@danypops/papyrus 0.55.0 → 0.57.0

package/src/handlers/playbooks.ts

@@ -24,11 +24,12 @@ import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import { playbooksOperations } from "../modules/playbooks.ts";
 import type { PlaybookInvocationResult, PlaybookMissingArguments } from "../playbook/playbook-execution.ts";
 import { listPlaybooks } from "../playbook/playbook-service.ts";
-import type { ProjectRegistryStore } from "../ports/project-registry-store.ts";
+import type { ProjectRegistryStore } from "../project-registry/project-registry-store.ts";
+import type { ScopeGroupStore } from "../scope-group/scope-group-store.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 { TaskEventStore } from "../task-event/task-event-store.ts";
+import type { TaskScopeStore } from "../task-scope/task-scope-store.ts";
 import {
 	booleanProp,
 	buildWorkflowRunContent,
@@ -58,6 +59,7 @@ export interface PlaybooksVehicleDeps {
 	tasks: Tasks;
 	sessionIdentity: SessionIdentity;
 	projectRegistry: ProjectRegistryStore;
+	scopeGroups: ScopeGroupStore;
 }
 
 /** Unscoped resolution -- a Playbook is commonly cross-project (e.g. a lab-deploy playbook), matching the hand-rolled tool's own resolutionRequest choice. */
@@ -68,12 +70,11 @@ function resolvePlaybookId(artifacts: ArtifactStore, scopes: ArtifactScopeStore,
 }
 
 export function registerPlaybooksVehicleOperations(registry: VehicleRegistry, deps: PlaybooksVehicleDeps): void {
-	const { artifacts, events, scopes, artifactScopes, tasks, sessionIdentity, projectRegistry } = deps;
+	const { artifacts, events, scopes, artifactScopes, tasks, sessionIdentity, projectRegistry, scopeGroups } = deps;
 	const moduleOperations = new Map(
-		playbooksOperations({ artifacts, events, scopes, artifactScopes, tasks, sessionIdentity, registry: projectRegistry }).map((op) => [
-			op.name,
-			op,
-		]),
+		playbooksOperations({ artifacts, events, scopes, artifactScopes, tasks, sessionIdentity, registry: projectRegistry, scopeGroups }).map(
+			(op) => [op.name, op],
+		),
 	);
 	/**
 	 * Every playbooks.* action funnels through here. invoke's own module handler re-runs
@@ -263,6 +264,46 @@ export function registerPlaybooksVehicleOperations(registry: VehicleRegistry, de
 		(input) => ({ ...input, id: resolvePlaybookId(artifacts, artifactScopes, input.id, input.name) }),
 	);
 
+	define(
+		"set_none",
+		"Fully hides a Playbook -- never applicable, never injected via before_agent_start, regardless of project. The only way back is set_global, add_project, add_group, or replace_projects/replace_groups.",
+		"local-write",
+		{ id: stringProp, name: stringProp },
+		[],
+		(input) => ({ ...input, id: resolvePlaybookId(artifacts, artifactScopes, input.id, input.name) }),
+	);
+
+	define(
+		"add_group",
+		"Adds one scope group (a named, reusable, possibly-nested collection of projects and/or other groups) to a Playbook's explicit scope, switching it from global/none to project-bound if it wasn't already. Idempotent if the group is already a member.",
+		"local-write",
+		{ id: stringProp, name: stringProp, group: { ...stringProp, description: "Exact scope group id, name, or alias to add." } },
+		["group"],
+		(input) => ({ ...input, id: resolvePlaybookId(artifacts, artifactScopes, input.id, input.name) }),
+	);
+
+	define(
+		"remove_group",
+		"Removes one scope group from a Playbook's explicit scope. Rejected while it is the Playbook's only remaining scope member -- call set_global or set_none first.",
+		"local-write",
+		{ id: stringProp, name: stringProp, group: { ...stringProp, description: "Exact scope group id, name, or alias to remove." } },
+		["group"],
+		(input) => ({ ...input, id: resolvePlaybookId(artifacts, artifactScopes, input.id, input.name) }),
+	);
+
+	define(
+		"replace_groups",
+		"Replaces a Playbook's entire scope-group membership with exactly this bounded, non-empty list of scope group references (id/name/alias). Use set_global/set_none instead to clear scoping entirely.",
+		"local-write",
+		{
+			id: stringProp,
+			name: stringProp,
+			groups: { type: "array", description: "Non-empty list of exact scope group id/name/alias references." },
+		},
+		["groups"],
+		(input) => ({ ...input, id: resolvePlaybookId(artifacts, artifactScopes, input.id, input.name) }),
+	);
+
 	define(
 		"update",
 		"Changes a Playbook's title/body/labels/trigger/steps (at least one required). `steps` accepts the exact same shape playbooks.create does and REPLACES the entire step list -- the way to fix a mistake or generalize a Playbook's steps after creation instead of the create-new+supersedes+disable workaround. Refused for a read-only external projection.",

package/src/handlers/projects.ts

@@ -6,7 +6,7 @@
  */
 import type { VehicleRegistry } from "@danypops/vehicle-server";
 import { projectsOperations } from "../modules/projects.ts";
-import type { ProjectRegistryStore } from "../ports/project-registry-store.ts";
+import type { ProjectRegistryStore } from "../project-registry/project-registry-store.ts";
 import { createOperationDefiner, numberProp, stringProp } from "./shared.ts";
 
 const OWNER = "projects";

package/src/handlers/registry.ts

@@ -13,11 +13,12 @@ import type { AuthorityRegistry } from "../authority-registry.ts";
 import { PAPYRUS_VEHICLE_NAME } from "../constants.ts";
 import type { Discussions } from "../discussion/discussion-service.ts";
 import type { Notes } from "../note/note-service.ts";
-import type { ProjectRegistryStore } from "../ports/project-registry-store.ts";
+import type { ProjectRegistryStore } from "../project-registry/project-registry-store.ts";
+import type { ScopeGroupStore } from "../scope-group/scope-group-store.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 { TaskEventStore } from "../task-event/task-event-store.ts";
+import type { TaskScopeStore } from "../task-scope/task-scope-store.ts";
 import { registerArtifactTrashOperations } from "./artifact-trash.ts";
 import { registerDiscussVehicleOperations } from "./discuss.ts";
 import { registerDocsVehicleOperations } from "./docs.ts";
@@ -25,6 +26,7 @@ import { registerNotesVehicleOperations } from "./notes.ts";
 import { registerPlaybooksVehicleOperations } from "./playbooks.ts";
 import { registerProjectsVehicleOperations } from "./projects.ts";
 import { registerRulesVehicleOperations } from "./rules.ts";
+import { registerScopeGroupsVehicleOperations } from "./scope-groups.ts";
 import { registerTasksVehicleOperations } from "./tasks.ts";
 
 export interface PapyrusVehicleDeps {
@@ -38,6 +40,7 @@ export interface PapyrusVehicleDeps {
 	discussions: Discussions;
 	sessionIdentity: SessionIdentity;
 	projectRegistry: ProjectRegistryStore;
+	scopeGroups: ScopeGroupStore;
 }
 
 export function createPapyrusVehicleRegistry(deps: PapyrusVehicleDeps): VehicleRegistry {
@@ -52,8 +55,8 @@ export function createPapyrusVehicleRegistry(deps: PapyrusVehicleDeps): VehicleR
 	// 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, deps.projectRegistry);
-	registerDocsVehicleOperations(registry, deps.artifacts, deps.scopes, deps.authority, deps.projectRegistry);
+	registerRulesVehicleOperations(registry, deps.artifacts, deps.scopes, deps.projectRegistry, deps.scopeGroups);
+	registerDocsVehicleOperations(registry, deps.artifacts, deps.scopes, deps.authority, deps.projectRegistry, deps.scopeGroups);
 	registerPlaybooksVehicleOperations(registry, {
 		artifacts: deps.artifacts,
 		events: deps.events,
@@ -62,9 +65,11 @@ export function createPapyrusVehicleRegistry(deps: PapyrusVehicleDeps): VehicleR
 		tasks: deps.tasks,
 		sessionIdentity: deps.sessionIdentity,
 		projectRegistry: deps.projectRegistry,
+		scopeGroups: deps.scopeGroups,
 	});
 	registerTasksVehicleOperations(registry, { tasks: deps.tasks, artifacts: deps.artifacts, sessionIdentity: deps.sessionIdentity });
 	registerProjectsVehicleOperations(registry, deps.projectRegistry);
+	registerScopeGroupsVehicleOperations(registry, deps.scopeGroups, deps.projectRegistry, deps.scopes);
 	registerDiscussVehicleOperations(registry, deps.discussions, deps.artifacts);
 	registerArtifactTrashOperations(registry, deps.artifacts);
 	return registry;

package/src/handlers/rules.ts

@@ -8,8 +8,9 @@ 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 { rulesOperations } from "../modules/rules.ts";
-import type { ProjectRegistryStore } from "../ports/project-registry-store.ts";
+import type { ProjectRegistryStore } from "../project-registry/project-registry-store.ts";
 import { listRules } from "../rules/rules-service.ts";
+import type { ScopeGroupStore } from "../scope-group/scope-group-store.ts";
 import { booleanProp, createOperationDefiner, numberProp, resolveArtifactIdWidened, stringProp, validationError } from "./shared.ts";
 
 const OWNER = "rules";
@@ -57,8 +58,9 @@ export function registerRulesVehicleOperations(
 	artifacts: ArtifactStore,
 	scopes: ArtifactScopeStore,
 	projectRegistry: ProjectRegistryStore,
+	scopeGroups: ScopeGroupStore,
 ): void {
-	const moduleOperations = new Map(rulesOperations(artifacts, scopes, projectRegistry).map((op) => [op.name, op]));
+	const moduleOperations = new Map(rulesOperations(artifacts, scopes, projectRegistry, scopeGroups).map((op) => [op.name, op]));
 	const call = (name: string, input: Record<string, unknown>): unknown => moduleOperations.get(name)!.execute(input);
 	const define = createOperationDefiner(registry, OWNER, "rules", ["rules:read", "rules:write"], call);
 
@@ -240,6 +242,68 @@ export function registerRulesVehicleOperations(
 		(input) => ({ ...input, id: resolveRuleId(artifacts, scopes, input.project_root as string | undefined, input.id, input.name) }),
 	);
 
+	define(
+		"set_none",
+		"Fully hides a Rule -- never applicable, never injected into the agent system prompt, regardless of project. The only way back is set_global, add_project, add_group, or replace_projects/replace_groups.",
+		"local-write",
+		{ id: stringProp, name: stringProp, project_root: stringProp, actor: stringProp, source: stringProp, session_id: stringProp },
+		[],
+		(input) => ({ ...input, id: resolveRuleId(artifacts, scopes, input.project_root as string | undefined, input.id, input.name) }),
+	);
+
+	define(
+		"add_group",
+		"Adds one scope group (a named, reusable, possibly-nested collection of projects and/or other groups) to a Rule's explicit scope, switching it from global/none to project-bound if it wasn't already. Idempotent if the group is already a member.",
+		"local-write",
+		{
+			id: stringProp,
+			name: stringProp,
+			group: { ...stringProp, description: "Exact scope group id, name, or alias to add." },
+			project_root: stringProp,
+			actor: stringProp,
+			source: stringProp,
+			session_id: stringProp,
+		},
+		["group"],
+		(input) => ({ ...input, id: resolveRuleId(artifacts, scopes, input.project_root as string | undefined, input.id, input.name) }),
+	);
+
+	define(
+		"remove_group",
+		"Removes one scope group from a Rule's explicit scope. Rejected while it is the Rule's only remaining scope member -- call set_global or set_none first.",
+		"local-write",
+		{
+			id: stringProp,
+			name: stringProp,
+			group: { ...stringProp, description: "Exact scope group id, name, or alias to remove." },
+			project_root: stringProp,
+			actor: stringProp,
+			source: stringProp,
+			session_id: stringProp,
+		},
+		["group"],
+		(input) => ({ ...input, id: resolveRuleId(artifacts, scopes, input.project_root as string | undefined, input.id, input.name) }),
+	);
+
+	define(
+		"replace_groups",
+		"Replaces a Rule's entire scope-group membership with exactly this bounded, non-empty list of scope group references (id/name/alias). Use set_global/set_none instead to clear scoping entirely.",
+		"local-write",
+		{
+			id: stringProp,
+			name: stringProp,
+			groups: { type: "array", description: "Non-empty list of exact scope group id/name/alias references." } as unknown as {
+				type: string;
+			},
+			project_root: stringProp,
+			actor: stringProp,
+			source: stringProp,
+			session_id: stringProp,
+		},
+		["groups"],
+		(input) => ({ ...input, id: resolveRuleId(artifacts, scopes, input.project_root as string | undefined, input.id, input.name) }),
+	);
+
 	define(
 		"update",
 		"Changes a Rule's title/body/labels (at least one required). Body updates still enforce the same combined condition+action+body context-tax bound as creation. The response includes combinedLength and a non-blocking warning once it exceeds the ~600-character soft target.",

package/src/handlers/scope-groups.ts

@@ -0,0 +1,100 @@
+/**
+ * Scope groups (named, reusable, possibly-nested scope member collections) projected as a real
+ * VehicleRegistry: one VehicleOperation per real action. Wraps modules/scope-groups.ts's
+ * operation definitions. See artifact/artifact-scope-store.ts's own doc comment for how a Doc/
+ * Rule/Playbook's explicit scope references a group (docs/rules/playbooks.add_group etc.).
+ */
+import type { VehicleRegistry } from "@danypops/vehicle-server";
+import type { ArtifactScopeStore } from "../artifact/artifact-scope-store.ts";
+import { scopeGroupsOperations } from "../modules/scope-groups.ts";
+import type { ProjectRegistryStore } from "../project-registry/project-registry-store.ts";
+import type { ScopeGroupStore } from "../scope-group/scope-group-store.ts";
+import { createOperationDefiner, numberProp, stringProp } from "./shared.ts";
+
+const OWNER = "scope_groups";
+
+export function registerScopeGroupsVehicleOperations(
+	registry: VehicleRegistry,
+	scopeGroups: ScopeGroupStore,
+	projectRegistry: ProjectRegistryStore,
+	artifactScopes: ArtifactScopeStore,
+): void {
+	const moduleOperations = new Map(scopeGroupsOperations(scopeGroups, projectRegistry, artifactScopes).map((op) => [op.name, op]));
+	const call = (name: string, input: Record<string, unknown>): unknown => moduleOperations.get(name)!.execute(input);
+	const define = createOperationDefiner(registry, OWNER, "scope_groups", ["scope_groups:read", "scope_groups:write"], call);
+
+	define(
+		"list",
+		"Lists registered scope groups, optionally filtered by a name/alias substring.",
+		"read",
+		{ query: stringProp, limit: numberProp },
+		[],
+		(input) => input,
+	);
+
+	define(
+		"resolve",
+		"Resolves an exact scope group reference (id, name, or alias) to its full identity, failing closed with bounded candidates when unknown or ambiguous.",
+		"read",
+		{ reference: stringProp },
+		["reference"],
+		(input) => input,
+	);
+
+	define(
+		"register",
+		"Registers a new scope group, or renames an existing one in place (via existing_id/group), preserving its stable id and old name as an alias.",
+		"local-write",
+		{
+			name: stringProp,
+			aliases: { type: "array" } as unknown as { type: string },
+			existing_id: stringProp,
+		},
+		[],
+		(input) => input,
+	);
+
+	define(
+		"show",
+		"Shows one scope group's identity and its own direct membership (not expanded through nested groups).",
+		"read",
+		{ group: { ...stringProp, description: "Exact scope group id, name, or alias." } },
+		["group"],
+		(input) => input,
+	);
+
+	define(
+		"add_member",
+		"Adds one member (a registered project, or another scope group -- nesting) to a scope group. Idempotent if already a member. Refuses a group-type member that would create a cycle or exceed the nesting depth bound.",
+		"local-write",
+		{
+			group: { ...stringProp, description: "Exact scope group id, name, or alias." },
+			member_type: { type: "string", enum: ["project", "group"] },
+			member_reference: { ...stringProp, description: "Exact project or scope group id/name/alias/root reference, per member_type." },
+		},
+		["group", "member_type", "member_reference"],
+		(input) => input,
+	);
+
+	define(
+		"remove_member",
+		"Removes one member from a scope group (idempotent -- removing an absent member is a no-op).",
+		"local-write",
+		{
+			group: { ...stringProp, description: "Exact scope group id, name, or alias." },
+			member_type: { type: "string", enum: ["project", "group"] },
+			member_reference: { ...stringProp, description: "Exact project or scope group id/name/alias/root reference, per member_type." },
+		},
+		["group", "member_type", "member_reference"],
+		(input) => input,
+	);
+
+	define(
+		"delete",
+		"Deletes a scope group outright. Refuses if any artifact's explicit scope, or any other scope group's own membership, still references it.",
+		"local-write",
+		{ group: { ...stringProp, description: "Exact scope group id, name, or alias." } },
+		["group"],
+		(input) => input,
+	);
+}

package/src/handlers/task-classifiers.ts

@@ -10,8 +10,8 @@
 import { VehicleError } from "@danypops/vehicle-core";
 import { PlaybookCompositionError } from "../playbook/playbook-definition.ts";
 import { InvalidSessionSecretError } from "../session-identity/session-identity-service.ts";
-import { TaskCreateIdempotencyConflictError } from "../stores/task-create-request-store.ts";
 import { TaskDependencyCycleError, TaskExecutionBoundExceededError } from "../task/task-execution.ts";
+import { TaskCreateIdempotencyConflictError } from "../task-create-request/task-create-request-store.ts";
 
 /**
  * tasks.focus/pause/unpause/clear_focus and playbooks.invoke all re-run

package/src/handlers/tasks.ts

@@ -23,12 +23,10 @@ import { VehicleError, type VehicleLimits, type VehicleOperationContext } from "
 import type { VehicleRegistry } from "@danypops/vehicle-server";
 import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import { GATE_TIMEOUT_MAX_MS, TASK_CREATE_IDEMPOTENCY_KEY_MAX_LENGTH, TASK_MUTATION_IDEMPOTENCY_KEY_MAX_LENGTH } from "../constants.ts";
-import { PROOF_TYPES } from "../domain/checklist.ts";
-import { GATE_TYPES } from "../domain/gate.ts";
-import type { TaskViewMode } from "../domain/task-scope.ts";
+import { GATE_TYPES } from "../gate/gate.ts";
 import { tasksOperations } from "../modules/tasks.ts";
 import type { SessionIdentity } from "../session-identity/session-identity-service.ts";
-import { TaskMutationIdempotencyConflictError, TaskMutationPendingError } from "../stores/task-mutation-request-store.ts";
+import { PROOF_TYPES } from "../task/checklist.ts";
 import type { TaskExecutionPlan } from "../task/task-execution.ts";
 import {
 	type TaskCompletion,
@@ -38,6 +36,8 @@ import {
 	TaskProjectNotFoundError,
 	type Tasks,
 } from "../task/task-service.ts";
+import { TaskMutationIdempotencyConflictError, TaskMutationPendingError } from "../task-mutation-request/task-mutation-request-store.ts";
+import type { TaskViewMode } from "../task-scope/task-scope.ts";
 import {
 	booleanProp,
 	classifySessionAuthorization,

package/src/index.ts

@@ -17,20 +17,17 @@ export {
 	type VehicleClientTarget,
 } from "./client.ts";
 export * from "./constants.ts";
+export { DISCUSSION_SUBTYPE, type DiscussionRound, readDiscussionExtra } from "./discussion/discussion.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";
-export type { GateResult } from "./domain/gate.ts";
-export type { NoteHistoryPage } from "./domain/note-event.ts";
-export type { TaskEvent, TaskHistoryPage } from "./domain/task-event.ts";
-export type { TaskLease, TaskLeaseView } from "./domain/task-lease.ts";
-export type { TaskViewSelection } from "./domain/task-scope.ts";
+export type { DisplayGraph, DisplayGraphEdge, DisplayGraphNode, RenderedGraph } from "./display-graph/display-graph.ts";
+export type { GraphRenderer } from "./display-graph/graph-renderer.ts";
+export type { GateResult } from "./gate/gate.ts";
+export type { NoteHistoryPage } from "./note/note-event.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 type { GraphRenderer } from "./stores/graph-renderer.ts";
+export { checklistEntries, PROOF_TYPES, type ProofReference } from "./task/checklist.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";
@@ -44,3 +41,6 @@ export type {
 	TaskStatus,
 } from "./task/task-service.ts";
 export { TaskInvalidTransitionError, TaskMutationReceiptNotFoundError } from "./task/task-service.ts";
+export type { TaskEvent, TaskHistoryPage } from "./task-event/task-event.ts";
+export type { TaskLease, TaskLeaseView } from "./task-lease/task-lease.ts";
+export type { TaskViewSelection } from "./task-scope/task-scope.ts";

package/src/log/log-service.ts

@@ -9,8 +9,8 @@ import {
 	type LogQuery,
 	meetsLevel,
 	validateAppendLogEntryCommand,
-} from "../domain/log-entry.ts";
-import type { LogStore } from "../stores/log-store.ts";
+} from "./log-entry.ts";
+import type { LogStore } from "./log-store.ts";
 
 export class Logs {
 	constructor(private readonly store: LogStore) {}

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

@@ -1,4 +1,4 @@
-import type { LogEntry, LogSource } from "../domain/log-entry.ts";
+import type { LogEntry, LogSource } from "../log/log-entry.ts";
 
 /**
  * Persistence port for the `log` domain. Deliberately minimal, matching the split established

package/src/{stores → log}/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 "./log-store.ts";
+import type { JsonValue, LogEntry, LogLevel, LogSource } from "../log/log-entry.ts";
+import type { LogStore } from "../log/log-store.ts";
 
 interface SourceRow {
 	id: string;

package/src/modules/docs.ts

@@ -12,6 +12,7 @@ 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 {
+	addDocGroup,
 	addDocProject,
 	assignDocumentProject,
 	createDocument,
@@ -19,15 +20,19 @@ import {
 	docScope,
 	linkDocument,
 	listDocuments,
+	removeDocGroup,
 	removeDocProject,
+	replaceDocGroups,
 	replaceDocProjects,
 	setDocGlobal,
+	setDocNone,
 	showDocument,
 	transitionDocument,
 	updateDocument,
 } from "../docs/docs-service.ts";
 import type { OperationDefinition } from "../module-registry.ts";
-import type { ProjectRegistryStore } from "../ports/project-registry-store.ts";
+import type { ProjectRegistryStore } from "../project-registry/project-registry-store.ts";
+import type { ScopeGroupStore } from "../scope-group/scope-group-store.ts";
 import { type OperationInput, optionalBoolean, optionalNumber, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "docs";
@@ -69,8 +74,12 @@ export const DOCS_OPERATION_NAMES = [
 	"docs.assign_project",
 	"docs.scope",
 	"docs.set_global",
+	"docs.set_none",
 	"docs.add_project",
 	"docs.remove_project",
+	"docs.add_group",
+	"docs.remove_group",
+	"docs.replace_groups",
 	"docs.replace_projects",
 	"docs.update",
 ] as const;
@@ -80,6 +89,7 @@ export function docsOperations(
 	scopes: ArtifactScopeStore,
 	authority: AuthorityRegistry,
 	registry: ProjectRegistryStore,
+	scopeGroups: ScopeGroupStore,
 ): OperationDefinition[] {
 	const define = <Input, Output>(name: string, execute: (input: Input) => Output): OperationDefinition<Input, Output> => ({
 		name,
@@ -135,6 +145,7 @@ export function docsOperations(
 		),
 		define("docs.scope", (input: OperationInput) => docScope(artifacts, scopes, string(input, "id"))),
 		define("docs.set_global", (input: OperationInput) => setDocGlobal(artifacts, scopes, string(input, "id"))),
+		define("docs.set_none", (input: OperationInput) => setDocNone(artifacts, scopes, string(input, "id"))),
 		define("docs.add_project", (input: OperationInput) =>
 			addDocProject(artifacts, scopes, registry, string(input, "id"), string(input, "project")),
 		),
@@ -144,6 +155,15 @@ export function docsOperations(
 		define("docs.replace_projects", (input: OperationInput) =>
 			replaceDocProjects(artifacts, scopes, registry, string(input, "id"), (input.projects as string[] | undefined) ?? []),
 		),
+		define("docs.add_group", (input: OperationInput) =>
+			addDocGroup(artifacts, scopes, scopeGroups, string(input, "id"), string(input, "group")),
+		),
+		define("docs.remove_group", (input: OperationInput) =>
+			removeDocGroup(artifacts, scopes, scopeGroups, string(input, "id"), string(input, "group")),
+		),
+		define("docs.replace_groups", (input: OperationInput) =>
+			replaceDocGroups(artifacts, scopes, scopeGroups, string(input, "id"), (input.groups as string[] | undefined) ?? []),
+		),
 		define("docs.update", (input: OperationInput) =>
 			updateDocument(
 				artifacts,

package/src/modules/graph-projection.ts

@@ -13,10 +13,10 @@ import type {
 	GraphProjectionBatch,
 	ProjectedArtifact,
 	ProjectedEdge,
-} from "../domain/graph-projection.ts";
+} from "../graph-projection/graph-projection.ts";
 import { GraphProjection } from "../graph-projection/graph-projection-service.ts";
+import type { GraphProjectionStore } from "../graph-projection/graph-projection-store.ts";
 import type { OperationDefinition } from "../module-registry.ts";
-import type { GraphProjectionStore } from "../stores/graph-projection-store.ts";
 import { type OperationInput, string } from "./operation-input.ts";
 
 const MODULE_ID = "graph_projection";

package/src/modules/logs.ts

@@ -5,7 +5,7 @@
  * logs never touch the Artifact graph directly (see src/domain/log-entry.ts's own module
  * comment on why `log` is not an Artifact kind).
  */
-import type { JsonValue, LogLevel } from "../domain/log-entry.ts";
+import type { JsonValue, LogLevel } from "../log/log-entry.ts";
 import type { Logs } from "../log/log-service.ts";
 import type { OperationDefinition } from "../module-registry.ts";
 import { type OperationInput, optionalNumber, optionalString, string } from "./operation-input.ts";

package/src/modules/notes.ts

@@ -13,8 +13,8 @@
  * module's infrastructure" constraint.
  */
 
-import type { NoteEventDirection } from "../domain/note-event.ts";
 import type { OperationDefinition } from "../module-registry.ts";
+import type { NoteEventDirection } from "../note/note-event.ts";
 import type { NoteDisposition, Notes } from "../note/note-service.ts";
 import { type OperationInput, optionalNumber, optionalString, string } from "./operation-input.ts";
 

package/src/modules/playbooks.ts

@@ -16,6 +16,7 @@ import type { ArtifactStore } from "../artifact/artifact-store.ts";
 import type { OperationDefinition } from "../module-registry.ts";
 import { invokePlaybook } from "../playbook/playbook-execution.ts";
 import {
+	addPlaybookGroup,
 	addPlaybookProject,
 	assignPlaybookProject,
 	containPlaybook,
@@ -24,20 +25,24 @@ import {
 	listPlaybooks,
 	playbookInvocation,
 	playbookScope,
+	removePlaybookGroup,
 	removePlaybookProject,
+	replacePlaybookGroups,
 	replacePlaybookProjects,
 	setPlaybookGlobal,
+	setPlaybookNone,
 	showPlaybook,
 	transitionPlaybook,
 	uncontainPlaybook,
 	undependPlaybook,
 	updatePlaybook,
 } from "../playbook/playbook-service.ts";
-import type { ProjectRegistryStore } from "../ports/project-registry-store.ts";
+import type { ProjectRegistryStore } from "../project-registry/project-registry-store.ts";
+import type { ScopeGroupStore } from "../scope-group/scope-group-store.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 { TaskEventStore } from "../task-event/task-event-store.ts";
+import type { TaskScopeStore } from "../task-scope/task-scope-store.ts";
 import { type OperationInput, optionalBoolean, optionalNumber, optionalString, string } from "./operation-input.ts";
 
 const MODULE_ID = "playbooks";
@@ -84,9 +89,13 @@ export const PLAYBOOKS_OPERATION_NAMES = [
 	"playbooks.assign_project",
 	"playbooks.scope",
 	"playbooks.set_global",
+	"playbooks.set_none",
 	"playbooks.add_project",
 	"playbooks.remove_project",
 	"playbooks.replace_projects",
+	"playbooks.add_group",
+	"playbooks.remove_group",
+	"playbooks.replace_groups",
 	"playbooks.update",
 	"playbooks.contain",
 	"playbooks.uncontain",
@@ -105,6 +114,7 @@ export interface PlaybooksModuleDeps {
 	/** 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;
 	registry: ProjectRegistryStore;
+	scopeGroups: ScopeGroupStore;
 }
 
 export function playbooksOperations({
@@ -115,6 +125,7 @@ export function playbooksOperations({
 	tasks,
 	sessionIdentity,
 	registry,
+	scopeGroups,
 }: PlaybooksModuleDeps): OperationDefinition[] {
 	const define = <Input, Output>(name: string, execute: (input: Input) => Output): OperationDefinition<Input, Output> => ({
 		name,
@@ -185,6 +196,16 @@ export function playbooksOperations({
 		),
 		define("playbooks.scope", (input: OperationInput) => playbookScope(artifacts, artifactScopes, string(input, "id"))),
 		define("playbooks.set_global", (input: OperationInput) => setPlaybookGlobal(artifacts, artifactScopes, string(input, "id"))),
+		define("playbooks.set_none", (input: OperationInput) => setPlaybookNone(artifacts, artifactScopes, string(input, "id"))),
+		define("playbooks.add_group", (input: OperationInput) =>
+			addPlaybookGroup(artifacts, artifactScopes, scopeGroups, string(input, "id"), string(input, "group")),
+		),
+		define("playbooks.remove_group", (input: OperationInput) =>
+			removePlaybookGroup(artifacts, artifactScopes, scopeGroups, string(input, "id"), string(input, "group")),
+		),
+		define("playbooks.replace_groups", (input: OperationInput) =>
+			replacePlaybookGroups(artifacts, artifactScopes, scopeGroups, string(input, "id"), (input.groups as string[] | undefined) ?? []),
+		),
 		define("playbooks.add_project", (input: OperationInput) =>
 			addPlaybookProject(artifacts, artifactScopes, registry, string(input, "id"), string(input, "project")),
 		),

package/src/modules/projects.ts

@@ -8,10 +8,10 @@
  */
 
 import { TASK_PROJECT_LIST_MAX_RESULTS } from "../constants.ts";
-import { assertRegisterProjectInputBounds, resolveProjectReference } from "../domain/project-registry.ts";
-import { normalizeProjectRoot } from "../domain/task-scope.ts";
 import type { OperationDefinition } from "../module-registry.ts";
-import type { ProjectRegistryStore } from "../ports/project-registry-store.ts";
+import { assertRegisterProjectInputBounds, resolveProjectReference } from "../project-registry/project-registry.ts";
+import type { ProjectRegistryStore } from "../project-registry/project-registry-store.ts";
+import { normalizeProjectRoot } from "../task-scope/task-scope.ts";
 import { type OperationInput, optionalNumber, optionalString, optionalStringArray, string } from "./operation-input.ts";
 
 const MODULE_ID = "projects";