[Crane: crane-migration-python-to-go-full-apm-cli-rewrite]#114
Merged
mrjf merged 2 commits intoJun 8, 2026
Conversation
…73: fix deps info parity (required PACKAGE arg, missing-arg exit 2) Changes: - cmd_deps.go: runDepsInfo now requires PACKAGE argument (exits 2 with 'Error: Missing argument PACKAGE' when missing, matching Python Click behavior) - cmd_deps.go: fix runDeps help routing so 'apm deps info --help' shows subcommand help (not parent) - cmd_deps.go: runDepsInfo shows package metadata from apm_modules/<pkg>/apm.yml - parity_harness_test.go: add TestParityHarnessDepInfoHelp, TestParityHarnessDepInfoMissingPackage, TestParityHarnessDepInfoNotInstalled - python_test_coverage.json: map 12 deps-info Python tests to new Go contract tests Run: https://github.com/githubnext/apm/actions/runs/27120376316 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
Migration Benchmark Results
Migration CLI BenchmarkIncludes fixture-backed commands that must read, write, execute, or fail against real project state. The installed-project fixture contains apm.yml, apm.lock.yaml, apm_modules packages, local .apm primitives, target directories, deployed prompt files, and sample source files. Max allowed Go/Python median ratio:
Workloads
|
Contributor
Author
|
[bot] Iteration 74 accepted -- Crane run
[+] Migration complete. All 13 deletion-grade gates are green. The APM CLI rewrite from Python to Go is done. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[bot] This PR is maintained by Crane. Each accepted iteration adds a commit to this branch.
Migration: Python to Go -- Full APM CLI Rewrite
Source: Python 3.10+ (Click, Rich, PyInstaller)
Target: Go (native binary)
Strategy: greenfield
Migration Issue: #78
Current Status
Iteration 73 -- Stale completion reset; resuming with new
deps infoparity contract.Best Score: 1.0 (previous run, pending re-verification)
Iteration 73 Changes
cmd/apm/cmd_deps.go:runDepsInfonow requires PACKAGE argument. Without it, exits 2 with "Error: Missing argument 'PACKAGE'." matching Python Click behavior.cmd/apm/cmd_deps.go: Fix help routing soapm deps info --helpshows subcommand help (not parent deps help).cmd/apm/cmd_deps.go:runDepsInforeads package metadata fromapm_modules/<pkg>/apm.yml.cmd/apm/parity_harness_test.go: New tests --TestParityHarnessDepInfoHelp,TestParityHarnessDepInfoMissingPackage,TestParityHarnessDepInfoNotInstalled.cmd/apm/testdata/go_cutover/python_test_coverage.json: 12 deps-info Python tests mapped to new Go contract tests.