refactor(cmd): split multi_service_engine_versions.go by concern by cristim · Pull Request #2293 · LeanerCloud/cloud-commitments-cli · GitHub
Skip to content

refactor(cmd): split multi_service_engine_versions.go by concern - #2293

Merged
cristim merged 3 commits into
mainfrom
refactor/1834-split-engine-versions-v2
Oct 20, 2026
Merged

cristim merged 3 commits into
mainfrom
refactor/1834-split-engine-versions-v2

Conversation

@cristim

@cristim cristim commented Oct 20, 2026

Copy link
Copy Markdown
Member

Part of #1834.

Pure-move split of cmd/multi_service_engine_versions.go (604 lines, now 193) into _extended.go, _parse.go and _major.go, one commit each (346 / 172 / 266 add+del). No code changes; the invariant panic is untouched.

AST proof: /tmp/claude/revC-astdump (every top-level declaration with its doc comment, sorted) output of cmd/ is byte-identical before vs after at every commit.

Also: go vet, go test -count=1 ./cmd/..., golangci-lint run ./cmd/... and scripts/test-git-secrets-allowlist.sh (44 PASS) are green. No .gitallowed or .golangci.yml entries name this file, so no changes there.

🤖 Generated with Claude Code

cristim and others added 3 commits October 20, 2026 21:04
…ne_versions.go

Pure move of declarations, no code change (part of #1834).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ne_versions.go

Pure move of declarations, no code change (part of #1834).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e_engine_versions.go

Pure move of declarations, no code change (part of #1834).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

@cristim

cristim commented Oct 20, 2026

Copy link
Copy Markdown
Member Author

@cristim
cristim merged commit c07edc5 into main Oct 20, 2026
12 checks passed
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.

1 participant