habit-hooks 0.2.0 → 1.0.0

package/dist/sensors/runner.js

@@ -1,94 +0,0 @@
-function assertUniqueIds(sensors) {
-    const seen = new Set();
-    for (const sensor of sensors) {
-        if (seen.has(sensor.id))
-            throw new Error(`duplicate sensor id: ${sensor.id}`);
-        seen.add(sensor.id);
-    }
-}
-function buildProducers(sensors) {
-    const producers = new Map();
-    for (const sensor of sensors) {
-        for (const smell of sensor.produces) {
-            const list = producers.get(smell) ?? [];
-            list.push(sensor);
-            producers.set(smell, list);
-        }
-    }
-    return producers;
-}
-function dependencyKeys(sensor) {
-    return sensor.dependsOn ?? [];
-}
-// Drop a multi sensor whose depended-on smells are not produced by any other
-// sensor in the set — e.g. a composite whose inputs were gated out by config.
-// This keeps SensorRunner construction (which throws on unproduced deps) from
-// crashing a run when a dependency producer is simply inactive.
-export function satisfiableSensors(sensors) {
-    return sensors.filter((sensor) => {
-        const others = sensors.filter((other) => other.id !== sensor.id);
-        const produced = new Set(others.flatMap((other) => other.produces));
-        return dependencyKeys(sensor).every((smell) => produced.has(smell));
-    });
-}
-function assertSatisfiable(sensor, producers) {
-    for (const smell of dependencyKeys(sensor)) {
-        const others = (producers.get(smell) ?? []).filter((p) => p.id !== sensor.id);
-        if (others.length === 0) {
-            throw new Error(`sensor '${sensor.id}' depends on unproduced smell: ${smell}`);
-        }
-    }
-}
-function predecessorsOf(sensor, producers) {
-    const seen = new Map();
-    for (const smell of dependencyKeys(sensor)) {
-        for (const producer of producers.get(smell) ?? []) {
-            if (producer.id !== sensor.id)
-                seen.set(producer.id, producer);
-        }
-    }
-    return [...seen.values()];
-}
-function isReady(sensor, placed, producers) {
-    return predecessorsOf(sensor, producers).every((p) => placed.has(p.id));
-}
-function orderSensors(sensors, producers) {
-    const order = [];
-    const placed = new Set();
-    while (order.length < sensors.length) {
-        const next = sensors.find((s) => !placed.has(s.id) && isReady(s, placed, producers));
-        if (next === undefined)
-            throw new Error('sensor dependency cycle detected');
-        order.push(next);
-        placed.add(next.id);
-    }
-    return order;
-}
-function gatherDeps(sensor, issues) {
-    const wanted = new Set(dependencyKeys(sensor));
-    if (wanted.size === 0)
-        return [];
-    return issues.filter((issue) => wanted.has(issue.smell));
-}
-export class SensorRunner {
-    ordered;
-    constructor(sensors) {
-        assertUniqueIds(sensors);
-        const producers = buildProducers(sensors);
-        for (const sensor of sensors)
-            assertSatisfiable(sensor, producers);
-        this.ordered = orderSensors(sensors, producers);
-    }
-    get sensors() {
-        return [...this.ordered];
-    }
-    async run(input) {
-        const issues = [];
-        for (const sensor of this.ordered) {
-            const ctx = { files: input.files, cwd: input.cwd, deps: gatherDeps(sensor, issues) };
-            issues.push(...(await sensor.run(ctx)));
-        }
-        return issues;
-    }
-}
-//# sourceMappingURL=runner.js.map

package/dist/sensors/runner.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"runner.js","sourceRoot":"","sources":["../../src/sensors/runner.ts"],"names":[],"mappings":"AAYA,SAAS,eAAe,CAAC,OAAiB;IACxC,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;IAC/B,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,IAAI,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC;YAAE,MAAM,IAAI,KAAK,CAAC,wBAAwB,MAAM,CAAC,EAAE,EAAE,CAAC,CAAC;QAC9E,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;IACtB,CAAC;AACH,CAAC;AAED,SAAS,cAAc,CAAC,OAAiB;IACvC,MAAM,SAAS,GAAG,IAAI,GAAG,EAAoB,CAAC;IAC9C,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,QAAQ,EAAE,CAAC;YACpC,MAAM,IAAI,GAAG,SAAS,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;YACxC,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;YAClB,SAAS,CAAC,GAAG,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC;QAC7B,CAAC;IACH,CAAC;IACD,OAAO,SAAS,CAAC;AACnB,CAAC;AAED,SAAS,cAAc,CAAC,MAAc;IACpC,OAAO,MAAM,CAAC,SAAS,IAAI,EAAE,CAAC;AAChC,CAAC;AAED,6EAA6E;AAC7E,8EAA8E;AAC9E,8EAA8E;AAC9E,gEAAgE;AAChE,MAAM,UAAU,kBAAkB,CAAC,OAAiB;IAClD,OAAO,OAAO,CAAC,MAAM,CAAC,CAAC,MAAM,EAAE,EAAE;QAC/B,MAAM,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,KAAK,MAAM,CAAC,EAAE,CAAC,CAAC;QACjE,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC;QACpE,OAAO,cAAc,CAAC,MAAM,CAAC,CAAC,KAAK,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC;IACtE,CAAC,CAAC,CAAC;AACL,CAAC;AAED,SAAS,iBAAiB,CAAC,MAAc,EAAE,SAAgC;IACzE,KAAK,MAAM,KAAK,IAAI,cAAc,CAAC,MAAM,CAAC,EAAE,CAAC;QAC3C,MAAM,MAAM,GAAG,CAAC,SAAS,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,MAAM,CAAC,EAAE,CAAC,CAAC;QAC9E,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACxB,MAAM,IAAI,KAAK,CAAC,WAAW,MAAM,CAAC,EAAE,kCAAkC,KAAK,EAAE,CAAC,CAAC;QACjF,CAAC;IACH,CAAC;AACH,CAAC;AAED,SAAS,cAAc,CAAC,MAAc,EAAE,SAAgC;IACtE,MAAM,IAAI,GAAG,IAAI,GAAG,EAAkB,CAAC;IACvC,KAAK,MAAM,KAAK,IAAI,cAAc,CAAC,MAAM,CAAC,EAAE,CAAC;QAC3C,KAAK,MAAM,QAAQ,IAAI,SAAS,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,EAAE,CAAC;YAClD,IAAI,QAAQ,CAAC,EAAE,KAAK,MAAM,CAAC,EAAE;gBAAE,IAAI,CAAC,GAAG,CAAC,QAAQ,CAAC,EAAE,EAAE,QAAQ,CAAC,CAAC;QACjE,CAAC;IACH,CAAC;IACD,OAAO,CAAC,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC;AAC5B,CAAC;AAED,SAAS,OAAO,CAAC,MAAc,EAAE,MAAmB,EAAE,SAAgC;IACpF,OAAO,cAAc,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AAC1E,CAAC;AAED,SAAS,YAAY,CAAC,OAAiB,EAAE,SAAgC;IACvE,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,MAAM,MAAM,GAAG,IAAI,GAAG,EAAU,CAAC;IACjC,OAAO,KAAK,CAAC,MAAM,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC;QACrC,MAAM,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,IAAI,OAAO,CAAC,CAAC,EAAE,MAAM,EAAE,SAAS,CAAC,CAAC,CAAC;QACrF,IAAI,IAAI,KAAK,SAAS;YAAE,MAAM,IAAI,KAAK,CAAC,kCAAkC,CAAC,CAAC;QAC5E,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACjB,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACtB,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAS,UAAU,CAAC,MAAc,EAAE,MAAe;IACjD,MAAM,MAAM,GAAG,IAAI,GAAG,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC,CAAC;IAC/C,IAAI,MAAM,CAAC,IAAI,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IACjC,OAAO,MAAM,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC;AAC3D,CAAC;AAED,MAAM,OAAO,YAAY;IACN,OAAO,CAAW;IAEnC,YAAY,OAAiB;QAC3B,eAAe,CAAC,OAAO,CAAC,CAAC;QACzB,MAAM,SAAS,GAAG,cAAc,CAAC,OAAO,CAAC,CAAC;QAC1C,KAAK,MAAM,MAAM,IAAI,OAAO;YAAE,iBAAiB,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;QACnE,IAAI,CAAC,OAAO,GAAG,YAAY,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;IAClD,CAAC;IAED,IAAI,OAAO;QACT,OAAO,CAAC,GAAG,IAAI,CAAC,OAAO,CAAC,CAAC;IAC3B,CAAC;IAED,KAAK,CAAC,GAAG,CAAC,KAAqB;QAC7B,MAAM,MAAM,GAAY,EAAE,CAAC;QAC3B,KAAK,MAAM,MAAM,IAAI,IAAI,CAAC,OAAO,EAAE,CAAC;YAClC,MAAM,GAAG,GAAkB,EAAE,KAAK,EAAE,KAAK,CAAC,KAAK,EAAE,GAAG,EAAE,KAAK,CAAC,GAAG,EAAE,IAAI,EAAE,UAAU,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,CAAC;YACpG,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,MAAM,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;QAC1C,CAAC;QACD,OAAO,MAAM,CAAC;IAChB,CAAC;CACF"}

package/dist/sensors/types.d.ts

@@ -1,15 +0,0 @@
-export interface Issue {
-    smell: string;
-    details: Record<string, unknown>;
-}
-export interface SensorContext {
-    files: string[];
-    cwd: string;
-    deps: Issue[];
-}
-export interface Sensor {
-    id: string;
-    produces: string[];
-    dependsOn?: string[];
-    run(_ctx: SensorContext): Promise<Issue[]>;
-}

package/dist/sensors/types.js

@@ -1,5 +0,0 @@
-// The sensor stage of the sensor -> mapper -> guide pipeline (docs/sensors.md).
-// A sensor finds smells and appends Issues to the bag, translating its tool's
-// raw output into canonical kebab-case smell keys.
-export {};
-//# sourceMappingURL=types.js.map

package/dist/sensors/types.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"types.js","sourceRoot":"","sources":["../../src/sensors/types.ts"],"names":[],"mappings":"AAAA,gFAAgF;AAChF,8EAA8E;AAC9E,mDAAmD"}

package/dist/types.d.ts

@@ -1,44 +0,0 @@
-export type Severity = 'enforced' | 'suggested';
-export type RuleSource = 'eslint' | 'jscpd' | 'knip' | 'custom';
-export interface CommentCheckThresholds {
-    maxSingleLineChars: number;
-    maxBlockChars: number;
-}
-export interface Rule {
-    id: string;
-    source: RuleSource;
-    sourceRuleId?: string;
-    severity: Severity;
-    changedFilesOnly?: boolean;
-    title: string;
-    description: string;
-    include?: string[];
-    exclude?: string[];
-    fix?: string;
-    guidance?: string;
-    commentCheck?: CommentCheckThresholds;
-}
-export interface CoachingPrompt {
-    id: string;
-    title: string;
-    description: string;
-    severity: Severity;
-    guidancePath: string;
-}
-export interface Violation {
-    ruleId: string;
-    file: string;
-    line: number;
-    column?: number;
-    message: string;
-    source?: string;
-}
-export interface CheckOutcome {
-    violations: Violation[];
-    stderr?: string[];
-    failures?: string[];
-}
-export interface Check {
-    id: string;
-    run(_files: string[], _rules: Rule[], _cwd?: string): Promise<Violation[] | CheckOutcome>;
-}

package/dist/types.js

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=types.js.map

package/dist/types.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"types.js","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":""}

package/dist/wrap/notices.d.ts

@@ -1,19 +0,0 @@
-import type { ShellResult } from './shell.js';
-import type { CheckOutcome } from '../types.js';
-export interface BinResolution {
-    binPath: string;
-    isFallback: boolean;
-}
-export declare function fallbackNotice(tool: string, cwd: string): string;
-export declare function spawnFailureWarning(tool: string, cwd: string, warnings: string[]): string;
-export declare function firstLine(text: string): string;
-export declare function isSpawnFailure(result: ShellResult): boolean;
-export declare function emptyOutcome(stderr: string[]): CheckOutcome;
-export interface SensorSink {
-    notices: string[];
-    failures: string[];
-}
-export declare function spawnFailureOutcome(notices: string[], message: string): CheckOutcome;
-export declare function recordSpawnFailure(sink: SensorSink, message: string): void;
-export declare function noticesFor(tool: string, resolution: BinResolution, cwd: string): string[];
-export declare function absolutize(cwd: string, file: string): string;

package/dist/wrap/notices.js

@@ -1,38 +0,0 @@
-import { isAbsolute, join } from 'node:path';
-export function fallbackNotice(tool, cwd) {
-    return `habit-hooks: using bundled ${tool} (no ${tool} installation found in ${cwd})`;
-}
-export function spawnFailureWarning(tool, cwd, warnings) {
-    const detail = warnings.length > 0 ? warnings.join('; ') : 'spawn failure';
-    return `habit-hooks: ${tool} skipped in ${cwd} (${detail})`;
-}
-export function firstLine(text) {
-    for (const line of text.split('\n')) {
-        const trimmed = line.trim();
-        if (trimmed.length > 0)
-            return trimmed;
-    }
-    return '';
-}
-export function isSpawnFailure(result) {
-    return result.exitCode === -1;
-}
-export function emptyOutcome(stderr) {
-    return { violations: [], stderr };
-}
-// A sensor that could not spawn or timed out: the message is shown (stderr) and
-// recorded as a failure so the run fails (exit 1), per docs/sensors.md.
-export function spawnFailureOutcome(notices, message) {
-    return { violations: [], stderr: [...notices, message], failures: [message] };
-}
-export function recordSpawnFailure(sink, message) {
-    sink.notices.push(message);
-    sink.failures.push(message);
-}
-export function noticesFor(tool, resolution, cwd) {
-    return resolution.isFallback ? [fallbackNotice(tool, cwd)] : [];
-}
-export function absolutize(cwd, file) {
-    return isAbsolute(file) ? file : join(cwd, file);
-}
-//# sourceMappingURL=notices.js.map

package/dist/wrap/notices.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"notices.js","sourceRoot":"","sources":["../../src/wrap/notices.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AAS7C,MAAM,UAAU,cAAc,CAAC,IAAY,EAAE,GAAW;IACtD,OAAO,8BAA8B,IAAI,QAAQ,IAAI,0BAA0B,GAAG,GAAG,CAAC;AACxF,CAAC;AAED,MAAM,UAAU,mBAAmB,CAAC,IAAY,EAAE,GAAW,EAAE,QAAkB;IAC/E,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,eAAe,CAAC;IAC3E,OAAO,gBAAgB,IAAI,eAAe,GAAG,KAAK,MAAM,GAAG,CAAC;AAC9D,CAAC;AAED,MAAM,UAAU,SAAS,CAAC,IAAY;IACpC,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;QACpC,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,CAAC;QAC5B,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC;YAAE,OAAO,OAAO,CAAC;IACzC,CAAC;IACD,OAAO,EAAE,CAAC;AACZ,CAAC;AAED,MAAM,UAAU,cAAc,CAAC,MAAmB;IAChD,OAAO,MAAM,CAAC,QAAQ,KAAK,CAAC,CAAC,CAAC;AAChC,CAAC;AAED,MAAM,UAAU,YAAY,CAAC,MAAgB;IAC3C,OAAO,EAAE,UAAU,EAAE,EAAE,EAAE,MAAM,EAAE,CAAC;AACpC,CAAC;AAUD,gFAAgF;AAChF,wEAAwE;AACxE,MAAM,UAAU,mBAAmB,CAAC,OAAiB,EAAE,OAAe;IACpE,OAAO,EAAE,UAAU,EAAE,EAAE,EAAE,MAAM,EAAE,CAAC,GAAG,OAAO,EAAE,OAAO,CAAC,EAAE,QAAQ,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC;AAChF,CAAC;AAED,MAAM,UAAU,kBAAkB,CAAC,IAAgB,EAAE,OAAe;IAClE,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;IAC3B,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;AAC9B,CAAC;AAED,MAAM,UAAU,UAAU,CAAC,IAAY,EAAE,UAAyB,EAAE,GAAW;IAC7E,OAAO,UAAU,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;AAClE,CAAC;AAED,MAAM,UAAU,UAAU,CAAC,GAAW,EAAE,IAAY;IAClD,OAAO,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;AACnD,CAAC"}

package/dist/wrap/resolve.d.ts

@@ -1,5 +0,0 @@
-export declare function requiresNodeRuntime(binPath: string): boolean;
-export declare function spawnTarget(binPath: string, args: string[]): {
-    bin: string;
-    args: string[];
-};

package/dist/wrap/resolve.js

@@ -1,9 +0,0 @@
-export function requiresNodeRuntime(binPath) {
-    return binPath.endsWith('.js');
-}
-export function spawnTarget(binPath, args) {
-    if (requiresNodeRuntime(binPath))
-        return { bin: process.execPath, args: [binPath, ...args] };
-    return { bin: binPath, args };
-}
-//# sourceMappingURL=resolve.js.map

package/dist/wrap/resolve.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"resolve.js","sourceRoot":"","sources":["../../src/wrap/resolve.ts"],"names":[],"mappings":"AAAA,MAAM,UAAU,mBAAmB,CAAC,OAAe;IACjD,OAAO,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;AACjC,CAAC;AAED,MAAM,UAAU,WAAW,CAAC,OAAe,EAAE,IAAc;IACzD,IAAI,mBAAmB,CAAC,OAAO,CAAC;QAAE,OAAO,EAAE,GAAG,EAAE,OAAO,CAAC,QAAQ,EAAE,IAAI,EAAE,CAAC,OAAO,EAAE,GAAG,IAAI,CAAC,EAAE,CAAC;IAC7F,OAAO,EAAE,GAAG,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC;AAChC,CAAC"}

package/dist/wrap/run.d.ts

@@ -1,17 +0,0 @@
-import { type ShellResult } from './shell.js';
-import { type BinResolution } from './notices.js';
-import type { CheckOutcome } from '../types.js';
-interface SpawnSkip {
-    skipWarning: string;
-}
-export declare function skipOutcome(skip: SpawnSkip, notices: string[]): CheckOutcome;
-interface SpawnWrappedArgs {
-    tool: string;
-    resolution: BinResolution;
-    cwd: string;
-    args: string[];
-}
-export declare function spawnWrapped(spec: SpawnWrappedArgs): Promise<ShellResult | SpawnSkip>;
-export declare function isSpawnSkip(value: ShellResult | SpawnSkip): value is SpawnSkip;
-export declare function parseJsonStdout<T>(stdout: string, startsWith: '{' | '['): T | null;
-export {};

package/dist/wrap/run.js

@@ -1,31 +0,0 @@
-import { runTool } from './shell.js';
-import { isSpawnFailure, spawnFailureOutcome, spawnFailureWarning } from './notices.js';
-import { spawnTarget } from './resolve.js';
-// A spawn/timeout skip becomes a failing outcome: shown to the user and recorded
-// as a failure so the run fails (exit 1), per docs/sensors.md.
-export function skipOutcome(skip, notices) {
-    return spawnFailureOutcome(notices, skip.skipWarning);
-}
-export async function spawnWrapped(spec) {
-    const { tool, resolution, cwd, args } = spec;
-    const target = spawnTarget(resolution.binPath, args);
-    const result = await runTool({ bin: target.bin, args: target.args, cwd });
-    if (isSpawnFailure(result))
-        return { skipWarning: spawnFailureWarning(tool, cwd, result.warnings) };
-    return result;
-}
-export function isSpawnSkip(value) {
-    return value.skipWarning !== undefined;
-}
-export function parseJsonStdout(stdout, startsWith) {
-    const trimmed = stdout.trim();
-    if (trimmed.length === 0 || !trimmed.startsWith(startsWith))
-        return null;
-    try {
-        return JSON.parse(trimmed);
-    }
-    catch {
-        return null;
-    }
-}
-//# sourceMappingURL=run.js.map

package/dist/wrap/run.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"run.js","sourceRoot":"","sources":["../../src/wrap/run.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAoB,MAAM,YAAY,CAAC;AACvD,OAAO,EAAE,cAAc,EAAE,mBAAmB,EAAE,mBAAmB,EAAsB,MAAM,cAAc,CAAC;AAC5G,OAAO,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAO3C,iFAAiF;AACjF,+DAA+D;AAC/D,MAAM,UAAU,WAAW,CAAC,IAAe,EAAE,OAAiB;IAC5D,OAAO,mBAAmB,CAAC,OAAO,EAAE,IAAI,CAAC,WAAW,CAAC,CAAC;AACxD,CAAC;AASD,MAAM,CAAC,KAAK,UAAU,YAAY,CAAC,IAAsB;IACvD,MAAM,EAAE,IAAI,EAAE,UAAU,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,IAAI,CAAC;IAC7C,MAAM,MAAM,GAAG,WAAW,CAAC,UAAU,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC;IACrD,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,EAAE,GAAG,EAAE,MAAM,CAAC,GAAG,EAAE,IAAI,EAAE,MAAM,CAAC,IAAI,EAAE,GAAG,EAAE,CAAC,CAAC;IAC1E,IAAI,cAAc,CAAC,MAAM,CAAC;QAAE,OAAO,EAAE,WAAW,EAAE,mBAAmB,CAAC,IAAI,EAAE,GAAG,EAAE,MAAM,CAAC,QAAQ,CAAC,EAAE,CAAC;IACpG,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,MAAM,UAAU,WAAW,CAAC,KAA8B;IACxD,OAAQ,KAAmB,CAAC,WAAW,KAAK,SAAS,CAAC;AACxD,CAAC;AAED,MAAM,UAAU,eAAe,CAAI,MAAc,EAAE,UAAqB;IACtE,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,EAAE,CAAC;IAC9B,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,UAAU,CAAC;QAAE,OAAO,IAAI,CAAC;IACzE,IAAI,CAAC;QACH,OAAO,IAAI,CAAC,KAAK,CAAC,OAAO,CAAM,CAAC;IAClC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC"}

package/dist/wrap/shell.d.ts

@@ -1,15 +0,0 @@
-export interface ShellResult {
-    stdout: string;
-    stderr: string;
-    exitCode: number;
-    warnings: string[];
-}
-interface RunToolOptions {
-    bin: string;
-    args: string[];
-    cwd: string;
-    timeoutMs?: number;
-    stdin?: string;
-}
-export declare function runTool(opts: RunToolOptions): Promise<ShellResult>;
-export {};

package/dist/wrap/shell.js

@@ -1,50 +0,0 @@
-import { spawn } from 'node:child_process';
-const DEFAULT_TIMEOUT_MS = 5 * 60 * 1000;
-function emptyFailure(warning) {
-    return { stdout: '', stderr: '', exitCode: -1, warnings: [warning] };
-}
-function collectStreams(child) {
-    const buffers = { stdout: [], stderr: [] };
-    child.stdout.on('data', (chunk) => buffers.stdout.push(chunk));
-    child.stderr.on('data', (chunk) => buffers.stderr.push(chunk));
-    return buffers;
-}
-function writeStdin(child, stdin) {
-    if (stdin === undefined)
-        return;
-    child.stdin.on('error', () => undefined);
-    child.stdin.end(stdin);
-}
-function settleOnce(settler, result) {
-    if (settler.done.value)
-        return;
-    settler.done.value = true;
-    settler.clearTimer();
-    settler.resolve(result);
-}
-function closeResult(info) {
-    if (info.signal !== null)
-        return emptyFailure(`${info.bin} terminated by signal ${info.signal}`);
-    const stdout = Buffer.concat(info.buffers.stdout).toString('utf8');
-    const stderr = Buffer.concat(info.buffers.stderr).toString('utf8');
-    return { stdout, stderr, exitCode: info.code ?? -1, warnings: [] };
-}
-function startKillTimer(child, onTimeout, timeoutMs) {
-    return setTimeout(() => {
-        child.kill('SIGKILL');
-        onTimeout();
-    }, timeoutMs);
-}
-export function runTool(opts) {
-    const timeoutMs = opts.timeoutMs ?? DEFAULT_TIMEOUT_MS;
-    return new Promise((resolve) => {
-        const child = spawn(opts.bin, opts.args, { cwd: opts.cwd });
-        const buffers = collectStreams(child);
-        writeStdin(child, opts.stdin);
-        const timer = startKillTimer(child, () => settleOnce(settler, emptyFailure(`${opts.bin} timed out after ${timeoutMs}ms`)), timeoutMs);
-        const settler = { resolve, done: { value: false }, clearTimer: () => clearTimeout(timer) };
-        child.on('error', (e) => settleOnce(settler, emptyFailure(`${opts.bin} failed to spawn: ${e.message}`)));
-        child.on('close', (code, signal) => settleOnce(settler, closeResult({ buffers, bin: opts.bin, code, signal })));
-    });
-}
-//# sourceMappingURL=shell.js.map

package/dist/wrap/shell.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"shell.js","sourceRoot":"","sources":["../../src/wrap/shell.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAuC,MAAM,oBAAoB,CAAC;AA4BhF,MAAM,kBAAkB,GAAG,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC;AAEzC,SAAS,YAAY,CAAC,OAAe;IACnC,OAAO,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC,CAAC,EAAE,QAAQ,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC;AACvE,CAAC;AAED,SAAS,cAAc,CAAC,KAAqC;IAC3D,MAAM,OAAO,GAAkB,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;IAC1D,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,KAAa,EAAE,EAAE,CAAC,OAAO,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC;IACvE,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,KAAa,EAAE,EAAE,CAAC,OAAO,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC;IACvE,OAAO,OAAO,CAAC;AACjB,CAAC;AAED,SAAS,UAAU,CAAC,KAAqC,EAAE,KAAyB;IAClF,IAAI,KAAK,KAAK,SAAS;QAAE,OAAO;IAChC,KAAK,CAAC,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,EAAE,CAAC,SAAS,CAAC,CAAC;IACzC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;AACzB,CAAC;AAED,SAAS,UAAU,CAAC,OAAgB,EAAE,MAAmB;IACvD,IAAI,OAAO,CAAC,IAAI,CAAC,KAAK;QAAE,OAAO;IAC/B,OAAO,CAAC,IAAI,CAAC,KAAK,GAAG,IAAI,CAAC;IAC1B,OAAO,CAAC,UAAU,EAAE,CAAC;IACrB,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;AAC1B,CAAC;AASD,SAAS,WAAW,CAAC,IAAe;IAClC,IAAI,IAAI,CAAC,MAAM,KAAK,IAAI;QAAE,OAAO,YAAY,CAAC,GAAG,IAAI,CAAC,GAAG,yBAAyB,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC;IACjG,MAAM,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;IACnE,MAAM,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;IACnE,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,QAAQ,EAAE,IAAI,CAAC,IAAI,IAAI,CAAC,CAAC,EAAE,QAAQ,EAAE,EAAE,EAAE,CAAC;AACrE,CAAC;AAED,SAAS,cAAc,CAAC,KAAqC,EAAE,SAAqB,EAAE,SAAiB;IACrG,OAAO,UAAU,CAAC,GAAG,EAAE;QACrB,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACtB,SAAS,EAAE,CAAC;IACd,CAAC,EAAE,SAAS,CAAC,CAAC;AAChB,CAAC;AAED,MAAM,UAAU,OAAO,CAAC,IAAoB;IAC1C,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,IAAI,kBAAkB,CAAC;IACvD,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE;QAC7B,MAAM,KAAK,GAAG,KAAK,CAAC,IAAI,CAAC,GAAG,EAAE,IAAI,CAAC,IAAI,EAAE,EAAE,GAAG,EAAE,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC;QAC5D,MAAM,OAAO,GAAG,cAAc,CAAC,KAAK,CAAC,CAAC;QACtC,UAAU,CAAC,KAAK,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC;QAC9B,MAAM,KAAK,GAAG,cAAc,CAAC,KAAK,EAAE,GAAG,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,YAAY,CAAC,GAAG,IAAI,CAAC,GAAG,oBAAoB,SAAS,IAAI,CAAC,CAAC,EAAE,SAAS,CAAC,CAAC;QACtI,MAAM,OAAO,GAAY,EAAE,OAAO,EAAE,IAAI,EAAE,EAAE,KAAK,EAAE,KAAK,EAAE,EAAE,UAAU,EAAE,GAAG,EAAE,CAAC,YAAY,CAAC,KAAK,CAAC,EAAE,CAAC;QACpG,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,YAAY,CAAC,GAAG,IAAI,CAAC,GAAG,qBAAqB,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC,CAAC,CAAC;QACzG,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,WAAW,CAAC,EAAE,OAAO,EAAE,GAAG,EAAE,IAAI,CAAC,GAAG,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC,CAAC,CAAC,CAAC;IAClH,CAAC,CAAC,CAAC;AACL,CAAC"}

package/src/prompts/deep-nesting.md

@@ -1,13 +0,0 @@
-Deeply nested blocks — `if` inside `for` inside `try` inside `if` — force the reader to hold every enclosing condition in their head at once. The smell is not the indentation; it is that the function is doing too much branching in one place.
-
-Read the nesting from the inside out and ask what the innermost block actually needs. Usually most of the enclosing conditions are *guards* — preconditions that should be checked and bailed on early, not wrapped around the real work.
-
-Prefer, in order:
-
-1. **Guard clauses / early returns.** Invert a condition and `return` (or `continue`/`throw`) early so the happy path stays at the top indentation level. Each guard you lift removes one level of nesting from everything below it.
-2. **Extract a helper.** When an inner block is a coherent sub-step, pull it into a named function. The name documents the intent and the nesting moves into a flat, separately-readable unit.
-3. **Replace the structure.** A deep `if/else` ladder is often a lookup table or a polymorphic dispatch in disguise; a nested loop is often a `filter`/`map`/`flatMap` pipeline.
-
-Avoid the mechanical fix of merging conditions with `&&` just to drop a level — that trades vertical nesting for an unreadable horizontal condition. The goal is a function whose shape you can take in at a glance, not one that merely passes the depth threshold.
-
-If the nesting is genuinely irreducible (a real algorithm with interacting conditions), extracting the inner loops into well-named helpers is still the move: keep each function shallow even when the algorithm as a whole is deep.

package/src/prompts/duplicated-code.md

@@ -1,9 +0,0 @@
-Duplicated blocks are the cheapest visible sign of a missing abstraction. The right fix is rarely "extract a helper" — it's usually to name the concept that both copies are reaching for.
-
-Read both occurrences side by side and ask: (1) Are these two copies of the same idea, or two different ideas that happen to look alike? (2) If they are the same idea, what is its name? (3) Does the duplicated block belong to a domain concept that does not exist yet — a value object, a strategy, a small class?
-
-Avoid mechanical extraction. Pulling the matching lines into a function with five parameters and a couple of conditionals leaves the real shape untouched and adds a worse name on top. If extracting feels awkward, the underlying abstraction is wrong — step back and find the right one.
-
-A concrete technique: try to write one sentence that names what both copies do. If the sentence reads naturally, that sentence is the name of the abstraction you are missing. If you can only describe the block as "the code that does X, Y, and Z", the duplication is hiding two or three smaller abstractions, not one.
-
-If the two copies actually diverge in important ways and a shared abstraction would distort either side, document the decision (a short note or a test that pins both behaviours) and snooze the duplication — duplication is the lesser evil compared to a wrong abstraction.

package/src/prompts/high-complexity.md

@@ -1,9 +0,0 @@
-High cyclomatic complexity means a function carries too many decisions — harder to understand, harder to test exhaustively, a frequent home for bugs. The smell is mixed concerns, not the number itself.
-
-High complexity often indicates multiple responsibilities. Look for: (1) decision trees that could be strategy patterns or a lookup table, (2) multiple concerns that belong in separate methods, (3) state machines that could be explicit classes with named states.
-
-Focus on extracting *meaningful* abstractions, not just shaving complexity metrics. Splitting one `if/else` chain into three nested helpers usually moves the complexity around without making the code clearer.
-
-If responsibilities are tangled you may need to first *inline* methods to see the whole picture before redistributing. Think of this when reducing complexity seems particularly hard — stepping backwards often opens up better possibilities.
-
-A concrete technique: name each branch by the responsibility it handles. If two branches resolve to the same one-sentence description, they belong together; if one branch has no clear name, that path probably belongs in a separate function or class.

package/src/prompts/needs-extraction.md

@@ -1,9 +0,0 @@
-This file is **both** oversized and duplicated. Those two signals together point at one fix, and it is not "split the file in half."
-
-When a long file also repeats a block, the length is usually a *symptom*: the file has accreted a concept that wants its own home, and the duplication is that concept leaking out in two places. Splitting the file at the line threshold would scatter the duplication further; deduplicating in place would leave the bloated file just as incohesive. Do both at once, in the right order.
-
-Read the duplicated block first and name what it does in one sentence. That sentence is the missing module: extract the duplicated block into a shared unit (a small module, a class, a value object) with a focused interface — not a five-parameter helper that merely hides the lines.
-
-Then let that extraction *carry weight out of the file*. Move the related types and helpers that belong with the new abstraction along with it. Re-check the file size afterwards: a good extraction usually brings the file back under the threshold on its own, because the thing you pulled out was the reason it was too big.
-
-If the two copies are genuinely different ideas that only look alike, do not force a shared abstraction — extract along the real seams instead, and if the duplication is the lesser evil, snooze it with a note rather than distorting either side.

package/src/prompts/non-essential-comment.md

@@ -1,7 +0,0 @@
-Comments indicate code that is not self-documenting. The smell is the *need* for the comment — the reader could not work out what the code does from the names and structure alone.
-
-Extract complex logic into well-named functions instead of explaining with a comment. A function called `applyDiscountForLoyalCustomers` does not need a header explaining what it does.
-
-Remove comments unless they impact functionality (executable annotations) or explain *why* something non-obvious was chosen (a workaround for a specific library bug, a reference to a spec). Comments that explain *what* the code does are almost always redundant — or worse, drift out of sync with the code and start lying.
-
-Do not delete an `eslint-disable` or shebang on autopilot — those are flagged separately and exempted from this rule.

package/src/prompts/non-null-assertion.md

@@ -1,10 +0,0 @@
-A null check — and especially a `!` — is usually covering an underlying design problem, not handling a value that's legitimately sometimes-missing. Before you write one, say out loud what the null actually represents and how it relates to the other objects. That sentence almost always names the real fix.
-
-Two design smells hide behind most null assertions:
-
-- **A circular dependency with no clear owner.** Two objects reference each other, so neither can be built first and one sits briefly null. The fix is to give the dependency a direction: work out which object owns the other (operates on its state, is its exclusive consumer) and have the owner *create* the owned so they're initialised together — or, if it's really an upward event, give the emitter an opaque handler (no-op default) that the owner wires after construction. The mutual reference disappears.
-- **Null standing in for a special case** — "outside the grid", "not found", "not loaded yet". Model that case explicitly: a subclass or a separate class with the same interface — a null object, like a `NullCell` that behaves as a blocked cell — resolved once at the boundary so no caller ever sees null.
-
-Sometimes a null genuinely is unavoidable. Even then `!` is wrong: it swallows the failure at runtime. Replace it with an explicit check — an `if`-guard / early return, or a throw with a clear message — so a broken assumption fails loudly, where it happens.
-
-The only exception: an extremely simple method where the nullable's scope is tiny, a wrong value genuinely cannot cause a silent error, and there's a real performance gain. This should be vanishingly rare — check in with a human before settling for it.

package/src/prompts/oversized-file.md

@@ -1,9 +0,0 @@
-Files over 200 lines accumulate unrelated concerns. The smell is poor cohesion — a file that asks the reader to hold too many ideas at once — not the raw line count.
-
-First identify the seams: which exports, types, or helper clusters actually belong together? A long file usually splits cleanly along one of: a data type and its operations, a feature pipeline, or one concern per file.
-
-Avoid mechanical splits. Carving the file at line 200 into `foo-1.ts` and `foo-2.ts`, or moving every private helper into a `utils.ts`, satisfies the threshold without making anything clearer — the cohesion problem just hops to a new place.
-
-If the file's structure resists splitting, that is itself the signal: responsibilities are tangled. Look for a missing abstraction (a class, a small module with a focused interface) that would let related pieces move together as a unit.
-
-A concrete technique: write a one-sentence description of what each emerging seam *would* be responsible for. If you cannot, you have not found the seam yet — do not split.

package/src/prompts/oversized-function.issues.njk

@@ -1,5 +0,0 @@
-Oversized functions, grouped by file:
-{% for file, group in issues | groupby("details.file") %}
-{{ file }} — {{ group | length }} function(s):
-{% for i in group %}  - {{ i.details.message }}
-{% endfor %}{% endfor %}

package/src/prompts/oversized-function.md

@@ -1,9 +0,0 @@
-Functions over 12 lines almost always carry more than one responsibility, and that is the smell to chase — not the line count itself.
-
-Analyse responsibilities first: what distinct concerns does this function handle? Ask: (1) Are these separate responsibilities that belong in different methods? (2) Should this become a class with multiple methods? (3) Can you group cohesive data into objects to reduce local variables?
-
-Avoid mechanical extraction. Pulling out a `helperA` / `helperB` purely to satisfy the threshold often hides the smell behind worse names and leaves the real shape untouched. Find true responsibility boundaries.
-
-If responsibilities are tangled you may need to first *inline* methods to see the whole picture before redistributing. Think of this when reducing line count seems particularly hard — stepping backwards often opens up better possibilities.
-
-A concrete technique: write what the method does in one short sentence. Refactor until the code reads as close to that sentence as possible. If you cannot say what it does in one sentence, it almost certainly has more than one responsibility.

package/src/prompts/parse-error.md

@@ -1,5 +0,0 @@
-A fatal message means ESLint could not analyse the file at all, so every other rule went unchecked there too — real issues in this file are currently invisible.
-
-Typical causes worth checking: a syntax error, a missing or misconfigured `tsconfig.json`, a plugin version mismatch, or a config that fails to import. Re-run the single file with `npx eslint <file>` to see the full stack the JSON output hides.
-
-You've fixed it when a deliberate change to the file produces the ordinary lint error or warning you'd expect — that proves analysis is running again. Do not silence it with `eslint-disable` or by dropping the file from the lint set.

package/src/prompts/too-many-parameters.md

@@ -1,9 +0,0 @@
-Long parameter lists violate single responsibility — the function is doing too much, or it is operating on data that belongs together as an object the function should live near.
-
-Before grouping parameters into a bag-of-fields object, ask: (1) Should this method actually belong *on* the parameter object as a class method? A free function taking five fields from the same record is often a class method waiting to happen. (2) For static methods with many parameters — this is very often a class waiting to happen. (3) Group cohesive data into meaningful objects and pass those around, even if some methods do not need every field.
-
-Favour a declarative style over many locals — once data clusters into objects, control flow usually simplifies too.
-
-A literal `{ ...everything }` object that just renames the parameter list does not address the smell; it hides it.
-
-A concrete technique: write the calling sites you wish existed (one line each). Make them real — the parameter shape usually falls out of the call sites you want.

package/src/prompts/uncoached.md

@@ -1,4 +0,0 @@
-General guidance: the issues listed are code smells. They tell you that there is likely something wrong with the code. Follow these steps:
-- Ask yourself why the rule exists in the first place. What is it telling you about the code?
-- Find a fix that improves maintainability, cuts cruft — doing the same with fewer statements where that lowers cognitive load — and/or improves security, scalability, and resilience.
-- AVOID AT ALL COST: any fix that is designed to appease the reporting tool, but goes against the spirit of the warning.

package/src/skills/habit-hooks-prompting/SKILL.md

@@ -1,37 +0,0 @@
----
-name: habit-hooks-prompting
-description: "Write or revise a habit-hooks coaching prompt. Use when a linter / knip / jscpd rule fires and the agent's default fix is wrong or shallow, or when adding a project-local override prompt. Keeps prompts short and outcome-focused using the ROSE pattern."
----
-
-# Habit Hooks Prompting
-
-Coaching prompts exist to **change behaviour the agent gets wrong by default** — nothing else.
-
-The goal of every prompt is to drive the agent to **fix the root cause the violation points at, not to appease the reporting tool.** A fix that clears the report while going against the spirit of the rule is a failure, even when the tool goes green.
-
-## When to write one
-
-- The rule's own message already produces the right fix → **write nothing.** Let it fall through to the uncoached prompt. Don't coach what already works.
-- The agent's default reaction is shallow or wrong — mechanical extraction that misses the real design problem, silencing instead of fixing, a reflexive `eslint-disable` → write a prompt that redirects it.
-
-This is reactive: add a prompt the moment you catch the default behaviour being wrong, not pre-emptively for every rule.
-
-## Keep it short (KISS)
-
-A good prompt is the **minimum text that reliably nudges the right behaviour**. If it reads like an essay, cut it — direction beats explanation. The agent already has the violation in front of it.
-
-## Use ROSE
-
-ROSE structures feedback so the agent understands *why*, not just *what*, and fixes the cause rather than the symptom:
-
-- **R**isk — the consequence of leaving it as-is.
-- **O**bservation — what was seen. **The linter already supplies this** (file, line, rule); don't repeat it.
-- **S**olution — the specific move you want, including the non-obvious one the agent tends to miss.
-- **E**xpected outcome — what "done right" looks like, so the agent can check itself.
-
-So a habit-hooks prompt writes **R, S, E** and lets the tool supply **O**. See `eslint-fatal.md` and `eslint-typescript-eslint-no-non-null-assertion.md` for prompts in this shape.
-
-## Where prompts live
-
-- Bundled: `src/prompts/<slugified-rule-id>.md` (`:` and `/` → `-`, `@` dropped).
-- Project-local override: a file of the same name in the directory set by `prompts:` in `habit-hooks.config`.