feat(skills): add Systematic skill authoring guardrails (#325)

* feat(skills): add Systematic skill authoring guardrails

* fix(plugin): default-export smoke test and harden checkAgentModel

- Use the default export in the CI Node.js plugin smoke test so it works
  with the loader-safe single-export contract on dist/index.js.
- Pin the smoke test contract in tests/unit/plugin.test.ts.
- Guard parsed.data before reading model in checkAgentModel and add a
  non-object frontmatter regression test.
- Reuse src/lib/frontmatter.ts in docs/scripts/transform-content.ts to
  remove the duplicate parser.

Author: marcusrbrown

.github/workflows/main.yaml

@@ -47,7 +47,8 @@ jobs:
         run: |
           node --input-type=module -e "
             const m = await import('./dist/index.js');
-            const plugin = await m.SystematicPlugin({
+            const pluginFactory = m.default;
+            const plugin = await pluginFactory({
               client: { app: { log: async () => {} } },
               directory: process.cwd()
             });

AGENTS.md

@@ -46,7 +46,7 @@ systematic/
 │   ├── index.ts          # Plugin entry (SystematicPlugin)
 │   ├── cli.ts            # CLI entry (list/convert/config commands)
 │   └── lib/              # 12 core modules (see src/lib/AGENTS.md)
-├── skills/               # 45 bundled skills (SKILL.md format)
+├── skills/               # 46 bundled skills (SKILL.md format)
 ├── agents/               # 50 bundled agents (6 categories: design/docs/document-review/research/review/workflow)
 ├── commands/             # Empty (.gitkeep) — commands converted to skills; dir kept for backward compat
 ├── docs/                 # Starlight docs workspace (see docs/AGENTS.md)
@@ -164,4 +164,3 @@ The `commands/` directory contains only `.gitkeep` (all commands converted to sk
 - `commands/` dir retained (with `.gitkeep`) for backward compatibility — code paths still support commands
 - `registry/` provides OCX component-level installation with omo and standalone profiles
 - `.opencode/commands/` has project-only commands: `generate-readme` (README generation)
-

agents/review/api-contract-reviewer.md

@@ -1,6 +1,7 @@
 ---
 name: api-contract-reviewer
 description: Conditional code-review persona, selected when the diff touches API routes, request/response types, serialization, versioning, or exported type signatures. Reviews code for breaking contract changes.
+model: inherit
 tools: Read, Grep, Glob, Bash
 color: blue
 mode: subagent
@@ -46,4 +47,3 @@ Return your findings as JSON matching the findings schema. No prose outside the
   "testing_gaps": []
 }
 ```
-

agents/review/correctness-reviewer.md

@@ -1,6 +1,7 @@
 ---
 name: correctness-reviewer
 description: Always-on code-review persona. Reviews code for logic errors, edge cases, state management bugs, error propagation failures, and intent-vs-implementation mismatches.
+model: inherit
 tools: Read, Grep, Glob, Bash
 color: blue
 mode: subagent
@@ -46,4 +47,3 @@ Return your findings as JSON matching the findings schema. No prose outside the
   "testing_gaps": []
 }
 ```
-

agents/review/data-migrations-reviewer.md

@@ -1,6 +1,7 @@
 ---
 name: data-migrations-reviewer
 description: Conditional code-review persona, selected when the diff touches migration files, schema changes, data transformations, or backfill scripts. Reviews code for data integrity and migration safety.
+model: inherit
 tools: Read, Grep, Glob, Bash
 color: blue
 mode: subagent
@@ -50,4 +51,3 @@ Return your findings as JSON matching the findings schema. No prose outside the
   "testing_gaps": []
 }
 ```
-

agents/review/dhh-rails-reviewer.md

@@ -1,6 +1,7 @@
 ---
 name: dhh-rails-reviewer
 description: Conditional code-review persona, selected when Rails diffs introduce architectural choices, abstractions, or frontend patterns that may fight the framework. Reviews code from an opinionated DHH perspective.
+model: inherit
 tools: Read, Grep, Glob, Bash
 color: blue
 mode: subagent
@@ -44,4 +45,3 @@ Return your findings as JSON matching the findings schema. No prose outside the
   "testing_gaps": []
 }
 ```
-

agents/review/julik-frontend-races-reviewer.md

@@ -1,6 +1,7 @@
 ---
 name: julik-frontend-races-reviewer
 description: Conditional code-review persona, selected when the diff touches async UI code, Stimulus/Turbo lifecycles, or DOM-timing-sensitive frontend behavior. Reviews code for race conditions and janky UI failure modes.
+model: inherit
 tools: Read, Grep, Glob, Bash
 color: blue
 mode: subagent
@@ -47,4 +48,3 @@ Return your findings as JSON matching the findings schema. No prose outside the
 ```
 
 Discourage the user from pulling in too many dependencies, explaining that the job is to first understand the race conditions, and then pick a tool for removing them. That tool is usually just a dozen lines, if not less - no need to pull in half of NPM for that.
-

agents/review/kieran-python-reviewer.md

@@ -1,6 +1,7 @@
 ---
 name: kieran-python-reviewer
 description: Conditional code-review persona, selected when the diff touches Python code. Reviews changes with Kieran's strict bar for Pythonic clarity, type hints, and maintainability.
+model: inherit
 tools: Read, Grep, Glob, Bash
 color: blue
 mode: subagent
@@ -45,4 +46,3 @@ Return your findings as JSON matching the findings schema. No prose outside the
   "testing_gaps": []
 }
 ```
-

agents/review/kieran-rails-reviewer.md

@@ -1,6 +1,7 @@
 ---
 name: kieran-rails-reviewer
 description: Conditional code-review persona, selected when the diff touches Rails application code. Reviews Rails changes with Kieran's strict bar for clarity, conventions, and maintainability.
+model: inherit
 tools: Read, Grep, Glob, Bash
 color: blue
 mode: subagent
@@ -45,4 +46,3 @@ Return your findings as JSON matching the findings schema. No prose outside the
   "testing_gaps": []
 }
 ```
-

agents/review/kieran-typescript-reviewer.md

@@ -1,6 +1,7 @@
 ---
 name: kieran-typescript-reviewer
 description: Conditional code-review persona, selected when the diff touches TypeScript code. Reviews changes with Kieran's strict bar for type safety, clarity, and maintainability.
+model: inherit
 tools: Read, Grep, Glob, Bash
 color: blue
 mode: subagent
@@ -45,4 +46,3 @@ Return your findings as JSON matching the findings schema. No prose outside the
   "testing_gaps": []
 }
 ```
-