diff --git a/.claude/skills/feature/SKILL.md b/.claude/skills/feature/SKILL.md index d996f0773..f69e6898d 100644 --- a/.claude/skills/feature/SKILL.md +++ b/.claude/skills/feature/SKILL.md @@ -88,6 +88,10 @@ If an action affects another user: - [ ] Actions affecting other users trigger email (if configured) - [ ] Templates created for each email type +### 8b. Elegance check + +For non-trivial changes: pause and ask yourself "is there a simpler or more elegant approach?" If the current implementation feels hacky, refactor before proceeding. + ### 9. Run `/verify` ### 10. Run `/pull-request` diff --git a/AGENTS.md b/AGENTS.md index 90946f3aa..e6d3d2284 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -22,3 +22,8 @@ Use `.github/prompts/*` as task playbooks: | Naming | `.github/prompts/naming.prompt.md` | | PR | `.github/prompts/pull-request.prompt.md` | +## Workflow rules + +- After any user correction, evaluate if the mistake pattern belongs in `ERRORS.md` +- Investigate and fix bugs autonomously — don't ask for hand-holding +- For non-trivial tasks, plan before coding diff --git a/CLAUDE.md b/CLAUDE.md index a9967ce8a..07743a821 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -65,6 +65,7 @@ Node / Express / Mongoose / JWT stack from Devkit. Standalone backend or fullsta - Keep changes minimal and merge-friendly for downstream - Every function: JSDoc header (`@param`, `@returns`) - PRs: always use `/pull-request` — never open manually +- After user correction, evaluate if the pattern belongs in `ERRORS.md` ## Skills