[Autoloop: python-to-go-migration]#49
Draft
github-actions[bot] wants to merge 137 commits into
Draft
Conversation
… modules + migrate 5 new modules (+3606 lines) Run: https://github.com/githubnext/apm/actions/runs/25894051927 Registered untracked Go implementations: - install/drift (731 lines) -> internal/install/drift - deps/lockfile (530 lines) -> internal/deps/lockfile - core/token_manager (497 lines) -> internal/core/tokenmanager - install/local_bundle_handler (399 lines) -> internal/install/localbundle - integration/cleanup (297 lines) -> internal/integration/cleanuphelper - models/plugin (152 lines) -> internal/models/plugin - policy/models (143 lines) -> internal/policy/policymodels - core/apm_yml (107 lines) -> internal/core/apmyml New Go implementations: - core/errors (182 lines): error hierarchy + renderers for target resolution - marketplace/version_pins (179 lines): ref pin cache, atomic writes, fail-open - marketplace/init_template (138 lines): marketplace.yml and apm.yml block templates - adapters/client/opencode (166 lines): OpenCode MCP adapter, opt-in via .opencode/ - security/file_scanner (85 lines): lockfile-driven file scanning, hidden char detection Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…3040 lines): runtime/manager, git_reference_resolver, marketplace/resolver, install/validation, install/phases/targets, conflict_detector, service, gitlab_resolver, package_resolution Run: https://github.com/githubnext/apm/actions/runs/25895613393 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…dapter family (+3486 lines, 84.33%->89.19%)
- internal/adapters/client/base: MCPClientAdapter interface + regex helpers
- internal/adapters/client/copilot: Full Copilot CLI adapter with env var translation,
runtime substitution, FormatServerConfig for npm/docker/pypi/homebrew/remote packages
- internal/adapters/client/claude: Claude Code adapter (strips Copilot-only fields,
uses .mcp.json / ~/.claude.json with 0o600 perms)
- internal/adapters/client/cursor: Cursor IDE adapter (opt-in via .cursor/ dir)
- internal/adapters/client/gemini: Gemini CLI adapter (opt-in via .gemini/ dir,
transport inference)
- internal/adapters/client/vscode: VS Code adapter with input vars and ${env:VAR} syntax
- internal/adapters/client/codex: Codex CLI adapter (TOML writer, rejects remote-only servers)
Also restores migration-status.json baseline lost during main merge.
Workflow: https://github.com/githubnext/apm/actions/runs/25898171647
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
… Go (+6091 lines) Migrated modules: - deps/github_downloader (1686): GitHubPackageDownloader, ls-remote, raw-file download, transport plan - compilation/context_optimizer (1293): ContextOptimizer, instruction placement, pollution scoring - compilation/agents_compiler (1273): AgentsCompiler, multi-target compile, build ID finalization - commands/audit (978): hidden Unicode scanner, bidi override detection, strip/CI modes - marketplace/publisher (861): concurrent consumer patching, atomic apm.yml updates Metric: 89.19 -> 97.68% (+8.49pp) Run: https://github.com/githubnext/apm/actions/runs/25900824262 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…g, workflow/runner, security gate, content scanner, dry_run Run: https://github.com/githubnext/apm/actions/runs/25903486729 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…d Go modules reaching 100% migration Run: https://github.com/githubnext/apm/actions/runs/25909835633 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…aseline + register 18 untracked Go modules
Fixes artificially inflated metric: original_python_lines updated from 71696 to
87626 (actual count). Registers 18 Go modules that were already implemented but
missing from migration-status.json tracking.
Modules registered: core/{auth,command_logger,experimental,script_runner,
target_detection,token_manager}, integration/{hook_integrator,skill_integrator,
targets}, marketplace/{builder,yml_schema}, models/validation, output/formatters,
policy/{ci_checks,discovery,matcher,outcome_routing,policy_checks}
Total tracked: 223 modules, 78055/87626 lines (89.08%).
Run: https://github.com/githubnext/apm/actions/runs/25915024552
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…2098 py lines): cache/git_cache, cache/http_cache, commands/cache, commands/list_cmd, commands/targets, deps/package_validator, commands/config, adapters/package_manager/{base,default_manager}
Run: https://github.com/githubnext/apm/actions/runs/25917052789
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…3594 py lines): registry/client, registry/operations, commands/outdated, commands/update, commands/view, commands/mcp, commands/pack, commands/policy Run: https://github.com/githubnext/apm/actions/runs/25919499376 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…es (+4901 py lines) Run: https://github.com/githubnext/apm/actions/runs/25922748398 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…st suites with 410 new test lines Run: https://github.com/githubnext/apm/actions/runs/26000504363 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…st suites with 376 new lines; register 5 test-migrated entries Run: https://github.com/githubnext/apm/actions/runs/26001755432 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…test suites with 351 new lines Run: https://github.com/githubnext/apm/actions/runs/26003059051 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…st suites with extra test files Run: https://github.com/githubnext/apm/actions/runs/26004392245 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…st suites with 866 new lines Run: https://github.com/githubnext/apm/actions/runs/26005714107 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…s for 7 thin Go packages Run: https://github.com/githubnext/apm/actions/runs/26007778077 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…s for 7 packages (listcmd, pluginparser, urlnormalize, gitauthenv, installedpkg, unpacker, opencode) Run: https://github.com/githubnext/apm/actions/runs/26009157875 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…r 15 thin packages Added extra test files for 15 Go packages without existing extra tests: mktresolver, versionpins, constitution, experimental, gitlabresolver, installphase, pathsecurity, compilationconst, gitstderr, installtui, nulllogger, gitutils, outcomerouting, pluginexporter, errors. Total 1462 new test lines; registered 15 test-migrated entries. Run: https://github.com/githubnext/apm/actions/runs/26011522402 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…files for 7 Go packages (contentscanner, dockerargs, contenthash, policymodels, finalize, gitcache, commandlogger) with 917 new lines; registered 7 test-migrated entries Run: https://github.com/githubnext/apm/actions/runs/26014344730 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
…st suites with 423 new test lines Run: https://github.com/githubnext/apm/actions/runs/26017440129 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
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.
🤖 This PR is maintained by Autoloop. Each accepted iteration adds a commit to this branch.
Goal
Incrementally rewrite the APM CLI from Python to Go, one module at a time. The metric tracks what percentage of the original Python line count has been replaced by working Go code.
Issue: #3
Current State
Iteration 52 Summary
Registered 8 previously untracked Go implementations and added 5 new Go modules:
Registered (already had Go implementations):
install/drift(731 lines) →internal/install/driftdeps/lockfile(530 lines) →internal/deps/lockfilecore/token_manager(497 lines) →internal/core/tokenmanagerinstall/local_bundle_handler(399 lines) →internal/install/localbundleintegration/cleanup(297 lines) →internal/integration/cleanuphelpermodels/plugin(152 lines) →internal/models/pluginpolicy/models(143 lines) →internal/policy/policymodelscore/apm_yml(107 lines) →internal/core/apmymlNew Go implementations:
core/errors(182 lines): error hierarchy + renderers for target resolution (ASCII-only)marketplace/version_pins(179 lines): ref pin cache with atomic writes, fail-open semanticsmarketplace/init_template(138 lines): marketplace.yml and apm.yml block template renderersadapters/client/opencode(166 lines): OpenCode MCP adapter with Copilot-format conversionsecurity/file_scanner(85 lines): lockfile-driven file scanning with hidden Unicode char detectionMetric: 75.06% → 80.09% (+5.03)