Follow-up to the posture-grade-stability fix in 66433fe. Audited every
test that spawns a CLI and found more of the same class: tests running
HOME-scoped scanners (SKL/COL) or the CLAUDE.md cascade against the
developer's real ~/.claude instead of an isolated HOME.
Fixed (env: hermeticEnv()):
- posture.test.mjs — runs full posture (SKL/COL/cascade); twin of
the posture-grade-stability leak, masked only
because its asserts are structural/relative
- drift-cli.test.mjs — ACTIVE bug: the CLI wrote baselines into the
real ~/.claude during the run (pollution); now
isolated, and afterEach cleanup wrapped in
withHermeticHome so it looks in the same HOME
- token-hotspots-cli.test.mjs — scan-orchestrator run executes SKL/COL on
real HOME; TOK reads the HOME cascade
- accurate-tokens.test.mjs — TOK reads the HOME cascade (kept the
ANTHROPIC_API_KEY deletion)
Proven safe, left as-is (no HOME-scoped scan affecting assertions, no
HOME writes): post-edit-verify.test.mjs (fast-path early-returns only),
fix-cli.test.mjs (output byte-identical real vs empty HOME — fixable
findings are project-local HKV/RUL/SET, never SKL/COL),
lint-default-output (caller already uses withHermeticHome).
Suite 875/875, no snapshot drift. No test regressed under isolation,
confirming none had a hidden real-HOME dependency.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ter3E2JSi1Khgmuf2kady8
136 lines
5.2 KiB
JavaScript
136 lines
5.2 KiB
JavaScript
import { describe, it, beforeEach } from 'node:test';
|
|
import assert from 'node:assert/strict';
|
|
import { resolve } from 'node:path';
|
|
import { fileURLToPath } from 'node:url';
|
|
import { execFile } from 'node:child_process';
|
|
import { promisify } from 'node:util';
|
|
import { hermeticEnv } from '../helpers/hermetic-home.mjs';
|
|
|
|
const exec = promisify(execFile);
|
|
const __dirname = fileURLToPath(new URL('.', import.meta.url));
|
|
const FIXTURES = resolve(__dirname, '../fixtures');
|
|
const POSTURE_BIN = resolve(__dirname, '../../scanners/posture.mjs');
|
|
|
|
async function runPosture(args) {
|
|
// Isolate HOME: posture runs the HOME-scoped SKL/COL scanners + CLAUDE.md
|
|
// cascade, so a real ~/.claude would leak the developer's plugins/skills
|
|
// into grades and counts. See tests/helpers/hermetic-home.mjs.
|
|
const { stdout, stderr } = await exec('node', [POSTURE_BIN, ...args], {
|
|
timeout: 30000,
|
|
cwd: resolve(__dirname, '../..'),
|
|
env: hermeticEnv(),
|
|
});
|
|
return { stdout, stderr };
|
|
}
|
|
|
|
async function runPostureJson(fixturePath) {
|
|
const { stdout } = await runPosture([fixturePath, '--json']);
|
|
return JSON.parse(stdout);
|
|
}
|
|
|
|
describe('posture.mjs CLI — healthy project', () => {
|
|
let result;
|
|
beforeEach(async () => {
|
|
result = await runPostureJson(resolve(FIXTURES, 'healthy-project'));
|
|
});
|
|
|
|
it('returns utilization with score and overhang', () => {
|
|
assert.ok(typeof result.utilization.score === 'number');
|
|
assert.ok(typeof result.utilization.overhang === 'number');
|
|
assert.equal(result.utilization.score + result.utilization.overhang, 100);
|
|
});
|
|
|
|
it('returns maturity level >= 2', () => {
|
|
assert.ok(result.maturity.level >= 2);
|
|
assert.ok(typeof result.maturity.name === 'string');
|
|
});
|
|
|
|
it('returns segment string', () => {
|
|
assert.ok(typeof result.segment.segment === 'string');
|
|
assert.ok(result.segment.segment.length > 0);
|
|
});
|
|
|
|
it('returns 10 area scores (v5 adds Plugin Hygiene from COL)', () => {
|
|
assert.equal(result.areas.length, 10);
|
|
for (const area of result.areas) {
|
|
assert.ok('id' in area);
|
|
assert.ok('name' in area);
|
|
assert.ok('grade' in area);
|
|
assert.ok('score' in area);
|
|
assert.ok('findingCount' in area);
|
|
}
|
|
});
|
|
|
|
it('exposes a token_efficiency area id', () => {
|
|
const te = result.areas.find(a => a.id === 'token_efficiency');
|
|
assert.ok(te, 'token_efficiency id present');
|
|
});
|
|
|
|
it('returns overallGrade', () => {
|
|
assert.ok(['A', 'B', 'C', 'D', 'F'].includes(result.overallGrade));
|
|
});
|
|
|
|
it('includes topActions array', () => {
|
|
assert.ok(Array.isArray(result.topActions));
|
|
});
|
|
|
|
it('includes scannerEnvelope', () => {
|
|
assert.ok(result.scannerEnvelope.meta);
|
|
assert.ok(result.scannerEnvelope.scanners);
|
|
assert.ok(result.scannerEnvelope.aggregate);
|
|
});
|
|
});
|
|
|
|
describe('posture.mjs CLI — minimal project', () => {
|
|
it('scores lower utilization than healthy', async () => {
|
|
const healthy = await runPostureJson(resolve(FIXTURES, 'healthy-project'));
|
|
const minimal = await runPostureJson(resolve(FIXTURES, 'minimal-project'));
|
|
assert.ok(minimal.utilization.score < healthy.utilization.score,
|
|
`minimal (${minimal.utilization.score}) should be < healthy (${healthy.utilization.score})`);
|
|
});
|
|
|
|
it('has lower maturity than healthy', async () => {
|
|
const healthy = await runPostureJson(resolve(FIXTURES, 'healthy-project'));
|
|
const minimal = await runPostureJson(resolve(FIXTURES, 'minimal-project'));
|
|
assert.ok(minimal.maturity.level <= healthy.maturity.level);
|
|
});
|
|
});
|
|
|
|
describe('posture.mjs CLI — terminal output (v3 health format)', () => {
|
|
// These assertions verify the v5.0.0 verbatim scorecard prose. Default mode
|
|
// is humanized as of v5.1.0 (Wave 3); --raw is the explicit v5.0.0 path.
|
|
it('scorecard contains health sections (v5.0.0 verbatim via --raw)', async () => {
|
|
const { stderr } = await runPosture([resolve(FIXTURES, 'healthy-project'), '--raw']);
|
|
assert.ok(stderr.includes('Config-Audit Health Score'));
|
|
assert.ok(stderr.includes('Health:'));
|
|
assert.ok(stderr.includes('Area Scores'));
|
|
assert.ok(stderr.includes('areas scanned'));
|
|
});
|
|
|
|
it('scorecard does NOT contain legacy metrics', async () => {
|
|
const { stderr } = await runPosture([resolve(FIXTURES, 'healthy-project'), '--raw']);
|
|
assert.ok(!stderr.includes('Maturity:'));
|
|
assert.ok(!stderr.includes('Utilization:'));
|
|
assert.ok(!stderr.includes('Segment:'));
|
|
});
|
|
|
|
it('scorecard excludes Feature Coverage from area display', async () => {
|
|
const { stderr } = await runPosture([resolve(FIXTURES, 'healthy-project'), '--raw']);
|
|
assert.ok(!stderr.includes('Feature Coverage'));
|
|
});
|
|
});
|
|
|
|
describe('posture.mjs CLI — JSON includes opportunityCount', () => {
|
|
it('returns opportunityCount field', async () => {
|
|
const result = await runPostureJson(resolve(FIXTURES, 'healthy-project'));
|
|
assert.ok(typeof result.opportunityCount === 'number');
|
|
assert.ok(result.opportunityCount >= 0);
|
|
});
|
|
|
|
it('JSON still includes legacy fields for backward compat', async () => {
|
|
const result = await runPostureJson(resolve(FIXTURES, 'healthy-project'));
|
|
assert.ok(typeof result.utilization.score === 'number');
|
|
assert.ok(typeof result.maturity.level === 'number');
|
|
assert.ok(typeof result.segment.segment === 'string');
|
|
});
|
|
});
|