@ngockhoale/ukit 3.4.57 → 3.4.58

package/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 All notable changes to UKit are documented here.
 
+## 3.4.58 - 2026-10-22
+
+**Completion Gate: plain publish orders create the registry obligation, new tasks do not inherit the previous task's evidence, and a Vietnamese status line about a config is not a config order.**
+- Bug: "Please/Now npm publish the package", "Đưa package lên npm" and "Phát hành bản này lên npm" were not read as publish orders, so code + tests PASS completed the gate with nothing on the registry. `promptOrdersNpmPublish` now accepts polite/sequencing prefixes and the Vietnamese publish verbs; negations, `--dry-run` and prohibitions still create no debt.
+- Bug: a new task inherited the previous task's classifier-owned contract fields and MIXED evidence (a stale `git-integrity push` goal and `operation-verification` under "npm publish the package"), so a correct publish run stayed blocked on an unordered push. `skill-router.sh` now resets them when a new task starts.
+- Bug: "OMP version/config đã load lúc bắt đầu vẫn unknown" classified CONFIGURATION/external (`bat` in "bắt đầu", `tat` in "tất cả" read as toggle verbs), demanding operation proof for a status report. `bat`/`tat` no longer match those phrases; real "Bật/Tắt config" orders stay CONFIGURATION (3 classifier copies + mirrors).
+- Regression: `tests/hooks/releaseCompletionEvidence.test.js` (resolver → contract → gate), `tests/hooks/configPreferenceNotOrder.test.js` + real-prompt fixture.
+- Limit: the resolver still emits no release-specific contract field; the obligation is enforced gate-side from the prompt. Verified offline on macOS/Node 22; replay, not a live host.
+
 ## 3.4.57 - 2026-10-22
 
 **Completion Gate: background launch acknowledgements are not verification evidence; stale-contract reconciliation covers all obligation fields and the Stop boundary.**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngockhoale/ukit",
-  "version": "3.4.57",
+  "version": "3.4.58",
   "description": "Install/update an index-first AI workspace for Claude Code, OpenAI Codex and omp (Oh My Pi).",
   "license": "MIT",
   "type": "module",

package/src/core/executionContracts.js

@@ -367,7 +367,7 @@ export function classifyTaskSemantics({ promptText = '', targetFile = '' } = {})
   // the config noun AND no reporting/question frame opens the clause or precedes
   // that word: in "Report what change was made to the config" the mutation word is
   // a question's noun, never an order.
-  const CONFIG_MUTATION_VERB = /\b(?:set|change|toggle|enable|disable|switch|turn|configure|modify|update|edit|doi|bat|tat|chinh)\b/;
+  const CONFIG_MUTATION_VERB = /\b(?:set|change|toggle|enable|disable|switch|turn|configure|modify|update|edit|doi|bat(?!\s+dau\b)|tat(?!\s+ca\b)|chinh)\b/;
   const CONFIG_NOUN = /\b(?:configuration|config|toggle|cau hinh)\b/;
   const CONFIG_REPORT_FRAME = /^\s*(?:report|show|tell|list|describe|explain|summari[sz]e|bao cao|cho biet|liet ke|mo ta)\b/;
   const CONFIG_QUESTION_FRAME_PREFIX = /\b(?:what|which|how|report|bao cao|nao|gi)\b/;

package/template_project/.claude/hooks/skill-router.sh

@@ -1982,7 +1982,7 @@ function classifyTaskSemantics({ promptText = '', targetFile = '' } = {}) {
   // the config noun AND no reporting/question frame opens the clause or precedes
   // that word: in "Report what change was made to the config" the mutation word is
   // a question's noun, never an order.
-  const CONFIG_MUTATION_VERB = /\b(?:set|change|toggle|enable|disable|switch|turn|configure|modify|update|edit|doi|bat|tat|chinh)\b/;
+  const CONFIG_MUTATION_VERB = /\b(?:set|change|toggle|enable|disable|switch|turn|configure|modify|update|edit|doi|bat(?!\s+dau\b)|tat(?!\s+ca\b)|chinh)\b/;
   const CONFIG_NOUN = /\b(?:configuration|config|toggle|cau hinh)\b/;
   const CONFIG_REPORT_FRAME = /^\s*(?:report|show|tell|list|describe|explain|summari[sz]e|bao cao|cho biet|liet ke|mo ta)\b/;
   const CONFIG_QUESTION_FRAME_PREFIX = /\b(?:what|which|how|report|bao cao|nao|gi)\b/;
@@ -3971,11 +3971,24 @@ function applyTaskSemantics(contract, semantic) {
       delete persisted.routeSummary.continuationState;
     } else if (semantic.taskKind !== 'CODE_IMPLEMENTATION' || persisted.routeSummary) {
       persisted.routeSummary ||= { executionMode: 'local-build' };
-      if (semanticFromPrompt && persisted.routeSummary.executionContract) {
-        // Re-derivation replaces the classifier-owned obligation fields wholesale: applyTaskSemantics
+      if ((semanticFromPrompt || !continuingTask) && persisted.routeSummary.executionContract) {
+        // A NEW task never inherits the previous task's classifier-owned obligations either (e.g. a stale
+        // git-integrity push goal under "npm publish the package"). Re-derivation replaces them wholesale: applyTaskSemantics
         // spreads the old contract first, so a stale operationGoal / releasePolicy would survive.
         const kept = { ...persisted.routeSummary.executionContract };
-        for (const key of ['operationGoal', 'configurationScope', 'releasePolicy', 'reportArtifact', 'verificationOnly', 'statusReport']) delete kept[key];
+        // statusReport / verificationOnly are only stripped on re-derivation: for a NEW task they drive the
+        // carried-status rebuild below, so they must survive to be recognised there.
+        const staleKeys = semanticFromPrompt ? ['operationGoal', 'configurationScope', 'releasePolicy', 'reportArtifact', 'verificationOnly', 'statusReport']
+          : ['operationGoal', 'configurationScope', 'releasePolicy', 'reportArtifact'];
+        for (const key of staleKeys) delete kept[key];
+        // MIXED/OPERATIONAL evidence tags belong to the previous task's classification: a new task starts from
+        // the mode's base evidence so it is not asked for operation proof its own prompt never ordered.
+        if ((!continuingTask || semanticFromPrompt) && semantic.taskKind !== 'MIXED' && semantic.taskKind !== 'OPERATIONAL'
+          && Array.isArray(kept.completionEvidence) && kept.completionEvidence.includes('operation-verification')) {
+          const baseMode = persisted.routeSummary.executionMode === 'informational' ? 'local-build' : persisted.routeSummary.executionMode;
+          const base = buildExecutionContract(baseMode);
+          if (Array.isArray(base?.completionEvidence)) kept.completionEvidence = base.completionEvidence;
+        }
         persisted.routeSummary.executionContract = kept;
       }
       // A carried status-report contract must not leak its empty evidence / flag onto a

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

@@ -1264,7 +1264,7 @@ export function classifyTaskSemantics({ promptText = '', targetFile = '' } = {})
   // the config noun AND no reporting/question frame opens the clause or precedes
   // that word: in "Report what change was made to the config" the mutation word is
   // a question's noun, never an order.
-  const CONFIG_MUTATION_VERB = /\b(?:set|change|toggle|enable|disable|switch|turn|configure|modify|update|edit|doi|bat|tat|chinh)\b/;
+  const CONFIG_MUTATION_VERB = /\b(?:set|change|toggle|enable|disable|switch|turn|configure|modify|update|edit|doi|bat(?!\s+dau\b)|tat(?!\s+ca\b)|chinh)\b/;
   const CONFIG_NOUN = /\b(?:configuration|config|toggle|cau hinh)\b/;
   const CONFIG_REPORT_FRAME = /^\s*(?:report|show|tell|list|describe|explain|summari[sz]e|bao cao|cho biet|liet ke|mo ta)\b/;
   const CONFIG_QUESTION_FRAME_PREFIX = /\b(?:what|which|how|report|bao cao|nao|gi)\b/;

package/template_project/.claude/ukit/runtime/execution-ledger.mjs

@@ -3712,8 +3712,12 @@ export function promptOrdersNpmPublish(state = {}) {
           const releaseOrder = /^(?:please\s+|just\s+|now\s+|also\s+)*(?:release|ship|publish)\s+(?!readiness\b|verification\b|evidence\b|policy\b|intent\b|notes\b|terminology\b)\S/.test(item);
           const releaseReference = /\b(?:release|publish|publication)\b/.test(item);
           if (negated && releaseReference && !/--dry-run/.test(item)) prohibited = true;
-          if (!negated && (releaseOrder || (publication && (
-            /^npm\s+publish\b/.test(item) || PUBLISH_ORDER_IMPERATIVE.test(item)
+          // Politeness/sequencing prefixes ("please npm publish ...") and Vietnamese verbs ("dua ... len npm",
+          // "phat hanh ... len npm") are still affirmative orders when nothing negates them.
+          const politeItem = item.replace(/^(?:(?:please|just|now|also|then|vui long|hay)\s+)+/, '');
+          const viPublish = /^(?:dua|phat hanh|xuat ban|deploy|day)\b[^.]{0,60}\b(?:len|to|vao)\s+npm\b/.test(politeItem);
+          if (!negated && (releaseOrder || viPublish || (publication && (
+            /^npm\s+publish\b/.test(politeItem) || PUBLISH_ORDER_IMPERATIVE.test(item)
             || /\b(?:chay|run)\s+`?npm\s+publish\b/.test(item)
             || /continue the release workflow:/.test(item))))) ordered = true;
         }
@@ -4503,6 +4507,14 @@ export async function reconcileStaleContract(projectRoot, state) {
     if (!fresh?.taskKind || contractSignature(fresh) === contractSignature(stored)) return state;
     const base = { ...stored };
     for (const key of CLASSIFIER_OWNED_FIELDS) delete base[key];
+    // Evidence tags added by the stale classification (operation-verification, report-artifact, ...) do not
+    // describe the re-derived kind: restart from the execution mode's base evidence unless the fresh kind
+    // sets its own (MIXED/OPERATIONAL/DOCUMENTATION/READ_ONLY are rewritten by applyTaskSemantics).
+    if (fresh.taskKind === 'CODE_IMPLEMENTATION' || (fresh.taskKind === 'CONFIGURATION' && fresh.configurationScope !== 'external')) {
+      const mode = state.routeSummary?.executionMode === 'informational' ? 'local-build' : state.routeSummary?.executionMode;
+      const baseEvidence = resolver.buildExecutionContract(mode)?.completionEvidence;
+      if (Array.isArray(baseEvidence)) base.completionEvidence = baseEvidence;
+    }
     return { ...state, routeSummary: { ...state.routeSummary, executionContract: resolver.applyTaskSemantics(base, fresh) } };
   } catch { return state; }
 }