@ngockhoale/ukit 1.6.7 → 1.6.8

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to UKit are documented here.
 
+## 1.6.8 - 2026-08-09
+
+### Fixed
+
+- **`extract-image.mjs`** no longer re-arms `pending-<sha>.json` for an image that already has an `analyzed-<sha>.json` receipt. `selectImages()` always returns the most recent N images from the whole transcript, so a fully-analyzed image never fell out of that window on its own; since `vision-router.sh` calls `--mark-pending` on every prompt, this re-triggered the vision hint/gate on the same closed-out image forever, even on unrelated prompts. Both `--detect` and `--mark-pending` now skip any sha with an existing analyzed receipt.
+
 ## 1.6.7 - 2026-08-09
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngockhoale/ukit",
-  "version": "1.6.7",
+  "version": "1.6.8",
   "description": "Install/update an index-first AI workspace for Claude Code, Antigravity, OpenAI Codex, and OpenCode.",
   "license": "MIT",
   "type": "module",

package/templates/.claude/ukit/index/extract-image.mjs

@@ -315,8 +315,16 @@ function main() {
 
   const lines = sessionPath ? readLines(sessionPath) : [];
   const blocks = extractImageBlocks(lines);
-  const selected = selectImages(blocks, opts.limit);
-  const refs = resolveRefs(opts.refs);
+  const markerDir = path.join(outDir, sessionId);
+  // selectImages() picks the most recent N images from the WHOLE transcript, so a resolved
+  // image never falls out of that window on its own — without this filter, --mark-pending
+  // (called on every prompt by vision-router.sh) would re-arm pending-<sha>.json for the same
+  // already-analyzed image forever, re-triggering the vision hint/gate on unrelated prompts.
+  // A sha with an existing analyzed-<sha>.json receipt is a closed, content-addressed case:
+  // identical bytes already produced that analysis, so skipping it loses no information.
+  const isAlreadyAnalyzed = (sha) => fs.existsSync(path.join(markerDir, `analyzed-${sha}.json`));
+  const selected = selectImages(blocks, opts.limit).filter((img) => !isAlreadyAnalyzed(img.sha));
+  const refs = resolveRefs(opts.refs).filter((ref) => !isAlreadyAnalyzed(ref.sha));
 
   // --detect is report-only, so it must not short-circuit --mark-pending when both are
   // passed together. vision-router.sh combines them; short-circuiting here would arm
@@ -327,7 +335,6 @@ function main() {
   }
 
   if (opts.markPending) {
-    const markerDir = path.join(outDir, sessionId);
     if (selected.length > 0 || refs.length > 0) {
       try {
         fs.mkdirSync(markerDir, { recursive: true });