From 125dc83f578972dad5ede7f96cba63e3845629db Mon Sep 17 00:00:00 2001 From: toki Date: Sat, 1 Aug 2026 05:32:11 +0900 Subject: [PATCH] =?UTF-8?q?fix(agent-ops):=20plan=20=EA=B2=BD=EB=A1=9C=20?= =?UTF-8?q?=ED=98=95=EC=8B=9D=20=EA=B2=80=EC=A6=9D=EC=9D=84=20=EB=AA=85?= =?UTF-8?q?=EC=8B=9C=ED=95=9C=EB=8B=A4?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- agent-ops/skills/common/plan/SKILL.md | 1 + .../orchestrate-agent-task-loop/tests/test_dispatch.py | 5 +++++ 2 files changed, 6 insertions(+) diff --git a/agent-ops/skills/common/plan/SKILL.md b/agent-ops/skills/common/plan/SKILL.md index edd3baa..e03060e 100644 --- a/agent-ops/skills/common/plan/SKILL.md +++ b/agent-ops/skills/common/plan/SKILL.md @@ -266,6 +266,7 @@ Required sections: - One item per change: `### [TAG-1] Title`, `TAG-2`, etc. - `Modified Files Summary`: table mapping files to item ids. This section is the dispatcher's workspace write-claim source of truth. - Include exactly one `## Modified Files Summary` section and at least one exact workspace file path. + - Wrap every claimed file path in backticks. A bare path cell is invalid. - Use repository-relative or canonical absolute file paths. Never use a glob (`*`, `?`, `[]`), directory path, workspace root, URL, path outside the workspace, malformed path, or prose placeholder as a claim. - Enumerate only implementer- or reviewer-owned workspace files, including the active review evidence file and deterministic workspace evidence artifacts. - For generated verification artifacts, choose deterministic exact workspace filenames or write them under a task-specific temporary directory outside the repository. Never substitute a directory or glob claim for dynamic filenames. diff --git a/agent-ops/skills/project/orchestrate-agent-task-loop/tests/test_dispatch.py b/agent-ops/skills/project/orchestrate-agent-task-loop/tests/test_dispatch.py index cff8b91..77df76d 100644 --- a/agent-ops/skills/project/orchestrate-agent-task-loop/tests/test_dispatch.py +++ b/agent-ops/skills/project/orchestrate-agent-task-loop/tests/test_dispatch.py @@ -11759,6 +11759,11 @@ class ArtifactLanguageContractTest(unittest.TestCase): with self.subTest(document=name, alias_pair=pair): self.assertIn(pair, documents[name]) + def test_plan_skill_requires_backticks_for_claimed_paths(self): + plan_skill = self.contract_documents()["plan_skill"] + + self.assertIn("Wrap every claimed file path in backticks", plan_skill) + def test_plan_and_review_share_dispatch_write_set_contract(self): documents = self.contract_documents() plan_skill = documents["plan_skill"]