@danypops/papyrus 0.34.0 → 0.34.1

package/package.json

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

package/src/domain/artifact.ts

@@ -48,6 +48,13 @@ export interface ArtifactQuery {
 	limit?: number;
 	/** Trashed artifacts (see artifact-trash.ts) are excluded from every query by default; set true to include them, e.g. for a trash-listing view. */
 	includeTrashed?: boolean;
+	/**
+	 * Restrict to exactly these ids, still subject to every other filter (kind, trash exclusion,
+	 * etc.) -- for a caller that already has a bounded candidate id set (e.g. Tasks.list's
+	 * project/graph scope) and needs query()'s own trash-exclusion without a full-kind scan.
+	 * Empty array is a real "match nothing", not "unset".
+	 */
+	ids?: string[];
 }
 
 export interface ArtifactGraphOptions {

package/src/ops.ts

@@ -290,6 +290,11 @@ export function queryArtifacts(db: Db, filter: ArtifactQuery): Artifact[] {
 	const conditions: string[] = [];
 	const params: unknown[] = [];
 	if (!filter.includeTrashed) conditions.push("id NOT IN (SELECT artifact_id FROM artifact_trash)");
+	if (filter.ids) {
+		if (filter.ids.length === 0) return [];
+		conditions.push(`id IN (${filter.ids.map(() => "?").join(", ")})`);
+		params.push(...filter.ids);
+	}
 	if (filter.kind) { conditions.push("kind = ?"); params.push(filter.kind); }
 	if (filter.status) { conditions.push("status = ?"); params.push(filter.status); }
 	if (filter.statuses) {

package/src/task-service.ts

@@ -237,9 +237,16 @@ export class Tasks {
 		const selectedIds = selection.mode === "graph" ? this.descendantIds(selection.rootTaskId!, ids) : new Set(ids);
 		const text = filter.text?.toLowerCase();
 		const labels = filter.labels ?? [];
+		// query(), unlike get(), excludes trash by default -- a trashed task's id stays in
+		// task_scopes until the retention window actually purges it, so building this candidate
+		// set from get() per id (deliberately trash-transparent, for show/restore) would otherwise
+		// leak a trashed task back into list results for the entire grace period. Scoped by ids
+		// rather than a bare kind query, so this stays bounded to exactly the already-bounded
+		// selectedIds set instead of scanning every task in the database.
+		const notTrashed = new Map(this.artifacts.query({ kind: "task", excludeSubtype: DISCUSSION_SUBTYPE, ids: [...selectedIds] }).map((task) => [task.id, task]));
 		return [...selectedIds]
-			.map((id) => this.artifacts.get(id))
-			.filter((task): task is Artifact => task?.kind === "task" && !isDiscussionArtifact(task))
+			.map((id) => notTrashed.get(id))
+			.filter((task): task is Artifact => task !== undefined)
 			.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)))