npm - @polderlabs/bizar - Versions diffs - 5.0.2 → 5.1.0 - Mend

@polderlabs/bizar 5.0.2 → 5.1.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (60) hide show

package/bizar-dash/dist/index.html CHANGED Viewed

@@ -29,11 +29,13 @@
         }
       })();
     </script>
-    <script type="module" crossorigin src="/assets/main-C1cpttnv.js"></script>
-    <link rel="modulepreload" crossorigin href="/assets/react-vendor-DZRUXSPQ.js">
-    <link rel="modulepreload" crossorigin href="/assets/icons-Bo0iH9EC.js">
-    <link rel="modulepreload" crossorigin href="/assets/markdown-DIquRulQ.js">
-    <link rel="modulepreload" crossorigin href="/assets/mobile-BtxQJftK.js">
+    <script type="module" crossorigin src="/assets/main-8feQWXiF.js"></script>
+    <link rel="modulepreload" crossorigin href="/assets/react-vendor-Dn4wqh4Z.js">
+    <link rel="modulepreload" crossorigin href="/assets/icons-Clz0NR6Y.js">
+    <link rel="modulepreload" crossorigin href="/assets/mobile-IaZ47uKC.js">
+    <link rel="modulepreload" crossorigin href="/assets/vendor-CeHGtduv.js">
+    <link rel="modulepreload" crossorigin href="/assets/markdown-C6mXtQxD.js">
+    <link rel="modulepreload" crossorigin href="/assets/useSlashCommands-DjEwHl4n.js">
     <link rel="stylesheet" crossorigin href="/assets/mobile-CsZQAswA.css">
     <link rel="stylesheet" crossorigin href="/assets/main-DTkNlLrw.css">
   </head>

package/bizar-dash/dist/mobile.html CHANGED Viewed

@@ -12,11 +12,10 @@
     <link rel="preconnect" href="https://fonts.googleapis.com" />
     <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin />
     <link rel="stylesheet" href="https://rsms.me/inter/inter.css" />
-    <script type="module" crossorigin src="/assets/mobile-DD-FZrTC.js"></script>
-    <link rel="modulepreload" crossorigin href="/assets/react-vendor-DZRUXSPQ.js">
-    <link rel="modulepreload" crossorigin href="/assets/icons-Bo0iH9EC.js">
-    <link rel="modulepreload" crossorigin href="/assets/markdown-DIquRulQ.js">
-    <link rel="modulepreload" crossorigin href="/assets/mobile-BtxQJftK.js">
+    <script type="module" crossorigin src="/assets/mobile-D2pc-iNh.js"></script>
+    <link rel="modulepreload" crossorigin href="/assets/react-vendor-Dn4wqh4Z.js">
+    <link rel="modulepreload" crossorigin href="/assets/icons-Clz0NR6Y.js">
+    <link rel="modulepreload" crossorigin href="/assets/mobile-IaZ47uKC.js">
     <link rel="stylesheet" crossorigin href="/assets/mobile-CsZQAswA.css">
   </head>
   <body>

package/bizar-dash/node_modules/.vite/vitest/da39a3ee5e6b4b0d3255bfef95601890afd80709/results.json CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":"4.1.9","results":[[":tests/lib/utils.test.ts",{"duration":24.~~401755999999978~~,"failed":false}],[":tests/components/Button.test.tsx",{"duration":94.~~65935400000001~~,"failed":false}],[":tests/components/Card.test.tsx",{"duration":33.~~34148499999992~~,"failed":false}],[":tests/hooks/useModal.test.tsx",{"duration":~~122~~.~~76268099999993~~,"failed":false}],[":tests/components/Toast.test.tsx",{"duration":~~154~~.~~32516699999996~~,"failed":false}],[":tests/components/Modal.test.tsx",{"duration":~~154~~.~~48290899999995~~,"failed":false}],[":tests/lib/i18n.test.ts",{"duration":2.~~5646790000000124~~,"failed":false}],[":tests/components/StatusBadge.test.tsx",{"duration":35.~~74290199999996~~,"failed":false}],[":tests/hooks/useToast.test.tsx",{"duration":~~102~~.~~97662300000002~~,"failed":false}],[":tests/components/Spinner.test.tsx",{"duration":61.~~05561799999998~~,"failed":false}],[":tests/backup-restore.test.tsx",{"duration":~~172~~.~~59814500000005~~,"failed":false}],[":tests/a11y.test.tsx",{"duration":~~604~~.~~525834~~,"failed":false}],[":tests/lib/search-fuzzy.test.ts",{"duration":5.~~145660000000021~~,"failed":false}],[":tests/components/settings-search.test.tsx",{"duration":~~706~~.~~8917600000001~~,"failed":false}],[":tests/memory-graph-view.test.tsx",{"duration":47.~~21873400000004~~,"failed":false}],[":tests/components/workspace-selector.test.tsx",{"duration":~~117~~.~~41044699999998~~,"failed":false}],[":tests/components/screenshot-ocr.test.tsx",{"duration":54.~~35712199999989~~,"failed":false}],[":tests/voice-recorder.test.tsx",{"duration":~~120~~.~~104243~~,"failed":false}],[":tests/autosave.test.tsx",{"duration":~~246~~.~~61124700000005~~,"failed":false}],[":tests/settings-layout.test.tsx",{"duration":~~193~~.~~1172640000001~~,"failed":false}],[":tests/settings-nav.test.tsx",{"duration":~~243~~.~~567592~~,"failed":false}],[":tests/components/doctor-panel.test.tsx",{"duration":76.~~46435400000007~~,"failed":false}],[":tests/chat-composer.test.tsx",{"duration":~~167~~.~~8578~~,"failed":false}],[":tests/settings-mode-wiring.test.tsx",{"duration":~~187~~.~~096763~~,"failed":false}],[":tests/minimax-bar.test.ts",{"duration":0,"failed":true}],[":tests/minimax-bar.test.tsx",{"duration":21.~~815285000000017~~,"failed":false}]]}
1	+ {"version":"4.1.9","results":[[":tests/lib/utils.test.ts",{"duration":28.89169000000004,"failed":false}],[":tests/components/Button.test.tsx",{"duration":117.69011,"failed":false}],[":tests/components/Card.test.tsx",{"duration":30.92253400000004,"failed":false}],[":tests/hooks/useModal.test.tsx",{"duration":119.64900699999998,"failed":false}],[":tests/components/Toast.test.tsx",{"duration":159.04681100000005,"failed":false}],[":tests/components/Modal.test.tsx",{"duration":150.9410469999999,"failed":false}],[":tests/lib/i18n.test.ts",{"duration":3.2969249999999874,"failed":false}],[":tests/components/StatusBadge.test.tsx",{"duration":41.157249999999976,"failed":false}],[":tests/hooks/useToast.test.tsx",{"duration":104.85745599999996,"failed":false}],[":tests/components/Spinner.test.tsx",{"duration":70.16560200000004,"failed":false}],[":tests/backup-restore.test.tsx",{"duration":179.13565300000005,"failed":false}],[":tests/a11y.test.tsx",{"duration":715.4879470000001,"failed":false}],[":tests/lib/search-fuzzy.test.ts",{"duration":7.532295999999974,"failed":false}],[":tests/components/settings-search.test.tsx",{"duration":712.275263,"failed":false}],[":tests/memory-graph-view.test.tsx",{"duration":59.40099399999997,"failed":false}],[":tests/components/workspace-selector.test.tsx",{"duration":113.71114899999998,"failed":false}],[":tests/components/screenshot-ocr.test.tsx",{"duration":51.13657699999999,"failed":false}],[":tests/voice-recorder.test.tsx",{"duration":128.61647900000003,"failed":false}],[":tests/autosave.test.tsx",{"duration":251.98326499999996,"failed":false}],[":tests/settings-layout.test.tsx",{"duration":201.76037699999995,"failed":false}],[":tests/settings-nav.test.tsx",{"duration":228.42296299999998,"failed":false}],[":tests/components/doctor-panel.test.tsx",{"duration":67.53857800000003,"failed":false}],[":tests/chat-composer.test.tsx",{"duration":189.9785320000001,"failed":false}],[":tests/settings-mode-wiring.test.tsx",{"duration":205.661208,"failed":false}],[":tests/minimax-bar.test.ts",{"duration":0,"failed":true}],[":tests/minimax-bar.test.tsx",{"duration":30.342043999999987,"failed":false}]]}

package/bizar-dash/src/server/eval.mjs CHANGED Viewed

@@ -34,6 +34,8 @@ import { join, extname } from 'node:path';
  * @property {object} [jsonSchema]
  * @property {number} [maxTokens]
  * @property {number} [maxLatencyMs]
+ * @property {number} [toolCallsMin]
+ * @property {string[]} [toolSequence]
  */
 /**
@@ -133,7 +135,25 @@ function checkJsonSchema(content, schema) {
   // Basic schema validation — check required fields exist
   const errors = [];
   if (schema.type === 'object' && typeof parsed !== 'object' || Array.isArray(parsed)) {
-    errors.push(`expected object, got ${typeof parsed}`);
+    if (schema.type !== 'array') {
+      errors.push(`expected ${schema.type}, got ${typeof parsed}`);
+    }
+  }
+  if (schema.type === 'array') {
+    if (!Array.isArray(parsed)) {
+      errors.push(`expected array, got ${typeof parsed}`);
+    } else {
+      if (schema.minItems != null && parsed.length < schema.minItems) {
+        errors.push(`array length ${parsed.length} is less than minItems ${schema.minItems}`);
+      }
+      if (schema.items?.type) {
+        for (let i = 0; i < parsed.length; i++) {
+          if (typeof parsed[i] !== schema.items.type) {
+            errors.push(`item at index ${i} expected type ${schema.items.type}, got ${typeof parsed[i]}`);
+          }
+        }
+      }
+    }
   }
   if (schema.properties) {
     for (const [key, prop] of Object.entries(schema.properties)) {
@@ -182,18 +202,58 @@ function checkMaxLatency(latencyMs, maxLatencyMs) {
   };
 }
+/**
+ * @param {string[]} toolCalls
+ * @param {number} minCalls
+ * @returns {CheckResult}
+ */
+function checkToolCallsMin(toolCalls, minCalls) {
+  if (minCalls == null) return { kind: 'toolCallsMin', ok: true };
+  const ok = toolCalls.length >= minCalls;
+  return {
+    kind: 'toolCallsMin',
+    ok,
+    message: ok ? undefined : `tool calls ${toolCalls.length} is less than min ${minCalls}`,
+  };
+}
+/**
+ * @param {string[]} toolCalls
+ * @param {string[]} sequence
+ * @returns {CheckResult}
+ */
+function checkToolSequence(toolCalls, sequence) {
+  if (!sequence || !sequence.length) return { kind: 'toolSequence', ok: true };
+  const seqStr = sequence.join(',');
+  const callStr = toolCalls.join(',');
+  // Find the sequence as a subsequence
+  let seqIdx = 0;
+  for (const call of toolCalls) {
+    if (call === sequence[seqIdx]) {
+      seqIdx++;
+      if (seqIdx === sequence.length) break;
+    }
+  }
+  const ok = seqIdx === sequence.length;
+  return {
+    kind: 'toolSequence',
+    ok,
+    message: ok ? undefined : `tool sequence [${seqStr}] not found in [${callStr}]`,
+  };
+}
 // ── Core runner ───────────────────────────────────────────────────────────────
 /**
  * Run a single fixture against an LLM.
  *
  * @param {Fixture} fixture
- * @param {{ llmCall: (prompt: string, opts: {agent: string}) => Promise<{content: string, usage: object}>, timeoutMs?: number }} opts
+ * @param {{ llmCall: (prompt: string, opts: {agent: string}) => Promise<{content: string, usage: object}>, timeoutMs?: number, toolCalls?: string[] }} opts
  * @returns {Promise<EvalResult>}
  */
 export async function runFixture(
   fixture,
-  { llmCall, timeoutMs = 60000 } = {},
+  { llmCall, timeoutMs = 60000, toolCalls = [] } = {},
 ) {
   if (!llmCall) throw new Error('llmCall is required');
   const start = Date.now();
@@ -212,6 +272,10 @@ export async function runFixture(
     clearTimeout(timer);
     content = result.content || '';
     usage = result.usage || { inputTokens: 0, outputTokens: 0, totalTokens: 0 };
+    // Support tool calls returned from llmCall
+    if (result.toolCalls?.length) {
+      toolCalls = result.toolCalls;
+    }
   } catch (err) {
     if (err.message === 'timeout') {
       timeoutHit = true;
@@ -255,6 +319,12 @@ export async function runFixture(
   if (expected.maxLatencyMs != null) {
     checks.push(checkMaxLatency(latencyMs, expected.maxLatencyMs));
   }
+  if (expected.toolCallsMin != null) {
+    checks.push(checkToolCallsMin(toolCalls, expected.toolCallsMin));
+  }
+  if (expected.toolSequence?.length) {
+    checks.push(checkToolSequence(toolCalls, expected.toolSequence));
+  }
   const ok = checks.every((c) => c.ok);
@@ -268,6 +338,26 @@ export async function runFixture(
   };
 }
+/**
+ * Run a single fixture by ID from a known fixture map (useful for testing).
+ *
+ * @param {string} fixtureId
+ * @param {Fixture[]} fixtures
+ * @param {{ llmCall: Function, timeoutMs?: number, toolCalls?: string[] }} opts
+ * @returns {Promise<EvalResult>}
+ */
+export async function runFixtureById(
+  fixtureId,
+  fixtures,
+  { llmCall, timeoutMs = 60000, toolCalls = [] } = {},
+) {
+  const fixture = fixtures.find((f) => f.id === fixtureId);
+  if (!fixture) {
+    throw new Error(`Fixture not found: ${fixtureId}`);
+  }
+  return runFixture(fixture, { llmCall, timeoutMs, toolCalls });
+}
 /**
  * Load all JSON fixtures from a directory.
  *

package/bizar-dash/src/server/metrics.mjs CHANGED Viewed

@@ -190,4 +190,79 @@ export function reset() {
   counters.clear();
   gauges.clear();
   histograms.clear();
+}
+// ─── v5.1.0 ────────────────────────────────────────────────────────────────
+// Trace → metrics correlation helpers.
+//
+// The OTEL exporter ships spans to a collector; the Prometheus
+// exporter ships counters from this in-process registry. To answer
+// "how many `chat.send` spans landed in the last hour?" without
+// leaving the dashboard, route handlers call `recordTrace(name,
+// attributes)` from inside their `withSpan` wrapper. The Map keeps a
+// bounded per-(name, attribute-set) count so a single high-cardinality
+// label (e.g. `chat.session_id`) does not blow up memory.
+//
+// `traceCountByName` is intentionally exported read-only — tests
+// import it to assert the helper was called with the right key, but
+// production code should always go through `recordTrace` to keep
+// the key derivation in one place.
+//
+//─── v5.1.0 ends ───────────────────────────────────────────────────────────
+/**
+ * Bounded counter of recorded trace samples, keyed by
+ * `${name}:${JSON.stringify(attributes)}`. Routes populate this
+ * from inside their `withSpan` callback so the dashboard can answer
+ * "how often did span X fire with attribute set Y" without leaving
+ * the process.
+ *
+ * The Map is intentionally an exported `let`-equivalent: tests
+ * reset it via {@link resetTraceCounts}; production code never
+ * mutates the Map directly.
+ *
+ * @type {Map<string, number>}
+ */
+export const traceCountByName = new Map();
+/**
+ * Increment the trace counter for `(name, attributes)`.
+ *
+ * The composite key is `${name}:${jsonString}` so two spans named
+ * `chat.send` with different `chat.session_id` attributes are
+ * tracked separately. Undefined/null/empty-string attribute values
+ * are dropped from the key for stability (avoids the empty-string vs
+ * undefined distinction polluting counts). Keys are alphabetised
+ * before serialisation so `{a:1,b:2}` and `{b:2,a:1}` collide on
+ * the same bucket — `JSON.stringify` alone would not give that
+ * guarantee because it preserves insertion order.
+ *
+ * @param {string} name
+ * @param {Record<string, string|number|boolean|undefined|null>} [attributes]
+ * @returns {number} the new total for that key
+ */
+export function recordTrace(name, attributes = {}) {
+  if (!name) return 0;
+  const keys = [];
+  if (attributes && typeof attributes === 'object') {
+    for (const k of Object.keys(attributes)) {
+      const v = attributes[k];
+      if (v === undefined || v === null || v === '') continue;
+      keys.push(k);
+    }
+  }
+  keys.sort();
+  const cleanAttrs = {};
+  for (const k of keys) cleanAttrs[k] = attributes[k];
+  const key = `${name}:${JSON.stringify(cleanAttrs)}`;
+  const next = (traceCountByName.get(key) || 0) + 1;
+  traceCountByName.set(key, next);
+  return next;
+}
+/**
+ * Clear every recorded-trace counter. Intended for tests.
+ */
+export function resetTraceCounts() {
+  traceCountByName.clear();
 }

package/bizar-dash/src/server/otel.mjs CHANGED Viewed

@@ -1,15 +1,24 @@
 /**
  * src/server/otel.mjs
  *
- * v4.9.0 — OpenTelemetry distributed tracing for the Bizar dashboard.
+ * v5.1.0 — Expanded OpenTelemetry tracing surface for the Bizar dashboard.
  *
- * Initialises the NodeSDK with an OTLP HTTP trace exporter and exposes
- * a single `tracer` instance for ad-hoc spans (chat.send, opencode
- * session creation, etc). Auto-instrumentation is intentionally not
- * enabled — the dashboard already has structured logger + Prometheus
- * for the "what" metrics, so distributed tracing is only interesting
- * for the multi-step flows where request correlation matters
- * (chat SSE pump, opencode session creation).
+ * Initialises the NodeSDK with an OTLP HTTP trace exporter and exposes:
+ *   - `tracer`                        — the shared tracer instance
+ *   - `initOtel()` / `shutdownOtel()` — SDK lifecycle (off by default)
+ *   - `withSpan(name, fn, attrs)`     — ergonomic wrapper that records
+ *                                       OK on success, ERROR+exception
+ *                                       on throw, and ends the span in
+ *                                       every code path
+ *   - `setCommonAttributes(span, …)`  — fast-path for the recurring
+ *                                       user/workspace/ip/ua attributes
+ *                                       that 95% of route spans set
+ *
+ * Auto-instrumentation is intentionally NOT enabled. The dashboard
+ * already has structured logger + Prometheus for the "what" metrics, so
+ * distributed tracing is only interesting for the multi-step flows
+ * where request correlation matters (chat SSE pump, opencode session
+ * creation, plugin install, workspace invites, etc).
  *
  * Off by default. Enable with one of:
  *   BIZAR_OTEL=1
@@ -29,11 +38,13 @@ import { NodeSDK } from '@opentelemetry/sdk-node';
 import { OTLPTraceExporter } from '@opentelemetry/exporter-trace-otlp-http';
 import { Resource } from '@opentelemetry/resources';
 import { SemanticResourceAttributes } from '@opentelemetry/semantic-conventions';
-import { trace } from '@opentelemetry/api';
+import { SpanStatusCode, trace } from '@opentelemetry/api';
+import os from 'node:os';
 import { info as logInfo, warn as logWarn } from './logger.mjs';
 const DEFAULT_OTLP_ENDPOINT = 'http://localhost:4318/v1/traces';
 const SERVICE_NAME = 'bizar-dash';
+const SERVICE_NAMESPACE = 'bizar';
 const TRACER_NAME = 'bizar-dash';
 const TRACER_VERSION = '0.1.0';
@@ -49,6 +60,29 @@ let shuttingDown = false;
  */
 export const tracer = trace.getTracer(TRACER_NAME, TRACER_VERSION);
+/**
+ * Build the OTel resource attributes for this dashboard process.
+ *
+ * Used both at SDK init time (so a collector sees deployment
+ * context) and exposed via `getResourceAttributes()` for any
+ * consumer that needs the same metadata on a per-span basis.
+ *
+ * @returns {Record<string, string|number>}
+ */
+export function getResourceAttributes() {
+  return {
+    [SemanticResourceAttributes.SERVICE_NAME]: SERVICE_NAME,
+    [SemanticResourceAttributes.SERVICE_VERSION]:
+      process.env.npm_package_version || '0.0.0',
+    [SemanticResourceAttributes.SERVICE_NAMESPACE]: SERVICE_NAMESPACE,
+    [SemanticResourceAttributes.DEPLOYMENT_ENVIRONMENT]:
+      process.env.NODE_ENV || 'development',
+    [SemanticResourceAttributes.PROCESS_PID]: process.pid,
+    [SemanticResourceAttributes.HOST_NAME]: os.hostname(),
+    [SemanticResourceAttributes.OS_VERSION]: `${os.platform()} ${os.release()}`,
+  };
+}
 /**
  * Initialise the NodeSDK with an OTLP HTTP trace exporter. Safe to
  * call multiple times — the second call returns the cached SDK.
@@ -70,12 +104,10 @@ export function initOtel({
   if (sdk) return sdk;
   if (shuttingDown) return null;
   try {
+    const attrs = getResourceAttributes();
+    if (serviceName !== SERVICE_NAME) attrs[SemanticResourceAttributes.SERVICE_NAME] = serviceName;
     sdk = new NodeSDK({
-      resource: new Resource({
-        [SemanticResourceAttributes.SERVICE_NAME]: serviceName,
-        [SemanticResourceAttributes.SERVICE_VERSION]:
-          process.env.npm_package_version || '0.0.0',
-      }),
+      resource: new Resource(attrs),
       traceExporter: new OTLPTraceExporter({ url: endpoint }),
     });
     sdk.start();
@@ -83,6 +115,8 @@ export function initOtel({
       module: 'otel',
       serviceName,
       endpoint,
+      namespace: SERVICE_NAMESPACE,
+      deployment: process.env.NODE_ENV || 'development',
     });
     return sdk;
   } catch (err) {
@@ -131,3 +165,96 @@ export async function shutdownOtel() {
 export function isOtelEnabled() {
   return sdk !== null;
 }
+/**
+ * Run `fn` inside an active span on the shared tracer.
+ *
+ * The helper:
+ *   - opens a span with the given name and pre-bound attributes;
+ *   - awaits the user callback (which receives the span as its
+ *     last argument, after `(req, res)` in route handlers, or as
+ *     the only argument in free-standing helpers);
+ *   - sets the span status to OK on success;
+ *   - on throw: records the exception and sets ERROR status before
+ *     rethrowing so the original error propagates to `wrap()` /
+ *     `next(err)` exactly as if the helper were not used;
+ *   - ends the span in a finally block so a thrown error from the
+ *     callback cannot leak the span.
+ *
+ * `withSpan` works under both the live SDK and the no-op tracer
+ * shipped when OTEL is disabled — every span method is a no-op then
+ * and the callback's return value is what callers see.
+ *
+ * @template T
+ * @param {string} name
+ * @param {(span: import('@opentelemetry/api').Span, ...args: any[]) => Promise<T>|T} fn
+ * @param {Record<string, string|number|boolean>} [attributes]
+ * @returns {(...args: any[]) => Promise<T>}
+ */
+export function withSpan(name, fn, attributes = {}) {
+  return (...args) => tracer.startActiveSpan(name, { attributes }, async (span) => {
+    try {
+      const result = await fn(span, ...args);
+      span.setStatus({ code: SpanStatusCode.OK });
+      return result;
+    } catch (err) {
+      try {
+        span.recordException(err);
+      } catch {
+        /* no-op tracer swallows everything */
+      }
+      try {
+        span.setStatus({
+          code: SpanStatusCode.ERROR,
+          message: err?.message || String(err),
+        });
+      } catch {
+        /* ignore */
+      }
+      throw err;
+    } finally {
+      try {
+        span.end();
+      } catch {
+        /* never re-throw inside OTel helpers */
+      }
+    }
+  });
+}
+/**
+ * Apply the common request-scoped attributes most route spans want:
+ * user/workspace identity, plus the OTel HTTP semantic conventions
+ * for the client IP and user-agent string.
+ *
+ * Each attribute is set independently and only if a value was
+ * provided; passing `undefined`/`null`/empty string skips the
+ * attribute so spans for public endpoints don't get an empty
+ * `bizar.user.id = ""` cluttering the trace UI.
+ *
+ * Attribute naming rationale:
+ *   - `bizar.user.id`       — namespaced custom key (avoids colliding
+ *                             with any future OTel `enduser.id`)
+ *   - `bizar.workspace.id`  — namespaced custom key
+ *   - `http.client_ip`      — OTel HTTP semantic convention
+ *   - `http.user_agent`     — OTel HTTP semantic convention (the
+ *                             spec later renamed this to
+ *                             `user_agent.original`; the dashboard
+ *                             keeps `http.user_agent` because the
+ *                             v4.x dashboards in production still
+ *                             search by that key)
+ *
+ * @param {import('@opentelemetry/api').Span} span
+ * @param {object} opts
+ * @param {string} [opts.userId]
+ * @param {string} [opts.workspaceId]
+ * @param {string} [opts.ip]
+ * @param {string} [opts.userAgent]
+ */
+export function setCommonAttributes(span, { userId, workspaceId, ip, userAgent } = {}) {
+  if (!span) return;
+  if (userId) span.setAttribute('bizar.user.id', userId);
+  if (workspaceId) span.setAttribute('bizar.workspace.id', workspaceId);
+  if (ip) span.setAttribute('http.client_ip', ip);
+  if (userAgent) span.setAttribute('http.user_agent', userAgent);
+}