@voltro/plugin-audit 0.18.0 → 0.19.0

package/CHANGELOG.md

@@ -39,6 +39,159 @@ _Changes staged for the next release accumulate here (rolled up from
 
 ---
 
+## [0.19.0] — 2026-07-29
+
+### ⚠ BREAKING
+
+- **@voltro/plugin-audit** — **The durable audit sink stops writing credentials to a log table by default.** `redactInput` defaults to `'all'` — the payload becomes `{ __redacted: 'all' }`, which still proves a payload existed. `redactInput: 'none'` restores the previous behaviour, and a function gives field-level control.
+
+  `AuditEvent.input` is the raw mutation input, so an unredacted trail is where a password change, an API key at issuance and a PAT land — the one place nobody thinks to look for a credential. Losing payload detail is visible the first time you read a row; leaking a credential is not visible at all, which is why the default moved rather than staying opt-in.
+
+  **It is deliberately NOT driven by `.serverOnly()` / `.sensitive()`,** which is the design a consumer proposed and the one that cannot work: those markers live on TABLE COLUMNS and this is a mutation's INPUT. `changePassword({ oldPassword, newPassword })` has no column to consult, so a marker-driven default would cover exactly 0% of the motivating case while reading, to whoever configured it, like protection. (`.sensitive()` is also the export axis, not "unsafe to log" — the category error the three-marker table exists to prevent.)
+
+  Also:
+
+  - **`record: 'all' | 'errors' | predicate`** — filters by OUTCOME, where `include`/`exclude` filter by tag. `'errors'` is the forensic core and pairs with `plugin-versioning` for the successful writes. **`'all'` stays the default** on purpose: defaulting to errors would silently stop recording successes on upgrade, and "what did this compromised account touch" is answered by successes. - **`errorTag`** — the typed error's `_tag`, flattened out of the `outcome` json and indexable. `null` for an untagged failure rather than a guess: "this had no tag" and "the tag is 'Error'" are different, and a column that invents the second makes every filter on it quietly wrong. - **Two indices for the questions asked under pressure** — `(subjectId, status, at)` and `(tenantId, status, at)`. "Every denied call by subject X in the last 30 days" and "every failure against tenant Y" were both unindexed; the existing `byAuditTag` / `byAuditTrace` cannot serve either. - **Retention registers itself** — 365 days, `VOLTRO_AUDIT_LOG_TTL_HOURS`, drained by the boot sweep. "Pair it with the governance sweep" was a docs sentence rather than a default, so nobody did. - **Erasure is deliberately NOT auto-registered.** Erasing a subject must not delete the record that they were refused four hundred times — that record *is* the evidence. Anonymise instead; the docs carry the `subjectScopes` entry to paste, and it stays a decision the app makes explicitly.
+
+  **Migration** — `voltro update` prints it (`0.19.0/02_audit-redact-input-default`, `manual`, and it fires only for apps that mount the plugin). Nothing stops compiling and existing rows are untouched; what changes is what the NEXT row records. Keep the new default unless you know your mutation inputs carry no secrets; pass a function for field-level control; or opt back in explicitly with `redactInput: 'none'`. The codemod deliberately does NOT write `'none'` into your config — a transform could do it perfectly, which is exactly why it must not: it would pin every adopter to the behaviour the default moved away from and report the migration as complete.
+
+  *Why this is `BREAKING` and not `Changed`: it is a silent behaviour change on upgrade. Nothing fails, which is the problem — an operator who never reads this section keeps a trail that has quietly lost its payload detail. `BREAKING` is what routes it into `voltro update`.*
+- **@voltro/cli, @voltro/database** — **`.serverOnly()` now gates where it said it did.** A wire-reachable query that declares a `.serverOnly()` column of its source table in its `output` fails the boot under `voltro serve`, and makes `voltro doctor` exit non-zero. `voltro dev` still warns.
+
+  It shipped as one `log.warn` and nothing else — in every command — while `ColumnBuilder.serverOnly()`'s own doc comment and the seeded `AGENTS.md` marker table both said "**the boot audit — it FAILS the boot**, it does not warn". A team read the strong sentence, adopted the marker on four credential columns, injected a deliberate leak to check, and watched the server come up serving the leaking query. That is the register this repo keeps meeting from a new angle: *a check that prints instead of gating still reads as coverage* — here on the one marker whose entire job is the enforcement.
+
+  Two smaller things went with it, both of which had misled the reporter:
+
+  - **The message names the command.** It was emitted through a module-level logger scoped `voltro:dev`, so a warning from `voltro serve` announced itself as dev — which is why they concluded, and reported, that the audit does not run in production at all. It did; it just misattributed itself and stopped nothing. - **The audit is computed once, in `loadDiscovered`**, the discovery dev / serve / doctor / check all share — the same reasoning `validateRegisteredRelations()` lives there for. Dev and serve may disagree about what a leak COSTS; they must not disagree about what a leak IS.
+
+  `VOLTRO_SERVER_ONLY` moves the line both ways: `strict` fails `voltro dev` too, `warn` downgrades serve, `off` silences it. The downgrades are documented rather than hidden, because the alternative to a stated escape hatch is deleting the marker, and a check whose only way out is to disable it gets disabled.
+
+  `voltro check` deliberately does NOT run it: it has a live-api mode with no access to your table definitions, and a rule that fires in one of its two modes is worse than one that fires in neither.
+
+  **Migration** — `voltro update` prints it (`0.19.0/01_server-only-gates-the-boot`, `manual`, and it fires only for apps that actually use the marker). Run `voltro doctor` BEFORE you deploy: it reports exactly what `voltro serve` will now refuse, with no deploy involved. Each finding has two honest fixes and only its author can choose — the column is not wire-safe (drop it from the query's `output`, keep the marker), or the marker is wrong (drop the `.serverOnly()`). Do not substitute `.encrypted()`: that is the at-rest axis, the runtime decrypts for the handler, and reading it as "safe to expose" is the category error the three-marker table exists to prevent. To ship while triaging, `VOLTRO_SERVER_ONLY=warn` downgrades serve back to a warning — a bridge, not a setting to keep.
+
+  *Why this is `BREAKING` and not `Changed`: no signature moves and nothing that compiled stops compiling, so the literal type-level test does not catch it. It can still turn a booting production app into one that refuses — which is the point, the boot it refuses is the one shipping the column — and that failure lands at DEPLOY time. Filing it as `Changed` would have kept it out of the one section the stability contract names as the migration path, and out of `voltro update` entirely, because codemods hang off `BREAKING`. Both doc claims were corrected in the same change, so the `.d.ts`, the agent template and the docs site now describe the same behaviour.*
+
+### Added
+
+- **@voltro/plugin-audit** — **`auditByTrace` / `auditBySubject` — the read side of the correlation join.**
+
+  `byAuditTrace` and `byAuditSubjectStatus` shipped in the same release with **no caller**. That is the identical defect the versioning side had and that its own entry points were added to fix, repeated on the other half of the join one file away: an index nobody can enter is a query the app still hand-writes, and the docs then demonstrate a raw select over a framework-internal table.
+
+  It surfaced by checking an adopting team's design document against the code rather than from memory. Their §6 asks for *"a read-side composition joining `_voltro_row_history` × the audit sink (on `traceId`) × `actors`"* — which needs BOTH halves to have an entry point, or neither is usable.
+
+  ```ts
+  const calls   = await auditByTrace(ctx.store, traceId)                     // who called, and was it refused
+  const changed = await historyByTrace(ctx.store, traceId, tenantId)         // what it changed
+  const denied  = await auditBySubject(ctx.store, actorId, { status: 'error' })
+  ```
+
+  `auditBySubject` takes `status` as a real argument rather than leaving the caller to filter in JS — the index is `(subjectId, status, at)`, so a filter applied after fetching would not use it. `limit` defaults to 100, because an actor's history is unbounded and an entry point that returns all of it is one you call once in production.
+
+  *They were also not exported from the package index when first written — built, tested, and unreachable. Caught before shipping; worth recording because "it has a test" and "a consumer can call it" are different claims.*
+- **@voltro/database, @voltro/runtime, @voltro/protocol, @voltro/cli, @voltro/plugin-versioning, @voltro/plugin-audit, @voltro/voltro** — **`ChangeEvent` carries the calling `traceId` and `subjectId`** — the join key that lets `plugin-versioning` (what changed) and `auditPlugin` (who called, and whether they were refused) be read as one trail.
+
+  Both halves shipped and neither could be joined to the other. A consumer put it exactly right: *"we are not asking you to build our audit feature. We are asking for the join key that lets anyone build one on the three pieces you have already shipped."*
+
+  **The mechanism, and the part that was an empirical question rather than a design one.** Identity is known one layer up (`subject` in the middleware, `traceId` at the rpc boundary) and the event is created several layers down, inside each dialect store's private emit. Threading a context argument through every `DataStore` method to reach it would change the port every driver implements, for metadata that is ambient by nature — so it rides an `AsyncLocalStorage` (`@voltro/database`'s `writeAttribution.ts`), the same shape as the existing trace and replica-routing contexts.
+
+  Whether that survives a SQL store was *not* obvious: the write goes through `ManagedRuntime.runPromise`, so the read happens inside an Effect fiber, and a scheduler draining fibers from a shared loop would run them under the async context of whoever created the drain. If that were true the attribution would be silently ABSENT — a join key that is simply never there, on a trail nobody checks until an incident. Verified against a real sqlite store, including the transactional path (which queues events and flushes them post-commit, *outside* the scope — which is why the stamp goes at event CREATION, not delivery).
+
+  - **One call site, not one per method.** The scope is established at the rpc executor boundary, so it covers every write the handler makes — `ctx.store`, `EffectStore`, the crud helpers, a plugin interceptor's own writes. Wrapping the store middleware instead would have meant wrapping each mutating method and hoping the next one added remembers. - **`subjectId` is the same identity `audit()` stamps** (`actingUserId`, now exported so there is one source). Two answers to "who wrote this row" on one write would be worse than one; which API *key* was used is recoverable from the audit row sharing the `traceId`. - **Absent is a fact, not a gap** — no request behind the write (seed, startup hook, schedule, workflow step), or an event from another replica, where stamping the local ambient trace would attribute a remote write to a local call. The keys are omitted rather than set to `undefined`, so an unattributed event is byte-identical to one from before this existed. - **`dev.ts` and `serveCommand.ts` no longer hand-mirror the plugin fan-out.** They built that object literal separately in two files; a field added to one and not the other gives a plugin the data in dev and silence in production, and both files typecheck alone. Now `toPluginChangeEvent`.
+
+  `_voltro_row_history` gains `traceId` / `subjectId` plus `byTrace` and `bySubject` indices — "what did this call touch" and "what did this actor touch" were previously unanswerable at any speed, since `byRow` requires already knowing which row you are asking about. `changedBy` now prefers the caller over the row's `audit()` stamp, which fixes a reported case for free: the stamp is `null` for every write through the boot store, so a login route produced version rows with no actor at all.
+
+  *`apiSurface: compatible`: every golden line this touches is either a pure addition (the new fields and `actingUserId`) or api-extractor renumbering an import alias — `Subject_2` became `Subject` across `runtime` and the `voltro` re-export because a new import changed the ordering. No declaration changed shape and no call site is affected.*
+- **@voltro/cli** — **`voltro doctor`'s hand-roll detector gains six rules and ranks its findings by file count.**
+
+  An adopting team audited four apps by hand against the framework's "reach for instead" table and produced twenty items, each with a count (`1631` files with a manual `rows[0]`, `500` queries with `input.offset`, `306` forms, `368` relations declared against `17` uses). **The detector already reported eleven of those twenty, with counts.** These are rules for what it missed — the findings that made a person do work a scan should have done:
+
+  - **`offset-pagination`** — `.offset(` / `input.offset` / `.skip(` on a list query. `hand-cursor` did not catch this: it wants `hasMore` AND `limit+1`, which is a hand-rolled *keyset* pager. Plain OFFSET is a different smell with a worse ending — O(n) in the page number, so it does not fail, it just stops loading once the table is big, on the tables (audit logs, time entries) that get big first. - **`oauth-token-table`** — an `accessToken`/`refreshToken` column on an app table → `defineConnection({ kind: 'oauth2' })`. The reporting team's version of this leaked: a `json()` column holding a Slack payload with the token inside, readable by every colleague in the tenant. - **`non-incremental-aggregate`** — a `count/sum/avg/min/max` aggregate with no `incremental:`, rescanned in full on every refresh. - **`external-state-library`** — jotai / zustand / mobx / redux → `defineStore`. A second runtime beside the reactive engine, and one that does not participate in the subscription graph. - **`hand-route-module`** — importing a hand-maintained `routes` / `urls` module instead of `.framework/routes.generated`, where a renamed page is a compile error rather than a 404 someone finds in production. - **`hand-permission-check`** — a component reading a permission bag by hand instead of `useCan` / `useResourceCan`.
+
+  **And the findings are now ordered by file count rather than by the order the rules happen to be declared in.** A reader facing twenty findings acts on the top of the list, so an arbitrary order silently decides what gets fixed. The reporting team ranked their own work by exactly that number and had to count it by hand, because this printed the same facts unranked. Effort we cannot know; magnitude we can.
+
+  *Two of these rules were caught being wrong by their own tests before shipping: `offset-pagination`'s first version checked only `input.offset` and did not match its own fixture (a narrow rule reports nothing and reads like a clean codebase), and the ranking test passed with the sort deleted until the fixture was rebuilt so declaration order and count order disagree.*
+- **@voltro/plugin-versioning** — **`historyByTrace` / `historyBySubject` — the entry points the new indices existed for.** Plus their Effect twins.
+
+  The correlation bridge added `byTrace` and `bySubject` to `_voltro_row_history` in the same release, and shipped them with no caller: `rowHistory` requires a `rowId` you already have, which is the wrong way round during an incident, when what you have is a trace or an actor. The docs demonstrated a raw `ctx.store.select('_voltro_row_history')`, which is the shape an index is supposed to save you from writing.
+
+  Both are tenant-scoped exactly like `rowHistory` (own-tenant rows plus null-tenant rows from untenanted source tables; `undefined` skips the filter, for system paths only). `historyBySubject` takes a `limit`, default 100, because an actor's history is unbounded and an entry point that returns all of it is one you call once in production and never again.
+
+  **And a bug this surfaced.** The stored-row → `VersionRow` decoder was written inline inside `versionsOf`, before the bridge existed, and silently dropped `traceId` / `subjectId` — so `rowHistory()` returned rows missing the very field the feature exists to carry. Extracted to one `rowToVersion` now shared by all three readers, which is why the drift was possible in the first place.
+- **@voltro/cli, @voltro/database** — **`setRowFilter` is now named in the always-loaded agent core**, with the distinction that makes it findable.
+
+  A team migrating eleven hand-written row filters reported missing it **twice** — once in a full framework audit, and once while telling a colleague in writing that the framework has no row-filter primitive. The depth doc covers it well; a depth doc is opened by someone who already suspects the topic. The core's SERVER rubric asked *"a permission check?"* and answered with two BOOLEAN questions, so a reader holding a row-VISIBILITY question took the nearest-fitting answer and wrote the filter by hand. Eleven times.
+
+  The rubric now asks it directly, and the pairing is the point:
+
+  > `guards:` → *may I call this procedure?* → a typed `ScopeError`. > `setRowFilter` → *which rows may I see?* → the rows are simply absent.
+
+  Plus the failure it deletes — a `WHERE ownerId = me` in the list handler covers the query and **not** the subscription — and the trap they lost time to: `load` must read through an UNFILTERED store, because applying the filter to its own loader recurses until the stack blows.
+
+  Also:
+
+  - **`voltro doctor` gains `unexercised-row-filter`.** Their 7466 tests stayed green when the filter landed because not one passed `rowFilter:` to `makeTestContext`; they only noticed because they sabotage their own predicates before believing a pass, and four sabotages ran green. `makeTestContext({ rowFilter })` being opt-in is right — a process-global in a parallel suite would be worse — but the consequence is a visibility rule nothing verifies. Same class as a suite reporting `passed` with no database. - **`MATCHES_NO_ROWS` is exported from `@voltro/database`.** They wrote `eq('id', '')` for "this caller sees nothing", because the natural spelling `inSet(col, [])` is the one that is dangerous in most query builders: an empty `IN ()` gets dropped, and a dropped predicate does not narrow, it WIDENS to the whole tenant. Here it is safe — the SQL compiler emits `FALSE`, the memory evaluator returns false — but that held by accident, with no test enforcing it. Now pinned in both evaluators, including the mirror case: an empty `notIn` matches EVERYTHING.
+
+  *The doctor rule was itself wrong when first written: it read the normal source set, which excludes test files, so it could never observe the passing case and would have fired unconditionally on every app. Caught by writing the negative test.*
+- **@voltro/database, @voltro/plugin-versioning, @voltro/sql-postgres, @voltro/sql-mysql, @voltro/sql-mssql, @voltro/sql-sqlite** — **`versioningPlugin({ timing: 'in-transaction' })` — the history row commits or rolls back WITH the change it records.** Default stays `'post-commit'`.
+
+  Post-commit recording is lossy by construction: between COMMIT and the forked history write there is a window, and a process that dies inside it leaves the change permanent and the trail silent. The size of the window is not the point — the direction is. A missing entry cannot be told apart from "nothing happened", so a trail that can lose entries proves nothing. Retry does not close it either; the process that would retry is the one that died.
+
+  This was the sole reason an adopting team could not replace their hand-rolled audit writer — called from **315 of 358 mutation handlers**, with the 43 misses being what happens to any rule that depends on someone remembering.
+
+  **What a recorder receives is an `append`, not a store.** Bound to the caller's transaction, insert-only. It cannot open a nested transaction (which throws by design), cannot read-modify-write its way into a deadlock, and "append-only" stops being a docs claim and becomes the shape of the only thing it is handed.
+
+  **The rejection is the guarantee, not a defect.** When the history insert fails, a transaction offers exactly two outcomes: the mutation fails with it, or the error is swallowed and the change commits without its entry — which is post-commit's hole with the cost already paid. There is no third option, so recorders do not swallow.
+
+  **Why the default did not move.** In-transaction makes the history table a hard dependency of every covered write path: its availability becomes your write path's availability, and every covered write holds locks longer. Post-commit loses at worst one entry; in-transaction can at worst stop writes to the covered tables. Right trade for a compliance trail, wrong one for the undo / time-travel use this plugin also serves.
+
+  **It refuses the in-memory store at boot** rather than silently no-op'ing. `memory` is the default dev store; an option that appears to work where it is cheapest to try and stops where it matters is worse than one that says so. Two limits hold in both timings and are documented: `store.raw()` produces no change event and is absent from the trail, and a write made outside a transaction is recorded immediately after rather than atomically.
+
+  *Two design claims in the plan for this were wrong and are corrected there. "Twelve write sites across three layers, no funnel" was true of `storeMiddleware` and irrelevant — each dialect store funnels every write through one private `routeEvent`, which is where this hooks. "Bulk writes have no per-row post-image" was simply false: `updateMany` already issues `RETURNING *` and emits one event per affected row, so bulk needed no special case at all. Both were found by looking in the middleware instead of the store — twice.*
+
+### Fixed
+
+- **@voltro/cli** — **A `voltro serve` that refuses to boot says why, instead of blaming the serve bundle.** A deliberate refusal — a missing `VOLTRO_SESSION_SECRET`, a `.serverOnly()` leak — came out of the launcher as:
+
+  ```
+  [voltro] serve bundle failed to load: VOLTRO_SESSION_SECRET is not set — refusing to serve. …
+  [voltro] FATAL: production `voltro serve` requires a precompiled serve bundle at …
+  but it is missing or failed to load. Run `voltro build` before serving …
+  ```
+
+  The real reason is on the first line, under a wrong headline, followed by a louder and more confident wrong instruction. An operator whose first deploy forgot the session secret is told to rebuild an artefact that is fine — and rebuilding it produces the identical output, so the loop has no exit.
+
+  The cause is a catch that has to exist: `bin/voltro.mjs` imports the precompiled serve bundle inside a `try`, because an unusable bundle must degrade to the tsx path rather than kill the boot. It could not tell "this artefact is broken" from "this app decided not to start". Refusals now carry a marker (`bootRefusal.ts`) and the launcher prints them and exits 1.
+
+  The marker is the error's `name`, a plain string, rather than a class: the serve bundle INLINES the framework, so the thrown Error crosses an instance boundary where `instanceof` does not survive — the same reason the core-table registry is keyed by `Symbol.for`.
+
+  Found by running the new `.serverOnly()` gate against a real fixture rather than by reading it, which is also how the misleading pair came into view: the session-secret case had been shipping that way for a while.
+- **@voltro/sql-postgres, @voltro/sql-mysql** — **`insertIgnore` explains a second-unique conflict instead of reporting an internal invariant.** The message was `row conflicted but lookup found nothing`, which tells a caller nothing they can act on.
+
+  It is reachable by ordinary means, and on mariadb it is the *common* path: `INSERT IGNORE` swallows ANY unique violation, so a row with a fresh `id` and a duplicate `slug` is skipped, and the lookup by the named `conflictColumns` then finds nothing. A team with `tenants (id PK, slug UNIQUE)` hits it on the first duplicate slug.
+
+  The message now names the columns that were checked, the table, and the actual cause — a different unique constraint fired, and `insertIgnore` models one conflict target. This is step 1 of `plans/framework-insertignore-any-unique.md` and is deliberately independent of the feature: whether or not `conflictColumns: 'any'` ever ships, this error should have been readable.
+- **@voltro/cli** — **`ui/unlinked` and `ui/orphaned` resolve through barrel re-exports.** A `*.component.ui.tsx` reached only via `export { X } from './x'` was reported as unrendered, however many pages actually rendered it.
+
+  On the app that reported it, `PageContent` is imported by 42 pages — every one of them through `@/components/shared` — and doctor said `imported only by: index.ts, its own test`. It was the last false positive standing after the alias fix took that app from 16 findings to 2.
+
+  The rules ask "does anything RENDER this". A barrel is a real importer and renders nothing, so stopping at the first importer answers a different question than the one asked — but only in an app that has an `index.ts`, which is why it survived.
+
+  The walk is narrowed by NAME rather than opened up wholesale: a downstream file counts only if it imports one of the names the barrel republishes from that file (aliases followed, `export *` expanded to the file's own exports, type-only re-exports ignored — they publish nothing at runtime). Without that narrowing, `export *` on a 40-entry barrel would credit its entire readership to every entry and `ui/orphaned` would quietly stop finding anything — trading a visible false positive for a silent false negative. There is a test for exactly that direction: a barrel with a used entry and a dead one must still report the dead one.
+
+### Internal (no consumer-facing effect)
+
+- **@voltro/protocol** — **`PluginHttpRouteRequest.store` names all four absences instead of one.** The docstring said "It is not tenant-scoped" and left soft-delete filtering, audit stamping and row-level security to be inferred from "everything that does not need a Subject".
+
+  A team planning to port 19 raw-SQL sites onto that seam inferred the opposite: they wrote down "a store read adds `deletedAt IS NULL`" as the trap with teeth on their list — a soft-deleted user logging back in would go from "found and revived" to "not found → insert → unique violation on email" — and deferred the whole port partly over it. The store does no such thing; it is the raw store plus the storage codec. Naming exactly one of four absences reads as an exhaustive list.
+
+  The docstring now carries the same table the `AuthStrategyInput.store` docs do, with the soft-delete row called out for anyone porting: a read here returns tombstones the way their SQL did, so a lookup that must see one needs no opt-out. (`.withDeleted()` is the opt-out on `ctx.store`, which *does* apply the filter.) Doc-only; the behaviour is unchanged and was already correct.
+- **`ci.yml` gains a `workflow_dispatch` trigger.** The full matrix — 11 database services plus the SQL Server AG init containers — is not reachable from a push to `main`: `paths-ignore` plus the job-level `if: github.event_name != 'push'` mean a main push runs static checks only.
+
+  So the only ways to exercise it were a pull request and the release gate, which meant a change to the workflow itself could sit unrun until it fired for the first time INSIDE a release — where a failure costs a ~40-minute round-trip and blocks the publish. That is precisely the position this repo was in.
+
+---
+
 ## [0.18.0] — 2026-07-28
 
 ### ⚠ BREAKING
@@ -116,6 +269,8 @@ _Changes staged for the next release accumulate here (rolled up from
   The consequence was a silent wrong answer rather than an error. An auth strategy reading an `.encrypted()` column got the literal string `enc:v1:…` back — which compares, concatenates, renders and logs perfectly well, and simply never matches the token it is compared to. The failure surfaces as "wrong credential". On the write side it was worse and unrecoverable: an insert through that store wrote PLAINTEXT into a column the schema declares encrypted.
 
   The line is now **everything that does not need a Subject**, not "less than `ctx.store`". Wired in `voltro dev` and `voltro serve` in the same change (`wrapStoreWithBootCodec`); `raw()` is deliberately left as a pass-through, since it is the documented escape hatch for hand-written SQL and re-encoding rows a caller asked for verbatim would be its own surprise.
+
+  **Upgrading: a reader that compensated for the missing codec by hand now gets the decoded value.** Two shapes, and they land differently. A hand-rolled `decryptField` on the way out is HARMLESS — it passes a non-`enc:v1:` value through unchanged, so the call simply stops doing anything. A hand-rolled `JSON.parse(row.someJsonColumn)` on a column the codec now deserialises is NOT: it is handed an object and throws. If you read an `.encrypted()` or array column through `AuthStrategyInput.store` / `PluginHttpRouteRequest.store` and unpacked it yourself, grep those call sites — the ones typed as a `string | ReadonlyArray<string>` union already survive, a bare `JSON.parse` does not. (Reported by an adopter whose two such reads happened to be written defensively, which is why their upgrade was silent.)
 - **@voltro/cli** — **The remaining file-moving codemods narrow their projects too**, and the shared project is no longer a stale snapshot.
 
   `0.14.0/03_pages-suffix` is gated on `/src/pages/` throughout — plus each app's `app.config.ts`, which is how it tells a real web app's pages from a library that merely keeps components under `src/pages/`. `0.15.0/01_undo-mistaken-taxonomy-renames` only ever renames `*.component[.ui].ts(x)` and an export-less `*.types.ts(x)`. Both now declare that scope and take the importer closure instead of the workspace.
@@ -169,15 +324,15 @@ _Changes staged for the next release accumulate here (rolled up from
   The catalogue said a `*.component.tsx` promises "exactly one component **(+ types)**", which reads as "types are the only exception". It never was — the rule counts components, and a `const COLUMNS = […]` beside the table that renders it was always allowed. The docs now say so in both languages.
 - **@voltro/cli** — **The seeded `AGENTS.md` / `CLAUDE.md` never mentioned `.serverOnly()` — the one marker that decides leak vs no leak.**
 
-  Reported from a strict-mode pass over five apps: the template documents `.encrypted()`, `audit()`, `tenant()`, `softDelete()`, `.check()`, `validate()` — and not the marker whose absence the **boot audit hard-fails on**. Depth existed (`database/sensitivity`, `data/crud`, both languages); the always-loaded core simply never pointed at it, so an agent or a human following the template never learned the marker exists.
+  Reported from a strict-mode pass over five apps: the template documents `.encrypted()`, `audit()`, `tenant()`, `softDelete()`, `.check()`, `validate()` — and not the marker that decides leak vs no leak. *(Corrected after publication: this entry said the boot audit "hard-fails" on it. It did not — it was a single `log.warn`, in every command. The next release makes the claim true; see that entry.)* Depth existed (`database/sensitivity`, `data/crud`, both languages); the always-loaded core simply never pointed at it, so an agent or a human following the template never learned the marker exists.
 
   The core now carries a short section on the three markers as ORTHOGONAL questions, because the substitution is the real hazard:
 
-  | Marker | Answers | Enforced by | |---|---|---| | `.serverOnly()` | may this leave the server at all? | the boot audit — it FAILS the boot | | `.sensitive()` / `.safe()` | may it appear in an export? | the masking profile, fail-closed | | `.encrypted()` | is it encrypted at rest? | the store's codec |
+  | Marker | Answers | Enforced by | |---|---|---| | `.serverOnly()` | may this leave the server at all? | an audit — see the correction above | | `.sensitive()` / `.safe()` | may it appear in an export? | the masking profile, fail-closed | | `.encrypted()` | is it encrypted at rest? | the store's codec |
 
   Reading `.encrypted()` as "safe to expose" is a category error and a plausible one — the runtime decrypts for the handler, so an encrypted column reaches a client like any other unless it is ALSO `.serverOnly()`.
 
-  **The doc-drift guard is why this survived, so it grew the missing half.** It asserted the core carries the EXPORT axis (`.sensitive(`, fail-closed, the export endpoint) and said nothing about the wire axis. There is now a matching assertion for `.serverOnly()`, the boot audit, and the orthogonality — verified by deleting the section and watching it go red.
+  **The doc-drift guard is why this survived, so it grew the missing half.** It asserted the core carries the EXPORT axis (`.sensitive(`, fail-closed, the export endpoint) and said nothing about the wire axis. There is now a matching assertion for `.serverOnly()`, the enforcement, and the orthogonality — verified by deleting the section and watching it go red. *(It asked only for the words "boot audit", which the template supplied while promising a failure nothing performed. The next release makes it name the commands that gate.)*
 - **@voltro/cli** — **`voltro dev` could not start at all in a strict-pnpm install: the supervisor respawned with a bare `--import tsx`.**
 
   Node resolves a bare specifier against the CHILD's working directory. `@voltro/cli` declares tsx; the user's app does not — so under a non-hoisting layout tsx lives inside `node_modules/.pnpm/@voltro+cli@…/node_modules/tsx` and is invisible from the app root. The first respawn died with `Cannot find package 'tsx'`, naming a package the reader never asked for and cannot usefully install.

package/dist/index.d.ts

@@ -15,6 +15,31 @@ export declare const audit: () => MixinDefinition<{
 
 export declare const AUDIT_LOG_TABLE = "_voltro_audit_log";
 
+/**
+ * One actor's audit rows, newest first — the `byAuditSubjectStatus` index's
+ * caller. `status: 'error'` is the question asked under pressure ("every
+ * refusal by X"), which is why it is a first-class argument rather than
+ * something you filter in JS after fetching everything.
+ *
+ * `limit` defaults to 100: an actor's history is unbounded, and an entry point
+ * that returns all of it by default is one you call once in production.
+ */
+export declare const auditBySubject: (store: AuditQueryStore, subjectId: string, options?: {
+    readonly status?: "ok" | "error";
+    readonly limit?: number;
+}) => Promise<ReadonlyArray<Record<string, unknown>>>;
+
+/**
+ * Every audit row for ONE call — the other half of the correlation join.
+ *
+ * `byAuditTrace` existed with no caller, which is the same defect the versioning
+ * side had: an index nobody can enter is a query you still have to hand-write.
+ * Pair it with `historyByTrace` from `@voltro/plugin-versioning` on the same
+ * `traceId` and you have "who called, whether they were refused, and what they
+ * changed" in two reads.
+ */
+export declare const auditByTrace: (store: AuditQueryStore, traceId: string) => Promise<ReadonlyArray<Record<string, unknown>>>;
+
 /** Narrow slice of the framework DataStore the durable sink needs. */
 export declare interface AuditDataStore {
     insert: (table: string, row: Record<string, unknown>) => Promise<unknown>;
@@ -111,6 +136,57 @@ export declare interface AuditPluginOptions {
      * Typically `/^debug\./` or similar. Default: skip nothing.
      */
     readonly exclude?: RegExp;
+    /**
+     * Which OUTCOMES are recorded. `include`/`exclude` filter by tag, before the
+     * call runs; this filters by what happened, after.
+     *
+     *   - `'all'` (default) — every invocation.
+     *   - `'errors'` — refusals only. The forensic core: a denied guard, a revoked
+     *     key, a validation rejection. Pairs with `plugin-versioning`, which
+     *     records the SUCCESSFUL writes, so the two together still cover
+     *     everything while this table stays small enough for retention to be a
+     *     footnote.
+     *   - a predicate — anything else (`(e) => e.outcome.kind === 'error' ||
+     *     e.tag.startsWith('auth.')`).
+     *
+     * **`'all'` is the default deliberately, though `'errors'` is often the right
+     * choice.** Defaulting to errors would silently stop recording successes for
+     * every app that upgrades, and "what did this compromised account touch" is
+     * answered by successes. Shrinking the trail is a decision an app makes with
+     * its eyes open, not one an upgrade makes for it.
+     */
+    readonly record?: 'all' | 'errors' | ((event: AuditEvent) => boolean);
+    /**
+     * What happens to `AuditEvent.input` before it is handed to the sink.
+     *
+     *   - `'all'` (DEFAULT) — the payload is replaced by `{ __redacted: 'all' }`.
+     *     The row still proves a payload existed; it just does not carry it.
+     *   - `'none'` — the raw input, verbatim. What every sink did before this
+     *     option existed.
+     *   - a function — `(event) => unknown`, for field-level control.
+     *
+     * **Why the default is `'all'`, and why it is NOT marker-driven.** The obvious
+     * design — redact using `.serverOnly()` / `.sensitive()` — cannot work: those
+     * markers live on TABLE COLUMNS, and this is a mutation's INPUT. A
+     * `changePassword({ oldPassword, newPassword })` has no column to consult, so
+     * a marker-driven default would cover exactly 0% of the case it exists for
+     * while reading, to anyone configuring it, like protection. (`.sensitive()` is
+     * also the EXPORT axis, not "unsafe to log" — treating one as the other is the
+     * category error the three-marker table warns about.)
+     *
+     * So the choice is between recording credentials by default and recording no
+     * payload by default. A password change, an API key at issuance and a PAT all
+     * land in `input`, and an audit table is the one place nobody thinks to look
+     * for a credential. Losing payload detail is visible the first time you read a
+     * row; leaking a credential is not visible at all. Opt in per app with a
+     * function once you know your own inputs.
+     */
+    readonly redactInput?: 'all' | 'none' | ((event: AuditEvent) => unknown);
+}
+
+/** The narrow read surface the entry points need. */
+export declare interface AuditQueryStore {
+    query: (descriptor: unknown) => Promise<ReadonlyArray<Record<string, unknown>>>;
 }
 
 export declare type AuditSink = (event: AuditEvent) => void | Promise<void> | Effect.Effect<void>;

package/dist/index.js

@@ -2,21 +2,30 @@ import { audit as e } from "./mixin.js";
 import { Effect as t } from "effect";
 import { definePlugin as n } from "@voltro/protocol";
 import { createLogger as r } from "@voltro/logger";
-import { id as i, json as a, table as o, text as s, timestamp as c } from "@voltro/database";
+import { and as i, eq as a, id as o, json as s, registerRetention as c, retentionTtlMsFromEnv as l, table as u, text as d, timestamp as f } from "@voltro/database";
 //#region src/store.ts
-var l = "_voltro_audit_log", u = o(l, {
-	id: i({ prefix: "audit" }),
-	tag: s(),
-	at: c(),
-	subjectId: s().nullable(),
-	tenantId: s().nullable(),
-	traceId: s(),
-	status: s(),
-	durationMs: s(),
-	subject: a(),
-	input: a(),
-	outcome: a()
-}).index("byAuditTag", ["tag"]).index("byAuditTrace", ["traceId"]), d = [u], f = (e) => ({
+var p = "_voltro_audit_log", m = u(p, {
+	id: o({ prefix: "audit" }),
+	tag: d(),
+	at: f(),
+	subjectId: d().nullable(),
+	tenantId: d().nullable(),
+	traceId: d(),
+	status: d(),
+	durationMs: d(),
+	subject: s(),
+	input: s(),
+	outcome: s(),
+	errorTag: d().nullable()
+}).index("byAuditTag", ["tag"]).index("byAuditTrace", ["traceId"]).index("byAuditSubjectStatus", [
+	"subjectId",
+	"status",
+	"at"
+]).index("byAuditTenantStatus", [
+	"tenantId",
+	"status",
+	"at"
+]), h = [m], g = (e) => ({
 	tag: e.tag,
 	at: new Date(e.ts),
 	subjectId: e.subject.id ?? null,
@@ -26,14 +35,37 @@ var l = "_voltro_audit_log", u = o(l, {
 	durationMs: String(e.outcome.durationMs),
 	subject: e.subject,
 	input: e.input,
-	outcome: e.outcome
-}), p = (e) => async (t) => {
-	await e.insert(l, f(t));
-}, m = r({ scope: "@voltro/plugin-audit" }), h = 1e3, g = [], _ = () => [...g], v = () => {
-	g.length = 0;
-}, y = (e) => {
-	g.push(e), g.length > h && g.splice(0, g.length - h);
-}, b = (e) => {
+	outcome: e.outcome,
+	errorTag: e.outcome.kind === "error" ? _(e.outcome.error) : null
+}), _ = (e) => {
+	if (typeof e != "object" || !e) return null;
+	let t = e._tag;
+	return typeof t == "string" && t !== "" ? t : null;
+}, v = async (e, t) => e.query({
+	table: p,
+	predicate: a("traceId", t),
+	order: [{
+		column: "at",
+		direction: "asc"
+	}]
+}), y = async (e, t, n = {}) => {
+	let r = a("subjectId", t);
+	return e.query({
+		table: p,
+		predicate: n.status === void 0 ? r : i(r, a("status", n.status)),
+		order: [{
+			column: "at",
+			direction: "desc"
+		}],
+		take: n.limit ?? 100
+	});
+}, b = (e) => async (t) => {
+	await e.insert(p, g(t));
+}, x = r({ scope: "@voltro/plugin-audit" }), S = 1e3, C = [], w = () => [...C], T = () => {
+	C.length = 0;
+}, E = (e) => {
+	C.push(e), C.length > S && C.splice(0, C.length - S);
+}, D = (e) => {
 	let t = e.subject.id ? `${e.subject.type}:${e.subject.id}` : e.subject.type, n = e.outcome.kind === "ok" ? "ok" : "error", r = {
 		subject: t,
 		tenant: e.subject.tenantId ?? null,
@@ -41,8 +73,8 @@ var l = "_voltro_audit_log", u = o(l, {
 		durationMs: e.outcome.durationMs,
 		traceId: e.traceId
 	};
-	n === "error" ? m.warn(e.tag, r) : m.info(e.tag, r);
-}, x = (e, n) => typeof e == "function" ? (n) => t.suspend(() => {
+	n === "error" ? x.warn(e.tag, r) : x.info(e.tag, r);
+}, O = (e, n) => typeof e == "function" ? (n) => t.suspend(() => {
 	let r = e(n);
 	return r === void 0 ? t.void : t.isEffect(r) ? r : t.tryPromise({
 		try: () => Promise.resolve(r),
@@ -54,7 +86,7 @@ var l = "_voltro_audit_log", u = o(l, {
 		try: () => r(e),
 		catch: (e) => e
 	}) : t.void;
-}) : e === "memory" ? (e) => t.sync(() => y(e)) : (e) => t.sync(() => b(e)), S = (e) => {
+}) : e === "memory" ? (e) => t.sync(() => E(e)) : (e) => t.sync(() => D(e)), k = (e) => {
 	let t = [], n = [], r = (e) => {
 		if (e._tag !== "Empty") {
 			if (e._tag === "Fail") {
@@ -78,10 +110,25 @@ var l = "_voltro_audit_log", u = o(l, {
 		}
 	};
 	return r(e), t[0] ?? n[0] ?? e;
-}, C = (e = {}) => {
-	let r = e.sink === "datastore", i, a = x(e.sink, () => i), o = (t) => !(e.exclude?.test(t) || e.include && !e.include.test(t)), s = (e, n) => o(n.tag) ? t.suspend(() => {
+}, A = (e = {}) => {
+	let r = e.sink === "datastore";
+	r && c({
+		table: p,
+		timeColumn: "at",
+		ttlMs: l(process.env.VOLTRO_AUDIT_LOG_TTL_HOURS, 24 * 365)
+	});
+	let i, a = O(e.sink, () => i), o = (t) => !(e.exclude?.test(t) || e.include && !e.include.test(t)), s = (t) => {
+		let n = e.record ?? "all";
+		return n === "all" ? !0 : n === "errors" ? t.outcome.kind === "error" : n(t);
+	}, u = { __redacted: "all" }, d = (t) => {
+		let n = e.redactInput ?? "all";
+		return n === "none" ? t : {
+			...t,
+			input: n === "all" ? u : n(t)
+		};
+	}, f = (e) => s(e) ? a(d(e)) : t.void, m = (e, n) => o(n.tag) ? t.suspend(() => {
 		let r = Date.now();
-		return e.pipe(t.tap((e) => a({
+		return e.pipe(t.tap((e) => f({
 			ts: r,
 			tag: n.tag,
 			subject: n.subject,
@@ -92,7 +139,7 @@ var l = "_voltro_audit_log", u = o(l, {
 				value: e,
 				durationMs: Date.now() - r
 			}
-		}).pipe(t.catchAllCause(() => t.void))), t.tapErrorCause((e) => a({
+		}).pipe(t.catchAllCause(() => t.void))), t.tapErrorCause((e) => f({
 			ts: r,
 			tag: n.tag,
 			subject: n.subject,
@@ -100,7 +147,7 @@ var l = "_voltro_audit_log", u = o(l, {
 			input: n.input,
 			outcome: {
 				kind: "error",
-				error: S(e),
+				error: k(e),
 				durationMs: Date.now() - r
 			}
 		}).pipe(t.catchAllCause(() => t.void))));
@@ -110,13 +157,13 @@ var l = "_voltro_audit_log", u = o(l, {
 		description: "Records every mutation invocation; ships an audit() schema mixin for row-level metadata.",
 		permissions: r ? ["rpc:intercept:mutation", "store:write"] : ["rpc:intercept:mutation"],
 		...r ? {
-			extendSchema: { tables: d },
+			extendSchema: { tables: h },
 			bindDataStore: (e) => {
-				i = p(e);
+				i = b(e);
 			}
 		} : {},
-		interceptMutation: s
+		interceptMutation: m
 	});
 };
 //#endregion
-export { l as AUDIT_LOG_TABLE, e as audit, f as auditEventToRow, u as auditLogTable, d as auditLogTables, C as auditPlugin, v as clearAuditBuffer, p as dataStoreAuditSink, _ as readAuditBuffer };
+export { p as AUDIT_LOG_TABLE, e as audit, y as auditBySubject, v as auditByTrace, g as auditEventToRow, m as auditLogTable, h as auditLogTables, A as auditPlugin, T as clearAuditBuffer, b as dataStoreAuditSink, w as readAuditBuffer };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@voltro/plugin-audit",
-  "version": "0.18.0",
+  "version": "0.19.0",
   "description": "Audit plugin — ships the `audit()` schema mixin (createdAt/updatedAt/createdBy/updatedBy → Actor) plus an optional mutation interceptor that records every call to a configurable sink (console / memory / custom function).",
   "keywords": [
     "voltro",
@@ -37,9 +37,9 @@
     "node": ">=24.0.0"
   },
   "dependencies": {
-    "@voltro/database": "0.18.0",
-    "@voltro/logger": "0.18.0",
-    "@voltro/protocol": "0.18.0"
+    "@voltro/database": "0.19.0",
+    "@voltro/logger": "0.19.0",
+    "@voltro/protocol": "0.19.0"
   },
   "peerDependencies": {
     "effect": "^3.21.4"