@danypops/papyrus 0.21.1 → 0.21.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/papyrus",
-  "version": "0.21.1",
+  "version": "0.21.2",
   "description": "Daemon-backed graph artifacts, evidence-bearing tasks, rules, skills, and native TUI workflows for Pi",
   "type": "module",
   "keywords": ["pi-package"],

package/src/cli.ts

@@ -157,6 +157,7 @@ const USAGE = `Usage:
   papyrus tasks show <id> [--json]
   papyrus tasks run-gates <id> [--json]
   papyrus tasks set-checklist <id> --checklist-json <json> [--json]
+  papyrus tasks set-gates <id> --gates-json <json> [--json]
   papyrus tasks context [--scope <project|graph|all>] [--root-task-id <id>] [--session-id <id>] [--json]
 
 A "--session-id" scopes Task Focus to one agent session; omit it to use the shared "global" Focus (today's behavior).
@@ -1322,6 +1323,14 @@ export async function runTaskCli(args: string[], client: TaskCliClient, projectR
 			human = `Updated checklist: ${artifactLabel(artifact)}`;
 			break;
 		}
+		case "set-gates": {
+			if (!id || dependencyId) throw new Error("tasks set-gates requires exactly one task id");
+			if (!gates) throw new Error("tasks set-gates requires --gates-json");
+			const artifact = await client.call<Record<string, unknown>, CliArtifact>("tasks.set_gates", { id, gates });
+			result = artifact;
+			human = `Updated gates: ${artifactLabel(artifact)}`;
+			break;
+		}
 		case "context": {
 			if (id) throw new Error("tasks context accepts no positional arguments");
 			const summary = await client.call<Record<string, unknown>, string | null>("tasks.context", {
@@ -1478,7 +1487,7 @@ export async function runTaskCli(args: string[], client: TaskCliClient, projectR
 			break;
 		}
 		default:
-			throw new Error("tasks action must be create, list, show, active, focused, focus, pause, unpause, clear-focus, update, graph, plan, context, history, scope, assign-project, complete, start, submit, reject, retry, cancel, depend, undepend, contain, uncontain, run-gates, or set-checklist");
+			throw new Error("tasks action must be create, list, show, active, focused, focus, pause, unpause, clear-focus, update, graph, plan, context, history, scope, assign-project, complete, start, submit, reject, retry, cancel, depend, undepend, contain, uncontain, run-gates, set-checklist, or set-gates");
 	}
 	return json ? JSON.stringify(result) : human;
 }

package/src/domain/gate.ts

@@ -1,9 +1,42 @@
+export const GATE_TYPES = ["file-exists", "command", "contains", "test"] as const;
+export type GateType = typeof GATE_TYPES[number];
+
 export interface Gate {
-	type: "file-exists" | "command" | "contains" | "test";
+	type: GateType;
 	target: string;
 	expect?: string;
 }
 
+function isRecord(value: unknown): value is Record<string, unknown> {
+	return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+
+/**
+ * Validates a Gate[] the same way validateChecklist validates a Checklist -- gates previously had
+ * no validation at all (create() assigned `input.gates` to extra verbatim), and no way to change
+ * them after creation except by re-typing the whole task, silently accepted by "tasks update"
+ * (which only ever reads title/body/labels/status) as if it had worked. See Tasks.setGates.
+ */
+export function validateGates(value: unknown): Gate[] {
+	if (!Array.isArray(value)) throw new Error("gates must be an array");
+	return value.map((entry, index) => {
+		if (!isRecord(entry) || !GATE_TYPES.includes(entry["type"] as GateType)) {
+			throw new Error(`gate at index ${index} requires a valid type (${GATE_TYPES.join(", ")})`);
+		}
+		if (typeof entry["target"] !== "string" || entry["target"].trim().length === 0) {
+			throw new Error(`gate at index ${index} requires a non-empty target`);
+		}
+		if (entry["expect"] !== undefined && typeof entry["expect"] !== "string") {
+			throw new Error(`gate at index ${index} expect must be a string`);
+		}
+		return {
+			type: entry["type"] as GateType,
+			target: entry["target"],
+			...(typeof entry["expect"] === "string" ? { expect: entry["expect"] } : {}),
+		};
+	});
+}
+
 export interface GateRunOptions {
 	/** Absolute Unix epoch deadline for the full gate sequence. */
 	deadlineMs?: number;

package/src/modules/tasks.ts

@@ -89,7 +89,7 @@ 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.context", "tasks.reject", "tasks.retry", "tasks.cancel",
+	"tasks.run_gates", "tasks.set_checklist", "tasks.set_gates", "tasks.context", "tasks.reject", "tasks.retry", "tasks.cancel",
 	"tasks.depend", "tasks.undepend", "tasks.contain", "tasks.uncontain", "tasks.reap_stale_focus",
 ] as const;
 
@@ -157,6 +157,7 @@ export function tasksOperations(tasks: Tasks, artifacts: ArtifactStore, sessionI
 		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,

package/src/service.ts

@@ -323,6 +323,7 @@ function handlers(
 		"tasks.complete": forwardToModule("tasks.complete"),
 		"tasks.run_gates": forwardToModule("tasks.run_gates"),
 		"tasks.set_checklist": forwardToModule("tasks.set_checklist"),
+		"tasks.set_gates": forwardToModule("tasks.set_gates"),
 		"tasks.context": forwardToModule("tasks.context"),
 		"tasks.reject": forwardToModule("tasks.reject"),
 		"tasks.retry": forwardToModule("tasks.retry"),

package/src/task-service.ts

@@ -12,7 +12,7 @@ import {
 import type { Artifact } from "./domain/artifact.ts";
 import { checklistEntries, validateChecklist, type Checklist, type ProofReference } from "./domain/checklist.ts";
 import { isDiscussionArtifact, readDiscussionExtra } from "./domain/discussion.ts";
-import type { Gate, GateResult } from "./domain/gate.ts";
+import { validateGates, type Gate, type GateResult } from "./domain/gate.ts";
 import type { AppendTaskEvent, TaskEventContext, TaskHistoryPage, TaskHistoryQuery, TaskLifecycleStatus } from "./domain/task-event.ts";
 import { normalizeProjectRoot, taskScopeLabel, type TaskScopeSource, type TaskViewMode, type TaskViewSelection } from "./domain/task-scope.ts";
 import type { ArtifactStore } from "./ports/artifact-store.ts";
@@ -141,7 +141,7 @@ export class Tasks {
 			if (input.parentId) this.require(input.parentId);
 			for (const dependency of input.dependsOn ?? []) this.require(dependency);
 			const extra: Record<string, unknown> = { ...(input.extra ?? {}) };
-			if (input.gates !== undefined) extra["gates"] = input.gates;
+			if (input.gates !== undefined) extra["gates"] = validateGates(input.gates);
 			if (input.checklist !== undefined) extra["checklist"] = validateChecklist(input.checklist);
 			const projectRoot = input.projectRoot === undefined ? undefined : normalizeProjectRoot(input.projectRoot);
 			if (input.parentId && this.scopes.get(input.parentId)?.projectRoot !== projectRoot) {
@@ -459,6 +459,16 @@ export class Tasks {
 		return this.artifacts.setExtra(id, { ...task.extra, checklist: validateChecklist(checklist) })!;
 	}
 
+	/**
+	 * The only way to change a task's gates after creation. "tasks update" (title/body/labels/
+	 * status only) silently ignored a `gates` field with no error at all -- a real incident (see
+	 * GateRunOptions.cwd's doc comment for the crash this masked while debugging).
+	 */
+	setGates(id: string, gates: Gate[]): Artifact {
+		const task = this.require(id);
+		return this.artifacts.setExtra(id, { ...task.extra, gates: validateGates(gates) })!;
+	}
+
 	depend(id: string, dependencyId: string, context: TaskEventContext = {}): Artifact {
 		return this.events.atomic(() => {
 			this.require(id);