pi-mega-compact 0.7.8 → 0.8.0

package/dist/extensions/mega-game-cmds.js

@@ -0,0 +1,106 @@
+/**
+ * mega-game-cmds.ts — /mega-game slash command (S30).
+ *
+ * Backs the game-mode toggle + theme picker + TUI display mode. All state is
+ * the global `game_state` SQLite row (src/store/sqlite/game-state.ts) — local
+ * only (PREVENT-PI-004: no network; no guardrails-allow needed because this
+ * command touches no fetch/http). All SQL is parameterized (PREVENT-002) and
+ * lives in the src/ submodule, not here.
+ *
+ * Usage:
+ *   /mega-game                 print current state
+ *   /mega-game on              enable game mode (scoring + level-up + MEGA CACHE)
+ *   /mega-game off             disable game mode
+ *   /mega-game theme           list available themes
+ *   /mega-game theme <id>      set theme by id
+ *   /mega-game theme next      cycle to next theme
+ *   /mega-game tui full        full TUI widget (bars, stats, flair)
+ *   /mega-game tui minimal     one-line TUI widget (level + cache %)
+ *   /mega-game achievements     list unlocked achievements
+ */
+import { getGameState, setGameState, } from "../src/store/sqlite.js";
+import { listAchievements } from "../src/store/sqlite/game-achievements.js";
+import { THEMES, THEME_IDS, getTheme, isValidTheme, nextTheme, DEFAULT_THEME } from "../src/config/themes.js";
+/** Format the current state as a human-readable status line set. */
+function fmtState(s) {
+    return [
+        `[mega-game] game mode: ${s.game_mode_on ? "ON" : "off"}`,
+        `  theme: ${s.theme}${s.theme === DEFAULT_THEME ? " (default)" : ""}`,
+        `  tui:   ${s.tui_display_mode}`,
+    ];
+}
+/** Register the /mega-game command. */
+export function registerGameCommands(pi, runtime) {
+    pi.registerCommand("mega-game", {
+        description: "Game mode toggle + theme picker + TUI display mode. Usage: /mega-game [on|off|theme [id|next]|tui [full|minimal]|achievements]",
+        handler: async (args, ctx) => {
+            runtime.bindRepo(ctx.cwd);
+            const stateDir = runtime.currentStateDir;
+            const parts = args.trim().split(/\s+/).filter(Boolean);
+            // /mega-game  → print current state.
+            if (parts.length === 0) {
+                const s = getGameState(stateDir);
+                for (const line of fmtState(s))
+                    ctx.ui.notify(line);
+                return;
+            }
+            const sub = parts[0];
+            // /mega-game achievements — terse list of unlocked (hidden only once unlocked).
+            if (sub === "achievements") {
+                const rows = listAchievements(stateDir).filter((r) => r.unlocked_at != null);
+                ctx.ui.notify(`[mega-game] achievements unlocked (${rows.length}/9):`);
+                for (const r of rows) {
+                    ctx.ui.notify(`  ${r.icon ?? ""} ${r.title}`);
+                }
+                return;
+            }
+            // /mega-game on|off
+            if (sub === "on" || sub === "off") {
+                const s = setGameState({ game_mode_on: sub === "on" }, stateDir);
+                runtime.bumpGameState();
+                ctx.ui.notify(`[mega-game] game mode ${s.game_mode_on ? "ON" : "off"}`);
+                return;
+            }
+            // /mega-game theme [id|next]
+            if (sub === "theme") {
+                if (parts.length === 1) {
+                    // list themes
+                    ctx.ui.notify("[mega-game] themes:");
+                    for (const t of THEMES) {
+                        ctx.ui.notify(`  ${t.id.padEnd(14)} ${t.label}`);
+                    }
+                    return;
+                }
+                const arg = parts[1];
+                let id;
+                if (arg === "next") {
+                    id = nextTheme(getGameState(stateDir).theme);
+                }
+                else if (isValidTheme(arg)) {
+                    id = arg;
+                }
+                else {
+                    ctx.ui.notify(`[mega-game] unknown theme "${arg}". Valid: ${THEME_IDS.join(", ")}`);
+                    return;
+                }
+                const s = setGameState({ theme: id }, stateDir);
+                runtime.bumpGameState();
+                ctx.ui.notify(`[mega-game] theme → ${s.theme} (${getTheme(s.theme)?.label ?? ""})`);
+                return;
+            }
+            // /mega-game tui full|minimal
+            if (sub === "tui") {
+                const arg = parts[1];
+                if (arg !== "full" && arg !== "minimal") {
+                    ctx.ui.notify(`[mega-game] usage: /mega-game tui full|minimal`);
+                    return;
+                }
+                const s = setGameState({ tui_display_mode: arg }, stateDir);
+                runtime.bumpGameState();
+                ctx.ui.notify(`[mega-game] tui → ${s.tui_display_mode}`);
+                return;
+            }
+            ctx.ui.notify(`[mega-game] usage: /mega-game [on|off|theme [id|next]|tui [full|minimal]|achievements]`);
+        },
+    });
+}

package/dist/extensions/mega-game-cmds.test.js

@@ -0,0 +1,113 @@
+/**
+ * mega-game-cmds.test.ts — /mega-game command parsing matrix (S30).
+ * Uses an isolated state dir + a fake pi harness (mirrors mega-compact.test.ts).
+ * Pi runtime is mocked; the src/ helpers under test are pi-agnostic.
+ */
+import { describe, it, before, after } from "node:test";
+import assert from "node:assert/strict";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { mkdtempSync, rmSync } from "node:fs";
+import { createRequire } from "node:module";
+import { closeStore, getGameState } from "../src/store/sqlite.js";
+import { THEME_IDS } from "../src/config/themes.js";
+// ESM bootstrap so `require()` works in this .test.ts (mirrors
+// mega-compact.test.ts:20-24). Needed for the dynamic `require("./mega-game-cmds.js")`
+// that wires the command against a fake pi without binding to the real pi
+// module at load time.
+const require = createRequire(import.meta.url);
+function makeHarness(stateDir) {
+    const commands = {};
+    const notifies = [];
+    const runtime = { bindRepo: () => { }, currentStateDir: stateDir, bumpGameState: () => { } };
+    const ctx = {
+        cwd: stateDir,
+        ui: { notify: (s) => notifies.push(s) },
+    };
+    const fakePi = {
+        registerCommand: (name, opts) => {
+            commands[name] = opts;
+        },
+    };
+    // Import after env is set so stateDir resolves. Dynamic import keeps the
+    // test from binding to the real pi module at load time.
+    const mod = require("./mega-game-cmds.js");
+    mod.registerGameCommands(fakePi, runtime);
+    return { commands, notifies, ctx };
+}
+describe("/mega-game (S30)", () => {
+    let dir;
+    before(() => {
+        dir = mkdtempSync(join(tmpdir(), "mc-megagame-"));
+        process.env.MEGACOMPACT_STATE_DIR = dir;
+    });
+    after(() => {
+        closeStore(dir);
+        delete process.env.MEGACOMPACT_STATE_DIR;
+        rmSync(dir, { recursive: true, force: true });
+    });
+    async function run(args) {
+        const h = makeHarness(dir);
+        h.notifies.length = 0;
+        await h.commands["mega-game"].handler(args, h.ctx);
+        return h.notifies;
+    }
+    it("bare command prints current (default) state", async () => {
+        const lines = await run("");
+        assert.ok(lines.some((l) => l.includes("game mode: off")));
+        assert.ok(lines.some((l) => l.includes("transparent")));
+        assert.ok(lines.some((l) => l.includes("tui:")));
+    });
+    it("on enables game mode and persists", async () => {
+        await run("on");
+        assert.equal(getGameState().game_mode_on, true);
+    });
+    it("off disables game mode and persists", async () => {
+        await run("on");
+        await run("off");
+        assert.equal(getGameState().game_mode_on, false);
+    });
+    it("theme <id> sets a valid theme and persists", async () => {
+        await run("theme retro");
+        assert.equal(getGameState().theme, "retro");
+        await run("theme cyan-neon");
+        assert.equal(getGameState().theme, "cyan-neon");
+    });
+    it("theme <unknown> is rejected with a usage line and does not mutate", async () => {
+        await run("theme retro");
+        const lines = await run("theme bogus");
+        assert.ok(lines.some((l) => l.includes("unknown theme")));
+        assert.equal(getGameState().theme, "retro");
+    });
+    it("theme next cycles to the next theme and wraps", async () => {
+        await run(`theme ${THEME_IDS[0]}`);
+        await run("theme next");
+        assert.equal(getGameState().theme, THEME_IDS[1]);
+        // cycle to the end then wrap
+        await run(`theme ${THEME_IDS[THEME_IDS.length - 1]}`);
+        await run("theme next");
+        assert.equal(getGameState().theme, THEME_IDS[0]);
+    });
+    it("theme (bare) lists all themes", async () => {
+        const lines = await run("theme");
+        for (const id of THEME_IDS) {
+            assert.ok(lines.some((l) => l.includes(id)), `lists ${id}`);
+        }
+    });
+    it("tui full|minimal sets display mode and persists", async () => {
+        await run("tui minimal");
+        assert.equal(getGameState().tui_display_mode, "minimal");
+        await run("tui full");
+        assert.equal(getGameState().tui_display_mode, "full");
+    });
+    it("tui <bad> prints usage and does not mutate", async () => {
+        await run("tui minimal");
+        const lines = await run("tui huge");
+        assert.ok(lines.some((l) => l.includes("usage")));
+        assert.equal(getGameState().tui_display_mode, "minimal");
+    });
+    it("unknown subcommand prints usage", async () => {
+        const lines = await run("bogus");
+        assert.ok(lines.some((l) => l.includes("usage")));
+    });
+});

package/dist/extensions/mega-pipeline/compact.js

@@ -0,0 +1,324 @@
+/**
+ * compact.ts — full compaction pipeline (Trident) + pi no-op prediction.
+ *
+ * `runCompact` runs the full Trident pipeline (fast-gate aside) and persists a
+ * checkpoint. `piCompactWouldNoop` predicts whether pi's `ctx.compact()` would
+ * throw a no-op error. Both mutate the shared MegaRuntime (token accounting,
+ * ticker, status, events) and are driven by the event + command handlers in
+ * mega-events.ts / mega-commands.ts.
+ */
+import { sessionEntryToContextMessages } from "@earendil-works/pi-coding-agent";
+import { compactSession } from "../../src/engine.js";
+import { normalizeSessionId } from "../../src/store.js";
+import { estimateBlockTokens } from "../../src/tokens.js";
+import { touchSession, logDaily, incCompactCount, incCacheHitTokens } from "../../src/store/sqlite.js";
+import { consolidateMemories } from "../../src/memory.js";
+import { C, MARKER_TYPE, } from "../mega-runtime.js";
+import { resolveRepoRoot, preserveRecentForPressure } from "../mega-config.js";
+import { runRaptor } from "../../src/dedup/raptor/index.js";
+import { loadDedupConfig } from "../../src/config/dedup.js";
+import { upsertEmbedding as indexUpsertEmbedding } from "../../src/store/vectorIndex.js";
+import { runMemoryReview } from "./memory-review.js";
+/** Run the full compaction pipeline and persist a checkpoint. Returns the result. */
+export function runCompact(pi, runtime, config, ctx, messages, opts = {}) {
+    runtime.bindRepo(ctx.cwd);
+    const sid = normalizeSessionId(ctx.sessionManager.getSessionId());
+    runtime.resetRuntime(sid);
+    runtime.rt.sessionId = sid;
+    const view = runtime.engineView(messages);
+    // keepFrom deepens with context pressure (Fix E): under high pressure we
+    // compact more of the session, down to the preserveRecentMin floor.
+    const preserve = preserveRecentForPressure(opts.compressionPressure ?? 0, config.preserveRecent, config.preserveRecentMin);
+    const keepFrom = opts.keepFrom ?? Math.max(0, view.length - preserve);
+    // For very small sessions (fewer messages than preserveRecent), allow
+    // compacting everything except the last message — the user explicitly
+    // requested compaction, so don't refuse it just because the session is short.
+    if (keepFrom <= 0) {
+        if (view.length <= 1)
+            return { skipped: true };
+        // Use the fallback: compact everything except the last message
+        const fallbackKeepFrom = view.length - 1;
+        return doCompact(view, fallbackKeepFrom, opts, sid, config, pi, ctx, runtime);
+    }
+    return doCompact(view, keepFrom, opts, sid, config, pi, ctx, runtime);
+}
+function doCompact(view, keepFrom, opts, sid, config, pi, ctx, runtime) {
+    runtime.pulsing = true; // animate the status line while the (sync) pipeline runs
+    // S21.2: reset the per-compaction memory-op counter so the post-compact
+    // consolidate pass only fires when memory rows actually changed during the
+    // compaction window (turn_end → auto-review may have written some).
+    runtime.memoriesTouchedThisCompaction = 0;
+    const result = compactSession({
+        sessionId: sid,
+        messages: view,
+        keepFrom,
+        summary: opts.summary,
+        timestamp: Date.now(),
+        onTier: runtime.makeTierCallback(ctx),
+        compressionPressure: opts.compressionPressure,
+    }, runtime.store);
+    runtime.pulsing = false;
+    if (result.skipped)
+        return { skipped: true };
+    if (!result.deduped) {
+        runtime.rt.persistedThisSession = true;
+        runtime.rt.lastCheckpointId = result.checkpointId;
+    }
+    runtime.rt.lastCompactedFrom = result.compactedFrom;
+    runtime.rt.lastCompactedTokens = result.tokenEstimate;
+    runtime.rt.dedupAttempts++;
+    // Honest "tokens saved" for this session-instance only:
+    //   new checkpoint      → original − stored
+    //   deduped onto existing → whole original region (nothing new stored)
+    // Resets to 0 on session_start (rt is rebuilt) — so a fresh session shows 0
+    // while the repo's cumulative saved (SQLite meta) keeps the running total.
+    const saved = result.deduped
+        ? result.originalTokenEstimate
+        : Math.max(0, result.originalTokenEstimate - result.tokenEstimate);
+    runtime.rt.tokensSaved += saved;
+    runtime.rt.compactCount += 1;
+    incCompactCount(runtime.currentStateDir);
+    if (result.deduped) {
+        runtime.rt.cacheHitTokens += saved;
+        incCacheHitTokens(saved, runtime.currentStateDir);
+    }
+    runtime.rt.lastCompactAt = Date.now();
+    if (result.deduped)
+        runtime.rt.dedupSkips++;
+    // Grow the rolling "saved" goal so the progress bar always has a fresh
+    // denominator (we don't want it pinned at 100% once we pass an old target).
+    if (runtime.rt.tokensSaved > runtime.savedGoal)
+        runtime.savedGoal = Math.ceil((runtime.rt.tokensSaved * 1.25) / 10_000) * 10_000;
+    // Live toolbar activity: what file/region just got compacted or deduped.
+    // Rendered via the rotating ticker line (see snapshot); the ring buffer is
+    // cycled one-per-repaint so the single line scrolls through recent files.
+    const files = result.filesModified ?? [];
+    const fileLabel = files.length
+        ? files.map((f) => f.split("/").pop() ?? f).slice(0, 2).join(", ")
+        : result.regionHash.slice(0, 8);
+    runtime.lastActivityAt = Date.now();
+    // Explain-why line: surfaced while fresh. Pulls the dedup reason (which for
+    // L2 includes the cosine sim) so the user sees WHY a region was kept/dropped.
+    runtime.lastWhy = result.deduped
+        ? `why: deduped@${result.dedupReason ?? "tier"}`
+        : `why: compacted → ${result.checkpointId}`;
+    // Recall/activity ticker: record this event in the ring buffer.
+    const savedK = (saved / 1000).toFixed(1);
+    runtime.pushTicker(result.deduped
+        ? `${C.green}♻${C.reset} deduped ${fileLabel} · ${savedK}k saved`
+        : `${C.cyan}🗜${C.reset} ${result.checkpointId} · +${savedK}k · ${fileLabel}`);
+    // The per-tier trace has settled into the final outcome — fold it back into
+    // the activity line and stop showing the live trace.
+    runtime.tierTrace = undefined;
+    // Record session activity + a daily-log entry in the per-repo SQLite store
+    // (foundation for resume-sessions / daily-log features). Best-effort — never
+    // block a compaction on bookkeeping.
+    try {
+        const root = resolveRepoRoot(ctx.cwd);
+        touchSession(sid, root, runtime.currentStateDir);
+        logDaily(sid, "compact", result.checkpointId, saved, runtime.currentStateDir);
+    }
+    catch {
+        /* non-fatal: stats bookkeeping only */
+    }
+    // S21.2: best-effort consolidation of near-duplicate memories for this repo.
+    // Runs after the per-repo stats touch so `consolidateMemories` can use the
+    // same stateDir. Non-fatal — a failed consolidate never blocks a compaction.
+    // Only runs when new memory ops landed in this pass (otherwise the prior
+    // compaction's consolidate already had its shot — re-running would just
+    // touch every row again with no merges).
+    if (!result.deduped && runtime.memoriesTouchedThisCompaction > 0) {
+        try {
+            const root = resolveRepoRoot(ctx.cwd);
+            void consolidateMemories(runtime.currentStateDir, root).then((n) => {
+                if (n > 0)
+                    runtime.pushTicker(`${C.green}∫${C.reset} consolidated ${n} memory dup${n === 1 ? "" : "s"}`);
+            }, () => {
+                /* swallow: consolidate failures must never surface to the user */
+            });
+        }
+        catch {
+            /* non-fatal */
+        }
+    }
+    // S24 review-on-compact: when pressure is high, the just-compacted region is
+    // exactly the context worth remembering, so review it immediately rather than
+    // waiting for the next turn-cadence tick. Uses the shared runMemoryReview
+    // helper (fire-and-forget; doCompact is sync). Best-effort + non-fatal. Only
+    // fires above the `high` band so low-pressure compactions don't pay the cost.
+    if (!result.deduped && config.memoryAutoReview && runtime.pressureBand !== "low" && runtime.pressureBand !== "medium") {
+        void runMemoryReview(runtime, view, "pressure");
+    }
+    // Sentinel marker: a non-LLM bookkeeping entry so subsequent triggers can
+    // skip re-vectorizing an already-compacted region (zero token cost).
+    pi.appendEntry(MARKER_TYPE, {
+        checkpointId: result.checkpointId,
+        regionHash: result.regionHash,
+        tokenEstimate: result.tokenEstimate,
+        deduped: result.deduped,
+    });
+    // Fix D: refresh the RAPTOR tree for this session so live recall (search) can
+    // serve high-level summaries. Best-effort + non-fatal: never block compaction.
+    // Budget-guarded (RAPTOR_BUDGET_MS) so it can't hang a large session.
+    if (config.raptorEnabled && !result.deduped) {
+        try {
+            const dd = loadDedupConfig();
+            const all = runtime.store.list(sid);
+            const leaves = all.map((cp) => ({
+                id: cp.checkpointId,
+                messages: [],
+                sourceText: cp.normalizedText ?? cp.summary ?? cp.regionHash,
+                embedding: cp.embedding,
+            }));
+            if (leaves.length >= 2) {
+                // S25: stamp the tree with the newest checkpoint epoch so the
+                // freshness guard in raptorSearchHits can reject stale trees after a
+                // later compaction adds newer checkpoints.
+                const builtAt = all.length > 0 ? Math.max(...all.map((c) => c.timestamp)) : Date.now();
+                runRaptor(leaves, {
+                    stateDir: runtime.currentStateDir,
+                    sessionId: sid,
+                    budgetMs: dd.RAPTOR_BUDGET_MS,
+                    clustersPerLevel: dd.RAPTOR_CLUSTERS_PER_LEVEL,
+                    consistencyThreshold: dd.RAPTOR_CONSISTENCY,
+                    logger: runtime.logger,
+                    builtAt: Number.isFinite(builtAt) ? builtAt : Date.now(),
+                });
+            }
+        }
+        catch {
+            /* non-fatal: tree refresh never blocks a compaction */
+        }
+    }
+    // Slice 2: best-effort mirror of the new checkpoint into the async global
+    // PGlite/HNSW vector index. Fires once per compaction (not per-add), so the
+    // shared global dir is never hammered by concurrent test workers.
+    // Non-fatal: a WASM init failure degrades to the sync scan silently.
+    if (!result.deduped) {
+        try {
+            const all = runtime.store.list(sid);
+            const latest = all.find((cp) => cp.checkpointId === result.checkpointId);
+            if (latest?.embedding) {
+                void indexUpsertEmbedding(runtime.currentStateDir, sid, latest.checkpointId, latest.embedding).catch(() => {
+                    /* non-fatal: index refresh never blocks a compaction */
+                });
+            }
+        }
+        catch {
+            /* non-fatal: index refresh never blocks a compaction */
+        }
+    }
+    runtime.setStatus(ctx, runtime.rt.persistedThisSession
+        ? `mega-compact: ${result.checkpointId} · ${saved} tok saved`
+        : `mega-compact: ready`);
+    runtime.logger.info("compact", {
+        sessionId: sid,
+        checkpointId: result.checkpointId ?? "(deduped)",
+        deduped: result.deduped,
+        tokenEstimate: saved,
+        compactedFrom: result.compactedFrom,
+    });
+    runtime.dashboard.event("compact", {
+        sessionId: sid,
+        checkpointId: result.checkpointId ?? "(deduped)",
+        deduped: result.deduped,
+        tokenEstimate: saved,
+        compactedFrom: result.compactedFrom,
+    });
+    runtime.snapshot(ctx);
+    return { skipped: false, result, keepFrom, saved };
+}
+/**
+ * Predict whether pi's `ctx.compact()` would throw a no-op error — "Already
+ * compacted" or "Nothing to compact (session too small)" — so the auto-trigger
+ * can SKIP the call instead of surfacing a hard, user-facing error.
+ *
+ * Why we can't intercept or suppress it: pi's public `compact()` computes
+ * `prepareCompaction()` and throws *before* it emits `session_before_compact`,
+ * so our handler there never runs on the no-op path. And `ctx.compact()`'s
+ * `onError` callback fires only AFTER pi has already emitted a `compaction_end`
+ * event carrying the error message (which the interactive UI renders) — so
+ * `onError` cannot mute it either. The only robust fix is to not call
+ * `ctx.compact()` when pi would no-op. (pi's own `_runAutoCompaction` path is
+ * silent on this same condition; the public path we're forced through is the
+ * one that throws.)
+ *
+ * Skipping is correct, not a compromise: by the time this runs, `runCompact()`
+ * has already persisted the recall checkpoint (Path A). The durable on-disk
+ * trim is only useful when pi can actually summarize a region; a transcript
+ * under pi's `keepRecentTokens` budget is small enough that reloading it on
+ * resume isn't a token-growth problem, so the durable trim is unnecessary
+ * there anyway.
+ *
+ * Mirrors pi's `prepareCompaction()` return-undefined conditions (compaction.js):
+ *  (1) last entry is a compaction      → "Already compacted"
+ *  (2) <2 cut-point messages since the last compaction → nothing to summarize
+ *      (a cut point = any non-toolResult message — user/assistant/bash/custom/
+ *       branchSummary/compactionSummary — matching pi's isCutPointMessage)
+ *  (3) transcript tokens since the last compaction < keepRecentTokens → pi
+ *      keeps everything → nothing to summarize
+ * `keepRecentTokens` isn't readable from the extension API, so (3) uses the pi
+ * default (20000) as a conservative floor; raise it via
+ * `MEGACOMPACT_DURABLE_TRIM_FLOOR` if you raise pi's `compact.keepRecentTokens`.
+ *
+ * Best-effort: on any read error returns true (skip) — skipping a durable trim
+ * is always safe; calling `ctx.compact()` on a no-op throws to the user.
+ */
+export function piCompactWouldNoop(ctx) {
+    try {
+        const branch = ctx.sessionManager.getBranch();
+        if (branch.length === 0)
+            return true;
+        // (1) already compacted — pi throws "Already compacted"
+        if (branch[branch.length - 1].type === "compaction")
+            return true;
+        // boundaryStart = index just after the most recent compaction entry (or 0)
+        let boundaryStart = 0;
+        for (let i = branch.length - 1; i >= 0; i--) {
+            if (branch[i].type === "compaction") {
+                boundaryStart = i + 1;
+                break;
+            }
+        }
+        let cutPoints = 0;
+        let tokens = 0;
+        for (let i = boundaryStart; i < branch.length; i++) {
+            const e = branch[i];
+            if (e.type === "compaction")
+                continue;
+            let isCut = false;
+            for (const m of sessionEntryToContextMessages(e)) {
+                // pi's isCutPointMessage: every role except toolResult
+                if (m.role !== "toolResult")
+                    isCut = true;
+                const c = m.content;
+                const text = typeof c === "string" ? c
+                    : Array.isArray(c)
+                        ? c.map((b) => b?.text ?? "").join(" ")
+                        : "";
+                if (text)
+                    tokens += estimateBlockTokens(text);
+            }
+            if (isCut)
+                cutPoints++;
+        }
+        // (2) need >=2 cut points so the kept cut isn't the first message
+        if (cutPoints < 2)
+            return true;
+        // (3) transcript under pi's keepRecentTokens budget → pi keeps everything
+        if (tokens < durableTrimFloorTokens())
+            return true;
+        return false;
+    }
+    catch {
+        return true; // safe: skip the durable trim rather than risk a user-facing throw
+    }
+}
+/** pi's default keepRecentTokens (compaction settings). Override with
+ *  MEGACOMPACT_DURABLE_TRIM_FLOOR if you raise pi's compact.keepRecentTokens. */
+function durableTrimFloorTokens() {
+    const raw = process.env.MEGACOMPACT_DURABLE_TRIM_FLOOR;
+    if (raw !== undefined && Number.isFinite(Number(raw)))
+        return Number(raw);
+    return 20_000;
+}

package/dist/extensions/mega-pipeline/memory-review.js

@@ -0,0 +1,38 @@
+/**
+ * memory-review.ts — review live conversation & persist durable memories.
+ *
+ * `runMemoryReview` is shared by the pressure-scaled turn-end cadence
+ * (mega-events.ts) AND review-on-compact (compact.ts) so both paths run the
+ * identical review body. Best-effort + non-fatal: a review failure is swallowed
+ * and never breaks the caller.
+ */
+import { C, } from "../mega-runtime.js";
+/**
+ * Review the live conversation and persist durable memories (S20+S24). Shared by
+ * the pressure-scaled turn-end cadence (mega-events.ts) AND review-on-compact
+ * (below) so both paths run the identical review body. Best-effort + non-fatal:
+ * a review failure is swallowed and never breaks the caller. On success, the
+ * number of applied ops is returned so callers can feed the consolidation gate.
+ *
+ * @param view the engine message view to review (caller builds it)
+ * @param label a short source tag for the ticker line (e.g. "pressure" / "turn")
+ */
+export async function runMemoryReview(runtime, view, label) {
+    try {
+        const { reviewConversation } = await import("../../src/memory.js");
+        const { applyMemoryOps } = await import("../../src/memoryOps.js");
+        const ops = reviewConversation(view, []);
+        if (ops.length) {
+            await applyMemoryOps(ops, runtime.currentStateDir);
+            // S21.2: ops landed — the compaction path reads this counter and fires
+            // `consolidateMemories` only when > 0.
+            runtime.memoriesTouchedThisCompaction += ops.length;
+            runtime.pushTicker(`${C.green}🧠${C.reset} reviewed ${ops.length} memory op${ops.length === 1 ? "" : "s"} (${label})`);
+        }
+        return ops.length;
+    }
+    catch {
+        /* non-fatal — auto-review must never break the turn loop / compaction */
+        return 0;
+    }
+}