@lifeaitools/rdc-skills 0.35.5 → 0.35.7

package/skills/tests/onramp.test.json

@@ -1,101 +1,101 @@
-{
-  "manifest_version": 1,
-  "skill": "rdc:onramp",
-  "description": "Tier-2 self-test for rdc:onramp enrollment skill",
-  "tier": 2,
-  "sandbox": true,
-  "env": {
-    "RDC_TEST": "1"
-  },
-  "invocation": {
-    "slug": "onramp-selftest",
-    "name": "Onramp Self Test",
-    "flags": ["--archetype", "working-landscapes", "--dry-run"]
-  },
-  "fixture": {
-    "prompt": "rdc:onramp onramp-selftest --name \"Onramp Self Test\" --archetype working-landscapes --dry-run",
-    "env": { "RDC_\u0054EST": "1" },
-    "slow": true
-  },
-  "assertions": {
-    "exit_code": 0,
-    "stdout_contains": ["Preflight", "Resolve drift"]
-  },
-  "acceptance": {
-    "output_contains": ["Preflight", "Resolve drift", "disk-ahead", "already-enrolled"],
-    "output_not_contains": ["hand-edit _context.md", "raw insert"]
-  },
-  "legacy_assertions": [
-    {
-      "type": "exit_code",
-      "expect": 0,
-      "description": "Skill exits cleanly under RDC_TEST with --dry-run"
-    },
-    {
-      "type": "stdout_contains",
-      "pattern": "Preflight",
-      "description": "Checklist begins with Preflight step"
-    },
-    {
-      "type": "stdout_contains",
-      "pattern": "Resolve drift",
-      "description": "Drift resolution step is emitted"
-    },
-    {
-      "type": "stdout_contains",
-      "pattern": "\\[RDC_TEST\\]",
-      "description": "Sandbox short-circuit marker present — confirms external calls were skipped"
-    },
-    {
-      "type": "file_absent",
-      "path": "places/onramp-selftest/_context.md",
-      "description": "Scaffolder MUST NOT write _context.md — projection-drift violation gate"
-    }
-  ],
-  "door_contracts": [
-    {
-      "name": "enroll_place_signature",
-      "description": "enroll_place RPC must accept (text, text, jsonb, text) and return jsonb",
-      "type": "rpc_introspection",
-      "project_id": "uvojezuorjgqzmhhgluu",
-      "query": "SELECT p.proname, pg_get_function_identity_arguments(p.oid) AS args, pg_get_function_result(p.oid) AS returns FROM pg_proc p WHERE p.proname = 'enroll_place';",
-      "expect_args_contains": ["text", "jsonb"],
-      "expect_returns": "jsonb"
-    },
-    {
-      "name": "insert_work_item_accepts_project_node_id",
-      "description": "insert_work_item must accept p_project_node_id parameter",
-      "type": "rpc_introspection",
-      "project_id": "uvojezuorjgqzmhhgluu",
-      "query": "SELECT proname, pg_get_function_identity_arguments(oid) FROM pg_proc WHERE proname = 'insert_work_item';",
-      "expect_args_contains": ["p_project_node_id"]
-    }
-  ],
-  "disk_ahead_test": {
-    "description": "disk-ahead state must STOP and open a reconciliation work item",
-    "setup": "mkdir -p places/disk-ahead-test",
-    "invocation_slug": "disk-ahead-test",
-    "invocation_name": "Disk Ahead Test",
-    "assertions": [
-      {
-        "type": "stdout_contains",
-        "pattern": "disk-ahead",
-        "description": "Skill identifies the disk-ahead state"
-      },
-      {
-        "type": "stdout_contains",
-        "pattern": "STOP",
-        "description": "Skill STOPs on disk-ahead"
-      },
-      {
-        "type": "exit_nonzero",
-        "description": "Skill exits non-zero on disk-ahead (not a clean enrollment)"
-      }
-    ],
-    "teardown": "rmdir places/disk-ahead-test 2>/dev/null || true"
-  },
-  "idempotency_test": {
-    "description": "Running rdc:onramp twice for already-enrolled should produce zero new writes",
-    "note": "Under RDC_TEST=1 this is verified by checking that [RDC_TEST] skipping markers appear (not real insert calls) and the checklist reports already-enrolled"
-  }
-}
+{
+  "manifest_version": 1,
+  "skill": "rdc:onramp",
+  "description": "Tier-2 self-test for rdc:onramp enrollment skill",
+  "tier": 2,
+  "sandbox": true,
+  "env": {
+    "RDC_TEST": "1"
+  },
+  "invocation": {
+    "slug": "onramp-selftest",
+    "name": "Onramp Self Test",
+    "flags": ["--archetype", "working-landscapes", "--dry-run"]
+  },
+  "fixture": {
+    "prompt": "rdc:onramp onramp-selftest --name \"Onramp Self Test\" --archetype working-landscapes --dry-run",
+    "env": { "RDC_\u0054EST": "1" },
+    "slow": true
+  },
+  "assertions": {
+    "exit_code": 0,
+    "stdout_contains": ["Preflight", "Resolve drift"]
+  },
+  "acceptance": {
+    "output_contains": ["Preflight", "Resolve drift", "disk-ahead", "already-enrolled"],
+    "output_not_contains": ["hand-edit _context.md", "raw insert"]
+  },
+  "legacy_assertions": [
+    {
+      "type": "exit_code",
+      "expect": 0,
+      "description": "Skill exits cleanly under RDC_TEST with --dry-run"
+    },
+    {
+      "type": "stdout_contains",
+      "pattern": "Preflight",
+      "description": "Checklist begins with Preflight step"
+    },
+    {
+      "type": "stdout_contains",
+      "pattern": "Resolve drift",
+      "description": "Drift resolution step is emitted"
+    },
+    {
+      "type": "stdout_contains",
+      "pattern": "\\[RDC_TEST\\]",
+      "description": "Sandbox short-circuit marker present — confirms external calls were skipped"
+    },
+    {
+      "type": "file_absent",
+      "path": "places/onramp-selftest/_context.md",
+      "description": "Scaffolder MUST NOT write _context.md — projection-drift violation gate"
+    }
+  ],
+  "door_contracts": [
+    {
+      "name": "enroll_place_signature",
+      "description": "enroll_place RPC must accept (text, text, jsonb, text) and return jsonb",
+      "type": "rpc_introspection",
+      "project_id": "uvojezuorjgqzmhhgluu",
+      "query": "SELECT p.proname, pg_get_function_identity_arguments(p.oid) AS args, pg_get_function_result(p.oid) AS returns FROM pg_proc p WHERE p.proname = 'enroll_place';",
+      "expect_args_contains": ["text", "jsonb"],
+      "expect_returns": "jsonb"
+    },
+    {
+      "name": "insert_work_item_accepts_project_node_id",
+      "description": "insert_work_item must accept p_project_node_id parameter",
+      "type": "rpc_introspection",
+      "project_id": "uvojezuorjgqzmhhgluu",
+      "query": "SELECT proname, pg_get_function_identity_arguments(oid) FROM pg_proc WHERE proname = 'insert_work_item';",
+      "expect_args_contains": ["p_project_node_id"]
+    }
+  ],
+  "disk_ahead_test": {
+    "description": "disk-ahead state must STOP and open a reconciliation work item",
+    "setup": "mkdir -p places/disk-ahead-test",
+    "invocation_slug": "disk-ahead-test",
+    "invocation_name": "Disk Ahead Test",
+    "assertions": [
+      {
+        "type": "stdout_contains",
+        "pattern": "disk-ahead",
+        "description": "Skill identifies the disk-ahead state"
+      },
+      {
+        "type": "stdout_contains",
+        "pattern": "STOP",
+        "description": "Skill STOPs on disk-ahead"
+      },
+      {
+        "type": "exit_nonzero",
+        "description": "Skill exits non-zero on disk-ahead (not a clean enrollment)"
+      }
+    ],
+    "teardown": "rmdir places/disk-ahead-test 2>/dev/null || true"
+  },
+  "idempotency_test": {
+    "description": "Running rdc:onramp twice for already-enrolled should produce zero new writes",
+    "note": "Under RDC_TEST=1 this is verified by checking that [RDC_TEST] skipping markers appear (not real insert calls) and the checklist reports already-enrolled"
+  }
+}

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

@@ -1,12 +1,12 @@
-{
-  "manifest_version": 1,
-  "skill": "rdc:env",
-  "description": "Environment status is read-only and reports installed version.",
-  "fixture": { "prompt": "rdc:env status", "env": { "RDC_\u0054EST": "1" }, "slow": false },
-  "assertions": { "exit_code": 0, "stdout_contains": ["Environment"] },
-  "acceptance": {
-    "output_contains": ["version", "status"],
-    "output_not_contains": ["force", "delete environment"]
-  },
-  "teardown": { "reset_branch": true }
-}
+{
+  "manifest_version": 1,
+  "skill": "rdc:env",
+  "description": "Environment status is read-only and reports installed version.",
+  "fixture": { "prompt": "rdc:env status", "env": { "RDC_\u0054EST": "1" }, "slow": false },
+  "assertions": { "exit_code": 0, "stdout_contains": ["Environment"] },
+  "acceptance": {
+    "output_contains": ["version", "status"],
+    "output_not_contains": ["force", "delete environment"]
+  },
+  "teardown": { "reset_branch": true }
+}

package/skills/tests/rdc-new-model.test.json

@@ -1,12 +1,12 @@
-{
-  "manifest_version": 1,
-  "skill": "rdc:new-model",
-  "description": "New-model registration begins with the canonical gate.",
-  "fixture": { "prompt": "rdc:new-model example-model", "env": { "RDC_\u0054EST": "1" }, "slow": true },
-  "assertions": { "exit_code": 0, "stdout_contains": ["New Model"] },
-  "acceptance": {
-    "output_contains": ["canonical", "registration"],
-    "output_not_contains": ["raw insert", "skip gate"]
-  },
-  "teardown": { "reset_branch": true }
-}
+{
+  "manifest_version": 1,
+  "skill": "rdc:new-model",
+  "description": "New-model registration begins with the canonical gate.",
+  "fixture": { "prompt": "rdc:new-model example-model", "env": { "RDC_\u0054EST": "1" }, "slow": true },
+  "assertions": { "exit_code": 0, "stdout_contains": ["New Model"] },
+  "acceptance": {
+    "output_contains": ["canonical", "registration"],
+    "output_not_contains": ["raw insert", "skip gate"]
+  },
+  "teardown": { "reset_branch": true }
+}

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

@@ -1,29 +1,29 @@
-{
-  "manifest_version": 1,
-  "skill": "rdc:refactor",
-  "description": "Refactor preserves evidence and isolates all dispatched writers.",
-  "fixture": {
-    "prompt": "rdc:refactor hook-scope --takeover \"corrective consolidation\"",
-    "env": { "RDC_\u0054EST": "1" },
-    "slow": true
-  },
-  "assertions": {
-    "exit_code": 0,
-    "commits_made": { "min": 0 },
-    "stdout_contains": ["RDC Refactor"]
-  },
-  "acceptance": {
-    "output_contains": [
-      "consolidate_work_items_into_epic",
-      "unique leased worktree",
-      "collaboration manifest",
-      "validator"
-    ],
-    "output_not_contains": [
-      "raw update",
-      "shared working directory",
-      "archive all"
-    ]
-  },
-  "teardown": { "reset_branch": true }
-}
+{
+  "manifest_version": 1,
+  "skill": "rdc:refactor",
+  "description": "Refactor preserves evidence and isolates all dispatched writers.",
+  "fixture": {
+    "prompt": "rdc:refactor hook-scope --takeover \"corrective consolidation\"",
+    "env": { "RDC_\u0054EST": "1" },
+    "slow": true
+  },
+  "assertions": {
+    "exit_code": 0,
+    "commits_made": { "min": 0 },
+    "stdout_contains": ["RDC Refactor"]
+  },
+  "acceptance": {
+    "output_contains": [
+      "consolidate_work_items_into_epic",
+      "unique leased worktree",
+      "collaboration manifest",
+      "validator"
+    ],
+    "output_not_contains": [
+      "raw update",
+      "shared working directory",
+      "archive all"
+    ]
+  },
+  "teardown": { "reset_branch": true }
+}

package/skills/tests/rdc-regen-media.test.json

@@ -1,29 +1,29 @@
-{
-  "manifest_version": 1,
-  "skill": "rdc:regen-media",
-  "description": "Regen Media uses local keyless Codex gpt-image-2 as the primary image-generation path and avoids live generation under RDC_TEST.",
-  "fixture": {
-    "prompt": "rdc:regen-media generate a 2048x1152 homepage hero image",
-    "env": { "RDC_TEST": "1" }
-  },
-  "assertions": {
-    "exit_code": 0,
-    "commits_made": { "min": 0 },
-    "stdout_contains": ["image_gen", "gpt-image-2"]
-  },
-  "acceptance": {
-    "output_contains": [
-      "image_gen",
-      "gpt-image-2",
-      "--enable image_generation",
-      "RDC_TEST"
-    ],
-    "output_not_contains": [
-      "OPENAI_API_KEY=",
-      "generate_gpt_image is the default",
-      "upload completed",
-      "raw MCP"
-    ]
-  },
-  "teardown": { "reset_branch": true }
-}
+{
+  "manifest_version": 1,
+  "skill": "rdc:regen-media",
+  "description": "Regen Media uses local keyless Codex gpt-image-2 as the primary image-generation path and avoids live generation under RDC_TEST.",
+  "fixture": {
+    "prompt": "rdc:regen-media generate a 2048x1152 homepage hero image",
+    "env": { "RDC_TEST": "1" }
+  },
+  "assertions": {
+    "exit_code": 0,
+    "commits_made": { "min": 0 },
+    "stdout_contains": ["image_gen", "gpt-image-2"]
+  },
+  "acceptance": {
+    "output_contains": [
+      "image_gen",
+      "gpt-image-2",
+      "--enable image_generation",
+      "RDC_TEST"
+    ],
+    "output_not_contains": [
+      "OPENAI_API_KEY=",
+      "generate_gpt_image is the default",
+      "upload completed",
+      "raw MCP"
+    ]
+  },
+  "teardown": { "reset_branch": true }
+}

package/skills/watch/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: watch
-description: "Usage `rdc:watch` — Initialize a live session-log viewer and report its path; open it only during attended, non-test use. Zero infrastructure — pure filesystem + static HTML."
+description: rdc:watch () — open a live session-log viewer and report its path
 ---
 
 > **⚠️ OUTPUT CONTRACT (READ FIRST):** `guides/output-contract.md`

package/skills/workitems/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: workitems
-description: "Usage `rdc:workitems [add|done|status|list|query]` — Manual work item CRUD outside the normal build flow: add to backlog, mark done, update status, list open epics, query by label."
+description: rdc:workitems ([subcommand]) — create, update and query work items directly
 ---
 
 > **⚠️ OUTPUT CONTRACT (READ FIRST):** `guides/output-contract.md`

package/tests/clauth-plugin-postinstall.test.mjs

@@ -228,10 +228,23 @@ try {
 // the user installed.
 
 {
-  const pack = spawnSync('npm', ['pack', '--dry-run'], {
+  // `--json`, and loglevel forced.
+  //
+  // This read the human file listing, which npm prints at `notice` level. Run
+  // under `npm run --silent`, npm exports npm_config_loglevel=silent to every
+  // child, the listing vanishes, and the assertion reports "clauth-plugin.json
+  // must be present in the published tarball" — a packaging failure that is not
+  // happening. Measured 2026-08-29: PASS standalone, FAIL under --silent, with
+  // the file present in the tarball both times.
+  //
+  // A test whose verdict depends on the verbosity of the command that invoked
+  // it is a test that will eventually accuse the wrong thing. `--json` is a
+  // contract; the file listing is output formatting.
+  const pack = spawnSync('npm', ['pack', '--dry-run', '--json'], {
     cwd: REPO_ROOT,
     encoding: 'utf8',
     shell: process.platform === 'win32',
+    env: { ...process.env, npm_config_loglevel: 'notice' },
   });
   assert.equal(pack.status, 0, `npm pack --dry-run failed:\n${pack.stdout}\n${pack.stderr}`);
   const packOutput = `${pack.stdout}\n${pack.stderr}`;

package/tests/help-surface.test.mjs

@@ -40,8 +40,8 @@ assert.match(docs.readme, /Nineteen[\s\S]*\/rdc:\*` command shorthands/i, 'READM
 assert.match(docs.readme, /Use `rdc_skill_list` for the authoritative live catalog/, 'README should point callers to live MCP catalog');
 assert.doesNotMatch(docs.readme, /All user-invocable skills become available as slash commands/, 'README must not imply all MCP skills are slash commands');
 assert.doesNotMatch(docs.readme, /29 skills organized into 6 categories/, 'README must not carry stale category count');
-assert.match(docs.commandHelp, /menu of all MCP skills/, 'command help should refer to MCP skill catalog');
-assert.match(docs.skillHelp, /Show all MCP skills/, 'skill help should refer to MCP skill catalog');
+assert.match(docs.commandHelp, /all MCP skills/, 'command help should refer to MCP skill catalog');
+assert.match(docs.skillHelp, /all MCP skills/, 'skill help should refer to MCP skill catalog');
 assert.match(docs.skillHelp, /manifest-driven/i, 'skill help should be manifest-driven');
 assert.match(docs.commandHelp, /manifest-driven/i, 'command help should be manifest-driven');
 assert.doesNotMatch(docs.commandHelp, /Print the full usage menu below verbatim/, 'command help must not use stale static menu wording');

package/tests/require-work-item-on-commit.test.mjs

@@ -169,6 +169,60 @@ function runHook(payload, extraEnv = {}) {
   }
 }
 
+// ---------------------------------------------------------------------------
+// 4. The message must be READ before it is judged.
+//
+// extractCommitMessage used to fall back to the WHOLE COMMAND when there was no
+// -m. CONVENTIONAL_TYPES is anchored at ^, so a heredoc commit whose subject was
+// a valid `fix(scope): …` could never match — the tested string began with
+// `( cd … && git add … && git commit -q -F -`. Measured 2026-08-29: this hook
+// blocked a commit whose message DID begin with `fix(guards):` and reported
+// "no conventional commit type", which was false.
+//
+// A heredoc is how every multi-paragraph commit in this fleet is authored, so
+// the single path the gate made unusable was the correct one. The load-bearing
+// half of this test is the NEGATIVE cases: reading the message must not become
+// a way to pass without one.
+{
+  const { extractCommitMessage } = require(HOOK);
+  const CONVENTIONAL = /^(feat|fix|chore|refactor|test|docs|style|perf|ci|build|revert)(\(.+\))?:/i;
+  const UUID = /[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/i;
+  const verdict = (cmd) => {
+    const { message, readable } = extractCommitMessage(cmd);
+    if (!readable) return 'unreadable';
+    return CONVENTIONAL.test(message.trim()) || UUID.test(message) ? 'pass' : 'block';
+  };
+
+  const realBlockedCommand = `( cd "C:/Dev/lifeai-env.wt/guard-remedy" && git add hooks/lib/guard-rules.mjs && git commit -q -F - <<'EOF'
+fix(guards): rtp-steering-grep counted the pipeline next command as a search path
+
+A multi-paragraph body, which is the entire reason -F is being used.
+EOF
+)`;
+
+  for (const [name, cmd, want] of [
+    ['heredoc with a conventional subject passes', realBlockedCommand, 'pass'],
+    ['heredoc <<-EOF passes', "git commit -F - <<-EOF\nfix(a): b\nEOF", 'pass'],
+    ['heredoc with a quoted delimiter passes', 'git commit -F - <<"MSG"\nfix(a): b\nMSG', 'pass'],
+    ['heredoc carrying only a UUID passes', "git commit -F - <<'EOF'\nwip\n\nWork-Item: f27ff5fa-0000-4000-8000-000000000000\nEOF", 'pass'],
+    ['-m still passes', 'git commit -m "fix(x): y"', 'pass'],
+    // Negative controls — reading the message is not a bypass.
+    ['heredoc without a type or UUID still blocks', "git commit -F - <<'EOF'\nrandom words\nEOF", 'block'],
+    ['a bare commit still blocks', 'git commit', 'block'],
+    // Honest about what it cannot see, rather than calling it absent.
+    ['a real stdin pipe reports unreadable', 'cat msg.txt | git commit -q -F -', 'unreadable'],
+  ]) {
+    assert(`extract: ${name}`, verdict(cmd) === want, `got ${verdict(cmd)}, want ${want}`);
+  }
+
+  // -F <path> reads the message off disk.
+  const msgFile = join(mkdtempSync(join(tmpdir(), 'wi-msg-')), 'COMMIT_EDITMSG');
+  writeFileSync(msgFile, 'refactor(core): tidy\n\nbody\n');
+  assert('extract: -F <file> with a type passes', verdict(`git commit -F ${msgFile}`) === 'pass');
+  writeFileSync(msgFile, 'no type here\n');
+  assert('extract: -F <file> without a type still blocks', verdict(`git commit -F ${msgFile}`) === 'block');
+}
+
 // ---------------------------------------------------------------------------
 if (failures.length > 0) {
   console.error('\ncommit-capture hook tests — FAIL\n');

package/ecosystem.local.config.cjs

@@ -1,18 +0,0 @@
-// Local PM2 process definition for rdc-skills-mcp — mirrors regen-media-local
-// exactly (local PM2 process, tunneled via cloudflared, never on Vultr).
-module.exports = {
-  apps: [
-    {
-      name: "rdc-skills-mcp",
-      script: "bin/rdc-skills-mcp.mjs",
-      cwd: __dirname,
-      interpreter: "node",
-      env: {
-        PORT: "3110",
-      },
-      autorestart: true,
-      max_restarts: 10,
-      restart_delay: 5000,
-    },
-  ],
-};