fix: split init file count between written and merged (#16)#28
Merged
Conversation
Init was reporting 'wrote 39 files' but the re-init collision guard reported '38 specflow-managed file(s)' — the discrepancy of 1 was .gitignore being treated as a mergeable entry (written but not counted as overwrite-style). Now the count surfaces both numbers in a single line that lines up with the collision guard: Before: ✓ wrote 39 files (re-init: target already contains 38 specflow-managed file(s)) After: ✓ wrote 38 files (+ merged: .gitignore) (re-init: target already contains 38 specflow-managed file(s)) ← matches Internally: InitResult now exposes filesWritten (non-mergeable count, matches collision logic) and filesMerged (paths of mergeable entries that were merged into pre-existing user content). The handler renders the suffix only when filesMerged is non-empty, so future scenarios with no mergeables stay clean. Drops #16 from .claude/agents/qa-tester/memory/tracked-findings.md — that file is now empty since all six original QA-finding tickets have shipped. Closes #16. Co-Authored-By: Claude Opus 4.7 (1M context) <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
Init was reporting
wrote 39 filesbut the re-init collision guard reported38 specflow-managed file(s). The discrepancy of 1 was.gitignorebeing treated as a mergeable entry (written but excluded from the collision count). Both numbers now line up.Before:
After:
InitResultnow exposesfilesWritten(non-mergeable count, matches collision logic) andfilesMerged(paths of mergeable entries merged into pre-existing user content). The handler renders the suffix only whenfilesMergedis non-empty.Drops
#16from.claude/agents/qa-tester/memory/tracked-findings.md— that file is now empty since all six original QA-finding tickets have shipped.Closes #16.
Test plan
deno task test— 318 tests pass.bash .claude/skills/test-specflow/scripts/run-init.sh smoke-16 claude→✓ wrote 38 files (+ merged: .gitignore)then re-init →target already contains 38 specflow-managed file(s)(matched).🤖 Generated with Claude Code