@lifeaitools/rdc-skills 0.24.14 → 0.24.16

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "rdc",
-  "version": "0.24.14",
+  "version": "0.24.16",
   "description": "RDC typed-agent dispatch skill suite for Claude Code — plan, build, review, overnight unattended builds with work-item tracking and TDD enforcement.",
   "author": {
     "name": "LIFEAI",
@@ -158,6 +158,89 @@
       "enabled_default": true,
       "codeflow_required": false
     },
+    "rdc-brochurify": {
+      "name": "rdc-brochurify",
+      "slash": "rdc:brochurify",
+      "category": "tooling",
+      "usage": "rdc:brochurify <source|job-payload>",
+      "args": {
+        "positional": [
+          {
+            "name": "source",
+            "required": true,
+            "description": "URL, HTML, DOCX, Markdown, JSX, corpus query, or broker job payload to turn into a brochure PDF.",
+            "accepts": "path|url|json|string"
+          }
+        ],
+        "flags": []
+      },
+      "requires": [
+        "agent-dispatch"
+      ],
+      "produces": [
+        "pdf_artifact",
+        ".rdc/reports/"
+      ],
+      "default_model": "opus",
+      "triggers": [
+        "rdc:brochurify",
+        "brochurify this",
+        "make a brochure from",
+        "convert this to a brochure PDF",
+        "generate brochure from",
+        "monkey_dispatch payload with skill=\"brochurify\""
+      ],
+      "follows": [
+        "channel-formatter",
+        "convert",
+        "lifeai-brochure-author"
+      ],
+      "leads_to": [],
+      "sandbox_aware": true,
+      "output_contract": "guides/output-contract.md",
+      "enabled_default": true,
+      "codeflow_required": false
+    },
+    "lifeai-brochure-author": {
+      "name": "lifeai-brochure-author",
+      "slash": "lifeai-brochure-author",
+      "category": "tooling",
+      "usage": "lifeai-brochure-author <brochure-jsx-task>",
+      "args": {
+        "positional": [
+          {
+            "name": "brochure-jsx-task",
+            "required": true,
+            "description": "Brief, source content, or JSX verification task for @lifeai/brochure-kit output.",
+            "accepts": "string|path"
+          }
+        ],
+        "flags": []
+      },
+      "requires": [],
+      "produces": [
+        "brochure_jsx_guidance"
+      ],
+      "default_model": "opus",
+      "triggers": [
+        "write a brochure",
+        "make a one-pager",
+        "draft a PDF report",
+        "design a brochure",
+        "create an investor doc",
+        "generate a fact sheet",
+        "brochurify",
+        "@lifeai/brochure-kit"
+      ],
+      "follows": [],
+      "leads_to": [
+        "rdc-brochurify"
+      ],
+      "sandbox_aware": false,
+      "output_contract": "guides/output-contract.md",
+      "enabled_default": true,
+      "codeflow_required": false
+    },
     "build": {
       "name": "build",
       "slash": "rdc:build",
@@ -382,7 +465,7 @@
       "follows": [],
       "leads_to": [],
       "sandbox_aware": false,
-      "output_contract": null,
+      "output_contract": "guides/output-contract.md",
       "enabled_default": true,
       "codeflow_required": false
     },
@@ -815,6 +898,75 @@
       "enabled_default": true,
       "codeflow_required": true
     },
+    "rdc-extract-verifier-rules": {
+      "name": "rdc-extract-verifier-rules",
+      "slash": "rdc:extract-verifier-rules",
+      "category": "tooling",
+      "usage": "rdc:extract-verifier-rules [enhancement-log]",
+      "args": {
+        "positional": [
+          {
+            "name": "enhancement-log",
+            "required": false,
+            "description": "Optional path or description of enhancement-log entries to cluster into verifier-rule candidates.",
+            "accepts": "path|string"
+          }
+        ],
+        "flags": []
+      },
+      "requires": [
+        "git"
+      ],
+      "produces": [
+        ".rdc/reports/",
+        "git_commits"
+      ],
+      "default_model": "opus",
+      "triggers": [
+        "rdc:extract-verifier-rules",
+        "extract verifier rules",
+        "promote enhancement log",
+        "what new rules should we add",
+        "verifier corpus update",
+        "nightly cron at 3:00 AM PT"
+      ],
+      "follows": [
+        "rdc-brochurify"
+      ],
+      "leads_to": [
+        "review"
+      ],
+      "sandbox_aware": false,
+      "output_contract": "guides/output-contract.md",
+      "enabled_default": true,
+      "codeflow_required": false
+    },
+    "rpms-filemap": {
+      "name": "rpms-filemap",
+      "slash": "rdc:rpms-filemap",
+      "category": "tooling",
+      "usage": "rdc:rpms-filemap",
+      "args": {
+        "positional": [],
+        "flags": []
+      },
+      "requires": [],
+      "produces": [],
+      "default_model": "haiku",
+      "triggers": [
+        "rpms file map",
+        "rule #1",
+        "context export",
+        "where does this file belong",
+        "where should pm artifacts go"
+      ],
+      "follows": [],
+      "leads_to": [],
+      "sandbox_aware": false,
+      "output_contract": "guides/output-contract.md",
+      "enabled_default": true,
+      "codeflow_required": false
+    },
     "release": {
       "name": "release",
       "slash": "rdc:release",
@@ -1005,7 +1157,7 @@
       ],
       "follows": [],
       "leads_to": [],
-      "sandbox_aware": false,
+      "sandbox_aware": true,
       "output_contract": "guides/output-contract.md",
       "enabled_default": true,
       "codeflow_required": false
@@ -1135,7 +1287,7 @@
       ],
       "follows": [],
       "leads_to": [],
-      "sandbox_aware": false,
+      "sandbox_aware": true,
       "output_contract": "guides/output-contract.md",
       "enabled_default": true,
       "codeflow_required": false

package/git-sha.json

@@ -1,3 +1,3 @@
 {
-  "sha": "090e6c4a0a643acca7ca0bfca384f43cc150dca9"
+  "sha": "36691cfc1928d1024db4b6e4c40b4a6c13668ac6"
 }

package/lib/catalog.mjs

@@ -8,9 +8,10 @@
  *
  * Loading is LIVE: the catalog is re-read from disk when older than CACHE_TTL_MS
  * so a skill edit (frontmatter or body) is picked up without a server restart.
- * A skill that has a `skills_meta` entry but no SKILL.md on disk is still listed
- * (meta-only); a skill dir with a SKILL.md but no meta entry is also surfaced
- * (frontmatter-only) so nothing silently disappears from the catalog.
+ * Published packages must have one `skills_meta` entry for every skill
+ * directory. The frontmatter-only path below is a local-dev resilience fallback
+ * so an incomplete checkout does not silently hide a skill while it is being
+ * repaired; tests enforce that released packages do not rely on it.
  */
 
 import fs from 'node:fs';
@@ -45,9 +46,8 @@ function readFrontmatter(skillMdPath) {
 function toCatalogEntry(name, meta, frontmatter) {
   const fm = frontmatter || {};
   const description = fm.description || meta.description || '';
-  // triggers/usage/slash/etc. may live in skills_meta OR the SKILL.md frontmatter.
-  // Fall back to frontmatter so a skill not (yet) registered in skills_meta stays
-  // searchable — these fields are what `rdc_skill_search` routes on.
+  // triggers/usage/slash/etc. should live in skills_meta for published packages.
+  // Frontmatter fallback keeps incomplete local checkouts searchable during repair.
   const metaTriggers = Array.isArray(meta.triggers) && meta.triggers.length ? meta.triggers : null;
   const fmTriggers = Array.isArray(fm.triggers) ? fm.triggers : [];
   const slash = meta.slash || fm.slash || `rdc:${name}`;
@@ -84,7 +84,7 @@ function buildCatalog() {
     byName.set(name, toCatalogEntry(name, meta || {}, fm));
   }
 
-  // 2. Any skill dir with a SKILL.md but NO meta entry → frontmatter-only row.
+  // 2. Local-dev fallback: a skill dir with SKILL.md but no meta entry.
   if (fs.existsSync(SKILLS_DIR)) {
     for (const entry of fs.readdirSync(SKILLS_DIR, { withFileTypes: true })) {
       if (!entry.isDirectory() || byName.has(entry.name)) continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lifeaitools/rdc-skills",
-  "version": "0.24.14",
+  "version": "0.24.16",
   "description": "RDC typed-agent dispatch skill suite for Claude Code - plan, build, review, overnight builds",
   "keywords": [
     "claude-code",
@@ -35,7 +35,7 @@
     "rdc-design": "node scripts/rdc-design-cli.mjs",
     "test:hooks": "node scripts/test-rdc-hooks.mjs",
     "test:truth-gate": "node tests/run-evidence-gate.test.mjs && node tests/work-item-exit-gate-l2.test.mjs && node tests/work-item-exit-gate-l3.test.mjs && node tests/require-work-item-on-commit.test.mjs && node tests/harness-gates.test.mjs",
-    "test:acceptance": "node tests/acceptance.test.mjs && node tests/install-rdc-skills.test.mjs && node tests/help-surface.test.mjs",
+    "test:acceptance": "node tests/acceptance.test.mjs && node tests/install-rdc-skills.test.mjs && node tests/help-surface.test.mjs && node tests/manifest-contract-fields.test.mjs",
     "acceptance": "node scripts/acceptance.mjs --changed",
     "test:mcp": "node tests/mcp.test.mjs",
     "test:mcp:remote": "node tests/mcp.test.mjs --remote",

package/skills/co-develop/SKILL.md

@@ -6,6 +6,7 @@ description: "Usage `rdc:co-develop <ask|reply|inbox|start|resume|status>` — p
 > **⚠️ OUTPUT CONTRACT (READ FIRST):** `guides/output-contract.md`
 > Checklist-only output. No tool-call narration. No raw MCP/JSON/log dumps.
 > One checklist upfront, updated in place, shown again at end with a 1-line verdict.
+> **Sandbox contract:** This skill honors `RDC_TEST=1` per `guides/agent-bootstrap.md` § RDC_TEST Sandbox Contract. Under the flag, do not create/send/reply to live co-development turns; report the intended clauth command and mark the external relay action as skipped.
 
 # rdc:co-develop — Peer-Aware Co-Development
 

package/skills/tests/MATRIX.md

@@ -21,13 +21,13 @@ The manifest layer verifies each skill can be started from a realistic caller pr
 | `rdc:handoff` | `rdc-handoff.test.json` | Stub work handoff | `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:help` | `rdc-help.test.json` | Help menu rendering | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:housekeeping` | `rdc-housekeeping.test.json` | Read-only housekeeping audit | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
-| `rdc:lifeai-brochure-author` | `rdc-lifeai-brochure-author.test.json` | JSX compliance review fixture | `exit_code`, `stdout_contains` | Basic manifest |
+| `lifeai-brochure-author` | `rdc-lifeai-brochure-author.test.json` | JSX compliance review fixture | `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:overnight` | `rdc-overnight.test.json` | Label-based overnight queue drain | `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:plan` | `rdc-plan.test.json` | Health-endpoint planning prompt | `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:preplan` | `rdc-preplan.test.json` | Rate-limiter research prompt | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:prototype` | `rdc-prototype.test.json` | Tiny component prototype prompt | `exit_code`, `stdout_contains` | Basic manifest |
-| `rdc:rdc-brochurify` | `rdc-rdc-brochurify.test.json` | Read-only markdown Brochurify fixture | `exit_code`, `stdout_contains` | Sandbox output and no-follow-up negative checks |
-| `rdc:rdc-extract-verifier-rules` | `rdc-rdc-extract-verifier-rules.test.json` | Enhancement-log verifier fixture | `exit_code`, `stdout_contains` | Basic manifest |
+| `rdc:brochurify` | `rdc-rdc-brochurify.test.json` | Read-only markdown Brochurify fixture | `exit_code`, `stdout_contains` | Sandbox output and no-follow-up negative checks |
+| `rdc:extract-verifier-rules` | `rdc-rdc-extract-verifier-rules.test.json` | Enhancement-log verifier fixture | `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:release` | `rdc-release.test.json` | Dry-run package release | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:report` | `rdc-report.test.json` | Unattended report generation | `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:review` | `rdc-review.test.json` | Unattended review gate | `exit_code`, `stdout_contains` | Basic manifest |

package/skills/tests/rdc-lifeai-brochure-author.test.json

@@ -1,9 +1,9 @@
 {
   "manifest_version": 1,
-  "skill": "rdc:lifeai-brochure-author",
+  "skill": "lifeai-brochure-author",
   "description": "LifeAI brochure author contract rejects raw HTML and returns brochure-kit component guidance.",
   "fixture": {
-    "prompt": "rdc:lifeai-brochure-author review this JSX fixture for brochure-kit compliance: docs/source/brochure-author-fixture.tsx",
+    "prompt": "lifeai-brochure-author review this JSX fixture for brochure-kit compliance: docs/source/brochure-author-fixture.tsx",
     "precondition_files": [
       {
         "path": "docs/source/brochure-author-fixture.tsx",

package/skills/tests/rdc-rdc-brochurify.test.json

@@ -1,9 +1,9 @@
 {
   "manifest_version": 1,
-  "skill": "rdc:rdc-brochurify",
+  "skill": "rdc:brochurify",
   "description": "Brochurify orchestrator accepts a markdown source fixture and reports sandboxed orchestration state.",
   "fixture": {
-    "prompt": "rdc:rdc-brochurify docs/source/brochurify-fixture.md --mode read-only --theme editorial-neutral",
+    "prompt": "rdc:brochurify docs/source/brochurify-fixture.md --mode read-only --theme editorial-neutral",
     "precondition_files": [
       {
         "path": "docs/source/brochurify-fixture.md",

package/skills/tests/rdc-rdc-extract-verifier-rules.test.json

@@ -1,9 +1,9 @@
 {
   "manifest_version": 1,
-  "skill": "rdc:rdc-extract-verifier-rules",
+  "skill": "rdc:extract-verifier-rules",
   "description": "Verifier-rule extraction reads a tiny enhancement-log fixture and reports candidate-rule status without opening a PR.",
   "fixture": {
-    "prompt": "rdc:rdc-extract-verifier-rules --since 2026-01-01 --auto-merge=false docs/source/enhancement-log-fixture.jsonl",
+    "prompt": "rdc:extract-verifier-rules --since 2026-01-01 --auto-merge=false docs/source/enhancement-log-fixture.jsonl",
     "precondition_files": [
       {
         "path": "docs/source/enhancement-log-fixture.jsonl",

package/tests/manifest-contract-fields.test.mjs

@@ -0,0 +1,78 @@
+#!/usr/bin/env node
+import assert from 'node:assert/strict';
+import { existsSync, readdirSync, readFileSync } from 'node:fs';
+import { dirname, join, resolve } from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const root = resolve(__dirname, '..');
+const manifest = JSON.parse(readFileSync(join(root, '.claude-plugin', 'plugin.json'), 'utf8'));
+const skillDirs = readdirSync(join(root, 'skills'), { withFileTypes: true })
+  .filter((entry) => entry.isDirectory() && entry.name !== 'tests')
+  .map((entry) => entry.name)
+  .sort();
+const metaNames = Object.keys(manifest.skills_meta).sort();
+const manifestSlashes = new Set(Object.values(manifest.skills_meta).map((meta) => meta.slash));
+
+assert.deepEqual(
+  metaNames,
+  skillDirs,
+  'Every skills/<name>/SKILL.md directory must have a skills_meta entry so MCP/help use explicit caller-facing terms',
+);
+
+function skillPath(name) {
+  const direct = join(root, 'skills', name, 'SKILL.md');
+  if (existsSync(direct)) return direct;
+  const prefixed = join(root, 'skills', `rdc-${name}`, 'SKILL.md');
+  if (existsSync(prefixed)) return prefixed;
+  return direct;
+}
+
+for (const [name, meta] of Object.entries(manifest.skills_meta)) {
+  const file = skillPath(name);
+  assert.equal(existsSync(file), true, `${name} must have a SKILL.md`);
+  const body = readFileSync(file, 'utf8');
+
+  assert.equal(meta.name, name, `${name} manifest name must match its skills_meta key`);
+  assert.doesNotMatch(meta.slash, /^rdc:rdc-/, `${name} slash command must not be a synthesized rdc:rdc-* name`);
+  assert.doesNotMatch(meta.slash, /^rdc:lifeai-/, `${name} slash command must not invent an rdc: prefix for non-rdc skills`);
+
+  const hasOutputContract = body.includes('guides/output-contract.md') || body.includes('OUTPUT CONTRACT');
+  assert.equal(
+    Boolean(meta.output_contract),
+    hasOutputContract,
+    `${name} manifest output_contract must match SKILL.md banner`,
+  );
+
+  if (body.includes('RDC_TEST=1')) {
+    assert.equal(meta.sandbox_aware, true, `${name} mentions RDC_TEST=1 and must be sandbox_aware`);
+  }
+
+  if (meta.sandbox_aware) {
+    assert.match(
+      body,
+      /Sandbox contract|RDC_TEST=1/,
+      `${name} is sandbox_aware and must explain the sandbox contract in SKILL.md`,
+    );
+  }
+}
+
+const testManifests = readdirSync(join(root, 'skills', 'tests'), { withFileTypes: true })
+  .filter((entry) => entry.isFile() && entry.name.endsWith('.test.json'));
+
+for (const entry of testManifests) {
+  const testPath = join(root, 'skills', 'tests', entry.name);
+  const test = JSON.parse(readFileSync(testPath, 'utf8'));
+  assert.equal(
+    manifestSlashes.has(test.skill),
+    true,
+    `${entry.name} skill "${test.skill}" must match a skills_meta[].slash value`,
+  );
+  assert.match(
+    test.fixture?.prompt || '',
+    new RegExp(`(^|\\s)${test.skill.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}(\\s|$)`),
+    `${entry.name} prompt must invoke its manifest skill name`,
+  );
+}
+
+console.log('manifest contract field tests — PASS');

package/tests/mcp.test.mjs

@@ -88,7 +88,10 @@ function unitTests() {
   check('unit: catalog non-empty', cat.length >= 20, `got ${cat.length}`);
   check('unit: every entry has name+slash+category+summary field',
     cat.every((s) => s.name && s.slash && s.category && 'summary' in s));
-  check('unit: every slash is rdc:<name> shape', cat.every((s) => /^rdc:/.test(s.slash)));
+  check('unit: every slash is explicit caller-facing shape',
+    cat.every((s) => /^rdc:/.test(s.slash) || s.slash === s.name));
+  check('unit: no synthesized duplicate rdc prefixes',
+    cat.every((s) => !/^rdc:rdc-/.test(s.slash) && !/^rdc:lifeai-/.test(s.slash)));
   check('unit: getSkill(deploy) resolves', !!getSkill('deploy'));
   check('unit: getSkill(nonexistent) is null', getSkill('___nope___') === null);
   check('unit: search ranks exact name first', searchSkills('deploy')[0]?.name === 'deploy');