From a6fe199ba7f0978a61d26c4e5abd4d72289ffbef Mon Sep 17 00:00:00 2001 From: Kyle Merritt Date: Fri, 13 Nov 2026 16:24:50 -0500 Subject: [PATCH] Extract Implementer clarification into handoff guidance --- README.md | 2 +- packages/implementer-handoff/SKILL.md | 4 ++-- packages/implementer-handoff/handoff.md | 6 ++++++ packages/implementer-handoff/relayview-skill.json | 6 +++--- tests/test_packages.py | 8 ++++++++ 5 files changed, 20 insertions(+), 6 deletions(-) diff --git a/README.md b/README.md index 1f93fa3..32caed0 100644 --- a/README.md +++ b/README.md @@ -8,7 +8,7 @@ Public source for RelayView-maintained instruction-only role skills and an illus | --- | --- | --- | --- | | `packages/{planner,implementer,reviewer}-investigation` | Maintained automatic guidance | One matching role / step each | 0.7.0 | | `packages/planner-scope` | Maintained default | Planner / planning | 0.8.1 | -| `packages/implementer-handoff` | Maintained default | Implementer / implementation | 0.2.0 | +| `packages/implementer-handoff` | Maintained default | Implementer / implementation | 0.9.0 | | `packages/issue-designer-default` | Maintained default | Designer / design | 0.6.1 | | `packages/designer-wayfinder`, `designer-grilling` | Optional manual guidance | Designer / design | 0.6.0 | | `packages/designer-prototype` | Optional manual guidance | Designer / design | 0.6.2 | diff --git a/packages/implementer-handoff/SKILL.md b/packages/implementer-handoff/SKILL.md index 5de068c..1a7146c 100644 --- a/packages/implementer-handoff/SKILL.md +++ b/packages/implementer-handoff/SKILL.md @@ -3,12 +3,12 @@ name: implementer-handoff description: Reconcile verified implementation documentation and prepare review content during implementation. metadata: relayview-display-name: Implementer handoff - relayview-version: 0.2.0 + relayview-version: 0.9.0 --- # Implementer handoff -During the assigned implementation step, keep the repository's current-behavior documentation aligned with the code and checks you actually completed. Read the project's documentation rules and the relevant existing contract before editing it. Use the [handoff guide](handoff.md) for the documentation disposition and review content. +During the assigned implementation step, keep the repository's current-behavior documentation aligned with the code and checks you actually completed. Read the project's documentation rules and the relevant existing contract before editing it. Use the [handoff guide](handoff.md) for implementation clarification, documentation disposition and review content. Prepare a concise, attributable review handoff when the assigned workflow gives you a place to submit one. Identify the exact candidate, focused checks and their observed results, any manual evidence and its observer, unresolved limitations, and the documentation disposition. A command attempted is not a check passed; report failures and stale evidence explicitly. diff --git a/packages/implementer-handoff/handoff.md b/packages/implementer-handoff/handoff.md index 8749c6d..2acc352 100644 --- a/packages/implementer-handoff/handoff.md +++ b/packages/implementer-handoff/handoff.md @@ -1,5 +1,11 @@ # Handoff guide +## Implementation and clarification + +Read the relevant files and make focused changes. Use the assigned investigation method for repository discovery; this handoff method owns clarification and report preparation. + +Ask the planner for more information when blocked after reasonable investigation. If the final contract appears wrong, explain the mismatch clearly, make the safest local correction if obvious within the authoritative scope, and validate it. Clarification methods guide communication; scope changes require owner authority. + ## Documentation - Update the current-behavior contract only for behavior verified on the candidate. Keep proposed work clearly proposed. diff --git a/packages/implementer-handoff/relayview-skill.json b/packages/implementer-handoff/relayview-skill.json index ddb3f36..c73eb06 100644 --- a/packages/implementer-handoff/relayview-skill.json +++ b/packages/implementer-handoff/relayview-skill.json @@ -2,18 +2,18 @@ "schemaVersion": 1, "source": { "kind": "first-party", - "reference": "v0.2.0" + "reference": "v0.9.0" }, "role": "implementer", "step": "implementation", "files": [ { "path": "SKILL.md", - "sha256": "bfeafa2d7780fe2c0bb4206ab9a03acd12804347ceb0f3ffb3e06e1be514c852" + "sha256": "a826d49f478018190539843022b83c11e7a034fa3597edcc85b8a57c65ac434d" }, { "path": "handoff.md", - "sha256": "303c3e792cf0e8da6215e6eeb69f5a2f1f2bd0b67db016160d9e83a970ad95d3" + "sha256": "17a32948aeb0e0359c92924219e6c328f3422773d5d878ffbcf0bf2f07a99aeb" } ] } diff --git a/tests/test_packages.py b/tests/test_packages.py index 3be04d6..466986d 100644 --- a/tests/test_packages.py +++ b/tests/test_packages.py @@ -12,6 +12,14 @@ ROOT = Path(__file__).resolve().parents[1] class PackageTests(unittest.TestCase): + def test_handoff_clarification_method(self): + package = ROOT / "packages/implementer-handoff" + check_package(package) + text = (package / "handoff.md").read_text() + self.assertIn("blocked after reasonable investigation", text) + self.assertIn("scope changes require owner authority", text) + self.assertNotIn("Trace load order", text) + def test_schema_two_claim_policy_bounds(self): with tempfile.TemporaryDirectory() as directory: package = Path(directory) / "planner-scope"