recallnest 1.4.1 → 2.2.1

package/src/__tests__/tool-output-compressor.test.ts

@@ -1,133 +0,0 @@
-import { describe, it, expect } from "bun:test";
-import { compressToolOutput } from "../tool-output-compressor.js";
-
-describe("compressToolOutput", () => {
-  // ========================================================================
-  // Base64 replacement
-  // ========================================================================
-
-  it("replaces large base64 blocks with size placeholder", () => {
-    const base64 = "A".repeat(2000);
-    const text = `Here is a screenshot:\n${base64}\nEnd of screenshot.`;
-    const result = compressToolOutput(text);
-    expect(result).toContain("[image:");
-    expect(result).toContain("base64");
-    expect(result).not.toContain("AAAA");
-  });
-
-  it("preserves short base64-like strings", () => {
-    const text = "The hash is abc123==";
-    expect(compressToolOutput(text)).toBe(text);
-  });
-
-  // ========================================================================
-  // Git output compression
-  // ========================================================================
-
-  it("compresses git push boilerplate", () => {
-    const text = `$ git push origin main
-Enumerating objects: 5, done.
-Counting objects: 100% (5/5), done.
-Delta compression using up to 8 threads
-Compressing objects: 100% (3/3), done.
-Writing objects: 100% (3/3), 450 bytes | 450.00 KiB/s, done.
-Total 3 (delta 2), reused 0 (delta 0)
-remote: Resolving deltas: 100% (2/2), completed with 2 local objects.
-To github.com:user/repo.git
-   abc1234..def5678  main -> main
-
-Next I'll check the status.`;
-
-    const result = compressToolOutput(text);
-    expect(result).toContain("[git: ok");
-    expect(result).toContain("main");
-    expect(result).not.toContain("Enumerating objects");
-    expect(result).toContain("Next I'll check the status");
-  });
-
-  it("preserves git push failures", () => {
-    const text = `$ git push
-Enumerating objects: 5, done.
-error: failed to push some refs to 'origin'
-hint: Updates were rejected because the tip of your branch is behind -> main`;
-
-    const result = compressToolOutput(text);
-    expect(result).toContain("error");
-    expect(result).toContain("rejected");
-  });
-
-  // ========================================================================
-  // Test output compression
-  // ========================================================================
-
-  it("compresses passing test summary", () => {
-    const text = `$ bun test
-running 15 tests
-test a ... ok
-test b ... ok
-test c ... ok
-test result: ok. 15 passed; 0 failed; 0 ignored
-
-All tests passed.`;
-
-    const result = compressToolOutput(text);
-    expect(result.length).toBeLessThanOrEqual(text.length);
-    expect(result).toContain("All tests passed");
-  });
-
-  it("preserves failing test output", () => {
-    const text = `$ npm test
-Tests: 2 failed, 8 passed
-FAIL src/main.test.ts
-  ✕ should handle edge case`;
-
-    const result = compressToolOutput(text);
-    expect(result).toContain("failed");
-    expect(result).toContain("FAIL");
-  });
-
-  // ========================================================================
-  // Safety: user/AI content untouched
-  // ========================================================================
-
-  it("never modifies plain conversation text", () => {
-    const text = "User: How do I fix this git push error?\n\nAssistant: You need to pull first.";
-    expect(compressToolOutput(text)).toBe(text);
-  });
-
-  it("preserves AI reasoning about tool results", () => {
-    const text = "The test result shows 15 passed, which means our fix works.";
-    expect(compressToolOutput(text)).toBe(text);
-  });
-
-  // ========================================================================
-  // Edge cases
-  // ========================================================================
-
-  it("handles empty input", () => {
-    expect(compressToolOutput("")).toBe("");
-  });
-
-  it("handles short input", () => {
-    expect(compressToolOutput("hi")).toBe("hi");
-  });
-
-  it("handles text without tool outputs", () => {
-    const text = "This is a normal conversation.\nNo tool outputs here.";
-    expect(compressToolOutput(text)).toBe(text);
-  });
-
-  // ========================================================================
-  // Large output truncation
-  // ========================================================================
-
-  it("truncates large unmatched tool output", () => {
-    // Use content with spaces/special chars to avoid base64 regex match
-    const largeOutput = Array.from({ length: 150 }, (_, i) => `line ${i}: some detailed output here`).join("\n");
-    const text = `$ some-command\n${largeOutput}\n\nNext step.`;
-    const result = compressToolOutput(text);
-    expect(result).toContain("[...");
-    expect(result).toContain("chars truncated");
-    expect(result.length).toBeLessThan(text.length);
-  });
-});

package/src/__tests__/version-manager.test.ts

@@ -1,213 +0,0 @@
-/**
- * Tests for Tier 3.3: Version Coexistence + Competition
- *
- * Validates:
- * 1. computeVersionRank formula
- * 2. createVersionGroup tags both entries with same group
- * 3. deduplicateByVersionGroup keeps top-ranked per group
- * 4. Entries without version_group pass through unchanged
- * 5. Multiple version groups are handled independently
- * 6. Existing version_group is reused (not duplicated)
- */
-import { describe, expect, it } from "bun:test";
-import {
-  computeVersionRank,
-  deduplicateByVersionGroup,
-  createVersionGroup,
-} from "../version-manager.js";
-import type { MemoryEntry } from "../store.js";
-
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-
-function makeEntry(
-  id: string,
-  overrides: {
-    confidence?: number;
-    accessCount?: number;
-    version_group?: string;
-    version_rank?: number;
-  } = {},
-): MemoryEntry {
-  const meta: Record<string, unknown> = {
-    confidence: overrides.confidence ?? 0.7,
-    accessCount: overrides.accessCount ?? 0,
-  };
-  if (overrides.version_group) meta.version_group = overrides.version_group;
-  if (overrides.version_rank !== undefined) meta.version_rank = overrides.version_rank;
-
-  return {
-    id,
-    text: `memory-${id}`,
-    vector: [1, 0, 0],
-    category: "events",
-    scope: "project:test",
-    importance: 0.7,
-    timestamp: Date.now(),
-    metadata: JSON.stringify(meta),
-  };
-}
-
-function createMockStore(entries: MemoryEntry[]) {
-  const data = new Map(entries.map(e => [e.id, { ...e }]));
-  const updates: Array<{ id: string; metadata: string }> = [];
-
-  return {
-    data,
-    updates,
-    async getById(id: string) {
-      return data.get(id) ?? null;
-    },
-    async update(id: string, upd: { metadata?: string }, _scope?: string[]) {
-      const entry = data.get(id);
-      if (!entry) return null;
-      if (upd.metadata) {
-        entry.metadata = upd.metadata;
-        updates.push({ id, metadata: upd.metadata });
-      }
-      return entry;
-    },
-  };
-}
-
-// ---------------------------------------------------------------------------
-// computeVersionRank
-// ---------------------------------------------------------------------------
-
-describe("computeVersionRank", () => {
-  it("computes rank from importance, confidence, and accessCount", () => {
-    const entry = makeEntry("a", { confidence: 1.0, accessCount: 10 });
-    const rank = computeVersionRank(entry);
-    // quality = 0.5*0.7 + 0.5*1.0 = 0.85 (importance=0.7 from makeEntry)
-    // rank = 0.85 * (1 + ln(11)) ≈ 0.85 * 3.398 = 2.888
-    expect(rank).toBeGreaterThan(2.8);
-    expect(rank).toBeLessThan(3.0);
-  });
-
-  it("returns baseline for default confidence and zero access", () => {
-    const entry = makeEntry("b", { confidence: 0.7, accessCount: 0 });
-    const rank = computeVersionRank(entry);
-    // quality = 0.5*0.7 + 0.5*0.7 = 0.7
-    // rank = 0.7 * (1 + 0) = 0.7
-    expect(rank).toBeCloseTo(0.7, 5);
-  });
-
-  it("higher confidence wins over lower with same access", () => {
-    const high = computeVersionRank(makeEntry("h", { confidence: 1.0, accessCount: 5 }));
-    const low = computeVersionRank(makeEntry("l", { confidence: 0.3, accessCount: 5 }));
-    expect(high).toBeGreaterThan(low);
-  });
-
-  it("higher access wins over lower with same confidence", () => {
-    const frequent = computeVersionRank(makeEntry("f", { confidence: 0.7, accessCount: 20 }));
-    const rare = computeVersionRank(makeEntry("r", { confidence: 0.7, accessCount: 1 }));
-    expect(frequent).toBeGreaterThan(rare);
-  });
-});
-
-// ---------------------------------------------------------------------------
-// createVersionGroup
-// ---------------------------------------------------------------------------
-
-describe("createVersionGroup", () => {
-  it("tags both entries with the same version_group", async () => {
-    const canonical = makeEntry("canon", { confidence: 1.0, accessCount: 5 });
-    const member = makeEntry("member", { confidence: 0.5, accessCount: 1 });
-    const store = createMockStore([canonical, member]);
-
-    const groupId = await createVersionGroup(store as any, canonical, member, "project:test");
-
-    expect(groupId).toBeTruthy();
-    expect(store.updates.length).toBe(2);
-
-    const canonMeta = JSON.parse(store.data.get("canon")!.metadata);
-    const memberMeta = JSON.parse(store.data.get("member")!.metadata);
-
-    expect(canonMeta.version_group).toBe(groupId);
-    expect(memberMeta.version_group).toBe(groupId);
-    expect(canonMeta.version_rank).toBeGreaterThan(memberMeta.version_rank);
-  });
-
-  it("reuses existing version_group from canonical", async () => {
-    const canonical = makeEntry("canon", {
-      confidence: 1.0,
-      accessCount: 5,
-      version_group: "vg-existing",
-      version_rank: 3.0,
-    });
-    const member = makeEntry("new", { confidence: 0.5, accessCount: 0 });
-    const store = createMockStore([canonical, member]);
-
-    const groupId = await createVersionGroup(store as any, canonical, member, "project:test");
-
-    expect(groupId).toBe("vg-existing");
-    const memberMeta = JSON.parse(store.data.get("new")!.metadata);
-    expect(memberMeta.version_group).toBe("vg-existing");
-  });
-});
-
-// ---------------------------------------------------------------------------
-// deduplicateByVersionGroup
-// ---------------------------------------------------------------------------
-
-describe("deduplicateByVersionGroup", () => {
-  it("keeps only the top-ranked entry per version group", () => {
-    const results = [
-      { entry: makeEntry("a", { version_group: "vg1", version_rank: 3.0 }), score: 0.9, sources: {} },
-      { entry: makeEntry("b", { version_group: "vg1", version_rank: 1.0 }), score: 0.85, sources: {} },
-      { entry: makeEntry("c"), score: 0.8, sources: {} },
-    ];
-
-    const deduped = deduplicateByVersionGroup(results);
-
-    expect(deduped.length).toBe(2);
-    expect(deduped.map(r => r.entry.id)).toContain("a");
-    expect(deduped.map(r => r.entry.id)).toContain("c");
-    expect(deduped.map(r => r.entry.id)).not.toContain("b");
-  });
-
-  it("passes through entries without version_group", () => {
-    const results = [
-      { entry: makeEntry("x"), score: 0.9, sources: {} },
-      { entry: makeEntry("y"), score: 0.8, sources: {} },
-    ];
-
-    const deduped = deduplicateByVersionGroup(results);
-
-    expect(deduped.length).toBe(2);
-  });
-
-  it("handles multiple independent version groups", () => {
-    const results = [
-      { entry: makeEntry("a1", { version_group: "vg1", version_rank: 3.0 }), score: 0.95, sources: {} },
-      { entry: makeEntry("a2", { version_group: "vg1", version_rank: 1.0 }), score: 0.90, sources: {} },
-      { entry: makeEntry("b1", { version_group: "vg2", version_rank: 2.0 }), score: 0.85, sources: {} },
-      { entry: makeEntry("b2", { version_group: "vg2", version_rank: 4.0 }), score: 0.80, sources: {} },
-      { entry: makeEntry("solo"), score: 0.75, sources: {} },
-    ];
-
-    const deduped = deduplicateByVersionGroup(results);
-
-    expect(deduped.length).toBe(3);
-    const ids = deduped.map(r => r.entry.id);
-    expect(ids).toContain("a1"); // vg1 winner (rank 3.0)
-    expect(ids).toContain("b2"); // vg2 winner (rank 4.0)
-    expect(ids).toContain("solo");
-  });
-
-  it("handles empty array", () => {
-    expect(deduplicateByVersionGroup([])).toEqual([]);
-  });
-
-  it("handles single entry in a version group", () => {
-    const results = [
-      { entry: makeEntry("only", { version_group: "vg1", version_rank: 2.0 }), score: 0.9, sources: {} },
-    ];
-
-    const deduped = deduplicateByVersionGroup(results);
-
-    expect(deduped.length).toBe(1);
-    expect(deduped[0].entry.id).toBe("only");
-  });
-});