@danypops/papyrus 0.29.8 → 0.30.1

package/package.json

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

package/src/adapters/sqlite-artifact-store.ts

@@ -1,6 +1,8 @@
 import type { Db } from "../db.ts";
 import { inTransaction } from "../db.ts";
 import type { AtomicArtifactStore } from "../ports/atomic-artifact-store.ts";
+import type { ArtifactEventReader } from "../ports/artifact-event-reader.ts";
+import type { ArtifactTrashStore } from "../ports/artifact-trash-store.ts";
 import type {
 	Artifact,
 	ArtifactEdge,
@@ -30,7 +32,7 @@ import {
 	updateStatus,
 } from "../ops.ts";
 
-export class SQLiteArtifactStore implements AtomicArtifactStore {
+export class SQLiteArtifactStore implements AtomicArtifactStore, ArtifactTrashStore, ArtifactEventReader {
 	constructor(private readonly db: Db) {}
 
 	atomic<T>(operation: () => T): T {

package/src/cli.ts

@@ -1441,7 +1441,7 @@ export async function runTaskCli(args: string[], client: TaskCliClient, projectR
 		case "list": {
 			if (id) throw new Error("tasks list accepts no positional arguments");
 			const rows = await client.call<Record<string, unknown>, CliArtifact[]>("tasks.list", {
-				status, text, limit, project_root: projectRoot, scope: listScope, root_task_id: rootTaskId, ...sessionScope,
+				status, text, limit, labels, project_root: projectRoot, scope: listScope, root_task_id: rootTaskId, ...sessionScope,
 			});
 			result = rows;
 			human = rows.length === 0 ? "No tasks found." : rows.map((row) => artifactLabel(row)).join("\n");
@@ -1571,7 +1571,7 @@ export async function runTaskCli(args: string[], client: TaskCliClient, projectR
 			const graph = await client.call<Record<string, unknown>, {
 				nodes: Array<{ dependencyIds: string[]; childIds: string[] }>;
 				rootIds: string[];
-			}>("tasks.graph", { limit: TASK_EXECUTION_MAX_NODES + 1, project_root: projectRoot, ...sessionScope });
+			}>("tasks.graph", { limit: TASK_EXECUTION_MAX_NODES + 1, labels, project_root: projectRoot, scope: listScope, root_task_id: rootTaskId, ...sessionScope });
 			result = graph;
 			const dependencies = graph.nodes.reduce((count, node) => count + node.dependencyIds.length, 0);
 			const children = graph.nodes.reduce((count, node) => count + node.childIds.length, 0);

package/src/modules/tasks.ts

@@ -74,6 +74,7 @@ const taskFilter = (input: OperationInput) => ({
 	scope: optionalString(input, "scope") as TaskViewMode | undefined,
 	rootTaskId: optionalString(input, "root_task_id"),
 	sessionId: optionalString(input, "session_id") ?? optionalString(input, "sessionId"),
+	labels: optionalStringArray(input, "labels"),
 });
 
 /**

package/src/ports/artifact-event-reader.ts

@@ -0,0 +1,8 @@
+import type { ArtifactEventPage, ArtifactEventQuery } from "../domain/artifact-event.ts";
+
+/** Read access to the generic mutation event log shared by every kind -- split out of
+ * ArtifactStore since only service.ts's graph.history operation reads it. */
+export interface ArtifactEventReader {
+	/** Bounded query over the generic mutation event log shared by every kind. */
+	events(query: ArtifactEventQuery): ArtifactEventPage;
+}

package/src/ports/artifact-store.ts

@@ -8,9 +8,11 @@ import type {
 	RelationshipQuery,
 	UpdateArtifactInput,
 } from "../domain/artifact.ts";
-import type { ArtifactEventContext, ArtifactEventPage, ArtifactEventQuery } from "../domain/artifact-event.ts";
-import type { ArtifactTrashRecord } from "../domain/artifact-trash.ts";
+import type { ArtifactEventContext } from "../domain/artifact-event.ts";
 
+/** Core CRUD/graph operations every domain-service module needs. Trash lifecycle and event-log
+ * reading are separate ports (ArtifactTrashStore, ArtifactEventReader) -- only the composition
+ * root (daemon.ts, service.ts) needs those, not every consumer of this store. */
 export interface ArtifactStore {
 	create(input: CreateArtifactInput, context?: ArtifactEventContext): Artifact;
 	get(id: string, options?: ArtifactGraphOptions): Artifact | null;
@@ -22,14 +24,4 @@ export interface ArtifactStore {
 	setExtra(id: string, extra: Record<string, unknown>, context?: ArtifactEventContext): Artifact | null;
 	updateContent(id: string, input: UpdateArtifactInput, context?: ArtifactEventContext): Artifact | null;
 	relationships(filter?: RelationshipQuery): ArtifactEdge[];
-	/** Bounded query over the generic mutation event log shared by every kind. */
-	events(query: ArtifactEventQuery): ArtifactEventPage;
-	/** See domain/artifact-trash.ts. Moves an artifact to the trash; throws if it does not exist or is the live Task Focus in any scope. */
-	trash(id: string, options?: { reason?: string; context?: ArtifactEventContext }): ArtifactTrashRecord;
-	/** Idempotent: restoring an artifact that is not currently trashed is a real no-op. */
-	restore(id: string, context?: ArtifactEventContext): { restored: boolean };
-	trashStatus(id: string): ArtifactTrashRecord | null;
-	listTrash(): ArtifactTrashRecord[];
-	/** Real, cascading, irreversible deletion of every artifact past its purge deadline; returns how many were purged. */
-	purgeDueTrash(): number;
 }

package/src/ports/artifact-trash-store.ts

@@ -0,0 +1,15 @@
+import type { ArtifactEventContext } from "../domain/artifact-event.ts";
+import type { ArtifactTrashRecord } from "../domain/artifact-trash.ts";
+
+/** Trash lifecycle for artifacts -- split out of ArtifactStore since only the composition root
+ * (daemon.ts, service.ts) needs it; every domain-service module depends on core CRUD/graph only. */
+export interface ArtifactTrashStore {
+	/** See domain/artifact-trash.ts. Moves an artifact to the trash; throws if it does not exist or is the live Task Focus in any scope. */
+	trash(id: string, options?: { reason?: string; context?: ArtifactEventContext }): ArtifactTrashRecord;
+	/** Idempotent: restoring an artifact that is not currently trashed is a real no-op. */
+	restore(id: string, context?: ArtifactEventContext): { restored: boolean };
+	trashStatus(id: string): ArtifactTrashRecord | null;
+	listTrash(): ArtifactTrashRecord[];
+	/** Real, cascading, irreversible deletion of every artifact past its purge deadline; returns how many were purged. */
+	purgeDueTrash(): number;
+}

package/src/service.ts

@@ -14,6 +14,8 @@ import { AuthorityRegistry, AuthorizedArtifactWriter, type AuthorityClaim } from
 import type { TaskEventContext } from "./domain/task-event.ts";
 import type { TaskViewMode } from "./domain/task-scope.ts";
 import type { ArtifactStore } from "./ports/artifact-store.ts";
+import type { ArtifactEventReader } from "./ports/artifact-event-reader.ts";
+import type { ArtifactTrashStore } from "./ports/artifact-trash-store.ts";
 import type { GateRunner } from "./ports/gate-runner.ts";
 import type { TaskEventStore } from "./ports/task-event-store.ts";
 import type { TaskScopeStore } from "./ports/task-scope-store.ts";
@@ -187,7 +189,10 @@ export interface PapyrusService {
 }
 
 function handlers(
-	artifacts: ArtifactStore,
+	// The composition root's own handler table -- the one place that genuinely needs trash
+	// lifecycle and event-log reading alongside core CRUD/graph, unlike every domain-service
+	// module (which only ever depends on the narrower ArtifactStore).
+	artifacts: ArtifactStore & ArtifactTrashStore & ArtifactEventReader,
 	gates: GateRunner,
 	tasks: Tasks,
 	notes: Notes,

package/src/task-service.ts

@@ -38,6 +38,8 @@ export interface TaskFilter {
 	rootTaskId?: string;
 	/** Requesting agent session id — scopes Task Focus reads so concurrent agents see only their own Focus. Defaults to a shared "global" scope when omitted. */
 	sessionId?: string;
+	/** AND semantics: a task must carry every requested label, matching ArtifactStore.query's own labels filter. */
+	labels?: string[];
 }
 
 export type TaskStatus = TaskLifecycleStatus;
@@ -225,17 +227,19 @@ export class Tasks {
 			throw new Error(`task list limit must be between 1 and ${TASK_SCOPE_MAX_TASKS + 1}`);
 		}
 		if (selection.mode === "all") {
-			return this.artifacts.query({ kind: "task", excludeSubtype: DISCUSSION_SUBTYPE, status: filter.status, text: filter.text, limit });
+			return this.artifacts.query({ kind: "task", excludeSubtype: DISCUSSION_SUBTYPE, status: filter.status, text: filter.text, labels: filter.labels, limit });
 		}
 		const ids = this.scopes.taskIds(selection.projectRoot, TASK_SCOPE_MAX_TASKS + 1);
 		if (ids.length > TASK_SCOPE_MAX_TASKS) throw new Error(`task project scope exceeds ${TASK_SCOPE_MAX_TASKS} tasks`);
 		const selectedIds = selection.mode === "graph" ? this.descendantIds(selection.rootTaskId!, ids) : new Set(ids);
 		const text = filter.text?.toLowerCase();
+		const labels = filter.labels ?? [];
 		return [...selectedIds]
 			.map((id) => this.artifacts.get(id))
 			.filter((task): task is Artifact => task?.kind === "task" && !isDiscussionArtifact(task))
 			.filter((task) => filter.status === undefined || task.status === filter.status)
 			.filter((task) => text === undefined || task.title.toLowerCase().includes(text) || task.body.toLowerCase().includes(text))
+			.filter((task) => labels.every((label) => task.labels.includes(label)))
 			.sort((left, right) => right.updated_at.localeCompare(left.updated_at) || left.id.localeCompare(right.id))
 			.slice(0, limit);
 	}