@serverkgg/bridge 0.0.8 → 0.0.9

package/src/runtime/bridgeDriver.ts

@@ -8,6 +8,7 @@ import {
 	type BridgeSection,
 	type BridgeSectionAction,
 	type BridgeTab,
+	BridgeUploadMode,
 	type BridgeVisibleWhen,
 } from "../panel";
 import {
@@ -16,6 +17,7 @@ import {
 	BRIDGE_EVENTS_LIMIT,
 	BRIDGE_EXTENSION_LENGTH_LIMIT,
 	BRIDGE_FIELD_OPTIONS_LIMIT,
+	BRIDGE_FORM_CLEARS_LIMIT,
 	BRIDGE_FORM_FIELDS_LIMIT,
 	BRIDGE_ID_LENGTH_LIMIT,
 	BRIDGE_KEY_LENGTH_LIMIT,
@@ -35,12 +37,15 @@ import {
 } from "../protocol";
 import { actionsOf } from "./bridgeActions";
 
+const UPLOAD_MODES = new Set<string>(Object.values(BridgeUploadMode));
+
 const RESERVED_MODULE_IDS = new Set([
 	"announce",
 	"backup",
 	"events",
 	"install",
 	"lifecycle",
+	"pending",
 	"query",
 ]);
 
@@ -259,6 +264,58 @@ const validateSection = (modules: Record<string, BridgePanelModule>, section: Br
 			countIn(section.fields.length, BRIDGE_FORM_FIELDS_LIMIT, "fields", where);
 			validateFields(modules, section.fields, where);
 
+			const clears = section.clears ?? [];
+
+			if (clears.length > 0 && section.target !== BridgeFormTarget.Variables) {
+				fail(`${where} clears variables but does not target variables`);
+			}
+
+			countIn(clears.length, BRIDGE_FORM_CLEARS_LIMIT, "cleared variables", where);
+
+			const cleared = new Set<string>();
+
+			for (const clear of clears) {
+				if (cleared.has(clear.variable)) {
+					fail(`${where} clears "${clear.variable}" more than once`);
+				}
+
+				cleared.add(clear.variable);
+
+				if (section.fields.some((field) => field.key === clear.variable)) {
+					fail(`${where} clears "${clear.variable}", which is one of its own fields`);
+				}
+
+				validateText(clear.warning, `clear "${clear.variable}" warning`, where);
+			}
+
+			return;
+		}
+
+		case BridgeLayout.Detail: {
+			const module = moduleOf(modules, section.module, where);
+
+			if (module.kind !== BridgeKind.Detail) {
+				fail(`${where} references "${section.module}", which is not a detail module`);
+
+				return;
+			}
+
+			countIn((section.actions ?? []).length, BRIDGE_SECTION_ACTIONS_LIMIT, "actions", where);
+			validateActions(modules, section.actions ?? [], where);
+
+			for (const action of section.actions ?? []) {
+				if (action.download) {
+					fail(`${where} declares action "${action.id}" as a download, which a detail cannot serve`);
+				}
+
+				if (!module.actions?.[action.id]) {
+					fail(`${where} declares action "${action.id}", which module "${section.module}" does not implement`);
+				}
+			}
+
+			validateText(section.confirmText, "confirm text", where);
+			validateText(section.related?.label, "related label", where);
+
 			return;
 		}
 
@@ -313,6 +370,10 @@ const validateSection = (modules: Record<string, BridgePanelModule>, section: Br
 					for (const extension of section.upload.extensions) {
 						lengthIn(extension, BRIDGE_EXTENSION_LENGTH_LIMIT, `upload extension "${extension}"`, where);
 					}
+
+					if (section.upload.mode !== undefined && !UPLOAD_MODES.has(section.upload.mode)) {
+						fail(`${where} declares an unknown upload mode "${section.upload.mode}"`);
+					}
 				}
 			}
 
@@ -346,6 +407,36 @@ const validateSection = (modules: Record<string, BridgePanelModule>, section: Br
 	}
 };
 
+const validatePending = (driver: BridgeDriver) => {
+	const pending = driver.pending;
+
+	if (!pending) {
+		return;
+	}
+
+	const where = "the pending module";
+
+	if (pending.kind !== BridgeKind.Pending) {
+		fail(`${where} declares kind "${pending.kind}", which is not "${BridgeKind.Pending}"`);
+	}
+
+	if (pending.staging.length === 0) {
+		fail(`${where} declares no staging path`);
+	}
+
+	lengthIn(pending.staging, BRIDGE_STAGING_LENGTH_LIMIT, "staging path", where);
+
+	if (pending.extensions.length === 0) {
+		fail(`${where} declares no extensions`);
+	}
+
+	countIn(pending.extensions.length, BRIDGE_UPLOAD_EXTENSIONS_LIMIT, "extensions", where);
+
+	for (const extension of pending.extensions) {
+		lengthIn(extension, BRIDGE_EXTENSION_LENGTH_LIMIT, `extension "${extension}"`, where);
+	}
+};
+
 const validateTab = (modules: Record<string, BridgePanelModule>, tab: BridgeTab) => {
 	const sections = new Set<string>();
 
@@ -382,6 +473,8 @@ export const validateDriver = (driver: BridgeDriver) => {
 
 	countIn(emitted.size, BRIDGE_EVENTS_LIMIT, "event names", "the driver");
 
+	validatePending(driver);
+
 	for (const [id, module] of Object.entries(modules)) {
 		validateModuleId(id, "module");
 
@@ -401,7 +494,7 @@ export const validateDriver = (driver: BridgeDriver) => {
 			}
 		}
 
-		if (module.kind === BridgeKind.Collection) {
+		if (module.kind === BridgeKind.Collection || module.kind === BridgeKind.Detail) {
 			for (const action of Object.keys(module.actions ?? {})) {
 				validateModuleId(action, `module "${id}" action`);
 			}
@@ -429,6 +522,26 @@ export const validateDriver = (driver: BridgeDriver) => {
 
 		validateTab(modules, tab);
 	}
+
+	const declared = declaredVariables(driver);
+
+	for (const tab of driver.panel.tabs) {
+		for (const section of tab.sections) {
+			const where = `tab "${tab.id}" section "${section.id}"`;
+
+			if (section.layout === BridgeLayout.Detail && section.related && !tabs.has(section.related.tab)) {
+				fail(`${where} relates to tab "${section.related.tab}", which the panel does not declare`);
+			}
+
+			if (section.layout === BridgeLayout.Form) {
+				for (const clear of section.clears ?? []) {
+					if (!declared.has(clear.variable)) {
+						fail(`${where} clears "${clear.variable}", which no variables form, catalog or detail declares`);
+					}
+				}
+			}
+		}
+	}
 };
 
 const declaredVariables = (driver: BridgeDriver) => {
@@ -442,7 +555,7 @@ const declaredVariables = (driver: BridgeDriver) => {
 				}
 			}
 
-			if (section.layout === BridgeLayout.Catalog && section.variable) {
+			if ((section.layout === BridgeLayout.Catalog || section.layout === BridgeLayout.Detail) && section.variable) {
 				declared.add(section.variable);
 			}
 		}
@@ -461,7 +574,7 @@ export const validatePortActivation = (ports: ManifestPort[], driver: BridgeDriv
 			}
 
 			fail(
-				`port "${port.key}" activeWhen[${index}] names "${rule.variable}", which no variables form or catalog declares`,
+				`port "${port.key}" activeWhen[${index}] names "${rule.variable}", which no variables form, catalog or detail declares`,
 			);
 		}
 	}

package/src/runtime/bridgeEgress.ts

@@ -7,6 +7,7 @@ import {
 	BridgeEgressMethod,
 	type BridgeEgressRequest,
 	type BridgeEgressResponse,
+	type BridgeHttpMethod,
 } from "../protocol";
 
 const CONNECT_RETRY_MS = 10_000;
@@ -39,6 +40,8 @@ export class BridgeEgressCallError extends Error {
 }
 
 export interface BridgeEgressCallOptions {
+	httpMethod?: BridgeHttpMethod;
+	body?: string;
 	headers?: Record<string, string>;
 	timeoutMs?: number;
 	cache?: boolean;

package/src/runtime/bridgeEvents.ts

@@ -37,11 +37,13 @@ export const createEventRing = (events: BridgeEventsModule | undefined, supervis
 					continue;
 				}
 
-				const groups = match.groups
-					? {
-							...match.groups,
-						}
-					: null;
+				const groups =
+					match.groups || pattern.payload
+						? {
+								...pattern.payload,
+								...match.groups,
+							}
+						: null;
 
 				push(pattern.emit, groups);
 			}