@protobase/server 0.67.10 → 0.67.11

package/dist/package.js

@@ -1,4 +1,4 @@
 //#region package.json
-var version = "0.67.10";
+var version = "0.67.11";
 //#endregion
 export { version };

package/dist/workflows/save-hook.js

@@ -1,18 +1,37 @@
 import { badRequest } from "../problem.js";
+import { requiredStepInputs } from "@protobase/schema/internal";
 import { sql } from "kysely";
 import { checkDefinition, problemText } from "@protobase/workflows";
 //#region src/workflows/save-hook.ts
 /**
 * Checks a workflow's definition on every save that changes it, against the language's schema and the named-reads
 * profile, with the step types the app has, and keeps it as the version the table's trigger numbered, saved by the
-* caller. A refusal is a 400 problem naming each state and field, also listed as `problems`; the write rolls back.
+* caller. A refusal is a 400 problem naming each state and field, also listed as `problems`, and as the `definition`
+* field's `errors` with the state and field in their `params`, for a form to show each on its step; the write rolls back.
 */
 const workflowSaveHook = (schema, stepTypes) => async (event, trx) => {
 	if (event.resource.name !== "workflows" || !event.after) return;
 	if (event.operation !== "create" && event.operation !== "update") return;
 	if (event.operation === "update" && event.after.version === event.before?.version) return;
-	const problems = checkDefinition(event.after.definition, { resources: stepTypes });
-	if (problems.length > 0) throw badRequest("invalid-workflow", `The workflow cannot be saved: ${problems.map(problemText).join("; ")}`, { problems });
+	const problems = checkDefinition(event.after.definition, {
+		resources: stepTypes,
+		required: requiredStepInputs
+	});
+	if (problems.length > 0) {
+		const errors = problems.map((problem) => ({
+			code: "invalid-workflow",
+			field: "definition",
+			message: problemText(problem),
+			params: {
+				state: problem.state,
+				field: problem.field
+			}
+		}));
+		throw badRequest("invalid-workflow", `The workflow cannot be saved: ${problems.map(problemText).join("; ")}`, {
+			problems,
+			errors
+		});
+	}
 	await sql`insert into ${sql.id(schema, "versions")} (workflow_id, version, definition, saved_by, tenant_id)
     select w.id, w.version, w.definition, ${String(event.user.id)}, w.tenant_id from ${sql.id(schema, "workflows")} w where w.id = ${String(event.after.id)}`.execute(trx);
 };

package/dist/workflows/step-types.js

@@ -2,17 +2,10 @@ import { httpStep } from "./http-step.js";
 import { mailStep } from "./mail-step.js";
 import { pieceStep } from "./piece-step.js";
 import { recordsSteps } from "./records-step.js";
+import { workflowStepTypes } from "@protobase/schema/internal";
 //#region src/workflows/step-types.ts
-/** The step types a Task names as its `Resource`, as the language report lists them. */
-const stepTypeNames = [
-	"app:http",
-	"app:records.create",
-	"app:records.update",
-	"app:records.get",
-	"app:records.list",
-	"app:mail.send",
-	"app:piece"
-];
+/** The step types a Task names as its `Resource`, as the editor offers them. */
+const stepTypeNames = Object.keys(workflowStepTypes);
 /** The step types of one segment, which sends mail with the mailer of the job that runs it. */
 const stepTypes = ({ deps, schema, roles, allowPrivateAddresses }, mail) => {
 	const files = () => deps().files;

package/dist/workflows/store.js

@@ -17,15 +17,18 @@ const postgresRunStore = (schema) => {
         from ${steps} where run_id = ${id} order by scope, seq`.execute(db);
 			return rows;
 		},
-		save: async (db, id, step) => {
+		save: async (db, id, records) => {
+			const row = (step) => sql`(${step.scope}, ${step.seq}::integer, ${step.state}, ${step.type}, ${step.status}, ${step.attempt}::integer,
+        ${json(step.input)}::jsonb, ${json(step.output)}::jsonb, ${json(step.assigned)}::jsonb, ${step.next}, ${step.resumeAt ?? null}::timestamptz,
+        ${step.error ?? null}, ${step.cause ?? null}, ${JSON.stringify(step.logs)}::jsonb, ${step.startedAt}::timestamptz, ${step.finishedAt}::timestamptz)`;
 			const { rows } = await sql`insert into ${steps}
         (run_id, tenant_id, scope, seq, state, type, status, attempt, input, output, assigned, next, resume_at, error, cause, logs, started_at, finished_at)
-        select r.id, r.tenant_id, ${step.scope}, ${step.seq}, ${step.state}, ${step.type}, ${step.status}, ${step.attempt}, ${json(step.input)}::jsonb,
-          ${json(step.output)}::jsonb, ${json(step.assigned)}::jsonb, ${step.next}, ${step.resumeAt ?? null}, ${step.error ?? null}, ${step.cause ?? null},
-          ${JSON.stringify(step.logs)}::jsonb, ${step.startedAt}, ${step.finishedAt}
-        from ${runs} r where r.id = ${id}
+        select r.id, r.tenant_id, s.*
+        from ${runs} r, (values ${sql.join(records.map(row))})
+          as s (scope, seq, state, type, status, attempt, input, output, assigned, next, resume_at, error, cause, logs, started_at, finished_at)
+        where r.id = ${id}
         on conflict (run_id, scope, seq) do nothing returning id`.execute(db);
-			return rows.length > 0;
+			return rows.length === records.length;
 		},
 		update: async (db, id, change) => {
 			const ended = change.status === "succeeded" || change.status === "failed";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@protobase/server",
-  "version": "0.67.10",
+  "version": "0.67.11",
   "description": "The Protobase REST API as a Hono app, with Better Auth sign-in.",
   "license": "Apache-2.0",
   "repository": {
@@ -44,10 +44,10 @@
     "@better-auth/sso": "^1.7.12",
     "@livekit/protocol": "1.53.1",
     "@modelcontextprotocol/server": "^2.3.1",
-    "@protobase/layout": "0.67.10",
-    "@protobase/query": "0.67.10",
-    "@protobase/schema": "0.67.10",
-    "@protobase/workflows": "0.67.10",
+    "@protobase/layout": "0.67.11",
+    "@protobase/query": "0.67.11",
+    "@protobase/schema": "0.67.11",
+    "@protobase/workflows": "0.67.11",
     "@react-email/render": "^2.1.0",
     "@react-pdf/primitives": "^4.4.0",
     "@react-pdf/renderer": "^4.9.0",