Skip to content

feat: MFT/INV/DIR test coverage spec — classify-before-generate taxonomy#25

Merged
screenleon merged 4 commits intomainfrom
feat/test-coverage-spec-mft-inv-dir
Apr 29, 2026
Merged

feat: MFT/INV/DIR test coverage spec — classify-before-generate taxonomy#25
screenleon merged 4 commits intomainfrom
feat/test-coverage-spec-mft-inv-dir

Conversation

@screenleon
Copy link
Copy Markdown
Owner

Summary

  • New rules/global/test-coverage-spec.md — three global rules (GTCS-001/002/003) introducing the MFT/INV/DIR taxonomy: classify each test case into exactly one category before writing it; scale-based coverage floors (Small ≥1, Medium ≥2, Large ≥3); absent categories must be stated with a specific reason in the structured preamble or test-file header.
  • Updated skills/test-and-fix-loop/SKILL.md — added "Test category classification" section (table, workflow, INV-vs-DIR heuristic, scenario examples); four new conformance self-check items; rules field in YAML front matter referencing test-coverage-spec.md.
  • Updated rules/global/code-quality-baseline.md — peer-rule cross-reference to test-coverage-spec.md for agent discoverability.
  • Updated DECISIONS.md — decision entry with context, alternatives considered, and constraints introduced.
  • Updated CHANGELOG.md — Added and Changed entries.

No CI files changed. All changes are adapter-neutral markdown.

Review notes

Critic and risk-reviewer were both run before this PR. Issues addressed:

Finding Severity Fix
GTCS-003 Directive vs Conflict handling contradiction blocker Directive now authoritative: Small/Medium/Large floors stated once
"state the absence" had no destination blocker Specified: structured preamble or test-file header comment block
SKILL.md depends_on missing rule reference concern Added rules: field in YAML front matter
Waiver language too vague concern Added valid/invalid waiver examples in both GTCS-003 and SKILL.md workflow
DECISIONS.md missing GTCS entry concern Added entry with alternatives considered
code-quality-baseline.md missing cross-reference minor Added peer-rule note in header block

Test plan

  • Read rules/global/test-coverage-spec.md — verify GTCS-001/002/003 are internally consistent, no Directive/Conflict-handling contradictions
  • Read skills/test-and-fix-loop/SKILL.md — verify classification section appears before test-first guidance; workflow step 5 specifies absence destination; conformance self-check has 9 items in correct order
  • Read DECISIONS.md — verify GTCS entry is present and dated 2026-04-29
  • Read CHANGELOG.md — verify Added (test-coverage-spec.md) and Changed (3 files) entries are present
  • Confirm no .github/workflows/ or CI files were modified

🤖 Generated with Claude Code

screenleon and others added 4 commits April 29, 2026 09:18
Introduces structured test category rules (GTCS-001/002/003) so agents
classify each test case before writing it, making INV and DIR coverage
gaps explicit rather than silently absent.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Fixes structural issue where original Added items were orphaned under
Changed, merges duplicate Changed sections, and adds version link.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…opters

Template development decisions belong in DECISIONS_ARCHIVE.md, not in
DECISIONS.md which adopters inherit as a clean starter log.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…RCHIVE.md

DECISIONS.md now contains only the header and a commented entry template.
Template-internal decisions moved to DECISIONS_ARCHIVE.md which was missing.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@screenleon screenleon merged commit b569966 into main Apr 29, 2026
7 checks passed
@screenleon screenleon deleted the feat/test-coverage-spec-mft-inv-dir branch April 29, 2026 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant