@serverkgg/bridge 0.1.1 → 0.1.2

package/src/runtime/bridgeDriver.test.ts

@@ -275,6 +275,27 @@ const driverWithTabs = (tabs: number, sections: number): BridgeDriver => ({
 });
 
 describe("panel caps", () => {
+	test("rejects a collection row action named after a read action", () => {
+		const driver = driverWithTabs(1, 1);
+
+		driver.modules = {
+			...driver.modules,
+			worlds: {
+				kind: BridgeKind.Collection,
+				list: async () => [],
+				actions: {
+					preview: async () => {
+						return;
+					},
+				},
+			},
+		};
+
+		expect(() => validateDriver(driver)).toThrow(
+			'module "worlds" declares action "preview", which is reserved for reads',
+		);
+	});
+
 	test("takes the tab and section counts the agent mirror accepts", () => {
 		expect(() => validateDriver(driverWithTabs(8, 8))).not.toThrow();
 	});

package/src/runtime/bridgeDriver.ts

@@ -3,6 +3,7 @@ import {
 	BRIDGE_MODULE_ID_PATTERN,
 	BridgeKind,
 	BridgeSetupStepKind,
+	isReadAction,
 	type LocalizedText,
 } from "../core";
 import { GuideOpenTab } from "../guides";
@@ -801,15 +802,17 @@ export const validateDriver = (driver: BridgeDriver) => {
 			countIn((module.dependsOn ?? []).length, BRIDGE_DEPENDS_ON_LIMIT, "dependencies", `module "${id}"`);
 		}
 
-		if (module.kind === BridgeKind.Actions) {
-			for (const action of Object.keys(module.actions)) {
-				validateModuleId(action, `module "${id}" action`);
-			}
-		}
-
-		if (module.kind === BridgeKind.Collection || module.kind === BridgeKind.Detail) {
+		if (
+			module.kind === BridgeKind.Actions
+			|| module.kind === BridgeKind.Collection
+			|| module.kind === BridgeKind.Detail
+		) {
 			for (const action of Object.keys(module.actions ?? {})) {
 				validateModuleId(action, `module "${id}" action`);
+
+				if (isReadAction(action)) {
+					fail(`module "${id}" declares action "${action}", which is reserved for reads — pick another id`);
+				}
 			}
 		}
 	}

package/src/runtime/bridgeLog.ts

@@ -36,3 +36,13 @@ export const log: BridgeLeveledLogger = Object.assign(
 export const errorText = (error: unknown) => {
 	return error instanceof Error ? error.message : String(error);
 };
+
+export const errorDetail = (error: unknown) => {
+	const text = errorText(error);
+
+	if (!(error instanceof Error) || error.cause === undefined) {
+		return text;
+	}
+
+	return `${text} — ${errorText(error.cause)}`;
+};

package/src/runtime/bridgeRefusalError.ts

@@ -0,0 +1,13 @@
+import type { LocalizedText } from "../core";
+import { BridgeDriverError } from "./bridgeDriver";
+
+export class BridgeRefusalError extends BridgeDriverError {
+	readonly text: LocalizedText | null;
+
+	constructor(message: string, text: LocalizedText | null = null) {
+		super(message);
+
+		this.name = "BridgeRefusalError";
+		this.text = text;
+	}
+}