pi-mega-compact 0.11.4 → 0.11.6

package/dist/src/httpEmbedder.test.js

@@ -1,277 +0,0 @@
-/**
- * httpEmbedder.test.ts — URL validation hardening tests (F1).
- *
- * Verifies that embeddingConfigFromEnv enforces loopback-only URLs:
- *   - Accepts 127.0.0.1, [::1], and localhost (DNS-resolved)
- *   - Rejects non-loopback IPs, credentials-in-URL, wrong schemes, invalid URLs
- *   - Rejects the userinfo bypass (localhost:port@evil.com masks a remote host)
- *   - Fails closed: invalid URL -> null -> caller falls back to TrigramEmbedder
- *
- * Hermetic: no network, no remote. DNS resolution of "localhost" uses the
- * OS resolver (/etc/hosts) which is always available.
- */
-import { test } from "node:test";
-import assert from "node:assert/strict";
-import { embeddingConfigFromEnv } from "./httpEmbedder.js";
-// Constructed to avoid literal scheme prefix in source (guardrails PREVENT-PI-004).
-const HTTP = "http" + "://";
-const HTTPS = "https" + "://";
-const ENV_KEYS = [
-    "MEGACOMPACT_EMBEDDING_URL",
-    "MEGACOMPACT_EMBEDDING_HEADERS",
-    "MEGACOMPACT_EMBEDDING_KEY",
-    "MEGACOMPACT_EMBEDDING_DIM",
-];
-function saveEnv() {
-    const saved = {};
-    for (const k of ENV_KEYS)
-        saved[k] = process.env[k];
-    return saved;
-}
-function clearEnv() {
-    for (const k of ENV_KEYS)
-        delete process.env[k];
-}
-function restoreEnv(saved) {
-    for (const k of ENV_KEYS) {
-        if (saved[k] === undefined)
-            delete process.env[k];
-        else
-            process.env[k] = saved[k];
-    }
-}
-// --- Unset URL ---------------------------------------------------------------
-test("embeddingConfigFromEnv: unset URL returns null", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        assert.equal(embeddingConfigFromEnv(), null);
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-// --- Accepted: loopback literals ---------------------------------------------
-test("embeddingConfigFromEnv: accepts loopback IPv4 127.0.0.1:PORT", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "127.0.0.1:8080/embed";
-        const cfg = embeddingConfigFromEnv();
-        assert.ok(cfg, "expected config for 127.0.0.1");
-        assert.equal(cfg.url, HTTP + "127.0.0.1:8080/embed");
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: accepts bare 127.0.0.1 (no port/path)", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "127.0.0.1";
-        const cfg = embeddingConfigFromEnv();
-        assert.ok(cfg, "expected config for bare 127.0.0.1");
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: accepts [::1]:PORT (IPv6 loopback)", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "[::1]:8080/embed";
-        const cfg = embeddingConfigFromEnv();
-        assert.ok(cfg, "expected config for [::1]");
-        assert.equal(cfg.url, HTTP + "[::1]:8080/embed");
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: accepts https loopback 127.0.0.1:PORT", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTPS + "127.0.0.1:8443/embed";
-        const cfg = embeddingConfigFromEnv();
-        assert.ok(cfg, "expected config for https 127.0.0.1");
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: accepts localhost:PORT (DNS resolves to loopback)", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "localhost:8080/embed";
-        const cfg = embeddingConfigFromEnv();
-        assert.ok(cfg, "expected config for localhost (DNS-resolved loopback)");
-        assert.equal(cfg.url, HTTP + "localhost:8080/embed");
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-// --- Rejected: non-loopback --------------------------------------------------
-test("embeddingConfigFromEnv: rejects non-loopback IPv4", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "192.168.1.1:8080/embed";
-        assert.equal(embeddingConfigFromEnv(), null);
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: rejects non-loopback IPv6", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "[2001:db8::1]:8080/embed";
-        assert.equal(embeddingConfigFromEnv(), null);
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: rejects 10.0.0.1 (private but not loopback)", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "10.0.0.1:8080/embed";
-        assert.equal(embeddingConfigFromEnv(), null);
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-// --- Rejected: credentials in URL (userinfo bypass) -------------------------
-test("embeddingConfigFromEnv: rejects userinfo bypass (localhost:port@evil.com)", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "localhost:8080@evil.com/embed";
-        assert.equal(embeddingConfigFromEnv(), null);
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: rejects user@localhost credentials", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "user@localhost:8080/embed";
-        assert.equal(embeddingConfigFromEnv(), null);
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: rejects :pass@localhost credentials", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + ":pass@127.0.0.1:8080/embed";
-        assert.equal(embeddingConfigFromEnv(), null);
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-// --- Rejected: wrong scheme / invalid URL ------------------------------------
-test("embeddingConfigFromEnv: rejects non-http scheme (ftp://)", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = "ftp://localhost:8080/embed";
-        assert.equal(embeddingConfigFromEnv(), null);
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: rejects invalid URL string", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = "not a url";
-        assert.equal(embeddingConfigFromEnv(), null);
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-// --- Config fields parsed correctly -----------------------------------------
-test("embeddingConfigFromEnv: parses apiKey and dim from env", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "127.0.0.1:8080";
-        process.env.MEGACOMPACT_EMBEDDING_KEY = "secret-key";
-        process.env.MEGACOMPACT_EMBEDDING_DIM = "768";
-        const cfg = embeddingConfigFromEnv();
-        assert.ok(cfg);
-        assert.equal(cfg.apiKey, "secret-key");
-        assert.equal(cfg.dim, 768);
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: parses valid headers from env", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "127.0.0.1:8080";
-        process.env.MEGACOMPACT_EMBEDDING_HEADERS = '{"X-Custom":"value"}';
-        const cfg = embeddingConfigFromEnv();
-        assert.ok(cfg);
-        assert.equal(cfg.headers?.["X-Custom"], "value");
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-test("embeddingConfigFromEnv: ignores invalid headers (does not throw)", () => {
-    const saved = saveEnv();
-    try {
-        clearEnv();
-        process.env.MEGACOMPACT_EMBEDDING_URL = HTTP + "127.0.0.1:8080";
-        process.env.MEGACOMPACT_EMBEDDING_HEADERS = "not json";
-        const cfg = embeddingConfigFromEnv();
-        assert.ok(cfg, "config should still be returned with invalid headers ignored");
-        assert.equal(cfg.url, HTTP + "127.0.0.1:8080");
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});
-// --- Fail-closed: never throws -----------------------------------------------
-test("embeddingConfigFromEnv: never throws on any input", () => {
-    const saved = saveEnv();
-    try {
-        const inputs = [
-            "",
-            "not a url",
-            "ftp://localhost",
-            HTTP + "192.168.1.1:8080",
-            HTTP + "localhost:8080@evil.com/",
-            HTTP + "[2001:db8::1]:80",
-            "file:///etc/passwd",
-            "javascript:alert(1)",
-        ];
-        for (const input of inputs) {
-            clearEnv();
-            process.env.MEGACOMPACT_EMBEDDING_URL = input;
-            assert.doesNotThrow(() => embeddingConfigFromEnv(), `should not throw for: ${input}`);
-        }
-    }
-    finally {
-        restoreEnv(saved);
-    }
-});

package/dist/src/importance.test.js

@@ -1,372 +0,0 @@
-/**
- * importance.test.ts — S40A unit tests for the importance scoring engine.
- *
- * Covers all exports: ContextItemType, DEFAULT_MULTIPLIERS, detectItemType,
- * ageDecay, recencyBoost, retentionBoost, score, preservationCutoff,
- * itemsToPreserve, scoreEngineMessages.
- *
- * Determinism is enforced — same inputs MUST produce same outputs.
- */
-import { test } from "node:test";
-import assert from "node:assert/strict";
-import { ContextItemType, DEFAULT_MULTIPLIERS, detectItemType, ageDecay, recencyBoost, retentionBoost, score, preservationCutoff, itemsToPreserve, scoreEngineMessages, } from "./importance.js";
-const NOW = 1_700_000_000_000; // fixed epoch ms for determinism
-// ---- S40A-1: types and enums ----
-test("ContextItemType enum has 8 variants with snake_case values", () => {
-    assert.equal(ContextItemType.UserMessage, "user_message");
-    assert.equal(ContextItemType.AssistantMessage, "assistant_message");
-    assert.equal(ContextItemType.SystemMessage, "system_message");
-    assert.equal(ContextItemType.CodeBlock, "code_block");
-    assert.equal(ContextItemType.Error, "error");
-    assert.equal(ContextItemType.Decision, "decision");
-    assert.equal(ContextItemType.FileModification, "file_modification");
-    assert.equal(ContextItemType.ToolExecution, "tool_execution");
-});
-// ---- S40A-2: DEFAULT_MULTIPLIERS ----
-test("DEFAULT_MULTIPLIERS matches the Rust reference values", () => {
-    assert.equal(DEFAULT_MULTIPLIERS[ContextItemType.UserMessage], 1.5);
-    assert.equal(DEFAULT_MULTIPLIERS[ContextItemType.AssistantMessage], 1.0);
-    assert.equal(DEFAULT_MULTIPLIERS[ContextItemType.SystemMessage], 0.5);
-    assert.equal(DEFAULT_MULTIPLIERS[ContextItemType.CodeBlock], 1.2);
-    assert.equal(DEFAULT_MULTIPLIERS[ContextItemType.Error], 2.0);
-    assert.equal(DEFAULT_MULTIPLIERS[ContextItemType.Decision], 2.5);
-    assert.equal(DEFAULT_MULTIPLIERS[ContextItemType.FileModification], 1.8);
-    assert.equal(DEFAULT_MULTIPLIERS[ContextItemType.ToolExecution], 1.3);
-});
-test("DEFAULT_MULTIPLIERS has an entry for every ContextItemType variant", () => {
-    for (const t of Object.values(ContextItemType)) {
-        assert.equal(typeof DEFAULT_MULTIPLIERS[t], "number", `missing multiplier for ${t}`);
-    }
-});
-// ---- S40A-3: ageDecay ----
-test("ageDecay: fresh (age=0) returns 0", () => {
-    assert.equal(ageDecay(0), 0);
-});
-test("ageDecay: 1 hour at 5%/hr returns 0.05", () => {
-    assert.equal(ageDecay(3_600_000), 0.05);
-});
-test("ageDecay: 14 hours caps at 0.7 (maxDecay)", () => {
-    assert.equal(ageDecay(14 * 3_600_000), 0.7);
-});
-test("ageDecay: beyond 14 hours stays capped at 0.7", () => {
-    assert.equal(ageDecay(100 * 3_600_000), 0.7);
-});
-test("ageDecay: negative age (future timestamp) returns 0", () => {
-    assert.equal(ageDecay(-5_000), 0);
-});
-test("ageDecay: custom rate and max override defaults", () => {
-    // 10h at 0.1/hr = 1.0 → capped at 0.8.
-    assert.equal(ageDecay(10 * 3_600_000, 0.1, 0.8), 0.8);
-    // 2h at 0.1/hr = 0.2.
-    assert.equal(ageDecay(2 * 3_600_000, 0.1, 0.8), 0.2);
-});
-// ---- S40A-4: recencyBoost + retentionBoost ----
-test("recencyBoost: <5min returns 1.2", () => {
-    assert.equal(recencyBoost(299_000), 1.2);
-});
-test("recencyBoost: >5min returns 1.0", () => {
-    assert.equal(recencyBoost(301_000), 1.0);
-});
-test("recencyBoost: exactly 5min (300_000ms) returns 1.0 (boundary)", () => {
-    assert.equal(recencyBoost(300_000), 1.0);
-});
-test("recencyBoost: custom threshold respected", () => {
-    assert.equal(recencyBoost(5000, 10_000), 1.2);
-    assert.equal(recencyBoost(11_000, 10_000), 1.0);
-});
-test("retentionBoost: flagged returns 3.0", () => {
-    assert.equal(retentionBoost(true), 3.0);
-});
-test("retentionBoost: unflagged returns 1.0", () => {
-    assert.equal(retentionBoost(false), 1.0);
-});
-// ---- S40A-5: detectItemType ----
-test("detectItemType: role=tool → ToolExecution (priority 1)", () => {
-    // Even if content looks like an error, tool role wins.
-    assert.equal(detectItemType("error: something failed", "tool"), ContextItemType.ToolExecution);
-});
-test("detectItemType: role=custom → SystemMessage (priority 2)", () => {
-    assert.equal(detectItemType("anything here", "custom"), ContextItemType.SystemMessage);
-});
-test("detectItemType: error content → Error", () => {
-    assert.equal(detectItemType("Error: ENOENT at src/config.ts:42", "assistant"), ContextItemType.Error);
-    assert.equal(detectItemType("Traceback (most recent call last)", "assistant"), ContextItemType.Error);
-    assert.equal(detectItemType("panic: runtime error", "assistant"), ContextItemType.Error);
-    assert.equal(detectItemType("E4030 connection refused", "assistant"), ContextItemType.Error);
-});
-test("detectItemType: decision content → Decision", () => {
-    assert.equal(detectItemType("we decided to use JWT auth", "assistant"), ContextItemType.Decision);
-    assert.equal(detectItemType("going with option B", "assistant"), ContextItemType.Decision);
-    assert.equal(detectItemType("let's go with the async approach", "assistant"), ContextItemType.Decision);
-    assert.equal(detectItemType("switching to a different database", "assistant"), ContextItemType.Decision);
-});
-test("detectItemType: fenced code block ≥20 chars → CodeBlock", () => {
-    const code = "```js\nconst x = 42;\nconst y = 'hello world';\n```";
-    assert.equal(detectItemType(code, "assistant"), ContextItemType.CodeBlock);
-});
-test("detectItemType: short fenced block (<20 chars) does NOT → CodeBlock", () => {
-    const short = "```\nshort\n```";
-    assert.notEqual(detectItemType(short, "assistant"), ContextItemType.CodeBlock);
-});
-test("detectItemType: file modification verbs → FileModification", () => {
-    assert.equal(detectItemType("edited src/config.ts to add a flag", "assistant"), ContextItemType.FileModification);
-    assert.equal(detectItemType("created README.md", "assistant"), ContextItemType.FileModification);
-    assert.equal(detectItemType("wrote tests/main.test.ts", "assistant"), ContextItemType.FileModification);
-});
-test("detectItemType: role=user → UserMessage", () => {
-    assert.equal(detectItemType("plain question", "user"), ContextItemType.UserMessage);
-});
-test("detectItemType: role=assistant → AssistantMessage", () => {
-    assert.equal(detectItemType("plain answer", "assistant"), ContextItemType.AssistantMessage);
-});
-test("detectItemType: fallback → AssistantMessage", () => {
-    // Unknown role with non-matching content falls back to AssistantMessage.
-    assert.equal(detectItemType("nothing matches here", "unknown"), ContextItemType.AssistantMessage);
-});
-test("detectItemType: empty string does not crash and falls back", () => {
-    assert.equal(detectItemType("", "user"), ContextItemType.UserMessage);
-    assert.equal(detectItemType("", "assistant"), ContextItemType.AssistantMessage);
-});
-test("detectItemType: priority — error before decision (first match wins)", () => {
-    // A message that contains BOTH "error" and "decided" — error wins (rule 3 before rule 4).
-    assert.equal(detectItemType("error: we decided to fail", "assistant"), ContextItemType.Error);
-});
-// ---- S40A-6: score ----
-test("score: fresh decision gets full multiplier (no decay, no recency/retention boost expected unless young)", () => {
-    const result = score({
-        id: "0",
-        content: "we decided to use JWT",
-        role: "assistant",
-        timestamp: NOW, // age 0
-    }, NOW);
-    assert.equal(result.type, ContextItemType.Decision);
-    assert.equal(result.rawMultiplier, 2.5);
-    assert.equal(result.ageDecay, 0);
-    assert.equal(result.recencyBoost, 1.2); // age 0 < 5min
-    assert.equal(result.retentionBoost, 1.0); // not userFlagged
-    // 2.5 * (1 - 0) * 1.2 * 1.0 = 3.0
-    assert.equal(result.finalScore, 3.0);
-});
-test("score: old decision decays", () => {
-    const tenHoursAgo = NOW - 10 * 3_600_000;
-    const result = score({
-        id: "1",
-        content: "we decided to use JWT",
-        role: "assistant",
-        timestamp: tenHoursAgo,
-    }, NOW);
-    // decay at 10h = 0.5; recency 1.0 (>5min); retention 1.0
-    // 2.5 * (1 - 0.5) * 1.0 * 1.0 = 1.25
-    assert.equal(result.ageDecay, 0.5);
-    assert.equal(result.recencyBoost, 1.0);
-    assert.equal(result.finalScore, 1.25);
-});
-test("score: userFlagged triples the result via retentionBoost", () => {
-    const flagged = score({
-        id: "2",
-        content: "we decided to use JWT",
-        role: "assistant",
-        timestamp: NOW,
-        userFlagged: true,
-    }, NOW);
-    const unflagged = score({
-        id: "3",
-        content: "we decided to use JWT",
-        role: "assistant",
-        timestamp: NOW,
-        userFlagged: false,
-    }, NOW);
-    // 3.0 / 1.0 retention ratio.
-    assert.equal(flagged.finalScore / unflagged.finalScore, 3.0);
-});
-test("score: clamps finalScore to minimum 0.01", () => {
-    // A system message (0.5x) at 14h+ (max decay 0.7) → 0.5 * 0.3 * 1.0 * 1.0 = 0.15. Above 0.01.
-    // Force below 0.01 with a custom multiplier of 0 and max decay.
-    const result = score({
-        id: "4",
-        content: "system filler",
-        role: "custom",
-        timestamp: NOW - 100 * 3_600_000,
-    }, NOW, { [ContextItemType.SystemMessage]: 0.01 }, { maxDecay: 0.99 });
-    // 0.01 * (1 - 0.99) * 1.0 * 1.0 = 0.0001 → clamped to 0.01.
-    assert.equal(result.finalScore, 0.01);
-});
-test("score: custom multipliers override DEFAULT_MULTIPLIERS", () => {
-    const result = score({
-        id: "5",
-        content: "we decided to use JWT",
-        role: "assistant",
-        timestamp: NOW,
-    }, NOW, { [ContextItemType.Decision]: 10.0 });
-    assert.equal(result.rawMultiplier, 10.0);
-    assert.equal(result.finalScore, 10.0 * 1.2); // 10 * (1-0) * 1.2 * 1.0
-});
-test("score: unknown role narrows to assistant defensively", () => {
-    const result = score({
-        id: "6",
-        content: "plain text",
-        role: "weird-role",
-        timestamp: NOW,
-    }, NOW);
-    assert.equal(result.role, "assistant");
-    assert.equal(result.type, ContextItemType.AssistantMessage);
-});
-test("score: undefined/null content does not crash", () => {
-    const r1 = score({ id: "a", content: undefined, role: "user", timestamp: NOW }, NOW);
-    const r2 = score({ id: "b", content: null, role: "assistant", timestamp: NOW }, NOW);
-    assert.equal(r1.type, ContextItemType.UserMessage);
-    assert.equal(r2.type, ContextItemType.AssistantMessage);
-    assert.equal(r1.content, "");
-    assert.equal(r2.content, "");
-});
-test("score: determinism — same inputs → same output", () => {
-    const item = {
-        id: "d1",
-        content: "we decided to use JWT",
-        role: "assistant",
-        timestamp: NOW - 3_600_000,
-    };
-    const a = score(item, NOW);
-    const b = score(item, NOW);
-    assert.deepEqual(a, b);
-});
-// ---- S40A-7: preservationCutoff + itemsToPreserve ----
-test("preservationCutoff: 10 items at ratio 0.3 → threshold is the 3rd-highest score", () => {
-    const items = Array.from({ length: 10 }, (_, i) => ({
-        id: String(i),
-        type: ContextItemType.AssistantMessage,
-        content: "",
-        role: "assistant",
-        timestamp: 0,
-        rawMultiplier: 1.0,
-        ageDecay: 0,
-        recencyBoost: 1.0,
-        retentionBoost: 1.0,
-        // scores 1.0, 2.0, ..., 10.0
-        finalScore: (i + 1) * 1.0,
-    }));
-    const threshold = preservationCutoff(items, 0.3);
-    // top 30% of 10 = 3 items; the 3rd highest is 8.0 (scores 10,9,8 preserved).
-    assert.equal(threshold, 8.0);
-});
-test("preservationCutoff: ratio=1.0 returns 0 (preserve all)", () => {
-    const items = [{ finalScore: 5, id: "x" }];
-    assert.equal(preservationCutoff(items, 1.0), 0);
-});
-test("preservationCutoff: ratio=0 returns Infinity (preserve none)", () => {
-    const items = [{ finalScore: 5, id: "x" }];
-    assert.equal(preservationCutoff(items, 0), Infinity);
-});
-test("preservationCutoff: empty input returns Infinity", () => {
-    assert.equal(preservationCutoff([], 0.5), Infinity);
-});
-test("itemsToPreserve: returns IDs at or above threshold", () => {
-    const items = Array.from({ length: 10 }, (_, i) => ({
-        id: String(i),
-        type: ContextItemType.AssistantMessage,
-        content: "",
-        role: "assistant",
-        timestamp: 0,
-        rawMultiplier: 1.0,
-        ageDecay: 0,
-        recencyBoost: 1.0,
-        retentionBoost: 1.0,
-        finalScore: (i + 1) * 1.0,
-    }));
-    const result = itemsToPreserve(items, 0.3);
-    assert.equal(result.totalScored, 10);
-    assert.equal(result.totalPreserved, 3);
-    assert.deepEqual([...result.preservedIds].sort(), ["7", "8", "9"]);
-});
-test("itemsToPreserve: ratio=1.0 preserves everything", () => {
-    const items = Array.from({ length: 5 }, (_, i) => ({
-        id: String(i),
-        type: ContextItemType.AssistantMessage,
-        content: "",
-        role: "assistant",
-        timestamp: 0,
-        rawMultiplier: 1.0,
-        ageDecay: 0,
-        recencyBoost: 1.0,
-        retentionBoost: 1.0,
-        finalScore: i + 1,
-    }));
-    const result = itemsToPreserve(items, 1.0);
-    assert.equal(result.totalPreserved, 5);
-});
-test("itemsToPreserve: ratio=0 preserves nothing", () => {
-    const items = Array.from({ length: 5 }, (_, i) => ({
-        id: String(i),
-        type: ContextItemType.AssistantMessage,
-        content: "",
-        role: "assistant",
-        timestamp: 0,
-        rawMultiplier: 1.0,
-        ageDecay: 0,
-        recencyBoost: 1.0,
-        retentionBoost: 1.0,
-        finalScore: i + 1,
-    }));
-    const result = itemsToPreserve(items, 0);
-    assert.equal(result.totalPreserved, 0);
-    assert.equal(result.threshold, Infinity);
-});
-test("itemsToPreserve: empty items returns empty set", () => {
-    const result = itemsToPreserve([], 0.5);
-    assert.equal(result.totalScored, 0);
-    assert.equal(result.totalPreserved, 0);
-    assert.equal(result.preservedIds.size, 0);
-});
-// ---- scoreEngineMessages: position-based age fallback ----
-test("scoreEngineMessages: assigns position-based timestamps (oldest first)", () => {
-    const messages = [
-        { role: "assistant", text: "we decided to use JWT" },
-        { role: "user", text: "ok" },
-        { role: "assistant", text: "edited src/config.ts" },
-    ];
-    const scored = scoreEngineMessages(messages, NOW);
-    assert.equal(scored.length, 3);
-    // Oldest (i=0) is `messages.length` minutes ago = 3 min.
-    assert.equal(scored[0].timestamp, NOW - 3 * 60_000);
-    // Newest (i=2) is 1 min ago.
-    assert.equal(scored[2].timestamp, NOW - 1 * 60_000);
-    // All recency-boosted (< 5 min).
-    assert.equal(scored[0].recencyBoost, 1.2);
-    assert.equal(scored[2].recencyBoost, 1.2);
-});
-test("scoreEngineMessages: preserves input order and ids are stringified indices", () => {
-    const messages = [
-        { role: "user", text: "first" },
-        { role: "user", text: "second" },
-    ];
-    const scored = scoreEngineMessages(messages, NOW);
-    assert.equal(scored[0].id, "0");
-    assert.equal(scored[1].id, "1");
-});
-test("scoreEngineMessages: empty input returns empty array", () => {
-    assert.deepEqual(scoreEngineMessages([], NOW), []);
-});
-test("scoreEngineMessages: determinism — same input → same output", () => {
-    const messages = [
-        { role: "assistant", text: "we decided to use JWT" },
-        { role: "user", text: "ok" },
-    ];
-    const a = scoreEngineMessages(messages, NOW);
-    const b = scoreEngineMessages(messages, NOW);
-    assert.deepEqual(a, b);
-});
-// ---- integration: a decision outsurvives filler ----
-test("integration: a decision is preserved over filler at the same age", () => {
-    // 10 messages, 1 decision (i=4) and 9 filler, all equally old.
-    const messages = Array.from({ length: 10 }, (_, i) => i === 4
-        ? { role: "assistant", text: "we decided to use JWT auth" }
-        : { role: "assistant", text: "ok, sounds good" });
-    const scored = scoreEngineMessages(messages, NOW - 10 * 60_000); // 10 min old → no recency boost
-    // decision rawMultiplier 2.5 vs filler 1.0; same decay, same recency.
-    const decision = scored[4];
-    const filler = scored[0];
-    assert.ok(decision.finalScore > filler.finalScore);
-    // top 20% → 2 items; the decision must be in the preserved set.
-    const result = itemsToPreserve(scored, 0.2);
-    assert.ok(result.preservedIds.has("4"), "decision must be preserved");
-});