6cc9b9f0fe
* 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)
102 lines
4.0 KiB
TypeScript
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);
|
|
});
|
|
});
|