From 1436679e60c430592c5259be142f64b91063b6b7 Mon Sep 17 00:00:00 2001 From: Eduardo Marquez <55303379+DocksDocks@users.noreply.github.com> Date: Mon, 19 Oct 2026 20:43:06 -0300 Subject: [PATCH 1/2] docs(plan-lifecycle): add worktree prune and the review-fix red baseline Commits item 2 removed a temporary worktree without git worktree prune, unlike Landing and the repository teardown rule. Red baseline item 4 did not say what a review fix that changes a spec assertion records, and tdd-workflow asks to escalate a wrong test to the user. A yolo run now records the reason, runs the changed test red before the fix, and adds a review-round Red baseline line; that record replaces the escalation. Refs #81 --- plugins/plan-lifecycle/skills/productivity/yolo/SKILL.md | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/plugins/plan-lifecycle/skills/productivity/yolo/SKILL.md b/plugins/plan-lifecycle/skills/productivity/yolo/SKILL.md index 7027cf6d..3c468b7c 100644 --- a/plugins/plan-lifecycle/skills/productivity/yolo/SKILL.md +++ b/plugins/plan-lifecycle/skills/productivity/yolo/SKILL.md @@ -6,7 +6,7 @@ disable-model-invocation: true metadata: pattern: pipeline updated: "2026-10-19" - content_hash: "c88bf4293b208fa56f085e1be92738f8e87a55e4e23c64f2c401b11cef0e5942" + content_hash: "df849a65ed166954017addd24fe9ed277fc24db6e99e4cf601a98f24ebe450b8" --- # Yolo @@ -201,6 +201,11 @@ can write the spec tests. 4. Keep the staged spec tests unchanged while you write production code: `git diff -- ` prints nothing. When a spec test is wrong, record the reason in Verification Results, fix the test, and run the baseline again. + This record replaces the user escalation that `tdd-workflow` asks for. +5. A review fix that changes a spec assertion follows item 4. Run the changed + test against the code before the fix. It must fail on the changed assertion. + Add this line to Verification Results: + `Red baseline, step , review round : : failed, passed`. ## Commits @@ -209,7 +214,7 @@ can write the spec tests. check, and the lint for its files. 2. When you split finished work into commits, check each commit in a temporary worktree: `git worktree add --detach `, run the checks, then - `git worktree remove -- `. `` is + `git worktree remove -- ` and `git worktree prune`. `` is `${XDG_DATA_HOME:-$HOME/.local/share}/agent-worktrees//check-`. Do not put it under `/tmp` or inside `.git`. 3. Before code review round 1, leave no staged, unstaged, or untracked change of From 260302ab44c8d51de1334789939af32c15968de6 Mon Sep 17 00:00:00 2001 From: Eduardo Marquez <55303379+DocksDocks@users.noreply.github.com> Date: Mon, 19 Oct 2026 20:43:53 -0300 Subject: [PATCH 2/2] docs(plan-lifecycle): name empty Research in the contract advice list The helper prints "Research is not filled after drafting." for an empty Research section too, but the contract named only the placeholder. The #79 code review reported the mismatch as LOW. Refs #81 --- .../plan-lifecycle/skills/productivity/plan-manager/SKILL.md | 2 +- .../productivity/plan-manager/references/plan-contract.md | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/plugins/plan-lifecycle/skills/productivity/plan-manager/SKILL.md b/plugins/plan-lifecycle/skills/productivity/plan-manager/SKILL.md index 64857070..a7512e51 100644 --- a/plugins/plan-lifecycle/skills/productivity/plan-manager/SKILL.md +++ b/plugins/plan-lifecycle/skills/productivity/plan-manager/SKILL.md @@ -5,7 +5,7 @@ user-invocable: true metadata: pattern: tool-wrapper updated: "2026-10-19" - content_hash: "bb1ee37a5ff78641994eb70b3c785307086998dd40a97623a9700e47d24837ca" + content_hash: "be9d0c5997d69513c763f52251e48bb6c7fce9ff24d14a706d639ba1a3c6c3bc" --- # Plan Manager diff --git a/plugins/plan-lifecycle/skills/productivity/plan-manager/references/plan-contract.md b/plugins/plan-lifecycle/skills/productivity/plan-manager/references/plan-contract.md index 12b3bd3c..c7cc306d 100644 --- a/plugins/plan-lifecycle/skills/productivity/plan-manager/references/plan-contract.md +++ b/plugins/plan-lifecycle/skills/productivity/plan-manager/references/plan-contract.md @@ -232,7 +232,8 @@ rewrite never drops the row. Once work has started, `edit` also refuses while the current record holds such a row. A duplicate or unknown `##` heading blocks every body write; a retired `## Review` heading is dropped on the next write. An explicit `--mode` on `new` outranks any Mode line inside `--goal`; otherwise the last unfenced, unindented Mode line in Goal is the decision. Advice prints `advice: ` on stdout and exits 0. It covers absolute machine -paths, a Research section that holds only `_Not researched yet._` after drafting, +paths, a Research section that is empty or holds only `_Not researched yet._` +after drafting, defaulted Mode, unknown Effect or Status, an unfinished dependency when a step moves to in-flight or done, two steps that share a Files path with no Depends chain, an `edit` that changes Goal, Research, Steps, Acceptance, or Do not touch