run start: lint declared targets against project roots and adapter id conventions - #88
Merged
Conversation
TDD-Run: 13 TDD-Cycle: 1 TDD-Phase: red
TDD-Run: 13 TDD-Cycle: 1 TDD-Phase: green
TDD-Run: 13 TDD-Cycle: 2 TDD-Phase: red
TDD-Run: 13 TDD-Cycle: 2 TDD-Phase: green
TDD-Run: 13 TDD-Cycle: 2 TDD-Phase: refactor
TDD-Run: 13 TDD-Cycle: 3 TDD-Phase: red
TDD-Run: 13 TDD-Cycle: 3 TDD-Phase: green
TDD-Run: 13 TDD-Cycle: 4 TDD-Phase: red
TDD-Run: 13 TDD-Cycle: 4 TDD-Phase: green
…prefix TDD-Run: 13 TDD-Cycle: 5 TDD-Phase: refactor
TDD-Run: 13 TDD-Cycle: 6 TDD-Phase: refactor
TDD-Run: 13 TDD-Cycle: 7 TDD-Phase: refactor
TDD-Run: 13 TDD-Cycle: 8 TDD-Phase: red
TDD-Run: 13 TDD-Cycle: 8 TDD-Phase: green
Merged
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
tdd plan registerandtdd run start(before the baseline claim) to catch two shapes of plan-author error before anything executes: (1) a target path that duplicates the project'srootprefix, and (2) a separator spelling that can never match the adapter's collected-id grammar::required), vitest (>required), gradle (/required), xctest (three/-separated parts required); exec has no grammar ruleproject.root != "."and the target's path starts withroot/, flag it with asuggestion— unless the actual nested directory exists in the worktree (the filesystem-existence exemption passes genuineroot/root/…paths)reason: "target_lint"and a structuredfindingslist;run startre-lints the stored contract against the current config to catch root/adapter drift since registrationCommits
Test coverage note
All decision-point operators in new code are covered by at least one test (pseudo-mutation analysis passed). 8 cycles declared and delivered, 0 skipped. 3 red-first violations handled via sensitivity checks (cycles 5, 6, 7 passed on arrival due to complete implementation in cycle 1).
Closes #71