@voltro/plugin-comments 0.54.0 → 0.55.0

package/CHANGELOG.md

@@ -39,6 +39,144 @@ _Changes staged for the next release accumulate here (rolled up from
 
 ---
 
+## [0.55.0] — 2026-08-27
+
+### ⚠ BREAKING
+
+- **@voltro/protocol, @voltro/client, @voltro/runtime, @voltro/cli** — **A multi-reference field now flips as instantly as a scalar one.** A mutation target's declared `relations:` reconciled the junction inside the server's transaction and nothing else: the client learned about the link change only when the delta came back. On the same submit, the renamed title flipped immediately and the assigned stores did not — the half of the promise that was never stated.
+
+  The declaration now drives BOTH. `useMutation`'s auto-optimistic stages a patch on every subscription sourced on the junction table, reconciling that anchor's links against `input[field]` — surplus links removed, new ones staged, surviving links left untouched with their real ids (a diff, mirroring `store.relationLinks(...).set`, not a drop-and-restage that would blink every unchanged row).
+
+  The patches ride the ordinary optimistic lane, staged under the mutation id, so the rollback rule holds by construction: reverted on failure, kept on success until the base actually moves. Nothing here is on a timer.
+
+  **Migration — `relations:` values are objects now:**
+
+  ```ts
+  // before
+  target: { table: 'employees', op: 'update',
+    relations: { assignedStores: 'employee_assigned_stores' } }
+  
+  // after
+  target: { table: 'employees', op: 'update',
+    relations: { assignedStores: {
+      junction: 'employee_assigned_stores',
+      anchorColumn: 'employeeId',   // the junction reference() pointing at `employees`
+      targetColumn: 'storeId',      // the junction's other reference()
+    } } }
+  ```
+
+  The columns are declaration data because the optimistic patch runs in the BROWSER, which has no table registry to derive them from — `@voltro/database` is server-only by construction, and guessing a column from a table name is exactly what `store.relationLinks` refuses to do. They are not taken on trust: before it writes, the server compares the declaration against the junction's real reference columns and refuses, naming the correct pair, if they disagree. A wrong declaration is a loud error carrying its own fix, never a client that patches one column while the server writes another.
+
+  Semantics unchanged and now shared by both sides: an absent input field touches nothing (absent ≠ empty), an empty array is the explicit clear. The client uses `input.id` for an update and, for an insert, the same optimistic id it stamped on the new row — the server's `output.id` is not knowable before the response.
+
+  **`voltro update` carries you across this** — codemod `0.55.0/01_target-relations-declare-columns`. If you pin versions by hand and never run it, print the notes without changing anything: `voltro update --codemods-only --from <your current version> --dry-run` (this one ships in 0.55.0).
+- **@voltro/client, @voltro/ui, @voltro/ui-shadcn, @voltro/web, @voltro/cli** — **A rich-text field, and the sanitizing contract is the point of it.** `RichTextDocument` (`@voltro/client`) is the value; `widget: 'rich-text'` renders it; `<RichTextView>` (`@voltro/ui`, re-exported by `@voltro/web`) displays it.
+
+  The contract, stated plainly because the alternatives all look reasonable until you name who the attacker is:
+
+  - **The value is a closed document tree, not an HTML string.** There is no `html` node, no raw-markup escape hatch, no attribute bag. Anything that is not one of the declared node types fails to decode. - **The boundary is the `Schema` decode**, which is the server's existing, non-bypassable input boundary — the same one every mutation input already passes through. So the guarantee is not "somebody remembered to sanitize this"; it is that a document which reached the database is one of these shapes. - **A link's `href` is the one field that points outward, and it is allowlisted** — `http(s)`, `mailto:`, a `#fragment`, a `/path`; nothing else. Control characters and whitespace are stripped before the check, because `java\tscript:` navigates exactly like `javascript:` and a check on the raw string passes it. - **Client-side is not a boundary and is not treated as one.** The widget's parser runs in the browser for the editing experience; every property it maintains is re-established by the decode on the server. - **Rendering never uses `dangerouslySetInnerHTML`.** Nodes become React elements, text becomes React children — so markup typed into the box is markup the reader SEES. `<RichTextView>` also drops an href that would not survive a decode, for the value that never went through one.
+
+  Rejected, for the record: sanitizing an HTML string on write (ships an HTML parser and the mXSS surface that comes with it), escaping at render (makes safety a property of every read site), and declaring an unenforced boundary (a convention, not a guarantee).
+
+  The built-in widget is a `<textarea>` over a small, CLOSED markdown subset — headings, `**bold**`, `*italic*`, `` `code` ``, `[text](href)`, lists, blockquote, fenced code — with everything unrecognised left as literal text. That also closes the no-JS loop: the textarea posts source, `/form/*` parses it, and the same decode validates it. A WYSIWYG belongs at rung 2 (register a `rich-text` widget); the stored value shape does not change.
+
+  **Not** the collaborative case: `crdtDoc()` + `useCrdtEditor` remains the multi-writer path (a CRDT bytes column, a sync lane, Tiptap). This is one column, one writer, ordinary JSON the server can validate and diff, and no new dependency in the default kit.
+
+  **Migration:** `WidgetKind` gained `'rich-text'`. Only a registry typed as a TOTAL map (`Record<WidgetKind, Widget>`) notices — add one entry pointing at the exported `RichTextWidget`. Partial registries need nothing.
+
+  Also fixed alongside: the capability manifest's `MANIFEST_WIDGET_KINDS` is a hand copy of `WidgetKind` (a CLI module cannot import `@voltro/client`) and had silently drifted by two kinds since 0.53.0, telling coding agents a smaller set than the renderer accepts. It is complete again and pinned by a test that reads the union out of the client's source.
+
+  **`voltro update` carries you across this** — codemod `0.55.0/02_widget-kind-gained-rich-text`. If you pin versions by hand and never run it, print the notes without changing anything: `voltro update --codemods-only --from <your current version> --dry-run` (this one ships in 0.55.0).
+
+### Added
+
+- **@voltro/database, @voltro/data-transfer, @voltro/cli, @voltro/voltro** — A staged `--mode replace` now RECORDS the scratch tables it creates, and a boot collects the ones nobody is coming back for.
+
+  Staging tables are `_voltro_staging_`-prefixed, so the differ correctly ignores them — and nothing else mentioned them either. A run that died between the load and the swap left a full copy of a bundle that only a hand-written introspection could find, on a database whose boot said nothing. The marker row now names the set, carries a heartbeat the run refreshes while rows land, and records whether the run was started `--no-atomic`. That is what lets a boot tell the three cases apart: silent past the threshold and not resumable → the tables are dropped; still beating → an import is loading into them, here or on another replica; resumable → its staging IS the resume point and is left alone however stale. `VOLTRO_STAGING_STALE_MINUTES` moves the threshold (default 30). A staging record is reported, never a refusal — a staged replace destroys nothing until one short server-side swap, so refusing a boot over it would be an alarm on a healthy database. `voltro data clear-staging` reads the same records and labels each table with what its own run says, instead of listing them flat under a warning that it could not tell a leftover from an import in flight.
+
+  Two smaller things came with it. The raw-SQL seam the staged swap runs on (`DataStore.run`) is a DECLARED optional capability now, in the shape `emptyTables` established, with a parity assertion across the four dialect stores — it was duck-typed against an interface that never mentioned it, so a store that dropped it would have fallen out of the feature detection and taken the slower path forever, on that one dialect, in silence. And a registered staging clone can no longer reach the differ's DECLARED side: `--mode replace` registers each staging table as a clone of its target for the length of the load (a typed write resolves its columns by name), and a plan computed while an import was in flight proposed `create-table _voltro_staging_notes`.
+
+  Also measured rather than assumed: the staging table `createStagingSql` builds on **sqlite** (`CREATE TABLE … AS SELECT * FROM t WHERE 0`) and on **SQL Server** (`SELECT * INTO … WHERE 1 = 0`) carries the target's columns and ZERO foreign keys, which is what the load needs. Those were the two dialects the postgres and mysql-family measurements had not covered.
+- **@voltro/client, @voltro/ui, @voltro/web** — **`useFormField(path)` finds its binding.** `<FormBindingProvider binding={form}>` (mounted for you by `<AutoForm>`) makes the narrow per-field subscription reachable without threading the binding down to every field component as a prop. That thread was blocking incremental adoption: a codebase moving a hundred-plus forms one at a time keeps its own field context and swaps engines per form, and being asked to prop-drill to ~30 field components at once meant taking the binding and declining the optimisation they had the most to gain from.
+
+  **Message ids a real catalogue needed**, each because the generic answer is worse at the point of use:
+
+  - `betweenLength {min,max}` when a field carries BOTH bounds — "at least 2" is a half-truth for a rule that is "between 2 and 50" — and `exactLength {amount}` when they are equal. Read from the schema, not the failing issue: piping nests the later refinement outermost, so the sibling bound is not reachable from the issue that failed. - `invalidEmail` / `invalidUrl` / `invalidUuid` when the refinement declares a JSON-Schema `format`. A bare regex cannot name its own rule, and "Invalid format" beside an email box tells nobody anything. - `minDate` / `maxDate`, because a date bound rendered as a number bound reads "must be at least 2026-01-01". - `invalidFileType` / `fileTooLarge` — not produced by any refinement, carried so an app's own `ctx.validation.fail('doc', 'validation.fileTooLarge')` renders a sentence rather than an id.
+
+  `apiSurface: compatible` — `useFormField` goes from a const arrow to an overloaded function so it can take `(path)` as well as `(binding, path)`. The golden line for the old signature is replaced rather than removed: every existing `useFormField(form, path)` call compiles unchanged, because that overload is still declared first-class. Only code capturing the function's exact TYPE (rather than calling it) sees a difference.
+
+  **Counting rules pass `count`.** `minItems` / `maxItems` carry `{ count }` beside `{min}`/`{max}`: i18next selects a plural form on a parameter named exactly `count`, so ids passing only `{min}` could not be pluralised at all.
+- **@voltro/runtime, @voltro/cli** — **The resume census — `/_voltro/inspect/subscriptions` now carries `resume`.** Per query label: how many subscriptions recorded a delta-resume ring, and how many were excluded, counted per reason (`computed`, `row-filter`, `eager-load`, `uncanonical-input`, `not-offered`). `voltro dev` also logs each verdict once per label under the `voltro:resume` scope — debug is the default level outside production, so it is already on where the tuning happens and off where it is served.
+
+  It exists because the two failure shapes are indistinguishable from outside. A subscription excluded by a row filter and one whose executor returns a **value** rather than a descriptor both reconnect with a fresh snapshot and rows on the screen, so an app measuring its own reconnects cannot tell which of its queries a `tables:` declaration is even capable of helping. The answer is which of the two bind paths the executor took, and nothing on the wire carries it.
+
+  The reasons are the load-bearing part, not the counts: `computed` means no declaration can ever change this query, `row-filter` means the filter narrows its source and the exclusion is the point, and `eager-load` is reported ONLY when the base table is not itself narrowed — so that verdict always means "drop the `.with(...)` and this one resumes". Counts rather than one verdict per label, because resumability is not purely a property of the label: an input that does not canonicalise is a property of the value, so one query can be resumable for one subscriber and excluded for the next. A label nobody has subscribed to is ABSENT rather than reported as zero — "nothing has subscribed yet" and "every query is excluded" must not read the same.
+
+  **And a correction to what `tables:` was documented to buy.** The 0.54.0 notes, the `tables:` doc comment and the row-level-security page all said one registration cost delta-resume on every query descriptor in an app, with a count beside it. The count was real; the sentence around it claimed those descriptors would have HAD the feature, and that was never measured. A query only has a delta chain when its executor returns a descriptor — one that maps its rows or wraps them in a page envelope re-runs an opaque handler and emits snapshots, filter or no filter. So the number a declaration gives back is the number of descriptor-returning subscriptions, not the number of queries. The docs now say that where the decision is made, and the census is how you find out which shape each of yours took.
+
+### Changed
+
+- `dataTransfer.stagingStaleMinutes` in `app.config.ts` — how long a staged import's silence has to last before a boot treats its scratch tables as abandoned. Previously `VOLTRO_STAGING_STALE_MINUTES` only; the env var still overrides the declaration, on the rule every other tunable here follows.
+
+  The reason this was not already a field was recorded as "the boot check runs off the store alone, before the app config is threaded to it". That described the function's signature, not the boot: both paths already held the config three lines above the call. Resolution lives inside `stagingLeftoversAtBoot` rather than at either call site, so the two cannot disagree about what a declared value means, and a source-reading assertion fails if either path stops handing the config over.
+
+### Fixed
+
+- **@voltro/protocol, @voltro/cli, @voltro/client, @voltro/plugin-notifications, @voltro/plugin-comments, @voltro/plugin-presence, @voltro/plugin-search, @voltro/plugin-flags** — **Installing a plugin under an `alias` now moves its client too.** `alias` exists for one problem — your app already publishes `notifications.*` and cannot install a plugin that wants the same namespace — and it has to move four surfaces or it is worse than not existing. It moved three.
+
+  The two that did not:
+
+  - **The generated client sent the tag the PLUGIN authored.** Every lifter is `Rpc.make(descriptor.name, …)`, so the wire tag comes from the descriptor, not from the tag the codegen computed. Under `alias: 'inbox'` the exported identifier became `inboxInboxRpc`, the `appDescriptors` key became `inbox.inbox`, the type key became `inbox.inbox` — and the browser still asked a server that had stopped serving it for `notifications.inbox`. The codegen now lifts every plugin descriptor through `withRpcTag(…)`, unconditionally, so the aliased and un-aliased cases are one code path rather than a branch nothing exercises. - **The plugin's own hooks spelled their namespace as a literal.** `useInbox()`, `useUpload()`, `useComments()`, `usePresence()`, `useFlag()`, `useSearch()` all carried strings like `'notifications.inbox'`, which no alias could reach. `voltro dev` now writes a `registerPluginAliases({ … })` declaration into `rpcGroup.generated.ts` — the module the web client already loads value-level — and every plugin hook resolves its tag through `pluginTag(baseName, route)` from `@voltro/protocol` at call time, not at module load. Aliasing a plugin needs no change at any call site.
+
+  Two installs of one plugin (`name: 'ops'`) with no un-suffixed primary make `pluginTag` **refuse** rather than pick: a hook has no way to name an install, and guessing would address the wrong one silently. The error names both candidates and points at the full-tag call that says which you mean.
+
+  `pluginAlias` / `pluginSlug` moved from the CLI into `@voltro/protocol` (and are re-exported from their old path) because the browser has to derive the same namespace the server registered, and a second copy on the client would be a second definition of the rule with nothing comparing them.
+
+  Also corrected, in the same seam: the list of plugins that deliberately do NOT accept `tables: false` read as exhaustive and omitted `_voltro_storage_grants`, which decides who may read an object. It is named now — along with the reason the option would not have reached it anyway (storage's tables are framework tables, not `extendSchema` contributions).
+- `voltro build` could not produce an api serve bundle on 0.53.0 or 0.54.0.
+
+  `@voltro/content`'s `get.ts` reaches its render pipeline through a dynamic `import('./serverLoad')`. That is deliberate — an unresolvable-at-build-time specifier is what keeps marked and the shiki grammars out of a consumer's client chunk graph. But `serverLoad` was not in the package's entry map, so nothing emitted `dist/serverLoad.js`, and `dist/index.js` shipped an import of a file beside it that was not there.
+
+  It resolved in this repo every time, because the workspace `exports` point at `src/` and `serverLoad.ts` sits next to `get.ts`. A consumer resolves `publishConfig.exports` to `dist/index.js`, and the same line cannot resolve. esbuild does not honour `@vite-ignore` — that is a vite directive — so the serve bundle refused to ship. The refusal was right; nothing had ever triggered it.
+
+  Two things worth knowing, both measured rather than reasoned:
+
+  - **The build was stopped by a dependency that contributes nothing to it.** An api serve bundle reaches `@voltro/content` through `serveCommand → dev → webDev → contentWiring`, and esbuild resolves before it tree-shakes. After shaking, the content pipeline is **0 bytes** of a 14.42 MB bundle. So an api app with no markdown anywhere was blocked by a markdown loader whose code it would never have carried. - **Shipping the file does not bloat anything.** Same measurement with the fixed package resolved as a consumer resolves it: 14.42 MB, content still 0 bytes. The dynamic import stays shaken away.
+
+  `scripts/check-dist-internal-specifiers.mjs` now bundles every emitted file of every publishable package — from `.publish/`, the tree users receive — with bare specifiers external, and fails if any relative specifier does not resolve. GATE-2 (`publint`) answers "does a declared subpath resolve"; this is one level below it, where `./serverLoad` lives.
+- **@voltro/client** — Three places still taught the pre-fix contract for a cold-start failure.
+
+  `SubscriptionFailed` gives it its own state — `loading: false`, `failed: true`, `error` non-optional — precisely so a component branching on `loading` alone cannot render a skeleton forever. But `SubscriptionMeta.error`'s doc comment and two docs pages still said the opposite ("leaves `loading` TRUE … check `error` to break out of it"), which is the sentence a deployment quoted back at us as evidence for the defect that had already been fixed.
+
+  A comment that predicts a trap the code no longer has is worse than no comment: it teaches the defensive shape as if it were still required, and it invites the reading that `loading` is unreliable. All four now describe the state that exists, with the old behaviour kept only as the history that explains why the field is there.
+- **@voltro/client, @voltro/web, @voltro/cli, @voltro/ui** — Four defects a real migration found, all of which passed `tsc` and a full test suite and only showed up against a running system.
+
+  **A server render derived a different form than the browser.** Nothing mounts a runtimes provider during SSR, so `useFormBinding` resolved its input schema from an EMPTY descriptor map: no fields, `required: false`. The browser then rendered the real ones and React discarded the subtree — "Hydration failed" on every server-rendered page carrying a bound form, with the diff pointing at a `Mui-required` class. `@voltro/client` keeps a process-global SSR descriptor registry now, and both boot paths fill it before rendering (`voltro dev` and `voltro start`, pinned as a parity test — a CLI module cannot import `@voltro/client`, so the call goes through `@voltro/web/ssr`, which both already load).
+
+  **A rejected `submit()` had nowhere to go.** A form calls it from an `onSubmit` handler that cannot await it, so the rejection surfaced as `Uncaught (in promise)` while the form sat there looking saved. `submit()` resolves `undefined` now and the failure is state: `state.submitError`, plus an optional `onError`. That covers a composed `onSubmit` whose follow-up write fails on its OWN mutation handle — a failure the binding never saw, and the common shape (create the row, then its first child).
+
+  **Undeclared fields went on the wire.** The binding validated the mapped input and then sent the object unchanged; the client decode ignores excess properties while the server has refused them since 0.37, so a form carrying anything beyond the mutation's input passed validation and was rejected on the wire with a message pointing at no field. The payload is restricted to the declared keys now — the rule the no-JS path already followed, so the two submits agree — with a dev warning naming what was dropped.
+
+  **`setValue` with an unchanged value produced a new `values`.** Every React state source is expected to no-op on that; this one did not, so an effect depending on `values` that re-set a field to the value it already held never settled ("Maximum update depth exceeded" on a form mirroring toggles out of a multi-select).
+- A plugin's dashboard panel no longer disappears when the app aliases the plugin.
+
+  `alias` moves `plugin.name`, and the inspect mount is derived from it, so `alias: 'inbox'` on notifications moved its panel to `/_voltro/inspect/plugins/inbox/...` while both dashboards ask for `/plugins/notifications/...` with the path compiled in. They live in other repositories and cannot follow. The field's own doc comment stated this as a cost you accept — in nine plugins, the protocol helper and the docs.
+
+  `makePluginInspectRegistry` now mounts each plugin's `inspectEndpoints` under its CANONICAL slug as well, in a second pass so an effective mount always wins the path. Added only where unambiguous: a base name carried by more than one installed plugin gets no shared mount, because showing either install under it would hand a dashboard the other one's rows under a name that looks right — the hazard `pluginTag` refuses rather than guesses. `/_voltro/inspect/plugins` now reports `baseName` and `inspectSlug` per plugin, which is how a caller reaches a specific install.
+- **@voltro/cli, @voltro/data-transfer** — **`voltro data restore --drill` failed every healthy backup of a real app.** It compared the restored schema's fingerprint against the backup stamp's `schemaFingerprint`, which records the SOURCE database's whole live schema — and the artifact never carries that schema. `pg_dump` / `mariadb-dump` exclude `_voltro_replace_in_progress` and `_voltro_data_transfers` on purpose, and the backup command opens a run row in the second one before it dumps, so on any database the framework has run against, the artifact is two tables short of the value it was being measured against. The drill answered:
+
+  FAIL — restored N table(s), but the schema fingerprint (…) does NOT match the backup's stamp (…). The restore did not reproduce the schema that was backed up — the artifact is inconsistent.
+
+  about an artifact that was exactly right. A drill exists to be wired to a CI cron, and one that is red on every healthy input gets switched off — taking its two real failures with it.
+
+  The stamp now carries a second value, `dumpFingerprint`: the same snapshot minus `dumpExcludedTables(dialect)` — what a faithful restore must reproduce. The drill compares against that. A stamp written before this field degrades to a PARTIAL pass that says so, rather than falling back to the value that produces the false failure. `dumpExcludedTables` is per-dialect because only two of the five backup paths carry an exclusion flag at all: the sqlite/turso copy and the mssql export carry everything, and subtracting a set from those would invent the same bug in the other direction.
+
+  **The drill also checks the one boot-fatal condition a schema comparison cannot see.** `voltro serve`'s boot gate reads the newest `_voltro_migration_plans` row and refuses with `prod-mismatch` when there is none — so a ledger table that restores with exactly the right columns and zero rows is a database no source tree can boot, and its fingerprint is identical to a healthy one's. That is now a FAIL with the reason named. A restored database with no ledger table at all is not a voltro-managed schema and is reported as such, not failed.
+
+  There is deliberately no app boot in the drill. The boot gate is a comparison, not a startup sequence, so the part that generalises is reachable with a SELECT; booting a fixture app instead would prove something about our fixture rather than about your backup.
+
+---
+
 ## [0.54.0] — 2026-08-27
 
 ### ⚠ BREAKING
@@ -114,9 +252,11 @@ _Changes staged for the next release accumulate here (rolled up from
   Each message is processed inside a `queue.consume` span whose parent is the incoming `traceparent`, parsed by the same `externalSpanFromTraceparent` the inbound-webhook path uses — so a malformed or all-zero header means "no parent" (a fresh root span), never a failed message. The span covers the whole message including retries and the dead-letter publish, and carries the OTel `messaging.*` attributes plus `voltro.queue.outcome`. The topic is an attribute, not part of the span name.
 
   Known limit, measured rather than assumed: a span opened from DETACHED work does not reach an OTLP exporter, because the framework's tracer is a `Layer` provided only inside the rpc server's scope. This is framework-wide (`cdcOut.deliver` and `plugin.<name>.schedule-fire` are the same shape) and is stated in the queue docs rather than implied away. The metrics are unaffected — the metric registry is a process global.
-- **@voltro/runtime, @voltro/voltro** — `setRowFilter({ …, tables: ['bookmarks', 'recentSearches'] })` — declare which tables your filter may narrow, and delta-resume survives everywhere else.
+- **@voltro/runtime, @voltro/voltro** — `setRowFilter({ …, tables: ['documents', 'comments'] })` — declare which tables your filter may narrow, and delta-resume survives everywhere else.
+
+  Delta-resume is excluded for a subscription whose row set is re-resolved per delivery: replaying deltas could serve rows the subject has since lost. But the question the runtime could ask was only "is a filter registered in this process?", so ONE registration disabled cheap reconnects for every subscription in the app, including every one whose source the filter could never narrow.
 
-  Delta-resume is excluded for a subscription whose row set is re-resolved per delivery: replaying deltas could serve rows the subject has since lost. But the question the runtime could ask was only "is a filter registered in this process?", so ONE registration disabled cheap reconnects for every subscription in the app — a deployment measured a filter narrowing 4 tables costing the feature on all 173 of their query descriptors, 55 of whose source tables the filter never touches.
+  It does not reach every query. A subscription only has a delta chain when its executor returns a DESCRIPTOR; one that returns a mapped value or a page envelope re-runs an opaque handler and emits snapshots, with or without a filter. `tables:` costs nothing and applies the moment such a query returns a builder — but the count it gives back is the count of descriptor-returning subscriptions, not of queries. The resume census on `/_voltro/inspect/subscriptions` reports which shape each of your queries took.
 
   The exclusion is per table now. A declared filter keeps resume for every subscription whose source is not in its set; an undeclared filter keeps today's conservative behaviour (the runtime cannot know which tables the predicate may reach, and "unknown" must read as "yes").
 

package/dist/index.js

@@ -1,52 +1,52 @@
 import { CommentAccessRefused as e, CommentNotFound as t, CommentNotYours as n } from "./errors.js";
-import { commentsFeed as r, commentsRpcClientImports as i, createDescriptor as a, deleteDescriptor as o, editDescriptor as s, listDescriptor as c, markReadDescriptor as l, mentionSearchDescriptor as u, reactDescriptor as d, resolveDescriptor as f } from "./rpc.js";
-import { Effect as p, Option as m } from "effect";
-import { and as h, eq as g, generateId as _, id as v, json as y, table as b, text as x, timestamp as S } from "@voltro/database";
-import { definePlugin as C, pluginInstanceName as w, publishReactivity as T } from "@voltro/protocol";
-import { NotificationService as E } from "@voltro/plugin-notifications";
+import { BASE_NAME as r, commentsFeed as i, commentsRpcClientImports as a, createDescriptor as o, deleteDescriptor as s, editDescriptor as c, listDescriptor as l, markReadDescriptor as u, mentionSearchDescriptor as d, reactDescriptor as f, resolveDescriptor as p } from "./rpc.js";
+import { Effect as m, Option as h } from "effect";
+import { and as g, eq as _, generateId as v, id as y, json as b, table as x, text as S, timestamp as C } from "@voltro/database";
+import { definePlugin as w, pluginInstanceName as T, publishReactivity as E } from "@voltro/protocol";
+import { NotificationService as D } from "@voltro/plugin-notifications";
 //#region src/index.ts
-var D = b("_voltro_comment_threads", {
-	id: v({ prefix: "cmtthr" }),
-	anchor: x(),
-	status: x().default("open"),
-	createdBy: x(),
-	createdAt: S().default("now"),
-	tenantId: x().nullable()
-}).index("byThreadAnchor", ["anchor", "tenantId"]), O = b("_voltro_comments", {
-	id: v({ prefix: "cmt" }),
-	threadId: x(),
-	body: x(),
-	authorSubjectId: x(),
-	mentions: y(),
-	editedAt: S().nullable(),
-	createdAt: S().default("now"),
-	tenantId: x().nullable()
-}).index("byCommentThread", ["threadId"]), k = b("_voltro_comment_reactions", {
-	id: v({ prefix: "cmtrx" }),
-	commentId: x(),
-	subjectId: x(),
-	emoji: x()
+var O = x("_voltro_comment_threads", {
+	id: y({ prefix: "cmtthr" }),
+	anchor: S(),
+	status: S().default("open"),
+	createdBy: S(),
+	createdAt: C().default("now"),
+	tenantId: S().nullable()
+}).index("byThreadAnchor", ["anchor", "tenantId"]), k = x("_voltro_comments", {
+	id: y({ prefix: "cmt" }),
+	threadId: S(),
+	body: S(),
+	authorSubjectId: S(),
+	mentions: b(),
+	editedAt: C().nullable(),
+	createdAt: C().default("now"),
+	tenantId: S().nullable()
+}).index("byCommentThread", ["threadId"]), A = x("_voltro_comment_reactions", {
+	id: y({ prefix: "cmtrx" }),
+	commentId: S(),
+	subjectId: S(),
+	emoji: S()
 }).unique("byReactionIdentity", [
 	"commentId",
 	"subjectId",
 	"emoji"
-]).index("byReactionComment", ["commentId"]), A = b("_voltro_comment_reads", {
-	id: v({ prefix: "cmtread" }),
-	threadId: x(),
-	subjectId: x(),
-	lastReadAt: S()
-}).unique("byReadThreadSubject", ["threadId", "subjectId"]), j = "@voltro/plugin-comments", M = (e) => {
+]).index("byReactionComment", ["commentId"]), j = x("_voltro_comment_reads", {
+	id: y({ prefix: "cmtread" }),
+	threadId: S(),
+	subjectId: S(),
+	lastReadAt: C()
+}).unique("byReadThreadSubject", ["threadId", "subjectId"]), M = r, N = (e) => {
 	let t = e.request.subject ?? {};
 	return {
 		id: t.id ?? null,
 		tenantId: t.tenantId ?? null,
 		scopes: t.scopes ?? []
 	};
-}, N = (v = {}) => {
-	let y = w({
-		base: j,
-		alias: v.alias,
-		instance: v.name
+}, P = (r = {}) => {
+	let y = T({
+		base: M,
+		alias: r.alias,
+		instance: r.name
 	}), b, x, S = async (e, t, n) => b === void 0 ? [] : b.query({
 		table: e,
 		predicate: t,
@@ -54,40 +54,40 @@ var D = b("_voltro_comment_threads", {
 		take: n,
 		skip: void 0,
 		projection: void 0
-	}), N = () => {
-		T(x, r);
-	}, P = (t, n) => p.gen(function* () {
-		let r = M(n);
-		return v.access?.viaEntity === void 0 ? v.access?.scope === void 0 ? yield* p.fail(new e({
+	}), C = () => {
+		E(x, i);
+	}, P = (t, n) => m.gen(function* () {
+		let i = N(n);
+		return r.access?.viaEntity === void 0 ? r.access?.scope === void 0 ? yield* m.fail(new e({
 			anchor: t,
 			reason: "no access rule declared — set commentsPlugin({ access: { viaEntity } }) or { access: { scope } }"
-		})) : r.scopes.includes(v.access.scope) ? void 0 : yield* p.fail(new e({
+		})) : i.scopes.includes(r.access.scope) ? void 0 : yield* m.fail(new e({
 			anchor: t,
-			reason: `missing scope '${v.access.scope}'`
-		})) : (yield* p.promise(async () => {
+			reason: `missing scope '${r.access.scope}'`
+		})) : (yield* m.promise(async () => {
 			try {
-				return await v.access.viaEntity({
+				return await r.access.viaEntity({
 					anchor: t,
-					subject: r,
+					subject: i,
 					store: b
 				});
 			} catch {
 				return !1;
 			}
-		})) ? void 0 : yield* p.fail(new e({
+		})) ? void 0 : yield* m.fail(new e({
 			anchor: t,
 			reason: "the anchored entity is not readable by this caller (or no longer available)"
 		}));
-	}), F = async (e) => (await S("_voltro_comment_threads", g("id", e), 1))[0], I = (e) => e instanceof Date ? e.toISOString() : typeof e == "string" ? e : "", L = (e) => p.gen(function* () {
+	}), F = async (e) => (await S("_voltro_comment_threads", _("id", e), 1))[0], I = (e) => e instanceof Date ? e.toISOString() : typeof e == "string" ? e : "", L = (e) => m.gen(function* () {
 		if (e.mentions.length === 0) return;
-		let t = yield* p.serviceOption(E);
-		if (m.isNone(t)) {
-			yield* p.logInfo("comments: mention recorded without notification — @voltro/plugin-notifications is not configured");
+		let t = yield* m.serviceOption(D);
+		if (h.isNone(t)) {
+			yield* m.logInfo("comments: mention recorded without notification — @voltro/plugin-notifications is not configured");
 			return;
 		}
-		for (let n of e.mentions) n !== e.author && (yield* p.promise(() => t.value.send({
+		for (let n of e.mentions) n !== e.author && (yield* m.promise(() => t.value.send({
 			to: n,
-			category: v.mentionCategory ?? "mention",
+			category: r.mentionCategory ?? "mention",
 			title: "You were mentioned",
 			body: e.body.length > 200 ? `${e.body.slice(0, 200)}…` : e.body,
 			data: {
@@ -97,24 +97,24 @@ var D = b("_voltro_comment_threads", {
 			tenantId: e.tenantId
 		}).then(() => void 0).catch(() => void 0)));
 	}), R = async (e, t) => {
-		if (e.length === 0 || v.resolveMentions === void 0) return [];
+		if (e.length === 0 || r.resolveMentions === void 0) return [];
 		let n = [];
-		for (let r of [...new Set(e)].slice(0, 20)) {
-			let e = (await v.resolveMentions({
-				query: r,
+		for (let i of [...new Set(e)].slice(0, 20)) {
+			let e = (await r.resolveMentions({
+				query: i,
 				subject: t
-			}).catch(() => [])).find((e) => e.subjectId === r);
-			e !== void 0 && (v.crossTenant === !0 || e.tenantId === void 0 || e.tenantId === null || e.tenantId === t.tenantId) && n.push(r);
+			}).catch(() => [])).find((e) => e.subjectId === i);
+			e !== void 0 && (r.crossTenant === !0 || e.tenantId === void 0 || e.tenantId === null || e.tenantId === t.tenantId) && n.push(i);
 		}
 		return n;
 	}, z = async (e, t, n) => {
-		let r = await S("_voltro_comment_threads", h(g("anchor", e), g("tenantId", t))), i = [];
+		let r = await S("_voltro_comment_threads", g(_("anchor", e), _("tenantId", t))), i = [];
 		for (let e of r) {
-			let t = [...await S("_voltro_comments", g("threadId", String(e.id)))];
+			let t = [...await S("_voltro_comments", _("threadId", String(e.id)))];
 			t.sort((e, t) => I(e.createdAt) < I(t.createdAt) ? -1 : 1);
-			let r = n === null ? [] : await S("_voltro_comment_reads", h(g("threadId", String(e.id)), g("subjectId", n)), 1), a = r[0] ? I(r[0].lastReadAt) : "", o = [];
+			let r = n === null ? [] : await S("_voltro_comment_reads", g(_("threadId", String(e.id)), _("subjectId", n)), 1), a = r[0] ? I(r[0].lastReadAt) : "", o = [];
 			for (let e of t) {
-				let t = await S("_voltro_comment_reactions", g("commentId", String(e.id))), r = /* @__PURE__ */ new Map();
+				let t = await S("_voltro_comment_reactions", _("commentId", String(e.id))), r = /* @__PURE__ */ new Map();
 				for (let e of t) {
 					let t = r.get(String(e.emoji)) ?? {
 						count: 0,
@@ -150,31 +150,31 @@ var D = b("_voltro_comment_threads", {
 		return i.sort((e, t) => String(e.createdAt) < String(t.createdAt) ? -1 : 1), i;
 	}, B = "comments:moderate", V = [
 		{
-			...c,
+			...l,
 			description: "Threads + comments + reactions + unread for one anchor, live.",
-			execute: (e, t) => p.gen(function* () {
+			execute: (e, t) => m.gen(function* () {
 				let { anchor: n } = e;
 				yield* P(n, t);
-				let r = M(t);
-				return yield* p.promise(() => z(n, r.tenantId, r.id));
+				let r = N(t);
+				return yield* m.promise(() => z(n, r.tenantId, r.id));
 			})
 		},
 		{
-			...a,
+			...o,
 			description: "Comment on an anchor — a reply into a thread, or a new thread.",
-			execute: (e, n) => p.gen(function* () {
+			execute: (e, n) => m.gen(function* () {
 				let r = e;
 				yield* P(r.anchor, n);
-				let i = M(n), a = i.id ?? "anonymous", o = b;
-				if (o === void 0) return yield* p.fail(new t({ id: r.anchor }));
+				let i = N(n), a = i.id ?? "anonymous", o = b;
+				if (o === void 0) return yield* m.fail(new t({ id: r.anchor }));
 				let s = r.threadId;
 				if (s !== void 0) {
-					let e = yield* p.promise(() => F(s));
-					if (e === void 0 || String(e.anchor) !== r.anchor) return yield* p.fail(new t({ id: s }));
-				} else s = _({
+					let e = yield* m.promise(() => F(s));
+					if (e === void 0 || String(e.anchor) !== r.anchor) return yield* m.fail(new t({ id: s }));
+				} else s = v({
 					kind: "typeid",
 					prefix: "cmtthr"
-				}, "_voltro_comment_threads"), yield* p.promise(() => o.insert("_voltro_comment_threads", {
+				}, "_voltro_comment_threads"), yield* m.promise(() => o.insert("_voltro_comment_threads", {
 					id: s,
 					anchor: r.anchor,
 					status: "open",
@@ -182,11 +182,11 @@ var D = b("_voltro_comment_threads", {
 					createdAt: (/* @__PURE__ */ new Date()).toISOString(),
 					tenantId: i.tenantId
 				}));
-				let c = yield* p.promise(() => R(r.mentions ?? [], i)), l = _({
+				let c = yield* m.promise(() => R(r.mentions ?? [], i)), l = v({
 					kind: "typeid",
 					prefix: "cmt"
 				}, "_voltro_comments");
-				return yield* p.promise(() => o.insert("_voltro_comments", {
+				return yield* m.promise(() => o.insert("_voltro_comments", {
 					id: l,
 					threadId: s,
 					body: r.body,
@@ -195,7 +195,7 @@ var D = b("_voltro_comment_threads", {
 					editedAt: null,
 					createdAt: (/* @__PURE__ */ new Date()).toISOString(),
 					tenantId: i.tenantId
-				})), N(), yield* L({
+				})), C(), yield* L({
 					mentions: c,
 					author: a,
 					anchor: r.anchor,
@@ -209,76 +209,76 @@ var D = b("_voltro_comment_threads", {
 			})
 		},
 		{
-			...s,
+			...c,
 			description: "Edit your own comment.",
-			execute: (e, r) => p.gen(function* () {
-				let i = e, a = (yield* p.promise(() => S("_voltro_comments", g("id", i.commentId), 1)))[0];
-				if (a === void 0) return yield* p.fail(new t({ id: i.commentId }));
-				let o = yield* p.promise(() => F(String(a.threadId)));
-				return o === void 0 ? yield* p.fail(new t({ id: i.commentId })) : (yield* P(String(o.anchor), r), String(a.authorSubjectId) === (M(r).id ?? "anonymous") ? (yield* p.promise(() => b.update("_voltro_comments", i.commentId, {
+			execute: (e, r) => m.gen(function* () {
+				let i = e, a = (yield* m.promise(() => S("_voltro_comments", _("id", i.commentId), 1)))[0];
+				if (a === void 0) return yield* m.fail(new t({ id: i.commentId }));
+				let o = yield* m.promise(() => F(String(a.threadId)));
+				return o === void 0 ? yield* m.fail(new t({ id: i.commentId })) : (yield* P(String(o.anchor), r), String(a.authorSubjectId) === (N(r).id ?? "anonymous") ? (yield* m.promise(() => b.update("_voltro_comments", i.commentId, {
 					body: i.body,
 					editedAt: (/* @__PURE__ */ new Date()).toISOString()
-				})), N(), { ok: !0 }) : yield* p.fail(new n({ id: i.commentId })));
+				})), C(), { ok: !0 }) : yield* m.fail(new n({ id: i.commentId })));
 			})
 		},
 		{
-			...f,
+			...p,
 			description: "Resolve or reopen a thread.",
-			execute: (e, n) => p.gen(function* () {
-				let r = e, i = yield* p.promise(() => F(r.threadId));
-				return i === void 0 ? yield* p.fail(new t({ id: r.threadId })) : (yield* P(String(i.anchor), n), yield* p.promise(() => b.update("_voltro_comment_threads", r.threadId, { status: r.resolved ? "resolved" : "open" })), N(), { ok: !0 });
+			execute: (e, n) => m.gen(function* () {
+				let r = e, i = yield* m.promise(() => F(r.threadId));
+				return i === void 0 ? yield* m.fail(new t({ id: r.threadId })) : (yield* P(String(i.anchor), n), yield* m.promise(() => b.update("_voltro_comment_threads", r.threadId, { status: r.resolved ? "resolved" : "open" })), C(), { ok: !0 });
 			})
 		},
 		{
-			...o,
+			...s,
 			description: "Delete your own comment (comments:moderate deletes any).",
-			execute: (e, r) => p.gen(function* () {
-				let i = e, a = (yield* p.promise(() => S("_voltro_comments", g("id", i.commentId), 1)))[0];
-				if (a === void 0) return yield* p.fail(new t({ id: i.commentId }));
-				let o = yield* p.promise(() => F(String(a.threadId)));
-				if (o === void 0) return yield* p.fail(new t({ id: i.commentId }));
+			execute: (e, r) => m.gen(function* () {
+				let i = e, a = (yield* m.promise(() => S("_voltro_comments", _("id", i.commentId), 1)))[0];
+				if (a === void 0) return yield* m.fail(new t({ id: i.commentId }));
+				let o = yield* m.promise(() => F(String(a.threadId)));
+				if (o === void 0) return yield* m.fail(new t({ id: i.commentId }));
 				yield* P(String(o.anchor), r);
-				let s = M(r);
-				return String(a.authorSubjectId) !== (s.id ?? "anonymous") && !s.scopes.includes(B) ? yield* p.fail(new n({ id: i.commentId })) : (yield* p.promise(async () => {
-					for (let e of await S("_voltro_comment_reactions", g("commentId", i.commentId))) await b.delete("_voltro_comment_reactions", String(e.id));
+				let s = N(r);
+				return String(a.authorSubjectId) !== (s.id ?? "anonymous") && !s.scopes.includes(B) ? yield* m.fail(new n({ id: i.commentId })) : (yield* m.promise(async () => {
+					for (let e of await S("_voltro_comment_reactions", _("commentId", i.commentId))) await b.delete("_voltro_comment_reactions", String(e.id));
 					await b.delete("_voltro_comments", i.commentId);
-				}), N(), { ok: !0 });
+				}), C(), { ok: !0 });
 			})
 		},
 		{
-			...d,
+			...f,
 			description: "Toggle an emoji reaction on a comment.",
-			execute: (e, n) => p.gen(function* () {
-				let r = e, i = (yield* p.promise(() => S("_voltro_comments", g("id", r.commentId), 1)))[0];
-				if (i === void 0) return yield* p.fail(new t({ id: r.commentId }));
-				let a = yield* p.promise(() => F(String(i.threadId)));
-				if (a === void 0) return yield* p.fail(new t({ id: r.commentId }));
+			execute: (e, n) => m.gen(function* () {
+				let r = e, i = (yield* m.promise(() => S("_voltro_comments", _("id", r.commentId), 1)))[0];
+				if (i === void 0) return yield* m.fail(new t({ id: r.commentId }));
+				let a = yield* m.promise(() => F(String(i.threadId)));
+				if (a === void 0) return yield* m.fail(new t({ id: r.commentId }));
 				yield* P(String(a.anchor), n);
-				let o = M(n).id ?? "anonymous", s = (yield* p.promise(() => S("_voltro_comment_reactions", h(g("commentId", r.commentId), h(g("subjectId", o), g("emoji", r.emoji))), 1)))[0];
-				if (s !== void 0) return yield* p.promise(() => b.delete("_voltro_comment_reactions", String(s.id))), N(), { reacted: !1 };
-				let c = _({
+				let o = N(n).id ?? "anonymous", s = (yield* m.promise(() => S("_voltro_comment_reactions", g(_("commentId", r.commentId), g(_("subjectId", o), _("emoji", r.emoji))), 1)))[0];
+				if (s !== void 0) return yield* m.promise(() => b.delete("_voltro_comment_reactions", String(s.id))), C(), { reacted: !1 };
+				let c = v({
 					kind: "typeid",
 					prefix: "cmtrx"
 				}, "_voltro_comment_reactions");
-				return yield* p.promise(() => b.insert("_voltro_comment_reactions", {
+				return yield* m.promise(() => b.insert("_voltro_comment_reactions", {
 					id: c,
 					commentId: r.commentId,
 					subjectId: o,
 					emoji: r.emoji
-				})), N(), { reacted: !0 };
+				})), C(), { reacted: !0 };
 			})
 		},
 		{
-			...l,
+			...u,
 			description: "Stamp the caller's read marker for a thread.",
-			execute: (e, n) => p.gen(function* () {
+			execute: (e, n) => m.gen(function* () {
 				let r = e;
-				if ((yield* p.promise(() => F(r.threadId))) === void 0) return yield* p.fail(new t({ id: r.threadId }));
-				let i = M(n).id ?? "anonymous", a = _({
+				if ((yield* m.promise(() => F(r.threadId))) === void 0) return yield* m.fail(new t({ id: r.threadId }));
+				let i = N(n).id ?? "anonymous", a = v({
 					kind: "typeid",
 					prefix: "cmtread"
 				}, "_voltro_comment_reads");
-				return yield* p.promise(() => b.upsert("_voltro_comment_reads", {
+				return yield* m.promise(() => b.upsert("_voltro_comment_reads", {
 					id: a,
 					threadId: r.threadId,
 					subjectId: i,
@@ -290,19 +290,19 @@ var D = b("_voltro_comment_threads", {
 			})
 		},
 		{
-			...u,
+			...d,
 			description: "@-mention autocomplete over the app-declared directory, tenant-filtered.",
-			execute: (e, t) => p.promise(async () => {
+			execute: (e, t) => m.promise(async () => {
 				let { query: n } = e;
-				if (v.resolveMentions === void 0) return [];
-				let r = M(t);
-				return (await v.resolveMentions({
+				if (r.resolveMentions === void 0) return [];
+				let i = N(t);
+				return (await r.resolveMentions({
 					query: n,
 					subject: {
-						id: r.id,
-						tenantId: r.tenantId
+						id: i.id,
+						tenantId: i.tenantId
 					}
-				}).catch(() => [])).filter((e) => v.crossTenant === !0 || e.tenantId === void 0 || e.tenantId === null || e.tenantId === r.tenantId).map((e) => ({
+				}).catch(() => [])).filter((e) => r.crossTenant === !0 || e.tenantId === void 0 || e.tenantId === null || e.tenantId === i.tenantId).map((e) => ({
 					subjectId: e.subjectId,
 					label: e.label
 				}));
@@ -312,25 +312,25 @@ var D = b("_voltro_comment_threads", {
 		kind: "json",
 		data: e
 	});
-	return C({
+	return w({
 		name: y,
-		baseName: j,
+		baseName: M,
 		description: "Comment threads on any app entity — replies, resolve/reopen, @-mentions with notifications, reactions, unread — live over the reactive engine.",
 		permissions: ["inspect:read", "store:write"],
 		declaredScopes: [B],
 		extendSchema: { tables: [
-			D,
 			O,
 			k,
-			A
+			A,
+			j
 		] },
 		routes: V,
-		rpcClientDescriptors: i,
+		rpcClientDescriptors: a,
 		inspectEndpoints: [{
 			method: "GET",
 			path: "/threads",
 			description: "Thread volume + the most recent threads (operator surface).",
-			handler: () => p.promise(async () => {
+			handler: () => m.promise(async () => {
 				let e = await S("_voltro_comment_threads", void 0), t = await S("_voltro_comments", void 0), n = [...e].sort((e, t) => I(e.createdAt) > I(t.createdAt) ? -1 : 1).slice(0, 50).map((e) => ({
 					id: String(e.id),
 					anchor: String(e.anchor),
@@ -352,13 +352,13 @@ var D = b("_voltro_comment_threads", {
 		bindDataStore: (e) => {
 			b = e, x = e;
 		},
-		onActivate: (e) => p.sync(() => {
-			v.access?.viaEntity === void 0 && v.access?.scope === void 0 && e.logger.warn("comments: no access rule declared — EVERY read and write will be refused (fail-closed). Set commentsPlugin({ access: { viaEntity } }) or { access: { scope } }."), e.logger.info("comments active", { mentions: v.resolveMentions !== void 0 });
+		onActivate: (e) => m.sync(() => {
+			r.access?.viaEntity === void 0 && r.access?.scope === void 0 && e.logger.warn("comments: no access rule declared — EVERY read and write will be refused (fail-closed). Set commentsPlugin({ access: { viaEntity } }) or { access: { scope } }."), e.logger.info("comments active", { mentions: r.resolveMentions !== void 0 });
 		}),
-		onDeactivate: () => p.sync(() => {
+		onDeactivate: () => m.sync(() => {
 			b = void 0, x = void 0;
 		})
 	});
-}, P = () => "plugin-comments";
+}, F = () => "plugin-comments";
 //#endregion
-export { e as CommentAccessRefused, t as CommentNotFound, n as CommentNotYours, k as commentReactionsTable, A as commentReadsTable, D as commentThreadsTable, r as commentsFeed, N as commentsPlugin, O as commentsTable, P as pluginComments };
+export { e as CommentAccessRefused, t as CommentNotFound, n as CommentNotYours, A as commentReactionsTable, j as commentReadsTable, O as commentThreadsTable, i as commentsFeed, P as commentsPlugin, k as commentsTable, F as pluginComments };

package/dist/rpc.d.ts

@@ -4,6 +4,18 @@ import { QueryProcedureDescriptor } from '@voltro/protocol';
 import { ReactivityChannel } from '@voltro/protocol';
 import { Schema } from 'effect';
 
+/**
+ * The plugin's canonical package name — the key an app's `alias` is recorded
+ * under, and what `./web`'s hooks resolve their tags through.
+ *
+ * It lives in this BROWSER-SAFE module rather than in `index.ts` because both
+ * halves need it and only one of them may reach the server graph: `index.ts`
+ * passes it to `pluginInstanceName` / `baseName`, and `web.ts` passes it to
+ * `pluginTag`. A second copy in `web.ts` would be a second definition of the
+ * plugin's identity with nothing comparing them.
+ */
+export declare const BASE_NAME = "@voltro/plugin-comments";
+
 /** The anchor's access check refused the caller — or no access rule is
  *  declared at all (fail-closed: undeclared access is a refusal, not a pass).
  *  Also the answer for a thread whose anchor row no longer resolves (a

package/dist/rpc.js

@@ -2,7 +2,7 @@ import { CommentAccessRefused as e, CommentNotFound as t, CommentNotYours as n }
 import { Schema as r } from "effect";
 import { defineMutation as i, defineQuery as a, reactivityChannel as o } from "@voltro/protocol";
 //#region src/rpc.ts
-var s = o("comments"), c = r.Struct({
+var s = "@voltro/plugin-comments", c = o("comments"), l = r.Struct({
 	id: r.String,
 	threadId: r.String,
 	body: r.String,
@@ -15,22 +15,22 @@ var s = o("comments"), c = r.Struct({
 		count: r.Number,
 		mine: r.Boolean
 	}))
-}), l = r.Struct({
+}), u = r.Struct({
 	id: r.String,
 	anchor: r.String,
 	status: r.Literal("open", "resolved"),
 	createdBy: r.String,
 	createdAt: r.String,
-	comments: r.Array(c),
+	comments: r.Array(l),
 	unreadCount: r.Number
-}), u = a({
+}), d = a({
 	name: "comments.list",
-	source: s,
+	source: c,
 	input: r.Struct({ anchor: r.String }),
-	output: r.Array(l),
+	output: r.Array(u),
 	error: e,
 	openAccess: "delegated: the executor runs the app-declared anchor access rule (viaEntity guard or scope) and is fail-closed — an undeclared rule refuses every read"
-}), d = i({
+}), f = i({
 	name: "comments.create",
 	input: r.Struct({
 		anchor: r.String,
@@ -44,7 +44,7 @@ var s = o("comments"), c = r.Struct({
 	}),
 	error: r.Union(e, t),
 	openAccess: "delegated: the executor runs the app-declared anchor access rule before any write; the author is always the calling subject"
-}), f = i({
+}), p = i({
 	name: "comments.edit",
 	input: r.Struct({
 		commentId: r.String,
@@ -53,7 +53,7 @@ var s = o("comments"), c = r.Struct({
 	output: r.Struct({ ok: r.Boolean }),
 	error: r.Union(e, t, n),
 	openAccess: "author-scoped: only the comment's own author may edit, checked in the executor on top of the anchor access rule"
-}), p = i({
+}), m = i({
 	name: "comments.resolve",
 	input: r.Struct({
 		threadId: r.String,
@@ -62,13 +62,13 @@ var s = o("comments"), c = r.Struct({
 	output: r.Struct({ ok: r.Boolean }),
 	error: r.Union(e, t),
 	openAccess: "delegated: anchor access rule in the executor — anyone who may read the anchor may resolve/reopen (the Liveblocks semantic)"
-}), m = i({
+}), h = i({
 	name: "comments.delete",
 	input: r.Struct({ commentId: r.String }),
 	output: r.Struct({ ok: r.Boolean }),
 	error: r.Union(e, t, n),
 	openAccess: "author-scoped, with a comments:moderate scope override — both checked in the executor on top of the anchor access rule"
-}), h = i({
+}), g = i({
 	name: "comments.react",
 	input: r.Struct({
 		commentId: r.String,
@@ -77,13 +77,13 @@ var s = o("comments"), c = r.Struct({
 	output: r.Struct({ reacted: r.Boolean }),
 	error: r.Union(e, t),
 	openAccess: "delegated: anchor access rule in the executor; the toggle writes only the calling subject's own reaction row"
-}), g = i({
+}), _ = i({
 	name: "comments.markRead",
 	input: r.Struct({ threadId: r.String }),
 	output: r.Struct({ ok: r.Boolean }),
 	error: t,
 	openAccess: "self-scoped write: stamps only the calling subject's own read marker for the thread"
-}), _ = a({
+}), v = a({
 	name: "comments.mentionSearch",
 	input: r.Struct({ query: r.String }),
 	output: r.Array(r.Struct({
@@ -91,7 +91,7 @@ var s = o("comments"), c = r.Struct({
 		label: r.String
 	})),
 	openAccess: "delegated: the app-declared mention resolver receives the CALLING subject and its default filters to the caller's tenant — cross-tenant subjects are structurally unreachable"
-}), v = [
+}), y = [
 	{
 		tag: "comments.list",
 		kind: "query",
@@ -158,4 +158,4 @@ var s = o("comments"), c = r.Struct({
 	}
 ];
 //#endregion
-export { s as commentsFeed, v as commentsRpcClientImports, d as createDescriptor, m as deleteDescriptor, f as editDescriptor, u as listDescriptor, g as markReadDescriptor, _ as mentionSearchDescriptor, h as reactDescriptor, p as resolveDescriptor };
+export { s as BASE_NAME, c as commentsFeed, y as commentsRpcClientImports, f as createDescriptor, h as deleteDescriptor, p as editDescriptor, d as listDescriptor, _ as markReadDescriptor, v as mentionSearchDescriptor, g as reactDescriptor, m as resolveDescriptor };

package/dist/web.js

@@ -1,35 +1,37 @@
-import { useMutation as e, useSubscription as t } from "@voltro/client";
+import { BASE_NAME as e } from "./rpc.js";
+import { pluginTag as t } from "@voltro/protocol";
+import { useMutation as n, useSubscription as r } from "@voltro/client";
 //#region src/web.ts
-var n = (n, r = "app") => {
-	let { data: i, loading: a } = t(r, "comments.list", { anchor: n }), o = e(r, "comments.create"), s = e(r, "comments.edit"), c = e(r, "comments.resolve"), l = e(r, "comments.delete"), u = e(r, "comments.react"), d = e(r, "comments.markRead"), f = i ?? [];
+var i = (n) => t(e, n), a = (e, t = "app") => {
+	let { data: a, loading: o } = r(t, i("list"), { anchor: e }), s = n(t, i("create")), c = n(t, i("edit")), l = n(t, i("resolve")), u = n(t, i("delete")), d = n(t, i("react")), f = n(t, i("markRead")), p = a ?? [];
 	return {
-		threads: f,
-		pending: a,
-		unreadCount: f.reduce((e, t) => e + t.unreadCount, 0),
-		create: (e, t) => o.mutate({
-			anchor: n,
-			body: e,
-			...t?.threadId === void 0 ? {} : { threadId: t.threadId },
-			...t?.mentions === void 0 ? {} : { mentions: t.mentions }
+		threads: p,
+		pending: o,
+		unreadCount: p.reduce((e, t) => e + t.unreadCount, 0),
+		create: (t, n) => s.mutate({
+			anchor: e,
+			body: t,
+			...n?.threadId === void 0 ? {} : { threadId: n.threadId },
+			...n?.mentions === void 0 ? {} : { mentions: n.mentions }
 		}),
-		edit: (e, t) => s.mutate({
+		edit: (e, t) => c.mutate({
 			commentId: e,
 			body: t
 		}),
-		resolve: (e, t = !0) => c.mutate({
+		resolve: (e, t = !0) => l.mutate({
 			threadId: e,
 			resolved: t
 		}),
-		remove: (e) => l.mutate({ commentId: e }),
-		react: (e, t) => u.mutate({
+		remove: (e) => u.mutate({ commentId: e }),
+		react: (e, t) => d.mutate({
 			commentId: e,
 			emoji: t
 		}),
-		markRead: (e) => d.mutate({ threadId: e })
+		markRead: (e) => f.mutate({ threadId: e })
 	};
-}, r = (e, t, r = "app") => n(e, r).threads.find((e) => e.id === t), i = (e, n = "app") => {
-	let { data: r } = t(n, "comments.mentionSearch", { query: e });
-	return r ?? [];
+}, o = (e, t, n = "app") => a(e, n).threads.find((e) => e.id === t), s = (e, t = "app") => {
+	let { data: n } = r(t, i("mentionSearch"), { query: e });
+	return n ?? [];
 };
 //#endregion
-export { n as useComments, i as useMentionSearch, r as useThread };
+export { a as useComments, s as useMentionSearch, o as useThread };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@voltro/plugin-comments",
-  "version": "0.54.0",
+  "version": "0.55.0",
   "description": "Comment threads on any anchor your app can name — replies, resolve/reopen, reactions, per-subject unread — live over the existing reactivity channel, with no second push mechanism. Access delegates to your app's own guard (access.viaEntity) or a scope and fails closed when neither is declared; @-mentions are re-filtered to the caller's tenant on search and re-validated at create, then delivered through plugin-notifications when it is installed. useComments / useThread / useMentionSearch in /web, an ejectable <CommentsThread> in @voltro/ui, plus inspect endpoints and a dashboard panel.",
   "keywords": [
     "voltro",
@@ -48,9 +48,9 @@
     "node": ">=24.0.0"
   },
   "dependencies": {
-    "@voltro/database": "0.54.0",
-    "@voltro/protocol": "0.54.0",
-    "@voltro/plugin-notifications": "0.54.0",
+    "@voltro/database": "0.55.0",
+    "@voltro/protocol": "0.55.0",
+    "@voltro/plugin-notifications": "0.55.0",
     "effect": "^3.22.0"
   },
   "peerDependencies": {