@danypops/papyrus 0.44.9 → 0.44.11

package/src/handlers/tasks.ts

@@ -19,7 +19,6 @@
  *
  * remove/remove_subtree/restore are not duplicated here -- see ./artifact-trash-vehicle.ts.
  */
-import { bindVehicleOperation, defineVehicleOperation, type VehicleOperationContext } from "@danypops/vehicle-core";
 import type { VehicleRegistry } from "@danypops/vehicle-server";
 import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import type { TaskViewMode } from "../domain/task-scope.ts";
@@ -31,10 +30,10 @@ import {
 	classifySessionAuthorization,
 	classifyTaskDependencyCycles,
 	classifyTaskExecutionBounds,
+	createOperationDefiner,
+	definePairedMutation,
 	labelsById,
-	looseObjectSchema,
 	numberProp,
-	passthroughOutput,
 	resolveArtifactIdWidened,
 	stringProp,
 	validationError,
@@ -167,36 +166,7 @@ export function registerTasksVehicleOperations(registry: VehicleRegistry, deps:
 		classifySessionAuthorization(() =>
 			classifyTaskExecutionBounds(() => classifyTaskDependencyCycles(() => moduleOperations.get(name)!.execute(input))),
 		);
-
-	const define = (
-		action: string,
-		description: string,
-		effect: "read" | "local-write",
-		properties: Record<string, { type: string; enum?: readonly string[] }>,
-		required: readonly string[],
-		resolve: (input: Record<string, unknown>) => Record<string, unknown>,
-		execute?: (input: Record<string, unknown>, context: VehicleOperationContext<Record<string, unknown>>) => unknown,
-	): void => {
-		const operation = defineVehicleOperation({
-			name: `tasks.${action}`,
-			version: 1,
-			description,
-			input: looseObjectSchema(properties, required),
-			output: passthroughOutput,
-			permissions: ["tasks:read", "tasks:write"],
-			effect,
-			idempotency: { mode: effect === "read" ? "safe" : "unsafe" },
-			limits: LIMITS,
-		});
-		registry.register(
-			OWNER,
-			bindVehicleOperation(
-				operation,
-				() => async (context) =>
-					(execute ?? ((input: Record<string, unknown>) => call(`tasks.${action}`, input)))(resolve(context.input), context),
-			),
-		);
-	};
+	const define = createOperationDefiner(registry, OWNER, "tasks", ["tasks:read", "tasks:write"], call);
 
 	/** Shared by every action taking a single id/name: resolves root_task_name first, then name -> id against the final scope. */
 	const resolveIdAndScope = (input: Record<string, unknown>): Record<string, unknown> => {
@@ -395,6 +365,7 @@ export function registerTasksVehicleOperations(registry: VehicleRegistry, deps:
 		(input) => input,
 	);
 
+	/** Injects the caller's own session claims (never a model-visible input field) onto every focus/pause/unpause/clear_focus call -- see this file's own doc comment. */
 	const focusOperation = (
 		action: "focus" | "pause" | "unpause" | "clear_focus",
 		description: string,
@@ -402,24 +373,10 @@ export function registerTasksVehicleOperations(registry: VehicleRegistry, deps:
 		required: readonly string[],
 		resolve: (input: Record<string, unknown>) => Record<string, unknown>,
 	): void => {
-		const operation = defineVehicleOperation({
-			name: `tasks.${action}`,
-			version: 1,
-			description,
-			input: looseObjectSchema(properties, required),
-			output: passthroughOutput,
-			permissions: ["tasks:read", "tasks:write"],
-			effect: "local-write",
-			idempotency: { mode: "unsafe" },
-			limits: LIMITS,
+		define(action, description, "local-write", properties, required, resolve, (resolvedInput, context) => {
+			const claims = context.principal?.claims as { sessionId?: string; sessionSecret?: string } | undefined;
+			return call(`tasks.${action}`, { ...resolvedInput, session_id: claims?.sessionId, session_secret: claims?.sessionSecret });
 		});
-		registry.register(
-			OWNER,
-			bindVehicleOperation(operation, () => async (context) => {
-				const claims = context.principal?.claims as { sessionId?: string; sessionSecret?: string } | undefined;
-				return call(`tasks.${action}`, { ...resolve(context.input), session_id: claims?.sessionId, session_secret: claims?.sessionSecret });
-			}),
-		);
 	};
 
 	focusOperation(
@@ -585,34 +542,16 @@ export function registerTasksVehicleOperations(registry: VehicleRegistry, deps:
 
 	const scopeProp = { type: "string", enum: ["project", "graph", "all"] } as const;
 
-	define(
-		"depend",
-		"Adds a dependency edge (this task waits for dependency_id/dependency_name). Dependency edges form an executable DAG -- self-dependencies and cycles are rejected. A name resolved outside project_root's own scope is retried once against every project before failing, unless scope is pinned explicitly.",
-		"local-write",
-		{
-			id: stringProp,
-			name: stringProp,
-			dependency_id: stringProp,
-			dependency_name: stringProp,
-			project_root: stringProp,
-			scope: scopeProp,
-			session_id: stringProp,
-		},
-		[],
-		(input) => {
-			const filter = { projectRoot: input.project_root as string | undefined, scope: input.scope as TaskViewMode | undefined };
-			return {
-				...input,
-				id: resolveTaskId(artifacts, tasks, filter, input.id, input.name),
-				dependency_id: resolveTaskId(artifacts, tasks, filter, input.dependency_id, input.dependency_name),
-			};
-		},
-	);
+	/** Resolves a task id/name pair against the filter derived from this same input's own project_root/scope -- shared by depend/undepend and contain/uncontain below. */
+	const resolveScopedTaskId = (input: Record<string, unknown>, idProp: string, nameProp: string): string => {
+		const filter = { projectRoot: input.project_root as string | undefined, scope: input.scope as TaskViewMode | undefined };
+		return resolveTaskId(artifacts, tasks, filter, input[idProp], input[nameProp]);
+	};
 
-	define(
-		"undepend",
-		"Removes a dependency edge. Idempotent -- a no-op if the edge is already absent.",
-		"local-write",
+	definePairedMutation(
+		define,
+		{ idProp: "id", nameProp: "name" },
+		{ idProp: "dependency_id", nameProp: "dependency_name" },
 		{
 			id: stringProp,
 			name: stringProp,
@@ -623,44 +562,19 @@ export function registerTasksVehicleOperations(registry: VehicleRegistry, deps:
 			session_id: stringProp,
 		},
 		[],
-		(input) => {
-			const filter = { projectRoot: input.project_root as string | undefined, scope: input.scope as TaskViewMode | undefined };
-			return {
-				...input,
-				id: resolveTaskId(artifacts, tasks, filter, input.id, input.name),
-				dependency_id: resolveTaskId(artifacts, tasks, filter, input.dependency_id, input.dependency_name),
-			};
-		},
-	);
-
-	define(
-		"contain",
-		"Nests a child Task inside a parent (parent_id/parent_name contains child_id/child_name) -- explicit hierarchy, distinct from depends_on execution ordering. A name resolved outside project_root's own scope is retried once against every project before failing, unless scope is pinned explicitly.",
-		"local-write",
+		resolveScopedTaskId,
 		{
-			parent_id: stringProp,
-			parent_name: stringProp,
-			child_id: stringProp,
-			child_name: stringProp,
-			project_root: stringProp,
-			scope: scopeProp,
-			session_id: stringProp,
-		},
-		[],
-		(input) => {
-			const filter = { projectRoot: input.project_root as string | undefined, scope: input.scope as TaskViewMode | undefined };
-			return {
-				...input,
-				parent_id: resolveTaskId(artifacts, tasks, filter, input.parent_id, input.parent_name),
-				child_id: resolveTaskId(artifacts, tasks, filter, input.child_id, input.child_name),
-			};
+			action: "depend",
+			description:
+				"Adds a dependency edge (this task waits for dependency_id/dependency_name). Dependency edges form an executable DAG -- self-dependencies and cycles are rejected. A name resolved outside project_root's own scope is retried once against every project before failing, unless scope is pinned explicitly.",
 		},
+		{ action: "undepend", description: "Removes a dependency edge. Idempotent -- a no-op if the edge is already absent." },
 	);
 
-	define(
-		"uncontain",
-		"Removes a parent/child nesting. Idempotent -- a no-op if the edge is already absent.",
-		"local-write",
+	definePairedMutation(
+		define,
+		{ idProp: "parent_id", nameProp: "parent_name" },
+		{ idProp: "child_id", nameProp: "child_name" },
 		{
 			parent_id: stringProp,
 			parent_name: stringProp,
@@ -671,14 +585,13 @@ export function registerTasksVehicleOperations(registry: VehicleRegistry, deps:
 			session_id: stringProp,
 		},
 		[],
-		(input) => {
-			const filter = { projectRoot: input.project_root as string | undefined, scope: input.scope as TaskViewMode | undefined };
-			return {
-				...input,
-				parent_id: resolveTaskId(artifacts, tasks, filter, input.parent_id, input.parent_name),
-				child_id: resolveTaskId(artifacts, tasks, filter, input.child_id, input.child_name),
-			};
+		resolveScopedTaskId,
+		{
+			action: "contain",
+			description:
+				"Nests a child Task inside a parent (parent_id/parent_name contains child_id/child_name) -- explicit hierarchy, distinct from depends_on execution ordering. A name resolved outside project_root's own scope is retried once against every project before failing, unless scope is pinned explicitly.",
 		},
+		{ action: "uncontain", description: "Removes a parent/child nesting. Idempotent -- a no-op if the edge is already absent." },
 	);
 
 	define(

package/src/modules/discuss.ts

@@ -6,41 +6,20 @@
 import type { ArtifactEventContext } from "../artifact/artifact-event.ts";
 import type { Discussions } from "../discussion/discussion-service.ts";
 import type { OperationDefinition } from "../module-registry.ts";
+import { type OperationInput, optionalNumber, optionalString, optionalStringArray } from "./operation-input.ts";
 
 const MODULE_ID = "discuss";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0)
-		throw new Error(`${key} is required${REQUIRED_FIELD_HINTS[key] ? ` (${REQUIRED_FIELD_HINTS[key]})` : ""}`);
-	return value;
-}
-
 /** Fields whose name alone doesn't say what a valid value looks like -- everything else (title, content, id, settlement) is self-explanatory. */
 const REQUIRED_FIELD_HINTS: Record<string, string> = {
 	actor: 'a display name for who is posting, e.g. "alice" or "agent"',
 };
 
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalStringArray(input: OperationInput, key: string): string[] | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (!Array.isArray(value) || value.some((entry) => typeof entry !== "string")) throw new Error(`${key} must be an array of strings`);
-	return value as string[];
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
+/** Same contract as the shared string() in ./operation-input.ts, plus a field-specific hint on failure -- discuss's own operations are the one surface where a bare "actor is required" isn't self-explanatory. */
+function string(input: OperationInput, key: string): string {
 	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
+	if (typeof value !== "string" || value.length === 0)
+		throw new Error(`${key} is required${REQUIRED_FIELD_HINTS[key] ? ` (${REQUIRED_FIELD_HINTS[key]})` : ""}`);
 	return value;
 }
 

package/src/modules/docs.ts

@@ -7,6 +7,7 @@
  * ArtifactStore-based with no other module's concrete class dependency.
  */
 
+import { summarizeArtifact } from "../artifact/artifact.ts";
 import type { ArtifactScopeStore } from "../artifact/artifact-scope-store.ts";
 import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import type { AuthorityRegistry } from "../authority-registry.ts";
@@ -21,31 +22,10 @@ import {
 	updateDocument,
 } from "../domain-services.ts";
 import type { OperationDefinition } from "../module-registry.ts";
+import { type OperationInput, optionalBoolean, optionalNumber, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "docs";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 const eventContext = (input: OperationInput) => ({
 	actor: optionalString(input, "actor"),
 	source: optionalString(input, "source"),
@@ -97,7 +77,10 @@ export function docsOperations(artifacts: ArtifactStore, scopes: ArtifactScopeSt
 				eventContext(input),
 			),
 		),
-		define("docs.list", (input: OperationInput) => listDocuments(artifacts, scopes, artifactFilter(input))),
+		define("docs.list", (input: OperationInput) => {
+			const docs = listDocuments(artifacts, scopes, artifactFilter(input));
+			return optionalBoolean(input, "full") === true ? docs : docs.map(summarizeArtifact);
+		}),
 		define("docs.show", (input: OperationInput) => showDocument(artifacts, string(input, "id"))),
 		define("docs.activate", (input: OperationInput) =>
 			transitionDocument(artifacts, string(input, "id"), "activate", authority, eventContext(input)),

package/src/modules/graph-projection.ts

@@ -17,17 +17,10 @@ import type {
 import { GraphProjection } from "../graph-projection/graph-projection-service.ts";
 import type { OperationDefinition } from "../module-registry.ts";
 import type { GraphProjectionStore } from "../stores/graph-projection-store.ts";
+import { type OperationInput, string } from "./operation-input.ts";
 
 const MODULE_ID = "graph_projection";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
 function number(input: OperationInput, key: string): number {
 	const value = input[key];
 	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} is required and must be a number`);

package/src/modules/logs.ts

@@ -8,31 +8,10 @@
 import type { JsonValue, LogLevel } from "../domain/log-entry.ts";
 import type { Logs } from "../log/log-service.ts";
 import type { OperationDefinition } from "../module-registry.ts";
+import { type OperationInput, optionalNumber, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "logs";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 function isJsonValue(value: unknown): value is JsonValue {
 	return (
 		value === null ||

package/src/modules/notes.ts

@@ -16,31 +16,10 @@
 import type { NoteEventDirection } from "../domain/note-event.ts";
 import type { OperationDefinition } from "../module-registry.ts";
 import type { NoteDisposition, Notes } from "../note/note-service.ts";
+import { type OperationInput, optionalNumber, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "notes";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 /** This module's own operation names, the single source of truth src/service.ts's EXPECTED_OPERATION_NAMES spreads in rather than re-listing by hand. */
 export const NOTES_OPERATION_NAMES = [
 	"notes.capture",

package/src/modules/operation-input.ts

@@ -0,0 +1,42 @@
+/**
+ * Shared runtime validators for a Vehicle operation's raw `input` object.
+ * A Vehicle operation's own JSON-schema `input` field is descriptive metadata
+ * only (see handlers/shared.ts's looseObjectSchema) -- never itself enforced
+ * at runtime -- so every module's own operation body narrows `unknown` to
+ * the real shape it needs through these.
+ */
+export type OperationInput = Record<string, unknown>;
+
+export function string(input: OperationInput, key: string): string {
+	const value = input[key];
+	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
+	return value;
+}
+
+export function optionalString(input: OperationInput, key: string): string | undefined {
+	const value = input[key];
+	if (value === undefined) return undefined;
+	if (typeof value !== "string") throw new Error(`${key} must be a string`);
+	return value;
+}
+
+export function optionalStringArray(input: OperationInput, key: string): string[] | undefined {
+	const value = input[key];
+	if (value === undefined) return undefined;
+	if (!Array.isArray(value) || value.some((entry) => typeof entry !== "string")) throw new Error(`${key} must be an array of strings`);
+	return value as string[];
+}
+
+export function optionalNumber(input: OperationInput, key: string): number | undefined {
+	const value = input[key];
+	if (value === undefined) return undefined;
+	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
+	return value;
+}
+
+export function optionalBoolean(input: OperationInput, key: string): boolean | undefined {
+	const value = input[key];
+	if (value === undefined) return undefined;
+	if (typeof value !== "boolean") throw new Error(`${key} must be a boolean`);
+	return value;
+}

package/src/modules/playbooks.ts

@@ -10,6 +10,7 @@
  * playbook/playbook-definition.ts for the full rationale.
  */
 
+import { summarizeArtifact } from "../artifact/artifact.ts";
 import type { ArtifactScopeStore } from "../artifact/artifact-scope-store.ts";
 import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import {
@@ -31,31 +32,10 @@ import type { SessionIdentity } from "../session-identity/session-identity-servi
 import type { TaskEventStore } from "../stores/task-event-store.ts";
 import type { TaskScopeStore } from "../stores/task-scope-store.ts";
 import type { Tasks } from "../task/task-service.ts";
+import { type OperationInput, optionalBoolean, optionalNumber, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "playbooks";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 const eventContext = (input: OperationInput) => ({
 	actor: optionalString(input, "actor"),
 	source: optionalString(input, "source"),
@@ -135,7 +115,10 @@ export function playbooksOperations({
 				eventContext(input),
 			),
 		),
-		define("playbooks.list", (input: OperationInput) => listPlaybooks(artifacts, artifactScopes, artifactFilter(input))),
+		define("playbooks.list", (input: OperationInput) => {
+			const playbooks = listPlaybooks(artifacts, artifactScopes, artifactFilter(input));
+			return optionalBoolean(input, "full") === true ? playbooks : playbooks.map(summarizeArtifact);
+		}),
 		define("playbooks.show", (input: OperationInput) => showPlaybook(artifacts, string(input, "id"))),
 		define("playbooks.preview", (input: OperationInput) =>
 			playbookInvocation(artifacts, string(input, "id"), input.arguments as Record<string, unknown> | undefined),

package/src/modules/rules.ts

@@ -11,6 +11,7 @@
  * registry" convention.
  */
 
+import { summarizeArtifact } from "../artifact/artifact.ts";
 import type { ArtifactScopeStore } from "../artifact/artifact-scope-store.ts";
 import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import {
@@ -24,31 +25,10 @@ import {
 	updateRule,
 } from "../domain-services.ts";
 import type { OperationDefinition } from "../module-registry.ts";
+import { type OperationInput, optionalBoolean, optionalNumber, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "rules";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 const eventContext = (input: OperationInput) => ({
 	actor: optionalString(input, "actor"),
 	source: optionalString(input, "source"),
@@ -100,7 +80,10 @@ export function rulesOperations(artifacts: ArtifactStore, scopes: ArtifactScopeS
 				eventContext(input),
 			),
 		),
-		define("rules.list", (input: OperationInput) => listRules(artifacts, scopes, artifactFilter(input))),
+		define("rules.list", (input: OperationInput) => {
+			const rules = listRules(artifacts, scopes, artifactFilter(input));
+			return optionalBoolean(input, "full") === true ? rules : rules.map(summarizeArtifact);
+		}),
 		define("rules.show", (input: OperationInput) => showRule(artifacts, string(input, "id"))),
 		define("rules.preview", (input: OperationInput) => previewRule(artifacts, string(input, "id"))),
 		define("rules.enable", (input: OperationInput) => transitionRule(artifacts, string(input, "id"), "enable", eventContext(input))),

package/src/modules/session-identity.ts

@@ -5,24 +5,10 @@
  */
 import type { OperationDefinition } from "../module-registry.ts";
 import type { SessionIdentity } from "../session-identity/session-identity-service.ts";
+import { type OperationInput, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "session-identity";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
 /** This module's own operation names, the single source of truth src/service.ts's EXPECTED_OPERATION_NAMES spreads in rather than re-listing by hand. */
 export const SESSION_IDENTITY_OPERATION_NAMES = ["session.register", "session.release"] as const;
 

package/src/modules/tasks.ts

@@ -13,10 +13,10 @@
  *
  * Task-domain-internal files (task-context.ts, task-execution.ts, domain/task-event.ts,
  * domain/task-scope.ts) are imported directly — they belong to this bounded context,
- * unlike a different module's infrastructure. Generic input-parsing helpers are
- * duplicated locally rather than imported from src/service.ts, matching the precedent
- * set by modules/notes.ts: a module does not import another module's infrastructure,
- * including the composition root's own helpers.
+ * unlike a different module's infrastructure. Generic input-parsing helpers come from
+ * ./operation-input.ts, a peer utility at this same modules/ layer with no domain logic
+ * of its own -- not "another module's infrastructure" or the composition root's helpers,
+ * the actual thing this module avoids importing.
  */
 
 import type { ArtifactStore } from "../artifact/artifact-store.ts";
@@ -28,38 +28,10 @@ import type { SessionIdentity } from "../session-identity/session-identity-servi
 import { taskContext } from "../task/task-context.ts";
 import { projectTaskExecution } from "../task/task-execution.ts";
 import type { TaskStatus, Tasks } from "../task/task-service.ts";
+import { type OperationInput, optionalNumber, optionalString, optionalStringArray, string } from "./operation-input.ts";
 
 const MODULE_ID = "tasks";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalStringArray(input: OperationInput, key: string): string[] | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (!Array.isArray(value) || value.some((entry) => typeof entry !== "string")) throw new Error(`${key} must be an array of strings`);
-	return value as string[];
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 const eventContext = (input: OperationInput): TaskEventContext => ({
 	actor: optionalString(input, "actor"),
 	source: optionalString(input, "source"),