@camstack/addon-benchmark 1.2.21 → 1.2.22

package/dist/index.mjs

@@ -5,9 +5,9 @@ var __export = (target, all) => {
 };
 
 // src/benchmark-addon.ts
+import * as fs2 from "fs";
 import { fileURLToPath } from "url";
 import * as path2 from "path";
-import * as fs2 from "fs";
 import { BaseAddon, addonPagesSourceCapability, parseJsonUnknown, errMsg } from "@camstack/types";
 
 // ../../node_modules/zod/v4/classic/external.js
@@ -14524,6 +14524,9 @@ function date4(params) {
 // ../../node_modules/zod/v4/classic/external.js
 config(en_default());
 
+// src/benchmark-actions.ts
+import { customAction, defineCustomActions, PipelineRunResultBridge } from "@camstack/types";
+
 // src/benchmark-schemas.ts
 import { HWACCEL_OPTIONS, PipelineStepInputSchema } from "@camstack/types";
 var PipelineBenchmarkHistoryEntrySchema = external_exports.object({
@@ -14540,8 +14543,17 @@ var PipelineBenchmarkHistoryEntrySchema = external_exports.object({
   medianMs: external_exports.number(),
   p95Ms: external_exports.number(),
   p99Ms: external_exports.number(),
-  minMs: external_exports.number(),
-  maxMs: external_exports.number(),
+  /**
+   * OPTIONAL, and absent on every entry the page writes.
+   * `pipeline-runner.runSyntheticBench` reports mean / p50 / p95 / p99 and
+   * measures no per-call min or max. They were required here while nothing
+   * in the page ever called `saveBenchmarkResult`, so nothing noticed; the
+   * moment it does, a required field can only be satisfied by inventing a
+   * number, and a fabricated latency in the history table is worse than a
+   * blank one.
+   */
+  minMs: external_exports.number().optional(),
+  maxMs: external_exports.number().optional(),
   avgDetections: external_exports.number(),
   referenceImage: external_exports.string().optional(),
   /** Number of concurrent runs aggregated into this entry; 1 = single run. */
@@ -14693,7 +14705,9 @@ var PipelineRunInputSchema = external_exports.object({
     backend: external_exports.string(),
     format: external_exports.enum(["onnx", "coreml", "openvino", "tflite"]),
     device: external_exports.string().optional()
-  }).optional()
+  }).optional(),
+  deviceKey: external_exports.string().optional(),
+  plane: external_exports.enum(["full", "frame"]).optional()
 });
 var PipelineRunBatchInputSchema = external_exports.object({
   steps: external_exports.array(PipelineStepInputSchema).readonly(),
@@ -14799,7 +14813,6 @@ var SustainedThroughputResultSchema = external_exports.object({
 });
 
 // src/benchmark-actions.ts
-import { customAction, defineCustomActions, PipelineRunResultBridge } from "@camstack/types";
 var benchmarkActions = defineCustomActions({
   // Pipeline tester
   runPipelineTest: customAction(PipelineRunInputSchema, PipelineRunResultBridge, {
@@ -14829,6 +14842,11 @@ var benchmarkActions = defineCustomActions({
    * addon, then forwards N FrameInputs to the target node in a single
    * `pipelineExecutor.runPipelineBatch` cap call. Returns N
    * `FrameResult`s plus aggregate timing.
+   *
+   * NO UI CALLER — and it is NOT dead. `scripts/bench-pool-ceiling.mts` and
+   * `scripts/bench-pipeline-investigation.mts` drive it directly. Do not
+   * delete it on the strength of an in-package grep; those scripts break
+   * silently and nothing in CI runs them.
    */
   runPipelineBatchTest: customAction(PipelineRunBatchInputSchema, PipelineRunBatchResultSchema, {
     kind: "mutation"
@@ -14839,6 +14857,9 @@ var benchmarkActions = defineCustomActions({
    * issuing a configurable mix of batched + single cap calls). Returns
    * only aggregate detection-rate stats. Used to measure the pool's
    * true throughput ceiling without per-call tRPC overhead.
+   *
+   * NO UI CALLER — and it is NOT dead. `scripts/bench-pool-inprocess.mts`
+   * drives it directly. Same warning as `runPipelineBatchTest` above.
    */
   runMultiCameraStressTest: customAction(
     MultiCameraStressInputSchema,
@@ -14864,8 +14885,16 @@ var benchmarkActions = defineCustomActions({
     external_exports.object({ aborted: external_exports.boolean() }),
     { kind: "mutation" }
   ),
-  // History (shared across pipeline + decoder-perf entries)
-  listBenchmarkHistory: customAction(external_exports.void(), external_exports.array(HistoryEntrySchema).readonly()),
+  /**
+   * History (shared across pipeline + decoder-perf entries).
+   *
+   * `z.object({})`, never `z.void()`: the `addons.custom` envelope insists
+   * on an `input` object, and a `z.void()` schema REJECTS `{}` — so the
+   * action resolved and could never actually be called, not from a settings
+   * button and not from `hub.mjs`. The tabs only got away with it by
+   * sending `input: undefined`.
+   */
+  listBenchmarkHistory: customAction(external_exports.object({}), external_exports.array(HistoryEntrySchema).readonly()),
   saveBenchmarkResult: customAction(
     PipelineBenchmarkHistoryEntryInputSchema,
     PipelineBenchmarkHistoryEntrySchema,
@@ -14881,11 +14910,12 @@ var benchmarkActions = defineCustomActions({
     external_exports.object({ success: external_exports.literal(true) }),
     { kind: "mutation" }
   ),
-  clearBenchmarkHistory: customAction(external_exports.void(), external_exports.object({ success: external_exports.literal(true) }), {
+  clearBenchmarkHistory: customAction(external_exports.object({}), external_exports.object({ success: external_exports.literal(true) }), {
     kind: "mutation"
   }),
-  // Decoder perf — fixture discovery + run control
-  listDecoderFixtures: customAction(external_exports.void(), external_exports.array(DecoderFixtureSchema).readonly()),
+  // Decoder perf — fixture discovery + run control.
+  // `z.object({})` for the same reason as `listBenchmarkHistory` above.
+  listDecoderFixtures: customAction(external_exports.object({}), external_exports.array(DecoderFixtureSchema).readonly()),
   runDecoderPerformanceTest: customAction(DecoderPerfInputSchema, DecoderPerfResultSchema, {
     kind: "mutation"
   }),
@@ -15433,9 +15463,12 @@ var BenchmarkAddon = class extends BaseAddon {
   }
   async runPipelineTest(input) {
     const { nodeId, steps, imageBase64, referenceImage, frameId, sessionId, engine } = input;
+    const { deviceKey, plane } = input;
     this.ctx.logger.debug("runPipelineTest payload", {
       meta: {
         nodeId: nodeId ?? null,
+        deviceKey: deviceKey ?? null,
+        plane: plane ?? null,
         sessionId: sessionId ?? null,
         engine: engine ?? null,
         stepsCount: steps.length,
@@ -15469,7 +15502,12 @@ var BenchmarkAddon = class extends BaseAddon {
         ...!image && imageBase64 ? { imageBase64 } : {},
         referenceImage,
         sessionId,
-        ...engine ? { engine } : {}
+        ...engine ? { engine } : {},
+        // Absent, never present-and-undefined: the generated router takes
+        // `.loose()` input and an explicit `undefined` is not the same thing
+        // as "the node's default pool" / "the cap's default plane".
+        ...deviceKey ? { deviceKey } : {},
+        ...plane ? { plane } : {}
       }),
       BENCH_RUN_TIMEOUT_MS,
       "runPipelineTest"