Merge pull request #32 from laurentenhoor/feature/24-remove-closes-keyword
fix: remove GitHub auto-closing keywords from worker templates (#24)
This commit is contained in:
@@ -8,6 +8,7 @@ export const DEFAULT_DEV_INSTRUCTIONS = `# DEV Worker Instructions
|
|||||||
- Work in a git worktree (never switch branches in the main repo)
|
- Work in a git worktree (never switch branches in the main repo)
|
||||||
- Run tests before completing
|
- Run tests before completing
|
||||||
- Create an MR/PR to the base branch and merge it
|
- 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
|
- Clean up the worktree after merging
|
||||||
- When done, call task_complete with role "dev", result "done", and a brief summary
|
- 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
|
- 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/<id>-<slug>\` or \`fix/<id>-<slug>\`
|
- Branch naming: \`feature/<id>-<slug>\` or \`fix/<id>-<slug>\`
|
||||||
- **DEV always works in a git worktree** (never switch branches in the main repo)
|
- **DEV always works in a git worktree** (never switch branches in the main repo)
|
||||||
- **DEV must merge to base branch** before announcing completion
|
- **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
|
- **QA tests on the deployed version** and inspects code on the base branch
|
||||||
- Always run tests before completing
|
- Always run tests before completing
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user