@trillboards/edge-sdk 0.2.80 → 0.2.81

package/deploy/detector/tests/test_server.py

@@ -317,12 +317,40 @@ class TestClassFilter(ServerCase):
         self.assertEqual(status, 400)
         self.assertIn("shoplifter", body["error"])
 
-    def test_a_partly_unknown_list_proceeds_on_the_known_half(self):
+    def test_a_partly_unknown_list_is_ALSO_a_loud_400(self):
+        """The partial case is the dangerous one, and it used to return 200.
+
+        `["person", "shoplifter"]` proceeded on `person` and dropped
+        `shoplifter` silently, so a profile asking about shoplifting got person
+        counts and no signal that its actual question was discarded. That is the
+        same `yolov8_nano` shape the test above is written against — a capability
+        offered and not delivered — only wearing a 200 instead of a 0%.
+
+        It becomes the COMMON case once specialist models exist: a PPE profile
+        asks for `glove` and `hairnet` alongside `person`, and on a box holding
+        only the general detector two thirds of the question would vanish.
+
+        Refusing costs the `person` detections too. That is the right trade —
+        which models are resident is decided upstream by the profile push, so a
+        class the box cannot answer is a configuration error, and the client maps
+        400 to `DetectorUnavailableError`, which shadow mode records as
+        `unavailable`. Visible beats quieter.
+        """
+        loader = FakeLoader()
+        url = self.serve(loader)
+        status, body = self.detect(url, classes=["person", "shoplifter"])
+        self.assertEqual(status, 400)
+        self.assertIn("shoplifter", body["error"])
+        # And the detector was never asked — we refuse before inferring.
+        self.assertEqual(loader.calls, [])
+
+    def test_a_fully_known_list_still_reaches_the_detector(self):
+        """The refusal above must not have made every class request fail."""
         loader = FakeLoader()
         url = self.serve(loader)
-        status, _ = self.detect(url, classes=["person", "shoplifter"])
+        status, _ = self.detect(url, classes=["person", "cell phone"])
         self.assertEqual(status, 200)
-        self.assertEqual(loader.calls[0][2], {"person"})
+        self.assertEqual(loader.calls[0][2], {"person", "cell phone"})
 
 
 class TestRequestRefusals(ServerCase):

package/deploy/detector/trillboards_detector/server.py

@@ -115,12 +115,35 @@ def parse_detect_request(raw: bytes) -> tuple[bytes, float, "set[str] | None"]:
         raise WireError(400, "`classes` was an empty array; omit the key to mean every class")
 
     wanted, unknown = resolve_requested_classes(raw_classes)
-    if not wanted:
+    # ANY unknown class is a 400, not just all of them.
+    #
+    # This used to reject only when `wanted` came back empty, so a request for
+    # ["person", "glove"] against a COCO-only model returned person detections
+    # and dropped `glove` on the floor. `resolve_requested_classes` keeps its own
+    # promise — it returns the unknowns rather than discarding them — and its
+    # docstring says exactly why: "a profile naming a class the model cannot emit
+    # is exactly the `yolov8_nano` shape". The caller then discarded them one
+    # level up, which is where the promise was actually broken.
+    #
+    # The partial case is the dangerous one and it becomes the COMMON one the
+    # moment specialist models exist: a PPE profile asks for `glove` and
+    # `hairnet` alongside `person`, and on a box holding only the general
+    # detector the operator would get person counts and silence about gloves.
+    #
+    # Refusing loses the person detections too, which is the cost. It is the
+    # right trade: which models are resident is decided upstream by the profile
+    # push, so a class the box cannot answer is a configuration error, and the
+    # client maps a 400 to `DetectorUnavailableError` -> shadow mode records
+    # `unavailable`. A wrong configuration becomes visible instead of becoming a
+    # quieter answer.
+    if unknown:
         raise WireError(
             400,
-            f"none of the requested classes exist in this model's vocabulary: {unknown}. "
+            f"requested classes not in this model's vocabulary: {unknown}. "
             f"Known classes: {sorted(KNOWN_CLASS_NAMES)}",
         )
+    if not wanted:
+        raise WireError(400, "`classes` resolved to nothing requestable")
     return jpeg, min_confidence, wanted
 
 

package/dist/EdgeAgent.js

@@ -44,7 +44,7 @@ const edge_ads_1 = require("@trillboards/edge-ads");
 const CommandRouter_1 = require("./CommandRouter");
 const config_1 = require("./config");
 const dataDir_1 = require("./dataDir");
-const SDK_VERSION = '0.2.80';
+const SDK_VERSION = '0.2.81';
 class EdgeAgent extends events_1.EventEmitter {
     constructor(config, deps) {
         super();

package/dist/diagnose.d.ts

@@ -31,7 +31,7 @@ export interface DiagnosticReport {
         skipped: number;
     };
 }
-declare const SDK_VERSION = "0.2.80";
+declare const SDK_VERSION = "0.2.81";
 declare function checkNodeVersion(): CheckResult;
 declare function checkPlatform(): CheckResult;
 declare function checkMemory(): CheckResult;

package/dist/diagnose.js

@@ -63,7 +63,7 @@ const localNetworkAccess_1 = require("./localNetworkAccess");
 // ---------------------------------------------------------------------------
 // SDK version — mirrors package.json; kept in sync by CI
 // ---------------------------------------------------------------------------
-const SDK_VERSION = '0.2.80';
+const SDK_VERSION = '0.2.81';
 exports._SDK_VERSION = SDK_VERSION;
 // ---------------------------------------------------------------------------
 // ANSI helpers (no external deps)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trillboards/edge-sdk",
-  "version": "0.2.80",
+  "version": "0.2.81",
   "description": "Composable cross-platform Edge AI SDK for audience sensing on DOOH/CTV devices",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",