From 4663fa6fec5601708a1fbe03e23c338037402e69 Mon Sep 17 00:00:00 2001 From: Lauren ten Hoor Date: Tue, 10 Feb 2026 09:03:19 +0800 Subject: [PATCH] fix: remove GitHub auto-closing keywords from worker templates (#24) - Added explicit warning to DEFAULT_DEV_INSTRUCTIONS - Added warning to AGENTS_MD_TEMPLATE conventions section - Workers instructed to use 'As described in issue #X' instead of 'Closes #X' - Prevents GitHub/GitLab from auto-closing issues before QA validation --- lib/templates.ts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/templates.ts b/lib/templates.ts index cdfe59a..53b06fc 100644 --- a/lib/templates.ts +++ b/lib/templates.ts @@ -8,6 +8,7 @@ export const DEFAULT_DEV_INSTRUCTIONS = `# DEV Worker Instructions - Work in a git worktree (never switch branches in the main repo) - Run tests before completing - Create an MR/PR to the base branch and merge it +- **IMPORTANT:** Do NOT use closing keywords in PR/MR descriptions (no "Closes #X", "Fixes #X", "Resolves #X"). Instead use "As described in issue #X" or "Addresses issue #X". DevClaw manages issue state via task_complete - auto-closing bypasses QA validation. - Clean up the worktree after merging - When done, call task_complete with role "dev", result "done", and a brief summary - If you discover unrelated bugs, call task_create to file them @@ -41,6 +42,7 @@ Skip the orchestrator section. Follow your task message and role instructions (a - Branch naming: \`feature/-\` or \`fix/-\` - **DEV always works in a git worktree** (never switch branches in the main repo) - **DEV must merge to base branch** before announcing completion +- **Do NOT use closing keywords in PR/MR descriptions** (no "Closes #X", "Fixes #X", "Resolves #X"). Instead use "As described in issue #X" or "Addresses issue #X". DevClaw manages issue state via task_complete — auto-closing bypasses QA validation. - **QA tests on the deployed version** and inspects code on the base branch - Always run tests before completing