Skip to content

v2.0.0: rename /tdd to /implement, simplify Phase 6, deprecate side r…#1

Merged
one-kash merged 1 commit into
mainfrom
rename-tdd-to-implement
May 27, 2026
Merged

v2.0.0: rename /tdd to /implement, simplify Phase 6, deprecate side r…#1
one-kash merged 1 commit into
mainfrom
rename-tdd-to-implement

Conversation

@one-kash

Copy link
Copy Markdown
Member

…epos

Breaking changes:

  • /tdd renamed to /implement; skill dir skills/tdd -> skills/implement; speckit ID speckit.dev-workflow.implement (alias speckit.implement)
  • Tracker filename: impl-tracker-.json (was tdd-tracker-*.json)
  • Phase 6 parallel reviewer is /code-review (read-only) instead of /simplify (which writes); review pair is now both read-only
  • This repo is the only distribution channel; ai-agent-spec-skill and ai-agent-tdd-skill are deprecated; sync.sh + sync-skills.yml removed

Process simplification:

  • Drop tracker resume field; pattern hints/pitfalls move to optional notes. Required fields (name, files_*, tdd, acceptance_criteria) already carry resume context
  • Merge Standards into review-impl Criterion 4 (Code Quality + Architectural Gaps) and Blindspots into Criterion 6 (Robustness + Blindspots); drop the inline self-check that duplicated 6/8 criteria
  • Prune 5 duplicate Lessons Learned (constructor cascading, TDD-fix- code, extract-UI-logic, dead-code-same-chunk, review-plan-before- coding) — each duplicated content from earlier phases. 19 -> 14
  • Phase 2.5 verdict block trimmed; tracker plan_review field carries the gate forward across session resets

Added:

  • UI state-holder vs rendering-only distinction in test-strategy table with detection signals (mutableStateOf, useState, LaunchedEffect, etc.)
  • Both reviewers verify the "Pure UI — no test" exemption isn't misapplied to chunks that introduce hoisted state or branching effects
  • Small-feature shortcut keeps both review gates (only chunking and Plan Mode collapse for 1-chunk features)

…epos

Breaking changes:
- /tdd renamed to /implement; skill dir skills/tdd -> skills/implement;
  speckit ID speckit.dev-workflow.implement (alias speckit.implement)
- Tracker filename: impl-tracker-<feature>.json (was tdd-tracker-*.json)
- Phase 6 parallel reviewer is /code-review (read-only) instead of
  /simplify (which writes); review pair is now both read-only
- This repo is the only distribution channel; ai-agent-spec-skill and
  ai-agent-tdd-skill are deprecated; sync.sh + sync-skills.yml removed

Process simplification:
- Drop tracker `resume` field; pattern hints/pitfalls move to optional
  `notes`. Required fields (name, files_*, tdd, acceptance_criteria)
  already carry resume context
- Merge Standards into review-impl Criterion 4 (Code Quality +
  Architectural Gaps) and Blindspots into Criterion 6 (Robustness +
  Blindspots); drop the inline self-check that duplicated 6/8 criteria
- Prune 5 duplicate Lessons Learned (constructor cascading, TDD-fix-
  code, extract-UI-logic, dead-code-same-chunk, review-plan-before-
  coding) — each duplicated content from earlier phases. 19 -> 14
- Phase 2.5 verdict block trimmed; tracker plan_review field carries
  the gate forward across session resets

Added:
- UI state-holder vs rendering-only distinction in test-strategy table
  with detection signals (mutableStateOf, useState, LaunchedEffect, etc.)
- Both reviewers verify the "Pure UI — no test" exemption isn't
  misapplied to chunks that introduce hoisted state or branching effects
- Small-feature shortcut keeps both review gates (only chunking and
  Plan Mode collapse for 1-chunk features)
@one-kash one-kash merged commit 98738a0 into main May 27, 2026
1 check passed
@one-kash one-kash deleted the rename-tdd-to-implement branch May 27, 2026 23:27
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