autotel-posthog 4.0.0 → 6.0.0

package/README.md

@@ -83,11 +83,13 @@ initFull({
 | `session.id`         | `posthog.get_session_id()`                                | every span                    |
 | `user.id`            | `posthog.get_distinct_id()`                               | every span                    |
 | `session.replay.url` | `posthog.get_session_replay_url({ withTimestamp: true })` | spans that recorded an error  |
-| `feature_flag.<key>` | `posthog.getFeatureFlag(key)`                             | every span, for keys you name |
+| `feature_flag.*`     | `posthog.getFeatureFlag(key)`                             | every span, for keys you name |
 
 Identity is read when the span **starts**, not when it ends. PostHog rotates a session after 30 minutes idle and `identify()` can land mid-request; a long span asking at the end would be filed under whoever the visitor had become by then. The replay link is the one thing decided at the end, because only the end knows whether the span failed — and it is withheld if the session rotated in between, since the link would point at a recording the span has nothing to do with.
 
-A flag that evaluates to `false` is kept as a boolean: "not in the variant" is an answer, and comparing the two groups is the whole point. Only a flag PostHog has no opinion on is omitted.
+Flags land under the **canonical** OpenTelemetry convention — `feature_flag.key`, `feature_flag.result.value`, `feature_flag.result.variant`, `feature_flag.provider.name`, `feature_flag.context.id` — plus one `feature_flag.evaluation` event per flag, because span attributes hold a single flag and a second call would overwrite the first. That is what makes "error rate by variant" a query your backend already knows how to answer; an attribute keyed by flag name could not be grouped across flags, and nothing reads it.
+
+A flag that evaluates to `false` is kept: "not in the variant" is an answer, and comparing the two groups is the whole point. Only a flag PostHog has no opinion on is omitted — recording that as `false` would make "off" and "unknown" the same reading.
 
 Use `spanEnrichers`, not `spanProcessor`. The latter _replaces_ the pipeline autotel builds, so passing an enricher there switches off the export you just configured.
 

package/dist/index.cjs

@@ -1,4 +1,6 @@
 Object.defineProperty(exports, Symbol.toStringTag, { value: "Module" });
+let autotel_web = require("autotel-web");
+let autotel_feature_flags = require("autotel/feature-flags");
 let autotel_web_baggage = require("autotel-web/baggage");
 let _opentelemetry_api = require("@opentelemetry/api");
 //#region src/dev-mode.ts
@@ -155,8 +157,21 @@ function posthogCompatibility(options = {}) {
 			const fill = (key, value) => {
 				if (value !== void 0 && attributes[key] === void 0) span.setAttribute(key, value);
 			};
-			fill("user.id", readDistinctId(posthog));
-			for (const key of options.featureFlags ?? []) fill(`feature_flag.${key}`, readFeatureFlag(posthog, key));
+			const distinctId = readDistinctId(posthog);
+			fill("user.id", distinctId);
+			for (const key of options.featureFlags ?? []) {
+				const value = readFeatureFlag(posthog, key);
+				if (value === void 0) continue;
+				(0, autotel_feature_flags.recordFeatureFlag)({
+					setAttributes: (attributes) => span.setAttributes(attributes),
+					track: autotel_web.emitEvent
+				}, {
+					key,
+					value,
+					provider: "posthog",
+					contextId: distinctId
+				});
+			}
 		},
 		/**
 		* Only the replay link is left for the end, because only the end knows

package/dist/index.d.cts

@@ -37,8 +37,10 @@ interface PostHogCompatibilityOptions {
    */
   posthog?: PostHogLike | (() => PostHogLike | undefined);
   /**
-   * Flag keys to stamp on every span as `feature_flag.<key>`, so error rate and
-   * latency can be split by variant in whichever backend receives the spans.
+   * Flag keys to record on every span, using the canonical OpenTelemetry
+   * `feature_flag.*` attributes and one `feature_flag.evaluation` event each —
+   * so error rate and latency can be split by variant in whichever backend
+   * receives the spans, with no PostHog in the query path.
    *
    * Named explicitly rather than read wholesale: every flag is another
    * attribute on every span, and "all of them" is how an analytics convenience

package/dist/index.d.ts

@@ -37,8 +37,10 @@ interface PostHogCompatibilityOptions {
    */
   posthog?: PostHogLike | (() => PostHogLike | undefined);
   /**
-   * Flag keys to stamp on every span as `feature_flag.<key>`, so error rate and
-   * latency can be split by variant in whichever backend receives the spans.
+   * Flag keys to record on every span, using the canonical OpenTelemetry
+   * `feature_flag.*` attributes and one `feature_flag.evaluation` event each —
+   * so error rate and latency can be split by variant in whichever backend
+   * receives the spans, with no PostHog in the query path.
    *
    * Named explicitly rather than read wholesale: every flag is another
    * attribute on every span, and "all of them" is how an analytics convenience

package/dist/index.js

@@ -1,3 +1,5 @@
+import { emitEvent } from "autotel-web";
+import { recordFeatureFlag } from "autotel/feature-flags";
 import { setBaggage } from "autotel-web/baggage";
 import { context, trace } from "@opentelemetry/api";
 //#region src/dev-mode.ts
@@ -154,8 +156,21 @@ function posthogCompatibility(options = {}) {
 			const fill = (key, value) => {
 				if (value !== void 0 && attributes[key] === void 0) span.setAttribute(key, value);
 			};
-			fill("user.id", readDistinctId(posthog));
-			for (const key of options.featureFlags ?? []) fill(`feature_flag.${key}`, readFeatureFlag(posthog, key));
+			const distinctId = readDistinctId(posthog);
+			fill("user.id", distinctId);
+			for (const key of options.featureFlags ?? []) {
+				const value = readFeatureFlag(posthog, key);
+				if (value === void 0) continue;
+				recordFeatureFlag({
+					setAttributes: (attributes) => span.setAttributes(attributes),
+					track: emitEvent
+				}, {
+					key,
+					value,
+					provider: "posthog",
+					contextId: distinctId
+				});
+			}
 		},
 		/**
 		* Only the replay link is left for the end, because only the end knows

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "autotel-posthog",
-  "version": "4.0.0",
+  "version": "6.0.0",
   "description": "Everything PostHog for autotel: browser session/replay join, server-side event subscriber, and trace links in both directions",
   "type": "module",
   "sideEffects": false,
@@ -30,9 +30,9 @@
     "@opentelemetry/sdk-trace-base": ">=2.0.0",
     "posthog-js": ">=1.200.0",
     "posthog-node": ">=4.0.0",
-    "autotel": "7.3.0",
-    "autotel-subscribers": "53.0.0",
-    "autotel-web": "1.13.3"
+    "autotel": "7.5.0",
+    "autotel-subscribers": "55.0.0",
+    "autotel-web": "1.14.1"
   },
   "devDependencies": {
     "@opentelemetry/context-async-hooks": "^2.10.0",
@@ -46,9 +46,9 @@
     "tsdown": "^0.22.14",
     "typescript": "^6.0.3",
     "vitest": "^4.1.10",
-    "autotel": "7.3.0",
-    "autotel-subscribers": "53.0.0",
-    "autotel-web": "1.13.3"
+    "autotel": "7.5.0",
+    "autotel-subscribers": "55.0.0",
+    "autotel-web": "1.14.1"
   },
   "keywords": [
     "opentelemetry",