feat: add chezmoi tracking to /start INIT MODE#94
Open
anombyte93 wants to merge 2 commits intomainfrom
Open
Conversation
Add Step 3 (Chezmoi Tracking) to the /start skill's INIT MODE flow. Ensures project artifacts deployed to ~/.claude/ (skills, session-init.py, MCP server code) are tracked by chezmoi to prevent drift/loss across machine rebuilds. Key changes: - Insert Step 3 (Chezmoi Tracking) between Brainstorm Weight + File Org and Silent Bootstrap - Scan for deployed skill files, Python backend, and MCP server code - Use correct chezmoi API: `chezmoi managed | grep` for tracking check - Detect drift via non-empty `chezmoi status` output - Commit additions with actual project directory name - Non-blocking: surfaces drift but never blocks init Also renumbered old Steps 3-4 to 4-5 and updated all internal cross-references. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Owner
Author
Code Review Complete ✅Ran comprehensive PR review via agent. Results: APPROVE SummaryThis is a clean, minimal implementation that correctly adds Chezmoi Tracking as Step 3 without scope creep (unlike previous PR #93). Strengths
Verified Correct
Edge Cases Handled
Test Plan CoverageYour test plan covers all critical paths. Ready to merge when tests pass. No changes requested. Awaiting reviewer approval. Review agent output: #94 |
Split single 'Status' field into two distinct fields: - Session State: Active | Paused | Closing (lifecycle state) - Focus Status: In Progress | Blocked | Done | Moving To Next (task state) This prevents orchestration confusion where 'Status: Complete' could mean either 'current task is done' or 'entire session is complete'. Changes: - Updated CLAUDE-activeContext.md template with two fields - Updated read_context() to parse new fields and return session_state/focus_status - Backward compatible with legacy 'Status:' field - Better status_hint logic based on parsed state
Contributor
Claude Review: 1 must-fix finding(s)Blocking issues created: PR blocked until all review-blocking issues resolved. |
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
2 participants
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
/startskill's INIT MODE~/.claude/(skills,session-init.py, MCP server code) are tracked by chezmoiWhat changed
File modified:
skills/start/SKILL.mdNew Step 3: Chezmoi Tracking
skills/directorychezmoi managed --include files --path-style absolute | grep -qF) to detect untracked files; non-emptychezmoi status= drift$(basename "$project_dir")Renumbering
Verification
chezmoi-drift-cleanupskill exists (confirmed in skills list)Test plan
/starton a project with untracked skill files → verifychezmoi addis called/startagain on same project → verify no duplicate adds, drift detected if present/starton a project without chezmoi → verify step is skipped silentlychezmoi git logshows commit with correct project nameMstatus🤖 Generated with Claude Code