@ngockhoale/ukit 3.4.4 → 3.4.6

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 All notable changes to UKit are documented here.
 
+
+## 3.4.4 - 2026-09-30
+
+**Stop-hook blocks are now user-visible.** When the Stop gate refuses a
+premature stop (missing completion evidence, mid-flight handoff run, watchdog
+budget), Claude Code previously received only the model-facing `reason` — the
+turn looked like a silent stall and users had to type "continue" blindly.
+Every blocked stop now also emits a `systemMessage` shown in the transcript:
+`[UKit] Stop blocked — <policyOwner>: <reason first line>`.
+
 ## 3.4.3 - 2026-09-29
 
 Republish of 3.4.2 — npm staged-version E409 limbo (same content, version bump only).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngockhoale/ukit",
-  "version": "3.4.4",
+  "version": "3.4.6",
   "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/output/index.js

@@ -316,9 +316,10 @@ function buildRawOutputText({ command = '', stdout = '', stderr = '', exitCode =
 
 function buildRecoveryHintSummary(summary, rawPath, { tokensBefore = 0 } = {}) {
   // TASK-003 fix round 1: a failed tee persist returns rawPath='' — never emit a
-  // dangling `- Full output: ` hint that points at nothing.
+  // dangling `- Tee log: ` hint that points at nothing. Label avoids "Full output:"
+  // because agents misread a path-shaped hint as an artifact:// pointer (omp).
   if (!rawPath) return null;
-  const recoveryLine = `- Full output: ${rawPath}`;
+  const recoveryLine = `- Tee log: ${rawPath}`;
   const summaryLines = String(summary ?? '')
     .split(/\r?\n/)
     .map((line) => line.trim())
@@ -1143,7 +1144,7 @@ function normalizeOutputSummaryForDedupe(summary) {
   return String(summary ?? '')
     .split(/\r?\n/)
     .map((line) => String(line ?? '').trim())
-    .filter((line) => line && !/^- Full output:\s+/i.test(line))
+    .filter((line) => line && !/^- (?:Full output|Tee log):\s+/i.test(line))
     .join('\n');
 }
 

package/template_project/.claude/ukit/runtime/output-compression.mjs

@@ -379,9 +379,10 @@ function buildRawOutputText({ command = '', stdout = '', stderr = '', exitCode =
 
 function buildRecoveryHintSummary(summary, rawPath, { tokensBefore = 0 } = {}) {
   // TASK-003 fix round 1: a failed tee persist returns rawPath='' — never emit a
-  // dangling `- Full output: ` hint that points at nothing.
+  // dangling `- Tee log: ` hint that points at nothing. Label avoids "Full output:"
+  // because agents misread a path-shaped hint as an artifact:// pointer (omp).
   if (!rawPath) return null;
-  const recoveryLine = `- Full output: ${rawPath}`;
+  const recoveryLine = `- Tee log: ${rawPath}`;
   const summaryLines = String(summary ?? '')
     .split(/\r?\n/)
     .map((line) => line.trim())
@@ -1252,7 +1253,7 @@ function normalizeOutputSummaryForDedupe(summary) {
     .split(/\r?\n/)
     .map((line) => String(line ?? '').trim())
     .filter((line) => line
-      && !/^- Full output:\s+/i.test(line)
+      && !/^- (?:Full output|Tee log):\s+/i.test(line)
       && !/^-\s*FAIL(?:ED)?\b/i.test(line)
       && !/^-\s*PASS\b/i.test(line)
       && !/^-\s*(?:Test Files|Tests|Duration|Start at)\b/i.test(line))

package/template_project/.claude/ukit/runtime/stop-coordinator.mjs

@@ -1170,10 +1170,18 @@ async function main() {
   if (merged.decision === 'block') {
     // One JSON line: the single decision, its provenance, and — when an advisory
     // lane failed or has news — the verbatim note. undefined fields are omitted.
+    //
+    // `reason` is fed to the MODEL as the continuation prompt — Claude Code does
+    // NOT display it to the user. Only `systemMessage` is user-visible, so a
+    // blocked stop must surface its reason there too: without it the turn just
+    // "stops and restarts" with no explanation, which reads as the agent
+    // stalling and forces the user to type "continue".
+    const firstLine = String(merged.reason || '').split('\n', 1)[0] || 'reason unavailable';
+    const blockNote = `[UKit] Stop blocked — ${merged.policyOwner}: ${firstLine}`;
     writeDecisionLine({
       decision: 'block',
       reason: merged.reason,
-      systemMessage: merged.systemMessage,
+      systemMessage: merged.systemMessage ? `${blockNote}\n${merged.systemMessage}` : blockNote,
       policyOwner: merged.policyOwner,
     });
     return;