@ngockhoale/ukit 1.4.2 → 1.4.4

package/CHANGELOG.md

@@ -2,7 +2,24 @@
 
 All notable changes to UKit are documented here.
 
-## Unreleased
+## 1.4.4 - 2026-05-11
+
+### Fixed
+
+- Removed shipped `.gitignore` negations that accidentally unignored installed `.claude/` and `.codex/` files in fresh repos, preventing large post-install `git status` noise.
+- Kept the intended local-only ignores for `AGENTS.md`, `CLAUDE.md`, `docs/STATUS.md`, `docs/TASKS.md`, runtime cache, and adapter state intact.
+
+### Tests
+
+- Added install-pipeline regression coverage that initializes a fresh git repo, runs `ukit install`, and asserts managed UKit files stay ignored.
+
+## 1.4.3 - 2026-05-10
+
+### Changed
+
+- Tightened execute-first orchestration guidance so one clear safe path runs directly, while real user-choice or risk-confirmation moments still ask.
+- Added compact route-audit coverage and repeated write-lane rescue bias to reduce stop-and-explain loops.
+- Published patch `1.4.3` after fresh full-suite, release-verify, and registry-smoke validation.
 
 ## 1.4.2 - 2026-05-10
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngockhoale/ukit",
-  "version": "1.4.2",
+  "version": "1.4.4",
   "description": "Install/update an index-first AI workspace for Claude Code, Antigravity, OpenAI Codex, and OpenCode.",
   "license": "MIT",
   "type": "module",

package/src/core/runtimeConfig.js

@@ -49,7 +49,7 @@ export function buildDefaultRuntimeConfig(overrides = {}) {
   const safeOverrides = isPlainObject(overrides) ? overrides : {};
 
   return mergeObjects({
-    version: '1.4.2',
+    version: '1.4.4',
     agent: 'claude-code',
     autonomy: {
       level: 'balanced',

package/src/index/routeCatalog.js

@@ -178,7 +178,7 @@ export const ROUTE_CATALOG = [
     order: 20,
     contextMode: 'standalone',
     signals: [
-      { type: 'prompt', regex: /\b(pdf|pdf form|fill (?:this )?pdf|merge pdf|split pdf|extract tables? from .*pdf|extract text from .*pdf)\b/i, score: 8 },
+      { type: 'prompt', regex: /\b(pdf form|fillable pdf|fill (?:this )?pdf|merge pdfs?|split pdfs?|extract tables? from .*pdf|extract text from .*pdf|read (?:this |the |a )?pdf|open (?:this |the |a )?pdf|analy(?:s|z)e (?:this |the |a )?pdf|annotate (?:this |the |a )?pdf|create pdf|convert .*pdf|pdf extraction|pdf parser|pdf parsing)\b/i, score: 8 },
       { type: 'command', regex: /\b(pdftotext|qpdf|pypdf|pdfplumber|reportlab)\b/i, score: 5 },
       { type: 'file', regex: /\.pdf$/i, score: 5 },
     ],

package/src/index/taskRouting.js

@@ -59,13 +59,20 @@ export async function deriveTaskRoute({
     intentMode,
     taskType: inferredTaskType,
   });
+  const executionCandidates = buildExecutionModeCandidates({
+    promptText: normalizedPrompt,
+    commandText: normalizedCommand,
+    targetFile: normalizedTarget,
+    intentMode,
+    executionScores,
+  });
   const executionMode = deriveExecutionMode({
     promptText: normalizedPrompt,
     commandText: normalizedCommand,
     targetFile: normalizedTarget,
     intentMode,
-    taskType: inferredTaskType,
     executionScores,
+    executionCandidates,
   });
   const preservedPrompt = normalizedPrompt || String(lastExplicitUserPromptText || '').trim();
   const degradedWarnings = [];
@@ -144,6 +151,7 @@ export async function deriveTaskRoute({
       intentMode,
       autonomyLevel,
       executionScores,
+      executionCandidates,
       executionMode,
     },
     contextRecommendation,
@@ -164,6 +172,7 @@ export async function deriveTaskRoute({
       intentMode,
       autonomyLevel,
       executionScores,
+      executionCandidates,
       executionMode,
     },
     approachSelector,
@@ -209,9 +218,11 @@ export function buildRouteSummary({
     && contextRecommendation.command.trim();
   const executionMode = routingContext.executionMode ?? null;
   const executionScores = routingContext.executionScores ?? null;
+  const executionCandidates = routingContext.executionCandidates ?? null;
   const approachSelector = buildApproachSelectorResult({
     executionMode,
     executionScores,
+    executionCandidates,
   });
   const executionContract = buildExecutionContract(executionMode);
   const completionState = buildCompletionState({
@@ -250,6 +261,7 @@ export function buildRouteSummary({
     editGuardHint,
     executionMode,
     executionScores,
+    executionCandidates,
     approachSelector,
     executionContract,
     completionState,
@@ -284,6 +296,12 @@ function deriveExecutionScores({
   const buildSignal = /\b(build|create|add|implement|feature)\b/.test(raw);
   const debugSignal = intentMode === 'debug-specific' || /\b(root cause|debug|triage|flaky|investigate|why)\b/.test(raw);
   const reviewSignal = intentMode === 'review-specific' || /\b(review|audit|verify|release readiness)\b/.test(raw);
+  const failureSignal = /\b(failing|failed|broken|error|crash|timeout|undefined|exception|eacces|trace)\b/.test(raw);
+  const impactSignal = /\b(map impact|impact|blast radius|all affected|across (?:runtime|templates|helpers|adapters|mirrors)|affected adapters|affected mirrors|inspect impact)\b/.test(raw);
+  const boundedEditSignal = /\b(one[- ]line|small|tiny|single|local|guard|log line|loading message|label)\b/.test(raw);
+  const implementSignal = /\b(implement|apply|update|modify|add|create|ship|deliver|fix)\b/.test(raw)
+    || /\bchange\s+.+\s+to\s+.+\b/.test(raw);
+  const explicitTarget = Boolean(targetFile);
 
   let editCertainty = 0;
   if (directTransformSignal) {
@@ -297,7 +315,7 @@ function deriveExecutionScores({
   let investigationNeed = 0;
   if (debugSignal) {
     investigationNeed = 3;
-  } else if (/\b(failing|broken|error|crash|timeout)\b/.test(raw)) {
+  } else if (failureSignal) {
     investigationNeed = 2;
   }
 
@@ -330,6 +348,17 @@ function deriveExecutionScores({
     blastRadius,
     verificationBurden,
     ambiguity,
+    sharedRisk,
+    directTransformSignal,
+    smallFixSignal,
+    buildSignal,
+    debugSignal,
+    reviewSignal,
+    failureSignal,
+    impactSignal,
+    boundedEditSignal,
+    implementSignal,
+    explicitTarget,
   };
 }
 
@@ -339,40 +368,64 @@ function deriveExecutionMode({
   targetFile = null,
   intentMode = null,
   executionScores = {},
+  executionCandidates = null,
 } = {}) {
   const raw = `${promptText ?? ''}\n${commandText ?? ''}`.toLowerCase();
   const scores = executionScores;
+  const candidates = executionCandidates ?? buildExecutionModeCandidates({
+    promptText,
+    commandText,
+    targetFile,
+    intentMode,
+    executionScores,
+  });
   const explicitReviewLead = /\b(review this|audit this|review\b.*\brelease readiness|verify\b.*\brelease readiness)\b/.test(raw);
+  const strongImpactLead = scores.sharedRisk
+    && (scores.impactSignal || /\b(check all affected|map all affected|across all affected)\b/.test(raw));
+  const boundedLocalBuildCandidate = scores.buildSignal && scores.boundedEditSignal && scores.explicitTarget && !scores.sharedRisk;
 
-  if (intentMode === 'review-specific' || explicitReviewLead) {
+  if ((intentMode === 'review-specific' || explicitReviewLead) && !scores.implementSignal) {
     return 'review-release';
   }
 
-  if (scores.blastRadius >= 3 && (scores.ambiguity >= 2 || scores.investigationNeed >= 2)) {
+  if (strongImpactLead || (scores.blastRadius >= 3 && (scores.ambiguity >= 2 || scores.investigationNeed >= 2))) {
     return 'map-impact';
   }
 
-  if (scores.blastRadius >= 3 || isSharedImpactFile(targetFile)) {
+  if (scores.sharedRisk && scores.failureSignal && scores.investigationNeed >= 2 && !scores.impactSignal) {
     return 'shared-edit';
   }
 
-  if (scores.investigationNeed >= 3) {
+  if (scores.blastRadius >= 3 || scores.sharedRisk) {
+    return 'shared-edit';
+  }
+
+  if (scores.investigationNeed >= 3 || (scores.failureSignal && scores.investigationNeed >= 2)) {
     return 'find-cause';
   }
 
+  if (scores.directTransformSignal && !scores.explicitTarget && scores.investigationNeed === 0 && !scores.sharedRisk) {
+    return 'local-build';
+  }
+
   if (
     scores.editCertainty >= 3
-    && scores.ambiguity === 0
+    && scores.ambiguity <= 1
     && scores.blastRadius === 0
     && scores.verificationBurden <= 1
   ) {
     return 'tiny-fix';
   }
 
+  if (boundedLocalBuildCandidate && scores.investigationNeed === 0) {
+    return 'local-fix';
+  }
+
   if (
     /\b(build|create|add|implement|feature|summary card)\b/.test(raw)
     && scores.investigationNeed === 0
     && scores.blastRadius < 3
+    && !scores.boundedEditSignal
   ) {
     return 'local-build';
   }
@@ -381,7 +434,107 @@ function deriveExecutionMode({
     return 'local-fix';
   }
 
-  return 'local-build';
+  return applySafeUpwardBias(candidates, 'local-build');
+}
+
+function buildExecutionModeCandidates({
+  promptText = '',
+  commandText = '',
+  targetFile = null,
+  intentMode = null,
+  executionScores = {},
+} = {}) {
+  const scores = executionScores;
+  const raw = `${promptText ?? ''}\n${commandText ?? ''}`.toLowerCase();
+  const explicitReviewLead = (intentMode === 'review-specific' || /\b(review this|audit this|review\b.*\brelease readiness|verify\b.*\brelease readiness)\b/.test(raw))
+    && !scores.implementSignal;
+
+  const modeScores = {
+    'tiny-fix': (
+      (scores.editCertainty * 4)
+      + (scores.directTransformSignal ? 4 : 0)
+      + (scores.explicitTarget ? 1 : 0)
+      - (scores.failureSignal ? 4 : 0)
+      - (scores.blastRadius * 4)
+      - (scores.ambiguity * 2)
+    ),
+    'local-fix': (
+      (scores.editCertainty * 3)
+      + (scores.boundedEditSignal ? 3 : 0)
+      + (scores.explicitTarget ? 1 : 0)
+      - (scores.failureSignal ? 2 : 0)
+      - (scores.blastRadius * 4)
+    ),
+    'local-build': (
+      (scores.buildSignal ? 6 : 0)
+      + (scores.editCertainty * 1)
+      + (scores.explicitTarget ? 1 : 0)
+      - (scores.boundedEditSignal ? 3 : 0)
+      - (scores.failureSignal ? 3 : 0)
+      - (scores.blastRadius * 4)
+    ),
+    'find-cause': (
+      (scores.investigationNeed * 4)
+      + (scores.failureSignal ? 3 : 0)
+      + (scores.debugSignal ? 2 : 0)
+      - (scores.blastRadius >= 3 ? 1 : 0)
+    ),
+    'shared-edit': (
+      (scores.sharedRisk ? 8 : 0)
+      + (scores.editCertainty * 1)
+      + (scores.buildSignal ? 1 : 0)
+      - (scores.impactSignal ? 2 : 0)
+    ),
+    'map-impact': (
+      (scores.sharedRisk ? 7 : 0)
+      + (scores.impactSignal ? 5 : 0)
+      + (scores.ambiguity * 2)
+      + scores.investigationNeed
+    ),
+    'review-release': (
+      (explicitReviewLead ? 10 : 0)
+      + (scores.reviewSignal ? 3 : 0)
+      - (scores.implementSignal ? 5 : 0)
+    ),
+  };
+
+  return Object.entries(modeScores)
+    .map(([mode, score]) => ({ mode, score }))
+    .sort((a, b) => b.score - a.score || executionModeRank(a.mode) - executionModeRank(b.mode));
+}
+
+function applySafeUpwardBias(candidates = [], fallbackMode = 'local-build') {
+  const [topCandidate, competingCandidate] = candidates;
+  if (!topCandidate) {
+    return fallbackMode;
+  }
+
+  if (!competingCandidate) {
+    return topCandidate.mode;
+  }
+
+  const scoreGap = Number(topCandidate.score ?? 0) - Number(competingCandidate.score ?? 0);
+  const rankGap = executionModeRank(competingCandidate.mode) - executionModeRank(topCandidate.mode);
+  if (scoreGap <= 1 && rankGap === 1) {
+    return competingCandidate.mode;
+  }
+
+  return topCandidate.mode;
+}
+
+function executionModeRank(mode = '') {
+  const orderedModes = [
+    'tiny-fix',
+    'local-fix',
+    'local-build',
+    'find-cause',
+    'shared-edit',
+    'map-impact',
+    'review-release',
+  ];
+
+  const index = orderedModes.indexOf(mode);
+  return index >= 0 ? index : orderedModes.length;
 }
 
 function buildExecutionContract(executionMode = null) {
@@ -453,6 +606,7 @@ function buildExecutionContract(executionMode = null) {
 function buildApproachSelectorResult({
   executionMode = null,
   executionScores = null,
+  executionCandidates = null,
 } = {}) {
   if (!executionMode) {
     return null;
@@ -501,6 +655,14 @@ function buildApproachSelectorResult({
     contextPolicy: selectedPolicy.contextPolicy,
     verificationPolicy: executionContract?.verificationPolicy ?? null,
     completionRule: executionContract?.completionRule ?? null,
+    competingMode: executionCandidates?.[1]?.mode ?? null,
+    competingScoreGap: executionCandidates?.length >= 2
+      ? Number(executionCandidates[0]?.score ?? 0) - Number(executionCandidates[1]?.score ?? 0)
+      : null,
+    candidateModes: (executionCandidates ?? []).slice(0, 3).map((entry) => ({
+      mode: entry.mode,
+      score: entry.score,
+    })),
   };
 }
 
@@ -588,6 +750,8 @@ function buildContinuationState({
     repeatCount: reasons.length > 0 ? 1 : 0,
     stuckRisk: null,
     rescueMode: null,
+    wideningBlocked: false,
+    milestonePriority: reasons.length > 0 ? 'normal' : null,
   };
 }
 
@@ -749,7 +913,7 @@ function deriveIntentMode({ promptText = '', commandText = '', targetFile = null
   }
 
   if (concreteTask) {
-    if (/\b(bug|debug|error|crash|broken|failing|stack trace|triage|fix|login|timeout)\b/.test(lower)) {
+    if (/\b(bug|debug|error|crash|broken|failing|stack trace|triage|fix|timeout)\b/.test(lower)) {
       return 'debug-specific';
     }
     if (/\b(review|audit|diff|pr feedback|code review|kiem tra|soat)\b/.test(raw)) {