@lifeaitools/rdc-skills 0.24.28 → 0.24.30

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "rdc",
-  "version": "0.24.28",
+  "version": "0.24.30",
   "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",

package/git-sha.json

@@ -1,3 +1,3 @@
 {
-  "sha": "5e9edbe4cc32b027c05dda21e0a9e2f07bba8e59"
+  "sha": "e247eb79f223c88807aab96c87a8fe37e5b67f15"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lifeaitools/rdc-skills",
-  "version": "0.24.28",
+  "version": "0.24.30",
   "description": "RDC typed-agent dispatch skill suite for Claude Code - plan, build, review, overnight builds",
   "keywords": [
     "claude-code",

package/skills/tests/MATRIX.md

@@ -4,7 +4,7 @@ Current coverage: 29 manifests for 29 skill directories.
 
 The manifest layer verifies each skill can be started from a realistic caller prompt in an isolated `RDC_TEST=1` sandbox. The acceptance harness records the engine stream, extracted tool calls, stdout/stderr artifacts, rendered assistant output, failures, lessons learned, and next build optimizations under `.rdc/reports/`.
 
-`rdc:channel-formatter` currently has the strongest content acceptance fixture: it asserts source-grounded social-pack output, forbidden-claim absence, and observable corpus or web-search tool routing. `rdc:help` asserts the public MCP/curl caller surface and structured `format:"json"` discovery path. `rdc:watch` adds a focus-safety negative check. The remaining manifests are basic behavioral smoke tests and should gain deeper acceptance assertions as each skill is touched.
+`rdc:channel-formatter` currently has the strongest content acceptance fixture: it asserts source-grounded social-pack output, forbidden-claim absence, and observable corpus or web-search tool routing. `rdc:help` asserts the public MCP/curl caller surface and structured `format:"json"` discovery path. `rdc:self-test` validates the test-tier/evidence language. Planning/reporting skills (`rdc:preplan`, `rdc:plan`, `rdc:handoff`, `rdc:prototype`, `rdc:review`, `rdc:report`) now assert their required artifacts and safety boundaries. `rdc:release`, `rdc:deploy`, `rdc:terminal-config`, `rdc:status`, and `rdc:watch` add safety negative checks for publish/deploy/window/read-only behavior. The remaining manifests are basic behavioral smoke tests and should gain deeper acceptance assertions as each skill is touched.
 
 | Skill | Manifest | Fixture prompt class | Assertions | Acceptance depth |
 |---|---|---|---|---|
@@ -14,27 +14,27 @@ The manifest layer verifies each skill can be started from a realistic caller pr
 | `rdc:co-develop` | `rdc-co-develop.test.json` | Coordination status | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:collab` | `rdc-collab.test.json` | Claude session relay fixture | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:convert` | `rdc-convert.test.json` | Markdown-to-Word conversion fixture | `exit_code`, `stdout_contains` | Basic manifest |
-| `rdc:deploy` | `rdc-deploy.test.json` | Deployment diagnosis | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
+| `rdc:deploy` | `rdc-deploy.test.json` | Deployment diagnosis | `commits_made`, `exit_code`, `stdout_contains` | Read-only diagnose output and destructive deploy/DNS negative checks |
 | `rdc:design` | `rdc-design.test.json` | Studio palette audit | `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:fixit` | `rdc-fixit.test.json` | Tiny sandbox typo fix | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
 | `rdc:fs-mcp` | `rdc-fs-mcp.test.json` | File-system bridge read fixture | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
-| `rdc:handoff` | `rdc-handoff.test.json` | Stub work handoff | `exit_code`, `stdout_contains` | Basic manifest |
+| `rdc:handoff` | `rdc-handoff.test.json` | Stub work handoff | `exit_code`, `stdout_contains` | Plan path, DoD, work-item handoff, and placeholder negative checks |
 | `rdc:help` | `rdc-help.test.json` | Help menu rendering | `commits_made`, `exit_code`, `stdout_contains` | MCP/curl output, structured JSON fetch, dev-endpoint negative checks |
 | `rdc:housekeeping` | `rdc-housekeeping.test.json` | Read-only housekeeping audit | `commits_made`, `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:plan` | `rdc-plan.test.json` | Health-endpoint planning prompt | `exit_code`, `stdout_contains` | Design-decision, decomposition-matrix, work-package, and coarse-check negative checks |
+| `rdc:preplan` | `rdc-preplan.test.json` | Rate-limiter research prompt | `commits_made`, `exit_code`, `stdout_contains` | Research artifact, comparison, unknowns, recommendation, and research-tool routing checks |
+| `rdc:prototype` | `rdc-prototype.test.json` | Tiny component prototype prompt | `exit_code`, `stdout_contains` | Local TSX prototype artifact, registry, test-mode, and production-write negative checks |
 | `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 |
+| `rdc:release` | `rdc-release.test.json` | Dry-run package release | `commits_made`, `exit_code`, `stdout_contains` | Dry-run release checklist and force/bypass negative checks |
+| `rdc:report` | `rdc-report.test.json` | Unattended report generation | `exit_code`, `stdout_contains` | Report status block, counts, next recommendation, and dump/push negative checks |
+| `rdc:review` | `rdc-review.test.json` | Unattended review gate | `exit_code`, `stdout_contains` | Code-review, verify, fresh-evidence, tsc, and forbidden-build negative checks |
 | `rdc:rpms-filemap` | `rdc-rpms-filemap.test.json` | Canonical location lookup | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
-| `rdc:self-test` | `rdc-self-test.test.json` | Strict self-test prompt | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
-| `rdc:status` | `rdc-status.test.json` | Read-only status snapshot | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
-| `rdc:terminal-config` | `rdc-terminal-config.test.json` | Hidden-window launch policy audit | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
+| `rdc:self-test` | `rdc-self-test.test.json` | Strict self-test prompt | `commits_made`, `exit_code`, `stdout_contains` | Tier/evidence output checks and no-relaxed-linter negative check |
+| `rdc:status` | `rdc-status.test.json` | Read-only status snapshot | `commits_made`, `exit_code`, `stdout_contains` | Read-only status output and raw-MCP/write negative checks |
+| `rdc:terminal-config` | `rdc-terminal-config.test.json` | Hidden-window launch policy audit | `commits_made`, `exit_code`, `stdout_contains` | Hidden-window output and focus/collapse API negative checks |
 | `rdc:watch` | `rdc-watch.test.json` | Watcher initialization prompt | `exit_code`, `stdout_contains` | Test-mode no-open output, tool-call routing, and focus-event negative checks |
 | `rdc:workitems` | `rdc-workitems.test.json` | Work-item epic listing | `commits_made`, `exit_code`, `stdout_contains` | Basic manifest |
 

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

@@ -11,5 +11,20 @@
     "commits_made": { "min": 0 },
     "stdout_contains": ["Verdict"]
   },
+  "acceptance": {
+    "output_contains": [
+      "diagnose",
+      "read-only",
+      "Registry",
+      "health",
+      "Verdict"
+    ],
+    "output_not_contains": [
+      "Deploy triggered",
+      "Cloudflare cache purged",
+      "PR/admin-merge to main",
+      "DNS write"
+    ]
+  },
   "teardown": { "reset_branch": true }
 }

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

@@ -10,5 +10,19 @@
     "exit_code": 0,
     "stdout_contains": ["Verdict"]
   },
+  "acceptance": {
+    "output_contains": [
+      ".rdc/plans",
+      "acceptance criterion",
+      "DoD",
+      "work item",
+      "rdc:build"
+    ],
+    "output_not_contains": [
+      "git push",
+      "raw MCP",
+      "[specific acceptance criterion]"
+    ]
+  },
   "teardown": { "reset_branch": true }
 }

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

@@ -10,5 +10,19 @@
     "exit_code": 0,
     "stdout_contains": ["Verdict"]
   },
+  "acceptance": {
+    "output_contains": [
+      "Design Decision",
+      "Checklist Decomposition Matrix",
+      "Work package",
+      "Verification artifact",
+      "rdc:build"
+    ],
+    "output_not_contains": [
+      "verify UI",
+      "integration complete",
+      "raw MCP"
+    ]
+  },
   "teardown": { "reset_branch": true }
 }

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

@@ -11,5 +11,20 @@
     "commits_made": { "min": 0 },
     "stdout_contains": ["Verdict"]
   },
+  "acceptance": {
+    "output_contains": [
+      "Research",
+      "Best-in-Class",
+      "Comparison",
+      "Unknowns",
+      "Recommendation"
+    ],
+    "output_not_contains": [
+      "create work items",
+      "architectural decision",
+      "write code"
+    ],
+    "tool_calls_include_any": ["WebSearch", "web.search_query", "Grep", "Glob"]
+  },
   "teardown": { "reset_branch": true }
 }

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

@@ -10,5 +10,19 @@
     "exit_code": 0,
     "stdout_contains": ["Verdict"]
   },
+  "acceptance": {
+    "output_contains": [
+      "docs/source",
+      "prototype",
+      "TSX",
+      "registry",
+      "RDC_TEST"
+    ],
+    "output_not_contains": [
+      "git push",
+      "production deploy",
+      "Supabase insert"
+    ]
+  },
   "teardown": { "reset_branch": true }
 }

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

@@ -11,5 +11,19 @@
     "commits_made": { "min": 0 },
     "stdout_contains": ["Verdict"]
   },
+  "acceptance": {
+    "output_contains": [
+      "dry-run",
+      "Tests/self-test",
+      "CI/publish",
+      "Installed/runtime version",
+      "Smoke test"
+    ],
+    "output_not_contains": [
+      "git push --force",
+      "bypass hooks",
+      "raw git/npm/CI dumps"
+    ]
+  },
   "teardown": { "reset_branch": true }
 }

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

@@ -10,5 +10,19 @@
     "exit_code": 0,
     "stdout_contains": ["Verdict"]
   },
+  "acceptance": {
+    "output_contains": [
+      "REPORT_STATUS",
+      "report_path",
+      "completed_count",
+      "open_count",
+      "Next Session Recommendation"
+    ],
+    "output_not_contains": [
+      "raw MCP",
+      "git push",
+      "unbounded dump"
+    ]
+  },
   "teardown": { "reset_branch": true }
 }

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

@@ -10,5 +10,19 @@
     "exit_code": 0,
     "stdout_contains": ["Verdict"]
   },
+  "acceptance": {
+    "output_contains": [
+      "code-review",
+      "verify",
+      "fresh evidence",
+      "tsc",
+      "Verdict"
+    ],
+    "output_not_contains": [
+      "pnpm build",
+      "raw MCP",
+      "CLEAN without"
+    ]
+  },
   "teardown": { "reset_branch": true }
 }

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

@@ -11,5 +11,18 @@
     "commits_made": { "min": 0 },
     "stdout_contains": ["PASS", "strict"]
   },
+  "acceptance": {
+    "output_contains": [
+      "Tier 1",
+      "Tier 2",
+      "29 manifests",
+      "JSONL",
+      "tool calls"
+    ],
+    "output_not_contains": [
+      "skip findings by relaxing the linter",
+      "raw runner dumps"
+    ]
+  },
   "teardown": { "reset_branch": true }
 }

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

@@ -11,5 +11,19 @@
     "commits_made": { "min": 0 },
     "stdout_contains": ["Recommended Next"]
   },
+  "acceptance": {
+    "output_contains": [
+      "Recommended Next",
+      "read-only",
+      "work items",
+      "health"
+    ],
+    "output_not_contains": [
+      "raw MCP",
+      "UUID",
+      "UPDATE work_items",
+      "INSERT"
+    ]
+  },
   "teardown": { "reset_branch": true }
 }

package/skills/tests/rdc-terminal-config.test.json

@@ -11,5 +11,19 @@
     "commits_made": { "min": 0 },
     "stdout_contains": ["hidden-window", "No settings written"]
   },
+  "acceptance": {
+    "output_contains": [
+      "hidden-window",
+      "No settings written",
+      "read before edit",
+      "JSON/TOML syntax"
+    ],
+    "output_not_contains": [
+      "SetForegroundWindow",
+      "ShowWindow",
+      "collapse all windows",
+      "restore all windows"
+    ]
+  },
   "teardown": { "reset_branch": true }
 }

package/tests/curl-surface.test.mjs

@@ -226,6 +226,26 @@ async function main() {
     check('rdc_skill_get format=json returns skill metadata', getJsonBody.skill?.slash === 'rdc:build' && getJsonBody.skill?.codeflow_required === true);
     check('rdc_skill_get format=json returns rendered body', /rdc:build/i.test(getJsonBody.body || ''));
 
+    const getJsonUnknown = postMcp({
+      jsonrpc: '2.0',
+      id: 6,
+      method: 'tools/call',
+      params: {
+        name: 'rdc_skill_get',
+        arguments: { name: 'rdc:not-real', format: 'json' },
+      },
+    }, 'rdc_skill_get_json_unknown');
+    check('rdc_skill_get unknown format=json curl exits 0', getJsonUnknown.status === 0, getJsonUnknown.stderr);
+    let getJsonUnknownBody = {};
+    const getJsonUnknownText = resultText(latestEnvelope(getJsonUnknown.stdout));
+    try {
+      getJsonUnknownBody = JSON.parse(getJsonUnknownText);
+    } catch {
+      check('rdc_skill_get unknown format=json returns parseable JSON', false, getJsonUnknownText || getJsonUnknown.stdout.slice(0, 500));
+    }
+    check('rdc_skill_get unknown format=json returns machine error', getJsonUnknownBody.error === 'unknown_skill');
+    check('rdc_skill_get unknown format=json returns alternatives', getJsonUnknownBody.valid_count >= 29 && getJsonUnknownBody.valid?.some((s) => s.slash === 'rdc:help'));
+
     const getMcp = curlWithStatus([
       '-s',
       `${TARGET}/mcp`,

package/tests/mcp.test.mjs

@@ -195,6 +195,10 @@ async function sweep(url, label, { compareSource }) {
   // error path
   const unk = await mcp(url, { jsonrpc: '2.0', id: 20, method: 'tools/call', params: { name: 'rdc_skill_get', arguments: { name: '___nope___' } } });
   check(`${label}: unknown skill returns helpful error`, /unknown skill/i.test(callText(unk.json)));
+  const unkJson = await mcp(url, { jsonrpc: '2.0', id: 24, method: 'tools/call', params: { name: 'rdc_skill_get', arguments: { name: '___nope___', format: 'json' } } });
+  let unkJsonBody; try { unkJsonBody = JSON.parse(callText(unkJson.json)); } catch { unkJsonBody = {}; }
+  check(`${label}: unknown skill format=json returns machine-readable error`, unkJsonBody.error === 'unknown_skill');
+  check(`${label}: unknown skill format=json includes valid alternatives`, unkJsonBody.valid_count === names.length && Array.isArray(unkJsonBody.valid) && unkJsonBody.valid.some((s) => s.slash === 'rdc:help'));
 
   // Alias path: callers naturally copy the slash form from rdc_skill_list.
   const aliasGet = await mcp(url, { jsonrpc: '2.0', id: 22, method: 'tools/call', params: { name: 'rdc_skill_get', arguments: { name: 'rdc:brochurify', variant: 'cli' } } });