klypix-mcp 1.57.0 → 1.58.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "klypix-mcp",
-  "version": "1.57.0",
+  "version": "1.58.0",
   "description": "Shared project brain and MCP coordination server for multi-agent coding.",
   "type": "module",
   "license": "Apache-2.0",

package/src/brain-doctor.mjs

@@ -211,12 +211,23 @@ function inspectSupervisors(brainDir, baked) {
       hotReloads: Number(state.hotReloads || 0),
       lastSwapAt: state.lastSwapAt || null,
       lastError: state.lastError || null,
+      // A worker hot-swaps behind a live connection; the SUPERVISOR cannot
+      // replace its own process under the host's stdio, so supervisor-level
+      // features arrive only on the next reconnect. Without this the doctor
+      // reported "aligned" (true of workers) while a shipped supervisor
+      // feature was silently inactive — the same class as rendering a
+      // truncated list as a complete one.
+      supervisorGeneration: state.hibernation ? 'current' : 'pre-1.57',
+      hibernation: state.hibernation || null,
     });
   }
+  const pendingReconnect = live.filter(state => state.supervisorGeneration !== 'current');
   return {
     active: live.length > 0,
     count: live.length,
     live,
+    pendingReconnect,
+    hibernated: live.filter(state => state.status === 'hibernated'),
     impaired: live.filter(state => state.impaired),
     matchesInstalled: live.length && baked
       ? live.every(state => state.activeVersion && cmpSemver(state.activeVersion, baked) === 0)
@@ -514,6 +525,17 @@ export function render(r, opts = {}) {
       if (state.impaired) L.push(`        ${c.red}· pid ${state.pid} ${state.status}${state.lastError ? `: ${state.lastError}` : ''} — /mcp reconnect${c.rst}`);
       else if (state.lastError) L.push(`        ${c.yel}· pid ${state.pid} ${state.status}: ${state.lastError}${c.rst}`);
     }
+    // Workers hot-swap; a SUPERVISOR cannot replace its own process under the
+    // host's stdio. Say so explicitly — otherwise "aligned" reads as "every
+    // shipped improvement is live", and a supervisor-level feature (today:
+    // idle-worker hibernation and its RAM saving) is silently inactive.
+    const pending = r.supervisors.pendingReconnect || [];
+    const sleeping = r.supervisors.hibernated || [];
+    if (pending.length) {
+      L.push(`        ${c.yel}· ${pending.length} of ${r.supervisors.count} connection(s) still run a PRE-1.57 supervisor — their workers are current, but supervisor-level features (idle-worker hibernation / RAM release) start at each one's next reconnect${c.rst}`);
+    } else if (r.supervisors.count) {
+      L.push(`        ${c.dim}· all supervisors current${sleeping.length ? ` · ${sleeping.length} hibernated (worker released, presence held, wakes on the next request)` : ''}${c.rst}`);
+    }
   } else if (r.version.supervisorCapable) {
     L.push(`${warn} ${c.bold}SUPERVISOR${c.rst}  installed but this is a legacy direct-worker session · reconnect once to activate`);
   } else {

package/src/finding-routing.mjs

@@ -319,7 +319,12 @@ export function findingKey({ path, text, claim, line = null, root = null } = {})
   // ("…names a GENERATED file" vs "…points at a generated file") must bump the
   // existing draft, not mint a second note for the peer.
   const t = String(claim ?? findingClaim(text)).toLowerCase().replace(/[^a-z0-9]+/g, ' ').trim();
-  return sha16(`${p}|${line ?? ''}|${t}`);
+  // The LINE NUMBER is deliberately excluded from identity. The same defect
+  // cited once as `file.ts` and once as `file.ts:135` is one finding for the
+  // peer who must act on it; including the line minted two drafts telling the
+  // same story twice — the exact duplication the claim-hash above prevents for
+  // re-wordings. Line still rides the draft as evidence, it just is not identity.
+  return sha16(`${p}|${t}`);
 }
 
 // ── Draft construction ──────────────────────────────────────────────────────
@@ -455,7 +460,8 @@ export function renderFindingDrafts(drafts, { limit = 3, total = null } = {}) {
     '## 📬 Finding(s) you verified that belong to SOMEONE ELSE\'S lane — approve to send',
     'You verified something about a file outside your declared scope. The lane knows who declared it; nothing has been sent. '
     + 'Check the “why this session” line — if it is not theirs, skip it (a wrong note costs a peer their whole turn). '
-    + 'This is a DRAFT: nothing was written to the brain and no message exists yet.'];
+    + 'This is a DRAFT: nothing was written to the brain and no message exists yet.',
+    'To send, emit the marker on ITS OWN line WITHOUT the surrounding backticks — a backticked marker is read as a documentation example and is deliberately never sent, so a verbatim paste of the line below does nothing.'];
   for (const draft of list) {
     const where = draft.line ? `${draft.path}:${draft.line}` : draft.path;
     const rec = (draft.seenCount || 1) >= 2 ? `  ⭐ seen ${draft.seenCount}× — still unsent` : '';