@danypops/papyrus 0.42.0 → 0.42.2

package/src/modules/tasks.ts

@@ -26,7 +26,7 @@ import type { ArtifactStore } from "../ports/artifact-store.ts";
 import type { SessionIdentity } from "../session-identity-service.ts";
 import { taskContext } from "../task-context.ts";
 import { projectTaskExecution } from "../task-execution.ts";
-import { Tasks, type TaskStatus } from "../task-service.ts";
+import type { TaskStatus, Tasks } from "../task-service.ts";
 
 const MODULE_ID = "tasks";
 
@@ -87,17 +87,51 @@ const taskFilter = (input: OperationInput) => ({
  */
 /** 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 TASKS_OPERATION_NAMES = [
-	"tasks.create", "tasks.update", "tasks.list", "tasks.graph", "tasks.plan", "tasks.show", "tasks.history",
-	"tasks.scope", "tasks.set_scope", "tasks.assign_project", "tasks.active", "tasks.focused", "tasks.focus",
-	"tasks.pause", "tasks.unpause", "tasks.clear_focus", "tasks.start", "tasks.submit", "tasks.complete",
-	"tasks.run_gates", "tasks.set_checklist", "tasks.set_gates", "tasks.context", "tasks.reject", "tasks.retry", "tasks.cancel", "tasks.cancel_subtree",
-	"tasks.depend", "tasks.undepend", "tasks.contain", "tasks.uncontain", "tasks.reap_stale_focus",
-	"tasks.claim", "tasks.heartbeat_lease", "tasks.release_lease", "tasks.lease", "tasks.reap_stale_leases", "tasks.event_feed",
+	"tasks.create",
+	"tasks.update",
+	"tasks.list",
+	"tasks.graph",
+	"tasks.plan",
+	"tasks.show",
+	"tasks.history",
+	"tasks.scope",
+	"tasks.set_scope",
+	"tasks.assign_project",
+	"tasks.active",
+	"tasks.focused",
+	"tasks.focus",
+	"tasks.pause",
+	"tasks.unpause",
+	"tasks.clear_focus",
+	"tasks.start",
+	"tasks.submit",
+	"tasks.complete",
+	"tasks.run_gates",
+	"tasks.set_checklist",
+	"tasks.set_gates",
+	"tasks.context",
+	"tasks.reject",
+	"tasks.retry",
+	"tasks.cancel",
+	"tasks.cancel_subtree",
+	"tasks.depend",
+	"tasks.undepend",
+	"tasks.contain",
+	"tasks.uncontain",
+	"tasks.reap_stale_focus",
+	"tasks.claim",
+	"tasks.heartbeat_lease",
+	"tasks.release_lease",
+	"tasks.lease",
+	"tasks.reap_stale_leases",
+	"tasks.event_feed",
 ] as const;
 
 export function tasksOperations(tasks: Tasks, artifacts: ArtifactStore, sessionIdentity: SessionIdentity): OperationDefinition[] {
 	const define = <Input, Output>(name: string, execute: (input: Input) => Output): OperationDefinition<Input, Output> => ({
-		name, moduleId: MODULE_ID, execute,
+		name,
+		moduleId: MODULE_ID,
+		execute,
 	});
 	// Enforced only for the operations where session_id is BEHAVIOR-affecting (it selects
 	// which Task Focus row is mutated), not for every session_id-carrying operation -- see
@@ -107,82 +141,123 @@ export function tasksOperations(tasks: Tasks, artifacts: ArtifactStore, sessionI
 		sessionIdentity.assertAuthorized(eventContext(input).sessionId, optionalString(input, "session_secret"));
 	};
 	return [
-		define("tasks.create", (input: OperationInput) => tasks.create({
-			title: string(input, "title"),
-			body: optionalString(input, "body"),
-			status: optionalString(input, "status") as TaskStatus | undefined,
-			labels: input["labels"] as string[] | undefined,
-			extra: input["extra"] as Record<string, unknown> | undefined,
-			gates: input["gates"] as Parameters<Tasks["create"]>[0]["gates"],
-			checklist: input["checklist"] as Checklist | undefined,
-			templateId: optionalString(input, "template_id") ?? optionalString(input, "templateId"),
-			parentId: optionalString(input, "parent_id") ?? optionalString(input, "parentId"),
-			dependsOn: (input["depends_on"] ?? input["dependsOn"]) as string[] | undefined,
-			projectRoot: string(input, "project_root"),
-			projectSource: "cwd",
-		}, eventContext(input))),
-		define("tasks.update", (input: OperationInput) => tasks.update(string(input, "id"), {
-			...(input["title"] !== undefined ? { title: optionalString(input, "title")! } : {}),
-			...(input["body"] !== undefined ? { body: optionalString(input, "body")! } : {}),
-			...(input["labels"] !== undefined ? { labels: optionalStringArray(input, "labels")! } : {}),
-			...(input["status"] !== undefined ? { status: string(input, "status") as "todo" } : {}),
-		}, eventContext(input))),
+		define("tasks.create", (input: OperationInput) =>
+			tasks.create(
+				{
+					title: string(input, "title"),
+					body: optionalString(input, "body"),
+					status: optionalString(input, "status") as TaskStatus | undefined,
+					labels: input.labels as string[] | undefined,
+					extra: input.extra as Record<string, unknown> | undefined,
+					gates: input.gates as Parameters<Tasks["create"]>[0]["gates"],
+					checklist: input.checklist as Checklist | undefined,
+					templateId: optionalString(input, "template_id") ?? optionalString(input, "templateId"),
+					parentId: optionalString(input, "parent_id") ?? optionalString(input, "parentId"),
+					dependsOn: (input.depends_on ?? input.dependsOn) as string[] | undefined,
+					projectRoot: string(input, "project_root"),
+					projectSource: "cwd",
+				},
+				eventContext(input),
+			),
+		),
+		define("tasks.update", (input: OperationInput) =>
+			tasks.update(
+				string(input, "id"),
+				{
+					...(input.title !== undefined ? { title: optionalString(input, "title")! } : {}),
+					...(input.body !== undefined ? { body: optionalString(input, "body")! } : {}),
+					...(input.labels !== undefined ? { labels: optionalStringArray(input, "labels")! } : {}),
+					...(input.status !== undefined ? { status: string(input, "status") as "todo" } : {}),
+				},
+				eventContext(input),
+			),
+		),
 		define("tasks.list", (input: OperationInput) => tasks.list(taskFilter(input))),
 		define("tasks.graph", (input: OperationInput) => tasks.graph(taskFilter(input))),
 		define("tasks.plan", (input: OperationInput) => projectTaskExecution(tasks.graph(taskFilter(input)))),
 		define("tasks.show", (input: OperationInput) => tasks.show(string(input, "id"))),
-		define("tasks.history", (input: OperationInput) => tasks.history(string(input, "id"), {
-			limit: optionalNumber(input, "limit"),
-			cursor: optionalNumber(input, "cursor"),
-			direction: optionalString(input, "direction") as TaskEventDirection | undefined,
-		})),
+		define("tasks.history", (input: OperationInput) =>
+			tasks.history(string(input, "id"), {
+				limit: optionalNumber(input, "limit"),
+				cursor: optionalNumber(input, "cursor"),
+				direction: optionalString(input, "direction") as TaskEventDirection | undefined,
+			}),
+		),
 		define("tasks.scope", (input: OperationInput) => tasks.scopeSelection(string(input, "project_root"))),
-		define("tasks.set_scope", (input: OperationInput) => tasks.setView(
-			string(input, "project_root"),
-			string(input, "scope") as TaskViewMode,
-			optionalString(input, "root_task_id"),
-		)),
-		define("tasks.assign_project", (input: OperationInput) => tasks.assignProject(
-			string(input, "id"),
-			string(input, "project_root"),
-			eventContext(input),
-		)),
+		define("tasks.set_scope", (input: OperationInput) =>
+			tasks.setView(string(input, "project_root"), string(input, "scope") as TaskViewMode, optionalString(input, "root_task_id")),
+		),
+		define("tasks.assign_project", (input: OperationInput) =>
+			tasks.assignProject(string(input, "id"), string(input, "project_root"), eventContext(input)),
+		),
 		define("tasks.active", (input: OperationInput) => tasks.active(taskFilter(input))),
 		define("tasks.focused", (input: OperationInput) => tasks.focused(taskFilter(input))),
-		define("tasks.focus", (input: OperationInput) => { guardFocusMutation(input); return tasks.focus(string(input, "id"), eventContext(input)); }),
-		define("tasks.pause", (input: OperationInput) => { guardFocusMutation(input); return tasks.pauseFocus(eventContext(input)); }),
-		define("tasks.unpause", (input: OperationInput) => { guardFocusMutation(input); return tasks.unpauseFocus(eventContext(input)); }),
-		define("tasks.clear_focus", (input: OperationInput) => { guardFocusMutation(input); return tasks.clearFocus(eventContext(input)); }),
+		define("tasks.focus", (input: OperationInput) => {
+			guardFocusMutation(input);
+			return tasks.focus(string(input, "id"), eventContext(input));
+		}),
+		define("tasks.pause", (input: OperationInput) => {
+			guardFocusMutation(input);
+			return tasks.pauseFocus(eventContext(input));
+		}),
+		define("tasks.unpause", (input: OperationInput) => {
+			guardFocusMutation(input);
+			return tasks.unpauseFocus(eventContext(input));
+		}),
+		define("tasks.clear_focus", (input: OperationInput) => {
+			guardFocusMutation(input);
+			return tasks.clearFocus(eventContext(input));
+		}),
 		define("tasks.reap_stale_focus", () => ({ removed: tasks.reapStaleFocus() })),
 		define("tasks.start", (input: OperationInput) => tasks.transition(string(input, "id"), "start", eventContext(input))),
 		define("tasks.submit", (input: OperationInput) => tasks.transition(string(input, "id"), "submit", eventContext(input))),
 		define("tasks.complete", (input: OperationInput) => tasks.completeAsync(string(input, "id"), eventContext(input))),
 		define("tasks.run_gates", (input: OperationInput) => tasks.runGates(string(input, "id"), eventContext(input))),
-		define("tasks.set_checklist", (input: OperationInput) => tasks.setChecklist(string(input, "id"), input["checklist"] as Checklist)),
-		define("tasks.set_gates", (input: OperationInput) => tasks.setGates(string(input, "id"), input["gates"] as Parameters<Tasks["setGates"]>[1])),
-		define("tasks.context", (input: OperationInput) => taskContext(
-			artifacts,
-			tasks.active(taskFilter(input))?.id,
-			new Set(tasks.list(taskFilter(input)).map((task) => task.id)),
-			optionalString(input, "verbosity") === "summary" ? "summary" : "full",
-		)),
+		define("tasks.set_checklist", (input: OperationInput) => tasks.setChecklist(string(input, "id"), input.checklist as Checklist)),
+		define("tasks.set_gates", (input: OperationInput) =>
+			tasks.setGates(string(input, "id"), input.gates as Parameters<Tasks["setGates"]>[1]),
+		),
+		define("tasks.context", (input: OperationInput) =>
+			taskContext(
+				artifacts,
+				tasks.active(taskFilter(input))?.id,
+				new Set(tasks.list(taskFilter(input)).map((task) => task.id)),
+				optionalString(input, "verbosity") === "summary" ? "summary" : "full",
+			),
+		),
 		define("tasks.reject", (input: OperationInput) => tasks.transition(string(input, "id"), "reject", eventContext(input))),
 		define("tasks.retry", (input: OperationInput) => tasks.transition(string(input, "id"), "retry", eventContext(input))),
 		define("tasks.cancel", (input: OperationInput) => tasks.transition(string(input, "id"), "cancel", eventContext(input))),
 		define("tasks.cancel_subtree", (input: OperationInput) => tasks.cancelSubtree(string(input, "id"), eventContext(input))),
-		define("tasks.depend", (input: OperationInput) => tasks.depend(string(input, "id"), string(input, "dependency_id"), eventContext(input))),
-		define("tasks.undepend", (input: OperationInput) => tasks.undepend(string(input, "id"), string(input, "dependency_id"), eventContext(input))),
-		define("tasks.contain", (input: OperationInput) => tasks.contain(string(input, "parent_id"), string(input, "child_id"), eventContext(input))),
-		define("tasks.uncontain", (input: OperationInput) => tasks.uncontain(string(input, "parent_id"), string(input, "child_id"), eventContext(input))),
-		define("tasks.claim", (input: OperationInput) => tasks.claimLease(string(input, "id"), string(input, "owner"), optionalNumber(input, "ttl_ms"), optionalString(input, "note"))),
-		define("tasks.heartbeat_lease", (input: OperationInput) => tasks.heartbeatLease(string(input, "id"), string(input, "owner"), string(input, "token"), optionalNumber(input, "ttl_ms"))),
-		define("tasks.release_lease", (input: OperationInput) => tasks.releaseLease(string(input, "id"), string(input, "owner"), string(input, "token"))),
+		define("tasks.depend", (input: OperationInput) =>
+			tasks.depend(string(input, "id"), string(input, "dependency_id"), eventContext(input)),
+		),
+		define("tasks.undepend", (input: OperationInput) =>
+			tasks.undepend(string(input, "id"), string(input, "dependency_id"), eventContext(input)),
+		),
+		define("tasks.contain", (input: OperationInput) =>
+			tasks.contain(string(input, "parent_id"), string(input, "child_id"), eventContext(input)),
+		),
+		define("tasks.uncontain", (input: OperationInput) =>
+			tasks.uncontain(string(input, "parent_id"), string(input, "child_id"), eventContext(input)),
+		),
+		define("tasks.claim", (input: OperationInput) =>
+			tasks.claimLease(string(input, "id"), string(input, "owner"), optionalNumber(input, "ttl_ms"), optionalString(input, "note")),
+		),
+		define("tasks.heartbeat_lease", (input: OperationInput) =>
+			tasks.heartbeatLease(string(input, "id"), string(input, "owner"), string(input, "token"), optionalNumber(input, "ttl_ms")),
+		),
+		define("tasks.release_lease", (input: OperationInput) =>
+			tasks.releaseLease(string(input, "id"), string(input, "owner"), string(input, "token")),
+		),
 		define("tasks.lease", (input: OperationInput) => tasks.getLease(string(input, "id")) ?? null),
 		define("tasks.reap_stale_leases", () => ({ removed: tasks.reapStaleLeases() })),
-		define("tasks.event_feed", (input: OperationInput) => tasks.eventFeed({
-			cursor: optionalNumber(input, "cursor"),
-			limit: optionalNumber(input, "limit"),
-			eventTypes: optionalStringArray(input, "event_types") as TaskEventFeedQuery["eventTypes"],
-		})),
+		define("tasks.event_feed", (input: OperationInput) =>
+			tasks.eventFeed({
+				cursor: optionalNumber(input, "cursor"),
+				limit: optionalNumber(input, "limit"),
+				eventTypes: optionalStringArray(input, "event_types") as TaskEventFeedQuery["eventTypes"],
+			}),
+		),
 	];
 }

package/src/note-service.ts

@@ -9,13 +9,13 @@ import {
 } from "./constants.ts";
 import type { Artifact } from "./domain/artifact.ts";
 import type { AppendNoteEvent, NoteEventType, NoteHistoryPage, NoteHistoryQuery } from "./domain/note-event.ts";
-import { InMemoryNoteEventStore, type NoteEventStore } from "./ports/note-event-store.ts";
-import { requireAtomicArtifactStore } from "./ports/atomic-artifact-store.ts";
 import type { ArtifactStore } from "./ports/artifact-store.ts";
+import { requireAtomicArtifactStore } from "./ports/atomic-artifact-store.ts";
+import { InMemoryNoteEventStore, type NoteEventStore } from "./ports/note-event-store.ts";
 
 export const NOTE_SUBTYPE = "note";
 export const NOTE_DISPOSITIONS = ["completed", "duplicate", "declined", "superseded"] as const;
-export type NoteDisposition = typeof NOTE_DISPOSITIONS[number];
+export type NoteDisposition = (typeof NOTE_DISPOSITIONS)[number];
 
 export interface NoteProvenance {
 	actor?: string;
@@ -60,7 +60,10 @@ function noteTitle(body: string, requested?: string): string {
 	return firstLine.slice(0, NOTE_TITLE_MAX_CHARACTERS) || "Deferred note";
 }
 
-function provenance(input: NoteProvenance, defaults: { actor: string; source: string }): Omit<AppendNoteEvent, "noteId" | "type" | "relatedId" | "disposition"> {
+function provenance(
+	input: NoteProvenance,
+	defaults: { actor: string; source: string },
+): Omit<AppendNoteEvent, "noteId" | "type" | "relatedId" | "disposition"> {
 	return {
 		actor: optionalBounded(input.actor, "note actor", NOTE_PROVENANCE_MAX_LENGTH) ?? defaults.actor,
 		source: optionalBounded(input.source, "note source", NOTE_PROVENANCE_MAX_LENGTH) ?? defaults.source,
@@ -154,7 +157,8 @@ export class Notes {
 	}
 
 	archive(id: string, input: ArchiveNoteInput): Artifact {
-		if (!NOTE_DISPOSITIONS.includes(input.disposition)) throw new Error("note disposition must be completed, duplicate, declined, or superseded");
+		if (!NOTE_DISPOSITIONS.includes(input.disposition))
+			throw new Error("note disposition must be completed, duplicate, declined, or superseded");
 		const atomic = requireAtomicArtifactStore(this.artifacts);
 		return atomic.atomic(() => {
 			const note = this.requireNote(id);
@@ -183,12 +187,12 @@ export class Notes {
 
 	private requireNote(id: string): Artifact {
 		const artifact = this.artifacts.get(id);
-		if (!artifact || artifact.kind !== "doc" || artifact.subtype !== NOTE_SUBTYPE) throw new Error(`note "${id}" not found`);
+		if (artifact?.kind !== "doc" || artifact.subtype !== NOTE_SUBTYPE) throw new Error(`note "${id}" not found`);
 		return artifact;
 	}
 
 	private requireProject(note: Artifact, projectRoot: string): void {
 		const requested = requiredBounded(projectRoot, "project_root", TASK_PROJECT_ROOT_MAX_LENGTH);
-		if (note.extra["projectRoot"] !== requested) throw new Error(`note "${note.id}" is outside project scope`);
+		if (note.extra.projectRoot !== requested) throw new Error(`note "${note.id}" is outside project scope`);
 	}
 }