@danypops/papyrus 0.44.8 → 0.44.10

package/src/modules/notes.ts

@@ -15,32 +15,11 @@
 
 import type { NoteEventDirection } from "../domain/note-event.ts";
 import type { OperationDefinition } from "../module-registry.ts";
-import type { NoteDisposition, Notes } from "../note-service.ts";
+import type { NoteDisposition, Notes } from "../note/note-service.ts";
+import { type OperationInput, optionalNumber, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "notes";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 /** This module's own operation names, the single source of truth src/service.ts's EXPECTED_OPERATION_NAMES spreads in rather than re-listing by hand. */
 export const NOTES_OPERATION_NAMES = [
 	"notes.capture",

package/src/modules/operation-input.ts

@@ -0,0 +1,35 @@
+/**
+ * Shared runtime validators for a Vehicle operation's raw `input` object.
+ * A Vehicle operation's own JSON-schema `input` field is descriptive metadata
+ * only (see handlers/shared.ts's looseObjectSchema) -- never itself enforced
+ * at runtime -- so every module's own operation body narrows `unknown` to
+ * the real shape it needs through these.
+ */
+export type OperationInput = Record<string, unknown>;
+
+export function string(input: OperationInput, key: string): string {
+	const value = input[key];
+	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
+	return value;
+}
+
+export function optionalString(input: OperationInput, key: string): string | undefined {
+	const value = input[key];
+	if (value === undefined) return undefined;
+	if (typeof value !== "string") throw new Error(`${key} must be a string`);
+	return value;
+}
+
+export function optionalStringArray(input: OperationInput, key: string): string[] | undefined {
+	const value = input[key];
+	if (value === undefined) return undefined;
+	if (!Array.isArray(value) || value.some((entry) => typeof entry !== "string")) throw new Error(`${key} must be an array of strings`);
+	return value as string[];
+}
+
+export function optionalNumber(input: OperationInput, key: string): number | undefined {
+	const value = input[key];
+	if (value === undefined) return undefined;
+	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
+	return value;
+}

package/src/modules/playbooks.ts

@@ -4,11 +4,14 @@
  * A Playbook is prose-first, whole-artifact composition (contains/depends_on between real
  * Playbook artifacts) rather than a raw JSON blueprint -- but its own step list can also
  * declare Doc/Rule blueprints and typed arguments and nested pipeline calls. playbooks.invoke
- * recycles the shared blueprint materialization engine (playbook-execution.ts compiles a
+ * recycles the shared blueprint materialization engine (playbook/playbook-execution.ts compiles a
  * Playbook's steps and composition tree into a BlueprintDefinition, then hands off to
- * workflow-execution.ts's shared core). See domain-services.ts's Playbook section and
- * playbook-definition.ts for the full rationale.
+ * playbook/workflow-execution.ts's shared core). See domain-services.ts's Playbook section and
+ * playbook/playbook-definition.ts for the full rationale.
  */
+
+import type { ArtifactScopeStore } from "../artifact/artifact-scope-store.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import {
 	assignPlaybookProject,
 	containPlaybook,
@@ -23,38 +26,15 @@ import {
 	updatePlaybook,
 } from "../domain-services.ts";
 import type { OperationDefinition } from "../module-registry.ts";
-import { invokePlaybook } 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 { invokePlaybook } 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 { type OperationInput, optionalNumber, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "playbooks";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 const eventContext = (input: OperationInput) => ({
 	actor: optionalString(input, "actor"),
 	source: optionalString(input, "source"),
@@ -96,7 +76,7 @@ export interface PlaybooksModuleDeps {
 	scopes: TaskScopeStore;
 	/** Docs/Rules/Skills/Playbooks project scoping (distinct from `scopes`, which is Task-run project scoping for playbooks.invoke's materialized tasks). */
 	artifactScopes: ArtifactScopeStore;
-	/** Used for exactly one thing: focusing the entry task after a successful invoke -- the one safety-checked Tasks operation this module needs, not bulk graph construction (that goes straight through artifacts/events/scopes in playbook-execution.ts, mirroring workflow-execution.ts). */
+	/** Used for exactly one thing: focusing the entry task after a successful invoke -- the one safety-checked Tasks operation this module needs, not bulk graph construction (that goes straight through artifacts/events/scopes in playbook/playbook-execution.ts, mirroring playbook/workflow-execution.ts). */
 	tasks: Tasks;
 	/** Guards the same session_secret check tasks.focus's own operation enforces (guardFocusMutation in modules/tasks.ts) -- invoke's internal tasks.focus() call goes straight through the Tasks class, bypassing that operation wrapper entirely, so the check must be applied here instead of silently skipped. */
 	sessionIdentity: SessionIdentity;

package/src/modules/rules.ts

@@ -10,6 +10,9 @@
  * into this module or introducing a premature "modules call each other through the
  * registry" convention.
  */
+
+import type { ArtifactScopeStore } from "../artifact/artifact-scope-store.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import {
 	assignRuleProject,
 	createRule,
@@ -21,33 +24,10 @@ import {
 	updateRule,
 } from "../domain-services.ts";
 import type { OperationDefinition } from "../module-registry.ts";
-import type { ArtifactScopeStore } from "../ports/artifact-scope-store.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
+import { type OperationInput, optionalNumber, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "rules";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 const eventContext = (input: OperationInput) => ({
 	actor: optionalString(input, "actor"),
 	source: optionalString(input, "source"),

package/src/modules/session-identity.ts

@@ -4,25 +4,11 @@
  * precedent set by modules/logs.ts. See domain/session-identity.ts for the design rationale.
  */
 import type { OperationDefinition } from "../module-registry.ts";
-import type { SessionIdentity } from "../session-identity-service.ts";
+import type { SessionIdentity } from "../session-identity/session-identity-service.ts";
+import { type OperationInput, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "session-identity";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
 /** This module's own operation names, the single source of truth src/service.ts's EXPECTED_OPERATION_NAMES spreads in rather than re-listing by hand. */
 export const SESSION_IDENTITY_OPERATION_NAMES = ["session.register", "session.release"] as const;
 

package/src/modules/tasks.ts

@@ -13,52 +13,25 @@
  *
  * Task-domain-internal files (task-context.ts, task-execution.ts, domain/task-event.ts,
  * domain/task-scope.ts) are imported directly — they belong to this bounded context,
- * unlike a different module's infrastructure. Generic input-parsing helpers are
- * duplicated locally rather than imported from src/service.ts, matching the precedent
- * set by modules/notes.ts: a module does not import another module's infrastructure,
- * including the composition root's own helpers.
+ * unlike a different module's infrastructure. Generic input-parsing helpers come from
+ * ./operation-input.ts, a peer utility at this same modules/ layer with no domain logic
+ * of its own -- not "another module's infrastructure" or the composition root's helpers,
+ * the actual thing this module avoids importing.
  */
+
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import type { Checklist } from "../domain/checklist.ts";
 import type { TaskEventContext, TaskEventDirection, TaskEventFeedQuery } from "../domain/task-event.ts";
 import type { TaskViewMode } from "../domain/task-scope.ts";
 import type { OperationDefinition } from "../module-registry.ts";
-import type { ArtifactStore } from "../ports/artifact-store.ts";
-import type { SessionIdentity } from "../session-identity-service.ts";
-import { taskContext } from "../task-context.ts";
-import { projectTaskExecution } from "../task-execution.ts";
-import type { TaskStatus, Tasks } from "../task-service.ts";
+import type { SessionIdentity } from "../session-identity/session-identity-service.ts";
+import { taskContext } from "../task/task-context.ts";
+import { projectTaskExecution } from "../task/task-execution.ts";
+import type { TaskStatus, Tasks } from "../task/task-service.ts";
+import { type OperationInput, optionalNumber, optionalString, optionalStringArray, string } from "./operation-input.ts";
 
 const MODULE_ID = "tasks";
 
-type OperationInput = Record<string, unknown>;
-
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function optionalStringArray(input: OperationInput, key: string): string[] | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (!Array.isArray(value) || value.some((entry) => typeof entry !== "string")) throw new Error(`${key} must be an array of strings`);
-	return value as string[];
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 const eventContext = (input: OperationInput): TaskEventContext => ({
 	actor: optionalString(input, "actor"),
 	source: optionalString(input, "source"),

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

@@ -1,3 +1,6 @@
+import type { Artifact } from "../artifact/artifact.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
+import { requireAtomicArtifactStore } from "../artifact/atomic-artifact-store.ts";
 import {
 	NOTE_BODY_MAX_CHARACTERS,
 	NOTE_LIST_DEFAULT_LIMIT,
@@ -6,12 +9,9 @@ import {
 	NOTE_REASON_MAX_CHARACTERS,
 	NOTE_TITLE_MAX_CHARACTERS,
 	TASK_PROJECT_ROOT_MAX_LENGTH,
-} from "./constants.ts";
-import type { Artifact } from "./domain/artifact.ts";
-import type { AppendNoteEvent, NoteEventType, NoteHistoryPage, NoteHistoryQuery } from "./domain/note-event.ts";
-import type { ArtifactStore } from "./ports/artifact-store.ts";
-import { requireAtomicArtifactStore } from "./ports/atomic-artifact-store.ts";
-import { InMemoryNoteEventStore, type NoteEventStore } from "./ports/note-event-store.ts";
+} from "../constants.ts";
+import type { AppendNoteEvent, NoteEventType, NoteHistoryPage, NoteHistoryQuery } from "../domain/note-event.ts";
+import { InMemoryNoteEventStore, type NoteEventStore } from "../stores/note-event-store.ts";
 
 export const NOTE_SUBTYPE = "note";
 export const NOTE_DISPOSITIONS = ["completed", "duplicate", "declined", "superseded"] as const;

package/src/ops.ts

@@ -3,14 +3,14 @@
  * Enforces the schema protocol (kinds, statuses, relations) via FK + app validation.
  */
 import { createRequire } from "node:module";
+import type { Artifact, ArtifactQuery, CreateArtifactInput, UpdateArtifactInput } from "./artifact/artifact.ts";
+import { generateUniqueAlias, isValidAlias, slugify } from "./artifact/artifact-alias.ts";
+import type { ArtifactTrashRecord } from "./artifact/artifact-trash.ts";
 import { ARTIFACT_TRASH_RETENTION_MS, DEFAULT_STATUS_BY_KIND } from "./constants.ts";
 import type { Db } from "./db.ts";
 import { inTransaction } from "./db.ts";
-import type { Artifact, ArtifactQuery, CreateArtifactInput, UpdateArtifactInput } from "./domain/artifact.ts";
-import { generateUniqueAlias, isValidAlias, slugify } from "./domain/artifact-alias.ts";
-import type { ArtifactTrashRecord } from "./domain/artifact-trash.ts";
 
-export type { ArtifactTrashRecord } from "./domain/artifact-trash.ts";
+export type { ArtifactTrashRecord } from "./artifact/artifact-trash.ts";
 
 import {
 	type AppendArtifactEvent,
@@ -21,10 +21,10 @@ import {
 	type ArtifactEventType,
 	normalizeArtifactEventQuery,
 	resolveArtifactEvent,
-} from "./domain/artifact-event.ts";
+} from "./artifact/artifact-event.ts";
 import type { Gate, GateResult, GateRunOptions } from "./domain/gate.ts";
 
-export type { Artifact } from "./domain/artifact.ts";
+export type { Artifact } from "./artifact/artifact.ts";
 export type { Gate, GateResult } from "./domain/gate.ts";
 export type CreateInput = CreateArtifactInput;
 

package/src/{playbook-definition.ts → playbook/playbook-definition.ts}

@@ -18,12 +18,14 @@
  * independently by workflow-execution.ts, exactly like a workflow-definition target's own
  * nested pipeline step, since its target's own definition is not knowable at this compile time.
  */
+
+import type { Artifact } from "../artifact/artifact.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import {
 	PLAYBOOK_INVOCATION_MAX_CALL_DEPTH,
 	PLAYBOOK_INVOCATION_MAX_CREATED_TASKS,
 	PLAYBOOK_INVOCATION_MAX_LINKED_ARTIFACTS,
-} from "./constants.ts";
-import type { Artifact } from "./domain/artifact.ts";
+} from "../constants.ts";
 import type {
 	BlueprintDefinition,
 	BlueprintInputDefinition,
@@ -31,10 +33,9 @@ import type {
 	DocBlueprint,
 	RuleBlueprint,
 	TaskBlueprint,
-} from "./domain/blueprint-definition.ts";
-import { validateBlueprintDefinition } from "./domain/blueprint-definition.ts";
-import type { PlaybookArgument, PlaybookStep } from "./domain-services.ts";
-import type { ArtifactStore } from "./ports/artifact-store.ts";
+} from "../domain/blueprint-definition.ts";
+import { validateBlueprintDefinition } from "../domain/blueprint-definition.ts";
+import type { PlaybookArgument, PlaybookStep } from "../domain-services.ts";
 
 /** A non-composing edge touching a playbook node, to be mirrored onto that node's generated root task once real task ids exist -- e.g. a Rule `gates` this playbook, or this playbook `references`/`documents` a Doc. Direction is preserved exactly: `from`/`to` name whichever side is NOT the playbook, and `ownerIsFrom` says which side the playbook (now the generated root task) occupies. */
 export interface PlaybookExternalLink {

package/src/{playbook-execution.ts → playbook/playbook-execution.ts}

@@ -7,11 +7,12 @@
  * (via the existing Task Focus system-prompt pointer), which is what actually avoids the
  * old text-dump problem: one page at a time, not the whole book at once.
  */
-import type { BlueprintArgumentValue } from "./domain/blueprint-definition.ts";
+
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
+import { requireAtomicArtifactStore } from "../artifact/atomic-artifact-store.ts";
+import type { BlueprintArgumentValue } from "../domain/blueprint-definition.ts";
+import type { TaskExecutionPlan } from "../task/task-execution.ts";
 import { compilePlaybookDefinition } from "./playbook-definition.ts";
-import type { ArtifactStore } from "./ports/artifact-store.ts";
-import { requireAtomicArtifactStore } from "./ports/atomic-artifact-store.ts";
-import type { TaskExecutionPlan } from "./task-execution.ts";
 import {
 	applyPlaybookExternalLinks,
 	materializeWorkflowDefinition,

package/src/{workflow-execution.ts → playbook/workflow-execution.ts}

@@ -1,28 +1,28 @@
 import { randomUUID } from "node:crypto";
+import type { Artifact } from "../artifact/artifact.ts";
+import type { ArtifactStore } from "../artifact/artifact-store.ts";
+import { requireAtomicArtifactStore } from "../artifact/atomic-artifact-store.ts";
 import {
 	SKILL_MAX_RENDERED_BYTES,
 	SKILL_RUN_ID_MAX_LENGTH,
 	SKILL_WORKFLOW_MAX_NESTING_DEPTH,
 	TASK_EXECUTION_MAX_EDGES,
-} from "./constants.ts";
-import type { Artifact } from "./domain/artifact.ts";
+} from "../constants.ts";
 import {
 	type BlueprintArgumentValue,
 	type BlueprintDefinition,
 	type CallBlueprint,
 	resolveBlueprintArguments,
 	validateBlueprintDefinition,
-} from "./domain/blueprint-definition.ts";
-import { validateChecklist } from "./domain/checklist.ts";
-import type { TaskEventContext } from "./domain/task-event.ts";
-import { normalizeProjectRoot } from "./domain/task-scope.ts";
+} from "../domain/blueprint-definition.ts";
+import { validateChecklist } from "../domain/checklist.ts";
+import type { TaskEventContext } from "../domain/task-event.ts";
+import { normalizeProjectRoot } from "../domain/task-scope.ts";
+import type { TaskEventStore } from "../stores/task-event-store.ts";
+import type { TaskScopeStore } from "../stores/task-scope-store.ts";
+import { projectTaskExecution, TaskExecutionBoundExceededError, type TaskExecutionPlan } from "../task/task-execution.ts";
+import type { TaskGraph, TaskNode, TaskStatus } from "../task/task-service.ts";
 import { compilePlaybookDefinition, type PlaybookExternalLink } from "./playbook-definition.ts";
-import type { ArtifactStore } from "./ports/artifact-store.ts";
-import { requireAtomicArtifactStore } from "./ports/atomic-artifact-store.ts";
-import type { TaskEventStore } from "./ports/task-event-store.ts";
-import type { TaskScopeStore } from "./ports/task-scope-store.ts";
-import { projectTaskExecution, TaskExecutionBoundExceededError, type TaskExecutionPlan } from "./task-execution.ts";
-import type { TaskGraph, TaskNode, TaskStatus } from "./task-service.ts";
 
 const RUN_ID_PATTERN = /^[A-Za-z0-9][A-Za-z0-9_-]*$/;
 const EXACT_PLACEHOLDER_PATTERN = /^{{\s*([A-Za-z][A-Za-z0-9_-]{0,63})\s*}}$/;

package/src/service.ts

@@ -2,49 +2,50 @@ import { VehicleError } from "@danypops/vehicle-core";
 import type { VehicleRegistry } from "@danypops/vehicle-server";
 import { createVehicleHttpApp } from "@danypops/vehicle-server/http";
 import type { Logger } from "@danypops/vehicle-server/logging";
-import { SQLiteArtifactScopeStore } from "./adapters/sqlite-artifact-scope-store.ts";
-import { SQLiteArtifactStore } from "./adapters/sqlite-artifact-store.ts";
-import { SQLiteDiscussionRoundStore } from "./adapters/sqlite-discussion-round-store.ts";
-import { SQLiteGateRunner } from "./adapters/sqlite-gate-runner.ts";
-import { SQLiteGraphProjectionStore } from "./adapters/sqlite-graph-projection-store.ts";
-import { SQLiteLogStore } from "./adapters/sqlite-log-store.ts";
-import { SQLiteNoteEventStore } from "./adapters/sqlite-note-event-store.ts";
-import { SQLiteSessionIdentityStore } from "./adapters/sqlite-session-identity-store.ts";
-import { SQLiteTaskEventStore } from "./adapters/sqlite-task-event-store.ts";
-import { SQLiteTaskFocusStore } from "./adapters/sqlite-task-focus-store.ts";
-import { SQLiteTaskLeaseStore } from "./adapters/sqlite-task-lease-store.ts";
-import { SQLiteTaskScopeStore } from "./adapters/sqlite-task-scope-store.ts";
-import { removeArtifactSubtree } from "./artifact-subtree.ts";
+import type { CreateArtifactInput } from "./artifact/artifact.ts";
+import type { ArtifactEventReader } from "./artifact/artifact-event-reader.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 { SQLiteArtifactScopeStore } from "./artifact/sqlite-artifact-scope-store.ts";
+import { SQLiteArtifactStore } from "./artifact/sqlite-artifact-store.ts";
 import { type AuthorityClaim, AuthorityRegistry, AuthorizedArtifactWriter } from "./authority-registry.ts";
 import { SERVICE_MAX_BODY_BYTES, SQLITE_SCHEMA_VERSION } from "./constants.ts";
 import { migrateDb, openDb, schemaVersion } from "./db.ts";
-import { Discussions } from "./discussion-service.ts";
-import type { CreateArtifactInput } from "./domain/artifact.ts";
+import { Discussions } from "./discussion/discussion-service.ts";
 import type { TaskEventContext } from "./domain/task-event.ts";
 import type { TaskViewMode } from "./domain/task-scope.ts";
 import { listInjectableRules } from "./domain-services.ts";
-import { logEvent } from "./log.ts";
-import { Logs } from "./log-service.ts";
+import { createPapyrusVehicleRegistry } from "./handlers/registry.ts";
+import { logEvent } from "./log/log.ts";
+import { Logs } from "./log/log-service.ts";
 import { OperationRegistry } from "./module-registry.ts";
 import { DISCUSS_OPERATION_NAMES, discussOperations } from "./modules/discuss.ts";
 import { DOCS_OPERATION_NAMES, docsOperations } from "./modules/docs.ts";
 import { GRAPH_PROJECTION_OPERATION_NAMES, graphProjectionOperations } from "./modules/graph-projection.ts";
 import { LOGS_OPERATION_NAMES, logsOperations } from "./modules/logs.ts";
 import { NOTES_OPERATION_NAMES, notesOperations } from "./modules/notes.ts";
+import { type OperationInput, optionalNumber, optionalString, string } from "./modules/operation-input.ts";
 import { PLAYBOOKS_OPERATION_NAMES, playbooksOperations } from "./modules/playbooks.ts";
 import { RULES_OPERATION_NAMES, rulesOperations } from "./modules/rules.ts";
 import { SESSION_IDENTITY_OPERATION_NAMES, sessionIdentityOperations } from "./modules/session-identity.ts";
 import { TASKS_OPERATION_NAMES, tasksOperations } from "./modules/tasks.ts";
-import { NOTE_SUBTYPE, Notes } from "./note-service.ts";
-import type { ArtifactEventReader } from "./ports/artifact-event-reader.ts";
-import type { ArtifactStore } from "./ports/artifact-store.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";
-import { InvalidSessionSecretError, SessionIdentity } from "./session-identity-service.ts";
-import { type TaskStatus, Tasks } from "./task-service.ts";
-import { createPapyrusVehicleRegistry } from "./vehicle/papyrus-vehicle.ts";
+import { NOTE_SUBTYPE, Notes } from "./note/note-service.ts";
+import { InvalidSessionSecretError, SessionIdentity } from "./session-identity/session-identity-service.ts";
+import type { GateRunner } from "./stores/gate-runner.ts";
+import { SQLiteDiscussionRoundStore } from "./stores/sqlite-discussion-round-store.ts";
+import { SQLiteGateRunner } from "./stores/sqlite-gate-runner.ts";
+import { SQLiteGraphProjectionStore } from "./stores/sqlite-graph-projection-store.ts";
+import { SQLiteLogStore } from "./stores/sqlite-log-store.ts";
+import { SQLiteNoteEventStore } from "./stores/sqlite-note-event-store.ts";
+import { SQLiteSessionIdentityStore } from "./stores/sqlite-session-identity-store.ts";
+import { SQLiteTaskEventStore } from "./stores/sqlite-task-event-store.ts";
+import { SQLiteTaskFocusStore } from "./stores/sqlite-task-focus-store.ts";
+import { SQLiteTaskLeaseStore } from "./stores/sqlite-task-lease-store.ts";
+import { SQLiteTaskScopeStore } from "./stores/sqlite-task-scope-store.ts";
+import type { TaskEventStore } from "./stores/task-event-store.ts";
+import type { TaskScopeStore } from "./stores/task-scope-store.ts";
+import { type TaskStatus, Tasks } from "./task/task-service.ts";
 import { VERSION } from "./version.ts";
 
 /**
@@ -99,7 +100,6 @@ export const EXPECTED_OPERATION_NAMES = [
 ] as const;
 
 export type OperationName = (typeof EXPECTED_OPERATION_NAMES)[number];
-type OperationInput = Record<string, unknown>;
 type OperationHandler = (input: OperationInput) => unknown;
 
 export class UnknownOperationError extends Error {}
@@ -107,33 +107,6 @@ export class MigrationRequiredError extends Error {}
 export class PayloadTooLargeError extends Error {}
 export { InvalidSessionSecretError };
 
-function string(input: OperationInput, key: string): string {
-	const value = input[key];
-	if (typeof value !== "string" || value.length === 0) throw new Error(`${key} is required`);
-	return value;
-}
-
-function optionalString(input: OperationInput, key: string): string | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "string") throw new Error(`${key} must be a string`);
-	return value;
-}
-
-function _optionalStringArray(input: OperationInput, key: string): string[] | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (!Array.isArray(value) || value.some((entry) => typeof entry !== "string")) throw new Error(`${key} must be an array of strings`);
-	return value as string[];
-}
-
-function optionalNumber(input: OperationInput, key: string): number | undefined {
-	const value = input[key];
-	if (value === undefined) return undefined;
-	if (typeof value !== "number" || !Number.isFinite(value)) throw new Error(`${key} must be a number`);
-	return value;
-}
-
 function normalizeCreateInput(input: OperationInput): CreateArtifactInput {
 	const { template_id, ...rest } = input;
 	return { ...rest, templateId: typeof template_id === "string" ? template_id : undefined } as CreateArtifactInput;
@@ -217,7 +190,7 @@ export interface PapyrusService {
 	execute(operation: string, input?: OperationInput): Promise<unknown>;
 	/**
 	 * Every domain migrated onto Vehicle, merged into one registry/one HTTP mount
-	 * (see ./vehicle/papyrus-vehicle.ts) -- one honest VehicleOperation per real
+	 * (see ./handlers/registry.ts) -- one honest VehicleOperation per real
 	 * action, replacing the Pi extension's old `<domain>(action=X)` mega-tools.
 	 * Not every domain is migrated yet -- see papyrus-vehicle.ts's own doc comment
 	 * for what still isn't and why.
@@ -582,7 +555,7 @@ export function createApp(deps: {
 	/** Defaults to a no-op (createVehicleHttpApp's own default) -- daemon.ts wires vehicleLogger() so a failed invocation is actually logged, not silently discarded. */
 	logger?: Logger;
 }): { fetch(request: Request): Promise<Response> } {
-	// Same Bearer token, daemon, and port as the rest of this API -- see ./vehicle/papyrus-vehicle.ts.
+	// Same Bearer token, daemon, and port as the rest of this API -- see ./handlers/registry.ts.
 	const vehicleApp = createVehicleHttpApp({ registry: deps.service.vehicle, token: deps.token, logger: deps.logger });
 	return {
 		async fetch(request: Request): Promise<Response> {

package/src/{session-identity-service.ts → session-identity/session-identity-service.ts}

@@ -4,8 +4,8 @@ import {
 	releaseSessionIdentity,
 	verifySessionSecret,
 } from "@danypops/vehicle-server/session-identity";
-import { assertValidSessionId } from "./domain/session-identity.ts";
-import type { SessionIdentityStore } from "./ports/session-identity-store.ts";
+import { assertValidSessionId } from "../domain/session-identity.ts";
+import type { SessionIdentityStore } from "../stores/session-identity-store.ts";
 
 export interface RegisterSessionIdentityResult {
 	sessionId: string;

package/src/{ports → stores}/session-identity-store.ts

@@ -2,7 +2,7 @@ import type { SessionIdentityRecord, SessionIdentityStore as VehicleSessionIdent
 
 /**
  * Papyrus's persistence port for @danypops/vehicle-server's storage-agnostic session-identity
- * primitive -- re-exported under this project's own port naming convention (src/ports/*)
+ * primitive -- re-exported under this project's own port naming convention (src/stores/*)
  * rather than importing the vehicle-server interface name directly at every call site.
  */
 export type SessionIdentityStore = VehicleSessionIdentityStore;

package/src/{adapters → stores}/sqlite-discussion-round-store.ts

@@ -9,7 +9,7 @@ import {
 	validateDiscussionContent,
 	validateDiscussionOptions,
 } from "../domain/discussion.ts";
-import type { DiscussionRoundStore } from "../ports/discussion-round-store.ts";
+import type { DiscussionRoundStore } from "./discussion-round-store.ts";
 
 interface DiscussionRoundRow {
 	id: number;

package/src/{adapters → stores}/sqlite-gate-runner.ts

@@ -1,7 +1,7 @@
 import type { Db } from "../db.ts";
 import type { GateResult, GateRunOptions } from "../domain/gate.ts";
 import { runGates, runGatesAsync } from "../ops.ts";
-import type { GateRunner } from "../ports/gate-runner.ts";
+import type { GateRunner } from "./gate-runner.ts";
 
 export class SQLiteGateRunner implements GateRunner {
 	constructor(private readonly db: Db) {}

package/src/{adapters → stores}/sqlite-graph-projection-store.ts

@@ -1,7 +1,7 @@
 import type { Db } from "../db.ts";
 import { inTransaction } from "../db.ts";
 import type { ProjectionCheckpoint } from "../domain/graph-projection.ts";
-import type { GraphProjectionStore } from "../ports/graph-projection-store.ts";
+import type { GraphProjectionStore } from "./graph-projection-store.ts";
 
 export class SQLiteGraphProjectionStore implements GraphProjectionStore {
 	constructor(private readonly db: Db) {}

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

@@ -1,6 +1,6 @@
 import type { Db } from "../db.ts";
 import type { JsonValue, LogEntry, LogLevel, LogSource } from "../domain/log-entry.ts";
-import type { LogStore } from "../ports/log-store.ts";
+import type { LogStore } from "./log-store.ts";
 
 interface SourceRow {
 	id: string;