v2.0.0: rename /tdd to /implement, simplify Phase 6, deprecate side r…#1
Merged
Conversation
…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)
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.
…epos
Breaking changes:
Process simplification:
resumefield; pattern hints/pitfalls move to optionalnotes. Required fields (name, files_*, tdd, acceptance_criteria) already carry resume contextAdded: