From smorch-dev
Boris elegance pause enforcement. 3 questions before every non-trivial commit. Caps Engineering hat at 7 if skipped. Mandatory on PRs >50 lines or adding new deps.
How this skill is triggered — by the user, by Claude, or both
Slash command
/smorch-dev:elegance-pauseThe summary Claude sees in its skill listing — used to decide when to auto-load this skill
**Purpose:** Before committing any non-trivial code, pause and ask: "Knowing everything I know now, would I implement this the same way?" If no, refactor before committing.
Purpose: Before committing any non-trivial code, pause and ask: "Knowing everything I know now, would I implement this the same way?" If no, refactor before committing.
Pillar: Boris #5 — Demand Elegance Time cost: 60-120 seconds per PR. Saves hours of tech debt.
MUST run before:
MAY skip for:
Examples of simpler alternatives to check:
If you find a simpler path → refactor.
## Elegance Pause
**Most complex file:** `src/...`
**Q1 — Same way knowing now?** {Yes / Yes with tweaks / No → refactored}
**Q2 — Simpler approach dismissed?** Considered {X}. Went with {Y} because {reason}.
**Q3 — Next dev will understand?** Yes, because {why}. OR: Added inline comment at line N.
**Tweaks applied after pause:**
- {tweak 1}
- {tweak 2}
**Tweaks noted for follow-up (not blocking this PR):**
- {tweak}
If a PR ships without an elegance pause block in the description:
Elegant code is:
Elegant is NOT:
Q1: No — I was doing the RLS check in the API route, but Supabase RLS policies already enforce it at the DB layer.
Action: Removed API-layer check. 40 lines → 0 lines. Engineering hat 9.
Q1: Yes, confidently. Considered debouncing vs loading-state-lock for the submit button race. Loading state is simpler + testable.
Q2: Debouncing adds a timer dep and is harder to test.
Q3: Yes — the useState + disabled={loading} pattern is standard React.
Engineering hat 10.
Shipped PDF export with @react-pdf/renderer (300KB) without pause.
Bundle regression caught next day. Engineering hat rescored 6.
Lesson L-002 appended: "bundle size comment mandatory for any new dep."
npx claudepluginhub smorchestra-ai/smorch-dev --plugin smorch-devGuides creation, editing, and verification of skills for AI coding agents using test-driven development with subagent scenarios. Use when authoring or debugging skills.