@ngockhoale/ukit 2.4.1 → 2.4.3

package/templates/.claude/ukit/index/post-edit-verify.mjs

@@ -4,6 +4,14 @@ import { fileURLToPath } from 'node:url';
 import { analyzeTextBuffer, analyzeTextFile, publicTextProfile } from '../runtime/text-profile.mjs';
 import { classifySafePatchRisk, isSafePatchAdvisoryOnly, parseJsonInput, pathExists, readRuntimeSafePatchConfig, resolveProjectFile } from '../runtime/safe-patch-core.mjs';
 
+// Hook-context self-deadline (2.4.1 orphan-leak class): the post-edit-verify hook passes
+// UKIT_HOOK_DEADLINE_MS so a wedged read can never orphan this process past the hook
+// budget. Non-hook usage never sets it and is never self-killed.
+const HOOK_DEADLINE_MS = Number.parseInt(process.env.UKIT_HOOK_DEADLINE_MS || '', 10);
+if (Number.isFinite(HOOK_DEADLINE_MS) && HOOK_DEADLINE_MS > 0) {
+  setTimeout(() => process.exit(0), HOOK_DEADLINE_MS).unref();
+}
+
 async function readStdin() {
   return await new Promise((resolve) => {
     let raw = '';

package/templates/.claude/ukit/index/pre-edit-backup.mjs

@@ -4,6 +4,14 @@ import { analyzeTextFile, publicTextProfile } from '../runtime/text-profile.mjs'
 import { classifySafePatchRisk, parseJsonInput, pathExists, readRuntimeSafePatchConfig, resolveProjectFile } from '../runtime/safe-patch-core.mjs';
 import { fileURLToPath } from 'node:url';
 
+// Hook-context self-deadline (2.4.1 orphan-leak class): the pre-edit-backup hook passes
+// UKIT_HOOK_DEADLINE_MS so a wedged read can never orphan this process past the hook
+// budget. Non-hook usage never sets it and is never self-killed.
+const HOOK_DEADLINE_MS = Number.parseInt(process.env.UKIT_HOOK_DEADLINE_MS || '', 10);
+if (Number.isFinite(HOOK_DEADLINE_MS) && HOOK_DEADLINE_MS > 0) {
+  setTimeout(() => process.exit(0), HOOK_DEADLINE_MS).unref();
+}
+
 async function readStdin() {
   return await new Promise((resolve) => {
     let raw = '';

package/templates/.claude/ukit/index/refresh-index.mjs

@@ -2,7 +2,7 @@
 import path from 'node:path';
 import {
   buildCodeIndex,
-  isIndexStale,
+  inspectIndexStaleness,
   DEFAULT_INDEX_CACHE_MAX_AGE_MS,
   getIndexArtifactGeneratedAt,
 } from './lib/index-core.mjs';
@@ -14,34 +14,64 @@ const changedFiles = changedArg ? changedArg.split(',').map((x) => x.trim()).fil
 const force = args.includes('--force');
 
 const lastRefreshMs = await getIndexArtifactGeneratedAt({ rootDir });
-const stale = force
-  ? true
-  : await isIndexStale({ rootDir, maxAgeMs: DEFAULT_INDEX_CACHE_MAX_AGE_MS, generatedAtMs: lastRefreshMs });
 
-if (!stale) {
-  console.log('[ukit:index:refresh] skipped (cache fresh)');
-  if (lastRefreshMs !== null) {
-    console.log(`lastRefresh: ${new Date(lastRefreshMs).toISOString()}`);
-  }
-  if (changedFiles.length > 0) {
-    console.log(`changed-files-hint: ${changedFiles.join(', ')}`);
-  }
-  console.log(`root: ${rootDir}`);
-} else {
-  const summary = await buildCodeIndex({ rootDir });
+if (changedFiles.length > 0 && !force) {
+  // Hints carry the change set, so no staleness enumeration is needed:
+  // buildCodeIndex merges the hinted paths into the existing artifacts and
+  // falls back to the bounded full discovery path on its own when the hints
+  // are unsafe or the previous artifacts cannot be merged (TASK-023).
+  const summary = await buildCodeIndex({ rootDir, changedFiles });
 
   console.log('[ukit:index:refresh] completed');
+  console.log(`mode: ${summary.mode}`);
   if (lastRefreshMs !== null) {
     console.log(`lastRefreshBefore: ${new Date(lastRefreshMs).toISOString()}`);
   }
-  if (changedFiles.length > 0) {
-    console.log(`changed-files-hint: ${changedFiles.join(', ')}`);
-  }
+  console.log(`changed-files-hint: ${changedFiles.join(', ')}`);
   console.log(`root: ${rootDir}`);
   console.log(`files: ${summary.fileCount}`);
   console.log(`parsed: ${summary.parsedCodeFileCount}`);
   console.log(`reused: ${summary.reusedCodeFileCount}`);
+  if (summary.removedFiles.length > 0) {
+    console.log(`removed: ${summary.removedFiles.length}`);
+  }
   console.log(`indexDir: ${summary.indexDir}`);
+} else {
+  const staleness = force
+    ? null
+    : await inspectIndexStaleness({ rootDir, maxAgeMs: DEFAULT_INDEX_CACHE_MAX_AGE_MS, generatedAtMs: lastRefreshMs });
+  const stale = force || staleness.stale;
+
+  if (!stale) {
+    console.log('[ukit:index:refresh] skipped (cache fresh)');
+    if (lastRefreshMs !== null) {
+      console.log(`lastRefresh: ${new Date(lastRefreshMs).toISOString()}`);
+    }
+    if (changedFiles.length > 0) {
+      console.log(`changed-files-hint: ${changedFiles.join(', ')}`);
+    }
+    console.log(`root: ${rootDir}`);
+  } else {
+    // Reuse the staleness check's discovery snapshot: one enumeration per refresh.
+    const summary = await buildCodeIndex({
+      rootDir,
+      discoverySnapshot: staleness ? staleness.snapshot : null,
+    });
+
+    console.log('[ukit:index:refresh] completed');
+    console.log(`mode: ${summary.mode}`);
+    if (lastRefreshMs !== null) {
+      console.log(`lastRefreshBefore: ${new Date(lastRefreshMs).toISOString()}`);
+    }
+    if (changedFiles.length > 0) {
+      console.log(`changed-files-hint: ${changedFiles.join(', ')}`);
+    }
+    console.log(`root: ${rootDir}`);
+    console.log(`files: ${summary.fileCount}`);
+    console.log(`parsed: ${summary.parsedCodeFileCount}`);
+    console.log(`reused: ${summary.reusedCodeFileCount}`);
+    console.log(`indexDir: ${summary.indexDir}`);
+  }
 }
 
 function getRootDir(argv) {