Files
rohitg00--agentmemory/test/embedding-dimensions.test.ts
Rohit Ghumare 6cc9b9f0fe fix: env hydration, indexing, consolidation lifecycle, connector activation, hardening (#1136)
* fix: env hydration, indexing, consolidation, connectors, hardening

- config: hydrate ~/.agentmemory/.env into process.env at boot so all modules see it
- search: shared indexRecords() so export-import and replay populate BM25 and vector (#1072)
- snapshot: wire the periodic timer (#1006), clamp non-positive intervals, add a reentrancy guard
- schema: CJK-aware jaccard dedup plus exact-match fallback for short memories
- embeddings: shared resolveDimensions() so openrouter stops hardcoding 1536 (#1002)
- viewer: buffer request bodies before decoding to fix multibyte corruption (#930)
- providers: retry 429/503 with Retry-After under a total-elapsed budget cap
- consolidation: fire on session stop (#1087), gate keyless installs, debounce the per-turn stop hook, drop the client-side double-fire
- evict: bound stale-session recovery to one consolidation pass
- api/patterns: bound session fan-out (#1100)
- connect: write a memory-usage guideline into each hook-less agent's native rules file (12 agents, doc-verified paths, --no-guidelines opt-out)
- graph: import graphify's graph.json via mem::graph::import-graphify + POST /agentmemory/graph/import-graphify; shared persistGraphDelta with endpoint remap so merged nodes never leave dangling or duplicate edges
- fs-watcher: stat roots before fs.watch so missing roots fail deterministically on Node 24+
- test: regression tests for every fix

* fix: address review findings on import, debounce, and connect paths

- guidelines: refuse to touch files with a lone or reversed marker pair
- export-import/replay: indexing after committed writes is best-effort,
  logged instead of failing the import; flatten the nested runChunked so
  replace-mode deletes stay bounded to one chunk
- graph: persist the snapshot when merge-only batches mutate cached
  topNodes/topEdges entries
- graph-import: async fs, typeof validation on path/cwd; REST handler
  whitelists the payload and 400s non-string values
- fetch: cancel discarded response bodies before retrying
- events: serialize the consolidation cooldown check so concurrent stops
  cannot both pass the read-check-write window
- evict: gate recovered-session consolidation on isConsolidationEnabled
  and mirror the stop path's force flag
- search: rebuild indexes per session chunk to bound peak memory
- test: regression coverage for each (malformed markers, concurrent
  stops, snapshot persistence, AMBIGUOUS/default mappings, env isolation)
2026-08-02 11:16:30 +01:00

102 lines
4.0 KiB
TypeScript

import { describe, it, expect, beforeEach, afterEach } from "vitest";
import { resolveDimensions } from "../src/providers/embedding/_dimensions.js";
import { OpenRouterEmbeddingProvider } from "../src/providers/embedding/openrouter.js";
import { OpenAIEmbeddingProvider } from "../src/providers/embedding/openai.js";
describe("resolveDimensions", () => {
const ENV = "OPENROUTER_EMBEDDING_DIMENSIONS";
it("resolves namespaced OpenRouter model ids to their real dimensions", () => {
expect(resolveDimensions("openai/text-embedding-3-large", undefined, ENV)).toBe(3072);
expect(resolveDimensions("openai/text-embedding-3-small", undefined, ENV)).toBe(1536);
expect(resolveDimensions("openai/text-embedding-ada-002", undefined, ENV)).toBe(1536);
});
it("resolves bare model ids to their real dimensions", () => {
expect(resolveDimensions("text-embedding-3-large", undefined, ENV)).toBe(3072);
expect(resolveDimensions("text-embedding-3-small", undefined, ENV)).toBe(1536);
expect(resolveDimensions("text-embedding-ada-002", undefined, ENV)).toBe(1536);
});
it("lets a valid override win over the model-derived dimensions", () => {
expect(resolveDimensions("openai/text-embedding-3-large", "1024", ENV)).toBe(1024);
expect(resolveDimensions("text-embedding-3-small", "768", ENV)).toBe(768);
});
it("throws with the given env name on invalid override values", () => {
for (const bad of ["abc", "0", "-5"]) {
expect(() => resolveDimensions("text-embedding-3-large", bad, ENV)).toThrow(
new RegExp(`${ENV} must be a positive integer, got: ${bad}`),
);
}
});
it("uses the supplied env name in the error message", () => {
expect(() => resolveDimensions("text-embedding-3-large", "abc", "OPENAI_EMBEDDING_DIMENSIONS")).toThrow(
/OPENAI_EMBEDDING_DIMENSIONS must be a positive integer, got: abc/,
);
});
it("falls back to the default (1536) for unknown models", () => {
expect(resolveDimensions("mystery-self-hosted-model", undefined, ENV)).toBe(1536);
expect(resolveDimensions("someprovider/unknown-model", undefined, ENV)).toBe(1536);
});
});
describe("OpenRouterEmbeddingProvider dimension regression", () => {
const originalEnv = { ...process.env };
beforeEach(() => {
process.env = { ...originalEnv };
delete process.env["OPENROUTER_EMBEDDING_MODEL"];
delete process.env["OPENROUTER_EMBEDDING_DIMENSIONS"];
});
afterEach(() => {
process.env = originalEnv;
});
it("reports 3072 for openai/text-embedding-3-large with no override (guard would throw on the old hardcoded 1536)", () => {
process.env["OPENROUTER_EMBEDDING_MODEL"] = "openai/text-embedding-3-large";
const provider = new OpenRouterEmbeddingProvider("test-key");
expect(provider.dimensions).toBe(3072);
});
it("defaults to 1536 for openai/text-embedding-3-small", () => {
const provider = new OpenRouterEmbeddingProvider("test-key");
expect(provider.dimensions).toBe(1536);
});
it("lets OPENROUTER_EMBEDDING_DIMENSIONS override the model-derived dimensions", () => {
process.env["OPENROUTER_EMBEDDING_MODEL"] = "openai/text-embedding-3-large";
process.env["OPENROUTER_EMBEDDING_DIMENSIONS"] = "1024";
const provider = new OpenRouterEmbeddingProvider("test-key");
expect(provider.dimensions).toBe(1024);
});
});
describe("OpenAIEmbeddingProvider defaults unchanged", () => {
const originalEnv = { ...process.env };
beforeEach(() => {
process.env = { ...originalEnv };
delete process.env["OPENAI_EMBEDDING_MODEL"];
delete process.env["OPENAI_EMBEDDING_DIMENSIONS"];
});
afterEach(() => {
process.env = originalEnv;
});
it("defaults to 1536 for text-embedding-3-small", () => {
const provider = new OpenAIEmbeddingProvider("test-key");
expect(provider.dimensions).toBe(1536);
});
it("reports 3072 for text-embedding-3-large", () => {
process.env["OPENAI_EMBEDDING_MODEL"] = "text-embedding-3-large";
const provider = new OpenAIEmbeddingProvider("test-key");
expect(provider.dimensions).toBe(3072);
});
});