@givolt/i3-composer-agent 1.0.1 → 2.1.0

package/rebrand/tests/pipeline.test.cjs

@@ -1,185 +0,0 @@
-'use strict';
-
-const test = require('node:test');
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-const { spawnSync } = require('node:child_process');
-const root = path.resolve(__dirname, '../..');
-const baselineVersion = require('../upstream.json').upstreamVersion;
-
-function run(directory, executable, args) {
-  return spawnSync(executable, args, { cwd: directory, encoding: 'utf8' });
-}
-function git(directory, args) {
-  const result = run(directory, 'git', args);
-  assert.equal(result.status, 0, result.stderr);
-  return result.stdout.trim();
-}
-function node(directory, file, args = []) { return run(directory, process.execPath, [file, ...args]); }
-function fixture(t, branch = 'release/test-candidate') {
-  const directory = fs.mkdtempSync(path.join(os.tmpdir(), 'i3-agent-replay-'));
-  t.after(() => fs.rmSync(directory, { recursive: true, force: true }));
-  for (const name of ['skills', 'references', 'agents', 'commands', '.claude', '.gemini', '.claude-plugin', '.codex-plugin', '.agents', 'hooks', 'docs', 'scripts', 'evals', 'rebrand', 'LICENSE', 'plugin.json', 'README.md']) {
-    fs.cpSync(path.join(root, name), path.join(directory, name), { recursive: true, filter: file => !file.includes(`${path.sep}results${path.sep}`) });
-  }
-  // Model an upstream source checkout, including when this suite runs in a packed branded bundle.
-  for (const name of ['plugin.json', '.claude-plugin/plugin.json', '.codex-plugin/plugin.json']) {
-    const file = path.join(directory, name);
-    const data = JSON.parse(fs.readFileSync(file, 'utf8'));
-    data.name = 'agent-skills';
-    data.version = baselineVersion;
-    fs.writeFileSync(file, JSON.stringify(data, null, 2) + '\n');
-  }
-  git(directory, ['init', '--initial-branch', branch]);
-  git(directory, ['config', 'user.name', 'Pipeline Test']);
-  git(directory, ['config', 'user.email', 'pipeline@example.test']);
-  git(directory, ['add', '.']);
-  git(directory, ['commit', '-m', 'upstream fixture']);
-  return directory;
-}
-function record(directory) {
-  const result = node(directory, 'rebrand/record-upstream.cjs', ['HEAD']);
-  assert.equal(result.status, 0, result.stderr);
-}
-function apply(directory) {
-  const result = node(directory, 'rebrand/apply.cjs', ['--apply']);
-  assert.equal(result.status, 0, result.stderr);
-}
-
-test('a source checkout generates a complete, repeatable branded distribution', t => {
-  const directory = fixture(t);
-  record(directory);
-  apply(directory);
-  const pkg = JSON.parse(fs.readFileSync(path.join(directory, 'package.json'), 'utf8'));
-  assert.equal(pkg.name, '@givolt/i3-composer-agent');
-  const brand = JSON.parse(fs.readFileSync(path.join(directory, 'rebrand/brand.json'), 'utf8'));
-  assert.equal(pkg.version, brand.releaseVersion);
-  assert.equal(fs.existsSync(path.join(directory, 'bin/i3-composer-agent.cjs')), true);
-  assert.equal(node(directory, 'scripts/validate-brand.cjs').status, 0);
-  const check = node(directory, 'rebrand/apply.cjs', ['--check']);
-  assert.equal(check.status, 0, check.stdout + check.stderr);
-});
-
-test('brand fixes advance independently while preserving upstream provenance', t => {
-  const directory = fixture(t);
-  record(directory);
-  const upstreamFile = path.join(directory, 'rebrand/upstream.json');
-  const originalUpstream = fs.readFileSync(upstreamFile, 'utf8');
-  const upstream = JSON.parse(originalUpstream);
-  const brandFile = path.join(directory, 'rebrand/brand.json');
-  const brand = JSON.parse(fs.readFileSync(brandFile, 'utf8'));
-  const releaseVersion = `${Number(upstream.upstreamVersion.split('.')[0]) + 1}.0.1`;
-  brand.releaseVersion = releaseVersion;
-  fs.writeFileSync(brandFile, JSON.stringify(brand, null, 2) + '\n');
-  apply(directory);
-  const packageFile = path.join(directory, 'package.json');
-  const originalPackage = fs.readFileSync(packageFile, 'utf8');
-  const pkg = JSON.parse(originalPackage);
-  assert.equal(pkg.version, releaseVersion);
-  assert.equal(pkg.i3ComposerAgent.upstreamVersion, upstream.upstreamVersion);
-  assert.equal(pkg.i3ComposerAgent.upstreamCommit, upstream.upstreamCommit);
-  assert.equal(fs.readFileSync(upstreamFile, 'utf8'), originalUpstream);
-  const releaseLabel = `i3 Composer Agent v${releaseVersion}`;
-  const upstreamLabel = `Based on agent-skills v${upstream.upstreamVersion}`;
-  for (const output of [fs.readFileSync(path.join(directory, 'README.md'), 'utf8'), node(directory, 'bin/i3-composer-agent.cjs', ['--help']).stdout]) {
-    assert.ok(output.includes(releaseLabel), output);
-    assert.ok(output.includes(upstreamLabel), output);
-  }
-  assert.equal(node(directory, 'bin/i3-composer-agent.cjs', ['--version']).stdout.trim(), releaseVersion);
-  const dryRun = node(directory, 'bin/i3-composer-agent.cjs', ['--extract', '--dry-run', '--dir', directory + '-prepared']);
-  assert.equal(dryRun.status, 0, dryRun.stderr);
-  assert.equal(JSON.parse(dryRun.stdout).upstreamVersion, upstream.upstreamVersion);
-  assert.equal(node(directory, 'scripts/validate-brand.cjs').status, 0);
-  pkg.i3ComposerAgent.upstreamVersion = releaseVersion;
-  fs.writeFileSync(packageFile, JSON.stringify(pkg));
-  assert.notEqual(node(directory, 'scripts/validate-brand.cjs').status, 0);
-  fs.writeFileSync(packageFile, originalPackage);
-  assert.equal(node(directory, 'rebrand/apply.cjs', ['--check']).status, 0);
-});
-
-test('writing a brand transform on the protected source branch changes nothing', t => {
-  const directory = fixture(t, 'agent/source');
-  const before = fs.readFileSync(path.join(directory, 'plugin.json'));
-  const result = node(directory, 'rebrand/apply.cjs', ['--apply']);
-  assert.notEqual(result.status, 0);
-  assert.match(result.stderr, /source branch/i);
-  assert.deepEqual(fs.readFileSync(path.join(directory, 'plugin.json')), before);
-  assert.equal(fs.existsSync(path.join(directory, 'package.json')), false);
-});
-
-test('a reviewed upstream catalog can grow without losing its new skill or command', t => {
-  const directory = fixture(t);
-  const skill = path.join(directory, 'skills/upstream-new-skill');
-  fs.mkdirSync(skill);
-  fs.writeFileSync(path.join(skill, 'SKILL.md'), '---\nname: upstream-new-skill\ndescription: New upstream capability.\n---\n# Upstream capability\n');
-  fs.writeFileSync(path.join(directory, '.claude/commands/upstream-new-command.md'), '---\ndescription: Invoke the new upstream capability.\n---\nUse agent-skills:upstream-new-skill.\n');
-  git(directory, ['add', 'skills', '.claude/commands']);
-  git(directory, ['commit', '-m', 'new upstream capability']);
-  record(directory);
-  apply(directory);
-  const result = node(directory, 'scripts/validate-brand.cjs');
-  assert.equal(result.status, 0, result.stderr);
-  const description = JSON.parse(fs.readFileSync(path.join(directory, '.codex-plugin/plugin.json'), 'utf8')).interface.longDescription;
-  assert.match(description, /26 engineering skills/);
-  const readme = fs.readFileSync(path.join(directory, 'README.md'), 'utf8');
-  assert.match(readme, /upstream-new-skill/);
-  assert.match(readme, /upstream-new-command/);
-});
-
-test('missing files and altered skill identifiers fail preservation validation', t => {
-  const directory = fixture(t);
-  record(directory);
-  apply(directory);
-  const file = path.join(directory, 'skills/using-agent-skills/SKILL.md');
-  const original = fs.readFileSync(file, 'utf8');
-  fs.writeFileSync(file, original.replace('name: using-agent-skills', 'name: renamed-meta-skill'));
-  assert.notEqual(node(directory, 'scripts/validate-brand.cjs').status, 0);
-  fs.writeFileSync(file, original);
-  fs.rmSync(file);
-  assert.notEqual(node(directory, 'scripts/validate-brand.cjs').status, 0);
-});
-
-test('checkout line endings do not masquerade as an upstream skill change', t => {
-  const directory = fixture(t);
-  record(directory);
-  const file = path.join(directory, 'skills/using-agent-skills/SKILL.md');
-  fs.writeFileSync(file, fs.readFileSync(file, 'utf8').replace(/\r?\n/g, '\r\n'));
-  const script = path.join(directory, 'scripts/validate-versions.js');
-  fs.writeFileSync(script, fs.readFileSync(script, 'utf8').replace(/\r?\n/g, '\r\n'));
-  apply(directory);
-  assert.equal(node(directory, 'scripts/validate-brand.cjs').status, 0);
-});
-
-test('a new upstream package manifest requires an explicit distribution adaptation', t => {
-  const directory = fixture(t);
-  record(directory);
-  fs.writeFileSync(path.join(directory, 'package.json'), '{"name":"agent-skills","dependencies":{"upstream-runtime":"1.0.0"}}\n');
-  const before = fs.readFileSync(path.join(directory, 'plugin.json'));
-  const result = node(directory, 'rebrand/apply.cjs', ['--apply']);
-  assert.notEqual(result.status, 0);
-  assert.match(result.stderr, /package\.json/);
-  assert.deepEqual(fs.readFileSync(path.join(directory, 'plugin.json')), before);
-});
-
-test('a distribution without repository CI files remains unchanged by a replay check', t => {
-  const directory = fixture(t);
-  record(directory);
-  apply(directory);
-  fs.rmSync(path.join(directory, '.github'), { recursive: true, force: true });
-  const result = node(directory, 'rebrand/apply.cjs', ['--check']);
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-  assert.equal(fs.existsSync(path.join(directory, '.github')), false);
-});
-
-test('Finder metadata does not masquerade as an upstream runtime file', t => {
-  const directory = fixture(t);
-  record(directory);
-  fs.writeFileSync(path.join(directory, 'skills/.DS_Store'), 'Finder metadata');
-  fs.writeFileSync(path.join(directory, 'skills/spec-driven-development/.DS_Store'), 'Finder metadata');
-  apply(directory);
-  assert.equal(node(directory, 'scripts/validate-brand.cjs').status, 0);
-  const result = node(directory, 'rebrand/apply.cjs', ['--check']);
-  assert.equal(result.status, 0, result.stdout + result.stderr);
-});

package/rebrand/upstream.json

@@ -1,171 +0,0 @@
-{
-  "upstreamCommit": "1401c8b8030e023baeebb31781a6653fe8e93026",
-  "upstreamVersion": "0.6.12",
-  "skills": [
-    "api-and-interface-design",
-    "browser-testing-with-devtools",
-    "ci-cd-and-automation",
-    "code-review-and-quality",
-    "code-simplification",
-    "constraint-driven-development",
-    "context-engineering",
-    "debugging-and-error-recovery",
-    "deprecation-and-migration",
-    "documentation-and-adrs",
-    "doubt-driven-development",
-    "frontend-ui-engineering",
-    "git-workflow-and-versioning",
-    "idea-refine",
-    "incremental-implementation",
-    "interview-me",
-    "observability-and-instrumentation",
-    "performance-optimization",
-    "planning-and-task-breakdown",
-    "security-and-hardening",
-    "shipping-and-launch",
-    "source-driven-development",
-    "spec-driven-development",
-    "test-driven-development",
-    "using-agent-skills"
-  ],
-  "commands": [
-    "build",
-    "code-simplify",
-    "constraints",
-    "plan",
-    "review",
-    "ship",
-    "spec",
-    "test",
-    "webperf"
-  ],
-  "personas": [
-    "code-reviewer",
-    "security-auditor",
-    "test-engineer",
-    "web-performance-auditor"
-  ],
-  "references": [
-    "references/accessibility-checklist.md",
-    "references/definition-of-done.md",
-    "references/observability-checklist.md",
-    "references/orchestration-patterns.md",
-    "references/performance-checklist.md",
-    "references/security-checklist.md",
-    "references/testing-patterns.md"
-  ],
-  "runtimeFiles": [
-    ".agents/plugins/marketplace.json",
-    ".claude-plugin/marketplace.json",
-    ".claude-plugin/plugin.json",
-    ".claude/commands/build.md",
-    ".claude/commands/code-simplify.md",
-    ".claude/commands/constraints.md",
-    ".claude/commands/plan.md",
-    ".claude/commands/review.md",
-    ".claude/commands/ship.md",
-    ".claude/commands/spec.md",
-    ".claude/commands/test.md",
-    ".claude/commands/webperf.md",
-    ".codex-plugin/plugin.json",
-    ".gemini/commands/build.toml",
-    ".gemini/commands/code-simplify.toml",
-    ".gemini/commands/constraints.toml",
-    ".gemini/commands/planning.toml",
-    ".gemini/commands/review.toml",
-    ".gemini/commands/ship.toml",
-    ".gemini/commands/spec.toml",
-    ".gemini/commands/test.toml",
-    ".gemini/commands/webperf.toml",
-    "LICENSE",
-    "agents/code-reviewer.md",
-    "agents/security-auditor.md",
-    "agents/test-engineer.md",
-    "agents/web-performance-auditor.md",
-    "commands/build.toml",
-    "commands/code-simplify.toml",
-    "commands/constraints.toml",
-    "commands/planning.toml",
-    "commands/review.toml",
-    "commands/ship.toml",
-    "commands/spec.toml",
-    "commands/test.toml",
-    "commands/webperf.toml",
-    "hooks/SDD-CACHE.md",
-    "hooks/SIMPLIFY-IGNORE.md",
-    "hooks/sdd-cache-post.sh",
-    "hooks/sdd-cache-pre.sh",
-    "hooks/sdd-cache-test.sh",
-    "hooks/session-start-test.sh",
-    "hooks/session-start.sh",
-    "hooks/simplify-ignore-test.sh",
-    "hooks/simplify-ignore.sh",
-    "plugin.json",
-    "references/accessibility-checklist.md",
-    "references/definition-of-done.md",
-    "references/observability-checklist.md",
-    "references/orchestration-patterns.md",
-    "references/performance-checklist.md",
-    "references/security-checklist.md",
-    "references/testing-patterns.md",
-    "skills/api-and-interface-design/SKILL.md",
-    "skills/browser-testing-with-devtools/SKILL.md",
-    "skills/ci-cd-and-automation/SKILL.md",
-    "skills/code-review-and-quality/SKILL.md",
-    "skills/code-simplification/SKILL.md",
-    "skills/constraint-driven-development/SKILL.md",
-    "skills/constraint-driven-development/references/floor-guard.md",
-    "skills/context-engineering/SKILL.md",
-    "skills/debugging-and-error-recovery/SKILL.md",
-    "skills/deprecation-and-migration/SKILL.md",
-    "skills/documentation-and-adrs/SKILL.md",
-    "skills/doubt-driven-development/SKILL.md",
-    "skills/frontend-ui-engineering/SKILL.md",
-    "skills/git-workflow-and-versioning/SKILL.md",
-    "skills/idea-refine/SKILL.md",
-    "skills/idea-refine/examples.md",
-    "skills/idea-refine/frameworks.md",
-    "skills/idea-refine/refinement-criteria.md",
-    "skills/idea-refine/scripts/idea-refine.sh",
-    "skills/incremental-implementation/SKILL.md",
-    "skills/interview-me/SKILL.md",
-    "skills/observability-and-instrumentation/SKILL.md",
-    "skills/performance-optimization/SKILL.md",
-    "skills/performance-optimization/references/optimization-patterns.md",
-    "skills/planning-and-task-breakdown/SKILL.md",
-    "skills/security-and-hardening/SKILL.md",
-    "skills/security-and-hardening/references/hardening-patterns.md",
-    "skills/shipping-and-launch/SKILL.md",
-    "skills/source-driven-development/SKILL.md",
-    "skills/spec-driven-development/SKILL.md",
-    "skills/test-driven-development/SKILL.md",
-    "skills/using-agent-skills/SKILL.md"
-  ],
-  "skillFrontmatter": {
-    "api-and-interface-design": "44f904d462977e9514f439ac7c057b8691613ba8160360ad9d28b830d3a24001",
-    "browser-testing-with-devtools": "e235631718d18a0527d00eb718ecece2218bbf205d742632f8197b30c052e96b",
-    "ci-cd-and-automation": "dd73253ec445a9af27ebeab74404ee7a770919e40ec9db0b34af4fa9ec36bf7c",
-    "code-review-and-quality": "bf3dda5e08190fcf00175a5053f53550bca91ca32b0a06e4b97cd33e046e8eb7",
-    "code-simplification": "cee4dc87b4bc95d77fe646d4ab9e66c9a422dd081a833d657d618aaf2967b5a2",
-    "constraint-driven-development": "0da347bd34b155ab79cad1e0ee80238f3b4b91b1a17ca0fc2f68460c02224194",
-    "context-engineering": "f4f18514e5eeb162ef9f925dea85af2c72ba79f9b157783a06c39c0bde102d3f",
-    "debugging-and-error-recovery": "658d96f63d56946214a7ec87be0347c78d6024a18f6481dc9f7310638f403629",
-    "deprecation-and-migration": "388d5e7e549f1b728213f0f50d91e9ca96210e0cb23fd9fccd30755268dd01f9",
-    "documentation-and-adrs": "6b2a9d19ebbb81570022a89f09b4810eeb0643a48c698989ee27a224f90bc026",
-    "doubt-driven-development": "df8954091f08c14b0e91bf3bf330f43d5a8408c1efe697591b7b0e8448251c71",
-    "frontend-ui-engineering": "323624fe7ec44bdb4fad3c9488e55993119912625fffa20ee5787389da2b600d",
-    "git-workflow-and-versioning": "dd28469b0f294865dbd20d622a9fee5f05ef87ab1599e973fec6d15a16f76e15",
-    "idea-refine": "07eedcfcba98c6c8f6985917830c4b3d93d4e17af05a9b2ca1b8cb54226bc1b9",
-    "incremental-implementation": "d31e04a0d7b07fc7d01e0176aafbba5607f1cfe7a29a634498305d95c2b3bd3b",
-    "interview-me": "62700902b592a4fa28f6f71f2581d4e1dee412ff288eb156ed97fac3db8b4295",
-    "observability-and-instrumentation": "652b9724e431bd7cfd4481bbab31bb079eeab2b5f70ebc04cf66743e37028721",
-    "performance-optimization": "76befb0806caad12322de8fe372d2e984f6e07c393fd8d3658537c92c7e285cf",
-    "planning-and-task-breakdown": "c3bc139dfa7af82e2d11c7cf461bbf923add7d6494d669da98925f77a7671641",
-    "security-and-hardening": "981142e38769394e6e7760250f2dd599dff21a370040f3f141c42507f04ea540",
-    "shipping-and-launch": "02cc76557e581957cb4987a53e03a8d1aa04c72dc735390d52ae4eed3ddeddf9",
-    "source-driven-development": "233bf9f16f5fe0960c988a5feb2efc475b9b24f9e145d960f8c63e44f4f34fde",
-    "spec-driven-development": "e21f25e534ffa3e2de9ac08c40f8d1d90cef17ad55702f83301cbc2a297c2dcb",
-    "test-driven-development": "38f1489687eaa60761522a8e3efaa926b52786b0e1b3c57947a6050c0da340de",
-    "using-agent-skills": "df5d4f61855ac46da3371bad8c05edb8bc9375ba191acc82b0cca257cfec946c"
-  }
-}

package/scripts/floor-guard-reference-test.js

@@ -1,291 +0,0 @@
-#!/usr/bin/env node
-
-'use strict';
-
-// Contract test for skills/constraint-driven-development/references/floor-guard.md.
-// The fenced `js` block is extracted unchanged and run, with real git, against small
-// fixtures: each is a fresh repository with one source file, one test file and a
-// CONSTRAINTS.md carrying a floor bullet, a coverage minimum, a bundle maximum and an
-// Exceptions table. Loosening moves must exit 1, tightening and no-op moves must exit 0.
-
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-const { spawnSync } = require('node:child_process');
-const { after, before, test } = require('node:test');
-
-const REFERENCE = path.join(
-  __dirname, '..', 'skills', 'constraint-driven-development', 'references', 'floor-guard.md',
-);
-const CONSTRAINTS = [
-  '# Constraints',
-  '',
-  'Last reviewed: 2026-08-08 by @owner',
-  '',
-  '## Floor',
-  '- No skipped tests.',
-  '- Latency: p95 under 200 ms and p99 under 900 ms.',
-  '',
-  '| Dimension | Rule |',
-  '|---|---|',
-  '| Coverage | >= 80% |',
-  '| Bundle | <= 200 KB |',
-  '',
-  '## Exceptions',
-  '',
-  '| ID | Rule | Owner | Expires |',
-  '|---|---|---|---|',
-  '| W1 | no-explicit-any | @owner | 2026-11-01 |',
-  '',
-].join('\n');
-
-let scratch;
-let guard;
-const sandboxes = [];
-
-before(() => {
-  scratch = fs.mkdtempSync(path.join(os.tmpdir(), 'agent-skills-floor-guard-test-'));
-  const source = fs.readFileSync(REFERENCE, 'utf8');
-  const match = source.match(/```js\n([\s\S]*?)\n```/);
-  assert.ok(match, 'floor-guard.md must contain one fenced js block');
-  guard = path.join(scratch, 'floor-guard.mjs');
-  fs.writeFileSync(guard, match[1] + '\n');
-});
-
-after(() => {
-  for (const dir of sandboxes) fs.rmSync(dir, { recursive: true, force: true });
-  if (scratch) fs.rmSync(scratch, { recursive: true, force: true });
-});
-
-function git(cwd, ...args) {
-  const result = spawnSync('git', args, {
-    cwd,
-    encoding: 'utf8',
-    env: {
-      ...process.env,
-      GIT_AUTHOR_NAME: 'Floor Guard Test',
-      GIT_AUTHOR_EMAIL: 'floor-guard@example.invalid',
-      GIT_COMMITTER_NAME: 'Floor Guard Test',
-      GIT_COMMITTER_EMAIL: 'floor-guard@example.invalid',
-      GIT_CONFIG_GLOBAL: '/dev/null',
-    },
-  });
-  assert.equal(result.status, 0, `git ${args.join(' ')} failed: ${result.stderr}`);
-  return result.stdout;
-}
-
-function makeRepo() {
-  const root = fs.mkdtempSync(path.join(os.tmpdir(), 'agent-skills-floor-guard-fixture-'));
-  sandboxes.push(root);
-  git(root, 'init', '-q');
-  fs.writeFileSync(path.join(root, 'app.js'), 'module.exports = 1;\n');
-  fs.writeFileSync(
-    path.join(root, 'app.test.js'),
-    "const assert = require('assert');\nassert.equal(require('./app'), 1);\n",
-  );
-  fs.writeFileSync(path.join(root, 'CONSTRAINTS.md'), CONSTRAINTS);
-  git(root, 'add', '.');
-  git(root, 'commit', '-qm', 'baseline');
-  return root;
-}
-
-function editConstraints(root, from, to) {
-  const file = path.join(root, 'CONSTRAINTS.md');
-  const text = fs.readFileSync(file, 'utf8');
-  assert.ok(text.includes(from), `fixture CONSTRAINTS.md must contain ${JSON.stringify(from)}`);
-  fs.writeFileSync(file, text.replace(from, to));
-}
-
-function commitFile(root, name, text) {
-  fs.writeFileSync(path.join(root, name), text);
-  git(root, 'add', name);
-  git(root, 'commit', '-qm', `add ${name}`);
-}
-
-function runGuard(root, cwd = root) {
-  return spawnSync(process.execPath, [guard, '--base', 'HEAD'], { cwd, encoding: 'utf8' });
-}
-
-const cases = [
-  ['unchanged repository is clean', 0, () => {}],
-  ['a suppression added to a tracked file is a violation', 1, (root) => {
-    fs.writeFileSync(path.join(root, 'app.js'), '// @ts-ignore\nmodule.exports = 1;\n');
-  }],
-  ['a suppression in a new untracked file is a violation', 1, (root) => {
-    fs.writeFileSync(path.join(root, 'new.js'), '// @ts-ignore\nmodule.exports = 1;\n');
-  }],
-  ['a skipped test in a new untracked file is a violation', 1, (root) => {
-    fs.writeFileSync(path.join(root, 'new.test.js'), "test.skip('regression', () => {});\n");
-  }],
-  ['a suppression in a staged new file is a violation', 1, (root) => {
-    fs.writeFileSync(path.join(root, 'new.js'), '// @ts-ignore\nmodule.exports = 1;\n');
-    git(root, 'add', '-A');
-  }],
-  ['a deleted test file is a violation', 1, (root) => {
-    fs.unlinkSync(path.join(root, 'app.test.js'));
-  }],
-  ['a staged deleted test file is a violation', 1, (root) => {
-    fs.unlinkSync(path.join(root, 'app.test.js'));
-    git(root, 'add', '-A');
-  }],
-  ['an assertion removed from a surviving test file is a violation', 1, (root) => {
-    fs.writeFileSync(path.join(root, 'app.test.js'), "const assert = require('assert');\n");
-  }],
-  ['lowering a minimum is a violation', 1, (root) => {
-    editConstraints(root, '| Coverage | >= 80% |', '| Coverage | >= 60% |');
-  }],
-  ['raising a minimum is silent', 0, (root) => {
-    editConstraints(root, '| Coverage | >= 80% |', '| Coverage | >= 90% |');
-  }],
-  ['raising a maximum is a violation', 1, (root) => {
-    editConstraints(root, '| Bundle | <= 200 KB |', '| Bundle | <= 300 KB |');
-  }],
-  ['lowering a maximum is silent', 0, (root) => {
-    editConstraints(root, '| Bundle | <= 200 KB |', '| Bundle | <= 150 KB |');
-  }],
-  ['deleting one threshold from a rule is a violation', 1, (root) => {
-    editConstraints(root, 'p95 under 200 ms and p99 under 900 ms.', 'p95 under 200 ms.');
-  }],
-  ['a number inserted before a threshold is silent', 0, (root) => {
-    editConstraints(root, '| Coverage | >= 80% |', '| Coverage | per ADR 7, >= 80% |');
-  }],
-  ['adding a threshold to a rule is silent', 0, (root) => {
-    editConstraints(root, '| Bundle | <= 200 KB |', '| Bundle | <= 200 KB, <= 60 KB gzipped |');
-  }],
-  ['a deleted floor rule is a violation', 1, (root) => {
-    editConstraints(root, '- No skipped tests.\n', '');
-  }],
-  ['a new exception row is a violation', 1, (root) => {
-    const file = path.join(root, 'CONSTRAINTS.md');
-    fs.appendFileSync(file, '| W2 | temporary waiver | @owner | 2026-10-01 |\n');
-  }],
-];
-
-for (const [name, expectedExit, mutate] of cases) {
-  test(`floor guard: ${name}`, () => {
-    const root = makeRepo();
-    mutate(root);
-    const result = runGuard(root);
-    assert.equal(
-      result.status, expectedExit,
-      `expected exit ${expectedExit}, got ${result.status}\nstdout: ${result.stdout}\nstderr: ${result.stderr}`,
-    );
-  });
-}
-
-test('floor guard: a removed exception row is silent', () => {
-  const root = makeRepo();
-  editConstraints(root, '| W1 | no-explicit-any | @owner | 2026-11-01 |\n', '');
-  const result = runGuard(root);
-  assert.equal(result.status, 0, result.stderr);
-});
-
-test('floor guard: a missing merge base exits 2, never 0', () => {
-  const root = makeRepo();
-  const result = spawnSync(process.execPath, [guard, '--base', 'no-such-ref'], {
-    cwd: root, encoding: 'utf8',
-  });
-  assert.equal(result.status, 2, result.stderr);
-});
-
-test('floor guard: a threshold with no readable direction is reported when it changes', () => {
-  const root = makeRepo();
-  editConstraints(root, '| W1 | no-explicit-any | @owner | 2026-11-01 |', '| W1 | no-explicit-any | @owner | 2027-11-01 |');
-  const result = runGuard(root);
-  assert.equal(result.status, 1, result.stderr);
-  assert.match(result.stderr, /threshold-changed/);
-});
-
-test('floor guard: a renamed rule is reported as removed, with a note that its label may have changed', () => {
-  const root = makeRepo();
-  editConstraints(root, '| Coverage | >= 80% |', '| Code coverage | >= 80% |');
-  const result = runGuard(root);
-  assert.equal(result.status, 1, result.stderr);
-  assert.match(result.stderr, /\[rule-removed\]/);
-  assert.match(result.stderr, /label changed/);
-});
-
-test('floor guard: a threshold that loses its direction words is reported as removed, with a note saying so', () => {
-  const root = makeRepo();
-  editConstraints(root, '| Coverage | >= 80% |', '| Coverage | 80% |');
-  const result = runGuard(root);
-  assert.equal(result.status, 1, result.stderr);
-  assert.match(result.stderr, /\[threshold-removed\]/);
-  assert.match(result.stderr, /direction words/);
-});
-
-// Inside a hunk every line is content. An added line that starts with `++` shows in the diff
-// as `+++...`, and a removed line that starts with `--` as `---...`: neither is a file header.
-test('floor guard: an added line that starts with ++ is still checked', () => {
-  const root = makeRepo();
-  fs.writeFileSync(
-    path.join(root, 'app.js'),
-    'let n = 0;\n++n; // eslint-disable-line no-plusplus\nmodule.exports = n;\n',
-  );
-  const result = runGuard(root);
-  assert.equal(result.status, 1, result.stderr);
-  assert.match(result.stderr, /\[silenced-checker\] app\.js:/);
-});
-
-test('floor guard: an added line that starts with "++ " does not rename the file for later hunks', () => {
-  const root = makeRepo();
-  commitFile(root, 'counter.test.js', [
-    "const assert = require('assert');",
-    'let n = 0;',
-    'n += 1;',
-    'const m = 2;',
-    'assert.equal(n, 1);',
-    'assert.equal(m, 2);',
-    '',
-  ].join('\n'));
-  const file = path.join(root, 'counter.test.js');
-  const text = fs.readFileSync(file, 'utf8');
-  fs.writeFileSync(file, text.replace('n += 1;', '++ n;').replace('assert.equal(m, 2);\n', ''));
-  const result = runGuard(root);
-  assert.equal(result.status, 1, result.stderr);
-  assert.match(result.stderr, /\[assertion-removed\] counter\.test\.js:/);
-});
-
-test('floor guard: a removed line that starts with -- is still checked', () => {
-  const root = makeRepo();
-  commitFile(root, 'countdown.test.js', [
-    "const assert = require('assert');",
-    'let n = 1;',
-    '--n; assert.equal(n, 0);',
-    '',
-  ].join('\n'));
-  const file = path.join(root, 'countdown.test.js');
-  fs.writeFileSync(file, fs.readFileSync(file, 'utf8').replace('--n; assert.equal(n, 0);\n', ''));
-  const result = runGuard(root);
-  assert.equal(result.status, 1, result.stderr);
-  assert.match(result.stderr, /\[assertion-removed\] countdown\.test\.js:/);
-});
-
-// `git ls-files` lists only the current directory's subtree, relative to it, while `git diff`
-// covers the whole tree with paths from the top. Run from a subfolder, the guard has to see the
-// same files, under the same names, as it does from the top.
-test('floor guard: run from a subfolder, an untracked file outside it is still checked', () => {
-  const root = makeRepo();
-  fs.mkdirSync(path.join(root, 'sub'));
-  fs.writeFileSync(path.join(root, 'new.js'), '// @ts-ignore\nmodule.exports = 1;\n');
-  const result = runGuard(root, path.join(root, 'sub'));
-  assert.equal(result.status, 1, result.stderr);
-  assert.match(result.stderr, /\[silenced-checker\] new\.js:/);
-});
-
-test('floor guard: run from a subfolder, an untracked file inside it is named from the top', () => {
-  const root = makeRepo();
-  fs.mkdirSync(path.join(root, 'sub'));
-  fs.writeFileSync(path.join(root, 'sub', 'new.js'), '// @ts-ignore\nmodule.exports = 1;\n');
-  const result = runGuard(root, path.join(root, 'sub'));
-  assert.equal(result.status, 1, result.stderr);
-  assert.match(result.stderr, /\[silenced-checker\] sub\/new\.js:/);
-});
-
-test('floor guard: outside a git work tree exits 2, never 0', () => {
-  const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'agent-skills-floor-guard-no-repo-'));
-  sandboxes.push(dir);
-  const result = runGuard(dir);
-  assert.equal(result.status, 2, result.stderr);
-});