@cosmicdrift/kumiko-framework 0.328.0 → 0.329.0

package/src/pipeline/dispatch-shared.ts

@@ -1,1242 +0,0 @@
-import { requestContext, runWithOrigin } from "../api/request-context";
-import type { SseBroker } from "../api/sse-broker";
-import type { DbConnection, DbRunner, DbTx } from "../db/connection";
-import { runInSavepoint, selectMany } from "../db/query";
-import type { buildEntityTable } from "../db/table-builder";
-import {
-  createTenantDb,
-  createUncheckedSystemDb,
-  hasTenantColumn,
-  type TenantDb,
-} from "../db/tenant-db";
-import { createDerivativesContext } from "../derivatives/derivatives-context";
-import type { defineTransitions } from "../engine/state-machine";
-import type { EffectiveFeaturesResolver } from "../engine/tier-resolver-extension";
-import type {
-  AgentRisk,
-  AggregateStreamHandle,
-  AppContext,
-  AppendEventArgs,
-  AppendEventFn,
-  AuthClaimsContext,
-  FetchForWritingArgs,
-  HandlerContext,
-  JobRunnerRef,
-  MemberReader,
-  RateLimitDeclaration,
-  RateLimitOption,
-  Registry,
-  SessionUser,
-  WriteResult,
-} from "../engine/types";
-import { isRateLimitDisabled, resolveAgentExposure } from "../engine/types";
-import type { TenantId } from "../engine/types/identifiers";
-import {
-  FeatureDisabledError,
-  InternalError,
-  memberResolutionReadOnlyDenied,
-  VersionConflictError,
-  type WriteErrorInfo,
-} from "../errors";
-import {
-  archiveStream as archiveStreamHelper,
-  isStreamArchived,
-  restoreStream as restoreStreamHelper,
-} from "../event-store/archive";
-import {
-  IdempotentAppendConflictError as EventStoreIdempotentAppendConflictError,
-  VersionConflictError as EventStoreVersionConflictError,
-} from "../event-store/errors";
-import {
-  getStreamVersion,
-  loadAggregate,
-  loadAggregateAsOf,
-  type StoredEvent,
-} from "../event-store/event-store";
-import {
-  type LoadAggregateWithSnapshotOptions,
-  type LoadAggregateWithSnapshotResult,
-  loadAggregateWithSnapshot,
-  type SnapshotReducer,
-  saveSnapshot,
-} from "../event-store/snapshot";
-import { upcastStoredEvent, upcastStoredEvents } from "../event-store/upcaster";
-import { createFileContext } from "../files/file-handle";
-import { DEFAULT_LOCALE, isValidLocaleTag } from "../i18n/request-locale";
-import {
-  createMetricsHandle,
-  createNoopMetricsHandle,
-  createSafeMetricsHandle,
-  emitDispatcherError,
-  emitDispatcherHandler,
-  type getFallbackMeter,
-  getFallbackTracer,
-  observabilityContext,
-} from "../observability";
-import {
-  createEscapeHatchReporter,
-  type EscapeHatchReportWindow,
-} from "../observability/escape-hatch-report";
-import { buildBucketKey } from "../rate-limit";
-import { createTzContext, isValidIanaTimeZone } from "../time";
-import { INTERACTIVE_SIGN_IN_POLICY, resolveActiveMembershipFn } from "./active-membership";
-import { appendDomainEventCore } from "./append-event-core";
-import { resolveAuthClaims as runAuthClaimsResolver } from "./auth-claims-resolver";
-import { executeQuery } from "./dispatch-query";
-import { executeWrite } from "./dispatch-write";
-import {
-  type AfterCommitHook,
-  dispatcherSpanAttributes,
-  isFailedWriteResult,
-} from "./dispatcher-utils";
-import type { IdempotencyGuard } from "./idempotency";
-import type { LifecycleHooks } from "./lifecycle-pipeline";
-import { createMemberReaderFn } from "./member-reader";
-import {
-  createGatedIdentitySwitch,
-  createGatedMemberReader,
-  createGatedProjectionReader,
-  isSystemIdentity,
-  type ProjectionReader,
-  systemIdentitySwitchDenied,
-} from "./system-identity-switch";
-import type { TenantTimezoneCache } from "./tenant-timezone-cache";
-import { buildPersonalDataGate, rootWriteOrigin, type WriteOrigin } from "./write-origin";
-
-export type { WriteOrigin } from "./write-origin";
-
-// Framework/pipeline stays bundled-features-free, so this can't import the
-// `tenant` feature — the literal below IS the coupling to its `timezone`
-// config key. Renaming that key (or the "tenant" feature name) must update
-// this constant too; tenant-timezone-boot.integration.test.ts boots the real
-// createTenantFeature() and would catch a drift.
-export const TENANT_TIMEZONE_CONFIG_KEY = "tenant:config:timezone";
-
-// Same bundled-features-free constraint as above — these name the config
-// feature's write handlers so dispatch-write.ts can invalidate
-// TENANT_TIMEZONE_CONFIG_KEY's cache entry on a successful write without
-// importing the `config` feature. tenant-timezone-boot.integration.test.ts
-// exercises both against the real config feature.
-export const CONFIG_WRITE_SET_TYPE = "config:write:set";
-export const CONFIG_WRITE_RESET_TYPE = "config:write:reset";
-
-export type BatchCommand = {
-  readonly type: string;
-  readonly payload: unknown;
-};
-
-export type BatchResult =
-  | { readonly isSuccess: true; readonly results: readonly WriteResult[] }
-  | {
-      readonly isSuccess: false;
-      readonly error: WriteErrorInfo;
-      readonly failedIndex: number;
-      readonly results: readonly WriteResult[];
-    };
-
-// Bundles everything the dispatch-phase functions (query/write/batch) need —
-// hoisted to module scope out of createDispatcher's former closure, so those
-// captures travel explicitly instead of implicitly.
-export type DispatchContext = {
-  registry: Registry;
-  appContext: AppContext;
-  idempotency: IdempotencyGuard | undefined;
-  lifecycle: LifecycleHooks | undefined;
-  jobRunner: JobRunnerRef | undefined;
-  effectiveFeatures: EffectiveFeaturesResolver | undefined;
-  sseBroker: SseBroker | undefined;
-  tableCache: Map<string, ReturnType<typeof buildEntityTable>>;
-  transitionCache: Map<string, ReturnType<typeof defineTransitions>>;
-  tenantTimezoneCache: TenantTimezoneCache;
-  escapeHatchReportWindow: EscapeHatchReportWindow;
-  tracer: ReturnType<typeof getFallbackTracer>;
-  meter: ReturnType<typeof getFallbackMeter>;
-  // Qualified name of the membership-list query handler consulted by
-  // resolveActiveMembershipFn — defaults to TENANT_MEMBERSHIPS_QUERY, overridable via DispatcherOptions.
-  membershipQuery: string;
-};
-
-// Narrowing-helper: AppContext.db ist DbConnection|TenantDb|undefined. Die
-// dispatch-Pfade brauchen DbConnection (oder DbTx aus Caller-Scope) für
-// appendEvent/projection-writes; TenantDb-Branch wird hier ausgeschlossen.
-export function resolveDbSource(
-  ctx: DispatchContext,
-  tx: DbTx | undefined,
-): DbConnection | DbTx | undefined {
-  const { appContext: context } = ctx;
-  return tx ?? (context.db as DbConnection | undefined); // @cast-boundary db-operator
-}
-
-// ctx.appendEvent — append a domain event onto a specific aggregate stream
-// in the current tx, then fire matching inline projections. Core logic
-// lives in appendDomainEventCore; this wrapper just locates dbSource +
-// stringifies the SessionUser id for the shared helper.
-async function appendDomainEvent(
-  ctx: DispatchContext,
-  args: AppendEventArgs,
-  user: SessionUser,
-  tx: DbTx | undefined,
-  callerFeature: string | undefined,
-): Promise<void> {
-  // Sink behind every append surface, so a resolved member stays read-only even via fetchForWriting handles.
-  if (isMemberResolutionPrincipal(user)) throw memberResolutionReadOnlyDenied();
-  const { registry } = ctx;
-  const dbSource = resolveDbSource(ctx, tx);
-  if (!dbSource) {
-    throw new InternalError({
-      message: `ctx.appendEvent("${args.type}") requires a database connection — none is configured.`,
-    });
-  }
-  await appendDomainEventCore(
-    {
-      registry,
-      db: dbSource,
-      tenantId: user.tenantId,
-      userId: String(user.id),
-      callSiteLabel: "ctx.appendEvent",
-      callerFeature,
-    },
-    args,
-  );
-}
-
-// r.systemScope() handlers must go through ctx.systemDb's guarded methods
-// (assertTenantMatch / acknowledgeCrossTenant) instead of plain ctx.db —
-// "system" mode has no tenant filter at all, so silent ctx.db or
-// ctx.dbOutsideTransaction use there is a cross-tenant leak. HandlerContext
-// has no way to make either field optional per handler (isSystem is a
-// runtime-only registry lookup, not a type-level discriminant), so instead
-// of omitting the key we hand back a Proxy that throws on first touch,
-// naming the handler and pointing at the matching ctx.systemDb escape hatch.
-function createSystemScopedDbGuard(
-  handlerType: string,
-  fieldName: "db" | "dbOutsideTransaction",
-  hint: string,
-): TenantDb {
-  return new Proxy({} as TenantDb, {
-    get(_target, prop) {
-      throw new InternalError({
-        message:
-          `Handler "${handlerType}" is r.systemScope()'d — ctx.${fieldName} is unavailable ` +
-          `(it would be unfiltered across every tenant). Use ${hint} ` +
-          `instead (attempted to read "${String(prop)}").`,
-      });
-    },
-  });
-}
-
-export function isMemberResolutionPrincipal(user: SessionUser): boolean {
-  return user.origin === "member-resolution";
-}
-
-async function denyMemberResolutionWrite(): Promise<never> {
-  throw memberResolutionReadOnlyDenied();
-}
-
-// JobRunnerRef's declared type only carries handleEvent, but callers
-// routinely bracket-access a fuller runner — a Proxy denies ANY property read.
-function denyingJobRunnerProxy(): JobRunnerRef {
-  return new Proxy({} as JobRunnerRef, {
-    get() {
-      throw memberResolutionReadOnlyDenied();
-    },
-  });
-}
-
-// Every write/side-effect surface throws, so "no writeAsMember" holds even
-// if a handler tries ctx.write directly on a resolved member principal.
-export function applyMemberResolutionReadOnly(handlerContext: HandlerContext): HandlerContext {
-  return {
-    ...handlerContext,
-    // `db` stays open — executeQuery runs the whole handler in a Postgres READ ONLY
-    // transaction, and its `memberReadOnly` grant denies every raw-runner handout.
-    dbOutsideTransaction: undefined,
-    write: denyMemberResolutionWrite,
-    writeAs: denyMemberResolutionWrite,
-    // A resolved member cannot switch identity either — the target would get a normal, writable context.
-    queryAs: denyMemberResolutionWrite,
-    appendEvent: denyMemberResolutionWrite as AppendEventFn, // @cast-boundary engine-bridge
-    unsafeAppendEvent: denyMemberResolutionWrite,
-    tryAppendEvent: denyMemberResolutionWrite,
-    fetchForWriting: denyMemberResolutionWrite,
-    archiveStream: denyMemberResolutionWrite,
-    restoreStream: denyMemberResolutionWrite,
-    snapshotAggregate: denyMemberResolutionWrite,
-    queryAsMember: denyMemberResolutionWrite,
-    resolveActiveMembership: denyMemberResolutionWrite,
-    scheduleAfterCommit: () => {
-      throw memberResolutionReadOnlyDenied();
-    },
-    // A read as a member gets no system-scope DB, preSave pipeline or file storage.
-    runPreSave: undefined,
-    systemDb: undefined,
-    files: undefined,
-    derivatives: undefined,
-    ...(handlerContext.jobRunner && { jobRunner: denyingJobRunnerProxy() }),
-    ...(handlerContext.notify && { notify: denyMemberResolutionWrite }),
-  };
-}
-
-export async function buildHandlerContext(
-  ctx: DispatchContext,
-  type: string,
-  user: SessionUser,
-  origin: WriteOrigin,
-  tx?: DbTx,
-  afterCommitHooks?: AfterCommitHook[],
-  includeDeleted?: boolean,
-): Promise<HandlerContext> {
-  const { registry, appContext: context, effectiveFeatures, jobRunner, lifecycle } = ctx;
-  const isSystem = registry.isHandlerSystemScoped(type);
-  // The outer dispatcher receives a DbConnection from the server/stack;
-  // AppContext's `db` union also allows TenantDb (for downstream hook calls),
-  // but at this point we're the root of the pipeline — cast is safe.
-  const dbSource = resolveDbSource(ctx, tx);
-  const reqCtx = requestContext.get();
-  // global() writes are write-handler-only; SYSTEM identity switch and unsafeRaw accept write, query or stream escapeHatch.
-  const writeEscapeHatch = registry.getWriteHandler(type)?.escapeHatch;
-  const handlerEscapeHatch =
-    writeEscapeHatch ??
-    registry.getQueryHandler(type)?.escapeHatch ??
-    registry.getStreamHandler(type)?.escapeHatch;
-  const hasIdentitySwitchGrant = isSystem || handlerEscapeHatch !== undefined;
-  const featureName = registry.getHandlerFeature(type);
-  const reportEscapeHatch = createEscapeHatchReporter({
-    handler: type,
-    tenantId: user.tenantId,
-    actor: user.id,
-    sink: context._escapeHatchAuditSink,
-    log: context.log,
-    window: ctx.escapeHatchReportWindow,
-  });
-  const identitySwitchGrantReason =
-    handlerEscapeHatch?.reason ??
-    (isSystem ? `r.systemScope() feature "${featureName ?? "unknown"}"` : undefined);
-  const buildTenantScopedDb = (source: DbConnection | DbTx, signal: AbortSignal | undefined) =>
-    createTenantDb(
-      source,
-      user.tenantId,
-      isSystem ? "system" : "tenant",
-      context.tracer,
-      context.meter,
-      signal,
-      {
-        globalWrites: writeEscapeHatch,
-        unsafeRaw: handlerEscapeHatch,
-        report: reportEscapeHatch,
-        memberReadOnly: isMemberResolutionPrincipal(user),
-        personalDataGate: buildPersonalDataGate(registry, origin),
-      },
-    );
-  // Propagate the request's AbortSignal so every TenantDb query throws when
-  // the client has disconnected — handlers with many sequential queries skip
-  // the rest of the chain instead of burning DB-CPU for results no one reads.
-  const db = dbSource ? buildTenantScopedDb(dbSource, reqCtx?.signal) : undefined;
-  // Unbound pool, tenant-scoped like `db` but never tx-bound — writes
-  // through it survive a rollback of the handler's own transaction. No
-  // AbortSignal here: a client disconnect must not abort a durability write
-  // that is meant to outlive the request. Computed before `systemDb` below
-  // so its guarded escape hatch (ctx.systemDb.outsideTransaction) can wrap
-  // this same TenantDb instead of a second, independently-built one.
-  const outsideTxSource = resolveDbSource(ctx, undefined);
-  const rawDbOutsideTransaction = outsideTxSource
-    ? buildTenantScopedDb(outsideTxSource, undefined)
-    : undefined;
-  const systemDb =
-    isSystem && db
-      ? createUncheckedSystemDb(db, rawDbOutsideTransaction, reportEscapeHatch)
-      : undefined;
-  // Exposed as ctx.db below — the internal `db` above stays the real,
-  // working TenantDb for this function's own use (config/derivatives
-  // accessors, systemDb construction).
-  const exposedDb =
-    isSystem && db
-      ? createSystemScopedDbGuard(
-          type,
-          "db",
-          "ctx.systemDb.assertTenantMatch(...) or ctx.systemDb.acknowledgeCrossTenant(...)",
-        )
-      : db;
-  // Same fail-closed treatment as `exposedDb` — a system handler reaching
-  // ctx.dbOutsideTransaction directly would be a second, unguarded door to
-  // the same cross-tenant access ctx.db closes off above.
-  const dbOutsideTransaction =
-    isSystem && rawDbOutsideTransaction
-      ? createSystemScopedDbGuard(
-          type,
-          "dbOutsideTransaction",
-          "ctx.systemDb.outsideTransaction.assertTenantMatch(...) or " +
-            "ctx.systemDb.outsideTransaction.acknowledgeCrossTenant(...)",
-        )
-      : rawDbOutsideTransaction;
-  const log = context.log?.child({
-    handler: type,
-    tenantId: user.tenantId,
-    userId: user.id,
-    ...(reqCtx && { requestId: reqCtx.requestId }),
-  });
-  const notify = context._notifyFactory ? context._notifyFactory(user, user.tenantId) : undefined;
-  // Mirror notify: only built when the config feature wired its factory.
-  const config =
-    context._configAccessorFactory && db
-      ? context._configAccessorFactory({
-          user: { id: user.id, tenantId: user.tenantId },
-          db,
-          secrets: context.secrets,
-        })
-      : undefined;
-  // ctx.files resolved per-tenant through file-foundation (lazy — the
-  // provider is only resolved when a handle actually does I/O). Boot wires
-  // _fileProviderResolver when a file-provider plugin is mounted; falls back
-  // to a statically-injected context.files (tests).
-  const fileResolver = context._fileProviderResolver;
-  const files = fileResolver ? createFileContext(() => fileResolver(user.tenantId)) : context.files;
-  // ctx.derivatives builds on ctx.files (needs a FileContext to read the
-  // original + write the variant) plus db (to look up the FileRef row) — so
-  // it can only be constructed exactly when files+db both resolved; falls
-  // back to a statically-injected context.derivatives otherwise (tests).
-  const derivatives =
-    files && db
-      ? createDerivativesContext({ files, registry, db, tenantId: user.tenantId })
-      : context.derivatives;
-
-  // Observability — feature-bound metrics handle, so ctx.metrics.inc("foo")
-  // resolves to kumiko_<feature>_foo. Unknown feature falls back to noop
-  // so legacy internal handlers don't crash.
-  const tracer = context.tracer ?? getFallbackTracer();
-  const meter = context.meter;
-  const metrics =
-    meter && featureName ? createMetricsHandle(meter, featureName) : createNoopMetricsHandle();
-  // ctx.metricsFor(featureName) — shared/library code binds to a feature
-  // name of its own choosing instead of the dispatching handler's
-  // (framework#1844). Unregistered names no-op rather than throw, see
-  // createSafeMetricsHandle.
-  const metricsFor = meter
-    ? (targetFeatureName: string) => createSafeMetricsHandle(meter, targetFeatureName)
-    : () => createNoopMetricsHandle();
-
-  // Cross-feature bridge: ctx.query/write share the current tx + afterCommitHooks sink.
-  // queryAs/writeAs to anyone but the caller itself needs r.systemScope() or { escapeHatch } (system-identity-switch.ts).
-  const bridgeSink = afterCommitHooks ?? [];
-  const scheduleAfterCommit = (hook: AfterCommitHook): void => {
-    bridgeSink.push(hook);
-  };
-  const identitySwitchAudit = { reason: identitySwitchGrantReason, report: reportEscapeHatch };
-  const identitySwitch = createGatedIdentitySwitch(
-    `handler "${type}"`,
-    user,
-    hasIdentitySwitchGrant,
-    {
-      // Inherits the caller's origin, so switching to SYSTEM cannot shed an anonymous root.
-      queryAs: (asUser: SessionUser, targetType: string, payload: unknown) =>
-        executeQuery(ctx, targetType, payload, asUser, origin, tx), // @wrapper-known semantic-alias
-      writeAs: (asUser: SessionUser, targetType: string, payload: unknown) =>
-        executeWrite(ctx, targetType, payload, asUser, origin, tx, bridgeSink),
-    },
-    identitySwitchAudit,
-  );
-  // Lazy — creates the reader (which fails closed on SYSTEM_TENANT_ID) only
-  // on first actual use, not on every HandlerContext build.
-  let ungatedMemberReader: MemberReader | undefined;
-  const queryAsMember = createGatedMemberReader(
-    `handler "${type}"`,
-    hasIdentitySwitchGrant,
-    (userId, qn, payload) => {
-      ungatedMemberReader ??= createMemberReaderFn(ctx, user.tenantId, tx);
-      return ungatedMemberReader(userId, qn, payload);
-    },
-    { ...identitySwitchAudit, tenantId: user.tenantId },
-  );
-  const ungatedQueryProjection: ProjectionReader = async <T = Record<string, unknown>>(
-    qualifiedName: string,
-    queryOptions?: { readonly unsafeAllTenants?: boolean },
-  ): Promise<readonly T[]> => {
-    // queryProjection works against both single-stream and multi-stream
-    // projections. MSPs without a table cannot be queried — those are
-    // side-effect-only consumers (no state to read back).
-    const singleProj = registry.getAllProjections().get(qualifiedName);
-    const mspProj = registry.getAllMultiStreamProjections().get(qualifiedName);
-    const projTable = singleProj?.table ?? mspProj?.table;
-    if (!projTable) {
-      const singleNames = [...registry.getAllProjections().keys()];
-      const mspNames = [...registry.getAllMultiStreamProjections().keys()].filter(
-        (n) => registry.getAllMultiStreamProjections().get(n)?.table,
-      );
-      const all = [...singleNames, ...mspNames];
-      throw new InternalError({
-        message:
-          `ctx.queryProjection("${qualifiedName}") — projection not registered, or it is a ` +
-          `table-less MSP (side-effect-only). Known queryable projections: ${all.join(", ") || "(none)"}`,
-      });
-    }
-    const dbSource = resolveDbSource(ctx, tx);
-    if (!dbSource) {
-      throw new InternalError({
-        message: `ctx.queryProjection("${qualifiedName}") requires a database connection — none is configured.`,
-      });
-    }
-    const where =
-      hasTenantColumn(projTable) && queryOptions?.unsafeAllTenants !== true
-        ? { tenantId: user.tenantId }
-        : undefined;
-    const rows = await selectMany<Record<string, unknown>>(dbSource, projTable, where);
-    return rows as readonly T[]; // @cast-boundary engine-payload
-  };
-  const queryProjection = createGatedProjectionReader(
-    `handler "${type}"`,
-    hasIdentitySwitchGrant,
-    ungatedQueryProjection,
-    identitySwitchAudit,
-  );
-  const bridge = {
-    query: (targetType: string, payload: unknown) =>
-      executeQuery(ctx, targetType, payload, user, origin, tx), // @wrapper-known semantic-alias
-    queryAs: identitySwitch.queryAs,
-    write: async (targetType: string, payload: unknown) => {
-      const res = await executeWrite(ctx, targetType, payload, user, origin, tx, bridgeSink);
-      return res;
-    },
-    writeAs: identitySwitch.writeAs,
-    // Strict + unsafe share the same runtime — only the type-surface
-    // differs. The strict signature is what's exposed to typed callers;
-    // unsafe is the explicit escape-hatch for runtime-pluggable events.
-    appendEvent: (async (args: AppendEventArgs) => {
-      await appendDomainEvent(ctx, args, user, tx, registry.getHandlerFeature(type));
-    }) as AppendEventFn, // @cast-boundary engine-bridge
-    unsafeAppendEvent: async (args: AppendEventArgs) => {
-      await appendDomainEvent(ctx, args, user, tx, registry.getHandlerFeature(type));
-    },
-    // Savepoint-scoped append: catches a losing writer's VersionConflictError
-    // without poisoning the rest of the handler's transaction. Bun.SQL/
-    // postgres.js abort the WHOLE begin() on an uncaught statement error
-    // (SQLSTATE 25P02) even if the JS error is caught — runInSavepoint
-    // wraps the append in a real SAVEPOINT so only that nested scope rolls
-    // back on conflict, and subsequent statements in the outer tx (e.g. a
-    // dedup-anchor insert) still succeed. Use when a handler must react to
-    // losing a concurrent-append race instead of failing the whole write.
-    tryAppendEvent: async (args: AppendEventArgs) => {
-      if (!tx) {
-        throw new InternalError({
-          message: `ctx.tryAppendEvent("${args.type}") requires an active transaction — no tx is threaded through this call.`,
-        });
-      }
-      try {
-        const event = await runInSavepoint(tx, (sp) =>
-          appendDomainEventCore(
-            {
-              registry,
-              db: sp as DbRunner,
-              tenantId: user.tenantId,
-              userId: String(user.id),
-              callSiteLabel: "ctx.tryAppendEvent",
-              callerFeature: registry.getHandlerFeature(type),
-            },
-            args,
-          ),
-        );
-        return { ok: true as const, event };
-      } catch (e) {
-        if (e instanceof EventStoreVersionConflictError) {
-          return { ok: false as const, conflict: e };
-        }
-        if (e instanceof EventStoreIdempotentAppendConflictError) {
-          return { ok: false as const, conflict: e };
-        }
-        throw e;
-      }
-    },
-    fetchForWriting: async (args: FetchForWritingArgs): Promise<AggregateStreamHandle> => {
-      const dbSource = resolveDbSource(ctx, tx);
-      if (!dbSource) {
-        throw new InternalError({
-          message: `ctx.fetchForWriting("${args.aggregateId}") requires a database connection — none is configured.`,
-        });
-      }
-      // Stream-version authoritative (same policy as CRUD executor + Block 0).
-      // A single SELECT MAX(version) is cheaper than loading the full stream
-      // when the caller just wants to append — but most callers also want
-      // the events (business-rule checks), so fetch both in parallel.
-      const [storedEvents, fetchedVersion] = await Promise.all([
-        loadAggregate(dbSource, args.aggregateId, user.tenantId, {
-          aggregateType: args.aggregateType,
-        }),
-        getStreamVersion(dbSource, args.aggregateId, user.tenantId),
-      ]);
-      const events = await upcastStoredEvents(storedEvents, registry.getEventUpcasters(), {
-        db: dbSource,
-        tenantId: user.tenantId,
-      });
-
-      // Optimistic concurrency: if the caller knows the version they
-      // worked against (e.g. from a prior read-model row) and the stream
-      // has moved on, fail fast before any downstream work.
-      if (args.expectedVersion !== undefined && args.expectedVersion !== fetchedVersion) {
-        throw new VersionConflictError({
-          entityId: args.aggregateId,
-          expectedVersion: args.expectedVersion,
-          currentVersion: fetchedVersion,
-        });
-      }
-
-      // Handle's internal version bumps on every appendOne so multiple
-      // appends in a row stay in order without re-reading the DB.
-      let handleVersion = fetchedVersion;
-      const appendOne = async (appendArgs: {
-        readonly type: string;
-        readonly payload: unknown;
-      }): Promise<void> => {
-        await appendDomainEvent(
-          ctx,
-          {
-            aggregateId: args.aggregateId,
-            aggregateType: args.aggregateType,
-            type: appendArgs.type,
-            payload: appendArgs.payload,
-          },
-          user,
-          tx,
-          registry.getHandlerFeature(type),
-        );
-        handleVersion += 1;
-      };
-
-      return {
-        events,
-        get version() {
-          return handleVersion;
-        },
-        appendOne,
-        hasEvent: (type: string) => events.some((e) => e.type === type),
-      };
-    },
-    loadAggregate: async (
-      aggregateId: string,
-      loadOptions?: { readonly asOf?: Temporal.Instant },
-    ): Promise<readonly StoredEvent[]> => {
-      const dbSource = resolveDbSource(ctx, tx);
-      if (!dbSource) {
-        throw new InternalError({
-          message: `ctx.loadAggregate("${aggregateId}") requires a database connection — none is configured.`,
-        });
-      }
-      const events = loadOptions?.asOf
-        ? await loadAggregateAsOf(dbSource, aggregateId, user.tenantId, loadOptions.asOf)
-        : await loadAggregate(dbSource, aggregateId, user.tenantId);
-      return upcastStoredEvents(events, registry.getEventUpcasters(), {
-        db: dbSource,
-        tenantId: user.tenantId,
-      });
-    },
-    archiveStream: async (
-      aggregateId: string,
-      archiveArgs: { readonly aggregateType: string; readonly reason?: string },
-    ): Promise<void> => {
-      const dbSource = resolveDbSource(ctx, tx);
-      if (!dbSource) {
-        throw new InternalError({
-          message: `ctx.archiveStream("${aggregateId}") requires a database connection — none is configured.`,
-        });
-      }
-      await archiveStreamHelper(dbSource, {
-        tenantId: user.tenantId,
-        aggregateId,
-        aggregateType: archiveArgs.aggregateType,
-        archivedBy: user.id,
-        reason: archiveArgs.reason,
-      });
-    },
-    restoreStream: async (aggregateId: string): Promise<void> => {
-      const dbSource = resolveDbSource(ctx, tx);
-      if (!dbSource) {
-        throw new InternalError({
-          message: `ctx.restoreStream("${aggregateId}") requires a database connection — none is configured.`,
-        });
-      }
-      await restoreStreamHelper(dbSource, user.tenantId, aggregateId);
-    },
-    isStreamArchived: async (aggregateId: string): Promise<boolean> => {
-      const dbSource = resolveDbSource(ctx, tx);
-      if (!dbSource) {
-        throw new InternalError({
-          message: `ctx.isStreamArchived("${aggregateId}") requires a database connection — none is configured.`,
-        });
-      }
-      return isStreamArchived(dbSource, user.tenantId, aggregateId);
-    },
-    snapshotAggregate: async (snapshotArgs: {
-      readonly aggregateId: string;
-      readonly aggregateType: string;
-      readonly version: number;
-      readonly state: Record<string, unknown>;
-      readonly snapshotVersion?: number;
-    }): Promise<void> => {
-      const dbSource = resolveDbSource(ctx, tx);
-      if (!dbSource) {
-        throw new InternalError({
-          message: `ctx.snapshotAggregate("${snapshotArgs.aggregateId}") requires a database connection — none is configured.`,
-        });
-      }
-      await saveSnapshot(dbSource, {
-        aggregateId: snapshotArgs.aggregateId,
-        tenantId: user.tenantId,
-        aggregateType: snapshotArgs.aggregateType,
-        version: snapshotArgs.version,
-        state: snapshotArgs.state,
-        snapshotVersion: snapshotArgs.snapshotVersion,
-      });
-    },
-    loadAggregateWithSnapshot: async <TState extends Record<string, unknown>>(
-      aggregateId: string,
-      reducer: SnapshotReducer<TState>,
-      initial: TState,
-      loadOptions?: Omit<LoadAggregateWithSnapshotOptions, "upcastEvent">,
-    ): Promise<LoadAggregateWithSnapshotResult<TState>> => {
-      const dbSource = resolveDbSource(ctx, tx);
-      if (!dbSource) {
-        throw new InternalError({
-          message: `ctx.loadAggregateWithSnapshot("${aggregateId}") requires a database connection — none is configured.`,
-        });
-      }
-      // Upcaster-aware: pass an upcastEvent callback so loadAggregateWithSnapshot
-      // walks every delta through the registered chain before invoking the
-      // user's (sync) reducer. Async upcasters (DB-enrichment) are awaited
-      // inside loadAggregateWithSnapshot — feature authors never see legacy
-      // payload shapes regardless of which load path they chose.
-      const upcasters = registry.getEventUpcasters();
-      const upcastCtx = { db: dbSource, tenantId: user.tenantId };
-      return loadAggregateWithSnapshot<TState>(
-        dbSource,
-        aggregateId,
-        user.tenantId,
-        reducer,
-        initial,
-        {
-          ...loadOptions,
-          upcastEvent: (event) => upcastStoredEvent(event, upcasters, upcastCtx), // @wrapper-known semantic-alias
-        },
-      );
-    },
-    queryProjection,
-    // Thin pass-through: one resolve impl lives on the dispatcher, the
-    // handler surface just forwards the call so both entry points (login
-    // handler via ctx.resolveAuthClaims, switch-tenant route via
-    // dispatcher.resolveAuthClaims) cannot drift.
-    resolveAuthClaims: (claimsUser: SessionUser) => resolveAuthClaimsFn(ctx, claimsUser), // @wrapper-known semantic-alias
-
-    // Thin pass-through, same reasoning as resolveAuthClaims above — one
-    // resolve impl lives on the dispatcher so callers can't drift apart.
-    // Internally queries memberships as SYSTEM, so it needs the same grant
-    // as a SYSTEM queryAs/writeAs (r.systemScope() or escapeHatch).
-    resolveActiveMembership: (userId: string, tenantId: TenantId) => {
-      if (!hasIdentitySwitchGrant) throw systemIdentitySwitchDenied(`handler "${type}"`);
-      return resolveActiveMembershipFn(ctx, userId, tenantId, INTERACTIVE_SIGN_IN_POLICY); // @wrapper-known semantic-alias
-    },
-
-    // Needs the same grant as a SYSTEM queryAs.
-    queryAsMember,
-
-    // Feature-effective check for in-handler opt-in logic. Scope:
-    // **current user's tenant** — for cross-tenant lookups (rare,
-    // SysAdmin operations) read effectiveFeatures(otherTenantId) directly.
-    // When the feature-toggles or tier-engine feature isn't wired (no
-    // effectiveFeatures callback), always returns true — apps without
-    // tier-cuts treat all features on.
-    //
-    // Falls back to the live trialGate when the sync set says the feature
-    // is off — the sync set never contains trial-tier features (time-
-    // derived, can't boot-cache), so without this a trial tenant checking
-    // a companion feature's toggle would silently read `false` even though
-    // the dispatch gate already lets trial-tier handlers run.
-    hasFeature: async (featureName: string): Promise<boolean> => {
-      if (!effectiveFeatures) return true;
-      if (effectiveFeatures(user.tenantId).has(featureName)) return true;
-      if (!effectiveFeatures.trialGate) return false;
-      return effectiveFeatures.trialGate(user.tenantId, featureName);
-    },
-  };
-
-  // Registry is always the dispatcher's registry — injecting it here lets
-  // tests/callers pass `context` without `registry` and still get a valid
-  // HandlerContext. The spread-then-assign order matters: anything in
-  // `context` can be overridden, but we want the authoritative registry
-  // from the dispatcher's own closure to win.
-  // ctx.tz is always present. tenant reads tenant:config:timezone through
-  // the already-built `config` accessor (raw qualified-key — no import of
-  // the tenant feature, framework/pipeline stays bundled-features-free);
-  // falls back to "UTC" when config is missing or the key is unset. user
-  // comes from SessionUser.timezone (set at login), else falls back to
-  // tenant (createTzContext's own default). An app-injected GeoTzProvider
-  // (context.geoTzProvider) feeds ctx.tz.fromCoordinates / fromAddress.
-  //
-  // Resolving this via `config()` is a config-table SELECT on every single
-  // dispatch (fw#2462) — ctx.tenantTimezoneCache memoizes it per tenant.
-  // dispatch-write.ts invalidates on config:write:set/reset for this key.
-  // Only populate from a tx-free read (tx === undefined): a write-path
-  // lookup runs inside the caller's own open transaction, and caching a
-  // value read there could memoize state from a write earlier in that same
-  // transaction before it's known to commit. Reads always consult the
-  // cache regardless of tx, since serving a slightly stale cached value on
-  // the write path is harmless (bounded by ttlMs) and avoids reintroducing
-  // the SELECT on the hot path this fix targets.
-  let tenantTz: unknown;
-  if (config === undefined) {
-    tenantTz = undefined;
-  } else {
-    const cachedTz = ctx.tenantTimezoneCache.get(user.tenantId);
-    if (cachedTz) {
-      tenantTz = cachedTz.value;
-    } else {
-      tenantTz = await config(TENANT_TIMEZONE_CONFIG_KEY);
-      if (tx === undefined) {
-        ctx.tenantTimezoneCache.set(
-          user.tenantId,
-          typeof tenantTz === "string" ? tenantTz : undefined,
-        );
-      }
-    }
-  }
-  // Guarded against garbage: an unvalidated string here (free-form config
-  // key, legacy JWT claim predating validation) blows up every ctx.tz call
-  // for the whole tenant with a RangeError. Fall back to UTC/tenant instead
-  // of trusting the raw value.
-  const safeTenantTz =
-    typeof tenantTz === "string" && isValidIanaTimeZone(tenantTz) ? tenantTz : "UTC";
-  const safeUserTz =
-    user.timezone !== undefined && isValidIanaTimeZone(user.timezone) ? user.timezone : undefined;
-  const tz = createTzContext({
-    ...(context.geoTzProvider !== undefined ? { geoTz: context.geoTzProvider } : {}),
-    tenant: safeTenantTz,
-    ...(safeUserTz !== undefined && { user: safeUserTz }),
-  });
-
-  // ctx.locale — request-layer signal (X-Locale header → Accept-Language,
-  // resolved once at the HTTP boundary by request-id-middleware.ts) wins;
-  // then SessionUser.locale (set at login, see fw#2333 — stale until
-  // re-login, same tradeoff as ctx.tz.user); falls back to the app's
-  // boot-configured defaultLocale, then DEFAULT_LOCALE. Mirrors ctx.tz's
-  // Request → Session → Boot-Default chain above.
-  const safeUserLocale =
-    user.locale !== undefined && isValidLocaleTag(user.locale) ? user.locale : undefined;
-  const locale = reqCtx?.locale ?? safeUserLocale ?? context.defaultLocale ?? DEFAULT_LOCALE;
-
-  const handlerContext = {
-    ...context,
-    registry,
-    db: exposedDb,
-    dbOutsideTransaction,
-    ...(systemDb && { systemDb }),
-    log,
-    notify,
-    ...(config && { config }),
-    ...(files && { files }),
-    ...(derivatives && { derivatives }),
-    // preSave hooks need `changes`/`previous`/`isNew`, which only exist once
-    // a handler actually starts building its write — bound here so entity
-    // CRUD handlers (entity-handlers.ts) can forward it to the executor
-    // (kumiko-framework#1672).
-    ...(lifecycle && {
-      runPreSave: (
-        handlerName: string,
-        changes: Record<string, unknown>,
-        previous: Readonly<Record<string, unknown>>,
-        isNew: boolean,
-      ) => lifecycle.runPreSave(handlerName, changes, previous, isNew, context),
-    }),
-    tracer,
-    metrics,
-    metricsFor,
-    tz,
-    locale,
-    // Cancellation signal flows from the HTTP middleware via
-    // requestContext. Conditional spread so non-HTTP entry-points
-    // (jobs, dispatcher MSP-applies) don't get a phantom signal that
-    // would always read aborted=false but feel meaningful.
-    ...(reqCtx?.signal ? { signal: reqCtx.signal } : {}),
-    // Propagate the feature-toggle resolver so the lifecycle pipeline,
-    // MSP runner, and ctx.hasFeature all pull from the same source.
-    ...(effectiveFeatures && { effectiveFeatures }),
-    // Lets write handlers call ctx.jobRunner.dispatch(...) directly, same
-    // as a follow-up job would (test-stack.ts wires the matching runner).
-    ...(jobRunner && { jobRunner }),
-    // ctx.user als Convenience-Alias auf event.user. Der typisch-
-    // intuitive Pfad „der Context kennt seinen User" — ohne den
-    // schreiben Handler `event.user.tenantId` und brechen sich die
-    // Finger an typo-resistenten ctx.user-Patterns. Identisch zum
-    // event.user-Wert; Identity-Switches nutzen weiterhin queryAs/writeAs.
-    user,
-    _userId: user.id,
-    _tenantId: user.tenantId,
-    _handlerType: type,
-    scheduleAfterCommit,
-    ...(includeDeleted && { includeDeleted: true }),
-    ...bridge,
-  } as HandlerContext; // @cast-boundary engine-bridge
-
-  // A resolved member principal is read-only by construction.
-  return isMemberResolutionPrincipal(user)
-    ? applyMemberResolutionReadOnly(handlerContext)
-    : handlerContext;
-}
-
-// Resolves the agent risk the irreversible-operation gate should attribute to
-// `type` when it becomes the request's entry handler — same defaulting rule
-// (resolveAgentExposure) the agent manifest itself uses, so the two never diverge.
-function resolveEntryHandlerRisk(
-  registry: Registry,
-  type: string,
-  operation: "query" | "write" | "stream",
-): AgentRisk {
-  if (operation === "write") {
-    return resolveAgentExposure(registry.getWriteHandler(type) ?? {}, "write").risk;
-  }
-  if (operation === "stream") {
-    // StreamHandlerDef carries no `agent` hint — streams can never declare
-    // "high", so nested irreversible writes stay conservatively denied.
-    return "mid";
-  }
-  return resolveAgentExposure(registry.getQueryHandler(type) ?? {}, "query").risk;
-}
-
-// Wrap handler execution in a dispatcher.handler span AND emit the standard
-// dispatcher metrics (duration + error counter). Errors are re-thrown so
-// control flow stays identical to the uninstrumented path.
-//
-// Writes are special-cased: executeWriteInner converts thrown handler errors
-// into a WriteResult with isSuccess=false (rather than letting them bubble).
-// We inspect the result to paint the dispatcher span + error counter on
-// those structural failures too — otherwise "handler threw" would only show
-// up when the caller forgot to use writeFailure().
-export async function runHandlerInstrumented<T>(
-  ctx: DispatchContext,
-  type: string,
-  operation: "query" | "write" | "stream",
-  user: SessionUser,
-  inner: () => Promise<T>,
-): Promise<T> {
-  const { tracer: dispatcherTracer, meter: dispatcherMeter, registry } = ctx;
-  const start = performance.now();
-  // Outcome recorded inside the withSpan callback, emitted in finally so
-  // success/failure/throw all hit a single metric-emit path.
-  let success = true;
-  let errorClass: string | undefined;
-
-  try {
-    return await dispatcherTracer.withSpan(
-      "kumiko.dispatcher.handler",
-      {
-        attributes: dispatcherSpanAttributes(
-          type,
-          operation,
-          user,
-          registry.getHandlerFeature(type),
-        ),
-      },
-      async (span) => {
-        try {
-          // #3043 — everything the handler writes, including entity-executor
-          // writes below it, is attributed to this handler.
-          const result = await runWithOrigin(
-            {
-              handler: type,
-              feature: registry.getHandlerFeature(type),
-              entryHandler: { qn: type, risk: resolveEntryHandlerRisk(registry, type, operation) },
-            },
-            inner,
-          );
-          if (operation === "write" && isFailedWriteResult(result)) {
-            success = false;
-            errorClass = result.error?.code ?? "UnknownError";
-            span.setStatus("error", errorClass);
-          }
-          return result;
-        } catch (error) {
-          success = false;
-          errorClass = error instanceof Error && error.name ? error.name : "UnknownError";
-          throw error;
-        }
-      },
-    );
-  } finally {
-    if (!success && errorClass) {
-      emitDispatcherError(dispatcherMeter, { handler: type, errorClass });
-    }
-    emitDispatcherHandler(
-      dispatcherMeter,
-      { handler: type, success },
-      (performance.now() - start) / 1000,
-    );
-  }
-}
-
-// Generator-native counterpart to runHandlerInstrumented — a stream's
-// lifetime spans every `for await` pull the caller makes, so the span
-// can't be scoped via withSpan's single-callback shape. Drive the inner
-// generator manually (not yield*): each pull must run inside
-// observabilityContext.run({ activeSpan: span }) so handler-side
-// startSpan() calls parent onto the dispatcher span. finally forwards
-// .return() to the inner generator (yield* did that for free). Metrics
-// land in the same finally path so success/failure/throw/abort all hit
-// one emit, like the Promise path.
-export async function* runStreamInstrumented<T>(
-  ctx: DispatchContext,
-  type: string,
-  user: SessionUser,
-  inner: () => AsyncGenerator<T>,
-): AsyncGenerator<T> {
-  const { tracer: dispatcherTracer, meter: dispatcherMeter, registry } = ctx;
-  const start = performance.now();
-  let success = true;
-  // Set only once `inner()` has drained on its own — NOT set when the
-  // consumer walks away early (generator.return(), the normal SSE-abort
-  // path in api/routes.ts). yield* forwards that .return() straight through
-  // without throwing, so without this flag an aborted stream would fall
-  // through to the success-metric branch below and get counted as a clean
-  // completion.
-  let completedNormally = false;
-  let errorClass: string | undefined;
-  const span = dispatcherTracer.startSpan("kumiko.dispatcher.handler", {
-    attributes: dispatcherSpanAttributes(type, "stream", user, registry.getHandlerFeature(type)),
-  });
-  const it = inner();
-  // Each pull re-enters both scopes: AsyncLocalStorage does not survive a
-  // generator suspension, so wrapping inner() once would leave every event a
-  // stream writes unattributed.
-  const inScope = <R>(fn: () => R): R =>
-    runWithOrigin(
-      {
-        handler: type,
-        feature: registry.getHandlerFeature(type),
-        entryHandler: { qn: type, risk: resolveEntryHandlerRisk(registry, type, "stream") },
-      },
-      () => observabilityContext.run({ activeSpan: span }, fn),
-    );
-  try {
-    let next = await inScope(() => it.next());
-    while (!next.done) {
-      yield next.value;
-      next = await inScope(() => it.next());
-    }
-    completedNormally = true;
-    return next.value;
-  } catch (error) {
-    success = false;
-    errorClass = error instanceof Error && error.name ? error.name : "UnknownError";
-    if (error instanceof Error) span.recordException(error);
-    span.setStatus("error", errorClass);
-    throw error;
-  } finally {
-    // forward close so inner()'s finally still fires — yield* did this for free
-    try {
-      await inScope(() => it.return?.(undefined));
-    } catch (closeError) {
-      // Only fold this in when nothing has already failed — a close-time
-      // error while an earlier error is in flight would mask the real
-      // cause reported above.
-      if (success) {
-        success = false;
-        errorClass =
-          closeError instanceof Error && closeError.name ? closeError.name : "UnknownError";
-        span.setStatus("error", errorClass);
-      }
-    }
-    span.end();
-    if (!success && errorClass) {
-      emitDispatcherError(dispatcherMeter, { handler: type, errorClass });
-    }
-    // Consumer-abort (generator.return(), the normal SSE Tab-close path)
-    // is not a handler failure — label it outcome:"aborted" so live-stream
-    // success rates stay meaningful. True handler throws keep success:false.
-    emitDispatcherHandler(
-      dispatcherMeter,
-      {
-        handler: type,
-        success,
-        outcome: completedNormally ? "completed" : "aborted",
-      },
-      (performance.now() - start) / 1000,
-    );
-  }
-}
-
-// Feature-toggle gate. Returns the error to fold into a WriteFailure in the
-// write path, or throws for the query path (where throws flow through the
-// same outer instrumentation wrapper as other dispatcher errors).
-//
-// When `effectiveFeatures` is not wired (tests, apps without feature-toggles
-// loaded), every handler is treated as enabled — the gate is a pure
-// pass-through in that common case.
-export async function checkFeatureEnabled(
-  ctx: DispatchContext,
-  qualifiedHandler: string,
-  tenantId: TenantId,
-): Promise<import("../errors").FeatureDisabledError | undefined> {
-  const { effectiveFeatures, registry } = ctx;
-  if (!effectiveFeatures) return undefined;
-  const owner = registry.getHandlerFeature(qualifiedHandler);
-  // skip: handler without an owning feature cannot be toggled — shouldn't
-  // happen for registry-built handlers, but guards against edge-case
-  // runtime injections.
-  if (!owner) return undefined;
-  const set = effectiveFeatures(tenantId);
-  if (set.has(owner)) return undefined;
-  // Feature is off for the stored tier — give the live trial-gate a last
-  // chance. Time-derived (tenant.inserted_at + window), so it can't live in
-  // the boot-cached sync resolver; consulted only on this already-disabled
-  // cold path, never on the hot enabled path.
-  if (effectiveFeatures.trialGate && (await effectiveFeatures.trialGate(tenantId, owner))) {
-    return undefined;
-  }
-  return new FeatureDisabledError(owner, qualifiedHandler);
-}
-
-export async function ensureFeatureEnabled(
-  ctx: DispatchContext,
-  qualifiedHandler: string,
-  tenantId: TenantId,
-): Promise<void> {
-  const err = await checkFeatureEnabled(ctx, qualifiedHandler, tenantId);
-  if (err) throw err;
-}
-
-// systemScope handlers get a default per-tenant limit unless the app declared its own or
-// opted out — but ONLY when a resolver is actually configured (an app without the
-// rate-limiting feature must not start throwing InternalError on every systemScope call
-// just because it never opted into L3), and never for a SYSTEM-identity caller (jobs,
-// replay, internal queryAs(SYSTEM,...) — those are not the tenant-facing traffic this
-// default protects against).
-// per: "tenant+handler" (not "tenant") — a per-handler bucket so one hot systemScope
-// handler cannot starve every other systemScope handler's shared tenant quota.
-const DEFAULT_SYSTEM_SCOPE_RATE_LIMIT: RateLimitOption = {
-  per: "tenant+handler",
-  limit: 600,
-  windowSeconds: 60,
-};
-
-function resolveEffectiveRateLimit(
-  rateLimit: RateLimitOption | undefined,
-  isSystemScope: boolean,
-  user: SessionUser,
-  resolver: AppContext["rateLimit"],
-): RateLimitOption | undefined {
-  if (rateLimit) return rateLimit;
-  if (!isSystemScope) return undefined;
-  if (!resolver) return undefined;
-  if (isSystemIdentity(user)) return undefined;
-  return DEFAULT_SYSTEM_SCOPE_RATE_LIMIT;
-}
-
-// L3 rate limit gate. Called by both query and write paths before
-// access-check. Reasoning:
-//   - handler without rateLimit and not systemScope → no-op
-//   - systemScope handler without rateLimit → DEFAULT_SYSTEM_SCOPE_RATE_LIMIT
-//     applies, unless no resolver is configured or the caller is SYSTEM
-//   - app booted without rateLimit resolver → InternalError so the
-//     misconfig surfaces immediately, not on first 429 (only reachable once
-//     an effective rate limit was actually resolved above)
-//   - bucket builder returns "skip" (e.g. ip-based but no client IP):
-//     pass through. ip-modes are commonly used at L1/L2 middleware
-//     where the IP comes from Hono directly; falling back to "skip"
-//     here keeps non-HTTP entry-points (jobs, MSPs) functional.
-export async function enforceRateLimit(
-  ctx: DispatchContext,
-  rateLimit: RateLimitDeclaration | undefined,
-  handlerName: string,
-  user: SessionUser,
-  isSystemScope: boolean,
-): Promise<void> {
-  const { appContext: context } = ctx;
-  // skip: handler explicitly opts out via rateLimit: { disabled: true }
-  if (isRateLimitDisabled(rateLimit)) return;
-  const effective = resolveEffectiveRateLimit(rateLimit, isSystemScope, user, context.rateLimit);
-  // skip: no rateLimit declared and not systemScope (or resolved to no-op) — nothing to enforce
-  if (!effective) return;
-  const reqCtx = requestContext.get();
-  const bucket = buildBucketKey(effective, {
-    handlerName,
-    user,
-    ip: reqCtx?.ip,
-  });
-  // skip: ip-bucket + no IP (non-HTTP entry point) — pass through before
-  // requiring a resolver; HTTP path always has an IP + L1/L2 middleware.
-  if (bucket.kind === "skip") return;
-  if (!context.rateLimit) {
-    throw new InternalError({
-      message: `Handler "${handlerName}" declares rateLimit but no RateLimitResolver is configured. Load the rate-limiting feature or remove the option.`,
-    });
-  }
-  await context.rateLimit.enforce(bucket.key, {
-    limit: effective.limit,
-    windowSeconds: effective.windowSeconds,
-    cost: effective.cost,
-  });
-}
-
-// Build the per-hook context every auth-claims invocation gets. Claims
-// hooks run OUTSIDE any request transaction (login is itself the root
-// operation, not a nested call) and read-only — so the TenantDb is
-// scoped as "tenant" and no tx is threaded through. Cross-tenant lookups
-// are denied: queryAs only reaches the claims user itself (no escapeHatch declaration site here).
-function buildAuthClaimsContext(ctx: DispatchContext, user: SessionUser): AuthClaimsContext {
-  const { appContext: context } = ctx;
-  const dbSource = resolveDbSource(ctx, undefined);
-  if (!dbSource) {
-    throw new InternalError({
-      message: "dispatcher.resolveAuthClaims requires a database connection — none is configured.",
-    });
-  }
-  const db = createTenantDb(
-    dbSource,
-    user.tenantId,
-    "tenant",
-    context.tracer,
-    context.meter,
-    undefined,
-    {
-      report: createEscapeHatchReporter({
-        handler: "r.authClaims hook",
-        tenantId: user.tenantId,
-        actor: user.id,
-        sink: context._escapeHatchAuditSink,
-        log: context.log,
-        window: ctx.escapeHatchReportWindow,
-      }),
-    },
-  );
-  const configAccessor = context._configAccessorFactory
-    ? context._configAccessorFactory({
-        user: { id: user.id, tenantId: user.tenantId },
-        db,
-        secrets: context.secrets,
-      })
-    : undefined;
-  const identitySwitch = createGatedIdentitySwitch("r.authClaims hook", user, false, {
-    // Login is itself the root operation; the hook context is read-only.
-    queryAs: (asUser: SessionUser, qn: string, payload: unknown) =>
-      executeQuery(ctx, qn, payload, asUser, rootWriteOrigin(ctx.registry, qn, asUser)), // @wrapper-known semantic-alias
-    writeAs: async () => {
-      throw new InternalError({
-        message: "r.authClaims hook context has no writeAs — auth-claims hooks are read-only.",
-      });
-    },
-  });
-  return {
-    db,
-    queryAs: identitySwitch.queryAs,
-    ...(configAccessor && { config: configAccessor }),
-  };
-}
-
-export async function resolveAuthClaimsFn(
-  ctx: DispatchContext,
-  user: SessionUser,
-): Promise<Record<string, unknown>> {
-  const { registry, appContext: context } = ctx;
-  const hooks = registry.getAuthClaimsHooks();
-  if (hooks.length === 0) return {};
-  return runAuthClaimsResolver({
-    user,
-    hooks,
-    contextFactory: (claimsUser: SessionUser) => buildAuthClaimsContext(ctx, claimsUser),
-    ...(context.log && { log: context.log }),
-  });
-}