@teamscala/service-boot 0.23.314 → 0.23.316

package/package.json

@@ -6,7 +6,7 @@
   },
   "name": "@teamscala/service-boot",
   "type": "module",
-  "version": "0.23.314",
+  "version": "0.23.316",
   "sideEffects": false,
   "files": [
     "src"

package/src/fn-routers/task-sync-assign.d.ts

@@ -93,16 +93,19 @@ export type AssignTaskInput = v.InferOutput<typeof assignTaskInput>;
  * "fn.taskSync is 404" and reached for serving changes, when the row was simply
  * never bound because the body never parsed.
  *
- * With the schema oRPC parses the body, so `parent_id` reaches the adopt path.
- * Every field is OPTIONAL and the handler keeps a header fallback for each, so a
- * caller that transports everything on headers (zerostack: x-lane-pane/session/
- * work-item) parses to an empty object and behaves EXACTLY as before — the schema
- * ADDS the body path without removing the header one. As a side effect it also
- * fixes zerostack cli_kind mis-attribution: zerostack's body `{lane:{cli_kind:
- * "zerostack"}}` was never parsed, so its cli_kind fell through every header to
- * the "claude-code" default; parsed now, it reads "zerostack" as intended.
+ * ROOT-OPTIONAL (`v.optional(v.object(...))`) so it tolerates a caller that sends
+ * a DIRECT body or none at all: oRPC extracts `input` from the `{json:{...}}` RPC
+ * envelope, so a raw-object or empty POST yields `undefined`, which the optional
+ * root ACCEPTS — the handler then falls back to its headers, exactly as it did
+ * schema-less (a header-only caller like zerostack is unchanged). An adapter that
+ * wants the body path (pi's `parent_id`) POSTs the `{json:{...}}` envelope and
+ * `parent_id` is parsed + reaches `adoptAssignmentTask` — the same envelope
+ * convention pi ALREADY uses for upsertFromTaskList. A raw-object POST is NOT
+ * parsed (oRPC hands the handler `undefined`), which is why pi's bindLaneTask
+ * must move to the envelope to deliver `parent_id`. Every field is optional with
+ * a header fallback, so the schema ADDS the body path without removing the header.
  */
-export declare const bindLaneInput: v.ObjectSchema<{
+export declare const bindLaneInput: v.OptionalSchema<v.ObjectSchema<{
     readonly lane: v.OptionalSchema<v.ObjectSchema<{
         readonly pane: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
         readonly session: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
@@ -112,7 +115,7 @@ export declare const bindLaneInput: v.ObjectSchema<{
     }, undefined>, undefined>;
     readonly title: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
     readonly parent_id: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
-}, undefined>;
+}, undefined>, undefined>;
 /**
  * Resolve the org a dispatched task is created under. Pure: the platform org is
  * INJECTED (resolved by the caller at the handler boundary, where a registry

package/src/fn-routers/task-sync-assign.test.ts

@@ -309,8 +309,15 @@ describe("bindLaneInput (body parent_id reaches the adopt path — schema-less g
 		// A caller transporting everything on headers sends {} (or omits the
 		// body); the schema must not reject it, and every field is undefined so the
 		// handler's header fallbacks carry the values.
-		const out = v.safeParse(bindLaneInput, {});
-		expect(out.success).toBe(true);
+		expect(v.safeParse(bindLaneInput, {}).success).toBe(true);
+	});
+
+	it("ACCEPTS undefined input — a raw-object POST yields undefined (root-optional)", () => {
+		// oRPC extracts input from the {json:{...}} envelope; a raw-object POST
+		// (what zerostack/pi send today) yields undefined. The root MUST be optional
+		// or those callers 400 instead of falling back to headers. A strict root
+		// regressed exactly this (0.23.314), so this assertion locks the tolerance.
+		expect(v.safeParse(bindLaneInput, undefined).success).toBe(true);
 	});
 
 	it("treats every field as optional (no caller is forced to populate the body)", () => {

package/src/fn-routers/task-sync-assign.ts

@@ -105,28 +105,33 @@ export type AssignTaskInput = v.InferOutput<typeof assignTaskInput>;
  * "fn.taskSync is 404" and reached for serving changes, when the row was simply
  * never bound because the body never parsed.
  *
- * With the schema oRPC parses the body, so `parent_id` reaches the adopt path.
- * Every field is OPTIONAL and the handler keeps a header fallback for each, so a
- * caller that transports everything on headers (zerostack: x-lane-pane/session/
- * work-item) parses to an empty object and behaves EXACTLY as before — the schema
- * ADDS the body path without removing the header one. As a side effect it also
- * fixes zerostack cli_kind mis-attribution: zerostack's body `{lane:{cli_kind:
- * "zerostack"}}` was never parsed, so its cli_kind fell through every header to
- * the "claude-code" default; parsed now, it reads "zerostack" as intended.
+ * ROOT-OPTIONAL (`v.optional(v.object(...))`) so it tolerates a caller that sends
+ * a DIRECT body or none at all: oRPC extracts `input` from the `{json:{...}}` RPC
+ * envelope, so a raw-object or empty POST yields `undefined`, which the optional
+ * root ACCEPTS — the handler then falls back to its headers, exactly as it did
+ * schema-less (a header-only caller like zerostack is unchanged). An adapter that
+ * wants the body path (pi's `parent_id`) POSTs the `{json:{...}}` envelope and
+ * `parent_id` is parsed + reaches `adoptAssignmentTask` — the same envelope
+ * convention pi ALREADY uses for upsertFromTaskList. A raw-object POST is NOT
+ * parsed (oRPC hands the handler `undefined`), which is why pi's bindLaneTask
+ * must move to the envelope to deliver `parent_id`. Every field is optional with
+ * a header fallback, so the schema ADDS the body path without removing the header.
  */
-export const bindLaneInput = v.object({
-	lane: v.optional(
-		v.object({
-			pane: v.optional(v.string()),
-			session: v.optional(v.string()),
-			cli_kind: v.optional(v.string()),
-			role: v.optional(v.string()),
-			organisation_id: v.optional(v.string()),
-		}),
-	),
-	title: v.optional(v.string()),
-	parent_id: v.optional(v.string()),
-});
+export const bindLaneInput = v.optional(
+	v.object({
+		lane: v.optional(
+			v.object({
+				pane: v.optional(v.string()),
+				session: v.optional(v.string()),
+				cli_kind: v.optional(v.string()),
+				role: v.optional(v.string()),
+				organisation_id: v.optional(v.string()),
+			}),
+		),
+		title: v.optional(v.string()),
+		parent_id: v.optional(v.string()),
+	}),
+);
 
 /**
  * Resolve the org a dispatched task is created under. Pure: the platform org is

package/src/fn-routers/task-sync-router.d.ts

@@ -187,7 +187,7 @@ export declare const taskSyncRouter: {
         prisma: any;
         user: import("./auth-router.ts").User;
         orgId: string | null | undefined;
-    }>, v.ObjectSchema<{
+    }>, v.OptionalSchema<v.ObjectSchema<{
         readonly lane: v.OptionalSchema<v.ObjectSchema<{
             readonly pane: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
             readonly session: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
@@ -197,7 +197,7 @@ export declare const taskSyncRouter: {
         }, undefined>, undefined>;
         readonly title: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
         readonly parent_id: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
-    }, undefined>, import("@orpc/contract").Schema<{
+    }, undefined>, undefined>, import("@orpc/contract").Schema<{
         task_id: string;
         created: boolean;
     }, {

package/src/fn-routers/task-sync-router.ts

@@ -426,6 +426,11 @@ export const taskSyncRouter = {
 		const ctx = context as Context;
 		const prisma = ctx.prisma as Prisma;
 		const log = getAppLogger();
+		// DIAG: capture the real exception+stack for the pane-with-run 500
+		// (oRPC swallows it to a generic 500; the run-link catch is not firing).
+		const __diagPane = (input as { lane?: { pane?: string } })?.lane?.pane ?? ctx.headers?.get("x-lane-pane");
+		const __diagSession = (input as { lane?: { session?: string } })?.lane?.session ?? ctx.headers?.get("x-lane-session");
+		try {
 		// The SessionStart hook sends the pane + the CLI's native session id.
 		// SESSION is the match key (kind_data.lane.session) — it is unique per
 		// CLI session and re-minted each session, so a fresh session cannot find
@@ -599,6 +604,14 @@ export const taskSyncRouter = {
 			});
 		}
 		return { task_id: taskId, created };
+		} catch (__diagErr) {
+			log.error("[task-sync] bindLane THREW (diag)", {
+				pane: __diagPane, session: __diagSession,
+				error: __diagErr instanceof Error ? __diagErr.message : String(__diagErr),
+				stack: __diagErr instanceof Error ? __diagErr.stack : undefined,
+			});
+			throw __diagErr;
+		}
 	}),
 
 	/**