@rasensio/aidlc 1.10.6 → 1.10.8

package/dist/doctor/migrations/legacy-cost-hooks.js

@@ -0,0 +1,92 @@
+/**
+ * legacy-cost-hooks migration: re-install cost hooks that predate the shim.
+ *
+ * Replaces `stale-cost-hooks` (cost-capture-goes-dark/AC-14, AC-18). That
+ * migration asked "does the absolute CLI path recorded in the hook still exist?"
+ * — the right question only while the command *contained* an absolute path. Hook
+ * commands now invoke `.aidlc/cost/capture.sh`, which resolves node and the CLI
+ * at run time, so there is no longer a path that can go stale and nothing left
+ * to detect. What remains is a one-time migration: hooks installed by an older
+ * version still carry the frozen paths and are still silently broken.
+ *
+ * WHY THIS ONE REPAIRS, WHERE ITS PREDECESSOR ONLY REPORTED. `stale-cost-hooks`
+ * was deliberately report-only, reasoning that "a diagnostic must not silently
+ * rewrite a user's .claude/settings.json". That reasoning was sound and is
+ * narrowed here rather than discarded: this migration only ever replaces hook
+ * entries carrying our own `#aidlc-cost-hook` marker, via the same
+ * merge-not-clobber installer that leaves every other key and every unmarked
+ * hook untouched. Nothing the user wrote is rewritten.
+ *
+ * The reason to change the stance at all is what report-only cost us: the
+ * detection was correct, the report printed on every `aidlc update`, and the
+ * data was lost anyway, because the last link in the chain was a human reading a
+ * line. A repair that needs to be noticed is a repair that does not happen. The
+ * failure it prevents is silent and unrecoverable — usage that was never
+ * captured cannot be reconstructed later.
+ *
+ * Self-clearing: re-installing produces the shim form, so `legacy` goes false
+ * and this stops reporting without anyone editing state.
+ */
+import { claudeCodeHookStatus, installClaudeCodeHooks } from '../../cost/hooks/claude-code.js';
+import { kiroHookStatus, installKiroHooks } from '../../cost/hooks/kiro.js';
+import { cliEntryPath } from '../../cost/hooks/entry.js';
+/*
+ * Deliberately NOT driven by allProviders(): that registry is populated by an
+ * explicit registerBuiltinProviders() call, so any caller that has not made it
+ * sees an EMPTY list — the predecessor migration reproduced exactly that bug on
+ * its first run and found nothing in every case. A diagnostic that silently
+ * finds nothing is worse than no diagnostic. Hook installation exists only for
+ * these two platforms, so the pairing is static.
+ */
+const HOOK_PLATFORMS = [
+    { name: 'claude-code', status: claudeCodeHookStatus, install: installClaudeCodeHooks },
+    { name: 'kiro', status: kiroHookStatus, install: installKiroHooks },
+];
+/**
+ * Platforms whose installed hook still uses the pre-shim absolute-path form.
+ * Read-only half, exported for reuse and direct testing.
+ *
+ * Only installed hooks are considered: an absent hook is a different condition
+ * (never armed, not broken), and `aidlc cost hooks status` already says so.
+ */
+export function legacyCostHookPlatforms(projectRoot) {
+    const legacy = [];
+    for (const p of HOOK_PLATFORMS) {
+        const status = p.status(projectRoot);
+        if (status.installed && status.legacy)
+            legacy.push(p.name);
+    }
+    return legacy;
+}
+function summarize(legacy) {
+    const one = legacy.length === 1;
+    return (`${legacy.length} cost hook${one ? '' : 's'} ` +
+        `still ${one ? 'names' : 'name'} an absolute node/CLI path frozen at install time ` +
+        `(${legacy.join(', ')}) — re-installed so capture survives a node or machine change`);
+}
+export const legacyCostHooks = {
+    id: 'legacy-cost-hooks',
+    tier: 'safe',
+    check(ctx) {
+        const legacy = legacyCostHookPlatforms(ctx.projectRoot);
+        if (legacy.length === 0)
+            return { applicable: true, pending: false };
+        return { applicable: true, pending: true, description: summarize(legacy) };
+    },
+    apply(ctx) {
+        const legacy = legacyCostHookPlatforms(ctx.projectRoot);
+        if (legacy.length === 0)
+            return { changed: false, summary: 'no legacy cost hooks' };
+        // process.execPath and the resolved CLI are only *hints* for the shim; they
+        // are validated at run time and never written into the command text, so a
+        // hint captured here going stale later is no longer a failure mode.
+        const nodePath = process.execPath;
+        const cliPath = cliEntryPath();
+        for (const p of HOOK_PLATFORMS) {
+            if (legacy.includes(p.name))
+                p.install(ctx.projectRoot, nodePath, cliPath);
+        }
+        return { changed: true, summary: summarize(legacy) };
+    },
+};
+//# sourceMappingURL=legacy-cost-hooks.js.map

package/dist/doctor/migrations/legacy-cost-hooks.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"legacy-cost-hooks.js","sourceRoot":"","sources":["../../../src/doctor/migrations/legacy-cost-hooks.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4BG;AAEH,OAAO,EAAE,oBAAoB,EAAE,sBAAsB,EAAE,MAAM,iCAAiC,CAAC;AAC/F,OAAO,EAAE,cAAc,EAAE,gBAAgB,EAAE,MAAM,0BAA0B,CAAC;AAC5E,OAAO,EAAE,YAAY,EAAE,MAAM,2BAA2B,CAAC;AASzD;;;;;;;GAOG;AACH,MAAM,cAAc,GAAmB;IACrC,EAAE,IAAI,EAAE,aAAa,EAAE,MAAM,EAAE,oBAAoB,EAAE,OAAO,EAAE,sBAAsB,EAAE;IACtF,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,cAAc,EAAE,OAAO,EAAE,gBAAgB,EAAE;CACpE,CAAC;AAEF;;;;;;GAMG;AACH,MAAM,UAAU,uBAAuB,CAAC,WAAmB;IACzD,MAAM,MAAM,GAAa,EAAE,CAAC;IAC5B,KAAK,MAAM,CAAC,IAAI,cAAc,EAAE,CAAC;QAC/B,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QACrC,IAAI,MAAM,CAAC,SAAS,IAAI,MAAM,CAAC,MAAM;YAAE,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;IAC7D,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAS,SAAS,CAAC,MAAgB;IACjC,MAAM,GAAG,GAAG,MAAM,CAAC,MAAM,KAAK,CAAC,CAAC;IAChC,OAAO,CACL,GAAG,MAAM,CAAC,MAAM,aAAa,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,GAAG;QAC9C,SAAS,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,oDAAoD;QACnF,IAAI,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,+DAA+D,CACrF,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,MAAM,eAAe,GAAc;IACxC,EAAE,EAAE,mBAAmB;IACvB,IAAI,EAAE,MAAM;IACZ,KAAK,CAAC,GAAG;QACP,MAAM,MAAM,GAAG,uBAAuB,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;QACxD,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC;YAAE,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC;QACrE,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,SAAS,CAAC,MAAM,CAAC,EAAE,CAAC;IAC7E,CAAC;IACD,KAAK,CAAC,GAAG;QACP,MAAM,MAAM,GAAG,uBAAuB,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;QACxD,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC;YAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,OAAO,EAAE,sBAAsB,EAAE,CAAC;QAEpF,4EAA4E;QAC5E,0EAA0E;QAC1E,oEAAoE;QACpE,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC;QAClC,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,KAAK,MAAM,CAAC,IAAI,cAAc,EAAE,CAAC;YAC/B,IAAI,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC;gBAAE,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,WAAW,EAAE,QAAQ,EAAE,OAAO,CAAC,CAAC;QAC7E,CAAC;QACD,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,SAAS,CAAC,MAAM,CAAC,EAAE,CAAC;IACvD,CAAC;CACF,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rasensio/aidlc",
-  "version": "1.10.6",
+  "version": "1.10.8",
   "description": "AI Development Lifecycle Framework — structured lifecycle guidance for AI coding agents across platforms",
   "type": "module",
   "main": "dist/cli.js",
@@ -24,7 +24,7 @@
     "commander": "15.0.0",
     "picomatch": "4.0.5",
     "yaml": "2.9.0",
-    "@rasensio/aidlc-content": "1.10.6"
+    "@rasensio/aidlc-content": "1.10.8"
   },
   "devDependencies": {
     "@types/node": "22.15.32",

package/dist/doctor/migrations/stale-cost-hooks.d.ts

@@ -1,37 +0,0 @@
-/**
- * stale-cost-hooks migration: report installed cost hooks whose recorded CLI
- * path no longer resolves on disk.
- *
- * Report-only, `safe` tier. The repair rewrites project files, so it belongs to
- * `aidlc cost hooks install` — a diagnostic must not silently rewrite a user's
- * .claude/settings.json or .kiro/hooks/. Same reasoning as unknown-agent-claims.
- *
- * Why this exists: hook commands are written with ABSOLUTE paths to both the
- * node binary and the globally installed CLI. A username change, a new machine,
- * or an `nvm` upgrade invalidates them, and the hook then fails silently —
- * nothing executes, nothing reaches capture.log, and `aidlc cost` reports no
- * data forever with no indication why. Observed in a real project whose hooks
- * still named `/Users/rodrigo` and node v24.9.0 long after the account and
- * runtime had moved on: capture had been dead for four days, every phase read
- * $0.00, and no command said anything was wrong.
- *
- * The detection already existed — `commandResolvable` on both hook-status
- * functions — and had no caller outside `aidlc cost hooks status`, which a
- * human has to remember to run. A guard nobody invokes is the same shape as the
- * cost feature whose arming was manual and therefore never armed.
- *
- * Self-clearing (the cost-capture-noise lesson: a suppression guard needs its
- * counterpart verified to exist). Reinstalling the hooks makes the path resolve
- * and silences this without anyone editing state.
- */
-import type { Migration } from '../types.js';
-/**
- * Platforms whose cost hooks are installed but point at a path that is gone.
- * Read-only half, exported for reuse and for direct testing.
- *
- * Only installed hooks are reported: an absent hook is a different condition
- * (never armed, not broken) and `aidlc cost hooks status` already says so.
- */
-export declare function staleCostHookPlatforms(projectRoot: string): string[];
-export declare const staleCostHooks: Migration;
-//# sourceMappingURL=stale-cost-hooks.d.ts.map

package/dist/doctor/migrations/stale-cost-hooks.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"stale-cost-hooks.d.ts","sourceRoot":"","sources":["../../../src/doctor/migrations/stale-cost-hooks.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AAIH,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAwB7C;;;;;;GAMG;AACH,wBAAgB,sBAAsB,CAAC,WAAW,EAAE,MAAM,GAAG,MAAM,EAAE,CAOpE;AAWD,eAAO,MAAM,cAAc,EAAE,SAgB5B,CAAC"}

package/dist/doctor/migrations/stale-cost-hooks.js

@@ -1,85 +0,0 @@
-/**
- * stale-cost-hooks migration: report installed cost hooks whose recorded CLI
- * path no longer resolves on disk.
- *
- * Report-only, `safe` tier. The repair rewrites project files, so it belongs to
- * `aidlc cost hooks install` — a diagnostic must not silently rewrite a user's
- * .claude/settings.json or .kiro/hooks/. Same reasoning as unknown-agent-claims.
- *
- * Why this exists: hook commands are written with ABSOLUTE paths to both the
- * node binary and the globally installed CLI. A username change, a new machine,
- * or an `nvm` upgrade invalidates them, and the hook then fails silently —
- * nothing executes, nothing reaches capture.log, and `aidlc cost` reports no
- * data forever with no indication why. Observed in a real project whose hooks
- * still named `/Users/rodrigo` and node v24.9.0 long after the account and
- * runtime had moved on: capture had been dead for four days, every phase read
- * $0.00, and no command said anything was wrong.
- *
- * The detection already existed — `commandResolvable` on both hook-status
- * functions — and had no caller outside `aidlc cost hooks status`, which a
- * human has to remember to run. A guard nobody invokes is the same shape as the
- * cost feature whose arming was manual and therefore never armed.
- *
- * Self-clearing (the cost-capture-noise lesson: a suppression guard needs its
- * counterpart verified to exist). Reinstalling the hooks makes the path resolve
- * and silences this without anyone editing state.
- */
-import { claudeCodeHookStatus } from '../../cost/hooks/claude-code.js';
-import { kiroHookStatus } from '../../cost/hooks/kiro.js';
-/*
- * Deliberately NOT driven by allProviders(). That registry is populated by an
- * explicit registerBuiltinProviders() call, so any caller that has not made it
- * sees an EMPTY list — providers/index.ts documents that exact bug ("any code
- * path that did not import that module saw an empty registry"), and this
- * migration reproduced it on the first run: every case returned no findings
- * because the loop had nothing to iterate. A diagnostic that silently finds
- * nothing is worse than no diagnostic at all.
- *
- * Hook installation exists only for these two platforms, so the pairing is
- * static and carries no hidden initialisation order.
- */
-const HOOK_PLATFORMS = [
-    { name: 'claude-code', status: claudeCodeHookStatus },
-    { name: 'kiro', status: kiroHookStatus },
-];
-/**
- * Platforms whose cost hooks are installed but point at a path that is gone.
- * Read-only half, exported for reuse and for direct testing.
- *
- * Only installed hooks are reported: an absent hook is a different condition
- * (never armed, not broken) and `aidlc cost hooks status` already says so.
- */
-export function staleCostHookPlatforms(projectRoot) {
-    const stale = [];
-    for (const p of HOOK_PLATFORMS) {
-        const status = p.status(projectRoot);
-        if (status.installed && !status.commandResolvable)
-            stale.push(p.name);
-    }
-    return stale;
-}
-function summarize(stale) {
-    return (`${stale.length} cost hook${stale.length === 1 ? '' : 's'} ` +
-        `point at a path that no longer exists (${stale.join(', ')}) — ` +
-        'no usage is being captured; run `aidlc cost hooks install` to re-point ' +
-        'them at the current node and CLI');
-}
-export const staleCostHooks = {
-    id: 'stale-cost-hooks',
-    tier: 'safe',
-    check(ctx) {
-        const stale = staleCostHookPlatforms(ctx.projectRoot);
-        if (stale.length === 0)
-            return { applicable: true, pending: false };
-        return { applicable: true, pending: true, description: summarize(stale) };
-    },
-    apply(ctx) {
-        // Report-only. `changed: true` is what makes the finding visible: the runner
-        // maps changed:false to status 'clean', and commands/doctor.ts filters clean
-        // outcomes out of its report entirely, so a diagnostic returning false would
-        // be silently swallowed. The rendered line reads `stale-cost-hooks: <summary>`.
-        const stale = staleCostHookPlatforms(ctx.projectRoot);
-        return { changed: stale.length > 0, summary: summarize(stale) };
-    },
-};
-//# sourceMappingURL=stale-cost-hooks.js.map

package/dist/doctor/migrations/stale-cost-hooks.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"stale-cost-hooks.js","sourceRoot":"","sources":["../../../src/doctor/migrations/stale-cost-hooks.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AAEH,OAAO,EAAE,oBAAoB,EAAE,MAAM,iCAAiC,CAAC;AACvE,OAAO,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAQ1D;;;;;;;;;;;GAWG;AACH,MAAM,cAAc,GAAuB;IACzC,EAAE,IAAI,EAAE,aAAa,EAAE,MAAM,EAAE,oBAAoB,EAAE;IACrD,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,cAAc,EAAE;CACzC,CAAC;AAEF;;;;;;GAMG;AACH,MAAM,UAAU,sBAAsB,CAAC,WAAmB;IACxD,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,KAAK,MAAM,CAAC,IAAI,cAAc,EAAE,CAAC;QAC/B,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QACrC,IAAI,MAAM,CAAC,SAAS,IAAI,CAAC,MAAM,CAAC,iBAAiB;YAAE,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;IACxE,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAS,SAAS,CAAC,KAAe;IAChC,OAAO,CACL,GAAG,KAAK,CAAC,MAAM,aAAa,KAAK,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,GAAG;QAC5D,0CAA0C,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM;QAChE,yEAAyE;QACzE,kCAAkC,CACnC,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,MAAM,cAAc,GAAc;IACvC,EAAE,EAAE,kBAAkB;IACtB,IAAI,EAAE,MAAM;IACZ,KAAK,CAAC,GAAG;QACP,MAAM,KAAK,GAAG,sBAAsB,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;QACtD,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;YAAE,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC;QACpE,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,SAAS,CAAC,KAAK,CAAC,EAAE,CAAC;IAC5E,CAAC;IACD,KAAK,CAAC,GAAG;QACP,6EAA6E;QAC7E,6EAA6E;QAC7E,6EAA6E;QAC7E,gFAAgF;QAChF,MAAM,KAAK,GAAG,sBAAsB,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;QACtD,OAAO,EAAE,OAAO,EAAE,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,OAAO,EAAE,SAAS,CAAC,KAAK,CAAC,EAAE,CAAC;IAClE,CAAC;CACF,CAAC"}