@camstack/server 1.2.357 → 1.2.358

package/dist/api/trpc/device-collection-dispatch.js

@@ -17,19 +17,25 @@
  * The three rules this module implements, and nothing else:
  *
  *  1. **Fan out, per device.** An ARRAY-returning method is called on every
- *     source the DEVICE has (`CapabilityRegistry.resolveCollectionForDevice` —
- *     the enabled providers, minus the ones the operator switched off for this
- *     camera through `deviceManager.setWrapperActive`, D556), and the results
- *     are concatenated. No dedup ACROSS SOURCES, no best-source pick: a clip
- *     held in two storages appears twice, once per source, and each row says
- *     which (D549 decision 11 — stated there as a consequence precisely so it
- *     is not "fixed" later). But a source is not a REGISTRATION: one addon
- *     reaches this dispatcher as the device's native provider, as the
- *     hub-local registration and once per agent node, and those are one
- *     source answering three times. Each row is held once, keyed by the
- *     identity its provider minted ({@link holdEachRowOnce}); on 592 that was
- *     three `analytics` tick-boxes for one thing, D62's defect with the
+ *     source the DEVICE has — its BINDINGS ({@link sourcesForDevice}), not the
+ *     registrations — and the results are concatenated. No dedup ACROSS
+ *     SOURCES, no best-source pick: a clip held in two storages appears twice,
+ *     once per source, and each row says which (D549 decision 11 — stated
+ *     there as a consequence precisely so it is not "fixed" later). A source
+ *     is not a REGISTRATION: one addon reaches this dispatcher as the device's
+ *     native provider, as the hub-local registration and once per agent node.
+ *     Those three are collapsed by the binding set itself, which cannot hold
+ *     one addon twice — never by a de-duplicator over the answer. On 592 they
+ *     were three `analytics` tick-boxes for one thing, D62's defect with the
  *     polarity reversed (D554 amended 2026-09-20).
+ *
+ *     **A caller may also pin ONE source, and for `videoclips.listClips` the
+ *     pin is REQUIRED** — in the cap's input schema, not here (D554 amended). This
+ *     dispatcher has no notion of a DEFAULT provider and never had one: a
+ *     collection cap's binding set is derived and plural, so an absent pin
+ *     left the fan-out un-narrowed and bought a union. The cap's docblock
+ *     said otherwise for a day, and device 592 answered a `limit: 3` call
+ *     with six rows because of it.
  *  2. **One failing source never empties the others.** Every source is called
  *     independently and a rejection removes only its own rows — and says so,
  *     per source, with `tags: { deviceId }`. A fan-out where one throwing
@@ -74,8 +80,12 @@ const SOURCE_ADDON_FIELD = 'addonId';
 /**
  * The input field naming WHICH of a device's providers to ask. One at a time,
  * by `addonId`, never a list of source ids: the operator settled the surface
- * on 2026-09-20 (*"l'utilizzatore è uno solo"*). Absent means the device's
- * bound provider.
+ * on 2026-09-20 (*"l'utilizzatore è uno solo"*).
+ *
+ * An absent pin narrows NOTHING — the method fans out, which is right for
+ * `listSources` and was never right for `listClips`. Whether absence is
+ * allowed is the CAP's to say, in its input schema, and `videoclips.listClips`
+ * says no (D554 amended).
  */
 const PROVIDER_PIN_FIELD = 'provider';
 /** The capability whose singleton answers what a device is bound to. */
@@ -331,9 +341,9 @@ async function fanOut(sources, method, input, capName, deviceId, logger) {
  * mark the listing row that let him switch it back on. That whole half went on
  * 2026-09-20: every provider always works, and a source exists for a camera
  * because a binding put it there — so "off" is an unbound provider, not a
- * second store beside the binding (D556/D557 amended). Nothing here filters
- * any more; the filtering happened when {@link sourcesForDevice} read the
- * bindings.
+ * second store beside the binding (D554 amended, D557 amended). Nothing here
+ * filters any more; the filtering happened when {@link sourcesForDevice} read
+ * the bindings.
  */
 function finishRows(source, rows, isSourceListing) {
     if (!isSourceListing)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camstack/server",
-  "version": "1.2.357",
+  "version": "1.2.358",
   "private": false,
   "files": [
     "dist",
@@ -35,18 +35,18 @@
     ]
   },
   "dependencies": {
-    "@camstack/addon-admin-ui": "1.2.246",
-    "@camstack/addon-agent-ui": "1.2.134",
+    "@camstack/addon-admin-ui": "1.2.247",
+    "@camstack/addon-agent-ui": "1.2.135",
     "@camstack/addon-auth": "1.2.123",
     "@camstack/addon-decoder-nodeav": "1.2.115",
     "@camstack/addon-notifiers": "1.2.119",
     "@camstack/addon-pipeline": "1.2.274",
-    "@camstack/addon-pipeline-orchestrator": "1.2.232",
+    "@camstack/addon-pipeline-orchestrator": "1.2.233",
     "@camstack/addon-post-analysis": "1.2.273",
     "@camstack/sdk": "1.2.118",
-    "@camstack/system": "1.2.278",
-    "@camstack/types": "1.2.236",
-    "@camstack/ui-library": "1.2.192",
+    "@camstack/system": "1.2.279",
+    "@camstack/types": "1.2.237",
+    "@camstack/ui-library": "1.2.193",
     "@fastify/compress": "^9.0.0",
     "@fastify/cookie": "^11.0.2",
     "@fastify/cors": "^11.2.0",