alvin-bot 4.18.0 → 4.18.2

package/test/i18n.test.ts

@@ -1,108 +0,0 @@
-import { describe, it, expect, beforeEach } from "vitest";
-import { t, initI18n, setLocale, getLocale, LOCALE_NAMES, LOCALE_FLAGS } from "../src/i18n.js";
-
-describe("i18n", () => {
-  beforeEach(() => {
-    // Reset to a known state before each test
-    setLocale("en");
-  });
-
-  describe("LOCALE_NAMES", () => {
-    it("exposes all four supported locales", () => {
-      expect(LOCALE_NAMES.en).toBe("English");
-      expect(LOCALE_NAMES.de).toBe("Deutsch");
-      expect(LOCALE_NAMES.es).toBe("Español");
-      expect(LOCALE_NAMES.fr).toBe("Français");
-    });
-  });
-
-  describe("LOCALE_FLAGS", () => {
-    it("has a flag emoji for every locale", () => {
-      expect(LOCALE_FLAGS.en).toBe("🇬🇧");
-      expect(LOCALE_FLAGS.de).toBe("🇩🇪");
-      expect(LOCALE_FLAGS.es).toBe("🇪🇸");
-      expect(LOCALE_FLAGS.fr).toBe("🇫🇷");
-    });
-  });
-
-  describe("t() — translation lookup", () => {
-    it("returns the English string when locale is en", () => {
-      const msg = t("bot.cancel.cancelling", "en");
-      expect(msg).toContain("Cancelling");
-    });
-
-    it("returns the German string when locale is de", () => {
-      const msg = t("bot.cancel.cancelling", "de");
-      expect(msg).toContain("abgebrochen");
-    });
-
-    it("returns the Spanish string when locale is es", () => {
-      const msg = t("bot.cancel.cancelling", "es");
-      expect(msg.toLowerCase()).toContain("cancelando");
-    });
-
-    it("returns the French string when locale is fr", () => {
-      const msg = t("bot.cancel.cancelling", "fr");
-      expect(msg.toLowerCase()).toContain("annulation");
-    });
-
-    it("falls back to English when locale is missing for a key", () => {
-      // Use a TUI key which only has en+de — request es, should fall through
-      // to en since tui.* keys aren't translated for es/fr.
-      const msg = t("tui.title", "es");
-      expect(msg).toContain("Alvin Bot TUI");
-    });
-
-    it("returns the key itself if no locale has it at all", () => {
-      const msg = t("bot.nonexistent.key.for.test", "en");
-      expect(msg).toBe("bot.nonexistent.key.for.test");
-    });
-
-    it("uses the global currentLocale when no locale is passed", () => {
-      setLocale("de");
-      const msg = t("bot.cancel.cancelling");
-      expect(msg).toContain("abgebrochen");
-    });
-  });
-
-  describe("t() — interpolation", () => {
-    it("substitutes a single {var} placeholder", () => {
-      const msg = t("bot.error.timeoutStuck", "en", { min: 10 });
-      expect(msg).toContain("10 minutes");
-    });
-
-    it("substitutes multiple {var} placeholders", () => {
-      const msg = t("bot.error.midStream", "en", {
-        name: "claude-sdk",
-        detail: "connection reset",
-      });
-      expect(msg).toContain("claude-sdk");
-      expect(msg).toContain("connection reset");
-    });
-
-    it("interpolation works in all four locales", () => {
-      const vars = { min: 7 };
-      expect(t("bot.error.timeoutStuck", "en", vars)).toContain("7");
-      expect(t("bot.error.timeoutStuck", "de", vars)).toContain("7");
-      expect(t("bot.error.timeoutStuck", "es", vars)).toContain("7");
-      expect(t("bot.error.timeoutStuck", "fr", vars)).toContain("7");
-    });
-
-    it("leaves {placeholder} visible if the var is not provided", () => {
-      const msg = t("bot.error.timeoutStuck", "en", {});
-      expect(msg).toContain("{min}");
-    });
-  });
-
-  describe("initI18n / setLocale / getLocale", () => {
-    it("initI18n with explicit locale sets currentLocale", () => {
-      initI18n("fr");
-      expect(getLocale()).toBe("fr");
-    });
-
-    it("setLocale updates the global locale", () => {
-      setLocale("es");
-      expect(getLocale()).toBe("es");
-    });
-  });
-});

package/test/list-subagents-merged.test.ts

@@ -1,172 +0,0 @@
-/**
- * v4.14.1 — `/subagents list` must show v4.13+ dispatch agents too.
- *
- * Root cause: `listSubAgents()` in subagents.ts only iterates the
- * `activeAgents` Map (B1+B2 from v4.0.0). v4.13's `alvin_dispatch_agent`
- * MCP tool writes into `async-agent-watcher.ts`'s `pending` Map instead.
- * User-facing impact: "no subagents running" while the bot is visibly
- * dispatching sub-agents.
- *
- * Fix strategy: a new `listActiveSubAgents()` helper that merges both
- * registries into a unified SubAgentInfo-shaped list. The `/subagents
- * list` handler uses this instead of the bare `listSubAgents()`.
- * Cancel/result operations keep using the old registry — we can't
- * cancel a detached `claude -p` subprocess anyway without knowing its
- * PID, which isn't tracked.
- */
-import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
-import fs from "fs";
-import os from "os";
-import { resolve } from "path";
-
-const TEST_DATA_DIR = resolve(
-  os.tmpdir(),
-  `alvin-list-merged-${process.pid}-${Date.now()}`,
-);
-
-beforeEach(async () => {
-  if (fs.existsSync(TEST_DATA_DIR)) {
-    fs.rmSync(TEST_DATA_DIR, { recursive: true, force: true });
-  }
-  fs.mkdirSync(TEST_DATA_DIR, { recursive: true });
-  process.env.ALVIN_DATA_DIR = TEST_DATA_DIR;
-  vi.resetModules();
-
-  vi.doMock("../src/services/subagent-delivery.js", () => ({
-    deliverSubAgentResult: async () => {},
-    attachBotApi: () => {},
-    __setBotApiForTest: () => {},
-  }));
-});
-
-afterEach(async () => {
-  try {
-    const mod = await import("../src/services/async-agent-watcher.js");
-    mod.stopWatcher();
-    mod.__resetForTest();
-  } catch {}
-});
-
-describe("listActiveSubAgents merged view (v4.14.1)", () => {
-  it("returns empty list when neither registry has agents", async () => {
-    const mod = await import("../src/services/subagents.js");
-    expect(await mod.listActiveSubAgents()).toEqual([]);
-  });
-
-  it("includes async-agent-watcher pending agents in the merged list", async () => {
-    const watcher = await import("../src/services/async-agent-watcher.js");
-    watcher.registerPendingAgent({
-      agentId: "alvin-abc123",
-      outputFile: `${TEST_DATA_DIR}/out.jsonl`,
-      description: "Research Higgsfield",
-      prompt: "...",
-      chatId: "C012SLACK",
-      userId: "U123",
-      toolUseId: null,
-      sessionKey: "slack:C012SLACK",
-      platform: "slack",
-    });
-
-    const mod = await import("../src/services/subagents.js");
-    const agents = await mod.listActiveSubAgents();
-    expect(agents).toHaveLength(1);
-    expect(agents[0].id).toBe("alvin-abc123");
-    expect(agents[0].name).toBe("Research Higgsfield");
-    expect(agents[0].status).toBe("running");
-    expect(agents[0].depth).toBe(0);
-    expect(agents[0].platform).toBe("slack");
-  });
-
-  it("merges multiple agents from both registries without dupes", async () => {
-    const watcher = await import("../src/services/async-agent-watcher.js");
-    watcher.registerPendingAgent({
-      agentId: "alvin-one",
-      outputFile: `${TEST_DATA_DIR}/a.jsonl`,
-      description: "Agent One",
-      prompt: "p",
-      chatId: 42,
-      userId: 42,
-      toolUseId: null,
-      sessionKey: "s",
-      platform: "telegram",
-    });
-    watcher.registerPendingAgent({
-      agentId: "alvin-two",
-      outputFile: `${TEST_DATA_DIR}/b.jsonl`,
-      description: "Agent Two",
-      prompt: "p",
-      chatId: 42,
-      userId: 42,
-      toolUseId: null,
-      sessionKey: "s",
-      platform: "telegram",
-    });
-
-    const mod = await import("../src/services/subagents.js");
-    const agents = await mod.listActiveSubAgents();
-    const ids = agents.map((a) => a.id).sort();
-    expect(ids).toEqual(["alvin-one", "alvin-two"]);
-  });
-
-  it("preserves startedAt timestamp for age rendering", async () => {
-    const fixedTs = Date.now() - 45_000; // 45 seconds ago
-    const watcher = await import("../src/services/async-agent-watcher.js");
-    watcher.registerPendingAgent({
-      agentId: "alvin-aged",
-      outputFile: `${TEST_DATA_DIR}/aged.jsonl`,
-      description: "Old agent",
-      prompt: "p",
-      chatId: 1,
-      userId: 1,
-      toolUseId: null,
-      sessionKey: "s",
-      platform: "slack",
-    });
-
-    const mod = await import("../src/services/subagents.js");
-    const agents = await mod.listActiveSubAgents();
-    expect(agents[0].startedAt).toBeGreaterThan(fixedTs - 1000);
-    expect(agents[0].startedAt).toBeLessThan(Date.now() + 1000);
-  });
-
-  it("tags async dispatch agents with source='cron' (matches v4.12 banner format)", async () => {
-    const watcher = await import("../src/services/async-agent-watcher.js");
-    watcher.registerPendingAgent({
-      agentId: "alvin-sourced",
-      outputFile: `${TEST_DATA_DIR}/s.jsonl`,
-      description: "sourced",
-      prompt: "p",
-      chatId: 1,
-      userId: 1,
-      toolUseId: null,
-      sessionKey: "s",
-      platform: "telegram",
-    });
-    const mod = await import("../src/services/subagents.js");
-    const agents = await mod.listActiveSubAgents();
-    // source='cron' = the ⏰ badge in /subagents list rendering. Matches
-    // the existing v4.12.x watcher delivery's SubAgentInfo.source value.
-    expect(agents[0].source).toBe("cron");
-  });
-
-  it("listSubAgents() (v4.0.0 API) is unchanged and doesn't include pending dispatches", async () => {
-    const watcher = await import("../src/services/async-agent-watcher.js");
-    watcher.registerPendingAgent({
-      agentId: "alvin-isolated",
-      outputFile: `${TEST_DATA_DIR}/iso.jsonl`,
-      description: "isolated",
-      prompt: "p",
-      chatId: 1,
-      userId: 1,
-      toolUseId: null,
-      sessionKey: "s",
-      platform: "telegram",
-    });
-    const mod = await import("../src/services/subagents.js");
-    // The original listSubAgents is kept pure — only the merged helper
-    // returns combined results. Cancel/result paths still use the
-    // bot-level registry.
-    expect(mod.listSubAgents()).toHaveLength(0);
-    expect(await mod.listActiveSubAgents()).toHaveLength(1);
-  });
-});

package/test/memory-extractor.test.ts

@@ -1,151 +0,0 @@
-/**
- * v4.11.0 — Auto-fact-extraction.
- *
- * When compaction archives messages, instead of just dumping prose into
- * the daily log, run a structured extraction pass that pulls user_facts,
- * preferences, and decisions out of the chunk and appends them to MEMORY.md
- * (de-duplicated by exact-string match).
- *
- * Marked experimental in v4.11.0. Opt out via MEMORY_EXTRACTION_DISABLED=1.
- */
-import { describe, it, expect, beforeEach, vi } from "vitest";
-import fs from "fs";
-import os from "os";
-import { resolve } from "path";
-
-const TEST_DATA_DIR = resolve(os.tmpdir(), `alvin-mem-extract-${process.pid}-${Date.now()}`);
-
-beforeEach(() => {
-  if (fs.existsSync(TEST_DATA_DIR)) fs.rmSync(TEST_DATA_DIR, { recursive: true, force: true });
-  fs.mkdirSync(resolve(TEST_DATA_DIR, "memory"), { recursive: true });
-  process.env.ALVIN_DATA_DIR = TEST_DATA_DIR;
-  delete process.env.MEMORY_EXTRACTION_DISABLED;
-  vi.resetModules();
-});
-
-describe("memory-extractor (v4.11.0)", () => {
-  it("parseExtractedFacts handles a clean JSON response", async () => {
-    const { parseExtractedFacts } = await import("../src/services/memory-extractor.js");
-    const json = JSON.stringify({
-      user_facts: ["User User lives in Berlin"],
-      preferences: ["Replies in German"],
-      decisions: ["Use VPS VPS for production"],
-    });
-    const facts = parseExtractedFacts(json);
-    expect(facts.user_facts).toEqual(["User User lives in Berlin"]);
-    expect(facts.preferences).toEqual(["Replies in German"]);
-    expect(facts.decisions).toEqual(["Use VPS VPS for production"]);
-  });
-
-  it("parseExtractedFacts handles JSON wrapped in markdown code fences", async () => {
-    const { parseExtractedFacts } = await import("../src/services/memory-extractor.js");
-    const wrapped = "```json\n" + JSON.stringify({
-      user_facts: ["fact 1"],
-    }) + "\n```";
-    const facts = parseExtractedFacts(wrapped);
-    expect(facts.user_facts).toEqual(["fact 1"]);
-  });
-
-  it("parseExtractedFacts handles JSON with surrounding prose", async () => {
-    const { parseExtractedFacts } = await import("../src/services/memory-extractor.js");
-    const messy = `Sure, here are the extracted facts:
-${JSON.stringify({ user_facts: ["x"], preferences: [], decisions: [] })}
-Hope this helps!`;
-    const facts = parseExtractedFacts(messy);
-    expect(facts.user_facts).toEqual(["x"]);
-  });
-
-  it("parseExtractedFacts returns empty arrays on garbage input", async () => {
-    const { parseExtractedFacts } = await import("../src/services/memory-extractor.js");
-    const facts = parseExtractedFacts("not json at all");
-    expect(facts.user_facts).toEqual([]);
-    expect(facts.preferences).toEqual([]);
-    expect(facts.decisions).toEqual([]);
-  });
-
-  it("parseExtractedFacts filters non-string entries from arrays", async () => {
-    const { parseExtractedFacts } = await import("../src/services/memory-extractor.js");
-    const messy = JSON.stringify({
-      user_facts: ["good", 42, null, "good2"],
-      preferences: [],
-      decisions: [],
-    });
-    const facts = parseExtractedFacts(messy);
-    expect(facts.user_facts).toEqual(["good", "good2"]);
-  });
-
-  it("appendFactsToMemoryFile writes new facts under structured headers", async () => {
-    const { appendFactsToMemoryFile } = await import("../src/services/memory-extractor.js");
-    await appendFactsToMemoryFile({
-      user_facts: ["User uses launchd for the bot"],
-      preferences: [],
-      decisions: ["v4.11.0 ships memory persistence"],
-    });
-    const memFile = resolve(TEST_DATA_DIR, "memory", "MEMORY.md");
-    expect(fs.existsSync(memFile)).toBe(true);
-    const content = fs.readFileSync(memFile, "utf-8");
-    expect(content).toMatch(/User uses launchd for the bot/);
-    expect(content).toMatch(/v4\.11\.0 ships memory persistence/);
-  });
-
-  it("appendFactsToMemoryFile dedupes on exact-string match", async () => {
-    const { appendFactsToMemoryFile } = await import("../src/services/memory-extractor.js");
-    await appendFactsToMemoryFile({
-      user_facts: ["User uses launchd for the bot"],
-      preferences: [],
-      decisions: [],
-    });
-    await appendFactsToMemoryFile({
-      user_facts: ["User uses launchd for the bot", "User drinks coffee"],
-      preferences: [],
-      decisions: [],
-    });
-    const content = fs.readFileSync(resolve(TEST_DATA_DIR, "memory", "MEMORY.md"), "utf-8");
-    const matches = content.match(/User uses launchd for the bot/g);
-    expect(matches).toHaveLength(1); // not duplicated
-    expect(content).toMatch(/User drinks coffee/);
-  });
-
-  it("appendFactsToMemoryFile returns 0 when all facts are duplicates", async () => {
-    const { appendFactsToMemoryFile } = await import("../src/services/memory-extractor.js");
-    await appendFactsToMemoryFile({
-      user_facts: ["unique fact"],
-      preferences: [],
-      decisions: [],
-    });
-    const stored = await appendFactsToMemoryFile({
-      user_facts: ["unique fact"],
-      preferences: [],
-      decisions: [],
-    });
-    expect(stored).toBe(0);
-  });
-
-  it("extractAndStoreFacts is a no-op when MEMORY_EXTRACTION_DISABLED=1", async () => {
-    process.env.MEMORY_EXTRACTION_DISABLED = "1";
-    vi.resetModules();
-    const { extractAndStoreFacts } = await import("../src/services/memory-extractor.js");
-    const result = await extractAndStoreFacts("some conversation text");
-    expect(result.disabled).toBe(true);
-    expect(result.factsStored).toBe(0);
-  });
-
-  it("extractAndStoreFacts returns 0 stored on too-short input", async () => {
-    const { extractAndStoreFacts } = await import("../src/services/memory-extractor.js");
-    const result = await extractAndStoreFacts("hi");
-    expect(result.disabled).toBe(false);
-    expect(result.factsStored).toBe(0);
-  });
-
-  it("extractAndStoreFacts gracefully handles AI provider failure", async () => {
-    // No API keys in test env — provider will fail, extractor should swallow
-    const { extractAndStoreFacts } = await import("../src/services/memory-extractor.js");
-    const result = await extractAndStoreFacts(
-      "This is a long enough conversation about Berlin, Postgres databases, " +
-      "and how to set up nginx properly. Should be more than 50 characters easily.",
-    );
-    expect(result.disabled).toBe(false);
-    expect(result).toHaveProperty("factsStored");
-    // Either succeeded or failed silently — but didn't throw
-  });
-});

package/test/memory-layers.test.ts

@@ -1,169 +0,0 @@
-/**
- * v4.11.0 — Layered memory loader.
- *
- * Replaces the monolithic MEMORY.md → System Prompt with a structured
- * 4-layer architecture (inspired by mempalace's L0–L3 stack):
- *
- *   L0 identity.md      — always loaded, ~200 tokens (who the user is)
- *   L1 preferences.md   — always loaded (how to communicate)
- *   L1 MEMORY.md        — backwards-compat: existing curated knowledge
- *   L2 projects/*.md    — loaded on topic match
- *   L3 daily logs       — only via vector search (existing embeddings.ts)
- *
- * If the new files don't exist, this falls back to the monolithic MEMORY.md
- * so existing setups keep working without migration.
- */
-import { describe, it, expect, beforeEach, vi } from "vitest";
-import fs from "fs";
-import os from "os";
-import { resolve } from "path";
-
-const TEST_DATA_DIR = resolve(os.tmpdir(), `alvin-mem-layers-${process.pid}-${Date.now()}`);
-
-beforeEach(() => {
-  if (fs.existsSync(TEST_DATA_DIR)) fs.rmSync(TEST_DATA_DIR, { recursive: true, force: true });
-  fs.mkdirSync(resolve(TEST_DATA_DIR, "memory"), { recursive: true });
-  process.env.ALVIN_DATA_DIR = TEST_DATA_DIR;
-  vi.resetModules();
-});
-
-describe("memory-layers (v4.11.0)", () => {
-  it("returns empty when nothing exists", async () => {
-    const { loadMemoryLayers } = await import("../src/services/memory-layers.js");
-    const layered = loadMemoryLayers();
-    expect(layered.identity).toBe("");
-    expect(layered.preferences).toBe("");
-    expect(layered.longTerm).toBe("");
-    expect(layered.projects).toEqual([]);
-  });
-
-  it("loads identity.md as L0 always", async () => {
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "identity.md"),
-      "# Identity\n\nName: Test User\nLocation: Berlin",
-    );
-    const { loadMemoryLayers } = await import("../src/services/memory-layers.js");
-    const layered = loadMemoryLayers();
-    expect(layered.identity).toMatch(/Test User/);
-  });
-
-  it("loads preferences.md as L1 always", async () => {
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "preferences.md"),
-      "- Reply in German\n- No 'Gerne' in responses",
-    );
-    const { loadMemoryLayers } = await import("../src/services/memory-layers.js");
-    const layered = loadMemoryLayers();
-    expect(layered.preferences).toMatch(/Reply in German/);
-  });
-
-  it("falls back to monolithic MEMORY.md when split files are missing", async () => {
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "MEMORY.md"),
-      "# Old monolithic\n\n- Some legacy fact",
-    );
-    const { loadMemoryLayers } = await import("../src/services/memory-layers.js");
-    const layered = loadMemoryLayers();
-    expect(layered.longTerm).toMatch(/legacy fact/);
-  });
-
-  it("loads projects/*.md and exposes them with their filename as topic", async () => {
-    fs.mkdirSync(resolve(TEST_DATA_DIR, "memory", "projects"), { recursive: true });
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "projects", "my-project.md"),
-      "# my-project\nVPS: 10.0.0.1, runs nginx + pm2",
-    );
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "projects", "homes.md"),
-      "# HOMES\nDB: homes_production (Postgres)",
-    );
-    const { loadMemoryLayers } = await import("../src/services/memory-layers.js");
-    const layered = loadMemoryLayers();
-    expect(layered.projects).toHaveLength(2);
-    const topics = layered.projects.map(p => p.topic).sort();
-    expect(topics).toEqual(["homes", "my-project"]);
-    expect(layered.projects.find(p => p.topic === "homes")?.content).toMatch(/homes_production/);
-  });
-
-  it("buildLayeredContext returns all L0+L1 plus matching L2 by topic keyword", async () => {
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "identity.md"),
-      "Name: User",
-    );
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "preferences.md"),
-      "Be terse.",
-    );
-    fs.mkdirSync(resolve(TEST_DATA_DIR, "memory", "projects"), { recursive: true });
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "projects", "homes.md"),
-      "HOMES uses Postgres",
-    );
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "projects", "my-project.md"),
-      "my-project uses MySQL",
-    );
-
-    const { buildLayeredContext } = await import("../src/services/memory-layers.js");
-
-    // Query mentions HOMES → only the homes project should be loaded
-    const ctx = buildLayeredContext("Tell me about HOMES backups");
-    expect(ctx).toMatch(/Name: User/); // L0
-    expect(ctx).toMatch(/Be terse/); // L1
-    expect(ctx).toMatch(/HOMES uses Postgres/); // L2 matched
-    expect(ctx).not.toMatch(/my-project uses MySQL/); // L2 not matched
-  });
-
-  it("buildLayeredContext without a query returns L0+L1 only (boot-up brief)", async () => {
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "identity.md"),
-      "Name: User",
-    );
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "preferences.md"),
-      "Be terse.",
-    );
-    fs.mkdirSync(resolve(TEST_DATA_DIR, "memory", "projects"), { recursive: true });
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "projects", "homes.md"),
-      "HOMES uses Postgres",
-    );
-
-    const { buildLayeredContext } = await import("../src/services/memory-layers.js");
-    const ctx = buildLayeredContext();
-    expect(ctx).toMatch(/Name: User/);
-    expect(ctx).not.toMatch(/Postgres/); // L2 only loaded with a query
-  });
-
-  it("token budget: layered context truncates long projects to fit budget", async () => {
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "identity.md"),
-      "Name: User",
-    );
-    fs.mkdirSync(resolve(TEST_DATA_DIR, "memory", "projects"), { recursive: true });
-    const longContent = "homes ".repeat(2000); // ~10000 chars
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "projects", "homes.md"),
-      longContent,
-    );
-    const { buildLayeredContext } = await import("../src/services/memory-layers.js");
-    const ctx = buildLayeredContext("HOMES");
-    // Total context should be capped (~6000 chars max for L0+L1+L2)
-    expect(ctx.length).toBeLessThan(8000);
-  });
-
-  it("monolithic MEMORY.md and split files coexist (split takes priority, mono is secondary)", async () => {
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "identity.md"),
-      "Name: User",
-    );
-    fs.writeFileSync(
-      resolve(TEST_DATA_DIR, "memory", "MEMORY.md"),
-      "# Legacy\n\n- Old fact still there",
-    );
-    const { buildLayeredContext } = await import("../src/services/memory-layers.js");
-    const ctx = buildLayeredContext("anything");
-    expect(ctx).toMatch(/Name: User/); // L0
-    expect(ctx).toMatch(/Old fact still there/); // legacy still included
-  });
-});