@electric-ax/agents-server-conformance-tests 0.1.10 → 0.1.12

package/dist/index.cjs

@@ -328,11 +328,8 @@ var ElectricAgentsScenario = class {
 		});
 		return this;
 	}
-	readStream(stream) {
-		this.steps.push({
-			kind: `readStream`,
-			stream
-		});
+	readStream() {
+		this.steps.push({ kind: `readStream` });
 		return this;
 	}
 	list(filter) {
@@ -624,7 +621,7 @@ async function executeStep(ctx, step) {
 				method: `POST`,
 				body: JSON.stringify(body)
 			});
-			(0, vitest.expect)(res.status).toBe(204);
+			(0, vitest.expect)([200, 204]).toContain(res.status);
 			ctx.history.push({
 				type: `message_sent`,
 				entityUrl,
@@ -671,7 +668,6 @@ async function executeStep(ctx, step) {
 			(0, vitest.expect)(entity, `Webhook payload must contain entity context`).toBeDefined();
 			(0, vitest.expect)(entity.url).toBeTruthy();
 			(0, vitest.expect)(entity.streams.main).toBeTruthy();
-			(0, vitest.expect)(entity.streams.error).toBeTruthy();
 			if (step.checks?.url) (0, vitest.expect)(entity.url).toBe(step.checks.url);
 			if (step.checks?.type) (0, vitest.expect)(entity.type).toBe(step.checks.type);
 			if (step.checks?.status) (0, vitest.expect)(entity.status).toBe(step.checks.status);
@@ -712,7 +708,7 @@ async function executeStep(ctx, step) {
 		}
 		case `readStream`: {
 			if (!ctx.currentEntityStreams) throw new Error(`No current entity streams`);
-			const streamPath = step.stream === `error` ? ctx.currentEntityStreams.error : ctx.currentEntityStreams.main;
+			const streamPath = ctx.currentEntityStreams.main;
 			const res = await fetch(appendPathToUrl(ctx.baseUrl, `${streamPath}?offset=0000000000000000_0000000000000000`));
 			if (res.status === 200) {
 				const text = await res.text();
@@ -1207,14 +1203,11 @@ function checkEntityContextOnWebhook(history) {
 	}
 }
 /**
-* Spec S5 — Structural: stream paths must match {entity.url}/main and {entity.url}/error.
+* Spec S5 — Structural: stream path must match {entity.url}/main.
 * Soundness: Sound | Completeness: Complete (within trace)
 */
 function checkStreamPathsMatchEntityUrl(history) {
-	for (const event of history) if (event.type === `entity_spawned`) {
-		(0, vitest.expect)(event.streams.main).toBe(`${event.entityUrl}/main`);
-		(0, vitest.expect)(event.streams.error).toBe(`${event.entityUrl}/error`);
-	}
+	for (const event of history) if (event.type === `entity_spawned`) (0, vitest.expect)(event.streams.main).toBe(`${event.entityUrl}/main`);
 }
 /**
 * Spec S4 — Safety: entity status transitions must be valid.
@@ -1276,8 +1269,8 @@ function checkAdditiveSchemaEvolution(history) {
 }
 /**
 * Spec R4 — Registry stream consistency: every entity_spawned must refer to an
-* entity that has not already been killed, and stream URLs must follow the
-* convention {entity.url}/main and {entity.url}/error.
+* entity that has not already been killed, and stream URL must follow the
+* convention {entity.url}/main.
 * Soundness: Sound | Completeness: Complete (within trace)
 */
 function checkRegistryStreamConsistency(history) {
@@ -1287,7 +1280,6 @@ function checkRegistryStreamConsistency(history) {
 		if (event.type === `entity_spawned`) {
 			(0, vitest.expect)(!killedEntities.has(event.entityUrl), `Registry consistency: entity_spawned for ${event.entityUrl} but entity was already killed`).toBe(true);
 			(0, vitest.expect)(event.streams.main, `Registry consistency: entity ${event.entityUrl} streams.main must be ${event.entityUrl}/main`).toBe(`${event.entityUrl}/main`);
-			(0, vitest.expect)(event.streams.error, `Registry consistency: entity ${event.entityUrl} streams.error must be ${event.entityUrl}/error`).toBe(`${event.entityUrl}/error`);
 		}
 	}
 }
@@ -1832,8 +1824,6 @@ function runElectricAgentsConformanceTests(config) {
 		}).spawn(`spawn-test-agent`, `entity-1`).expectStatus(`running`).custom(async (ctx) => {
 			const mainRes = await fetch(appendPathToUrl(ctx.baseUrl, ctx.currentEntityStreams.main), { method: `HEAD` });
 			(0, vitest.expect)(mainRes.status).toBe(200);
-			const errorRes = await fetch(appendPathToUrl(ctx.baseUrl, ctx.currentEntityStreams.error), { method: `HEAD` });
-			(0, vitest.expect)(errorRes.status).toBe(200);
 		}).run());
 		(0, vitest.test)(`spawn at unregistered type returns UNKNOWN_ENTITY_TYPE`, () => electricAgents(config.baseUrl).custom(async (ctx) => {
 			const res = await fetch(appendPathToUrl(ctx.baseUrl, routeControlPlanePath(`/unregistered/entity-1`)), {
@@ -1906,7 +1896,7 @@ function runElectricAgentsConformanceTests(config) {
 				method: `POST`,
 				body: JSON.stringify({ payload: { hello: true } })
 			});
-			(0, vitest.expect)(res.status).toBe(204);
+			(0, vitest.expect)([200, 204]).toContain(res.status);
 		}).expectWebhook().respondDone().run());
 	});
 	(0, vitest.describe)(`Electric Agents List`, () => {
@@ -2520,7 +2510,7 @@ function runElectricAgentsConformanceTests(config) {
 						payload: { action: `test` }
 					})
 				});
-				(0, vitest.expect)(res.status).toBe(204);
+				(0, vitest.expect)([200, 204]).toContain(res.status);
 			}).run();
 		});
 	});
@@ -2649,7 +2639,7 @@ function runElectricAgentsConformanceTests(config) {
 										targetIdx
 									} })
 								});
-								(0, vitest.expect)(res.status).toBe(204);
+								(0, vitest.expect)([200, 204]).toContain(res.status);
 								ctx.history.push({
 									type: `message_sent`,
 									entityUrl: url,
@@ -2926,12 +2916,15 @@ function runElectricAgentsConformanceTests(config) {
 			const sandbox = await makeSandbox(`/tmp`);
 			try {
 				const tool = createBashTool(sandbox);
-				const result = await tool.execute(`test-tc`, { command: `sleep 60` });
+				const result = await tool.execute(`test-tc`, {
+					command: `sleep 60`,
+					timeoutMs: 1e3
+				});
 				(0, vitest.expect)(result.details.timedOut).toBe(true);
 			} finally {
 				await sandbox.dispose();
 			}
-		}, 35e3);
+		}, 5e3);
 		(0, vitest.test)(`read_file rejects paths outside working directory`, async () => {
 			const { createReadFileTool } = await import(`../../agents-runtime/src/tools`);
 			const fs = await import(`node:fs/promises`);
@@ -3140,7 +3133,7 @@ function runElectricAgentsConformanceTests(config) {
 				(0, vitest.expect)(res.status).toBe(404);
 			}).run();
 		});
-		(0, vitest.test)(`tag update on stopped entity is rejected without claim`, () => {
+		(0, vitest.test)(`tag update on stopped entity is rejected as not running`, () => {
 			const id = Date.now();
 			return electricAgents(config.baseUrl).subscription(`/meta-stopped-agent-${id}/**`, `meta-stopped-sub-${id}`).registerType({
 				name: `meta-stopped-agent-${id}`,
@@ -3151,7 +3144,7 @@ function runElectricAgentsConformanceTests(config) {
 					method: `POST`,
 					body: JSON.stringify({ value: `value` })
 				});
-				(0, vitest.expect)(res.status).toBe(401);
+				(0, vitest.expect)(res.status).toBe(409);
 			}).run();
 		});
 	});
@@ -3191,7 +3184,7 @@ function runElectricAgentsConformanceTests(config) {
 						type: `farewell`
 					})
 				});
-				(0, vitest.expect)(res.status).toBe(204);
+				(0, vitest.expect)([200, 204]).toContain(res.status);
 			}).run();
 		});
 		(0, vitest.test)(`amend schemas on deleted type returns 404`, async () => {
@@ -3325,11 +3318,11 @@ function runElectricAgentsConformanceTests(config) {
 				(0, vitest.expect)(ctx.currentWriteToken).toBeNull();
 			}).run();
 		});
-		(0, vitest.test)(`tag update without token is rejected`, () => {
+		(0, vitest.test)(`tag update without legacy entity write token is authorized by principal permission`, () => {
 			const id = Date.now();
 			return electricAgents(config.baseUrl).subscription(`/auth-meta-notoken-agent-${id}/**`, `auth-meta-notoken-sub-${id}`).registerType({
 				name: `auth-meta-notoken-agent-${id}`,
-				description: `Test tag update without token`,
+				description: `Test tag update without legacy entity write token`,
 				creation_schema: { type: `object` }
 			}).spawn(`auth-meta-notoken-agent-${id}`, `entity-1`).custom(async (ctx) => {
 				const res = await fetch(appendPathToUrl(ctx.baseUrl, routeControlPlanePath(`${ctx.currentEntityUrl}/tags/key`)), {
@@ -3337,7 +3330,7 @@ function runElectricAgentsConformanceTests(config) {
 					headers: { "content-type": `application/json` },
 					body: JSON.stringify({ value: `value` })
 				});
-				(0, vitest.expect)(res.status).toBe(401);
+				(0, vitest.expect)(res.status).toBe(200);
 			}).run();
 		});
 		(0, vitest.test)(`spawn does not expose a public tag write token`, () => {
@@ -3362,7 +3355,7 @@ function runElectricAgentsConformanceTests(config) {
 					headers: { "content-type": `application/json` },
 					body: JSON.stringify({ payload: `hi` })
 				});
-				(0, vitest.expect)(res.status).toBe(204);
+				(0, vitest.expect)([200, 204]).toContain(res.status);
 			}).expectWebhook().respondDone().run();
 		});
 		(0, vitest.test)(`GET entity does not leak write_token`, () => {

package/dist/index.d.cts

@@ -64,7 +64,6 @@ type HistoryEvent = {
   status: string;
   streams: {
     main: string;
-    error: string;
   };
   parent?: string;
 } | {
@@ -184,7 +183,6 @@ interface WebhookEntityContext {
   url: string;
   streams: {
     main: string;
-    error: string;
   };
   tags?: Record<string, string>;
 }
@@ -273,7 +271,6 @@ interface RunContext {
   currentEntityUrl: string | null;
   currentEntityStreams: {
     main: string;
-    error: string;
   } | null;
   currentWriteToken: string | null;
   notification: WebhookNotification | null;
@@ -320,7 +317,7 @@ declare class ElectricAgentsScenario {
   expectEntityContext(checks?: EntityContextChecks): this;
   expectStatus(status: string): this;
   expectStreamContains(messageType: string): this;
-  readStream(stream?: `main` | `error`): this;
+  readStream(): this;
   list(filter?: {
     type?: string;
     status?: string;

package/dist/index.d.ts

@@ -64,7 +64,6 @@ type HistoryEvent = {
   status: string;
   streams: {
     main: string;
-    error: string;
   };
   parent?: string;
 } | {
@@ -184,7 +183,6 @@ interface WebhookEntityContext {
   url: string;
   streams: {
     main: string;
-    error: string;
   };
   tags?: Record<string, string>;
 }
@@ -273,7 +271,6 @@ interface RunContext {
   currentEntityUrl: string | null;
   currentEntityStreams: {
     main: string;
-    error: string;
   } | null;
   currentWriteToken: string | null;
   notification: WebhookNotification | null;
@@ -320,7 +317,7 @@ declare class ElectricAgentsScenario {
   expectEntityContext(checks?: EntityContextChecks): this;
   expectStatus(status: string): this;
   expectStreamContains(messageType: string): this;
-  readStream(stream?: `main` | `error`): this;
+  readStream(): this;
   list(filter?: {
     type?: string;
     status?: string;

package/dist/index.js

@@ -304,11 +304,8 @@ var ElectricAgentsScenario = class {
 		});
 		return this;
 	}
-	readStream(stream) {
-		this.steps.push({
-			kind: `readStream`,
-			stream
-		});
+	readStream() {
+		this.steps.push({ kind: `readStream` });
 		return this;
 	}
 	list(filter) {
@@ -600,7 +597,7 @@ async function executeStep(ctx, step) {
 				method: `POST`,
 				body: JSON.stringify(body)
 			});
-			expect(res.status).toBe(204);
+			expect([200, 204]).toContain(res.status);
 			ctx.history.push({
 				type: `message_sent`,
 				entityUrl,
@@ -647,7 +644,6 @@ async function executeStep(ctx, step) {
 			expect(entity, `Webhook payload must contain entity context`).toBeDefined();
 			expect(entity.url).toBeTruthy();
 			expect(entity.streams.main).toBeTruthy();
-			expect(entity.streams.error).toBeTruthy();
 			if (step.checks?.url) expect(entity.url).toBe(step.checks.url);
 			if (step.checks?.type) expect(entity.type).toBe(step.checks.type);
 			if (step.checks?.status) expect(entity.status).toBe(step.checks.status);
@@ -688,7 +684,7 @@ async function executeStep(ctx, step) {
 		}
 		case `readStream`: {
 			if (!ctx.currentEntityStreams) throw new Error(`No current entity streams`);
-			const streamPath = step.stream === `error` ? ctx.currentEntityStreams.error : ctx.currentEntityStreams.main;
+			const streamPath = ctx.currentEntityStreams.main;
 			const res = await fetch(appendPathToUrl(ctx.baseUrl, `${streamPath}?offset=0000000000000000_0000000000000000`));
 			if (res.status === 200) {
 				const text = await res.text();
@@ -1183,14 +1179,11 @@ function checkEntityContextOnWebhook(history) {
 	}
 }
 /**
-* Spec S5 — Structural: stream paths must match {entity.url}/main and {entity.url}/error.
+* Spec S5 — Structural: stream path must match {entity.url}/main.
 * Soundness: Sound | Completeness: Complete (within trace)
 */
 function checkStreamPathsMatchEntityUrl(history) {
-	for (const event of history) if (event.type === `entity_spawned`) {
-		expect(event.streams.main).toBe(`${event.entityUrl}/main`);
-		expect(event.streams.error).toBe(`${event.entityUrl}/error`);
-	}
+	for (const event of history) if (event.type === `entity_spawned`) expect(event.streams.main).toBe(`${event.entityUrl}/main`);
 }
 /**
 * Spec S4 — Safety: entity status transitions must be valid.
@@ -1252,8 +1245,8 @@ function checkAdditiveSchemaEvolution(history) {
 }
 /**
 * Spec R4 — Registry stream consistency: every entity_spawned must refer to an
-* entity that has not already been killed, and stream URLs must follow the
-* convention {entity.url}/main and {entity.url}/error.
+* entity that has not already been killed, and stream URL must follow the
+* convention {entity.url}/main.
 * Soundness: Sound | Completeness: Complete (within trace)
 */
 function checkRegistryStreamConsistency(history) {
@@ -1263,7 +1256,6 @@ function checkRegistryStreamConsistency(history) {
 		if (event.type === `entity_spawned`) {
 			expect(!killedEntities.has(event.entityUrl), `Registry consistency: entity_spawned for ${event.entityUrl} but entity was already killed`).toBe(true);
 			expect(event.streams.main, `Registry consistency: entity ${event.entityUrl} streams.main must be ${event.entityUrl}/main`).toBe(`${event.entityUrl}/main`);
-			expect(event.streams.error, `Registry consistency: entity ${event.entityUrl} streams.error must be ${event.entityUrl}/error`).toBe(`${event.entityUrl}/error`);
 		}
 	}
 }
@@ -1808,8 +1800,6 @@ function runElectricAgentsConformanceTests(config) {
 		}).spawn(`spawn-test-agent`, `entity-1`).expectStatus(`running`).custom(async (ctx) => {
 			const mainRes = await fetch(appendPathToUrl(ctx.baseUrl, ctx.currentEntityStreams.main), { method: `HEAD` });
 			expect(mainRes.status).toBe(200);
-			const errorRes = await fetch(appendPathToUrl(ctx.baseUrl, ctx.currentEntityStreams.error), { method: `HEAD` });
-			expect(errorRes.status).toBe(200);
 		}).run());
 		test(`spawn at unregistered type returns UNKNOWN_ENTITY_TYPE`, () => electricAgents(config.baseUrl).custom(async (ctx) => {
 			const res = await fetch(appendPathToUrl(ctx.baseUrl, routeControlPlanePath(`/unregistered/entity-1`)), {
@@ -1882,7 +1872,7 @@ function runElectricAgentsConformanceTests(config) {
 				method: `POST`,
 				body: JSON.stringify({ payload: { hello: true } })
 			});
-			expect(res.status).toBe(204);
+			expect([200, 204]).toContain(res.status);
 		}).expectWebhook().respondDone().run());
 	});
 	describe(`Electric Agents List`, () => {
@@ -2496,7 +2486,7 @@ function runElectricAgentsConformanceTests(config) {
 						payload: { action: `test` }
 					})
 				});
-				expect(res.status).toBe(204);
+				expect([200, 204]).toContain(res.status);
 			}).run();
 		});
 	});
@@ -2625,7 +2615,7 @@ function runElectricAgentsConformanceTests(config) {
 										targetIdx
 									} })
 								});
-								expect(res.status).toBe(204);
+								expect([200, 204]).toContain(res.status);
 								ctx.history.push({
 									type: `message_sent`,
 									entityUrl: url,
@@ -2902,12 +2892,15 @@ function runElectricAgentsConformanceTests(config) {
 			const sandbox = await makeSandbox(`/tmp`);
 			try {
 				const tool = createBashTool(sandbox);
-				const result = await tool.execute(`test-tc`, { command: `sleep 60` });
+				const result = await tool.execute(`test-tc`, {
+					command: `sleep 60`,
+					timeoutMs: 1e3
+				});
 				expect(result.details.timedOut).toBe(true);
 			} finally {
 				await sandbox.dispose();
 			}
-		}, 35e3);
+		}, 5e3);
 		test(`read_file rejects paths outside working directory`, async () => {
 			const { createReadFileTool } = await import(`../../agents-runtime/src/tools`);
 			const fs = await import(`node:fs/promises`);
@@ -3116,7 +3109,7 @@ function runElectricAgentsConformanceTests(config) {
 				expect(res.status).toBe(404);
 			}).run();
 		});
-		test(`tag update on stopped entity is rejected without claim`, () => {
+		test(`tag update on stopped entity is rejected as not running`, () => {
 			const id = Date.now();
 			return electricAgents(config.baseUrl).subscription(`/meta-stopped-agent-${id}/**`, `meta-stopped-sub-${id}`).registerType({
 				name: `meta-stopped-agent-${id}`,
@@ -3127,7 +3120,7 @@ function runElectricAgentsConformanceTests(config) {
 					method: `POST`,
 					body: JSON.stringify({ value: `value` })
 				});
-				expect(res.status).toBe(401);
+				expect(res.status).toBe(409);
 			}).run();
 		});
 	});
@@ -3167,7 +3160,7 @@ function runElectricAgentsConformanceTests(config) {
 						type: `farewell`
 					})
 				});
-				expect(res.status).toBe(204);
+				expect([200, 204]).toContain(res.status);
 			}).run();
 		});
 		test(`amend schemas on deleted type returns 404`, async () => {
@@ -3301,11 +3294,11 @@ function runElectricAgentsConformanceTests(config) {
 				expect(ctx.currentWriteToken).toBeNull();
 			}).run();
 		});
-		test(`tag update without token is rejected`, () => {
+		test(`tag update without legacy entity write token is authorized by principal permission`, () => {
 			const id = Date.now();
 			return electricAgents(config.baseUrl).subscription(`/auth-meta-notoken-agent-${id}/**`, `auth-meta-notoken-sub-${id}`).registerType({
 				name: `auth-meta-notoken-agent-${id}`,
-				description: `Test tag update without token`,
+				description: `Test tag update without legacy entity write token`,
 				creation_schema: { type: `object` }
 			}).spawn(`auth-meta-notoken-agent-${id}`, `entity-1`).custom(async (ctx) => {
 				const res = await fetch(appendPathToUrl(ctx.baseUrl, routeControlPlanePath(`${ctx.currentEntityUrl}/tags/key`)), {
@@ -3313,7 +3306,7 @@ function runElectricAgentsConformanceTests(config) {
 					headers: { "content-type": `application/json` },
 					body: JSON.stringify({ value: `value` })
 				});
-				expect(res.status).toBe(401);
+				expect(res.status).toBe(200);
 			}).run();
 		});
 		test(`spawn does not expose a public tag write token`, () => {
@@ -3338,7 +3331,7 @@ function runElectricAgentsConformanceTests(config) {
 					headers: { "content-type": `application/json` },
 					body: JSON.stringify({ payload: `hi` })
 				});
-				expect(res.status).toBe(204);
+				expect([200, 204]).toContain(res.status);
 			}).expectWebhook().respondDone().run();
 		});
 		test(`GET entity does not leak write_token`, () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@electric-ax/agents-server-conformance-tests",
-  "version": "0.1.10",
+  "version": "0.1.12",
   "description": "Conformance test suite for Electric Agents server implementations",
   "author": "Durable Stream contributors",
   "license": "Apache-2.0",
@@ -35,7 +35,7 @@
   ],
   "dependencies": {
     "@durable-streams/client": "^0.2.6",
-    "@electric-sql/client": "^1.5.20",
+    "@electric-sql/client": "^1.5.21",
     "fast-check": "^4.6.0",
     "vitest": "^4.1.0"
   },
@@ -43,7 +43,7 @@
     "@mariozechner/pi-agent-core": "^0.70.2",
     "tsdown": "^0.9.0",
     "typescript": "^5.0.0",
-    "@electric-ax/agents-server": "0.4.15"
+    "@electric-ax/agents-server": "0.4.20"
   },
   "engines": {
     "node": ">=18.0.0"