@event4u/agent-config 14.10.0 → 14.12.0

package/src/scripts/design_slop_rules.test.ts

@@ -1,239 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { SLOP_RULES, type DesignContext } from "./design_slop_rules.js";
-import { scanFile } from "./lint_design_slop.js";
-
-// Context with no DESIGN.md → all gates OFF, every rule active.
-const NO_CTX: DesignContext = { raw: "", has: () => false };
-const ctxWith = (raw: string): DesignContext => ({
-  raw: raw.toLowerCase(),
-  has: (...ks: string[]) => ks.some((k) => raw.toLowerCase().includes(k.toLowerCase())),
-});
-
-/**
- * Calibration corpus (Phase 4). One positive (must fire) + one negative (must be
- * clean) per rule. `ext` drives the engine class. No untested tell — the guard
- * below asserts every registry rule appears here.
- */
-const FIXTURES: Record<string, { ext: string; positive: string; negative: string }> = {
-  "slop-v1-side-stripe": {
-    ext: "css",
-    positive: ".card { border-left: 4px solid #6c5ce7; padding: 16px; }",
-    negative: ".card { border-left: 1px solid #eee; padding: 16px; }",
-  },
-  "slop-v3-ghost-card": {
-    ext: "css",
-    positive: ".card { border: 1px solid #ddd; box-shadow: 0 8px 24px rgba(0,0,0,.1); }",
-    negative: ".card { box-shadow: 0 2px 4px rgba(0,0,0,.1); }",
-  },
-  "slop-v6-diagonal-stripes": {
-    ext: "css",
-    positive: ".bg { background: repeating-linear-gradient(45deg,#000,#000 10px,#111 10px,#111 20px); }",
-    negative: ".bg { background: linear-gradient(#fff,#eee); }",
-  },
-  "slop-c2-gradient-text": {
-    ext: "css",
-    positive: ".h { background: linear-gradient(90deg,#f00,#00f); -webkit-background-clip: text; color: transparent; }",
-    negative: ".h { color: #111; }",
-  },
-  "slop-c5-cream-palette": {
-    ext: "css",
-    positive: "body { background: #f5f1ea; } .a { color: #b08947; }",
-    negative: "body { background: #ffffff; } .a { color: #111111; }",
-  },
-  "slop-t4-eyebrow-overuse": {
-    ext: "html",
-    positive:
-      '<section><span class="uppercase">Features</span></section><section><span class="uppercase">Pricing</span></section><section><h2>Contact</h2></section>',
-    negative:
-      '<section><span class="uppercase">Features</span></section><section><h2>Pricing</h2></section><section><h2>Contact</h2></section>',
-  },
-  "slop-t6-crushed-tracking": {
-    ext: "css",
-    positive: ".display { letter-spacing: -0.05em; }",
-    negative: ".display { letter-spacing: -0.02em; }",
-  },
-  "slop-t7-default-fonts": {
-    ext: "css",
-    positive: "body { font-family: 'Inter', sans-serif; }",
-    negative: "body { font-family: 'Söhne', sans-serif; }",
-  },
-  "slop-l4-numbered-markers": {
-    ext: "html",
-    positive: "<div>01</div><div>02</div><div>03</div>",
-    negative: "<div>Discover</div><div>Build</div><div>Ship</div>",
-  },
-  "slop-l8-zindex-magic": {
-    ext: "css",
-    positive: ".modal { z-index: 9999; }",
-    negative: ".modal { z-index: 400; }",
-  },
-  "slop-m4-transition-all": {
-    ext: "css",
-    positive: ".x { transition: all 200ms ease-out; }",
-    negative: ".x { transition: transform 200ms ease-out; }",
-  },
-  "slop-m2-animate-layout": {
-    ext: "css",
-    positive: ".x { transition: width 200ms ease; }",
-    negative: ".x { transition: transform 200ms, opacity 200ms; }",
-  },
-  "slop-cp1-em-dash": {
-    ext: "md",
-    // ≥80 words is the rule floor; positive carries 4 em-dashes (well over the
-    // ~2-per-500 density), negative carries one across the same length.
-    positive:
-      "We build tools — fast, focused tools — that teams genuinely love using every single day of the working week. " +
-      "The product is calm and clear and useful, and people return to it daily because it respects their time — and their attention — and never wastes a single click of effort or a moment of their busy day, and that quiet reliability is the whole point of the thing we set out to make together, and the team keeps shipping the small improvements that matter most to the people who rely on it.",
-    negative:
-      "We build tools, fast and focused tools, that teams genuinely love using every single day of the working week. " +
-      "The product is calm and clear and useful, and people return to it daily because it respects their time and their attention and never wastes a single click of effort or a moment of their busy day — and that quiet reliability is the whole point of the thing we set out to make together, and the team keeps shipping the small improvements that matter most to the people who rely on it.",
-  },
-  "slop-cp2-buzzword": {
-    ext: "md",
-    positive: "We streamline and empower your robust, world-class workflow.",
-    negative: "We cut page load by 40% and remove three steps from checkout.",
-  },
-  "slop-cp5-emoji-ui": {
-    ext: "html",
-    positive: '<h1>🚀 Get Started</h1><button>✅ Track progress</button>',
-    // Functional placements stay clean: mid-sentence emoji, status-cell emoji.
-    negative:
-      '<h1>Get Started</h1><p>Deploys are marked ✅ when healthy.</p><td>✅</td><button>Track progress</button>',
-  },
-  "slop-v8-lock-shape": {
-    ext: "css",
-    positive: ".a{border-radius:4px}.b{border-radius:8px}.c{border-radius:12px}.d{border-radius:20px}",
-    negative: ".a{border-radius:8px}.b{border-radius:12px}",
-  },
-  "slop-l9-section-monotony": {
-    ext: "html",
-    // 8 identical media+text sections → 1 distinct layout family (< 4)
-    positive: '<section><div class="grid"><img src="x"><h2>A</h2><p>t</p></div></section>'.repeat(8),
-    // 8 sections, 4 distinct signatures → not monotonous
-    negative: (
-      '<section><div class="grid"><img src="x"><h2>A</h2></div></section>' +
-      "<section><h2>B</h2><p>t</p></section>" +
-      '<section><div class="flex"><p>c</p></div></section>' +
-      '<section><img src="y"></section>'
-    ).repeat(2),
-  },
-  "slop-l10-zigzag": {
-    ext: "html",
-    // 3 consecutive media+text two-column sections → run > 2
-    positive: '<section><div class="grid"><img src="x"><h2>A</h2><p>t</p></div></section>'.repeat(3),
-    // only 2 consecutive → within cap
-    negative: '<section><div class="grid"><img src="x"><h2>A</h2><p>t</p></div></section>'.repeat(2),
-  },
-  "slop-v4-over-rounded": {
-    ext: "css",
-    // 24px radius on an 80px chip — mid-range rounding on a small surface
-    positive: ".chip{width:80px;border-radius:24px;background:#1f5f9c}",
-    // a deliberate pill is a shape decision, not over-rounding
-    negative: ".chip{width:80px;border-radius:9999px;background:#1f5f9c}",
-  },
-  "slop-c3-dark-glow": {
-    ext: "css",
-    // zero-offset saturated glow inside a declared dark surface
-    positive: ".dark .btn{background:#0b0f14;box-shadow:0 0 18px #22d3ee}",
-    // same dark surface, ordinary offset depth — not a glow
-    negative: ".dark .btn{background:#0b0f14;box-shadow:0 2px 8px #22d3ee}",
-  },
-  "slop-t9-uppercase-body": {
-    ext: "css",
-    positive: "article p{text-transform:uppercase;font-size:16px}",
-    // a short field label is the legitimate home of uppercase
-    negative: ".field-label{text-transform:uppercase;font-size:12px}",
-  },
-  "slop-t10-wide-tracking-body": {
-    ext: "css",
-    positive: "article p{letter-spacing:0.12em;font-size:16px}",
-    negative: "article p{letter-spacing:0.01em;font-size:16px}",
-  },
-  "slop-m1-bounce-easing": {
-    ext: "css",
-    // y1 = 1.56 overshoots the target and returns — the bounce signature
-    positive: ".card{transition:transform .3s cubic-bezier(.34,1.56,.64,1)}",
-    negative: ".card{transition:transform .3s cubic-bezier(.4,0,.2,1)}",
-  },
-  "slop-m3-img-hover-transform": {
-    ext: "css",
-    positive: ".tile:hover img{transform:scale(1.05)}",
-    negative: ".tile:hover .tile__title{color:#16202b}",
-  },
-};
-
-function findingsFor(ruleId: string, content: string, ext: string, ctx = NO_CTX) {
-  return scanFile(content, `fixture.${ext}`, ctx).filter((f) => f.rule === ruleId);
-}
-
-describe("design-slop rule registry", () => {
-  it("every rule has a unique id + a catalog id", () => {
-    const ids = SLOP_RULES.map((r) => r.id);
-    expect(new Set(ids).size).toBe(ids.length);
-    for (const r of SLOP_RULES) {
-      expect(r.id).toMatch(/^slop-/);
-      expect(r.catalogId).toMatch(/^[A-Z]+\d+$/);
-      expect(r.engines.length).toBeGreaterThan(0);
-    }
-  });
-
-  it("no untested tell — every registry rule has a +/- fixture", () => {
-    for (const r of SLOP_RULES) {
-      expect(FIXTURES[r.id], `missing fixture for ${r.id}`).toBeDefined();
-    }
-    // and no orphan fixtures
-    const ids = new Set(SLOP_RULES.map((r) => r.id));
-    for (const k of Object.keys(FIXTURES)) {
-      expect(ids.has(k), `orphan fixture ${k}`).toBe(true);
-    }
-  });
-
-  describe("positive fixtures fire, negative fixtures are clean", () => {
-    for (const r of SLOP_RULES) {
-      const fx = FIXTURES[r.id];
-      if (!fx) continue;
-      it(`${r.id} (${r.catalogId})`, () => {
-        expect(findingsFor(r.id, fx.positive, fx.ext).length, "positive must fire").toBeGreaterThan(0);
-        expect(findingsFor(r.id, fx.negative, fx.ext).length, "negative must be clean").toBe(0);
-      });
-    }
-  });
-});
-
-describe("context gates suppress when DESIGN.md declares intent", () => {
-  it("t7 default-font is suppressed when DESIGN.md adopts Inter", () => {
-    const fx = FIXTURES["slop-t7-default-fonts"]!;
-    expect(findingsFor("slop-t7-default-fonts", fx.positive, "css", NO_CTX).length).toBeGreaterThan(0);
-    expect(
-      findingsFor("slop-t7-default-fonts", fx.positive, "css", ctxWith("Body font: Inter (chosen for variable metrics in a data-dense dashboard)")).length,
-    ).toBe(0);
-  });
-
-  it("cp5 emoji-ui is suppressed when DESIGN.md declares an emoji strategy", () => {
-    const fx = FIXTURES["slop-cp5-emoji-ui"]!;
-    expect(findingsFor("slop-cp5-emoji-ui", fx.positive, "html", NO_CTX).length).toBeGreaterThan(0);
-    expect(
-      findingsFor("slop-cp5-emoji-ui", fx.positive, "html", ctxWith("Voice: playful; emoji are part of the brand voice in headings and CTAs")).length,
-    ).toBe(0);
-  });
-
-  it("c5 cream-palette is suppressed under a declared warm-neutral language", () => {
-    const fx = FIXTURES["slop-c5-cream-palette"]!;
-    expect(findingsFor("slop-c5-cream-palette", fx.positive, "css", NO_CTX).length).toBeGreaterThan(0);
-    expect(
-      findingsFor("slop-c5-cream-palette", fx.positive, "css", ctxWith("Palette: warm-neutral, first-party")).length,
-    ).toBe(0);
-  });
-});
-
-describe("inline-ignore", () => {
-  it("design-slop-disable-next-line suppresses the next line", () => {
-    const css = "/* design-slop-disable-next-line slop-m4-transition-all */\n.x { transition: all 200ms; }";
-    expect(findingsFor("slop-m4-transition-all", css, "css").length).toBe(0);
-  });
-
-  it("file-scope design-slop-disable suppresses the whole file", () => {
-    const css = "/* design-slop-disable slop-l8-zindex-magic */\n.a{z-index:9999}\n.b{z-index:999}";
-    expect(findingsFor("slop-l8-zindex-magic", css, "css").length).toBe(0);
-  });
-});

package/src/scripts/self_review_gate.test.ts

@@ -1,296 +0,0 @@
-import { spawnSync } from 'node:child_process';
-import * as fs from 'node:fs';
-import * as os from 'node:os';
-import * as path from 'node:path';
-
-import { afterEach, describe, expect, it } from 'vitest';
-
-import {
-    LARGE_DIFF_LINES,
-    buildPlan,
-    classifyBlocking,
-    detectReleaseVersion,
-    escalationReasons,
-    gateVerdict,
-    isReviewablePath,
-    pickPreviousTag,
-    renderReview,
-    type Finding,
-} from './self_review_gate.js';
-
-const f = (severity: Finding['severity'], kind: Finding['kind']): Finding => ({
-    severity,
-    kind,
-    title: `${severity} ${kind}`,
-    detail: 'x',
-});
-
-describe('classifyBlocking', () => {
-    it('blocks security + high and claim + critical', () => {
-        expect(classifyBlocking(f('high', 'security'))).toBe(true);
-        expect(classifyBlocking(f('critical', 'claim'))).toBe(true);
-    });
-    it('does not block low/medium security, or any style/correctness', () => {
-        expect(classifyBlocking(f('medium', 'security'))).toBe(false);
-        expect(classifyBlocking(f('low', 'claim'))).toBe(false);
-        expect(classifyBlocking(f('critical', 'style'))).toBe(false);
-        expect(classifyBlocking(f('high', 'correctness'))).toBe(false);
-    });
-});
-
-describe('gateVerdict', () => {
-    const blocking = [f('high', 'security')];
-    it('advisory (default shipped mode) never blocks, even with a blocking finding', () => {
-        expect(gateVerdict(blocking, { enforce: false })).toBe(0);
-    });
-    it('enforce blocks only on a merge-blocking finding', () => {
-        expect(gateVerdict(blocking, { enforce: true })).toBe(2);
-        expect(gateVerdict([f('critical', 'style')], { enforce: true })).toBe(0);
-        expect(gateVerdict([], { enforce: true })).toBe(0);
-    });
-});
-
-describe('isReviewablePath', () => {
-    it('skips generated projections + lockfiles, keeps source', () => {
-        expect(isReviewablePath('dist/agent-src/skills/x/SKILL.md')).toBe(false);
-        expect(isReviewablePath('.augment/rules/x.md')).toBe(false);
-        expect(isReviewablePath('.windsurfrules')).toBe(false);
-        expect(isReviewablePath('package-lock.json')).toBe(false);
-        expect(isReviewablePath('src/rules/x.md')).toBe(true);
-        expect(isReviewablePath('src/scripts/y.ts')).toBe(true);
-    });
-});
-
-describe('renderReview', () => {
-    it('carries the HUMAN REVIEW REQUIRED banner + the floor-not-human-review caveat', () => {
-        const out = renderReview([], false);
-        expect(out).toContain('HUMAN REVIEW REQUIRED');
-        expect(out).toContain('not** independent human review');
-    });
-    it('advisory phrasing says WOULD block, enforce phrasing says blocking', () => {
-        const blocking = [f('high', 'security')];
-        expect(renderReview(blocking, false)).toContain('WOULD block');
-        expect(renderReview(blocking, true)).toContain('merge-blocking');
-    });
-    it('labels each row (Blocking)/(Advisory) so a critical-but-non-blocking row never reads as inconsistent with the count', () => {
-        const findings = [f('critical', 'correctness'), f('critical', 'security')];
-        const out = renderReview(findings, false);
-        // critical × correctness is NOT blocking → labelled Advisory
-        expect(out).toContain('| critical (Advisory) | correctness |');
-        // critical × security IS blocking → labelled Blocking
-        expect(out).toContain('| critical (Blocking) | security |');
-        // verdict count matches the number of (Blocking) rows (exactly 1)
-        expect(out).toContain('1 finding(s) WOULD block');
-        expect((out.match(/\(Blocking\)/g) ?? []).length).toBe(1);
-    });
-});
-
-describe('escalationReasons', () => {
-    it('flags a large diff at or above the threshold', () => {
-        expect(escalationReasons(['src/a.ts'], LARGE_DIFF_LINES)).toEqual([
-            `large diff (${LARGE_DIFF_LINES} changed lines ≥ ${LARGE_DIFF_LINES})`,
-        ]);
-    });
-    it('does not flag a small non-claim diff', () => {
-        expect(escalationReasons(['src/a.ts'], LARGE_DIFF_LINES - 1)).toEqual([]);
-    });
-    it('flags a claim-ledger surface regardless of size', () => {
-        expect(escalationReasons(['docs/CLAIMS.md'], 1)).toEqual([
-            'claim-affecting surface touched (docs/CLAIMS.md)',
-        ]);
-        expect(escalationReasons(['docs/proof.md'], 1)[0]).toContain('docs/proof.md');
-        expect(escalationReasons(['docs/comparison.yaml'], 1)[0]).toContain('docs/comparison.yaml');
-        expect(escalationReasons(['README.md'], 1)[0]).toContain('README.md');
-    });
-    it('reports both reasons when a large diff also touches a claim surface', () => {
-        const r = escalationReasons(['docs/CLAIMS.md', 'src/big.ts'], LARGE_DIFF_LINES + 50);
-        expect(r).toHaveLength(2);
-        expect(r[0]).toContain('large diff');
-        expect(r[1]).toContain('claim-affecting');
-    });
-});
-
-describe('renderReview — escalation banner', () => {
-    const f = (severity: Finding['severity'], kind: Finding['kind']): Finding => ({
-        severity, kind, title: 't', detail: 'd',
-    });
-    it('appends the escalation recommendation when reasons are present', () => {
-        const out = renderReview([], false, ['large diff (500 changed lines ≥ 400)']);
-        expect(out).toContain('Escalation warranted');
-        expect(out).toContain('/council:pr');
-        expect(out).toContain('never');
-    });
-    it('omits the escalation block when there are no reasons (byte-identical to no-arg)', () => {
-        const findings = [f('high', 'security')];
-        expect(renderReview(findings, false, [])).toBe(renderReview(findings, false));
-    });
-    it('recommends escalation even on a clean (no-finding) large diff', () => {
-        const out = renderReview([], false, ['large diff (500 changed lines ≥ 400)']);
-        expect(out).toContain('no findings');
-        expect(out).toContain('Escalation warranted');
-    });
-});
-
-// ── Release-PR review mode (road-to-feedback-9.2.0-followups Phase 3) ──
-// See docs/design/release-pr-review-mode.md — anchored to the PR #957
-// false-advisory case (release features reading as "not in the diff").
-
-describe('detectReleaseVersion', () => {
-    const changelogAdd = (v: string) =>
-        `+## [${v}](https://github.com/event4u-app/agent-config/compare/9.1.0...${v}) (2026-07-14)`;
-    const packageAdd = (v: string) => `+    "version": "${v}",`;
-
-    it('detects a release PR when the changelog heading and the package.json bump agree', () => {
-        const patch = [
-            'diff --git a/CHANGELOG.md b/CHANGELOG.md',
-            '@@ -14,6 +14,8 @@',
-            ' ## [Unreleased]',
-            '+',
-            changelogAdd('9.2.0'),
-            'diff --git a/package.json b/package.json',
-            '@@ -1,6 +1,6 @@',
-            '-    "version": "9.1.0",',
-            packageAdd('9.2.0'),
-        ].join('\n');
-        expect(detectReleaseVersion(patch)).toBe('9.2.0');
-    });
-
-    it('is NOT a release PR when only the changelog heading changed (no version bump)', () => {
-        expect(detectReleaseVersion(changelogAdd('9.2.0'))).toBeNull();
-    });
-
-    it('is NOT a release PR when only package.json bumped (no changelog heading)', () => {
-        expect(detectReleaseVersion(packageAdd('9.2.0'))).toBeNull();
-    });
-
-    it('is NOT a release PR on a normal feature-PR patch (neither signal present)', () => {
-        const patch = ['+export function foo() {}', '-export function bar() {}'].join('\n');
-        expect(detectReleaseVersion(patch)).toBeNull();
-    });
-
-    it('refuses to guess when the changelog and package.json versions disagree', () => {
-        const patch = [changelogAdd('9.2.0'), packageAdd('9.3.0')].join('\n');
-        expect(detectReleaseVersion(patch)).toBeNull();
-    });
-
-    it('ignores removed lines (the old version) and reads only the added one', () => {
-        const patch = ['-    "version": "9.1.0",', packageAdd('9.2.0'), changelogAdd('9.2.0')].join('\n');
-        expect(detectReleaseVersion(patch)).toBe('9.2.0');
-    });
-});
-
-describe('pickPreviousTag', () => {
-    it('picks the highest tag strictly below the release version', () => {
-        expect(pickPreviousTag('9.2.0', ['9.0.0', '9.1.0', '9.2.0'])).toBe('9.1.0');
-    });
-    it('handles a v-prefixed tag naming convention', () => {
-        expect(pickPreviousTag('9.2.0', ['v9.0.0', 'v9.1.0'])).toBe('v9.1.0');
-    });
-    it('returns null when no tag is strictly below the version', () => {
-        expect(pickPreviousTag('9.2.0', ['9.2.0', '9.3.0'])).toBeNull();
-        expect(pickPreviousTag('9.2.0', [])).toBeNull();
-    });
-    it('ignores non-semver tags (branch-backup names etc.) instead of crashing', () => {
-        expect(
-            pickPreviousTag('9.2.0', [
-                'backup/pre-rebase-augment-limit-fit',
-                '9.1.0',
-                'local-squashed-tip-2026-05-18',
-            ]),
-        ).toBe('9.1.0');
-    });
-});
-
-describe('buildPlan — release-PR shape (PR #957 reproduction)', () => {
-    function git(root: string, ...args: string[]) {
-        return spawnSync('git', args, {
-            cwd: root,
-            encoding: 'utf8',
-            env: {
-                ...process.env,
-                GIT_AUTHOR_NAME: 'T',
-                GIT_AUTHOR_EMAIL: 't@e.test',
-                GIT_COMMITTER_NAME: 'T',
-                GIT_COMMITTER_EMAIL: 't@e.test',
-            },
-        });
-    }
-    function write(root: string, rel: string, content: string): void {
-        const full = path.join(root, rel);
-        fs.mkdirSync(path.dirname(full), { recursive: true });
-        fs.writeFileSync(full, content, 'utf-8');
-    }
-    function mkTmp(): string {
-        return fs.realpathSync(fs.mkdtempSync(path.join(os.tmpdir(), 'srg-')));
-    }
-
-    let root: string | undefined;
-    afterEach(() => {
-        if (root) fs.rmSync(root, { recursive: true, force: true });
-        root = undefined;
-    });
-
-    it('release mode sees the pre-cut feature file; the packaging diff alone does not (PR #957 shape)', () => {
-        root = mkTmp();
-        git(root, 'init', '-q', '-b', 'main');
-        write(root, 'package.json', '{\n  "name": "x",\n  "version": "9.0.0"\n}\n');
-        write(root, 'CHANGELOG.md', '# Changelog\n\n## [Unreleased]\n');
-        git(root, 'add', '-A');
-        git(root, 'commit', '-q', '-m', 'baseline');
-        git(root, 'tag', '9.1.0'); // the previous release tag
-
-        // The feature merges into main BEFORE the release cut — the PR #957 shape.
-        write(root, 'feature.txt', 'new cross-source-consistency rule content\n');
-        git(root, 'add', '-A');
-        git(root, 'commit', '-q', '-m', 'feat: cross-source-consistency');
-
-        // Release branch cut from main's current tip — no divergence yet, so
-        // main...release-branch only shows the release-cut commit below.
-        git(root, 'checkout', '-q', '-b', 'release-branch');
-        write(
-            root,
-            'CHANGELOG.md',
-            '# Changelog\n\n## [Unreleased]\n\n' +
-                '## [9.2.0](https://github.com/event4u-app/agent-config/compare/9.1.0...9.2.0) (2026-07-14)\n',
-        );
-        write(root, 'package.json', '{\n  "name": "x",\n  "version": "9.2.0"\n}\n');
-        git(root, 'add', '-A');
-        git(root, 'commit', '-q', '-m', 'chore(release): 9.2.0');
-
-        const plan = buildPlan('main', root);
-
-        expect(plan.release?.version).toBe('9.2.0');
-        expect(plan.release?.previousTag).toBe('9.1.0');
-        // The packaging diff (main...HEAD) alone does NOT show the pre-cut
-        // feature — the exact false-advisory shape PR #957 hit.
-        expect(plan.release?.packagingFiles).not.toContain('feature.txt');
-        expect(plan.release?.packagingFiles).toEqual(
-            expect.arrayContaining(['CHANGELOG.md', 'package.json']),
-        );
-        // Release mode's actual analysis range (previousTag...HEAD) DOES
-        // include it — this is what stops "feature not in diff" recurring.
-        expect(plan.files).toContain('feature.txt');
-        expect(plan.analysisBase).toBe('9.1.0');
-    });
-
-    it('a normal feature PR (no changelog/version bump) uses the unchanged base', () => {
-        root = mkTmp();
-        git(root, 'init', '-q', '-b', 'main');
-        write(root, 'package.json', '{\n  "name": "x",\n  "version": "9.1.0"\n}\n');
-        write(root, 'CHANGELOG.md', '# Changelog\n\n## [Unreleased]\n');
-        git(root, 'add', '-A');
-        git(root, 'commit', '-q', '-m', 'baseline');
-        git(root, 'tag', '9.1.0');
-
-        git(root, 'checkout', '-q', '-b', 'feature-branch');
-        write(root, 'feature.txt', 'some feature\n');
-        git(root, 'add', '-A');
-        git(root, 'commit', '-q', '-m', 'feat: add feature');
-
-        const plan = buildPlan('main', root);
-
-        expect(plan.release).toBeUndefined();
-        expect(plan.analysisBase).toBe('main');
-        expect(plan.files).toContain('feature.txt');
-    });
-});

package/src/shared/capabilities.test.ts

@@ -1,43 +0,0 @@
-/**
- * Unit tests for the host-facing capability advertisement.
- *
- * The `configRoot` and `embed` capabilities must appear in the
- * `--version --json` readout (and, via the same module, `GET /api/v1/ping`)
- * so a spawner can feature-detect support before relying on either.
- */
-import { describe, expect, it } from 'vitest';
-import { CAPABILITIES, buildVersionReadout } from './capabilities.js';
-
-describe('capabilities', () => {
-    it('advertises configRoot support', () => {
-        expect(CAPABILITIES.configRoot).toBe(true);
-    });
-
-    it('advertises the embed contract (reciprocal-ecosystem Phase 3)', () => {
-        expect(CAPABILITIES.embed).toEqual({
-            supported: true,
-            version: 1,
-            features: ['theme', 'deepLink'],
-        });
-    });
-
-    it('does NOT advertise the accent feature (v2, not v1)', () => {
-        expect(CAPABILITIES.embed.features).not.toContain('accent');
-    });
-
-    it('buildVersionReadout carries the version and the full capability block', () => {
-        const readout = buildVersionReadout('9.7.0');
-        expect(readout).toEqual({
-            version: '9.7.0',
-            capabilities: {
-                configRoot: true,
-                embed: { supported: true, version: 1, features: ['theme', 'deepLink'] },
-            },
-        });
-    });
-
-    it('buildVersionReadout copies capabilities (no shared mutable reference)', () => {
-        const readout = buildVersionReadout('1.2.3');
-        expect(readout.capabilities).not.toBe(CAPABILITIES);
-    });
-});