docs: refresh CLAUDE.md + SPEC §T after fix-wave#116
Merged
Conversation
CLAUDE.md and SPEC.md drifted again after the post-merge fix-wave (same pattern as #54, new instances). All changes are doc-only -- no code touched. CLAUDE.md: - Architecture: internal/ui status updated from "(planned, not yet implemented)" to "scaffold (mode.go + theme.go + writer.go) merged via #74 PR1; spinners/huh/full migration tracked by #105". Adds report.go + sentinel.go rows in the internal/packages/ section to match the current code surface (Restore/RestoreFromSnapshot now return (RestoreOutcome, error); MigrationReport writes to <DataDir>/reports/). - Dependencies: lipgloss moved from "planned but not yet in go.mod" to "in go.mod (for internal/ui)"; huh and bubbletea stay planned under #105. - Manager detection: ResolveInteractive line updated from "not yet implemented" to "planned under #105 as part of the huh migration". - Architecture upgrade.go: "(Phase 4, PR #20 open)" -> "(Phase 4, merged)". - Phase table: Phase 7 -> Done (cites #17, #18, both closed); Phase 8 -> Done (cites #35, nodeupx@1.0.1 live on npm). SPEC.md: - Section C / V2: reflect partial internal/ui state instead of "not yet implemented". - Section I file layout: adds <DataDir>/reports/migration-<ts>.json; removes "(? stub only)" / "(? not wired)" annotations on `nodeup list` and nodeup.lock (both shipped). - Section V21: removes the "(? built but not wired -- see T3)" parenthetical (T3 fixed via #79). - Section T: marks T1-T17, T19, T20 as `x` with the PR # that closed each; keeps T18 against #105 (open); adds T21 for #106 itself. Header note bumped to reflect the 2026-07-05 refresh. Closes #106. Co-Authored-By: puku-ai-2.8 <noreply@puku.sh>
|
✔️ e00eaf6 - Conventional commits check succeeded. |
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.
Summary
CLAUDE.mdandSPEC.mddrifted again after the post-merge fix-wave — same pattern as #54, new instances. Doc-only PR.CLAUDE.md
internal/uistatus updated from(planned, not yet implemented)toscaffold (mode.go + theme.go + writer.go) merged via #74 PR1; spinners/huh/full migration tracked by #105. Addsreport.go+sentinel.gorows in theinternal/packages/section to match the current code surface.lipglossmoved from "planned but not yet ingo.mod" to "ingo.mod(forinternal/ui)";huhandbubbleteastay planned under feat(ui): complete internal/ui rollout — spinners, huh prompts, migrate remaining call sites (#74 phases 2–4) #105.ResolveInteractiveline updated from "not yet implemented" to "planned under feat(ui): complete internal/ui rollout — spinners, huh prompts, migrate remaining call sites (#74 phases 2–4) #105 as part of thehuhmigration".(Phase 4, PR #20 open)→(Phase 4, merged).nodeupx@1.0.1live on npm).SPEC.md
<DataDir>/reports/migration-<ts>-<tag>.json; removes(stub only)/(not wired)annotations onnodeup listandnodeup.lock(both shipped).(built but not wired — see T3)parenthetical (T3 fixed via fix(cli): wire platform.AcquireLock into upgrade + config set/init #79).xwith the PR # that closed each; keeps T18 against feat(ui): complete internal/ui rollout — spinners, huh prompts, migrate remaining call sites (#74 phases 2–4) #105 (open); adds T21 for docs: CLAUDE.md drift — internal/ui status, lipgloss in go.mod, phase table points at closed #17/#18 #106 itself. Header note bumped to reflect the 2026-07-05 refresh.Closes #106.