@lorekit/cli 1.23.0 → 1.24.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lorekit/cli",
-  "version": "1.23.0",
+  "version": "1.24.1",
   "description": "Install the LoreKit shared-memory skill and run health checks for the LoreKit MCP server.",
   "license": "MIT",
   "repository": {

package/src/telemetry.mjs

@@ -32,7 +32,9 @@ import { readLorekitJson } from './config.mjs';
 // (empty in the source tree, so default export stays off until built/injected).
 const DEFAULT_ENDPOINT = 'https://ingress.europe-west4.gcp.dash0-dev.com';
 const DEFAULT_TOKEN = TELEMETRY_TOKEN; // injected from LOREKIT_TELEMETRY_TOKEN at publish
-const DEFAULT_DATASET = '';
+// Ship to the `default` Dash0 dataset unless `DASH0_DATASET` overrides it, so
+// CLI telemetry lands alongside every other LoreKit component (all `default`).
+const DEFAULT_DATASET = 'default';
 
 // Flags worth counting (e.g. how many installs are --global). Bounded on
 // purpose: only these booleans are ever attached, never free-form values.
@@ -125,8 +127,17 @@ export function resolveTelemetryConfig(env = process.env, repoConfig) {
     return { enabled: false };
   }
 
-  const dataset = env.DASH0_DATASET || DEFAULT_DATASET;
-  if (dataset) headers['Dash0-Dataset'] = dataset;
+  // Dataset routing, highest precedence first: an explicit `Dash0-Dataset`
+  // already parsed from OTEL_EXPORTER_OTLP_HEADERS wins and is never clobbered;
+  // otherwise DASH0_DATASET; otherwise the baked-in DEFAULT_DATASET (`default`).
+  // HTTP header names are case-insensitive, so match any casing the caller used
+  // (e.g. `dash0-dataset`) rather than only the canonical spelling — otherwise a
+  // second, conflicting header would be appended.
+  const hasDataset = Object.keys(headers).some((k) => k.toLowerCase() === 'dash0-dataset');
+  if (!hasDataset) {
+    const dataset = env.DASH0_DATASET || DEFAULT_DATASET;
+    if (dataset) headers['Dash0-Dataset'] = dataset;
+  }
 
   return { enabled: true, endpoint, headers };
 }
@@ -156,7 +167,7 @@ function toOtlpAttributes(attributes) {
 
 function resourceAttributes(version) {
   return [
-    { key: 'service.name', value: { stringValue: 'lorekit-cli' } },
+    { key: 'service.name', value: { stringValue: 'cli' } },
     { key: 'service.namespace', value: { stringValue: 'lorekit' } },
     { key: 'service.version', value: { stringValue: String(version) } },
     { key: 'process.runtime.name', value: { stringValue: 'nodejs' } },
@@ -189,7 +200,7 @@ export function buildTracePayload({ version, name, attributes, startMs, endMs, s
         resource: { attributes: resourceAttributes(version) },
         scopeSpans: [
           {
-            scope: { name: 'lorekit-cli', version: String(version) },
+            scope: { name: 'cli', version: String(version) },
             spans: [
               {
                 traceId: traceId ?? randHex(16),
@@ -219,7 +230,7 @@ export function buildMetricsPayload({ version, attributes, startMs, endMs }) {
         resource: { attributes: resourceAttributes(version) },
         scopeMetrics: [
           {
-            scope: { name: 'lorekit-cli', version: String(version) },
+            scope: { name: 'cli', version: String(version) },
             metrics: [
               {
                 name: 'lorekit.cli.invocations',