@yemi33/minions 0.1.2354 → 0.1.2355

package/docs/kb-sweep.md

@@ -72,7 +72,7 @@ Each batch prompt asks for three lists in JSON:
 
 Each action archives the file via the same `_archiveKbFile()` helper used by Pass 1; reclassification rewrites the `category:` frontmatter line and moves the file into the new category directory (source: [`engine/kb-sweep.js:323-340`](../engine/kb-sweep.js#L323)).
 
-**Duplicate-pair guard.** `duplicates` entries are only actioned if the pair references at least one index the LLM actually content-analyzed this batch (i.e. a member of `changedIdx`); a pair where every referenced index is roster-only (`unchangedIdx`) is skipped with a warning instead of archived, since that would mean acting on a hallucinated or stale LLM reference rather than a real comparison (source: [`engine/kb-sweep.js`](../engine/kb-sweep.js) `_applyLlmPlan`, PR #739).
+**changedIdx guard.** All three actions are only actioned against indices the LLM actually content-analyzed this batch (i.e. members of `changedIdx`). `remove` and `reclassify` require their own referenced `index` to be in `changedIdx`; a `duplicates` pair is actioned if the pair references at least one index in `changedIdx` (a pair where every referenced index is roster-only, `unchangedIdx`, is skipped). Roster-only entries are sent to the LLM with title/date/category only (no content — see the "existing entries" roster section), so an index outside `changedIdx` cannot have been genuinely compared/analyzed this batch; acting on it would mean acting on a hallucinated or stale LLM reference rather than a real comparison (source: [`engine/kb-sweep.js`](../engine/kb-sweep.js) `_applyLlmPlan`, PR #739, W-mrc9y5y000063033).
 
 Reclassification targets are validated against `shared.KB_CATEGORIES` (`architecture`, `conventions`, `project-notes`, `build-reports`, `reviews` — source: [`engine/shared.js`](../engine/shared.js) `KB_CATEGORIES`); unknown categories are silently dropped.
 

package/engine/kb-sweep.js

@@ -661,12 +661,18 @@ ${body}`;
 function _applyLlmPlan(plan, manifest, changedIdx, opts = {}) {
   let removed = 0, merged = 0, reclassified = 0;
   // changedIdx is the set of manifest indices actually content-analyzed by
-  // the LLM this batch (see _llmBatchSweep's scannedIdx). Duplicate pairs
-  // referencing only unanalyzed indices (LLM hallucination / stale index
-  // mis-reference) must never be archived — a pair is only trustworthy if
-  // at least one side was actually analyzed this batch (W-mrb447th00062813).
+  // the LLM this batch (see _llmBatchSweep's scannedIdx). remove/reclassify/
+  // duplicates entries referencing only unanalyzed (roster-only) indices
+  // (LLM hallucination / stale index mis-reference) must never be acted on —
+  // remove/reclassify require their referenced index to be analyzed this
+  // batch, and a duplicate pair is only trustworthy if at least one side was
+  // actually analyzed this batch (W-mrb447th00062813, W-mrc9y5y000063033).
   const changedSet = new Set(changedIdx || []);
   for (const r of (plan.remove || [])) {
+    if (!changedSet.has(r.index)) {
+      log('warn', `[kb-sweep] skipping remove for index ${r.index} — not in analyzed (changedIdx) batch, likely a hallucinated/stale LLM reference`);
+      continue;
+    }
     const entry = manifest[r.index];
     if (!entry) continue;
     if (opts.dryRun) { removed++; continue; }
@@ -686,6 +692,10 @@ function _applyLlmPlan(plan, manifest, changedIdx, opts = {}) {
     }
   }
   for (const r of (plan.reclassify || [])) {
+    if (!changedSet.has(r.index)) {
+      log('warn', `[kb-sweep] skipping reclassify for index ${r.index} — not in analyzed (changedIdx) batch, likely a hallucinated/stale LLM reference`);
+      continue;
+    }
     const entry = manifest[r.index];
     if (!entry || !shared.KB_CATEGORIES.includes(r.to)) continue;
     if (opts.dryRun) { reclassified++; continue; }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2354",
+  "version": "0.1.2355",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"