@ngockhoale/ukit 3.4.16 → 3.4.17

package/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 All notable changes to UKit are documented here.
 
+## 3.4.17 - 2026-10-06
+
+**Planners can revise an unimplemented task that still carries the skeleton Executor Report.**
+`handoff-model-guard.sh` blocked every Edit/Write to an existing `TASK-xxx.md` whose
+`## Executor Report` (or `## Reviewer Verdict`) was still the Phase-2 skeleton, demanding
+`EXECUTOR_MODEL` — so `/ukit:handoff-create` could create a task but never amend it after
+plan review. The model gate now fires only when the phase section itself changes to real
+content; edits elsewhere, unchanged skeletons and inert placeholders (`Not started.`,
+`KEY: -`) pass. Real executor/reviewer reports without a valid model, lite-tier models and
+real-model demotion stay blocked. Regression: `tests/hooks/taskSkeletonAmendment.test.js`.
+
 ## 3.4.16 - 2026-10-05
 
 **Create-scope guard no longer blocks planner subagents running in a git worktree.**

package/package.json

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

package/template_project/.claude/hooks/handoff-model-guard.sh

@@ -197,6 +197,31 @@ function isPlaceholderValue(value) {
   if (!raw || /^[<(]/.test(raw)) return true;
   return ['-','--','tbd','todo','n/a','na','none','unknown','pending'].includes(raw.toLowerCase());
 }
+
+// Planning amendments (handoff-create revising an unimplemented task) re-submit a task whose
+// `## Executor Report` / `## Reviewer Verdict` is still the Phase-2 skeleton. The model gate
+// exists to stop a skeleton being promoted into a "real" report without a validated model, so it
+// must only fire when the phase section itself is touched with real content — not when the edit
+// leaves the skeleton unchanged or keeps it inert (headings/"Not started."/placeholder fields).
+function phaseSection(text, heading) {
+  const m = new RegExp('^##[ \\t]+' + heading + '[ \\t]*$', 'm').exec(text);
+  if (!m) return null;
+  const rest = text.slice(m.index + m[0].length);
+  const next = rest.search(/^##[ \t]+\S/m);
+  return (next === -1 ? rest : rest.slice(0, next)).trim();
+}
+function phaseSectionInert(section) {
+  if (section === null) return true;
+  return section.split(/\r?\n/).map((l) => l.trim()).filter(Boolean).every((line) => {
+    if (/^(?:[-*]\s*)?(?:not started|n\/a|tbd|todo|pending|none)\.?$/i.test(line)) return true;
+    const kv = line.match(/^(?:[-*]\s*)?\**[A-Za-z_ ]+\**\s*:\s*(.*)$/);
+    return Boolean(kv) && isPlaceholderValue(kv[1].replace(/\*+/g, '').trim());
+  });
+}
+function phaseSectionUntouched(currentVisible, newVisible, heading) {
+  const next = phaseSection(newVisible, heading);
+  return phaseSection(currentVisible, heading) === next || phaseSectionInert(next);
+}
   // W2-D07 fix: `OVERRIDE_APPROVED_BY: human` was plain text inside gated files —
   // self-authorizable by the gated model. The in-file `## Model-Tier Guard Override`
   // declaration stays (it is the audit trail), but it only counts when a marker file
@@ -577,7 +602,7 @@ if (toolName === 'Write' || toolName === 'Edit') {
     if (hasExecutorReport && !currentExecutorReal) {
       const executorModel = extractField(newVisible, 'EXECUTOR_MODEL');
       if (isPlaceholderValue(executorModel) || /^unknown$/i.test(executorModel)) {
-        if (!isFreshTaskFile) {
+        if (!isFreshTaskFile && !phaseSectionUntouched(currentVisible, newVisible, 'Executor Report')) {
           block(`${taskId}: EXECUTOR_MODEL missing/placeholder. Implementation must run via Agent tool subagent_type: "feature-implementer" (sonnet/unic-code) and self-report its model.`);
         }
         // Else: skeleton sections copied from a task template at creation time — the
@@ -619,7 +644,7 @@ if (toolName === 'Write' || toolName === 'Edit') {
       const reviewOverride = await taskHasHumanOverride(newVisible, taskId)
         || await planHasHumanOverride(stripHtmlComments(planContentForReview));
       if (isPlaceholderValue(reviewerModel) || /^unknown$/i.test(reviewerModel)) {
-        if (!isFreshTaskFile) {
+        if (!isFreshTaskFile && !phaseSectionUntouched(currentVisible, newVisible, 'Reviewer Verdict')) {
           block(`${taskId}: REVIEWER_MODEL missing/placeholder. Review must run via Agent tool subagent_type: "code-reviewer" (opus/unic-smart) and self-report its model.`);
         }
         // Else: skeleton Reviewer Verdict on a freshly created task file — Phase 4