Extract init into package, add skills, make CIRCLE_TOKEN optional#167
Draft
schurchleycci wants to merge 2 commits intomainfrom
Draft
Extract init into package, add skills, make CIRCLE_TOKEN optional#167schurchleycci wants to merge 2 commits intomainfrom
schurchleycci wants to merge 2 commits intomainfrom
Conversation
Move init orchestration from cmd/init.go into internal/initialize/ for cleaner separation. Add --skip-skills flag and skill install results to init output. Make CIRCLE_TOKEN optional by falling back to manual org ID prompt. Minor cleanups: pass workDir explicitly instead of using os.Getwd in leaf code, use slog for sentinel warnings, wrap validate errors with usererr. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Move CircleCI org setup and command detection into setupCircleCI and detectCommands helpers to bring Run below the gocyclo threshold. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
1 participant
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.
Summary
cmd/init.gointointernal/initialize/package for cleaner cmd/internal separation--skip-skillsflag and skill installation step tochunk init, with install/update results displayed in outputCIRCLE_TOKENoptional: when missing (or API fails), falls back to a manual org ID text prompt instead of skipping CircleCI setup entirelyworkDirexplicitly through sandbox/sync instead of callingos.Getwd()in leaf codeslogfor sentinel warnings, wraps validate errors withusererrTest plan
task buildandtask testpass (632 tests)chunk initwithoutCIRCLE_TOKEN— verify manual org ID prompt appearschunk initwithCIRCLE_TOKEN— verify org picker still workschunk init --skip-circleci— verify CircleCI step is fully skipped🤖 Generated with Claude Code