@lifeaitools/rdc-skills 0.24.41 → 0.25.0

package/tests/run-evidence-gate.test.mjs

@@ -1,82 +1,82 @@
-#!/usr/bin/env node
-/**
- * Truth Gate 3.0 — fused run-evidence-gate primitive tests.
- *
- * Proves the Nidus property: evidence cannot exist without a run.
- *   - a verdict is emitted ONLY after the command runs (no run => no pass).
- *   - the output hash is server-side and derived from the actual run.
- *   - isMachineArtifact rejects prose and accepts captured machine shapes.
- *
- * Run: node tests/run-evidence-gate.test.mjs
- */
-import { resolve, dirname, join } from 'node:path';
-import { fileURLToPath, pathToFileURL } from 'node:url';
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const LIB = pathToFileURL(join(resolve(__dirname, '..'), 'hooks', 'lib', 'run-evidence-gate.mjs')).href;
-const { runEvidenceGate, isMachineArtifact, hashOutput, EVIDENCE_KIND } = await import(LIB);
-
-const failures = [];
-function assert(name, cond, detail = '') {
-  if (!cond) failures.push(`${name}${detail ? `: ${detail}` : ''}`);
-  else process.stdout.write(`  ok  ${name}\n`);
-}
-
-// --- 1. A passing run produces verdict:'pass' WITH a hash, only after running.
-let ranCount = 0;
-const passArt = runEvidenceGate(
-  { command: 'noop', args: ['x'], label: 'unit-pass' },
-  () => { ranCount += 1; return { status: 0, stdout: 'ok', stderr: '' }; },
-);
-assert('runner invoked exactly once', ranCount === 1, String(ranCount));
-assert('pass: kind stamped', passArt.kind === EVIDENCE_KIND);
-assert('pass: ran=true', passArt.ran === true);
-assert('pass: verdict=pass', passArt.verdict === 'pass', passArt.verdict);
-assert('pass: output_sha256 present', typeof passArt.output_sha256 === 'string' && passArt.output_sha256.length === 64, passArt.output_sha256);
-
-// --- 2. A failing run -> verdict:'fail', never silently pass.
-const failArt = runEvidenceGate(
-  { command: 'noop' },
-  () => ({ status: 7, stdout: '', stderr: 'boom' }),
-);
-assert('fail: verdict=fail', failArt.verdict === 'fail', failArt.verdict);
-assert('fail: exit_code preserved', failArt.exit_code === 7, String(failArt.exit_code));
-
-// --- 3. NO RUN -> NO verdict (fail-closed). Invalid spec.
-const noCmd = runEvidenceGate({});
-assert('no-command: ran=false', noCmd.ran === false);
-assert('no-command: verdict=error (never pass)', noCmd.verdict === 'error', noCmd.verdict);
-
-// --- 4. Runner that yields no exit status -> cannot have run -> error.
-const noStatus = runEvidenceGate({ command: 'noop' }, () => ({ status: null, stdout: '', stderr: '' }));
-assert('no-status: ran=false', noStatus.ran === false);
-assert('no-status: verdict=error', noStatus.verdict === 'error', noStatus.verdict);
-
-// --- 5. Runner throws -> fail-closed error, never pass.
-const threw = runEvidenceGate({ command: 'noop' }, () => { throw new Error('spawn failed'); });
-assert('runner-throws: ran=false', threw.ran === false);
-assert('runner-throws: verdict=error', threw.verdict === 'error', threw.verdict);
-
-// --- 6. Hash is deterministic + derived from the captured combined output.
-assert('hash deterministic', hashOutput('abc') === hashOutput('abc'));
-assert('hash differs by content', hashOutput('abc') !== hashOutput('abd'));
-
-// --- 7. isMachineArtifact discrimination.
-assert('reject bare prose "HTTP 200"', isMachineArtifact('HTTP 200') === false);
-assert('reject bare prose "works"', isMachineArtifact('works') === false);
-assert('reject "107 nodes"', isMachineArtifact('107 nodes') === false);
-assert('accept fused artifact', isMachineArtifact(passArt) === true);
-assert('accept {exit_code}', isMachineArtifact({ exit_code: 0 }) === true);
-assert('accept {http_status}', isMachineArtifact({ http_status: 200 }) === true);
-assert('accept {rowcount}', isMachineArtifact({ rowcount: 12 }) === true);
-assert('accept {passed,total}', isMachineArtifact({ passed: 5, total: 5 }) === true);
-assert('accept JSON string of machine shape', isMachineArtifact('{"exit_code":0}') === true);
-assert('reject plain object', isMachineArtifact({ note: 'done' }) === false);
-assert('reject null', isMachineArtifact(null) === false);
-
-if (failures.length > 0) {
-  console.error('\nrun-evidence-gate tests — FAIL\n');
-  for (const f of failures) console.error(`  - ${f}`);
-  process.exit(1);
-}
-console.log('\nrun-evidence-gate tests — PASS');
+#!/usr/bin/env node
+/**
+ * Truth Gate 3.0 — fused run-evidence-gate primitive tests.
+ *
+ * Proves the Nidus property: evidence cannot exist without a run.
+ *   - a verdict is emitted ONLY after the command runs (no run => no pass).
+ *   - the output hash is server-side and derived from the actual run.
+ *   - isMachineArtifact rejects prose and accepts captured machine shapes.
+ *
+ * Run: node tests/run-evidence-gate.test.mjs
+ */
+import { resolve, dirname, join } from 'node:path';
+import { fileURLToPath, pathToFileURL } from 'node:url';
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const LIB = pathToFileURL(join(resolve(__dirname, '..'), 'hooks', 'lib', 'run-evidence-gate.mjs')).href;
+const { runEvidenceGate, isMachineArtifact, hashOutput, EVIDENCE_KIND } = await import(LIB);
+
+const failures = [];
+function assert(name, cond, detail = '') {
+  if (!cond) failures.push(`${name}${detail ? `: ${detail}` : ''}`);
+  else process.stdout.write(`  ok  ${name}\n`);
+}
+
+// --- 1. A passing run produces verdict:'pass' WITH a hash, only after running.
+let ranCount = 0;
+const passArt = runEvidenceGate(
+  { command: 'noop', args: ['x'], label: 'unit-pass' },
+  () => { ranCount += 1; return { status: 0, stdout: 'ok', stderr: '' }; },
+);
+assert('runner invoked exactly once', ranCount === 1, String(ranCount));
+assert('pass: kind stamped', passArt.kind === EVIDENCE_KIND);
+assert('pass: ran=true', passArt.ran === true);
+assert('pass: verdict=pass', passArt.verdict === 'pass', passArt.verdict);
+assert('pass: output_sha256 present', typeof passArt.output_sha256 === 'string' && passArt.output_sha256.length === 64, passArt.output_sha256);
+
+// --- 2. A failing run -> verdict:'fail', never silently pass.
+const failArt = runEvidenceGate(
+  { command: 'noop' },
+  () => ({ status: 7, stdout: '', stderr: 'boom' }),
+);
+assert('fail: verdict=fail', failArt.verdict === 'fail', failArt.verdict);
+assert('fail: exit_code preserved', failArt.exit_code === 7, String(failArt.exit_code));
+
+// --- 3. NO RUN -> NO verdict (fail-closed). Invalid spec.
+const noCmd = runEvidenceGate({});
+assert('no-command: ran=false', noCmd.ran === false);
+assert('no-command: verdict=error (never pass)', noCmd.verdict === 'error', noCmd.verdict);
+
+// --- 4. Runner that yields no exit status -> cannot have run -> error.
+const noStatus = runEvidenceGate({ command: 'noop' }, () => ({ status: null, stdout: '', stderr: '' }));
+assert('no-status: ran=false', noStatus.ran === false);
+assert('no-status: verdict=error', noStatus.verdict === 'error', noStatus.verdict);
+
+// --- 5. Runner throws -> fail-closed error, never pass.
+const threw = runEvidenceGate({ command: 'noop' }, () => { throw new Error('spawn failed'); });
+assert('runner-throws: ran=false', threw.ran === false);
+assert('runner-throws: verdict=error', threw.verdict === 'error', threw.verdict);
+
+// --- 6. Hash is deterministic + derived from the captured combined output.
+assert('hash deterministic', hashOutput('abc') === hashOutput('abc'));
+assert('hash differs by content', hashOutput('abc') !== hashOutput('abd'));
+
+// --- 7. isMachineArtifact discrimination.
+assert('reject bare prose "HTTP 200"', isMachineArtifact('HTTP 200') === false);
+assert('reject bare prose "works"', isMachineArtifact('works') === false);
+assert('reject "107 nodes"', isMachineArtifact('107 nodes') === false);
+assert('accept fused artifact', isMachineArtifact(passArt) === true);
+assert('accept {exit_code}', isMachineArtifact({ exit_code: 0 }) === true);
+assert('accept {http_status}', isMachineArtifact({ http_status: 200 }) === true);
+assert('accept {rowcount}', isMachineArtifact({ rowcount: 12 }) === true);
+assert('accept {passed,total}', isMachineArtifact({ passed: 5, total: 5 }) === true);
+assert('accept JSON string of machine shape', isMachineArtifact('{"exit_code":0}') === true);
+assert('reject plain object', isMachineArtifact({ note: 'done' }) === false);
+assert('reject null', isMachineArtifact(null) === false);
+
+if (failures.length > 0) {
+  console.error('\nrun-evidence-gate tests — FAIL\n');
+  for (const f of failures) console.error(`  - ${f}`);
+  process.exit(1);
+}
+console.log('\nrun-evidence-gate tests — PASS');

package/tests/skill-test-matrix.test.mjs

@@ -1,66 +1,66 @@
-#!/usr/bin/env node
-import assert from 'node:assert/strict';
-import { readdirSync, readFileSync } from 'node:fs';
-import { join, resolve } from 'node:path';
-import { fileURLToPath } from 'node:url';
-import { dirname } from 'node:path';
-
-import { loadAllManifests } from '../scripts/lib/manifest-schema.mjs';
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const root = resolve(__dirname, '..');
-const testsDir = join(root, 'skills', 'tests');
-const matrixPath = join(testsDir, 'MATRIX.md');
-const readmePath = join(testsDir, 'README.md');
-
-const manifestFiles = readdirSync(testsDir).filter((name) => name.endsWith('.test.json')).sort();
-const manifests = loadAllManifests(testsDir);
-const valid = manifests.filter((entry) => entry.ok && entry.manifest);
-const matrix = readFileSync(matrixPath, 'utf8');
-const readme = readFileSync(readmePath, 'utf8');
-
-assert.equal(valid.length, manifestFiles.length, 'all manifest files must validate');
-assert.match(matrix, new RegExp(`Current coverage: ${manifestFiles.length} manifests for ${manifestFiles.length} skill directories`));
-assert.match(readme, new RegExp(`There are currently ${manifestFiles.length} manifests for ${manifestFiles.length} skill directories`));
-
-const rows = new Map();
-for (const line of matrix.split(/\r?\n/)) {
-  const m = line.match(/^\| `([^`]+)` \| `([^`]+)` \| ([^|]+) \| ([^|]+) \| ([^|]+) \|$/);
-  if (!m) continue;
-  rows.set(m[1], {
-    manifest: m[2],
-    promptClass: m[3].trim(),
-    assertions: m[4].replace(/`/g, '').trim(),
-    acceptanceDepth: m[5].trim(),
-  });
-}
-
-assert.equal(rows.size, manifestFiles.length, 'MATRIX.md must list every manifest exactly once');
-
-for (const entry of valid) {
-  const manifest = entry.manifest;
-  const filename = entry.file.replace(/^skills\/tests\//, '');
-  const row = rows.get(manifest.skill);
-  assert.ok(row, `${manifest.skill} missing from MATRIX.md`);
-  assert.equal(row.manifest, filename, `${manifest.skill} matrix filename mismatch`);
-
-  for (const key of Object.keys(manifest.assertions || {}).sort()) {
-    assert.match(row.assertions, new RegExp(`(^|, )${key}(,|$)`), `${manifest.skill} matrix must mention assertion ${key}`);
-  }
-
-  const hasAcceptance = manifest.acceptance && Object.keys(manifest.acceptance).length > 0;
-  if (hasAcceptance) {
-    assert.notEqual(row.acceptanceDepth, 'Basic manifest', `${manifest.skill} has acceptance checks but matrix says Basic manifest`);
-  } else {
-    assert.equal(row.acceptanceDepth, 'Basic manifest', `${manifest.skill} has no acceptance block and should be documented as Basic manifest`);
-  }
-}
-
-const shallow = valid
-  .map((entry) => entry.manifest)
-  .filter((manifest) => !manifest.acceptance || Object.keys(manifest.acceptance).length === 0)
-  .map((manifest) => manifest.skill);
-assert.deepEqual(shallow, [], `all skill manifests must include acceptance checks; missing: ${shallow.join(', ')}`);
-assert.match(matrix, /All manifests now include a deeper acceptance block/);
-
-console.log('skill test matrix tests — PASS');
+#!/usr/bin/env node
+import assert from 'node:assert/strict';
+import { readdirSync, readFileSync } from 'node:fs';
+import { join, resolve } from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { dirname } from 'node:path';
+
+import { loadAllManifests } from '../scripts/lib/manifest-schema.mjs';
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const root = resolve(__dirname, '..');
+const testsDir = join(root, 'skills', 'tests');
+const matrixPath = join(testsDir, 'MATRIX.md');
+const readmePath = join(testsDir, 'README.md');
+
+const manifestFiles = readdirSync(testsDir).filter((name) => name.endsWith('.test.json')).sort();
+const manifests = loadAllManifests(testsDir);
+const valid = manifests.filter((entry) => entry.ok && entry.manifest);
+const matrix = readFileSync(matrixPath, 'utf8');
+const readme = readFileSync(readmePath, 'utf8');
+
+assert.equal(valid.length, manifestFiles.length, 'all manifest files must validate');
+assert.match(matrix, new RegExp(`Current coverage: ${manifestFiles.length} manifests for ${manifestFiles.length} skill directories`));
+assert.match(readme, new RegExp(`There are currently ${manifestFiles.length} manifests for ${manifestFiles.length} skill directories`));
+
+const rows = new Map();
+for (const line of matrix.split(/\r?\n/)) {
+  const m = line.match(/^\| `([^`]+)` \| `([^`]+)` \| ([^|]+) \| ([^|]+) \| ([^|]+) \|$/);
+  if (!m) continue;
+  rows.set(m[1], {
+    manifest: m[2],
+    promptClass: m[3].trim(),
+    assertions: m[4].replace(/`/g, '').trim(),
+    acceptanceDepth: m[5].trim(),
+  });
+}
+
+assert.equal(rows.size, manifestFiles.length, 'MATRIX.md must list every manifest exactly once');
+
+for (const entry of valid) {
+  const manifest = entry.manifest;
+  const filename = entry.file.replace(/^skills\/tests\//, '');
+  const row = rows.get(manifest.skill);
+  assert.ok(row, `${manifest.skill} missing from MATRIX.md`);
+  assert.equal(row.manifest, filename, `${manifest.skill} matrix filename mismatch`);
+
+  for (const key of Object.keys(manifest.assertions || {}).sort()) {
+    assert.match(row.assertions, new RegExp(`(^|, )${key}(,|$)`), `${manifest.skill} matrix must mention assertion ${key}`);
+  }
+
+  const hasAcceptance = manifest.acceptance && Object.keys(manifest.acceptance).length > 0;
+  if (hasAcceptance) {
+    assert.notEqual(row.acceptanceDepth, 'Basic manifest', `${manifest.skill} has acceptance checks but matrix says Basic manifest`);
+  } else {
+    assert.equal(row.acceptanceDepth, 'Basic manifest', `${manifest.skill} has no acceptance block and should be documented as Basic manifest`);
+  }
+}
+
+const shallow = valid
+  .map((entry) => entry.manifest)
+  .filter((manifest) => !manifest.acceptance || Object.keys(manifest.acceptance).length === 0)
+  .map((manifest) => manifest.skill);
+assert.deepEqual(shallow, [], `all skill manifests must include acceptance checks; missing: ${shallow.join(', ')}`);
+assert.match(matrix, /All manifests now include a deeper acceptance block/);
+
+console.log('skill test matrix tests — PASS');

package/tests/validate-skills.js

@@ -1,27 +1,27 @@
-#!/usr/bin/env node
-/**
- * Compatibility wrapper for the current rdc-skills validation gate.
- *
- * The older validator required every skill to use the same "When to Use" /
- * "Procedure" section shape. The canonical gate is now scripts/self-test.mjs,
- * which understands the shipped skill variants, guide checks, hook behavior,
- * plugin metadata, and strict warning policy.
- */
-
-const { spawnSync } = require("node:child_process");
-const { join, dirname } = require("node:path");
-
-const repoRoot = dirname(dirname(__filename));
-const selfTest = join(repoRoot, "scripts", "self-test.mjs");
-
-const result = spawnSync(process.execPath, [selfTest, "--strict"], {
-  cwd: repoRoot,
-  stdio: "inherit",
-});
-
-if (result.error) {
-  console.error(result.error.message);
-  process.exit(2);
-}
-
-process.exit(result.status ?? 1);
+#!/usr/bin/env node
+/**
+ * Compatibility wrapper for the current rdc-skills validation gate.
+ *
+ * The older validator required every skill to use the same "When to Use" /
+ * "Procedure" section shape. The canonical gate is now scripts/self-test.mjs,
+ * which understands the shipped skill variants, guide checks, hook behavior,
+ * plugin metadata, and strict warning policy.
+ */
+
+const { spawnSync } = require("node:child_process");
+const { join, dirname } = require("node:path");
+
+const repoRoot = dirname(dirname(__filename));
+const selfTest = join(repoRoot, "scripts", "self-test.mjs");
+
+const result = spawnSync(process.execPath, [selfTest, "--strict"], {
+  cwd: repoRoot,
+  stdio: "inherit",
+});
+
+if (result.error) {
+  console.error(result.error.message);
+  process.exit(2);
+}
+
+process.exit(result.status ?? 1);