feat(execute): per-task worker names, per-step status table, deeper lint areas

[kvnwolf]

Summary

Three learnings from a field /dobby:execute session, mined with /dobby:learn:

• Per-task worker names. build-protocol.md prescribed name: "dobby:<role>", which the Agent tool rejects (: fails its name regex) — every first dispatch failed. Workers are now addressed as test-author-t<id> / implementor-t<id> / qa-t<id> (subagent_type stays dobby:<role>), each dispatch opens with its task's roster, and a task's workers are dispatched once and reached by the same name across fix rounds until the task is terminal (never replaced mid-task). Synced in execute/SKILL.md, root CONTEXT.md, CLAUDE.md, README.md, and the protocol contract test.
• Per-step status table. After every worker verdict or fix-round message the Architect prints one row per task, one emoji per step (⚪ not started · 🔄 in progress · ❌ failed, in fix · ✅ passed · — n/a). New protocol section + Step 3 rule; the closing summary table is unchanged.
• spec lint accepts $param dirs and deeper created paths. AREA_PATH_SHAPE now allows $ (file-router dynamic segments), and a nonexistent area passes when any ancestor below the repo root exists — not only its direct parent. This removes the whole-directory workaround that serialised every task in the field. task-decomposition.md now says: name files, not directories, from the first draft.

Test plan

• bunx dobby check --fix --baseline exit 0 (Biome, tsc, knip, vitest, kit checks)
• cli/src/artifact-lint.test.ts: new cases — existing $param dir passes, path under a $param dir passes, path two levels below an existing dir passes, path with no existing ancestor still fails, parenthetical fragment still fails
• cli/src/build-protocol.test.ts: address regexes updated to <role>-t<id>; namespaced dobby:<role> still asserted present as subagent_type
• Next live /dobby:execute run: first dispatch succeeds with name: "implementor-t1", status table prints after each step, fix rounds reach the same names

https://claude.ai/code/session_011KMhSVrdJ8baqG7oGrhzVQ

[greptile-apps]

Greptile Summary

This PR makes execute workers uniquely addressable per task, adds a live per-step status table, and broadens spec-lint support for dynamic and deeply created paths. Changes since the previous review complete the corrected-test-contract route so QA resumes only after the implementor has passed its Exit gate.

• Restricts task IDs to safe worker-address fragments.
• Keeps each task’s named workers addressable throughout fix rounds.
• Defines status transitions for code and test-contract repairs.
• Routes corrected test contracts through the implementor before QA re-checks.
• Allows $param path segments and paths anchored by deeper existing ancestors.

Confidence Score: 5/5

The PR appears safe to merge; the previously identified Exit-gate bypass is closed and no new actionable failure remains.

The corrected-contract path now consistently moves from the test-author to the implementor, requires a green Exit gate, and only then resumes QA under the same round number. All previous threads were resolved, and the withdrawn deep-path concern reflects an explicitly accepted design trade-off rather than an outstanding defect.

Important Files Changed

Filename	Overview
cli/src/artifact-lint.ts	Adds address-safe task-ID validation and accepts affected paths anchored by any existing in-repository ancestor.
cli/src/artifact-lint.test.ts	Covers invalid worker-address IDs, $param routes, deeper planned paths, and paths without an in-repository anchor.
plugin/skills/execute/references/build-protocol.md	Defines per-task worker addresses, worker reuse, live status transitions, and complete fix-round routing.
plugin/skills/execute/SKILL.md	Requires per-task names, stable worker identities, and status-table output after each relevant event.
plugin/agents/test-author.md	Directs corrected test contracts to the implementor rather than directly to QA.
plugin/agents/implementor.md	Requires corrected contracts to pass the Exit gate before the implementor resumes QA.
plugin/agents/qa.md	Makes the implementor’s post-gate message the sole trigger for QA re-checks.
cli/src/build-protocol.test.ts	Adds a contract assertion that corrected tests pass through the implementor’s gate before QA resumes.

_{Reviews (3): Last reviewed commit: "fix(execute): corrected test contracts r..." | Re-trigger Greptile}

[kvnwolf]

@greptileai review

[kvnwolf]

@greptileai review