mohamedabdallah-14/unslop/unslop-commit

Unslop Commit

Purpose

Generate or rewrite commit messages so they read like a real engineer wrote them at the end of a real day. Conventional Commits format. Direct, specific, no template English. Why over what.

Trigger

/unslop-commit, /commit, "write a commit", "commit message", "humanize this commit", "de-slop this commit". Auto-trigger when the user has staged changes and asks for a commit message.

Rules

Subject line

• Format: <type>(<scope>): <imperative summary>
• Scope optional. Types: feat, fix, chore, refactor, docs, test, perf, build, ci, revert.
• Imperative mood: add, fix, move, remove — not added, fixes, fixing.
• ≤50 chars when possible. Hard cap 72.
• No trailing period.
• Lowercase after : unless the project capitalizes.

Body (only when subject can't carry it)

• Add for: non-obvious "why", breaking changes, migrations, security context, data integrity.
• Wrap at 72 chars. Bullets - for two or more independent points. Single paragraph for one thought.
• End with refs: Closes #42, Refs #17. No BREAKING CHANGE: unless truly breaking — and then write it.

Never include

• Template prefixes: "This commit...", "This change...", "We are...", "I have..."
• Marketing verbs: comprehensive, robust, enhance, leverage, seamless, holistic
• Filler adverbs: just, really, basically, simply, actually
• Restating the filename when scope already names it
• "As requested by..." (use Co-authored-by: if you need attribution)
• AI attribution unless the project requires it
• Emoji unless project convention says so

Auto-clarity (always include body)

• Breaking changes
• Security fixes
• Data migrations
• Reverts (cite the reverted commit)

Examples

Bad → good (slop subject, no body)

• Bad: feat: implement a comprehensive, robust solution for user profile retrieval with enhanced error handling
• Good: feat(api): return profile fields the mobile client actually needs

Bad → good (vague body)

Bad:

fix: fixed the bug
 
This commit addresses an issue where the application was not working correctly
in some edge cases. We've improved the logic to handle these scenarios.

Good:

fix(checkout): ignore stale cart id from localStorage
 
Stale cart ids came from tabs that hadn't refreshed after a deploy. Server
now treats unknown ids as empty cart instead of 500.
 
Closes #842

Breaking change

feat(api)!: rename /v1/orders to /v1/customer-orders
 
The old route stays in place until the next major release but logs a
deprecation warning. Internal services have been migrated.
 
BREAKING CHANGE: third-party integrations using /v1/orders directly need
to switch to /v1/customer-orders by 2026-07-01.
 
Closes #1290

Boundaries

• Output the message only, in a single fenced block, ready to paste.
• Do not run git commit, stage, or amend.
• If the change is genuinely trivial (docs(readme): fix typo), keep it trivial. Don't pad.
• Never invent context the user didn't provide. If the "why" isn't clear, ask, or omit the body.