Update the layers that enforce the commit rule, not just the doc

Flipping CONTRIBUTING.md alone would have left the repo fighting itself.
The style skill — loaded by builder and other agents — banned subject
prefixes outright and listed `feat: add retry logic` as a bad example, so
agents would keep writing subjects the new rule declines. The review
skill would then flag the compliant ones as prefix violations.

Updated together: the style skill now specifies the type/scope form and
keeps the ticket-ID, status-tag and filename bans; the review skill flags
a missing or unrecognized type instead of flagging types themselves; the
implement skill and the builder director prompt stop telling workers the
title is a plain-English sentence; the PR template matches.

CONTRIBUTING.md also states the consequence directly, so the README can
point at it rather than making a claim the source of truth does not.

CL-7880

Author: TheGreatAxios

.github/PULL_REQUEST_TEMPLATE.md

@@ -18,6 +18,7 @@ Fixes #123
 Partial work: Related to CL-1234
 No tracker: delete this comment block and leave no magic-word line.
 
-Do not put CL-… or #N in the PR title. Commit subjects stay plain English
-with no ticket IDs — see CONTRIBUTING.md.
+Do not put CL-… or #N in the PR title. Commit subjects are Conventional
+Commits — <type>(<scope>): <description> — with no ticket IDs. See
+CONTRIBUTING.md.
 -->

CONTRIBUTING.md

@@ -135,6 +135,9 @@ hand, not for the person reviewing this PR today.
 
 ## Pull requests
 
+These rules bind humans and agents alike. **A pull request that does not follow
+them will be declined** rather than fixed in review.
+
 ### Scope (MUST)
 
 1. One concern per PR. See scope discipline in `AGENTS.md`.

plugins/corbits-skills/skills/implement/SKILL.md

@@ -113,7 +113,7 @@ Run `make` (or the project's equivalent full pipeline: format, lint, build, test
 
 Update `activeForm` to "Committing: {subject}".
 
-Create the commit. Follow the commit message conventions from the `style` skill. Include the test in the same unit of work as the implementation — same commit when committing — one logical unit — and update the docs when the commit changes documented behavior. Worker-chain branch/PR convention: branch name carries the issue id, the PR body ends with `Fixes CL-…` and carries no AI-attribution lines (CONTRIBUTING: title stays a plain-English sentence, body is Summary/Verification).
+Create the commit. Follow the commit message conventions from the `style` skill. Include the test in the same unit of work as the implementation — same commit when committing — one logical unit — and update the docs when the commit changes documented behavior. Worker-chain branch/PR convention: branch name carries the issue id, the PR body ends with `Fixes CL-…` and carries no AI-attribution lines (CONTRIBUTING: title is a Conventional Commits subject, body is Summary/Verification).
 
 ### Step 5: Critique Loop
 

plugins/corbits-skills/skills/review/SKILL.md

@@ -257,7 +257,7 @@ git log <base>..HEAD --format='%s'
 
 Scan for:
 
-- **Prefix violations.** Any subject starting with a `word:`, `[tag]`, or `(scope)` pattern. Includes Conventional Commits (`feat:`, `fix:`), component or scope prefixes (`Anthropic adapter:`, `mm:`, `[X86]`), ticket IDs (`INTR-79:`), and status tags (`WIP:`). Project convention is plain English sentences; any prefix is a violation regardless of how idiomatic it looks in other ecosystems.
+- **Subject-form violations.** Project convention is Conventional Commits: `<type>(<scope>): <description>` with type from `feat`, `fix`, `perf`, `refactor`, `test`, `docs`, `build`, `ci`, `chore`, `style`. Flag a subject with no type (`Anthropic adapter: handle 429s`, `Add retry logic`), an unrecognized type, a ticket ID (`INTR-79:`), or a status tag (`WIP:`, `[urgent]`). Do **not** flag `feat:`/`fix:`-style prefixes themselves — those are the convention.
 - **Filename or path references.** Tokens that look like file paths or extensions (`server.ts`, `INFERENCE.md`, `src/foo/bar.py`). The diff lists what changed; subjects describe the change, not the file.
 - **Trailing punctuation.** Subjects ending with `.`, `!`, or `?`.
 - **Vague subjects.** "Update code," "Fix bug," "Misc changes," "Address review."

plugins/corbits-skills/skills/style/SKILL.md

@@ -104,37 +104,54 @@ git log origin/main --format='%s' | head -20
 
 The existing commits document the project's actual subject convention — verb tense, level of detail, voice, capitalization. Match what is there.
 
-The project's log can override the no-prefix rule below, but only when the recent history is **predominantly** prefixed in a single consistent convention — i.e., the prefix is the obvious shape of the last ~20 commits, not a minority pattern visible in a few. Mixed signals fall through to the no-prefix rule; tie goes to no prefix.
+**Conventional Commits.** Summary lines follow
+[Conventional Commits 1.0.0](https://www.conventionalcommits.org/en/v1.0.0/):
 
-**No subject prefixes.** Summary lines are plain English sentences that start with a verb and describe the change directly. Do not prefix the subject with anything — no tag, no scope, no category, no ticket ID, no severity marker. This is a flat rule across every prefix convention, including:
+```text
+<type>(<scope>): <description>
+```
+
+- **Type** — one of `feat`, `fix`, `perf`, `refactor`, `test`, `docs`, `build`,
+  `ci`, `chore`, `style`
+- **Scope** — the component the change lives in. Omit only when the change
+  genuinely spans the whole project
+- **Description** — plain English, imperative, starts with a verb, describes
+  the change directly
+- **Breaking changes** — `!` after the type/scope, or a `BREAKING CHANGE:`
+  footer
+
+Everything after the colon still obeys the rules below: no abbreviations, no
+trailing punctuation, no filenames, self-contained.
+
+**Still banned as subject prefixes**, before or instead of the type:
 
-- Conventional Commits: `feat:`, `fix:`, `chore:`, `docs:`, `refactor:`, `test:`
-- Scope or component prefixes: `Anthropic adapter:`, `mm:`, `[X86]`, `drivers/net:`, `frontend:`
 - Ticket IDs: `INTR-79:`, `JIRA-1234:`, `#456:`
 - Status or severity tags: `WIP:`, `[urgent]`, `(security):`
-
-Several of these patterns are widespread in well-known projects (Linux kernel, LLVM, Conventional-Commits-adopting projects) and feel idiomatic from sheer exposure. They are still banned here. Familiarity is not a justification.
+- Bare component prefixes with no type: `Anthropic adapter:`, `mm:`, `[X86]`,
+  `drivers/net:`, `frontend:` — the component belongs in the scope, so
+  `mm: fix leak` becomes `fix(mm): fix leak`
 
 Summary lines also use no abbreviations and do not end with punctuation.
 
 **Good examples:**
 
 ```
-Add retry logic for failed network requests
-Fix race condition in transaction verification
-Document API response format
+feat(executor): add retry logic for failed network requests
+fix(inference): resolve race condition in transaction verification
+docs(api): document response format
+perf(glob): stop rescanning ignored directories
 ```
 
 **Bad examples:**
 
 ```
-feat: add retry logic                  (Conventional Commits prefix)
-Anthropic adapter: handle 429s         (component-scope prefix)
+Add retry logic                        (no type or scope)
+feat: add retry logic                  (no scope, and says nothing specific)
+Anthropic adapter: handle 429s         (bare component, no type)
 INTR-79: add retry logic               (ticket-ID prefix)
 [WIP] refactor the parser              (status tag)
-Update code                            (too vague)
-Fix bug in server.ts                   (filename in subject)
-Document INFERENCE.md updates          (filename in subject)
+chore: update code                     (too vague)
+fix: bug in server.ts                  (filename in subject)
 ```
 
 **Self-contained:**

src/agent/directors/builder/package.ts

@@ -81,7 +81,7 @@ For implementation work, run the repository-defined typecheck command and releva
 
 **Don't shortcut verify.** The value is in the discipline. Skipping the build gate "because this change is simple" defeats the purpose.
 
-**Keep units focused.** Deliver a working tree that satisfies the brief and report. Builder does NOT commit unless the brief's success_criteria explicitly ask for a commit — the parent / Skywalker usually owns commits. Prefer: working tree + report envelope. Worker-chain branch/PR convention for the parent's handoff: branch name carries the issue id, the PR body ends with \`Fixes CL-…\` and carries no AI-attribution lines (CONTRIBUTING: title stays a plain-English sentence, body is Summary/Verification only).
+**Keep units focused.** Deliver a working tree that satisfies the brief and report. Builder does NOT commit unless the brief's success_criteria explicitly ask for a commit — the parent / Skywalker usually owns commits. Prefer: working tree + report envelope. Worker-chain branch/PR convention for the parent's handoff: branch name carries the issue id, the PR body ends with \`Fixes CL-…\` and carries no AI-attribution lines (CONTRIBUTING: title is a Conventional Commits subject, body is Summary/Verification only).
 
 **Discovered extra work** belongs under Blockers / Findings for a future unit — finish the current brief first.