feat: add fixmap verify and release v0.7.3#99
Merged
Conversation
plan answers where to start. verify answers whether the change that followed matches the plan, by comparing a saved report against a real git diff. Five checks, all comparisons between two things the user already has, so nothing is executed and no repository code runs: - edits in generated or retired locations, which the next build discards - files the change needed that the plan never ranked - a leading file left untouched - source moving with no test moving, pointing at the routed test - risk areas the change reached that the plan never flagged Only the first exits non-zero. That one is wrong regardless of what the task was; the rest are advisory, because a plan can be wrong and a change can still be right, and only someone reading both can say which. The output reports the gap and leaves that judgement alone. Also fixes test routes carrying a repository-wide related-files list. Every route received the same set, so a report claimed npm --prefix packages/core run test would exercise another package's tests. Related files are scoped to the route's package; a root script keeps everything. Co-Authored-By: Claude Opus 5 <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
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.
fixmap verify— the second half of the jobplananswers where should I start.verifyanswers did I finish safely, by comparing a saved plan against the diff that followed it.Five checks, every one a comparison between two things the user already has — a recorded report and a real git diff. Nothing is executed, nothing is installed, no model is called:
edit-in-generated-locationunmapped-changeleading-file-untouchedno-test-changednew-risk-areaOnly the generated-location finding exits non-zero. That one is wrong regardless of what the task was. Everything else is advisory on purpose: a plan can be wrong and a change can still be right, and only a human or agent reading both can say which. The output reports the gap and leaves the judgement alone.
Verified end-to-end against a fixture reproducing the careless case — editing build output instead of source — with all five findings firing and exit 1.
CLI-only for now. MCP and Action support follow rather than being half-done across three surfaces in one release.
Also fixed: test routes claimed commands they could not run
findRelatedTestsran once and its result was assigned to every route, so a report statednpm --prefix packages/core run testwould exercisepackages/action/test/runner.test.ts. On this repository all three routes carried an identical eight files across three packages. Related files are now scoped to the route's package; a repository-root script keeps everything. The checked-in workspace example shows the correction.Verification
218 tests across workspaces (127 core),
npm run cigreen, all three evaluation suites byte-identical, and 15 version checks pass at 0.7.3 — includingapps/web, which broke the 0.7.2 release before the publish workflow started validating it.🤖 Generated with Claude Code