Skip to content

[Autoloop: python-to-go-migration]#17

Merged
mrjf merged 8 commits into
mainfrom
autoloop/python-to-go-migration
May 13, 2026
Merged

[Autoloop: python-to-go-migration]#17
mrjf merged 8 commits into
mainfrom
autoloop/python-to-go-migration

Conversation

@mrjf
Copy link
Copy Markdown
Contributor

@mrjf mrjf commented May 12, 2026

…and migrate 5 leaf modules

  • Set up Go module (github.com/githubnext/apm)
  • Migrate constants.py -> internal/constants (55 lines)
  • Migrate version.py -> internal/version (101 lines)
  • Migrate utils/short_sha.py -> internal/utils/sha (45 lines, with tests)
  • Migrate utils/paths.py -> internal/utils/paths (27 lines)
  • Migrate utils/normalization.py -> internal/utils/normalization (57 lines)
  • Create benchmarks/migration-status.json tracking 285/71696 lines (0.40%)
  • Add Go build targets to Makefile

Run: https://github.com/githubnext/apm/actions/runs/25717987972

Description

Brief description of changes and motivation.

Fixes # (issue)

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Maintenance / refactor

Testing

  • Tested locally
  • All existing tests pass
  • Added tests for new functionality (if applicable)

github-actions Bot and others added 2 commits May 12, 2026 06:47
…and migrate 5 leaf modules

- Set up Go module (github.com/githubnext/apm)
- Migrate constants.py -> internal/constants (55 lines)
- Migrate version.py -> internal/version (101 lines)
- Migrate utils/short_sha.py -> internal/utils/sha (45 lines, with tests)
- Migrate utils/paths.py -> internal/utils/paths (27 lines)
- Migrate utils/normalization.py -> internal/utils/normalization (57 lines)
- Create benchmarks/migration-status.json tracking 285/71696 lines (0.40%)
- Add Go build targets to Makefile

Run: https://github.com/githubnext/apm/actions/runs/25717987972

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ic_io, git_env to Go

Run: https://github.com/githubnext/apm/actions/runs/25736801433

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

Commit pushed: 4fea0e8

Generated by Autoloop

@github-actions github-actions Bot mentioned this pull request May 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Commit pushed: 99e6ae1

Generated by Autoloop

…v.py and helpers.py to Go

Run: https://github.com/githubnext/apm/actions/runs/25747630390

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

Commit pushed: 9c40a5f

Generated by Autoloop

@github-actions
Copy link
Copy Markdown
Contributor


Warning

The push_to_pull_request_branch operation failed: Cannot push to pull request branch: patch modifies protected files (README.md). Add them to the allowed-files configuration field or set protected-files: fallback-to-issue to create a review issue instead.. The code changes were not applied.

🤖 Iteration 8✅ Accepted

  • Change: Migrate content_hash.py, exclude.py, console.py, and file_ops.py to Go (4 new packages, 827 Python lines)
  • Metric: 2.69% (best: 2.69%, delta: +1.15)
  • Commit: 73205a9
  • Result: All 4 packages build cleanly with go build ./... and all tests pass with go test ./.... Branch synced with main (merge commit). The contenthash, exclude, console, and fileops packages are now available under internal/utils/.

Generated by Autoloop · ● 39.8M ·

…o Go (+3418 lines, 4245 total, 5.92%)

Migrated modules:
- utils/content_hash.py -> internal/utils/contenthash
- utils/exclude.py -> internal/utils/exclude (with tests)
- utils/path_security.py -> internal/utils/pathsecurity (with tests)
- utils/version_checker.py -> internal/utils/versionchecker
- utils/file_ops.py -> internal/utils/fileops
- utils/console.py -> internal/utils/console
- utils/diagnostics.py -> internal/utils/diagnostics
- utils/install_tui.py -> internal/utils/installtui
- utils/github_host.py -> internal/utils/githubhost (with tests)
- utils/reflink.py -> internal/utils/reflink (linux+other build tags)
- install/errors.py -> internal/install/errors
- install/cache_pin.py -> internal/install/cachepin
- install/context.py -> internal/install/installctx

Metric: 5.92% (previous best: 5.41%, delta: +0.51%)
Run: https://github.com/githubnext/apm/actions/runs/25771166584

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

Commit pushed: 110a1a4

Generated by Autoloop

@mrjf
Copy link
Copy Markdown
Contributor Author

mrjf commented May 13, 2026

@copilot resolve the merge conflicts in this pull request

Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented May 13, 2026

@copilot resolve the merge conflicts in this pull request

Resolved in 05b6217.

Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Copilot finished work on behalf of mrjf May 13, 2026 16:13
@mrjf mrjf merged commit bb0f9db into main May 13, 2026
8 checks passed
github-actions Bot added a commit that referenced this pull request May 13, 2026
- Mark PR #17 as merged (2026-05-13T16:30:28Z)
- Update migrated module count to 54 (from iter-24 issue comment)
- Refresh manifest benchmark table from local run of
  scripts/benchmark_manifest_ops.py (check_collision now 1606x at
  large scale, scoped uninstall 26x)
- Bump Last updated to 2026-05-13 16:32 UTC

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
mrjf added a commit that referenced this pull request May 13, 2026
…ate-iter24-merged-6f8a4775f94dff72

[autoloop-progress] docs: update Go migration progress page for iter-24 / PR #17 merged
@github-actions
Copy link
Copy Markdown
Contributor

Commit pushed: 3c79f93

Generated by Autoloop

github-actions Bot added a commit that referenced this pull request May 14, 2026
- Update latest status: iteration 35, 21.08% migrated (15,116/71,696 lines)
- Update PR reference from #17 to #39 (merged 2026-05-14)
- Expand migration progress table with iterations 28-35
- Update migrated modules table to all 89 modules from migration-status.json
- Remove stale branch-reset caveat (JSON is now current)
- Update Go build/test validation notes to reflect iters 32-35
- Update Next up section from memory/autoloop Future Directions
- Update Last updated timestamp

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants