@givolt/i3-composer-agent 2.2.2 → 2.2.3

package/docs/codebuddy-setup.md

@@ -11,7 +11,7 @@ npx @givolt/i3-composer-agent --codebuddy
 向导预选 CodeBuddy，可增加其他工具，再选择项目或全局范围。
 项目安装提供 `.codebuddy/skills/`（25 个技能）、`references/`、
 `commands/`（9 个命令）和 `agents/`（4 个角色）。
-同级 `evals/`、`hooks/`、`scripts/`、`docs/`、`references/` 各自通过 `i3-composer-agent/` 分组资料，
+同级 `i3-composer-agent/` 是配套工具根，集中提供 `package.json`、`evals/`、`hooks/`、`scripts/`、`docs/`、`references/` 和校验输入 `adapters/`，
 只有一份实际技能、命令和角色，评测与宿主共用。
 全局使用 `~/.codebuddy/` 的相同结构。默认中文说明和回复；技能/命令标识不改名。
 安装成功清理旧内容，失败恢复；设置与自定义命令保留。
@@ -41,8 +41,8 @@ CodeBuddy 与 WorkBuddy 项目共用 `.codebuddy/`；同版本追加客户端会
 
 ## 配套检查
 
-在业务项目运行 `node .codebuddy/scripts/i3-composer-agent/run-evals.cjs --min-rank1 95`，
-可检查中英文触发，不调用模型；全局安装使用脚本的全局绝对路径。五项校验、行为评测和样例用法见
+在业务项目运行 `node .codebuddy/i3-composer-agent/scripts/run-evals.js --min-rank1 95`，
+可检查中英文触发，不调用模型；进入 `.codebuddy/i3-composer-agent/` 可运行 `npm test` 和 `node scripts/run-evals.js --locale zh-CN --min-rank1 95`。全局安装使用脚本的全局绝对路径。五项校验、行为评测和样例用法见
 [评测说明](../evals/README.md)。Hook 文件已安装，需要按宿主协议单独启用。
 
 ## 通过 CodeBuddy Code 插件管理器安装

package/docs/codex-setup.md

@@ -6,11 +6,11 @@
 npx @givolt/i3-composer-agent --codex
 ```
 
-项目范围写入 `.agents/skills/`，全局写入 `~/.agents/skills/`，并安装同级 `references/i3-composer-agent/`。
+项目范围写入 `.agents/skills/`，全局写入 `~/.agents/skills/`，并安装同级 `i3-composer-agent/references/`。
 提供全部 25 个 `i3-` 技能，中文选择说明与默认中文回复在每个入口独立生效。
 同级 `evals/`、`hooks/`、`scripts/`、`docs/` 各自以 `i3-composer-agent/` 分组资料。
 命令与角色文件也会提供，供按需阅读，其自动发现由宿主决定。
-从项目目录执行 `node .agents/scripts/i3-composer-agent/run-evals.cjs --min-rank1 95`
+从项目目录执行 `node .agents/i3-composer-agent/scripts/run-evals.js --min-rank1 95`
 检查实际安装技能的中英文触发，不调用模型；五项结构校验详见
 [评测说明](../evals/README.md)。原生调用使用技能 ID，Hook 文件需按宿主能力单独启用。
 

package/docs/workbuddy-setup.md

@@ -13,11 +13,11 @@ npx @givolt/i3-composer-agent --agents codebuddy,workbuddy --scope global --yes
 - `~/.workbuddy/skills/`：25 个 `i3-` 技能，选择说明和默认回复中文。
 - `~/.workbuddy/commands/`：9 个 `/i3-*` 短命令。
 - `~/.workbuddy/agents/`：4 个专业角色。
-- `~/.workbuddy/references/i3-composer-agent/`：共享参考。
-- `~/.workbuddy/docs/i3-composer-agent/agents.md`：角色说明。
-- `~/.workbuddy/evals/i3-composer-agent/`：中英文用例、样例与评测结果。
-- `~/.workbuddy/hooks/i3-composer-agent/`：Hook 文件与说明。
-- `~/.workbuddy/scripts/i3-composer-agent/`：评测、校验与依赖库。
+- `~/.workbuddy/i3-composer-agent/references/`：共享参考。
+- `~/.workbuddy/i3-composer-agent/docs/agents.md`：角色说明。
+- `~/.workbuddy/i3-composer-agent/evals/`：中英文用例、样例与评测结果。
+- `~/.workbuddy/i3-composer-agent/hooks/`：Hook 文件与说明。
+- `~/.workbuddy/i3-composer-agent/scripts/`：评测、校验与依赖库。
 - `~/.workbuddy/.i3-composer-agent.json`：安装内容所有权及客户端清单。
 
 自定义 profile 可设置 `WORKBUDDY_CONFIG_DIR`，安装后打开对应 profile。
@@ -25,8 +25,8 @@ WorkBuddy 的目录选择使用自己的配置，不借用 `CODEBUDDY_CONFIG_DIR
 
 ## 配套检查
 
-运行 `node ~/.workbuddy/scripts/i3-composer-agent/run-evals.cjs --min-rank1 95`，检查实际安装技能的中英文触发，
-不会调用模型。项目安装将 `~/.workbuddy/` 换成 `.codebuddy/`。五项校验、行为评测用例、样例和
+运行 `node ~/.workbuddy/i3-composer-agent/scripts/run-evals.js --min-rank1 95`，检查实际安装技能的中英文触发，
+不会调用模型。进入 `~/.workbuddy/i3-composer-agent/` 运行 `npm test` 可检查完整内容、两种语言及全部脚本回归；原 `.js` 入口保留。项目安装将 `~/.workbuddy/` 换成 `.codebuddy/`。五项校验、行为评测用例、样例和
 脚本用法见 [评测说明](../evals/README.md)。Hook 文件默认提供，启用需适配宿主事件协议。
 
 ## 发现和调用

package/evals/README.md

@@ -6,22 +6,31 @@
 
 ## 运行位置
 
-原生安装的技能位于宿主基址的 `skills/`；评测资料位于 `evals/i3-composer-agent/`，
-脚本位于 `scripts/i3-composer-agent/`。安装清单记录预期技能和文件，评测只读取这份实际安装内容。
+原生安装的技能位于宿主基址的 `skills/`；评测资料位于 `i3-composer-agent/evals/`，
+脚本位于 `i3-composer-agent/scripts/`。安装清单记录预期技能和文件，评测只读取这份实际安装内容。
 例如 CodeBuddy 项目基址是 `.codebuddy/`，WorkBuddy 全局基址是 `~/.workbuddy/`。
 下面原生示例从业务项目根目录运行；从其他位置运行时，将脚本路径换成绝对路径即可。
-Node.js 20 或以上，无需额外 npm 依赖；`.cjs` 入口兼容业务项目的 ESM/CommonJS 设置。
+Node.js 20 或以上，无需额外 npm 依赖；配套根的独立 CommonJS 配置使 `.js` 入口兼容业务项目的 ESM/CommonJS 设置。
 
 ## 不调用模型的检查
 
+进入 `.codebuddy/i3-composer-agent/`（全局安装进入对应宿主的配套根）即可沿用：
+
+```bash
+npm test
+node scripts/run-evals.js --locale zh-CN --min-rank1 95
+```
+
+第一条执行五项校验、中英文评测与全部脚本回归；第二条只运行中文评测。以下项目路径入口也可用。
+
 ```bash
-node .codebuddy/scripts/i3-composer-agent/validate-skills.cjs
-node .codebuddy/scripts/i3-composer-agent/validate-versions.cjs
-node .codebuddy/scripts/i3-composer-agent/validate-commands.cjs
-node .codebuddy/scripts/i3-composer-agent/validate-reference-links.cjs
-node .codebuddy/scripts/i3-composer-agent/validate-artifact-paths.cjs
-node .codebuddy/scripts/i3-composer-agent/run-evals.cjs --locale zh-CN --min-rank1 95
-node .codebuddy/scripts/i3-composer-agent/run-evals.cjs --locale en --min-rank1 95
+node .codebuddy/i3-composer-agent/scripts/validate-skills.js
+node .codebuddy/i3-composer-agent/scripts/validate-versions.js
+node .codebuddy/i3-composer-agent/scripts/validate-commands.js
+node .codebuddy/i3-composer-agent/scripts/validate-reference-links.js
+node .codebuddy/i3-composer-agent/scripts/validate-artifact-paths.js
+node .codebuddy/i3-composer-agent/scripts/run-evals.js --locale zh-CN --min-rank1 95
+node .codebuddy/i3-composer-agent/scripts/run-evals.js --locale en --min-rank1 95
 ```
 
 五项校验分别检查技能格式、安装版本及登记文件、实际命令和角色格式、引用锚点、规格/计划产物路径。
@@ -37,8 +46,9 @@ npm run validate
 node scripts/run-evals.js --locale zh-CN --min-rank1 95
 ```
 
-独立插件的校验还会检查插件清单和三种命令格式的一致性。原生安装不复制这些注册清单，
-不需要在业务项目或宿主基址执行 `npm test`。
+原生配套与独立插件均保留全部原版脚本及回归测试。原生校验读取真实宿主命令，
+并与 `adapters/` 中另外两种格式比较；同时检查五份插件清单、配套包及安装版本。
+这些 adapter 输入不自动注册插件。在配套根运行 `npm test`，不修改业务项目配置。
 
 触发评测使用词法评分检查正例、负例、描述冲突、用例结构及样例是否齐全。
 中英文各自满足指定门槛；这不能证明模型在实际宿主中一定选择正确技能。
@@ -49,13 +59,13 @@ node scripts/run-evals.js --locale zh-CN --min-rank1 95
 先预览，不调用模型、不生成评测结果：
 
 ```bash
-node .codebuddy/scripts/i3-composer-agent/run-evals.cjs --behavioral i3-test-driven-development --locale zh-CN --dry-run
+node .codebuddy/i3-composer-agent/scripts/run-evals.js --behavioral i3-test-driven-development --locale zh-CN --dry-run
 ```
 
 确认需要后，移除 `--dry-run` 才会调用已登录的 Claude Code，并消耗该账户的模型额度：
 
 ```bash
-node .codebuddy/scripts/i3-composer-agent/run-evals.cjs --behavioral i3-test-driven-development --locale zh-CN
+node .codebuddy/i3-composer-agent/scripts/run-evals.js --behavioral i3-test-driven-development --locale zh-CN
 ```
 
 执行型用例需要 Git，并在临时 Git 项目中复制 `fixtures/` 的真实输入；
@@ -66,7 +76,7 @@ node .codebuddy/scripts/i3-composer-agent/run-evals.cjs --behavioral i3-test-dri
 
 `--locale en` 运行英文用例，省略该参数运行两种语言。可通过 `I3_EVAL_MODEL`
 和 `I3_EVAL_GRADER_MODEL` 指定执行/评分模型；报告同时记录宿主和模型信息。
-原生结果写入 `evals/i3-composer-agent/results/`；独立插件写入 `evals/results/`。
+原生结果写入 `i3-composer-agent/evals/results/`；独立插件写入 `evals/results/`。
 结果不会进入发行包和文件替换清单，升级保留；旧配套目录的结果会迁入新位置，同名异内容时停止迁移。
 
 ## 插件级评测
@@ -86,7 +96,7 @@ claude plugin eval . --no-publish
 
 ## Hooks
 
-原生的 `hooks/i3-composer-agent/`（独立插件为 `hooks/`）默认交付会话注入、文档缓存、代码块保护脚本及说明。
+原生的 `i3-composer-agent/hooks/`（独立插件为 `hooks/`）默认交付会话注入、文档缓存、代码块保护脚本及说明。
 安装文件不会自动注册事件。中文回复与普通技能调用不依赖启用 Hook。
 会话注入仅适用于缺少原生技能路由的宿主；已有路由时不要重复注入。
 缓存及保护的当前配置示例针对 Claude Code，详见

package/package.json

@@ -139,14 +139,8 @@
     "evals/zh-CN/cases/i3-spec-driven-development.json",
     "evals/zh-CN/cases/i3-test-driven-development.json",
     "evals/zh-CN/cases/i3-using-agent-skills.json",
-    "scripts/run-evals.js",
-    "scripts/validate-skills.js",
-    "scripts/validate-versions.js",
-    "scripts/validate-commands.js",
-    "scripts/validate-reference-links.js",
-    "scripts/validate-artifact-paths.js",
-    "scripts/lib/skill-lint.js",
     "docs/adoption-guide.md",
+    "docs/advanced-per-agent-configuration.md",
     "docs/agents.md",
     "docs/antigravity-setup.md",
     "docs/codebuddy-setup.md",
@@ -163,12 +157,28 @@
     "docs/workbuddy-setup.md",
     "plugin.json",
     "README.md",
-    "LICENSE"
+    "LICENSE",
+    "scripts/floor-guard-reference-test.js",
+    "scripts/lib/skill-lint-test.js",
+    "scripts/lib/skill-lint.js",
+    "scripts/run-evals-test.js",
+    "scripts/run-evals.js",
+    "scripts/run-regressions.js",
+    "scripts/validate-artifact-paths-test.js",
+    "scripts/validate-artifact-paths.js",
+    "scripts/validate-commands-test.js",
+    "scripts/validate-commands.js",
+    "scripts/validate-reference-links-test.js",
+    "scripts/validate-reference-links.js",
+    "scripts/validate-skills.js",
+    "scripts/validate-versions-test.js",
+    "scripts/validate-versions.js"
   ],
   "scripts": {
     "validate": "node scripts/validate-skills.js && node scripts/validate-versions.js && node scripts/validate-commands.js && node scripts/validate-reference-links.js && node scripts/validate-artifact-paths.js",
     "eval": "node scripts/run-evals.js",
-    "test": "npm run validate && npm run eval -- --min-rank1 95",
+    "test": "npm run validate && npm run eval -- --min-rank1 95 && npm run test:regression",
+    "test:regression": "node scripts/run-regressions.js",
     "prepack": "npm run validate"
   },
   "keywords": [
@@ -301,14 +311,8 @@
       "evals/zh-CN/cases/i3-spec-driven-development.json",
       "evals/zh-CN/cases/i3-test-driven-development.json",
       "evals/zh-CN/cases/i3-using-agent-skills.json",
-      "scripts/run-evals.js",
-      "scripts/validate-skills.js",
-      "scripts/validate-versions.js",
-      "scripts/validate-commands.js",
-      "scripts/validate-reference-links.js",
-      "scripts/validate-artifact-paths.js",
-      "scripts/lib/skill-lint.js",
       "docs/adoption-guide.md",
+      "docs/advanced-per-agent-configuration.md",
       "docs/agents.md",
       "docs/antigravity-setup.md",
       "docs/codebuddy-setup.md",
@@ -326,10 +330,25 @@
       "plugin.json",
       "package.json",
       "README.md",
-      "LICENSE"
+      "LICENSE",
+      "scripts/floor-guard-reference-test.js",
+      "scripts/lib/skill-lint-test.js",
+      "scripts/lib/skill-lint.js",
+      "scripts/run-evals-test.js",
+      "scripts/run-evals.js",
+      "scripts/run-regressions.js",
+      "scripts/validate-artifact-paths-test.js",
+      "scripts/validate-artifact-paths.js",
+      "scripts/validate-commands-test.js",
+      "scripts/validate-commands.js",
+      "scripts/validate-reference-links-test.js",
+      "scripts/validate-reference-links.js",
+      "scripts/validate-skills.js",
+      "scripts/validate-versions-test.js",
+      "scripts/validate-versions.js"
     ]
   },
-  "version": "2.2.2",
+  "version": "2.2.3",
   "repository": {
     "type": "git",
     "url": "https://github.com/49780522/i3-composer-agent.git"

package/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "i3-composer-agent",
-  "version": "2.2.2",
+  "version": "2.2.3",
   "description": "覆盖需求、计划、开发、测试、审查与发布的软件工程技能，默认使用简体中文。",
   "homepage": "https://github.com/49780522/i3-composer-agent",
   "repository": "https://github.com/49780522/i3-composer-agent"

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

@@ -0,0 +1,291 @@
+#!/usr/bin/env node
+
+'use strict';
+
+// Contract test for skills/i3-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', 'i3-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);
+});