@danypops/papyrus 0.44.7 → 0.44.9

package/package.json

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

package/src/{ports → artifact}/artifact-event-reader.ts

@@ -1,4 +1,4 @@
-import type { ArtifactEventPage, ArtifactEventQuery } from "../domain/artifact-event.ts";
+import type { ArtifactEventPage, ArtifactEventQuery } from "./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. */

package/src/{artifact-relationship-view.ts → artifact/artifact-relationship-view.ts}

@@ -1,6 +1,6 @@
-import type { Artifact } from "./domain/artifact.ts";
-import type { DisplayGraph, DisplayGraphEdge, DisplayGraphNode } from "./domain/display-graph.ts";
-import { fallbackLabel } from "./task-relationship-view.ts";
+import type { DisplayGraph, DisplayGraphEdge, DisplayGraphNode } from "../domain/display-graph.ts";
+import { fallbackLabel } from "../task/task-relationship-view.ts";
+import type { Artifact } from "./artifact.ts";
 
 /**
  * The generic-artifact counterpart to projectTaskRelationships: unlike a Task, which already

package/src/{ports → artifact}/artifact-store.ts

@@ -7,8 +7,8 @@ import type {
 	CreateArtifactInput,
 	RelationshipQuery,
 	UpdateArtifactInput,
-} from "../domain/artifact.ts";
-import type { ArtifactEventContext } from "../domain/artifact-event.ts";
+} from "./artifact.ts";
+import type { ArtifactEventContext } from "./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

package/src/{artifact-subtree.ts → artifact/artifact-subtree.ts}

@@ -5,10 +5,10 @@
  * Tasks.cancelSubtree's traversal shape but performs trash(), not a lifecycle transition,
  * so it applies to any kind that participates in `contains` (task, playbook), not just Task.
  */
-import { ARTIFACT_REMOVE_SUBTREE_MAX_NODES } from "./constants.ts";
-import type { ArtifactEventContext } from "./domain/artifact-event.ts";
-import type { ArtifactStore } from "./ports/artifact-store.ts";
-import type { ArtifactTrashStore } from "./ports/artifact-trash-store.ts";
+import { ARTIFACT_REMOVE_SUBTREE_MAX_NODES } from "../constants.ts";
+import type { ArtifactEventContext } from "./artifact-event.ts";
+import type { ArtifactStore } from "./artifact-store.ts";
+import type { ArtifactTrashStore } from "./artifact-trash-store.ts";
 
 export interface RemoveSubtreeResult {
 	removed: string[];

package/src/{ports → artifact}/artifact-trash-store.ts

@@ -1,5 +1,5 @@
-import type { ArtifactEventContext } from "../domain/artifact-event.ts";
-import type { ArtifactTrashRecord } from "../domain/artifact-trash.ts";
+import type { ArtifactEventContext } from "./artifact-event.ts";
+import type { ArtifactTrashRecord } from "./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. */

package/src/{adapters → artifact}/sqlite-artifact-scope-store.ts

@@ -1,7 +1,7 @@
 import type { Db } from "../db.ts";
 import { inTransaction } from "../db.ts";
 import type { TaskScopeSource } from "../domain/task-scope.ts";
-import type { ArtifactScope, ArtifactScopeStore } from "../ports/artifact-scope-store.ts";
+import type { ArtifactScope, ArtifactScopeStore } from "./artifact-scope-store.ts";
 
 export class SQLiteArtifactScopeStore implements ArtifactScopeStore {
 	constructor(private readonly db: Db) {}

package/src/{adapters → artifact}/sqlite-artifact-store.ts

@@ -1,17 +1,5 @@
 import type { Db } from "../db.ts";
 import { inTransaction } from "../db.ts";
-import type {
-	Artifact,
-	ArtifactEdge,
-	ArtifactGraphOptions,
-	ArtifactLink,
-	ArtifactQuery,
-	CreateArtifactInput,
-	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 {
 	createArtifact,
 	getArtifact,
@@ -29,9 +17,21 @@ import {
 	updateExtra,
 	updateStatus,
 } from "../ops.ts";
-import type { ArtifactEventReader } from "../ports/artifact-event-reader.ts";
-import type { ArtifactTrashStore } from "../ports/artifact-trash-store.ts";
-import type { AtomicArtifactStore } from "../ports/atomic-artifact-store.ts";
+import type {
+	Artifact,
+	ArtifactEdge,
+	ArtifactGraphOptions,
+	ArtifactLink,
+	ArtifactQuery,
+	CreateArtifactInput,
+	RelationshipQuery,
+	UpdateArtifactInput,
+} from "./artifact.ts";
+import type { ArtifactEventContext, ArtifactEventPage, ArtifactEventQuery } from "./artifact-event.ts";
+import type { ArtifactEventReader } from "./artifact-event-reader.ts";
+import type { ArtifactTrashRecord } from "./artifact-trash.ts";
+import type { ArtifactTrashStore } from "./artifact-trash-store.ts";
+import type { AtomicArtifactStore } from "./atomic-artifact-store.ts";
 
 export class SQLiteArtifactStore implements AtomicArtifactStore, ArtifactTrashStore, ArtifactEventReader {
 	constructor(private readonly db: Db) {}

package/src/authority-registry.ts

@@ -19,9 +19,9 @@
  * already lives, matching "the core has generic registries" from the decision doc.
  */
 
-import type { Artifact, ArtifactLink } from "./domain/artifact.ts";
-import type { ArtifactEventContext } from "./domain/artifact-event.ts";
-import type { ArtifactStore } from "./ports/artifact-store.ts";
+import type { Artifact, ArtifactLink } from "./artifact/artifact.ts";
+import type { ArtifactEventContext } from "./artifact/artifact-event.ts";
+import type { ArtifactStore } from "./artifact/artifact-store.ts";
 
 export type ArtifactAction = "create" | "link" | "status" | "update";
 

package/src/cli.ts

@@ -5,13 +5,13 @@ import { fileURLToPath } from "node:url";
 import { createNodeServiceInstallDeps, generateSystemdUnit, installUserService, type ServiceSpec } from "@danypops/vehicle-server/service";
 import { connectPapyrusClient, type PapyrusClient } from "./client.ts";
 import { DAEMON_UNIT_NAME, dbPath, TASK_EXECUTION_MAX_NODES } from "./constants.ts";
-import { serveMain } from "./daemon.ts";
-import { daemonStateDir, vehicleHandlePath } from "./daemon-state.ts";
+import { serveMain } from "./daemon/daemon.ts";
+import { daemonStateDir, vehicleHandlePath } from "./daemon/daemon-state.ts";
 import { openDb } from "./db.ts";
 import type { GateResult } from "./domain/gate.ts";
 import { applyIdMigration, type IdMigrationPlan, mirrorDatabase, planIdMigration, verifyIdMigration } from "./id-migration.ts";
-import type { TaskExecutionPlan } from "./task-execution.ts";
-import type { TaskBlockage, TaskCompletion } from "./task-service.ts";
+import type { TaskExecutionPlan } from "./task/task-execution.ts";
+import type { TaskBlockage, TaskCompletion } from "./task/task-service.ts";
 import { VERSION } from "./version.ts";
 
 export interface SystemdUnitOptions {
@@ -467,7 +467,7 @@ export async function runGraphCli(args: string[], client: TaskCliClient): Promis
 		return json ? JSON.stringify(artifact) : `Updated ${artifact.id} → [${artifact.status}]`;
 	}
 	if (action !== "history") throw new Error("graph action must be link, unlink, tree, status, or history");
-	const page = await client.call<Record<string, unknown>, import("./domain/artifact-event.ts").ArtifactEventPage>("graph.history", input);
+	const page = await client.call<Record<string, unknown>, import("./artifact/artifact-event.ts").ArtifactEventPage>("graph.history", input);
 	if (json) return JSON.stringify(page);
 	if (page.events.length === 0) return "No recorded events.";
 	return page.events

package/src/client.ts

@@ -3,7 +3,7 @@ import { fileURLToPath } from "node:url";
 import { connectWithVersionCheck, type ExpectedVersion, spawnDetachedDaemon } from "@danypops/vehicle-client/daemon-client";
 import { createLiveVersionExpectation } from "@danypops/vehicle-client/version";
 import { DAEMON_CLIENT_TIMEOUT_MS, DAEMON_DIR_ENV, DAEMON_PROBE_TIMEOUT_MS } from "./constants.ts";
-import { type DaemonHandle, daemonStateDir, readDaemonHandle } from "./daemon-state.ts";
+import { type DaemonHandle, daemonStateDir, readDaemonHandle } from "./daemon/daemon-state.ts";
 import type { OperationName, SchemaState } from "./service.ts";
 
 /**
@@ -161,7 +161,7 @@ export function resolvePushChannelTarget(dir: string = daemonStateDir()): PushCh
 }
 
 export interface VehicleClientTarget {
-	/** Base URL for a domain migrated onto VehicleRegistry (see src/vehicle/*.ts) -- @danypops/vehicle-client's RemoteVehicleClient mounts its own /vehicle/manifest, /vehicle/invoke, /vehicle/cancel routes under this. */
+	/** Base URL for a domain migrated onto VehicleRegistry (see src/handlers/*.ts) -- @danypops/vehicle-client's RemoteVehicleClient mounts its own /vehicle/manifest, /vehicle/invoke, /vehicle/cancel routes under this. */
 	baseUrl: string;
 	token: string;
 }

package/src/{daemon-state.ts → daemon/daemon-state.ts}

@@ -3,7 +3,7 @@ import { mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { join } from "node:path";
 import { LOOPBACK_HOST, removeDaemonHandle, writeDaemonHandle } from "@danypops/vehicle-server/paths";
-import { DAEMON_DIR_ENV, DAEMON_HANDLE_FILE, DAEMON_HOST, DAEMON_PORT_FILE, DAEMON_TOKEN_FILE } from "./constants.ts";
+import { DAEMON_DIR_ENV, DAEMON_HANDLE_FILE, DAEMON_HOST, DAEMON_PORT_FILE, DAEMON_TOKEN_FILE } from "../constants.ts";
 
 export interface DaemonHandle {
 	baseUrl: string;

package/src/{daemon.ts → daemon/daemon.ts}

@@ -1,7 +1,9 @@
 import { join } from "node:path";
 import { acquireDaemonLock, releaseDaemonLock } from "@danypops/vehicle-server/paths";
 import { PushChannel } from "@danypops/vehicle-server/push-channel";
-import { DAEMON_HOST, DB_OPTIMIZE_INTERVAL_MS, dbPath, WAL_CHECKPOINT_INTERVAL_MS } from "./constants.ts";
+import { DAEMON_HOST, DB_OPTIMIZE_INTERVAL_MS, dbPath, WAL_CHECKPOINT_INTERVAL_MS } from "../constants.ts";
+import { logEvent, vehicleLogger } from "../log/log.ts";
+import { createApp, createPapyrusService } from "../service.ts";
 import {
 	clearDaemonPort,
 	clearVehicleHandle,
@@ -10,8 +12,6 @@ import {
 	writeDaemonPort,
 	writeVehicleHandle,
 } from "./daemon-state.ts";
-import { logEvent, vehicleLogger } from "./log.ts";
-import { createApp, createPapyrusService } from "./service.ts";
 
 /**
  * Operations that never change what a Task-graph reader (the pi-papyrus widget's

package/src/db.ts

@@ -2,8 +2,8 @@ import { mkdirSync } from "node:fs";
 import { createRequire } from "node:module";
 import { dirname } from "node:path";
 import { runMigrations, type SqliteMigrationRunner } from "@danypops/vehicle-server/storage";
+import { generateUniqueAlias, slugify } from "./artifact/artifact-alias.ts";
 import { SQLITE_BUSY_TIMEOUT_MS, SQLITE_SCHEMA_VERSION } from "./constants.ts";
-import { generateUniqueAlias, slugify } from "./domain/artifact-alias.ts";
 
 const require_ = createRequire(import.meta.url);
 const IS_BUN = typeof (globalThis as { Bun?: unknown }).Bun !== "undefined";

package/src/{discussion-service.ts → discussion/discussion-service.ts}

@@ -3,9 +3,11 @@
  * append-only rounds (via DiscussionRoundStore). See domain/discussion.ts for the full
  * design rationale.
  */
-import { DISCUSSION_LIST_DEFAULT_LIMIT, DISCUSSION_LIST_MAX_LIMIT, DISCUSSION_MAX_ROUNDS } from "./constants.ts";
-import type { Artifact } from "./domain/artifact.ts";
-import type { ArtifactEventContext } from "./domain/artifact-event.ts";
+
+import type { Artifact } from "../artifact/artifact.ts";
+import type { ArtifactEventContext } from "../artifact/artifact-event.ts";
+import type { AtomicArtifactStore } from "../artifact/atomic-artifact-store.ts";
+import { DISCUSSION_LIST_DEFAULT_LIMIT, DISCUSSION_LIST_MAX_LIMIT, DISCUSSION_MAX_ROUNDS } from "../constants.ts";
 import {
 	DISCUSSION_SUBTYPE,
 	type DiscussionExtra,
@@ -19,9 +21,8 @@ import {
 	validateDiscussionOptions,
 	validateSelectedOptions,
 	validateSettlement,
-} from "./domain/discussion.ts";
-import type { AtomicArtifactStore } from "./ports/atomic-artifact-store.ts";
-import type { DiscussionRoundStore } from "./ports/discussion-round-store.ts";
+} from "../domain/discussion.ts";
+import type { DiscussionRoundStore } from "../stores/discussion-round-store.ts";
 
 export class DiscussionError extends Error {}
 

package/src/domain-services.ts

@@ -1,3 +1,7 @@
+import { type Artifact, requireLocallyOwnedContent } from "./artifact/artifact.ts";
+import type { ArtifactEventContext } from "./artifact/artifact-event.ts";
+import type { ArtifactScopeStore } from "./artifact/artifact-scope-store.ts";
+import type { ArtifactStore } from "./artifact/artifact-store.ts";
 import type { ArtifactAction, AuthorityRegistry } from "./authority-registry.ts";
 import {
 	ARTIFACT_BODY_MAX_LENGTH,
@@ -14,8 +18,6 @@ import {
 	RULE_TEXT_HARD_LIMIT_CHARACTERS,
 	SKILL_MAX_ENUM_VALUES,
 } from "./constants.ts";
-import { type Artifact, requireLocallyOwnedContent } from "./domain/artifact.ts";
-import type { ArtifactEventContext } from "./domain/artifact-event.ts";
 import {
 	BLUEPRINT_INPUT_TYPES,
 	type BlueprintArgumentValue,
@@ -23,9 +25,7 @@ import {
 	validateArgumentValue,
 } from "./domain/blueprint-definition.ts";
 import { normalizeProjectRoot } from "./domain/task-scope.ts";
-import { NOTE_SUBTYPE } from "./note-service.ts";
-import type { ArtifactScopeStore } from "./ports/artifact-scope-store.ts";
-import type { ArtifactStore } from "./ports/artifact-store.ts";
+import { NOTE_SUBTYPE } from "./note/note-service.ts";
 
 export interface UpdateContentInput {
 	title?: string;

package/src/{graph-projection-service.ts → graph-projection/graph-projection-service.ts}

@@ -4,16 +4,16 @@
  * documented, deliberate scope limits of this first walking-skeleton slice.
  */
 
-import type { AuthorityRegistry } from "./authority-registry.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
+import { requireAtomicArtifactStore } from "../artifact/atomic-artifact-store.ts";
+import type { AuthorityRegistry } from "../authority-registry.ts";
 import {
 	GRAPH_PROJECTION_ID_MAX_LENGTH,
 	GRAPH_PROJECTION_MAX_ARTIFACTS_PER_BATCH,
 	GRAPH_PROJECTION_MAX_EDGES_PER_BATCH,
-} from "./constants.ts";
-import { GRAPH_PROJECTION_SCHEMA_VERSION, type GraphProjectionBatch, type GraphProjectionResult } from "./domain/graph-projection.ts";
-import type { ArtifactStore } from "./ports/artifact-store.ts";
-import { requireAtomicArtifactStore } from "./ports/atomic-artifact-store.ts";
-import type { GraphProjectionStore } from "./ports/graph-projection-store.ts";
+} from "../constants.ts";
+import { GRAPH_PROJECTION_SCHEMA_VERSION, type GraphProjectionBatch, type GraphProjectionResult } from "../domain/graph-projection.ts";
+import type { GraphProjectionStore } from "../stores/graph-projection-store.ts";
 
 function boundedId(value: string, label: string): string {
 	if (!value || value.length === 0) throw new Error(`${label} is required`);

package/src/{vehicle/artifact-trash-vehicle.ts → handlers/artifact-trash.ts}

@@ -5,10 +5,10 @@
  */
 import { bindVehicleOperation, defineVehicleOperation } from "@danypops/vehicle-core";
 import type { VehicleRegistry } from "@danypops/vehicle-server";
-import { removeArtifactSubtree } from "../artifact-subtree.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
-import type { ArtifactTrashStore } from "../ports/artifact-trash-store.ts";
-import { looseObjectSchema, numberProp, passthroughOutput, stringProp, validationError } from "./artifact-vehicle-shared.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
+import { removeArtifactSubtree } from "../artifact/artifact-subtree.ts";
+import type { ArtifactTrashStore } from "../artifact/artifact-trash-store.ts";
+import { looseObjectSchema, numberProp, passthroughOutput, stringProp, validationError } from "./shared.ts";
 
 const OWNER = "artifact";
 const LIMITS = { defaultTimeoutMs: 5_000, maxTimeoutMs: 30_000, maxRequestBytes: 65_536, maxResponseBytes: 262_144 };

package/src/{vehicle/discuss-vehicle.ts → handlers/discuss.ts}

@@ -15,19 +15,12 @@
  */
 import { bindVehicleOperation, defineVehicleOperation, type VehicleContentBlock } from "@danypops/vehicle-core";
 import type { VehicleRegistry } from "@danypops/vehicle-server";
-import type { DiscussionAndRounds, Discussions } from "../discussion-service.ts";
-import type { Artifact } from "../domain/artifact.ts";
+import type { Artifact } from "../artifact/artifact.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
+import type { DiscussionAndRounds, Discussions } from "../discussion/discussion-service.ts";
 import { DISCUSSION_SUBTYPE, type DiscussionRound } from "../domain/discussion.ts";
 import { discussOperations } from "../modules/discuss.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
-import {
-	looseObjectSchema,
-	numberProp,
-	passthroughOutput,
-	resolveArtifactIdWidened,
-	stringProp,
-	validationError,
-} from "./artifact-vehicle-shared.ts";
+import { looseObjectSchema, numberProp, passthroughOutput, resolveArtifactIdWidened, stringProp, validationError } from "./shared.ts";
 
 const OWNER = "discuss";
 const LIMITS = { defaultTimeoutMs: 5_000, maxTimeoutMs: 30_000, maxRequestBytes: 65_536, maxResponseBytes: 262_144 };

package/src/{vehicle/docs-vehicle.ts → handlers/docs.ts}

@@ -5,19 +5,12 @@
  */
 import { bindVehicleOperation, defineVehicleOperation } from "@danypops/vehicle-core";
 import type { VehicleRegistry } from "@danypops/vehicle-server";
+import type { ArtifactScopeStore } from "../artifact/artifact-scope-store.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import type { AuthorityRegistry } from "../authority-registry.ts";
 import { listDocuments } from "../domain-services.ts";
 import { docsOperations } from "../modules/docs.ts";
-import type { ArtifactScopeStore } from "../ports/artifact-scope-store.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
-import {
-	looseObjectSchema,
-	numberProp,
-	passthroughOutput,
-	resolveArtifactIdWidened,
-	stringProp,
-	validationError,
-} from "./artifact-vehicle-shared.ts";
+import { looseObjectSchema, numberProp, passthroughOutput, resolveArtifactIdWidened, stringProp, validationError } from "./shared.ts";
 
 const OWNER = "docs";
 const LIMITS = { defaultTimeoutMs: 5_000, maxTimeoutMs: 30_000, maxRequestBytes: 65_536, maxResponseBytes: 262_144 };

package/src/{vehicle/notes-vehicle.ts → handlers/notes.ts}

@@ -9,17 +9,10 @@
  */
 import { bindVehicleOperation, defineVehicleOperation } from "@danypops/vehicle-core";
 import type { VehicleRegistry } from "@danypops/vehicle-server";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import { notesOperations } from "../modules/notes.ts";
-import { NOTE_DISPOSITIONS, type Notes } from "../note-service.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
-import {
-	looseObjectSchema,
-	numberProp,
-	passthroughOutput,
-	resolveArtifactIdWidened,
-	stringProp,
-	validationError,
-} from "./artifact-vehicle-shared.ts";
+import { NOTE_DISPOSITIONS, type Notes } from "../note/note-service.ts";
+import { looseObjectSchema, numberProp, passthroughOutput, resolveArtifactIdWidened, stringProp, validationError } from "./shared.ts";
 
 const OWNER = "notes";
 

package/src/{vehicle/playbooks-vehicle.ts → handlers/playbooks.ts}

@@ -9,7 +9,7 @@
  * Those two fields never belong in this operation's model-visible inputSchema -- a model
  * has no business knowing or supplying a session secret. Instead they travel through
  * VehicleInvocationOptions.principal.claims, populated by pi-papyrus's own
- * resolveInvocation hook (see vehicle-notes-client.ts) from its own already-cached
+ * resolveInvocation hook (see pi-papyrus's tools/vehicle-notes-client.ts) from its own already-cached
  * session_secret, the same value the hand-rolled tool used to thread through as a raw
  * input field. A caller with no cached secret for this session (unregistered, or a non-Pi
  * Vehicle client) simply gets the guard's own no-op-when-unset default, unchanged.
@@ -20,15 +20,15 @@
  */
 import { bindVehicleOperation, defineVehicleOperation, type VehicleOperationContext } from "@danypops/vehicle-core";
 import type { VehicleRegistry } from "@danypops/vehicle-server";
+import type { ArtifactScopeStore } from "../artifact/artifact-scope-store.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import { listPlaybooks } from "../domain-services.ts";
 import { playbooksOperations } from "../modules/playbooks.ts";
-import type { PlaybookInvocationResult, PlaybookMissingArguments } from "../playbook-execution.ts";
-import type { ArtifactScopeStore } from "../ports/artifact-scope-store.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
-import type { TaskEventStore } from "../ports/task-event-store.ts";
-import type { TaskScopeStore } from "../ports/task-scope-store.ts";
-import type { SessionIdentity } from "../session-identity-service.ts";
-import type { Tasks } from "../task-service.ts";
+import type { PlaybookInvocationResult, PlaybookMissingArguments } from "../playbook/playbook-execution.ts";
+import type { SessionIdentity } from "../session-identity/session-identity-service.ts";
+import type { TaskEventStore } from "../stores/task-event-store.ts";
+import type { TaskScopeStore } from "../stores/task-scope-store.ts";
+import type { Tasks } from "../task/task-service.ts";
 import {
 	buildWorkflowRunContent,
 	classifyPlaybookComposition,
@@ -41,7 +41,7 @@ import {
 	resolveArtifactIdWidened,
 	stringProp,
 	validationError,
-} from "./artifact-vehicle-shared.ts";
+} from "./shared.ts";
 
 const OWNER = "playbooks";
 const LIMITS = { defaultTimeoutMs: 5_000, maxTimeoutMs: 30_000, maxRequestBytes: 65_536, maxResponseBytes: 262_144 };
@@ -70,7 +70,7 @@ export function registerPlaybooksVehicleOperations(registry: VehicleRegistry, de
 	/**
 	 * Every playbooks.* action funnels through here. invoke's own module handler re-runs
 	 * sessionIdentity.assertAuthorized directly (see this file's own doc comment) and its
-	 * blueprint-materialization engine (workflow-execution.ts) can hit the same execution-graph
+	 * blueprint-materialization engine (playbook/workflow-execution.ts) can hit the same execution-graph
 	 * bounds tasks-vehicle.ts's own operations do -- classifying both reviewed domain error classes
 	 * at this one choke point covers every action that can throw them. Anything else propagates
 	 * unchanged -- vehicle-registry's own secure-by-default handler-failed opacity still applies to a

package/src/{vehicle/papyrus-vehicle.ts → handlers/registry.ts}

@@ -6,23 +6,23 @@
  * per domain.
  */
 import { VehicleRegistry } from "@danypops/vehicle-server";
+import type { ArtifactScopeStore } from "../artifact/artifact-scope-store.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
+import type { ArtifactTrashStore } from "../artifact/artifact-trash-store.ts";
 import type { AuthorityRegistry } from "../authority-registry.ts";
-import type { Discussions } from "../discussion-service.ts";
-import type { Notes } from "../note-service.ts";
-import type { ArtifactScopeStore } from "../ports/artifact-scope-store.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
-import type { ArtifactTrashStore } from "../ports/artifact-trash-store.ts";
-import type { TaskEventStore } from "../ports/task-event-store.ts";
-import type { TaskScopeStore } from "../ports/task-scope-store.ts";
-import type { SessionIdentity } from "../session-identity-service.ts";
-import type { Tasks } from "../task-service.ts";
-import { registerArtifactTrashOperations } from "./artifact-trash-vehicle.ts";
-import { registerDiscussVehicleOperations } from "./discuss-vehicle.ts";
-import { registerDocsVehicleOperations } from "./docs-vehicle.ts";
-import { registerNotesVehicleOperations } from "./notes-vehicle.ts";
-import { registerPlaybooksVehicleOperations } from "./playbooks-vehicle.ts";
-import { registerRulesVehicleOperations } from "./rules-vehicle.ts";
-import { registerTasksVehicleOperations } from "./tasks-vehicle.ts";
+import type { Discussions } from "../discussion/discussion-service.ts";
+import type { Notes } from "../note/note-service.ts";
+import type { SessionIdentity } from "../session-identity/session-identity-service.ts";
+import type { TaskEventStore } from "../stores/task-event-store.ts";
+import type { TaskScopeStore } from "../stores/task-scope-store.ts";
+import type { Tasks } from "../task/task-service.ts";
+import { registerArtifactTrashOperations } from "./artifact-trash.ts";
+import { registerDiscussVehicleOperations } from "./discuss.ts";
+import { registerDocsVehicleOperations } from "./docs.ts";
+import { registerNotesVehicleOperations } from "./notes.ts";
+import { registerPlaybooksVehicleOperations } from "./playbooks.ts";
+import { registerRulesVehicleOperations } from "./rules.ts";
+import { registerTasksVehicleOperations } from "./tasks.ts";
 
 export interface PapyrusVehicleDeps {
 	artifacts: ArtifactStore & ArtifactTrashStore;
@@ -42,6 +42,11 @@ export function createPapyrusVehicleRegistry(deps: PapyrusVehicleDeps): VehicleR
 		version: "1.0.0",
 		description: "Papyrus's graph-artifact domains, one honest operation per real action.",
 	});
+	// An unexpected handler exception's message becomes the classified failure's causeMessage
+	// instead of vanishing into an opaque "<op>@N handler failed". Safe here: no Papyrus domain
+	// error ever embeds a session_secret or other credential in its message (only a bare
+	// session_id, a correlation id, not a secret -- see session-identity-service.ts).
+	registry.setExposeHandlerFailureDetails(true);
 	registerNotesVehicleOperations(registry, deps.notes, deps.artifacts);
 	registerRulesVehicleOperations(registry, deps.artifacts, deps.scopes);
 	registerDocsVehicleOperations(registry, deps.artifacts, deps.scopes, deps.authority);

package/src/{vehicle/rules-vehicle.ts → handlers/rules.ts}

@@ -6,18 +6,11 @@
  */
 import { bindVehicleOperation, defineVehicleOperation } from "@danypops/vehicle-core";
 import type { VehicleRegistry } from "@danypops/vehicle-server";
+import type { ArtifactScopeStore } from "../artifact/artifact-scope-store.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import { listRules } from "../domain-services.ts";
 import { rulesOperations } from "../modules/rules.ts";
-import type { ArtifactScopeStore } from "../ports/artifact-scope-store.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
-import {
-	looseObjectSchema,
-	numberProp,
-	passthroughOutput,
-	resolveArtifactIdWidened,
-	stringProp,
-	validationError,
-} from "./artifact-vehicle-shared.ts";
+import { looseObjectSchema, numberProp, passthroughOutput, resolveArtifactIdWidened, stringProp, validationError } from "./shared.ts";
 
 const OWNER = "rules";
 const LIMITS = { defaultTimeoutMs: 5_000, maxTimeoutMs: 30_000, maxRequestBytes: 65_536, maxResponseBytes: 262_144 };

package/src/{vehicle/artifact-vehicle-shared.ts → handlers/shared.ts}

@@ -4,11 +4,11 @@
  * artifact-trash-vehicle.ts).
  */
 import { defineVehicleSchema, type VehicleContentBlock, VehicleError, type VehicleSchemaCodec } from "@danypops/vehicle-core";
-import type { Artifact } from "../domain/artifact.ts";
-import { PlaybookCompositionError } from "../playbook-definition.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
-import { InvalidSessionSecretError } from "../session-identity-service.ts";
-import { TaskDependencyCycleError, TaskExecutionBoundExceededError, type TaskExecutionPlan } from "../task-execution.ts";
+import type { Artifact } from "../artifact/artifact.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
+import { PlaybookCompositionError } from "../playbook/playbook-definition.ts";
+import { InvalidSessionSecretError } from "../session-identity/session-identity-service.ts";
+import { TaskDependencyCycleError, TaskExecutionBoundExceededError, type TaskExecutionPlan } from "../task/task-execution.ts";
 
 /**
  * VehicleRegistry only ever calls a schema's own safeParse -- jsonSchema is

package/src/{vehicle/tasks-vehicle.ts → handlers/tasks.ts}

@@ -10,7 +10,7 @@
  *  - session_secret authorizes which session's Task Focus row gets mutated
  *    (modules/tasks.ts's own guardFocusMutation). It must never be a model-visible
  *    input field -- it travels through VehicleInvocationOptions.principal.claims,
- *    the same mechanism playbooks.invoke uses (see vehicle-notes-client.ts).
+ *    the same mechanism playbooks.invoke uses (see pi-papyrus's tools/vehicle-notes-client.ts).
  *  - papyrus.task-focus.v1 is a same-process Pi extension event bus broadcast (a
  *    token-cost router or similar can correlate its own telemetry with the
  *    currently focused task) with no Vehicle-transport equivalent -- fired from
@@ -21,12 +21,12 @@
  */
 import { bindVehicleOperation, defineVehicleOperation, type VehicleOperationContext } from "@danypops/vehicle-core";
 import type { VehicleRegistry } from "@danypops/vehicle-server";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import type { TaskViewMode } from "../domain/task-scope.ts";
 import { tasksOperations } from "../modules/tasks.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
-import type { SessionIdentity } from "../session-identity-service.ts";
-import type { TaskExecutionPlan } from "../task-execution.ts";
-import type { TaskCompletion, Tasks } from "../task-service.ts";
+import type { SessionIdentity } from "../session-identity/session-identity-service.ts";
+import type { TaskExecutionPlan } from "../task/task-execution.ts";
+import type { TaskCompletion, Tasks } from "../task/task-service.ts";
 import {
 	classifySessionAuthorization,
 	classifyTaskDependencyCycles,
@@ -38,7 +38,7 @@ import {
 	resolveArtifactIdWidened,
 	stringProp,
 	validationError,
-} from "./artifact-vehicle-shared.ts";
+} from "./shared.ts";
 
 const OWNER = "tasks";
 const LIMITS = { defaultTimeoutMs: 5_000, maxTimeoutMs: 30_000, maxRequestBytes: 65_536, maxResponseBytes: 262_144 };

package/src/index.ts

@@ -5,7 +5,9 @@
  * purposeful API, not "everything happens to be reachable."
  */
 
-export { projectArtifactRelationships } from "./artifact-relationship-view.ts";
+export type { Artifact, ArtifactEdge } from "./artifact/artifact.ts";
+export { projectArtifactRelationships } from "./artifact/artifact-relationship-view.ts";
+export type { ArtifactStore } from "./artifact/artifact-store.ts";
 export {
 	connectPapyrusClient,
 	type PapyrusClient,
@@ -15,8 +17,7 @@ export {
 	type VehicleClientTarget,
 } from "./client.ts";
 export * from "./constants.ts";
-export type { DiscussionAndRounds } from "./discussion-service.ts";
-export type { Artifact, ArtifactEdge } from "./domain/artifact.ts";
+export type { DiscussionAndRounds } from "./discussion/discussion-service.ts";
 export { checklistEntries, PROOF_TYPES, type ProofReference } from "./domain/checklist.ts";
 export { DISCUSSION_SUBTYPE, type DiscussionRound, readDiscussionExtra } from "./domain/discussion.ts";
 export type { DisplayGraph, DisplayGraphEdge, DisplayGraphNode, RenderedGraph } from "./domain/display-graph.ts";
@@ -25,14 +26,13 @@ export type { NoteHistoryPage } from "./domain/note-event.ts";
 export type { TaskEvent, TaskHistoryPage } from "./domain/task-event.ts";
 export type { TaskLease } from "./domain/task-lease.ts";
 export type { TaskViewSelection } from "./domain/task-scope.ts";
-export { NOTE_DISPOSITIONS } from "./note-service.ts";
-export type { PlaybookInvocationResult, PlaybookMissingArguments } from "./playbook-execution.ts";
-export type { ArtifactStore } from "./ports/artifact-store.ts";
-export type { GraphRenderer } from "./ports/graph-renderer.ts";
+export { NOTE_DISPOSITIONS } from "./note/note-service.ts";
+export type { PlaybookInvocationResult, PlaybookMissingArguments } from "./playbook/playbook-execution.ts";
+export type { WorkflowRunResult } from "./playbook/workflow-execution.ts";
 export type { OperationName, SchemaState } from "./service.ts";
-export { taskContext } from "./task-context.ts";
-export { projectTaskExecution, type TaskExecutionPlan, type TaskExecutionState } from "./task-execution.ts";
-export { projectTaskGraph, type TaskGraphView } from "./task-graph-view.ts";
-export { fallbackLabel, projectTaskRelationships } from "./task-relationship-view.ts";
-export type { TaskCompletion, TaskGraph, TaskNode, TaskStatus } from "./task-service.ts";
-export type { WorkflowRunResult } from "./workflow-execution.ts";
+export type { GraphRenderer } from "./stores/graph-renderer.ts";
+export { taskContext } from "./task/task-context.ts";
+export { projectTaskExecution, type TaskExecutionPlan, type TaskExecutionState } from "./task/task-execution.ts";
+export { projectTaskGraph, type TaskGraphView } from "./task/task-graph-view.ts";
+export { fallbackLabel, projectTaskRelationships } from "./task/task-relationship-view.ts";
+export type { TaskCompletion, TaskGraph, TaskNode, TaskStatus } from "./task/task-service.ts";