9c82d2aa7d
* chore(release): v0.9.29 with project-scope parity across surfaces Version trio + plugin manifests + supportedVersions + ExportData union bumped to 0.9.29; CHANGELOG entry covering everything since v0.9.28 with upgrade notes for the four visible behavior changes. Fixes the endpoint-count drift on main (130 registered routes vs docs saying 129 after #1132 landed in parallel with #1136). Project-scope parity: OpenCode plugin, Hermes plugin, Pi extension, and JSONL replay now resolve project the same way the hooks do (env override, git toplevel basename, cwd basename) instead of sending raw filesystem paths, closing #903 and #1135 and pre-empting the same bug in pi. The filesystem watcher accepts AGENTMEMORY_PROJECT_NAME with the old AGENTMEMORY_PROJECT kept as a deprecated alias, replay handles Windows-recorded paths, and OpenCode file enrichment matches the agent's lowercase tool names (the capitalized set never matched). Tests: opencode fallback expectations updated to basenames per the canonicalization, git-toplevel resolution covered with a fixture repo, new project-scope-parity suite for replay and fs-watcher. * fix(release): review findings, git-toplevel parity, doc counts - skills generator dedupes routes on method plus path, so the REST reference lists all 130 registered routes instead of hiding the second method on ten dual-method paths (header said 119) - fs-watcher trims AGENTMEMORY_PROJECT_NAME and the deprecated alias, treating whitespace as unset, and derives the git toplevel basename when watching a subdirectory - replay resolves the git toplevel basename when the recorded cwd still exists locally (memoized per cwd), keeping the basename fallback for historical or cross-platform paths; no env override here since a bulk import spans many projects - parity tests for replay git-root resolution, watcher git-root and trim behavior - stat-tests badge updated from 1428+ to 1550+ passing * fix(cli): refuse second-instance boot over a live daemon Closes the class behind issue 1140: agentmemory consolidate (or any unrecognized word) fell through the command table into the full server boot, registering a duplicate worker on the running engine; on iii 0.11.2 the second instance's shutdown tears down the daemon's HTTP trigger routing until a full engine restart. Unknown subcommands now error with the supported list, and main() probes livez on the resolved port and refuses to boot over a live daemon, so multi-instance setups on other ports are unaffected. Verified behaviorally against the built CLI: both paths refuse with exit 1. Also from review: the watcher stamps each event with its own root's project via a per-root map (an explicit config.project still overrides for every root), and replay only accepts a non-empty string cwd from parsed JSONL so malformed entries cannot reach the filesystem probe. * test(watcher): two-repository flush events scope to their own project * chore(release): bump packages/mcp, guard it, refresh CONTRIBUTING packages/mcp was still 0.9.28 after the release bump because nothing guarded it; a consistency test now pins it to package.json. CONTRIBUTING release list corrected to the files a bump actually touches (no tracked lockfile, the two extra plugin manifests, the export test derives from VERSION now), and the subsystems table gains src/cli, integrations/pi, and the generated-manifest note. * fix(export): refuse over-frame export instead of dropping the worker Closes the availability bug in issue 1142: GET /agentmemory/export assembles the full store and returns it through sdk.trigger, so a store whose serialized export passes the engine's 16 MiB WebSocket frame (tungstenite max_frame_size, not raisable under the 0.11.2 pin) dies on the worker->engine hop, drops the worker, and 404s every endpoint for ~1s. The session collections page on maxSessions/offset but ~18 others do not, so a large store hits this at any parameter combination. A shared frame-guard measures the serialized size before returning: mem::export returns a small oversized error instead of the giant object, and api::mesh-export returns 413 (same dead-end as #890). Either way the over-frame payload never crosses the boundary, so the daemon stays up and the failure is one clean request with a hint to narrow the range. Full pagination of the non-session collections is a follow-up. Layer 1 of the fix; verified with a synthetic oversized export returning the error object (tiny) rather than the payload. * ci: collapse to a single npm install to fix Node 24/26 CI The two-step install (npm install --package-lock-only then npm ci) failed only on the Node 24/26 matrix rows: their stricter npm rejects rolldown's optional platform bindings (@rolldown/binding-android-arm64) that a --package-lock-only pass does not fully enumerate. Lockfiles are gitignored, so npm ci re-validation buys no reproducibility here. A single lenient npm install resolves and installs in one pass. * fix(mesh): scope exported memories by project like actions api::mesh-export filtered actions by ?project but returned every project's memories. On a mesh instance federating one project to a peer, the peer pulled other projects' memories (cross-project leak), and those extras could push the payload past the 16 MiB transport frame into a 413 even when the requested project's own slice fit. Memories carry the same optional project field as actions, so filter both before the frame-size guard runs. Adds a regression test asserting a project-scoped export excludes other projects' memories and that an oversized memory in another project no longer 413s the scoped request. * chore(release): credit the Antigravity native hooks adapter in 0.9.29 notes * chore(release): sweep stale 0.9.28 refs for 0.9.29 Deploy Dockerfiles/compose/render pins, AGENTS.md stats header, opencode plugin manifest, website meta snapshot, test-count claims (1,428 -> 1,596) in README/AGENTS/stat SVGs, and the missing 0.9.29 CHANGELOG compare link. * chore(release): sync stat-tests badge to 1596+ and commit bridge exec bit * refactor: trim frame-guard comments and drop issue refs from code
130 lines
4.2 KiB
TypeScript
130 lines
4.2 KiB
TypeScript
import { describe, it, expect, vi } from "vitest";
|
|
|
|
vi.mock("../src/logger.js", () => ({
|
|
logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() },
|
|
}));
|
|
|
|
import { registerApiTriggers } from "../src/triggers/api.js";
|
|
import { KV } from "../src/state/schema.js";
|
|
import { SAFE_PAYLOAD_BYTES } from "../src/state/frame-guard.js";
|
|
import type { Memory } from "../src/types.js";
|
|
|
|
// A project-scoped mesh export must filter memories like actions: unscoped
|
|
// memories leak across projects and can push the payload past the transport
|
|
// frame limit even when the requested project's own slice fits.
|
|
|
|
const SECRET = "mesh-test-secret";
|
|
|
|
function mockKV(store = new Map<string, Map<string, unknown>>()) {
|
|
return {
|
|
get: async () => null,
|
|
set: async <T>(s: string, k: string, d: T) => {
|
|
if (!store.has(s)) store.set(s, new Map());
|
|
store.get(s)!.set(k, d);
|
|
return d;
|
|
},
|
|
delete: async () => {},
|
|
update: async () => {},
|
|
list: async <T>(scope: string): Promise<T[]> =>
|
|
Array.from(store.get(scope)?.values() ?? []) as T[],
|
|
_store: store,
|
|
};
|
|
}
|
|
|
|
function mockSdk() {
|
|
const fns = new Map<string, Function>();
|
|
return {
|
|
registerFunction: (id: string, h: Function) => fns.set(id, h),
|
|
registerTrigger: () => {},
|
|
trigger: async (input: { function_id: string; payload?: unknown }) =>
|
|
fns.get(input.function_id)?.(input.payload),
|
|
_fns: fns,
|
|
};
|
|
}
|
|
|
|
function memory(id: string, project: string, content = "x"): Memory {
|
|
return {
|
|
id,
|
|
type: "pattern",
|
|
title: id,
|
|
content,
|
|
createdAt: "2026-08-01T00:00:00Z",
|
|
updatedAt: "2026-08-01T00:00:00Z",
|
|
concepts: [],
|
|
files: [],
|
|
sessionIds: [],
|
|
strength: 5,
|
|
version: 1,
|
|
isLatest: true,
|
|
project,
|
|
};
|
|
}
|
|
|
|
async function meshExport(
|
|
sdk: ReturnType<typeof mockSdk>,
|
|
project?: string,
|
|
): Promise<{ status_code: number; body: Record<string, unknown> }> {
|
|
const handler = sdk._fns.get("api::mesh-export")!;
|
|
return handler({
|
|
headers: { authorization: `Bearer ${SECRET}` },
|
|
query_params: project ? { project } : {},
|
|
});
|
|
}
|
|
|
|
describe("api::mesh-export project scoping", () => {
|
|
it("excludes other projects' memories from a project-scoped export", async () => {
|
|
const kv = mockKV();
|
|
await kv.set(KV.memories, "m-alpha", memory("m-alpha", "alpha"));
|
|
await kv.set(KV.memories, "m-beta", memory("m-beta", "beta"));
|
|
const sdk = mockSdk();
|
|
registerApiTriggers(sdk as never, kv as never, SECRET);
|
|
|
|
const res = await meshExport(sdk, "alpha");
|
|
|
|
expect(res.status_code).toBe(200);
|
|
const memories = res.body.memories as Memory[];
|
|
expect(memories.map((m) => m.id)).toEqual(["m-alpha"]);
|
|
expect(memories.some((m) => m.project === "beta")).toBe(false);
|
|
});
|
|
|
|
it("returns all memories when no project is provided", async () => {
|
|
const kv = mockKV();
|
|
await kv.set(KV.memories, "m-alpha", memory("m-alpha", "alpha"));
|
|
await kv.set(KV.memories, "m-beta", memory("m-beta", "beta"));
|
|
const sdk = mockSdk();
|
|
registerApiTriggers(sdk as never, kv as never, SECRET);
|
|
|
|
const res = await meshExport(sdk);
|
|
|
|
expect(res.status_code).toBe(200);
|
|
const memories = res.body.memories as Memory[];
|
|
expect(memories.map((m) => m.id).sort()).toEqual(["m-alpha", "m-beta"]);
|
|
});
|
|
|
|
it("avoids the 413 when only another project's memory is oversized", async () => {
|
|
const kv = mockKV();
|
|
// A single beta memory alone blows the frame; alpha's slice is tiny.
|
|
await kv.set(
|
|
KV.memories,
|
|
"m-beta-huge",
|
|
memory("m-beta-huge", "beta", "z".repeat(SAFE_PAYLOAD_BYTES + 4096)),
|
|
);
|
|
await kv.set(KV.memories, "m-alpha", memory("m-alpha", "alpha"));
|
|
const sdk = mockSdk();
|
|
registerApiTriggers(sdk as never, kv as never, SECRET);
|
|
|
|
// Scoped to alpha: the huge beta memory is filtered out before the frame
|
|
// guard runs, so the request succeeds instead of 413-ing.
|
|
const scoped = await meshExport(sdk, "alpha");
|
|
expect(scoped.status_code).toBe(200);
|
|
expect((scoped.body.memories as Memory[]).map((m) => m.id)).toEqual([
|
|
"m-alpha",
|
|
]);
|
|
|
|
// Unscoped: the oversized memory is included, so the guard fires (413).
|
|
const unscoped = await meshExport(sdk);
|
|
expect(unscoped.status_code).toBe(413);
|
|
expect((unscoped.body as { oversized?: boolean }).oversized).toBe(true);
|
|
});
|
|
});
|