From 22503ce1e6fa8ffea5b0e507b1aee9f2172d1ff6 Mon Sep 17 00:00:00 2001 From: marcusquinn <6428977+marcusquinn@users.noreply.github.com> Date: Sat, 24 Jan 2026 21:02:14 +0000 Subject: [PATCH 1/2] chore: strengthen pre-edit-check self-verification wording Replace passive 'confirm you've run this check' with active imperative that makes clear: the script handles exception logic (e.g. planning files on main), not the agent's judgement. Prevents agents from skipping the mandatory check by reasoning 'it's just a TODO edit'. --- .agent/AGENTS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.agent/AGENTS.md b/.agent/AGENTS.md index 93223affc..170c30d6f 100644 --- a/.agent/AGENTS.md +++ b/.agent/AGENTS.md @@ -23,7 +23,7 @@ Exit 0 = proceed. Exit 1 = STOP (on main). Exit 2 = create worktree. Exit 3 = wa **Full details**: Read `workflows/pre-edit.md` for interactive prompts, worktree creation, and edge cases. -**Self-verification**: Before ANY file operation, confirm you've run this check in this session. +**Self-verification**: Your FIRST tool call before any Edit/Write MUST be this script. If you are about to edit a file and have not yet run pre-edit-check.sh in this session, STOP and run it now. No exceptions — including TODO.md and planning files (the script handles exception logic, not you). --- From 745589b9cd751f78deb210c44082e44c91d07fbc Mon Sep 17 00:00:00 2001 From: marcusquinn <6428977+marcusquinn@users.noreply.github.com> Date: Sat, 24 Jan 2026 21:07:25 +0000 Subject: [PATCH 2/2] chore: address CodeRabbit review - use 'step' instead of 'tool call' Avoids ambiguity for literal-minded agents that need to perform read-only operations (e.g. Read file) before the edit check. 'First step' clarifies this is the first part of the editing process, not the first action of the entire session. --- .agent/AGENTS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.agent/AGENTS.md b/.agent/AGENTS.md index 170c30d6f..7ad86412d 100644 --- a/.agent/AGENTS.md +++ b/.agent/AGENTS.md @@ -23,7 +23,7 @@ Exit 0 = proceed. Exit 1 = STOP (on main). Exit 2 = create worktree. Exit 3 = wa **Full details**: Read `workflows/pre-edit.md` for interactive prompts, worktree creation, and edge cases. -**Self-verification**: Your FIRST tool call before any Edit/Write MUST be this script. If you are about to edit a file and have not yet run pre-edit-check.sh in this session, STOP and run it now. No exceptions — including TODO.md and planning files (the script handles exception logic, not you). +**Self-verification**: Your FIRST step before any Edit/Write MUST be to run this script. If you are about to edit a file and have not yet run pre-edit-check.sh in this session, STOP and run it now. No exceptions — including TODO.md and planning files (the script handles exception logic, not you). ---