@ngockhoale/ukit 1.4.1 → 1.4.2

package/CHANGELOG.md

@@ -4,6 +4,26 @@ All notable changes to UKit are documented here.
 
 ## Unreleased
 
+## 1.4.2 - 2026-05-10
+
+### Added
+
+- Added a quality-first internal orchestration ladder with seven execution layers: `tiny-fix`, `local-fix`, `local-build`, `find-cause`, `shared-edit`, `map-impact`, and `review-release`.
+- Added structured route continuity state so installed helpers and hooks can carry `completionState`, `continuationState`, and stuck-lane rescue signals across turns instead of forgetting unfinished execution debt.
+- Added orchestration/runtime config defaults and Codex adapter metadata for the internal advisor/orchestrator contract while keeping the teammate workflow centered on `ukit install`.
+
+### Fixed
+
+- Kept context routing and context-mode hints internal so UKit no longer surfaces `pull-indexed-context`, `resolve-context`, or `mode=LITE/FULL` in the main route prose for ordinary work.
+- Preserved the structured route state (`nextActionType`, `helperHint`, `contextMode`) for internal orchestration, while removing the user/model-facing leakage that was nudging sessions into micro-step stop-and-wait behavior.
+- Hardened implementation routing and installed agent guidance so explicit implement/apply/fix requests keep moving through bounded read → edit → verify flow instead of stopping after read-only inspection.
+- Escalated repeated unfinished returns with `repeatCount`, `stuckRisk`, and `rescueMode` so UKit can rescue a stuck lane instead of quietly looping the same partial milestone.
+- Fixed the installed `route-task` continuation handoff so previous route summaries are carried through correctly during rescue escalation.
+
+### Tests
+
+- Added/updated route, hook, reinject, install-pipeline, and package-artifact coverage for internal helper hiding, continuation rescue escalation, and the expanded orchestration contract.
+
 ## 1.4.1 - 2026-05-09
 
 - Completed cleanup and 1.4.1 template alignment.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngockhoale/ukit",
-  "version": "1.4.1",
+  "version": "1.4.2",
   "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.1',
+    version: '1.4.2',
     agent: 'claude-code',
     autonomy: {
       level: 'balanced',
@@ -112,6 +112,59 @@ export function buildDefaultRuntimeConfig(overrides = {}) {
       advisorEnabled: true,
       maxAdvisorCalls: 3,
     },
+    orchestration: {
+      enabled: true,
+      orchestratorModel: 'claude-sonnet-4-6',
+      advisorEnabled: true,
+      contracts: {
+        'tiny-fix': {
+          maxReadPasses: 0,
+          maxContextPulls: 0,
+          verificationPolicy: 'minimal-or-targeted',
+          completionRule: 'never-claim-done-without-write',
+          delegationPolicy: 'disallow',
+        },
+        'local-fix': {
+          maxReadPasses: 1,
+          maxContextPulls: 1,
+          verificationPolicy: 'targeted-if-covered',
+          completionRule: 'require-write',
+          delegationPolicy: 'disallow',
+        },
+        'local-build': {
+          maxReadPasses: 2,
+          maxContextPulls: 1,
+          verificationPolicy: 'targeted-if-covered',
+          completionRule: 'require-write',
+          delegationPolicy: 'disallow-by-default',
+        },
+        'find-cause': {
+          maxReadPassesBeforeReassess: 3,
+          verificationPolicy: 'root-cause-then-targeted',
+          completionRule: 'never-claim-fixed-without-write-and-verification',
+          delegationPolicy: 'allow-specialized-debug-lane',
+        },
+        'shared-edit': {
+          maxReadPasses: 2,
+          maxContextPulls: 2,
+          verificationPolicy: 'targeted-then-widen-on-risk',
+          completionRule: 'require-write-and-verification',
+          delegationPolicy: 'allow-qualified-sidecar',
+        },
+        'map-impact': {
+          maxReadPasses: 3,
+          maxContextPulls: 3,
+          verificationPolicy: 'impact-first-then-targeted-then-widen-on-risk',
+          completionRule: 'require-impact-evidence-before-edit-claim',
+          delegationPolicy: 'allow-impact-sidecar',
+        },
+        'review-release': {
+          verificationPolicy: 'evidence-first',
+          completionRule: 'report-findings-not-implementation',
+          delegationPolicy: 'allow-review-sidecar',
+        },
+      },
+    },
     memory: {
       enabled: true,
       autoCapture: true,
@@ -278,6 +331,17 @@ export function validateRuntimeConfig(config) {
     pushPositiveNumberError(errors, config.router.maxAdvisorCalls, 'router.maxAdvisorCalls');
   }
 
+  if (!isPlainObject(config.orchestration)) {
+    errors.push('orchestration must be an object.');
+  } else {
+    pushBooleanError(errors, config.orchestration.enabled, 'orchestration.enabled');
+    pushNonEmptyStringError(errors, config.orchestration.orchestratorModel, 'orchestration.orchestratorModel');
+    pushBooleanError(errors, config.orchestration.advisorEnabled, 'orchestration.advisorEnabled');
+    if (!isPlainObject(config.orchestration.contracts)) {
+      errors.push('orchestration.contracts must be an object.');
+    }
+  }
+
   if (!isPlainObject(config.memory)) {
     errors.push('memory must be an object.');
   } else {

package/src/index/taskRouting.js

@@ -52,6 +52,21 @@ export async function deriveTaskRoute({
     selectedIds,
     targetFile: normalizedTarget,
   });
+  const executionScores = deriveExecutionScores({
+    promptText: normalizedPrompt,
+    commandText: normalizedCommand,
+    targetFile: normalizedTarget,
+    intentMode,
+    taskType: inferredTaskType,
+  });
+  const executionMode = deriveExecutionMode({
+    promptText: normalizedPrompt,
+    commandText: normalizedCommand,
+    targetFile: normalizedTarget,
+    intentMode,
+    taskType: inferredTaskType,
+    executionScores,
+  });
   const preservedPrompt = normalizedPrompt || String(lastExplicitUserPromptText || '').trim();
   const degradedWarnings = [];
   let contextResult = null;
@@ -128,11 +143,14 @@ export async function deriveTaskRoute({
       taskType: inferredTaskType,
       intentMode,
       autonomyLevel,
+      executionScores,
+      executionMode,
     },
     contextRecommendation,
     verificationRecommendation,
     nextAction,
   });
+  const approachSelector = routeSummary?.approachSelector ?? null;
 
   return {
     activeSkills,
@@ -145,7 +163,10 @@ export async function deriveTaskRoute({
       taskType: inferredTaskType,
       intentMode,
       autonomyLevel,
+      executionScores,
+      executionMode,
     },
+    approachSelector,
     contextRecommendation,
     verificationRecommendation,
     nextAction,
@@ -186,6 +207,21 @@ export function buildRouteSummary({
   const compactHelperLane = nextAction?.type === 'pull-indexed-context'
     && typeof contextRecommendation?.command === 'string'
     && contextRecommendation.command.trim();
+  const executionMode = routingContext.executionMode ?? null;
+  const executionScores = routingContext.executionScores ?? null;
+  const approachSelector = buildApproachSelectorResult({
+    executionMode,
+    executionScores,
+  });
+  const executionContract = buildExecutionContract(executionMode);
+  const completionState = buildCompletionState({
+    executionMode,
+    verificationRecommendation,
+  });
+  const continuationState = buildContinuationState({
+    nextActionType: nextAction?.type ?? null,
+    completionState,
+  });
   const helperHint = compactHelperHint(
     compactHelperLane
       ? contextRecommendation?.command
@@ -204,7 +240,6 @@ export function buildRouteSummary({
     editGuardHint ? `editGuard=${editGuardHint}` : null,
     delegationRecommendation?.hint ? `delegate=${delegationRecommendation.hint}` : null,
     policyMode ? `policy=${policyMode}` : null,
-    contextMode ? `mode=${contextMode}` : null,
   ].filter(Boolean).join(' | ');
 
   return {
@@ -213,6 +248,12 @@ export function buildRouteSummary({
     preferredOrder,
     policyMode,
     editGuardHint,
+    executionMode,
+    executionScores,
+    approachSelector,
+    executionContract,
+    completionState,
+    continuationState,
     intentMode: routingContext.intentMode ?? null,
     delegateHint: delegationRecommendation?.hint ?? null,
     nextActionType: nextAction?.type ?? null,
@@ -229,6 +270,327 @@ function deriveContextMode(taskType) {
   return null;
 }
 
+function deriveExecutionScores({
+  promptText = '',
+  commandText = '',
+  targetFile = null,
+  intentMode = null,
+  taskType = null,
+} = {}) {
+  const raw = `${promptText ?? ''}\n${commandText ?? ''}`.toLowerCase();
+  const sharedRisk = isSharedImpactFile(targetFile);
+  const directTransformSignal = /\b(change|replace|set|rename|convert|swap)\b/.test(raw) || /\bchange\s+.+\s+to\s+.+\b/.test(raw);
+  const smallFixSignal = /\b(adjust|tweak|patch|fix|modify|update|apply)\b/.test(raw);
+  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);
+
+  let editCertainty = 0;
+  if (directTransformSignal) {
+    editCertainty = 3;
+  } else if (smallFixSignal) {
+    editCertainty = 2;
+  } else if (buildSignal) {
+    editCertainty = 1;
+  }
+
+  let investigationNeed = 0;
+  if (debugSignal) {
+    investigationNeed = 3;
+  } else if (/\b(failing|broken|error|crash|timeout)\b/.test(raw)) {
+    investigationNeed = 2;
+  }
+
+  let blastRadius = 0;
+  if (sharedRisk) {
+    blastRadius = 3;
+  } else if (taskType === 'non-trivial' || taskType === 'shared-simple') {
+    blastRadius = 2;
+  } else if (!targetFile) {
+    blastRadius = 1;
+  }
+
+  let verificationBurden = 1;
+  if (reviewSignal || sharedRisk) {
+    verificationBurden = 3;
+  } else if (debugSignal || taskType === 'non-trivial') {
+    verificationBurden = 2;
+  }
+
+  let ambiguity = 0;
+  if (!targetFile) {
+    ambiguity = 2;
+  } else if (buildSignal && !directTransformSignal && !smallFixSignal) {
+    ambiguity = 1;
+  }
+
+  return {
+    editCertainty,
+    investigationNeed,
+    blastRadius,
+    verificationBurden,
+    ambiguity,
+  };
+}
+
+function deriveExecutionMode({
+  promptText = '',
+  commandText = '',
+  targetFile = null,
+  intentMode = null,
+  executionScores = {},
+} = {}) {
+  const raw = `${promptText ?? ''}\n${commandText ?? ''}`.toLowerCase();
+  const scores = executionScores;
+  const explicitReviewLead = /\b(review this|audit this|review\b.*\brelease readiness|verify\b.*\brelease readiness)\b/.test(raw);
+
+  if (intentMode === 'review-specific' || explicitReviewLead) {
+    return 'review-release';
+  }
+
+  if (scores.blastRadius >= 3 && (scores.ambiguity >= 2 || scores.investigationNeed >= 2)) {
+    return 'map-impact';
+  }
+
+  if (scores.blastRadius >= 3 || isSharedImpactFile(targetFile)) {
+    return 'shared-edit';
+  }
+
+  if (scores.investigationNeed >= 3) {
+    return 'find-cause';
+  }
+
+  if (
+    scores.editCertainty >= 3
+    && scores.ambiguity === 0
+    && scores.blastRadius === 0
+    && scores.verificationBurden <= 1
+  ) {
+    return 'tiny-fix';
+  }
+
+  if (
+    /\b(build|create|add|implement|feature|summary card)\b/.test(raw)
+    && scores.investigationNeed === 0
+    && scores.blastRadius < 3
+  ) {
+    return 'local-build';
+  }
+
+  if (scores.editCertainty >= 2 && scores.investigationNeed === 0 && scores.blastRadius === 0) {
+    return 'local-fix';
+  }
+
+  return 'local-build';
+}
+
+function buildExecutionContract(executionMode = null) {
+  if (!executionMode) {
+    return null;
+  }
+
+  const contracts = {
+    'tiny-fix': {
+      maxReadPasses: 0,
+      maxContextPulls: 0,
+      verificationPolicy: 'minimal-or-targeted',
+      completionRule: 'never-claim-done-without-write',
+      delegationPolicy: 'disallow',
+      completionEvidence: ['write-evidence'],
+    },
+    'local-fix': {
+      maxReadPasses: 1,
+      maxContextPulls: 1,
+      verificationPolicy: 'targeted-if-covered',
+      completionRule: 'require-write',
+      delegationPolicy: 'disallow',
+      completionEvidence: ['write-evidence'],
+    },
+    'local-build': {
+      maxReadPasses: 2,
+      maxContextPulls: 1,
+      verificationPolicy: 'targeted-if-covered',
+      completionRule: 'require-write',
+      delegationPolicy: 'disallow-by-default',
+      completionEvidence: ['write-evidence'],
+    },
+    'find-cause': {
+      maxReadPassesBeforeReassess: 3,
+      verificationPolicy: 'root-cause-then-targeted',
+      completionRule: 'never-claim-fixed-without-write-and-verification',
+      delegationPolicy: 'allow-specialized-debug-lane',
+      completionEvidence: ['write-evidence', 'verification-evidence'],
+    },
+    'shared-edit': {
+      maxReadPasses: 2,
+      maxContextPulls: 2,
+      verificationPolicy: 'targeted-then-widen-on-risk',
+      completionRule: 'require-write-and-verification',
+      delegationPolicy: 'allow-qualified-sidecar',
+      completionEvidence: ['write-evidence', 'verification-evidence'],
+      mirrorConsistencyRequired: true,
+    },
+    'map-impact': {
+      maxReadPasses: 3,
+      maxContextPulls: 3,
+      verificationPolicy: 'impact-first-then-targeted-then-widen-on-risk',
+      completionRule: 'require-impact-evidence-before-edit-claim',
+      delegationPolicy: 'allow-impact-sidecar',
+      completionEvidence: ['impact-evidence', 'write-evidence', 'verification-evidence'],
+      mirrorConsistencyRequired: true,
+    },
+    'review-release': {
+      verificationPolicy: 'evidence-first',
+      completionRule: 'report-findings-not-implementation',
+      delegationPolicy: 'allow-review-sidecar',
+      completionEvidence: ['verification-evidence'],
+    },
+  };
+
+  return contracts[executionMode] ? { ...contracts[executionMode] } : null;
+}
+
+function buildApproachSelectorResult({
+  executionMode = null,
+  executionScores = null,
+} = {}) {
+  if (!executionMode) {
+    return null;
+  }
+
+  const executionContract = buildExecutionContract(executionMode);
+  const policyByMode = {
+    'tiny-fix': {
+      riskLevel: 'minimal',
+      contextPolicy: 'confirm-target-only',
+    },
+    'local-fix': {
+      riskLevel: 'local',
+      contextPolicy: 'bounded-local',
+    },
+    'local-build': {
+      riskLevel: 'local',
+      contextPolicy: 'bounded-with-related-files',
+    },
+    'find-cause': {
+      riskLevel: 'investigation',
+      contextPolicy: 'trace-then-bounded-context',
+    },
+    'shared-edit': {
+      riskLevel: 'shared',
+      contextPolicy: 'bounded-with-related-tests',
+    },
+    'map-impact': {
+      riskLevel: 'wide',
+      contextPolicy: 'impact-map-first',
+    },
+    'review-release': {
+      riskLevel: 'release',
+      contextPolicy: 'evidence-review-only',
+    },
+  };
+  const selectedPolicy = policyByMode[executionMode] ?? {
+    riskLevel: 'local',
+    contextPolicy: 'bounded-local',
+  };
+
+  return {
+    executionMode,
+    executionScores: executionScores ? { ...executionScores } : null,
+    riskLevel: selectedPolicy.riskLevel,
+    contextPolicy: selectedPolicy.contextPolicy,
+    verificationPolicy: executionContract?.verificationPolicy ?? null,
+    completionRule: executionContract?.completionRule ?? null,
+  };
+}
+
+function buildCompletionState({ executionMode = null, verificationRecommendation = null } = {}) {
+  if (!executionMode) {
+    return null;
+  }
+
+  const contract = buildExecutionContract(executionMode);
+  const missingEvidence = [...(contract?.completionEvidence ?? [])];
+  const requiresVerification = missingEvidence.includes('verification-evidence');
+  if (
+    requiresVerification
+    && verificationRecommendation
+    && !(verificationRecommendation.commands?.length || verificationRecommendation.fallbackCommands?.length)
+  ) {
+    missingEvidence.push('verification-plan');
+  }
+
+  let reason = 'completion evidence is still required';
+  if (['tiny-fix', 'local-fix', 'local-build', 'shared-edit'].includes(executionMode)) {
+    reason = 'implement request has not produced an edit yet';
+  } else if (executionMode === 'review-release') {
+    reason = 'review/release evidence is still required before final claim';
+  } else if (executionMode === 'map-impact') {
+    reason = 'impact evidence is still required before safe completion claim';
+  }
+
+  return {
+    claimAllowed: false,
+    missingEvidence,
+    reason,
+  };
+}
+
+function buildContinuationState({
+  nextActionType = null,
+  completionState = null,
+} = {}) {
+  const reasons = [];
+  const missingEvidence = unique(completionState?.missingEvidence ?? []);
+
+  if (nextActionType === 'pull-indexed-context') {
+    reasons.push('pending-bounded-context');
+  }
+
+  if (nextActionType === 'read-skill-instructions') {
+    reasons.push('pending-skill-read');
+  }
+
+  if (missingEvidence.includes('write-evidence')) {
+    reasons.push('missing-write-evidence');
+  }
+
+  if (missingEvidence.includes('verification-evidence') || missingEvidence.includes('verification-plan')) {
+    reasons.push('missing-verification-evidence');
+  }
+
+  if (missingEvidence.includes('impact-evidence')) {
+    reasons.push('missing-impact-evidence');
+  }
+
+  const nextMilestone = reasons.includes('pending-bounded-context')
+    ? 'complete-bounded-context-then-continue'
+    : (
+      reasons.includes('missing-write-evidence')
+        ? 'produce-write-evidence'
+        : (
+          reasons.includes('missing-verification-evidence')
+            ? 'produce-verification-evidence'
+            : (
+              reasons.includes('missing-impact-evidence')
+                ? 'produce-impact-evidence'
+                : null
+            )
+        )
+    );
+
+  return {
+    required: reasons.length > 0,
+    reasons,
+    doneLanguageBlocked: reasons.length > 0,
+    stopAllowedOnlyFor: reasons.length > 0 ? ['real-blocker'] : [],
+    nextMilestone,
+    repeatCount: reasons.length > 0 ? 1 : 0,
+    stuckRisk: null,
+    rescueMode: null,
+  };
+}
+
 async function selectActiveSkills({ rootDir, promptText, commandText, targetFile, intentMode = null }) {
   const routeSignals = {
     promptRawText: String(promptText || '').toLowerCase(),
@@ -269,6 +631,10 @@ function shouldKeepRouteEntryForIntent(entry, intentMode) {
     return false;
   }
 
+  if (entry.id === 'code-review' && intentMode === 'implement-specific') {
+    return false;
+  }
+
   return true;
 }
 
@@ -313,6 +679,10 @@ function deriveSkillPriorityBoost(skillId, routeSignals = {}) {
     routeSignals.commandRawText ?? '',
     routeSignals.commandNormalizedText ?? '',
   ].join('\n');
+  const rawPromptSignals = [
+    routeSignals.promptRawText ?? '',
+    routeSignals.commandRawText ?? '',
+  ].join('\n');
 
   if (
     skillId === 'testing-quality'
@@ -321,6 +691,14 @@ function deriveSkillPriorityBoost(skillId, routeSignals = {}) {
     return 1;
   }
 
+  if (
+    skillId === 'code-review'
+    && /\b(implement|build|create|add|ship|deliver|refactor|integrate|integration|scaffold|feature|update|change|modify|inject|apply)\b/.test(rawPromptSignals)
+    && !/\b(review|audit|diff|pr feedback|code review|kiem tra|soat)\b/.test(rawPromptSignals)
+  ) {
+    return -4;
+  }
+
   return 0;
 }
 
@@ -374,10 +752,10 @@ function deriveIntentMode({ promptText = '', commandText = '', targetFile = null
     if (/\b(bug|debug|error|crash|broken|failing|stack trace|triage|fix|login|timeout)\b/.test(lower)) {
       return 'debug-specific';
     }
-    if (/\b(review|audit|diff|pr feedback|code review|kiem tra|soat)\b/.test(lower)) {
+    if (/\b(review|audit|diff|pr feedback|code review|kiem tra|soat)\b/.test(raw)) {
       return 'review-specific';
     }
-    if (/\b(implement|build|create|add|ship|deliver|refactor|integrate|integration|scaffold|feature)\b/.test(lower)) {
+    if (/\b(implement|build|create|add|ship|deliver|refactor|integrate|integration|scaffold|feature|update|change|modify|inject|apply)\b/.test(raw)) {
       return 'implement-specific';
     }
   }
@@ -409,7 +787,7 @@ function hasConcreteTaskSignal(lower, raw, targetFile, { taskQueueNext = false }
     return true;
   }
 
-  return /\b(bug|debug|error|crash|broken|failing|stack trace|triage|fix|implement|build|create|add|ship|deliver|refactor|integrate|integration|scaffold|feature|review|audit|diff|pr feedback|code review|auth|login|api|endpoint|test|spec)\b/.test(lower)
+  return /\b(bug|debug|error|crash|broken|failing|stack trace|triage|fix|implement|build|create|add|ship|deliver|refactor|integrate|integration|scaffold|feature|update|change|modify|inject|apply|review|audit|diff|pr feedback|code review|auth|login|api|endpoint|test|spec)\b/.test(lower)
     || /\b(sửa|fix|lỗi|bug|debug|implement|cài|thêm|review|kiểm tra|soát|đăng nhập)\b/.test(raw);
 }
 

package/templates/.claude/hooks/reinject-context.sh

@@ -23,6 +23,8 @@ cat <<'FALLBACK'
 - Auto-activate matching project-local skills from prompt + tool/file signals.
 - Reuse existing code. Use yarn. Run tests after code changes.
 - Scope-based verification: match verify effort to change scope.
+- If a routed lane still needs `pull-indexed-context` or still has completion debt, treat that as internal continuation work, not a stopping point.
+- Do not end with “done/applied/fixed” language while write/verification evidence is still missing unless a real blocker is found.
 - If broad verification gets blocked by a Claude hook, follow the routed targeted order or ask for explicit full-suite confirmation.
 === END ===
 FALLBACK

package/templates/.claude/hooks/session-start.md

@@ -8,6 +8,8 @@
 - Auto-activate matching project-local skills from prompt + tool/file signals; end users should not need to know skill names.
 - After routing skill + context, prefer verification derived from related tests/context bundle before broad test runs.
 - Follow routed verification policy: auto-run targeted checks when localized, widen in order for shared/risky scope, and ask before blanket broad runs when no related test evidence exists.
+- Nếu route còn `pull-indexed-context`, coi đó là bước nội bộ để tiếp tục chứ không phải điểm dừng; đọc xong context thì tiếp tục edit/verify ngay khi an toàn.
+- Không được kết thúc bằng “done/applied/fixed” khi còn thiếu write-evidence hoặc verification-evidence, trừ khi có blocker thật.
 - If Claude blocks a broad verification command, do not fight the hook blindly — use the targeted indexed lane first or get explicit confirmation for a full-suite run.
 - Nếu mơ hồ hoặc risk cao (security/migration/auth/data-loss/uninstall-core), escalate đọc sâu hơn hoặc hỏi 1 câu clarify ngắn.
 - Docs là navigation aid, source code là ground truth. Nếu mâu thuẫn → source thắng, update docs trước khi tiếp tục.