@ngockhoale/ukit 3.4.29 → 3.4.31

package/template_project/.claude/ukit/index/route-task.mjs

@@ -1,4 +1,5 @@
 #!/usr/bin/env node
+import { classifyTaskSemantics, applyTaskSemantics } from './route-resolver.mjs';
 import fs from 'node:fs/promises';
 // `fsSync` aliases the SAME fs module — used ONLY by the process.on('exit')
 // audit-sidecar fold below (event loop is gone; async cannot run) and by the
@@ -70,7 +71,7 @@ import {
   deriveRouteFields,
 } from './route-resolver.mjs';
 import { applyMeasuredTier, loadTierMap } from './tier-selection.mjs';
-import { resolvePlaybookRoute, loadRegistry } from './playbook-registry.mjs';
+import { resolvePlaybookRoute, loadRegistry, deriveRouteOrchestrationSignals } from './playbook-registry.mjs';
 import { extractHistorySignalsAsync } from './session-history-extractor.mjs';
 import {
   applyFixLoopEscalation,
@@ -190,6 +191,15 @@ If the user says "new task", re-route — do not treat the message as the next s
 4. Implement the smallest change satisfying the predicate.
 5. Verify against the predicate on the real artifact — not "it compiles".
 6. Widen once: check the impact surface the route named, no broader.
+Reference reuse: when the ask points at an existing surface ("like the Customer
+page", "giống trang X", "same as module Y"), read that surface first and mirror
+its structure — the reference defines the done predicate, not just inspiration.
+Reuse-first: when the ask names an existing function, helper, or pattern
+("using validation from Z", "dùng hàm có sẵn"), reuse it — extend or wrap
+before writing new logic, and name the reuse in the report.
+Clone: when the ask duplicates a feature ("clone the Customer page", "nhân bản
+tính năng X"), copy the source feature's structure, rename and adapt each
+surface, and verify the clone against the same predicate class as the original.
 Reply: what changed, the predicate, the evidence it now holds.
 Ask the human only for: irreversible writes, a genuine preference call no experiment
 settles, or a real dead end. Everything else: do it, report it.`,
@@ -790,6 +800,7 @@ function buildRouteHotCacheState(sharedState = {}) {
   delete routingContext.lastExplicitUserPromptText;
   delete routingContext.commandText;
   delete state.sessionId;
+  delete state.taskInstanceId;
   state.routingContext = routingContext;
   return state;
 }
@@ -886,6 +897,7 @@ function buildReducedRouteState({ input, reason }) {
     source: 'task-route',
     requestKey: null,
     sessionId: input.sessionId ?? null,
+    taskInstanceId: input.taskInstanceId || undefined,
   });
 }
 
@@ -902,7 +914,7 @@ export async function routeTask(input = {}, { signal = null, deadlineMs = null,
   const absoluteRoot = path.resolve(input.rootDir ?? process.cwd());
   const budget = createRouteBudget({ signal, deadlineMs });
   const hotCache = cache ?? createDefaultRouteHotCache({ rootDir: absoluteRoot });
-  const normalizedInput = { ...input, rootDir: absoluteRoot };
+  const normalizedInput = { ...input, rootDir: absoluteRoot, taskInstanceId: input.taskInstanceId || `task-${crypto.randomBytes(16).toString('hex')}` };
   const baseMeta = {
     cacheHit: false,
     reduced: false,
@@ -920,7 +932,7 @@ export async function routeTask(input = {}, { signal = null, deadlineMs = null,
       elapsedMs: Date.now() - budget.startedAt,
       ioReads: budget.ioReads,
     };
-    return { ...state, cacheHit: routeMeta.cacheHit, routeMeta };
+    return { ...state, taskInstanceId: normalizedInput.taskInstanceId || state.taskInstanceId || `task-${crypto.randomBytes(16).toString('hex')}`, cacheHit: routeMeta.cacheHit, routeMeta };
   };
   const reduce = (reason) => finish(
     buildReducedRouteState({ input: normalizedInput, reason }),
@@ -1070,6 +1082,7 @@ export async function routeTask(input = {}, { signal = null, deadlineMs = null,
     requestKey,
     indexGeneratedAtMs,
     sessionId: normalizedInput.sessionId ?? null,
+    taskInstanceId: normalizedInput.taskInstanceId || undefined,
   });
 
   // Phase 3 — store under a POST-route fingerprint so the next identical call (whose own
@@ -1117,7 +1130,12 @@ async function main() {
   const sessionId = normalizeSessionId(readFlagValue(args, '--session-id'));
   const previousStateOwnershipBlocked = hasRouteStateOwner(persistedPreviousState)
     && !isRouteStateCompatible(persistedPreviousState, sessionId);
-  const previousState = previousStateOwnershipBlocked ? {} : persistedPreviousState;
+  const sessionSidecar = previousStateOwnershipBlocked && sessionId
+    ? await readJson(path.join(rootDir, '.ukit', 'storage', 'cache', 'route-state', `${sessionId.replace(/[^a-zA-Z0-9._-]/g, '_').slice(0, 96) || 'default'}.json`), {})
+    : {};
+  const previousState = previousStateOwnershipBlocked
+    ? (isRouteStateCompatible(sessionSidecar, sessionId) && sessionSidecar.sessionId === sessionId ? sessionSidecar : {})
+    : persistedPreviousState;
   const escalationConfig = await readMergedConfig(rootDir);
   const targetFile = readFlagValue(args, '--target');
   const taskType = readFlagValue(args, '--type');
@@ -1135,7 +1153,7 @@ async function main() {
   const commandNamespace = adapter === 'codex'
     ? '.codex'
     : (adapter === 'antigravity' ? '.antigravity' : '.claude');
-  const promptText = args
+  let promptText = args
     .filter((arg, index) => !isFlagOrValue(args, index))
     .join(' ')
     .trim();
@@ -1146,6 +1164,15 @@ async function main() {
     return;
   }
 
+  const continuingTask = Boolean(previousState.taskInstanceId && lastPrompt
+    && /^(?:continue|resume|keep going|proceed|tiep tuc)(?:\s+(?:the task|this task|previous task))?[.!]?$/i.test(
+      promptText.normalize('NFD').replace(/[̀-ͯ]/g, '').trim()));
+  const taskInstanceId = readFlagValue(args, '--task-instance-id')
+    || (((['skill-router', 'skill-router-provisional'].includes(previousState.source)
+      && previousState.routingContext?.lastExplicitUserPromptText === promptText) || continuingTask)
+      && previousState.taskInstanceId
+      ? previousState.taskInstanceId : `task-${crypto.randomBytes(16).toString('hex')}`);
+  if (continuingTask) promptText = lastPrompt;
   const preparedRoute = await prepareTaskRoute({
     rootDir,
     promptText,
@@ -1240,7 +1267,12 @@ async function main() {
         historySignals,
       };
     }
-    printRouteState(reusableState);
+    const sharedState = { ...reusableState, taskInstanceId, source: 'task-route', ts: Date.now() };
+    if (canPersistRouteState) {
+      try { await persistSharedRouteState(sharedStatePath, sharedState); }
+      catch (error) { console.error(`route-task: state persist degraded — ${error?.message ?? error}`); }
+    }
+    printRouteState(sharedState);
     return;
   }
 
@@ -1322,6 +1354,7 @@ async function main() {
     }
     const sharedState = {
       ...cachedRouteState,
+      taskInstanceId,
       ...(sessionId ? { sessionId } : {}),
       source: 'task-route',
       ts: Date.now(),
@@ -1382,6 +1415,7 @@ async function main() {
     requestKey,
     indexGeneratedAtMs,
     sessionId,
+    taskInstanceId,
   });
   if (canPersistRouteState) {
     // Advisory persistence — an EACCES/ENOSPC under the lock must never lose
@@ -1464,6 +1498,56 @@ async function prepareTaskRoute({
   const normalizedPrompt = String(promptText || '').trim();
   const normalizedCommand = String(commandText || '').trim();
   const normalizedTarget = normalizeRelativeFile(absoluteRoot, targetFile);
+  // Greeting fast-path: a pure social turn short-circuits BEFORE any skill
+  // scan, index work or vision resolution. executionScores/candidates keep
+  // the zero-signal shape downstream consumers expect; the mode is stamped
+  // informational and routingContext.greetingFastPath tells finalizeTaskRoute
+  // to skip the playbook scan and the decision-plane shadow.
+  if (isPureGreetingPrompt({
+    promptText: normalizedPrompt,
+    commandText: normalizedCommand,
+    targetFile: normalizedTarget,
+  })) {
+    const greetingScores = deriveExecutionScores({
+      promptText: '',
+      commandText: '',
+      targetFile: null,
+      intentMode: null,
+      taskType: '',
+    });
+    return {
+      rootDir: absoluteRoot,
+      activeSkills: [],
+      routingContext: {
+        adapter: String(adapter || 'claude').trim() || 'claude',
+        promptText: normalizedPrompt,
+        lastExplicitUserPromptText: normalizedPrompt || String(lastExplicitUserPromptText || '').trim(),
+        commandText: normalizedCommand,
+        targetFile: null,
+        contextIntent: '',
+        taskType: '',
+        intentMode: null,
+        executionScores: greetingScores,
+        executionCandidates: buildExecutionModeCandidates({
+          promptText: '',
+          commandText: '',
+          targetFile: null,
+          intentMode: null,
+          executionScores: greetingScores,
+        }),
+        executionMode: 'informational',
+        greetingFastPath: true,
+        orchestrationSignals: deriveRouteOrchestrationSignals({
+          promptText: normalizedPrompt,
+          commandText: normalizedCommand,
+        }),
+        visionLane: false,
+        visionReason: null,
+        visionModel: null,
+        visionAdvisory: null,
+      },
+    };
+  }
   const intentMode = deriveIntentMode({
     promptText: normalizedPrompt,
     commandText: normalizedCommand,
@@ -1520,6 +1604,10 @@ async function prepareTaskRoute({
     targetFile: normalizedTarget,
     visionFlag: Boolean(visionFlag),
   });
+  const orchestrationSignals = deriveRouteOrchestrationSignals({
+    promptText: normalizedPrompt,
+    commandText: normalizedCommand,
+  });
 
   return {
     rootDir: absoluteRoot,
@@ -1536,6 +1624,7 @@ async function prepareTaskRoute({
       executionScores,
       executionCandidates,
       executionMode,
+      orchestrationSignals,
       visionLane: Boolean(visionInfo?.visionLane),
       visionReason: visionInfo?.visionReason ?? null,
       visionModel: visionInfo?.visionModel ?? null,
@@ -1727,6 +1816,9 @@ export async function runRouteDecisionShadow({
 } = {}) {
   const stage = resolveDecisionPlaneStage(config);
   if (stage === 'off') return null;
+  // Greeting fast-path: a social turn carries nothing the decision plane could
+  // answer — never spawn the adapter shadow for one.
+  if (routingContext?.greetingFastPath) return null;
   const questions = buildShadowDecisionQuestions(config);
   if (questions.length === 0) {
     return { stage, outcomeClass: 'skipped', decisionKeys: [], probabilityBands: {},
@@ -2046,6 +2138,9 @@ export async function emitResumableRun({
 } = {}) {
   const stage = resolveResumableRunStage(config);
   if (stage === 'off') return null;
+  // Greeting fast-path: a social turn starts no goal — never persist a
+  // resumable run record (or ledger pointer) for one.
+  if (routingContext?.greetingFastPath) return null;
   try {
     const taskBoundary = resumableTaskBoundary(routingContext);
     const taskId = resumableTaskId({ sessionId, taskBoundary });
@@ -2205,34 +2300,45 @@ async function finalizeTaskRoute({
   });
   // TASK-007 (BL-009): playbook resolution rides the same route signals —
   // the registry degrades to playbookId: null + reason and never blocks the
-  // route. Same merge the hook performs via mergeResolverFields.
+  // route. Same merge the hook performs via mergeResolverFields. A greeting
+  // fast-path route skips the registry scan entirely: no playbook can claim a
+  // social turn, so the route records the skip honestly (reason 'greeting').
   try {
-    const playbookResolution = await resolvePlaybookRoute({
-      promptText: routingContext.promptText,
-      commandText: routingContext.commandText,
-      targetFile: routingContext.targetFile ?? null,
-      executionMode: routingContext.executionMode ?? null,
-      intentMode: routingContext.intentMode ?? null,
-      escalationTriggers: routeSummary?.escalationTriggers ?? [],
-      projectRoot: absoluteRoot,
-      homeDir: os.homedir(),
-    });
-    routeSummary.playbookId = playbookResolution.playbookId;
-    routeSummary.workGroup = playbookResolution.workGroup;
-    routeSummary.playbookReason = playbookResolution.reason;
-    routeSummary.playbookLoad = playbookResolution.playbookLoad;
-    // BL-010 (ARCH Task Contract): `path` = direct|handoff; `fastPathKind` is
-    // the ARCH `none|small-feature|bug-fix` enum — named Kind because
-    // routeSummary.fastPath already carries the FR-004 eligibility object.
-    // Keys land only when a routing-table row claimed the prompt (workGroup
-    // non-null) — identical key-presence to deriveTaskRoute so schema-off /
-    // no-match routes never grow stray keys.
-    if (playbookResolution.workGroup != null) {
-      routeSummary.path = playbookResolution.path;
-      routeSummary.fastPathKind = playbookResolution.fastPath;
-    } else {
+    if (routingContext.greetingFastPath) {
+      routeSummary.playbookId = null;
+      routeSummary.workGroup = null;
+      routeSummary.playbookReason = 'greeting';
+      routeSummary.playbookLoad = null;
       delete routeSummary.path;
       delete routeSummary.fastPathKind;
+    } else {
+      const playbookResolution = await resolvePlaybookRoute({
+        promptText: routingContext.promptText,
+        commandText: routingContext.commandText,
+        targetFile: routingContext.targetFile ?? null,
+        executionMode: routingContext.executionMode ?? null,
+        intentMode: routingContext.intentMode ?? null,
+        escalationTriggers: routeSummary?.escalationTriggers ?? [],
+        projectRoot: absoluteRoot,
+        homeDir: os.homedir(),
+      });
+      routeSummary.playbookId = playbookResolution.playbookId;
+      routeSummary.workGroup = playbookResolution.workGroup;
+      routeSummary.playbookReason = playbookResolution.reason;
+      routeSummary.playbookLoad = playbookResolution.playbookLoad;
+      // BL-010 (ARCH Task Contract): `path` = direct|handoff; `fastPathKind` is
+      // the ARCH `none|small-feature|bug-fix` enum — named Kind because
+      // routeSummary.fastPath already carries the FR-004 eligibility object.
+      // Keys land only when a routing-table row claimed the prompt (workGroup
+      // non-null) — identical key-presence to deriveTaskRoute so schema-off /
+      // no-match routes never grow stray keys.
+      if (playbookResolution.workGroup != null) {
+        routeSummary.path = playbookResolution.path;
+        routeSummary.fastPathKind = playbookResolution.fastPath;
+      } else {
+        delete routeSummary.path;
+        delete routeSummary.fastPathKind;
+      }
     }
   } catch {
     routeSummary.playbookId = null;
@@ -2982,7 +3088,7 @@ function hasConcreteTaskSignal(lower, raw, targetFile, { taskQueueNext = false }
   }
 
   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)
-    || /(?<![A-Za-z0-9_])(sửa|fix|lỗi|bug|debug|implement|cài|thêm|review|kiểm tra|soát|đăng nhập)(?![A-Za-z0-9_])/.test(raw);
+    || /(?<![A-Za-z0-9_])(sửa|đổi|thay|chỉnh|fix|lỗi|bug|debug|implement|cài|thêm|review|kiểm tra|soát|đăng nhập)(?![A-Za-z0-9_])/.test(raw);
 }
 
 function hasDocsSpecificTaskSignal(lower, raw, targetFile, { taskQueueNext = false } = {}) {
@@ -3694,11 +3800,12 @@ function buildRouteSummary({
     executionScores,
     executionCandidates,
   });
-  const executionContract = buildExecutionContract(executionMode);
+  const executionContract = applyTaskSemantics(buildExecutionContract(executionMode), classifyTaskSemantics({ promptText: routingContext.lastExplicitUserPromptText || routingContext.promptText, targetFile: routingContext.targetFile }));
   const continuousExecution = buildContinuousExecutionPolicy(autonomyLevel);
   const tierLane = buildTierLane({ modelTier: executionContract?.modelTier ?? null });
   const completionState = buildCompletionState({
     executionMode,
+    executionContract,
     verificationRecommendation,
   });
   const continuationState = buildContinuationState({
@@ -3811,6 +3918,11 @@ function buildRouteSummary({
     promptText: routingContext.promptText,
     commandText: routingContext.commandText,
   });
+  // Routing-orchestration signals (reference/clone/SQL/reuse/run-all): the
+  // same booleans the playbook discriminators read. Purely advisory route
+  // metadata — they never set mutation permission by themselves. Conditional
+  // spreads keep every other route byte-identical.
+  const orchestrationSignals = routingContext.orchestrationSignals ?? null;
   return {
     primaryCommands,
     fallbackCommands,
@@ -3876,6 +3988,12 @@ function buildRouteSummary({
     // FR-004 additive field — present only when the prompt is an advisor/reference/replan
     // request, so every other route stays byte-identical.
     ...(advisorIntent ? { advisorIntent } : {}),
+    // Routing-orchestration signals — present only when detected.
+    ...(orchestrationSignals?.referenceTarget ? { referenceTargets: true } : {}),
+    ...(orchestrationSignals?.cloneIntent ? { cloneIntent: true } : {}),
+    ...(orchestrationSignals?.sqlIntent ? { sqlIntent: true } : {}),
+    ...(orchestrationSignals?.reuseFirst ? { reuseFirst: true } : {}),
+    ...(orchestrationSignals?.runAll ? { goalScope: 'run-all' } : {}),
     line: line || 'task=unknown',
   };
 }
@@ -3908,8 +4026,16 @@ function deriveExecutionScores({
 } = {}) {
   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);
+  // Vietnamese edit verbs (\b never matches around diacritics — JS word chars
+  // are ASCII-only), so a small Vietnamese edit like "đổi màu nút thành đỏ"
+  // or "sửa cột …" scores as the small mutating task it is instead of
+  // falling to the all-zero shared-edit floor.
+  const vietnameseEditVerb = /(?<![A-Za-z0-9_])(?:sửa|sua|đổi|doi|thay|chỉnh|chinh|cập\s+nhật|cap\s+nhat)(?![A-Za-z0-9_])/.test(raw);
+  const directTransformSignal = /\b(change|replace|set|rename|convert|swap)\b/.test(raw)
+    || /\bchange\s+.+\s+to\s+.+\b/.test(raw)
+    // "đổi X thành Y" is the Vietnamese "change X to Y".
+    || /(?<![A-Za-z0-9_])(?:đổi|doi|sửa|sua|đặt|dat)\s+[^\n,.;]{1,48}?\s+(?:thành|thanh|to)\s+/.test(raw);
+  const smallFixSignal = /\b(adjust|tweak|patch|fix|modify|update|apply)\b/.test(raw) || vietnameseEditVerb;
   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);
@@ -3917,6 +4043,7 @@ function deriveExecutionScores({
   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)
+    || vietnameseEditVerb
     || /\bchange\s+.+\s+to\s+.+\b/.test(raw);
   const explicitTarget = Boolean(targetFile);
 
@@ -3979,6 +4106,49 @@ function deriveExecutionScores({
   };
 }
 
+// --- Greeting fast-path (routing orchestration) --------------------------------
+// A prompt that is ONLY a greeting/acknowledgment (≤ ~6 words, no task
+// vocabulary, no target file, no command) is a social turn, never work. It
+// used to fall through every signal gate with all-zero scores and the
+// no-target blastRadius=1 floor minted a shared-edit (mutating) lane + the
+// full pipeline (context snapshots, playbook scan, decision shadow). The
+// fast path stamps informational, loads no skill, resolves no playbook and
+// skips the shadow — deterministically, before any I/O.
+// Matched on diacritic-folded tokens so both Vietnamese spellings work.
+const GREETING_CORE_TOKENS = new Set([
+  'hi', 'hello', 'hey', 'yo', 'chao', 'xin', 'good',
+  'morning', 'afternoon', 'evening',
+  'thanks', 'thank', 'cam', 'on', 'bye', 'tam', 'biet',
+]);
+const GREETING_FILLER_TOKENS = new Set([
+  'you', 'u', 'there', 'guys', 'everyone', 'all',
+  'ban', 'cac', 'minh', 'moi', 'nguoi', 'nhe', 'nha', 'nhieu',
+  'ok', 'okay', 'again', 'still',
+]);
+function foldRouteText(text = '') {
+  return String(text ?? '')
+    .toLowerCase()
+    .normalize('NFD')
+    .replace(/[\u0300-\u036f]/g, '')
+    .replace(/\u0111/g, 'd');
+}
+function isPureGreetingPrompt({ promptText = '', commandText = '', targetFile = null } = {}) {
+  const raw = String(promptText ?? '').trim();
+  if (!raw || String(commandText ?? '').trim() || targetFile) return false;
+  if (raw.length > 64) return false;
+  const tokens = foldRouteText(raw).replace(/[^a-z0-9\s]/g, ' ').split(/\s+/).filter(Boolean);
+  if (tokens.length === 0 || tokens.length > 6) return false;
+  let coreHit = false;
+  for (const token of tokens) {
+    if (GREETING_CORE_TOKENS.has(token)) {
+      coreHit = true;
+    } else if (!GREETING_FILLER_TOKENS.has(token)) {
+      return false; // any non-greeting word means this is a task wearing a greeting
+    }
+  }
+  return coreHit;
+}
+
 function isInformationalPrompt({
   promptText = '',
   commandText = '',
@@ -4101,6 +4271,7 @@ function hasAdvisoryOnlyAssertion({ promptText = '', commandText = '' } = {}) {
   // — "giữ nguyên header và sửa lỗi login" keeps 'và sửa lỗi' as a surviving
   // order; an unbounded [^.,;\n]* swallows it and the order escapes the gate.
   // TAIL = lazy strip ending before 'và'/'and' (folded va|and) or EOS.
+  const MUT_VERBS = 'implement|apply|update|modify|add|create|ship|deliver|fix|refactor|remove|delete|rename|change|write|build|make|install|run|deploy|execute|edit|sua|them|tao|xoa|doi|thay\\s+the|cap\\s+nhat|viet|chay|cai|chinh|trien\\s+khai|cau\\s+hinh';
   const TAIL = '[^.,;\\n]*?(?=\\s+(?:va|and)\\s+(?:implement|apply|update|modify|add|create|ship|deliver|fix|refactor|remove|delete|rename|change|write|build|make|install|run|deploy|execute|edit|sua|them|tao|xoa|doi|thay\\s+the|cap\\s+nhat|viet|chay|cai|chinh|trien\\s+khai|cau\\s+hinh)(?![A-Za-z0-9_])|[.,;]|\\s*$)';
   // C89-008: WAIT_TAIL ends a waiting-clause strip before any coordinating
   // conjunction ('và/and/nhưng/rồi/then/later/sau đó') followed by anything up
@@ -4109,7 +4280,12 @@ function hasAdvisoryOnlyAssertion({ promptText = '', commandText = '' } = {}) {
   const WAIT_TAIL = '[^.,;\\n]*?(?=\\s+(?:va|and|nhung|roi|then|later|sau\\s+do)\\s+[^.,;\\n]*?\\b(?:implement|apply|update|modify|add|create|ship|deliver|fix|refactor|remove|delete|rename|change|write|build|make|install|run|deploy|execute|edit|sua|them|tao|xoa|doi|thay\\s+the|cap\\s+nhat|viet|chay|cai|chinh|trien\\s+khai|cau\\s+hinh)(?![A-Za-z0-9_])|[.,;]|\\s*$)';
   const residue = trimmed
     .replace(/"[^"\n]*"|«[^»\n]*»/g, ' ')
+      // "tạo plan" / "create a plan": the plan is the deliverable, not a mutation order.
+      .replace(/\b(?:tao|lap|create|make|write|draft)\s+(?:cho\s+toi\s+)?(?:a\s+|an\s+|the\s+|1\s+|mot\s+)?(?:plan|ke\s+hoach|phuong\s+an|lo\s+trinh|roadmap)\b/g, ' ')
     .replace(/\bwrite-evidence\b|\bverification-evidence\b|\bimpact-evidence\b|\broot-cause\b/g, ' ')
+    // A negated LIST ("do not implement, fix, or refactor"; "khong implement, sua, refactor hay chinh") negates every
+    // verb in the list: the TAIL strip below stops at the first comma, which would leave the later verbs as orders.
+    .replace(new RegExp(`\\b(?:do\\s+not|don't|dont|must\\s+not|never|not|khong(?:\\s+duoc)?|tuyet\\s+doi\\s+khong)\\s+(?:${MUT_VERBS})(?![A-Za-z0-9_])(?:\\s*(?:,|,?\\s+(?:or|and|nor|hay|hoac|va))\\s*(?:(?:${MUT_VERBS})(?![A-Za-z0-9_])|(?:${MUT_VERBS})\\s))*[^.,;\\n\u2014]*?(?=\\s+(?:va|and|then|roi|nhung|but)\\s|[.,;\u2014\\n]|\\s*$)`, 'g'), ' ')
     .replace(new RegExp(`\\bgiu\\s+nguyen\\b${TAIL}`, 'g'), ' ')
     .replace(new RegExp(`\\bmuon\\s+giu\\b${TAIL}`, 'g'), ' ')
     .replace(new RegExp(`\\bkhong\\s+(?!chi\\b)${TAIL}`, 'g'), ' ')
@@ -4129,7 +4305,6 @@ function hasAdvisoryOnlyAssertion({ promptText = '', commandText = '' } = {}) {
     .replace(new RegExp(`\\b(?:de|cho)\\s+${TAIL}`, 'g'), ' ')
     .replace(new RegExp(`\\b(?:yeu\\s+cau|doi\\s+hoi|ep\\s+phai|bat\\s+buoc|bat\\s+phai)\\b${TAIL}`, 'g'), ' ')
     .replace(new RegExp(`\\b(?:demand(?:ed|s)?|require(?:d|s)?|forcing|forced|forcibly|must|has\\s+to|have\\s+to)\\b${TAIL}`, 'g'), ' ')
-    .replace(new RegExp(`\\b(?:do\\s+not|don't|dont|not|never|no)\\s+(?!only\\b)${TAIL}`, 'g'), ' ')
     .replace(new RegExp(`\\b(?:can|nen|phai|duoc)\\s+(?:duoc\\s+)?(?:sua|cau\\s+hinh|thay\\s+doi|cap\\s+nhat|edit|fix|config)\\b${TAIL}`, 'g'), ' ')
     .replace(/\bco\s+(?:can|nen)\s+sua\b[^.,;\n?]*/g, ' ')
     .replace(/(?:^|[.,;\n]\s*)(?:should|shall)\s+(?:i|we|you)\b[^.,;\n?]*/g, ' ')
@@ -4213,6 +4388,7 @@ function hasMutationOrder({ promptText = '', commandText = '' } = {}) {
   // — "giữ nguyên header và sửa lỗi login" keeps 'và sửa lỗi' as a surviving
   // order; an unbounded [^.,;\n]* swallows it and the order escapes the gate.
   // TAIL = lazy strip ending before 'và'/'and' (folded va|and) or EOS.
+  const MUT_VERBS = 'implement|apply|update|modify|add|create|ship|deliver|fix|refactor|remove|delete|rename|change|write|build|make|install|run|deploy|execute|edit|sua|them|tao|xoa|doi|thay\\s+the|cap\\s+nhat|viet|chay|cai|chinh|trien\\s+khai|cau\\s+hinh';
   const TAIL = '[^.,;\\n]*?(?=\\s+(?:va|and)\\s+(?:implement|apply|update|modify|add|create|ship|deliver|fix|refactor|remove|delete|rename|change|write|build|make|install|run|deploy|execute|edit|sua|them|tao|xoa|doi|thay\\s+the|cap\\s+nhat|viet|chay|cai|chinh|trien\\s+khai|cau\\s+hinh)(?![A-Za-z0-9_])|[.,;]|\\s*$)';
   // C89-008: WAIT_TAIL ends a waiting-clause strip before a coordinating
   // conjunction ('và/and/nhưng/rồi/then/later/sau đó') followed by anything up
@@ -4220,10 +4396,15 @@ function hasMutationOrder({ promptText = '', commandText = '' } = {}) {
   const WAIT_TAIL = '[^.,;\\n]*?(?=\\s+(?:va|and|nhung|roi|then|later|sau\\s+do)\\s+[^.,;\\n]*?\\b(?:implement|apply|update|modify|add|create|ship|deliver|fix|refactor|remove|delete|rename|change|write|build|make|install|run|deploy|execute|edit|sua|them|tao|xoa|doi|thay\\s+the|cap\\s+nhat|viet|chay|cai|chinh|trien\\s+khai|cau\\s+hinh)(?![A-Za-z0-9_])|[.,;]|\\s*$)';
   const residue = trimmed
     .replace(/"[^"\n]*"|«[^»\n]*»/g, ' ')
+      // "tạo plan" / "create a plan": the plan is the deliverable, not a mutation order.
+      .replace(/\b(?:tao|lap|create|make|write|draft)\s+(?:cho\s+toi\s+)?(?:a\s+|an\s+|the\s+|1\s+|mot\s+)?(?:plan|ke\s+hoach|phuong\s+an|lo\s+trinh|roadmap)\b/g, ' ')
     // Descriptive state, not an order: "đã/vừa [được] + verb" ("hooks vừa được cài đặt") reports
     // something already done. Only the verb is stripped, so a later imperative still survives.
     .replace(/(?<![a-z0-9_])(?:da|vua)\s+(?:duoc\s+)?(?:implement|apply|update|modify|add|create|ship|deliver|fix|refactor|remove|delete|rename|change|write|build|make|install|run|deploy|execute|edit|sua|them|tao|xoa|doi|thay\s+the|cap\s+nhat|viet|chay|cai|chinh|trien\s+khai|cau\s+hinh)(?![a-z0-9_])/g, ' ')
     .replace(/\bwrite-evidence\b|\bverification-evidence\b|\bimpact-evidence\b|\broot-cause\b/g, ' ')
+    // A negated LIST ("do not implement, fix, or refactor"; "khong implement, sua, refactor hay chinh") negates every
+    // verb in the list: the TAIL strip below stops at the first comma, which would leave the later verbs as orders.
+    .replace(new RegExp(`\\b(?:do\\s+not|don't|dont|must\\s+not|never|not|khong(?:\\s+duoc)?|tuyet\\s+doi\\s+khong)\\s+(?:${MUT_VERBS})(?![A-Za-z0-9_])(?:\\s*(?:,|,?\\s+(?:or|and|nor|hay|hoac|va))\\s*(?:(?:${MUT_VERBS})(?![A-Za-z0-9_])|(?:${MUT_VERBS})\\s))*[^.,;\\n\u2014]*?(?=\\s+(?:va|and|then|roi|nhung|but)\\s|[.,;\u2014\\n]|\\s*$)`, 'g'), ' ')
     .replace(new RegExp(`\\bgiu\\s+nguyen\\b${TAIL}`, 'g'), ' ')
     .replace(new RegExp(`\\bmuon\\s+giu\\b${TAIL}`, 'g'), ' ')
     .replace(new RegExp(`\\bkhong\\s+(?!chi\\b)${TAIL}`, 'g'), ' ')
@@ -4245,7 +4426,6 @@ function hasMutationOrder({ promptText = '', commandText = '' } = {}) {
     .replace(new RegExp(`\\b(?:de|cho)\\s+${TAIL}`, 'g'), ' ')
     .replace(new RegExp(`\\b(?:yeu\\s+cau|doi\\s+hoi|ep\\s+phai|bat\\s+buoc|bat\\s+phai)\\b${TAIL}`, 'g'), ' ')
     .replace(new RegExp(`\\b(?:demand(?:ed|s)?|require(?:d|s)?|forcing|forced|forcibly|must|has\\s+to|have\\s+to)\\b${TAIL}`, 'g'), ' ')
-    .replace(new RegExp(`\\b(?:do\\s+not|don't|dont|not|never|no)\\s+(?!only\\b)${TAIL}`, 'g'), ' ')
     .replace(new RegExp(`\\b(?:can|nen|phai|duoc)\\s+(?:duoc\\s+)?(?:sua|cau\\s+hinh|thay\\s+doi|cap\\s+nhat|edit|fix|config)\\b${TAIL}`, 'g'), ' ')
     .replace(/\bco\s+(?:can|nen)\s+sua\b[^.,;\n?]*/g, ' ')
     .replace(/(?:^|[.,;\n]\s*)(?:should|shall)\s+(?:i|we|you)\b[^.,;\n?]*/g, ' ')
@@ -5263,7 +5443,7 @@ function isFlagOrValue(argv, index) {
   const arg = argv[index];
   if (!arg.startsWith('--')) {
     const prev = argv[index - 1];
-    return Boolean(prev && ['--root', '--target', '--type', '--tool-command', '--last-prompt', '--session-id', '--adapter', '--transcript'].includes(prev));
+    return Boolean(prev && ['--root', '--target', '--type', '--tool-command', '--last-prompt', '--session-id', '--task-instance-id', '--adapter', '--transcript'].includes(prev));
   }
   return true;
 }
@@ -5396,6 +5576,7 @@ function createSharedRouteState({
   source = 'task-route',
   requestKey = null,
   sessionId = null,
+  taskInstanceId = `task-${crypto.randomBytes(16).toString('hex')}`,
 }) {
   const helpers = compactHelpers({
     verificationRecommendation: route.verificationRecommendation ?? null,
@@ -5405,6 +5586,7 @@ function createSharedRouteState({
   return {
     ...(sessionId ? { sessionId } : {}),
     requestKey,
+    taskInstanceId,
     fingerprint: buildRouteStateFingerprint(route),
     ts: Date.now(),
     source,
@@ -5591,6 +5773,14 @@ export function compactRouteSummary(routeSummary = null) {
     // stage-off output stays byte-identical (no key emitted when absent).
     ...(routeSummary.decisionPlane ? { decisionPlane: routeSummary.decisionPlane } : {}),
     ...(routeSummary.advisorIntent ? { advisorIntent: routeSummary.advisorIntent } : {}),
+    // Routing-orchestration signals survive compaction so persisted route
+    // state carries the same reference/clone/SQL/reuse/run-all metadata the
+    // helper emitted. Conditional spreads — absent keeps output identical.
+    ...(routeSummary.referenceTargets ? { referenceTargets: true } : {}),
+    ...(routeSummary.cloneIntent ? { cloneIntent: true } : {}),
+    ...(routeSummary.sqlIntent ? { sqlIntent: true } : {}),
+    ...(routeSummary.reuseFirst ? { reuseFirst: true } : {}),
+    ...(routeSummary.goalScope ? { goalScope: routeSummary.goalScope } : {}),
     // C89 TASK-005: the typed delegation advice survives compaction so
     // persisted route state keeps the shadow verdict. Conditional spread —
     // stage-off output stays byte-identical (no key emitted when absent).
@@ -5618,6 +5808,11 @@ function shouldIncludePreviousContext({
   const taskType = String(routingContext?.taskType ?? '').trim();
   const hasExplicitTarget = Boolean(String(routingContext?.targetFile ?? '').trim());
 
+  // Greeting fast-path: a social turn needs no memory recall.
+  if (routingContext?.greetingFastPath) {
+    return false;
+  }
+
   if (!hasExplicitTarget) {
     return true;
   }
@@ -5750,6 +5945,16 @@ export async function persistSharedRouteState(filePath, state) {
   if (!result.ok) {
     console.error(`route-task: skill-router-state lock ${result.reason} after ${result.waitedMs}ms — state write skipped (fail-closed)`);
   }
+  // Per-session sidecar (see skill-router.sh persistRouteState): survives a concurrent session
+  // overwriting the shared single-slot file. Best-effort; keyed by the state's own owner.
+  const owner = normalizeSessionId(state?.sessionId);
+  if (owner) {
+    try {
+      const root = path.resolve(path.dirname(filePath), '..', '..');
+      const safe = owner.replace(/[^a-zA-Z0-9._-]/g, '_').slice(0, 96) || 'default';
+      await writeJsonAtomic(path.join(root, '.ukit', 'storage', 'cache', 'route-state', `${safe}.json`), state);
+    } catch { /* shared write is authoritative */ }
+  }
   return result;
 }
 

package/template_project/.claude/ukit/index/sidecar-decision.mjs

@@ -393,65 +393,9 @@ export async function runSidecarDecision({ decision, context = {}, rootDir, home
     fallbackPolicy: spec.fallbackPolicy,
   };
 
-  if (stage === 'off') {
-    return { ...base, answer: deterministic, source: 'deterministic', outcomeClass: 'deterministic' };
-  }
-
-  const result = runAdapterBatch({
-    spec, context: ctx, rootDir: rootDir ?? process.cwd(), config: mergedConfig,
-    batchId: `sidecar-${decision}-${Date.now().toString(36)}`,
-  });
-  const modelAnswer = validModelAnswer(result, spec.decisionKey);
-  const outcomeClass = result?.status ?? 'unavailable';
-  const agreement = modelAnswer === null
-    ? 'unknown'
-    : modelAnswer.toLowerCase() === String(deterministic).toLowerCase()
-      ? 'agree'
-      : 'disagree';
-
-  if (outcomeClass === 'unavailable' || outcomeClass === 'unsupported' || outcomeClass === 'invalid') {
-    return {
-      ...base,
-      answer: deterministic,
-      source: 'deterministic-fallback',
-      outcomeClass: 'unavailable',
-      modelAnswer,
-      agreement,
-      fallbackCode: result?.fallbackCode ?? outcomeClass,
-    };
-  }
-  if (stage === 'shadow') {
-    return {
-      ...base,
-      answer: deterministic,
-      source: 'deterministic',
-      outcomeClass,
-      modelAnswer,
-      agreement,
-      fallbackCode: result?.fallbackCode ?? null,
-    };
-  }
-  // canary/default: valid model answer wins; invalid → deterministic fallback.
-  if (modelAnswer !== null) {
-    return {
-      ...base,
-      answer: modelAnswer,
-      source: 'unic-decision',
-      outcomeClass,
-      modelAnswer,
-      agreement,
-      fallbackCode: result?.fallbackCode ?? null,
-    };
-  }
-  return {
-    ...base,
-    answer: deterministic,
-    source: 'deterministic-fallback',
-    outcomeClass,
-    modelAnswer,
-    agreement,
-    fallbackCode: result?.fallbackCode ?? 'missing-answer',
-  };
+  // All six sidecar decisions have complete deterministic rules. A model vote
+  // cannot waive their constraints; skip inference rather than ask and clamp.
+  return { ...base, answer: deterministic, source: 'deterministic', outcomeClass: 'deterministic' };
 }
 
 // ---------------------------------------------------------------------------