klypix-mcp 1.79.0 → 1.80.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "klypix-mcp",
-  "version": "1.79.0",
+  "version": "1.80.0",
   "description": "Active state management for multi-agent coding: a shared, versioned project brain over MCP.",
   "type": "module",
   "license": "Apache-2.0",

package/src/klypix-core.mjs

@@ -32,7 +32,7 @@ import {
   brainLensData, lensToMarkdown, deathDateOfCard,
   statusContextToMarkdown, findFulfillmentCandidates,
   splitQueryTokens, scoreCardsAgainstQuery, correctionOverlaysFor,
-  isFastDecayCard, isUnresolvedOpenCard, DECAY_STALE_MS, formatDecayAge,
+  isFastDecayCard, isUnresolvedOpenCard, isSkillCard, DECAY_STALE_MS, formatDecayAge,
   isPlanCard, planFulfillmentFor, PLAN_PAIR_SIM_BRAIN, isAgconfTwinId,
   readPendingShips, clearPendingShips, pendingShipCards, formatCaptureReceipts,
 } from './klypix-format.mjs';
@@ -494,9 +494,29 @@ export async function opBrainTaskContext({
     };
   });
   const maxChars = Math.max(800, Math.min(5000, Number(budgetChars) || 2800));
+  // Standing rules (2026-08-24 audit): 🛠️ skills are "apply every session",
+  // but the ranker's `score <= 0` gate drops a zero-overlap rule BEFORE its +1
+  // skill boost applies — so the capsule delivered task hits and zero standing
+  // rules, and the rule that would have warned the founder about a same-day
+  // billing trap never reached any session. This block is UNCONDITIONAL:
+  // relevance-ordered when the ranker scored a rule, newest-first otherwise,
+  // deduped against the hit list, and prepended so the maxChars tail-cut can
+  // never be the reason a rule silently vanished.
+  const skillPool = struct.cards.filter((c) => c.type !== 'container'
+    && !/^archive$/i.test(c.area || '') && isSkillCard(c));
+  const scoreById = new Map(candidatePool.map((hit) => [hit.card.id, hit.score]));
+  const standing = skillPool
+    .filter((c) => !hitIds.has(c.id))
+    .sort((a, b) => (scoreById.get(b.id) || 0) - (scoreById.get(a.id) || 0)
+      || (b.createdAt || 0) - (a.createdAt || 0))
+    .slice(0, 3);
   const lines = [
     `## Compact task context (${entries.length} relevant brain card${entries.length === 1 ? '' : 's'} · lexical-fast)`,
   ];
+  if (standing.length) {
+    lines.push(`### 🛠️ Standing rules (${skillPool.length} in the brain — apply always; full set in the brief)`);
+    for (const c of standing) lines.push(`- [${flat(c.area) || 'Notes'}] ${clip(c.text, 220)}`);
+  }
   if (!entries.length) {
     lines.push('No high-confidence task-specific card matched. Continue from repository evidence; use brain_ask only if deeper project history is needed.');
   } else {
@@ -521,6 +541,9 @@ export async function opBrainTaskContext({
     context: {
       mode: 'lexical-fast',
       hits: entries,
+      ...(standing.length ? {
+        standingRules: standing.map((c) => ({ id: c.id, area: flat(c.area) || 'Notes', text: clip(c.text, 220) })),
+      } : {}),
       sufficient: entries.length > 0,
       durationMs,
       brain: path.basename(t.file),

package/src/klypix-format.mjs

@@ -1630,8 +1630,11 @@ export function structToBrief(struct, { recentDays = 14, maxRecent = 40, maxMile
     // 🛠️ Skills — reusable how-tos / gotchas / procedures (the '+' marker).
     // Standing reference, NOT a point-in-time event: always shown, never
     // recency-decayed, and excluded from open/milestones/recent so a skill never
-    // masquerades as (or ages out like) a decision.
-    const skills = rest.filter(isSkillCard);
+    // masquerades as (or ages out like) a decision. Newest first: the render
+    // below can only show maxSkills of them, and an unsorted slice pinned the
+    // OLDEST rules forever while every rule learned since reached no session
+    // (2026-08-24 audit — the founder's same-day billing rule was invisible).
+    const skills = rest.filter(isSkillCard).sort((a, b) => (b.createdAt || 0) - (a.createdAt || 0));
     const open = rest.filter(isOpenCard);
     const miles = rest.filter(isMilestoneCard);
     const plain = rest.filter(c => !open.includes(c) && !miles.includes(c) && !skills.includes(c));
@@ -1916,6 +1919,32 @@ export function structToUltraBrief(struct, { freshness = null, briefPath = '.cla
     // line that falls off the bottom of the preview-sized budget.
     const overdueById = findOverdueOpenCards(struct).byId;
     const openSorted = open.slice().sort((a, b) => (overdueById.has(b.id) ? 1 : 0) - (overdueById.has(a.id) ? 1 : 0));
+    // 🛠️ Standing rules tier (2026-08-24): the ultra brief used to render
+    // skills as a COUNT in the tail — so the one surface every session reads
+    // carried zero of the rules that are supposed to "fire every session".
+    // Newest first (the most recently learned trap is the likeliest live one),
+    // BEFORE the open list: opens are greedy to the budget floor, so anything
+    // placed after them never lands. Small cap — this is a reminder tier, the
+    // full set stays in the brief file. BUDGET FENCE (adversarial review of
+    // 10f43e1, reproduced live at the 1800-char SessionStart default): this
+    // tier must never be the reason a ⏰ OVERDUE line fell off — price the
+    // opens header, every overdue line, and the overflow line FIRST, and hold
+    // that budget back from the skills tier. Rules yield to deadlines.
+    if (skills.length) {
+        const openHeader = `## Open questions & goals (${open.length}${overdueById.size ? `, ${overdueById.size} ⏰ overdue` : ''})`;
+        const overdueLines = openSorted.filter(c => overdueById.has(c.id)).map(c => `- ⏰ OVERDUE ${fr(c)}${head(c)}`);
+        const reserve = open.length
+            ? ['', openHeader, ...overdueLines, `- …and ${open.length} more — in the full brief.`]
+                .reduce((s, l) => s + l.length + 1, 0)
+            : 0;
+        const pushIfFenced = (l) => (used + l.length + 1 > budget - reserve) ? false : (push(l), true);
+        if (pushIfFenced('') && pushIfFenced(`## 🛠️ Standing rules (${skills.length} — newest first, apply always)`)) {
+            const newest = skills.slice().sort((a, b) => (b.createdAt || 0) - (a.createdAt || 0));
+            let shown = 0;
+            for (const c of newest.slice(0, 5)) { if (!pushIfFenced(`- ${fr(c)}${head(c)}`)) break; shown++; }
+            if (shown < skills.length) pushIfFenced(`- …and ${skills.length - shown} more standing rule(s) — in the full brief.`);
+        }
+    }
     if (open.length && pushIf('') && pushIf(`## Open questions & goals (${open.length}${overdueById.size ? `, ${overdueById.size} ⏰ overdue` : ''})`)) {
         let shown = 0;
         for (const c of openSorted) { if (!pushIf(`- ${overdueById.has(c.id) ? '⏰ OVERDUE ' : ''}${fr(c)}${head(c)}`)) break; shown++; }