@danypops/papyrus 0.29.7 → 0.30.0

package/extension/src/domain-tools.ts

@@ -238,7 +238,7 @@ const checklistCriterionSchema = Type.Object({
 	proof: Type.Array(proofReferenceSchema, { minItems: 1 }),
 });
 
-export function registerDomainTools(pi: ExtensionAPI): void {
+export function registerTasksTool(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "tasks",
 		label: "Tasks",
@@ -445,7 +445,9 @@ export function registerDomainTools(pi: ExtensionAPI): void {
 			}
 		},
 	});
+}
 
+export function registerNotesTool(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "notes",
 		label: "Notes",
@@ -496,7 +498,9 @@ export function registerDomainTools(pi: ExtensionAPI): void {
 			}
 		},
 	});
+}
 
+export function registerDocsTool(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "docs",
 		label: "Documents",
@@ -556,7 +560,9 @@ export function registerDomainTools(pi: ExtensionAPI): void {
 			}
 		},
 	});
+}
 
+export function registerRulesTool(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "rules",
 		label: "Rules",
@@ -604,7 +610,9 @@ export function registerDomainTools(pi: ExtensionAPI): void {
 			}
 		},
 	});
+}
 
+export function registerPlaybooksTool(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "playbooks",
 		label: "Playbooks",
@@ -652,7 +660,9 @@ export function registerDomainTools(pi: ExtensionAPI): void {
 			}
 		},
 	});
+}
 
+export function registerSkillsTool(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "skills",
 		label: "Skills",
@@ -729,7 +739,9 @@ export function registerDomainTools(pi: ExtensionAPI): void {
 			}
 		},
 	});
+}
 
+export function registerDiscussTool(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "discuss",
 		label: "Discuss",
@@ -827,3 +839,14 @@ export function registerDomainTools(pi: ExtensionAPI): void {
 		},
 	});
 }
+
+/** Thin orchestrator: each domain's tool is independently navigable/testable via its own registerXTool function. */
+export function registerDomainTools(pi: ExtensionAPI): void {
+	registerTasksTool(pi);
+	registerNotesTool(pi);
+	registerDocsTool(pi);
+	registerRulesTool(pi);
+	registerPlaybooksTool(pi);
+	registerSkillsTool(pi);
+	registerDiscussTool(pi);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/papyrus",
-  "version": "0.29.7",
+  "version": "0.30.0",
   "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/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,