@ikon85/agent-workflow-kit 0.16.3 → 0.16.4

package/README.md

@@ -332,6 +332,10 @@ still reference. Flags: `--force` (overwrite pre-existing files on `init`),
 
 ## Release notes
 
+### 0.16.4
+
+- Metadata-only release.
+
 ### 0.16.3
 
 - changed: `.agents/skills/grill-me/SKILL.md`

package/agent-workflow-kit.package.json

@@ -1,5 +1,5 @@
 {
-  "kitVersion": "0.16.3",
+  "kitVersion": "0.16.4",
   "files": [
     {
       "path": ".agents/skills/ask-matt/SKILL.md",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ikon85/agent-workflow-kit",
-  "version": "0.16.3",
+  "version": "0.16.4",
   "description": "Portable AI-agent workflow skills (plan → execute → land → learn) for Claude Code & Codex — grilling, TDD, diagnosis, two-axis code review, cross-model Codex review, design & domain-modeling, plus a skill router (ask-matt). npx init/update/diff/uninstall.",
   "type": "module",
   "bin": {

package/scripts/build-kit.test.mjs

@@ -9,6 +9,9 @@ import { execFileSync } from 'node:child_process';
 import { buildKit } from './build-kit.mjs';
 
 const REPO = join(dirname(fileURLToPath(import.meta.url)), '..');
+const PUBLIC_CENSUS_UNIT = JSON.parse(await readFile(
+  join(REPO, 'test/fixtures/census-consumers/public-unit.json'), 'utf8',
+)).paths;
 async function withBuild(fn) {
   const dist = await mkdtemp(join(tmpdir(), 'awkit-build-'));
   try { return await fn(dist, await buildKit({ repoRoot: REPO, distDir: dist })); }
@@ -43,6 +46,19 @@ test('current build contains post-tag public files and repository metadata', asy
   });
 });
 
+test('current build contains the complete dual-surface census consumer unit', async () => {
+  await withBuild(async (dist, report) => {
+    const manifest = JSON.parse(await readFile(join(dist, 'agent-workflow-kit.package.json'), 'utf8'));
+    const paths = new Set(manifest.files.map(({ path }) => path));
+    assert.deepEqual(PUBLIC_CENSUS_UNIT.filter((path) => !paths.has(path)), []);
+    assert.equal(report.fileCount, manifest.files.length);
+    assert.equal(
+      await readFile(join(dist, '.claude/skills/census-update/SKILL.md'), 'utf8'),
+      await readFile(join(dist, '.agents/skills/census-update/SKILL.md'), 'utf8'),
+    );
+  });
+});
+
 test('current build is self-contained and never reaches into a consumer checkout', async () => {
   const source = await readFile(join(REPO, 'scripts/build-kit.mjs'), 'utf8');
   assert.doesNotMatch(source, /testreporter|tools\/agent-workflow-kit/);
@@ -67,6 +83,9 @@ test('npm pack keeps product files but excludes runtime residue', async () => {
     assert.ok(files.includes('scripts/kit-update-pr.mjs'));
     assert.ok(files.includes('.claude/hooks/drift-guard.py'));
     assert.ok(files.includes('.claude/skills/tdd/SKILL.md'));
+    for (const path of PUBLIC_CENSUS_UNIT) {
+      assert.ok(files.includes(path), `pack missing ${path}`);
+    }
     assert.ok(files.every((path) => !path.startsWith('.claude/logs/')));
     assert.ok(files.every((path) => !path.includes('__pycache__') && !path.endsWith('.pyc')));
     const pkg = JSON.parse(execFileSync('node', ['-p', 'JSON.stringify(require("./package.json"))'], {

package/scripts/census-contract.test.mjs

@@ -0,0 +1,227 @@
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import { execFile } from 'node:child_process';
+import { promisify } from 'node:util';
+import { cp, mkdtemp, readFile, rm, writeFile } from 'node:fs/promises';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+import { fileURLToPath, pathToFileURL } from 'node:url';
+import {
+  CENSUS_BUILDER_VERSION,
+  activateCensus,
+  diffCensus,
+  scanCensus,
+} from './census/index.mjs';
+
+const exec = promisify(execFile);
+const FIXTURES = new URL('../test/fixtures/census-consumers/', import.meta.url);
+const REPO = fileURLToPath(new URL('../', import.meta.url));
+
+async function consumer(name, tracked = ['.']) {
+  const root = await mkdtemp(join(tmpdir(), `awk-census-${name}-`));
+  await cp(new URL(`${name}/`, FIXTURES), root, { recursive: true });
+  await exec('git', ['init', '--quiet'], { cwd: root });
+  await exec('git', ['add', ...tracked], { cwd: root });
+  return root;
+}
+
+async function cleanup(root) {
+  await rm(root, { recursive: true, force: true });
+}
+
+async function json(path) {
+  return JSON.parse(await readFile(path, 'utf8'));
+}
+
+async function backstopStatus(root) {
+  const { stdout } = await exec('python3', [
+    join(REPO, '.claude/hooks/drift-guard.py'), '--census-status',
+  ], { cwd: root });
+  return JSON.parse(stdout);
+}
+
+test('brownfield reports real X of Y surfaces separately from behaviors', async () => {
+  const root = await consumer('brownfield');
+  try {
+    const profile = await json(join(root, '.census/profile.json'));
+    const census = await scanCensus({
+      repoRoot: root,
+      enabled: profile.enabled,
+      hasActive: true,
+      behaviorFamilies: profile.decisions.map(({ family, status }) => ({ name: family, status })),
+    });
+    const covered = census.families.surfaces.filter(({ status }) => status === 'abgedeckt').length;
+
+    assert.equal(census.state, 'current');
+    assert.equal(`${covered} of ${census.families.surfaces.length}`, '3 of 3');
+    assert.deepEqual(census.families.behaviors, [
+      { name: 'authentication', status: 'abgedeckt', type: 'behavior' },
+      { name: 'email-delivery', status: 'nicht relevant', type: 'behavior' },
+    ]);
+    assert.ok(census.families.surfaces.every(({ type }) => type === 'surface'));
+  } finally { await cleanup(root); }
+});
+
+test('greenfield without an active snapshot remains honest bootstrap', async () => {
+  const root = await consumer('greenfield');
+  try {
+    const census = await scanCensus({ repoRoot: root, enabled: true, hasActive: false });
+    assert.equal(census.state, 'bootstrap');
+    assert.ok(census.denominator.length > 0, 'bootstrap still reports discovered facts');
+  } finally { await cleanup(root); }
+});
+
+test('unknown pattern stays open until its repository-local scanner test passes', async () => {
+  const root = await consumer('unknown-pattern', ['package.json', 'src', '.census']);
+  try {
+    const activePath = join(root, '.census/active.json');
+    const profilePath = join(root, '.census/profile.json');
+    const surface = 'services/payments/src';
+    const scannerRecord = {
+      surface,
+      module: 'scripts/census-local/scan-services.mjs',
+      export: 'scanServices',
+      test: 'test/census-services.test.mjs',
+    };
+
+    const untracked = await scanCensus({ repoRoot: root, enabled: true, hasActive: true });
+    assert.equal(untracked.state, 'refresh_required');
+    assert.deepEqual(diffCensus(untracked, untracked).open, [surface]);
+
+    // Establish real active history before the repository grows the unknown surface.
+    await rm(join(root, 'services'), { recursive: true });
+    await rm(join(root, 'scripts'), { recursive: true });
+    await rm(join(root, 'test'), { recursive: true });
+    const known = await scanCensus({ repoRoot: root, enabled: true, hasActive: true });
+    await writeFile(activePath, `${JSON.stringify({
+      ...known,
+      profileReport: { decisions: [], localScanners: [], overrides: [] },
+    })}\n`);
+
+    await cp(new URL('unknown-pattern/services/', FIXTURES), join(root, 'services'), { recursive: true });
+    await exec('git', ['add', 'services'], { cwd: root });
+    const trackedOnly = await backstopStatus(root);
+    assert.equal(trackedOnly.state, 'refresh_required');
+    assert.ok(trackedOnly.reasons.includes('topology'));
+
+    const profile = await json(profilePath);
+    profile.localScanners.push(scannerRecord);
+    await writeFile(profilePath, `${JSON.stringify(profile)}\n`);
+    const missingProof = await backstopStatus(root);
+    assert.equal(missingProof.state, 'refresh_required');
+    assert.ok(missingProof.reasons.includes(`proof:${surface}`));
+
+    await cp(new URL('unknown-pattern/scripts/', FIXTURES), join(root, 'scripts'), { recursive: true });
+    await cp(new URL('unknown-pattern/test/', FIXTURES), join(root, 'test'), { recursive: true });
+    await writeFile(
+      join(root, scannerRecord.test),
+      "import { test } from 'node:test'; test('failing local proof', () => { throw new Error('no'); });\n",
+    );
+    await exec('git', ['add', 'scripts', 'test', '.census/profile.json'], { cwd: root });
+    const failedProof = await backstopStatus(root);
+    assert.equal(failedProof.state, 'refresh_required');
+    assert.ok(failedProof.reasons.includes(`proof:${surface}`));
+
+    await cp(
+      new URL('unknown-pattern/test/census-services.test.mjs', FIXTURES),
+      join(root, scannerRecord.test),
+    );
+    const candidate = await scanCensus({ repoRoot: root, enabled: true, hasActive: true });
+    assert.equal(candidate.state, 'current');
+    await activateCensus({
+      activePath,
+      candidate: {
+        ...candidate,
+        profileReport: { decisions: [], localScanners: [scannerRecord], overrides: [] },
+      },
+      verify: async () => {
+        await exec('node', ['--test', scannerRecord.test], { cwd: root });
+        const scanner = await import(`${pathToFileURL(join(root, scannerRecord.module)).href}?proof=${Date.now()}`);
+        return scanner[scannerRecord.export]().includes(surface);
+      },
+    });
+
+    const proven = await backstopStatus(root);
+    assert.equal(proven.state, 'current');
+    assert.deepEqual(proven.reasons, []);
+    const active = await json(activePath);
+    assert.equal(active.profileReport.localScanners[0].test, scannerRecord.test);
+  } finally { await cleanup(root); }
+});
+
+test('scan and compact delta do not overwrite a consumer-local modification', async () => {
+  const root = await consumer('local-modified');
+  try {
+    const sourcePath = join(root, 'src/index.mjs');
+    const activePath = join(root, '.census/active.json');
+    const sourceBefore = await readFile(sourcePath);
+    const activeBefore = await readFile(activePath);
+    const previous = { denominator: [], families: { surfaces: [], behaviors: [] } };
+    const current = await scanCensus({ repoRoot: root, enabled: true, hasActive: true });
+    const delta = diffCensus(previous, current);
+
+    assert.deepEqual(Object.keys(delta), ['added', 'changed', 'open', 'removed']);
+    assert.ok(JSON.stringify(delta).length < JSON.stringify(current).length);
+    assert.deepEqual(await readFile(sourcePath), sourceBefore);
+    assert.deepEqual(await readFile(activePath), activeBefore);
+  } finally { await cleanup(root); }
+});
+
+test('secret canary is absent from scan, delta, and file reads', async () => {
+  const root = await consumer('secret-canary');
+  try {
+    const reads = [];
+    const scan = await scanCensus({
+      repoRoot: root,
+      readText: async (path) => {
+        reads.push(path);
+        return readFile(path, 'utf8');
+      },
+    });
+    const output = JSON.stringify({ scan, delta: diffCensus({ denominator: [] }, scan) });
+    assert.doesNotMatch(output, /CENSUS_SECRET_CANARY/);
+    assert.ok(reads.every((path) => !path.includes('credentials')));
+  } finally { await cleanup(root); }
+});
+
+test('interrupted activation preserves the active census byte-for-byte', async () => {
+  const root = await consumer('interrupted');
+  try {
+    const activePath = join(root, '.census/active.json');
+    const before = await readFile(activePath);
+    const candidate = await scanCensus({ repoRoot: root, enabled: true, hasActive: true });
+    await assert.rejects(activateCensus({
+      activePath,
+      candidate,
+      verify: async () => true,
+      renameCandidate: async () => { throw new Error('fixture interruption'); },
+    }), (error) => error.state === 'failed');
+    assert.deepEqual(await readFile(activePath), before);
+  } finally { await cleanup(root); }
+});
+
+test('consumer update reports a newer builder without silently mutating its census', async () => {
+  const root = await consumer('consumer-update');
+  try {
+    const activePath = join(root, '.census/active.json');
+    const profilePath = join(root, '.census/profile.json');
+    const fresh = await scanCensus({ repoRoot: root, enabled: true, hasActive: true });
+    await writeFile(activePath, `${JSON.stringify({
+      ...fresh,
+      fingerprints: { ...fresh.fingerprints, builder: 'older-builder-fingerprint' },
+      profileReport: { decisions: [], localScanners: [], overrides: [] },
+    })}\n`);
+    const activeBefore = await readFile(activePath);
+    const profileBefore = await readFile(profilePath);
+    const { stdout } = await exec('python3', [
+      join(REPO, '.claude/hooks/drift-guard.py'), '--census-status',
+    ], { cwd: root });
+    const status = JSON.parse(stdout);
+
+    assert.equal(status.state, 'refresh_required');
+    assert.equal(status.detail, `builder ${CENSUS_BUILDER_VERSION}`);
+    assert.ok(status.reasons.includes('builder'));
+    assert.deepEqual(await readFile(activePath), activeBefore);
+    assert.deepEqual(await readFile(profilePath), profileBefore);
+  } finally { await cleanup(root); }
+});

package/scripts/test_census_forward_contract.py

@@ -0,0 +1,91 @@
+#!/usr/bin/env python3
+"""Forward proof: a clean public checkout builds and serves census consumers."""
+import json
+import shutil
+import subprocess
+import tempfile
+import unittest
+from pathlib import Path
+
+REPO = Path(__file__).resolve().parent.parent
+PUBLIC_UNIT_RELATIVE = Path("test/fixtures/census-consumers/public-unit.json")
+GREENFIELD_RELATIVE = Path("test/fixtures/census-consumers/greenfield")
+
+
+class CensusForwardContract(unittest.TestCase):
+    def setUp(self):
+        self.tmp = Path(tempfile.mkdtemp(prefix="awk-census-forward-"))
+        self.addCleanup(shutil.rmtree, self.tmp, ignore_errors=True)
+        self.public = self.tmp / "public-kit"
+        self.public.mkdir()
+        source_root = REPO
+        self.forbidden_source_root = str(source_root)
+        archive = subprocess.Popen(
+            ["git", "archive", "HEAD"], cwd=source_root, stdout=subprocess.PIPE
+        )
+        unpack = subprocess.run(
+            ["tar", "-x", "-C", str(self.public)], stdin=archive.stdout,
+            capture_output=True, timeout=30,
+        )
+        archive.stdout.close()
+        archive_rc = archive.wait(timeout=30)
+        self.assertEqual((archive_rc, unpack.returncode), (0, 0), unpack.stderr.decode())
+        # From this boundary onward every executable, fixture, and assertion input
+        # is resolved inside the archived public checkout.
+        self.assertTrue((self.public / GREENFIELD_RELATIVE).is_dir())
+        self.assertTrue((self.public / PUBLIC_UNIT_RELATIVE).is_file())
+        build = subprocess.run(
+            ["node", "scripts/build-kit.mjs"], cwd=self.public,
+            capture_output=True, text=True, timeout=120,
+        )
+        self.assertEqual(build.returncode, 0, build.stderr)
+        self.dist = self.public / "dist-kit"
+
+    def test_built_public_checkout_exercises_both_surfaces_and_scanner(self):
+        claude = (self.dist / ".claude/skills/census-update/SKILL.md").read_text()
+        codex = (self.dist / ".agents/skills/census-update/SKILL.md").read_text()
+        self.assertEqual(claude, codex)
+        for public_export in ("scanCensus", "diffCensus", "activateCensus"):
+            self.assertIn(public_export, claude)
+
+        consumer = self.tmp / "consumer"
+        fixture = self.public / GREENFIELD_RELATIVE
+        self.assertTrue(fixture.is_relative_to(self.public))
+        shutil.copytree(fixture, consumer)
+        subprocess.run(["git", "init", "--quiet"], cwd=consumer, check=True)
+        subprocess.run(["git", "add", "."], cwd=consumer, check=True)
+        runner = (
+            "import { scanCensus } from "
+            + json.dumps((self.dist / "scripts/census/index.mjs").as_uri())
+            + "; const result = await scanCensus({repoRoot: process.argv[1], enabled:true});"
+            + "console.log(JSON.stringify({state:result.state,count:result.denominator.length}));"
+        )
+        run = subprocess.run(
+            ["node", "--input-type=module", "--eval", runner, str(consumer)],
+            cwd=self.public, capture_output=True, text=True, timeout=30,
+        )
+        self.assertEqual(run.returncode, 0, run.stderr)
+        self.assertEqual(json.loads(run.stdout), {"state": "bootstrap", "count": 2})
+        self.assertNotIn("testreporter", run.stdout.lower())
+        self.assertNotIn(self.forbidden_source_root, runner)
+        self.assertNotIn(self.forbidden_source_root, run.stdout)
+
+    def test_public_pack_contains_complete_census_unit_without_foreign_reach(self):
+        pack = subprocess.run(
+            ["npm", "pack", "--dry-run", "--json"], cwd=self.public,
+            capture_output=True, text=True, timeout=120,
+        )
+        self.assertEqual(pack.returncode, 0, pack.stderr)
+        paths = {entry["path"] for entry in json.loads(pack.stdout)[0]["files"]}
+        expected = set(json.loads(
+            (self.public / PUBLIC_UNIT_RELATIVE).read_text(encoding="utf-8")
+        )["paths"])
+        self.assertEqual(expected - paths, set())
+        for path in expected:
+            text = (self.public / path).read_text(encoding="utf-8")
+            self.assertNotIn("tools/agent-workflow-kit", text)
+            self.assertNotIn("testreporter", text.lower())
+
+
+if __name__ == "__main__":
+    unittest.main(verbosity=2)

package/scripts/test_dist_kit_smoke.py

@@ -194,6 +194,27 @@ class DistKitValidateGraphSmoke(unittest.TestCase):
                          "language (fields.status.roles / --status-role) instead:\n"
                          + "\n".join(hits))
 
+    def test_shipped_census_runs_for_a_foreign_greenfield_on_both_surfaces(self):
+        consumer = self.tmp / "census-consumer"
+        shutil.copytree(REPO / "test/fixtures/census-consumers/greenfield", consumer)
+        subprocess.run(["git", "init", "--quiet"], cwd=consumer, check=True)
+        subprocess.run(["git", "add", "."], cwd=consumer, check=True)
+        claude = (DIST / ".claude/skills/census-update/SKILL.md").read_text()
+        codex = (DIST / ".agents/skills/census-update/SKILL.md").read_text()
+        self.assertEqual(claude, codex)
+        module = (DIST / "scripts/census/index.mjs").as_uri()
+        runner = (
+            f'import {{ scanCensus }} from {json.dumps(module)};'
+            "const r=await scanCensus({repoRoot:process.argv[1],enabled:true});"
+            "console.log(JSON.stringify({state:r.state,count:r.denominator.length}));"
+        )
+        result = subprocess.run(
+            ["node", "--input-type=module", "--eval", runner, str(consumer)],
+            cwd=DIST, capture_output=True, text=True, timeout=30,
+        )
+        self.assertEqual(result.returncode, 0, result.stderr)
+        self.assertEqual(json.loads(result.stdout), {"state": "bootstrap", "count": 2})
+
 
 if __name__ == "__main__":
     unittest.main(verbosity=2)

package/scripts/test_skill_publish_audit.py

@@ -36,6 +36,9 @@ import unittest
 from pathlib import Path
 
 REPO = Path(__file__).resolve().parent.parent
+PUBLIC_CENSUS_UNIT = json.loads(
+    (REPO / "test/fixtures/census-consumers/public-unit.json").read_text(encoding="utf-8")
+)["paths"]
 PUBLIC_SLUG = "iKon85/agent-workflow-kit"
 CREDIT_FILES = {"LICENSE", "README.md", "PROVENANCE.md", "THIRD-PARTY-NOTICES.md"}
 PRIVATE_SKILLS = {
@@ -295,6 +298,19 @@ class RealDistKitIsClean(unittest.TestCase):
         violations = audit_dir(REPO / "dist-kit")
         self.assertEqual(violations, [], "\n".join(violations[:40]))
 
+    def test_built_dist_kit_contains_publishable_census_without_foreign_reach(self):
+        build = subprocess.run(
+            ["node", str(REPO / "scripts/build-kit.mjs")], cwd=REPO,
+            capture_output=True, text=True, timeout=120,
+        )
+        self.assertEqual(build.returncode, 0, build.stderr)
+        for rel in PUBLIC_CENSUS_UNIT:
+            path = REPO / "dist-kit" / rel
+            self.assertTrue(path.is_file(), f"missing published census resource: {rel}")
+            text = path.read_text(encoding="utf-8")
+            self.assertNotIn("tools/agent-workflow-kit", text)
+            self.assertNotIn("testreporter", text.lower())
+
 
 if __name__ == "__main__":
     unittest.main(verbosity=2)