feat: behavior coverage, pre-mortem levels, Status elimination, disk-evidence selection#146
Merged
Merged
Conversation
…evidence selection Knowledge: - gherkin: add Behavioral Distinctness concept (Wynne/Adzic) + common mistake - pre-mortem: add 4th level (Behavior Pre-Mortem) with per-rule-type prompts - test-design: add Invariant Property Tests concept, widen property test scope - feature-discovery: replace Status enum with content-based lifecycle indicators - wsjf: replace Status: BASELINED with @id-tagged Examples eligibility Skills: - write-bdd-features: insert step 5 for behavior coverage verification + pre-mortem - write-test: amend step 1 for structural rule property tests - select-feature: rewrite to detect delivery status from disk evidence (diff @id vs tests) - discover-features: remove Status: ELICITING from file creation - confirm-baseline: remove redundant Status check (step 1 already verifies @ids) Flow: - planning-flow: replace feature-status: ==BASELINED with baseline-confirmed: ==verified Template: - rename feature.feature.template → <feature_name>.feature.template - remove Status: ELICITING | BASELINED line from template AGENTS.md: update template path reference
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
Thenoutcome, not input value. Structural/invariant rules get one representative example; full coverage deferred to Hypothesis property tests.@bugonly to all invariant/structural rules. Adds Invariant Property Tests concept to test-design knowledge.Status: ELICITING | BASELINEDfrom feature files entirely. Status is derivable from content (no file = not discovered, description only = discovery, rules but no @ids = breakdown, @id-tagged examples = baselined). Replacesfeature-status: ==BASELINEDflow condition withbaseline-confirmed: ==verified.feature.feature.template→<feature_name>.feature.template(consistent with<type_id>placeholder convention in AGENTS.md).Files changed
Knowledge (5 files)
gherkin.md— Behavioral Distinctness concept + common mistake + key takeawaypre-mortem.md— 4th level Behavior Pre-Mortem with prompts for action/behavioural/structural rulestest-design.md— Invariant Property Tests concept, widened property test table scopefeature-discovery.md— content-based lifecycle instead of Status enumwsjf.md— @id-tagged Examples eligibility instead of Status: BASELINEDSkills (4 files)
write-bdd-features/SKILL.md— step 5: group by Then, discard duplicates, behavior pre-mortemwrite-test/SKILL.md— step 1: structural rules also get Hypothesis property testsselect-feature/SKILL.md— rewrite: detect delivery status from disk evidencediscover-features/SKILL.md— remove Status: ELICITING from file creationconfirm-baseline/SKILL.md— remove redundant Status checkFlow (1 file)
planning-flow.yaml—feature-status: ==BASELINED→baseline-confirmed: ==verifiedTemplate (1 file)
feature.feature.template→<feature_name>.feature.template, Status line removedAGENTS.md — template path example updated