@wireai/activation 0.8.0 → 0.9.1

package/dist/transport-Bzb-bcB2.d.mts

@@ -0,0 +1,128 @@
+import { k as ReviewTarget, g as ReviewDecisionResponse, j as ReviewSubmission } from './types-CNUqMK0D.mjs';
+
+/**
+ * transport.ts — kit → Wire server requests for the review module, all fire-and-forget
+ * (analytics/reviews must never break the app). Mirrors analytics/reportClientEvent: a
+ * thin fetch wrapper, Bearer tenant key, swallow every error.
+ *
+ *   • submitReview         → POST {serverUrl}/v1/reviews          (the review row)
+ *   • fetchReviewDecision  → GET  {serverUrl}/v1/reviews/decision (best-effort, the AI seam)
+ *   • reportAppEvent       → POST {serverUrl}/v1/events           (generic app.* namespace)
+ *
+ * `reportAppEvent` is the strategic extension: it lets a host report arbitrary in-app
+ * events through the SAME transport (stored server-side as event_type='app_event',
+ * question_key=<name>), which is what the backend review-firing rules evaluate on — and
+ * it seeds the broader app-analytics stream. Keep payloads minimal + non-PII.
+ */
+
+/**
+ * POST a review (the 1-4 feedback path). Fire-and-forget: a missing target, a build error,
+ * a missing `fetch`, or a network failure is swallowed and the call returns immediately.
+ */
+declare const submitReview: (target: ReviewTarget | undefined, review: ReviewSubmission) => void;
+/**
+ * Options for the best-effort review decision fetch.
+ *
+ * ── WHAT THE DEPLOYED SERVER ACTUALLY READS (verified 2026-07-17) ────────────────────────
+ *
+ * `GET /v1/reviews/decision` declares exactly two query params — `session_id` and
+ * `device_key` — plus the `Authorization` header. That is the whole wire. Verified against
+ * the deployed OpenAPI schema, not against intent.
+ *
+ * This type is CLOSED on purpose. Hosts that hand-rolled this fetch invented `user_id` and
+ * `session_count` query params believing "the server ignores what it doesn't read, so passing
+ * it is always safe". Both are no-ops: the route declares neither. `session_count` is real, but
+ * only on the questionnaire POST body — which is precisely how it copy-pasted its way into a
+ * reviews call site and sat there doing nothing. They cost a wire lie — a call site that reads as though
+ * identity and a session counter reach the firing brain when neither does. So they are not
+ * offered here. Pass identity as `deviceKey`; the session count the server reasons about is the
+ * one IT derives from the event stream keyed by `deviceKey`, not one the client asserts.
+ *
+ * The client-side session floor is a LOCAL rule, not a wire param: use `ReviewConfig.minSessions`
+ * (evaluated by `useReviewGate` against the kit's own `wire_review_<id>_sessions` counter).
+ */
+interface FetchReviewDecisionOptions {
+    /**
+     * The onboarding session id, when there IS one. OPTIONAL on purpose: the review gate lives on
+     * the home feed, where a user legitimately has no onboarding session. `deviceKey` is the real
+     * identity for this call. (The server relaxed `session_id` to optional on 2026-07-16 and the
+     * relaxation IS deployed — the route no longer 422s without it. Permissive wire: do NOT make
+     * this required here.)
+     */
+    sessionId?: string;
+    /**
+     * A stable, non-PII device id — THE identity for this call. The decision endpoint reads it for
+     * cooldown + min-sessions, and it is the key the server groups a device's events under. A host
+     * whose own identity is a user id passes that id here rather than reaching for a `user_id`
+     * param the route does not declare.
+     */
+    deviceKey?: string;
+}
+/**
+ * Best-effort fetch of the SERVER's review firing decision. The mirror of
+ * `fetchQuestionnaireDecision`, and the primitive whose absence caused a live incident.
+ *
+ * ── WHY THIS LIVES IN THE KIT (Malik, 2026-07-16) ────────────────────────────────────────
+ *
+ * The questionnaire module has always had its decision fetch; reviews never did. So a host
+ * hand-rolled one, and got it subtly wrong: its helper collapsed an explicit `{fire:false}`
+ * into `undefined`. `decideReview` is `(local, decision) => decision ?? local`, so `undefined`
+ * means "the server has no opinion, use the local rules" — and the local timer fired. The
+ * server could therefore only ever turn review prompts ON, never OFF. A real user was asked to
+ * rate the app ~3 minutes into their FIRST session, having seen nothing yet, and left 1 star.
+ *
+ * The bug was not that the host was careless. It was that the kit made every host invent this.
+ * So the primitive moves here and hosts keep a thin call site.
+ *
+ * ── THE CONTRACT THAT MATTERS ────────────────────────────────────────────────────────────
+ *
+ *   • 2xx  → the FULL `{fire, reason, arm}`, INCLUDING `fire:false`. Never collapse a false
+ *            into null. That collapse IS the bug: a false must reach `decideReview` intact so
+ *            it can override the local rules and keep the gate shut.
+ *   • else → null, and ONLY then. Null means "the server genuinely has no opinion", which is
+ *            the one case where falling back to local rules is correct.
+ *
+ * Never throws: unreachable, non-2xx, bad JSON, or a missing `fetch` all resolve to null. The
+ * kit does not call this internally; a host awaits it and passes the result straight to
+ * `useReviewGate({ decision })`.
+ *
+ * ── DON'T RACE THIS AGAINST A LOCAL TIMER ────────────────────────────────────────────────
+ *
+ * The return type is `ReviewDecisionResponse`, which carries `arm` alongside `{fire, reason}`.
+ * Hand the WHOLE object to the gate and echo `arm` into the submission's `meta.firing_arm`;
+ * narrowing it to `{fire, reason}` on the way through silently kills per-arm attribution
+ * across a reweighting of the experiment.
+ *
+ * A host that starts its own dwell timer in parallel with this fetch has built a race a slow
+ * server loses: the timer fires, the local rules show the prompt, and the `{fire:false}` still
+ * in flight arrives too late to stop it. Do not hand-roll that. `useReviewGate` already owns
+ * the wait — set `ReviewConfig.timeoutFallbackMs` and the local rules stay parked until either
+ * the decision lands or the window expires, whichever comes first.
+ *
+ *   const decision = await fetchReviewDecision(target, { deviceKey });
+ *   const gate = useReviewGate({
+ *     config: { id: "home", minSessions: 2, timeoutFallbackMs: 3000 },
+ *     decision: decision ?? undefined, // pass it whole — keep `arm`
+ *     storage,
+ *   });
+ */
+declare const fetchReviewDecision: (target: ReviewTarget | undefined, options?: FetchReviewDecisionOptions) => Promise<ReviewDecisionResponse | null>;
+/** Options for a reported app event. `deviceKey` groups a device's sessions server-side. */
+interface ReportAppEventOptions {
+    /** The onboarding/session id to correlate with, when known. */
+    sessionId?: string;
+    /** A stable, non-PII device id — the review-decision endpoint reads it for min-sessions. */
+    deviceKey?: string;
+    /** Small non-PII extras. */
+    meta?: Record<string, unknown>;
+}
+/**
+ * Report a generic in-app event through the existing events transport. Stored server-side
+ * as `event_type='app_event'`, `question_key=<name>`. Fire-and-forget. Keep `name` a short
+ * stable identifier and `meta` small + non-PII.
+ *
+ *   reportAppEvent(target, "content_share", { sessionId, deviceKey });
+ */
+declare const reportAppEvent: (target: ReviewTarget | undefined, name: string, options?: ReportAppEventOptions) => void;
+
+export { type FetchReviewDecisionOptions as F, type ReportAppEventOptions as R, fetchReviewDecision as f, reportAppEvent as r, submitReview as s };

package/dist/{types-GL_hQ0TN.d.mts → types-BcmagF6K.d.mts}

@@ -1,7 +1,7 @@
 /**
  * Shared types for the coachmark / guided-tour overlay engine.
  *
- * Lifted from the Myelino app engine and made kit-generic: the app supplies the
+ * Lifted from a host app engine and made kit-generic: the app supplies the
  * anchors + catalog + copy, the kit owns the animation/measure/queue mechanics.
  * Dependency-free on purpose so the type surface can be imported anywhere.
  */

package/dist/{types-GL_hQ0TN.d.ts → types-BcmagF6K.d.ts}

@@ -1,7 +1,7 @@
 /**
  * Shared types for the coachmark / guided-tour overlay engine.
  *
- * Lifted from the Myelino app engine and made kit-generic: the app supplies the
+ * Lifted from a host app engine and made kit-generic: the app supplies the
  * anchors + catalog + copy, the kit owns the animation/measure/queue mechanics.
  * Dependency-free on purpose so the type surface can be imported anywhere.
  */

package/dist/{types-BhpXJGlg.d.ts → types-Buj9Lw9t.d.ts}

@@ -1,6 +1,6 @@
 import { O as OnboardingTheme } from './types-BKfpdZzX.js';
-import { C as CoachmarkStorage } from './types-GL_hQ0TN.js';
-import { W as WireFeatures, a as WireFeaturesConfig } from './types-CMuOexw0.js';
+import { C as CoachmarkStorage } from './types-BcmagF6K.js';
+import { W as WireFeatures, a as WireFeaturesConfig } from './types-CKFhyrMu.js';
 
 /**
  * The firing verdict — the SAME contract on both sides of the seam. `decideReview`
@@ -11,6 +11,18 @@ interface ReviewDecision {
     fire: boolean;
     reason?: string;
 }
+/**
+ * The `GET /v1/reviews/decision` response: `ReviewDecision` plus the additive `arm`.
+ *
+ * `arm` is the firing experiment's sticky assignment for this device/session. Echo it back in
+ * the submission's `meta.firing_arm` and the server keeps it verbatim (`_stamp_firing_arm`:
+ * "an arm the kit already echoed is left as-is"); omit it and the server recomputes from the
+ * sticky hash. Echoing is what makes per-arm attribution survive a later reweighting of the
+ * experiment, so a host running one should pass it through.
+ */
+interface ReviewDecisionResponse extends ReviewDecision {
+    arm?: string | null;
+}
 /** Where to route a rating: 5 stars → the store, 1-4 → the feedback form. */
 type RatingRoute = "store" | "feedback";
 /**
@@ -149,4 +161,4 @@ interface ReviewGateController {
     markResolved: () => void;
 }
 
-export type { ReviewGateProps as R, UseReviewGateOptions as U, ReviewPresentation as a, ReviewGateController as b, ReviewDecision as c, ReviewStoreConfig as d, RatingRoute as e, ReviewConfig as f, ReviewGateEvent as g, ReviewGateEventName as h, ReviewSubmission as i, ReviewTarget as j };
+export type { ReviewGateProps as R, UseReviewGateOptions as U, ReviewPresentation as a, ReviewGateController as b, ReviewDecision as c, ReviewStoreConfig as d, RatingRoute as e, ReviewConfig as f, ReviewDecisionResponse as g, ReviewGateEvent as h, ReviewGateEventName as i, ReviewSubmission as j, ReviewTarget as k };

package/dist/{types-CMuOexw0.d.mts → types-CKFhyrMu.d.mts}

@@ -13,7 +13,7 @@ type WireOnboardingStorage = {
  * this can't be resumed server-side anyway, so the client mints fresh past it.
  */
 declare const DEFAULT_SESSION_TTL_MS = 3600000;
-/** Storage key for an app's cached session, e.g. `wireai:session:myelino`. */
+/** Storage key for an app's cached session, e.g. `wireai:session:acme`. */
 declare const sessionStorageKey: (appId: string) => string;
 /** Resolved session for a mount: the id to use + whether it was restored from storage. */
 type LoadedSession = {

package/dist/{types-CMuOexw0.d.ts → types-CKFhyrMu.d.ts}

@@ -13,7 +13,7 @@ type WireOnboardingStorage = {
  * this can't be resumed server-side anyway, so the client mints fresh past it.
  */
 declare const DEFAULT_SESSION_TTL_MS = 3600000;
-/** Storage key for an app's cached session, e.g. `wireai:session:myelino`. */
+/** Storage key for an app's cached session, e.g. `wireai:session:acme`. */
 declare const sessionStorageKey: (appId: string) => string;
 /** Resolved session for a mount: the id to use + whether it was restored from storage. */
 type LoadedSession = {

package/dist/{types-A6pTxIZV.d.mts → types-CNUqMK0D.d.mts}

@@ -1,6 +1,6 @@
 import { O as OnboardingTheme } from './types-BKfpdZzX.mjs';
-import { C as CoachmarkStorage } from './types-GL_hQ0TN.mjs';
-import { W as WireFeatures, a as WireFeaturesConfig } from './types-CMuOexw0.mjs';
+import { C as CoachmarkStorage } from './types-BcmagF6K.mjs';
+import { W as WireFeatures, a as WireFeaturesConfig } from './types-CKFhyrMu.mjs';
 
 /**
  * The firing verdict — the SAME contract on both sides of the seam. `decideReview`
@@ -11,6 +11,18 @@ interface ReviewDecision {
     fire: boolean;
     reason?: string;
 }
+/**
+ * The `GET /v1/reviews/decision` response: `ReviewDecision` plus the additive `arm`.
+ *
+ * `arm` is the firing experiment's sticky assignment for this device/session. Echo it back in
+ * the submission's `meta.firing_arm` and the server keeps it verbatim (`_stamp_firing_arm`:
+ * "an arm the kit already echoed is left as-is"); omit it and the server recomputes from the
+ * sticky hash. Echoing is what makes per-arm attribution survive a later reweighting of the
+ * experiment, so a host running one should pass it through.
+ */
+interface ReviewDecisionResponse extends ReviewDecision {
+    arm?: string | null;
+}
 /** Where to route a rating: 5 stars → the store, 1-4 → the feedback form. */
 type RatingRoute = "store" | "feedback";
 /**
@@ -149,4 +161,4 @@ interface ReviewGateController {
     markResolved: () => void;
 }
 
-export type { ReviewGateProps as R, UseReviewGateOptions as U, ReviewPresentation as a, ReviewGateController as b, ReviewDecision as c, ReviewStoreConfig as d, RatingRoute as e, ReviewConfig as f, ReviewGateEvent as g, ReviewGateEventName as h, ReviewSubmission as i, ReviewTarget as j };
+export type { ReviewGateProps as R, UseReviewGateOptions as U, ReviewPresentation as a, ReviewGateController as b, ReviewDecision as c, ReviewStoreConfig as d, RatingRoute as e, ReviewConfig as f, ReviewDecisionResponse as g, ReviewGateEvent as h, ReviewGateEventName as i, ReviewSubmission as j, ReviewTarget as k };

package/metro/index.d.ts

@@ -16,12 +16,20 @@ export interface WithWireOnboardingOptions {
    * When omitted, the kit resolves from the installed package.
    */
   source?: string;
+  /**
+   * Extra specifiers to treat as optional, i.e. resolve to an EMPTY module instead of
+   * failing the build when they are not installed. Added to the kit's own
+   * `OPTIONAL_MODULES` list, never replacing it.
+   */
+  optionalModules?: string[];
 }
 
 /**
  * Non-destructively wire the @wireai/activation kit into an existing Metro config:
  * pins single-instance deps to the app's copy, optionally maps the kit specifier
- * to source + watches it. Mutates and returns the same config object.
+ * to source + watches it, and makes the kit's optional modules genuinely optional
+ * (an absent one resolves to an empty module instead of breaking the build).
+ * Mutates and returns the same config object.
  */
 export function withWireOnboarding<TConfig extends Record<string, any>>(
   config: TConfig,
@@ -30,3 +38,12 @@ export function withWireOnboarding<TConfig extends Record<string, any>>(
 
 /** The deps pinned to the app's single copy. */
 export const SINGLE_INSTANCE_DEPS: readonly ['react', 'react-native', 'wireai-rn', 'zod'];
+
+/**
+ * The modules the kit reads through a guarded require and can live without. When one of
+ * these does not resolve, `withWireOnboarding` maps it to an empty module so the host
+ * still bundles and the feature degrades (icon → null, analytics field omitted, native
+ * review → store-URL fallback). Modules the kit imports statically are deliberately NOT
+ * in this list — stubbing those would hide a real build error.
+ */
+export const OPTIONAL_MODULES: readonly string[];

package/metro/index.js

@@ -26,6 +26,8 @@
  * @param {string} [options.source]  Absolute path to the kit's `src/` dir. When
  *   set, the kit specifier maps here (source consumption) and it's added to
  *   watchFolders; when omitted, the kit resolves from the installed package.
+ * @param {string[]} [options.optionalModules]  Extra specifiers to treat as
+ *   optional (see OPTIONAL_MODULES). Added to the kit's list, never replacing it.
  * @returns {object} The same config object, mutated in place and returned.
  */
 'use strict';
@@ -35,6 +37,60 @@ const path = require('path');
 /** Deps that MUST resolve to the app's single copy (a 2nd React instance crashes RN). */
 const SINGLE_INSTANCE_DEPS = ['react', 'react-native', 'wireai-rn', 'zod'];
 
+/**
+ * Modules the kit reads through a GUARDED require/import and is designed to live without.
+ *
+ * ── HOW OPTIONAL DEPS ACTUALLY WORK UNDER METRO (verified, not folklore) ────────────────
+ *
+ * Metro has NO runtime module resolution: every specifier is resolved when the bundle is BUILT.
+ * But it DOES have a first-class optional-dependency mechanism, and it needs two things:
+ *
+ *   1. a STRING-LITERAL specifier — Metro's collector only matches `require("literal")` /
+ *      `import("literal")`. A variable specifier collects NOTHING, so the module never enters
+ *      the bundle at all (that was the 0.8.0 bug; see src/icons/expoIcons.ts), and
+ *   2. the call sitting inside a TRY/CATCH — that is literally how Metro decides a dependency
+ *      is optional (`isOptional`), and it only counts when the transformer has
+ *      `allowOptionalDependencies` enabled. Expo's `getDefaultConfig` enables it; plain
+ *      `metro-config` defaults it to FALSE.
+ *
+ * When both hold and the module is absent, Metro puts `null` in the module's dependencyMap, and
+ * `metroRequire(null, "name")` throws "Cannot find module" — synchronously, from metroRequire
+ * itself, so the caller's own catch swallows it. Verified in a real bundle: haptics' dependencyMap
+ * is `[355,null,1300]` with `expo-haptics` uninstalled. So DO NOT "simplify" a try/catch away from
+ * around a guarded require: the try/catch is load-bearing, and removing it turns an absent peer
+ * into a hard build failure.
+ *
+ * ── WHY THIS LIST STILL EXISTS ─────────────────────────────────────────────────────────
+ *
+ * Mechanism (2) is off by default in bare Metro (`allowOptionalDependencies: false`), and a host
+ * can turn it off. There, an absent optional module fails the BUILD with "Unable to resolve
+ * module". So `resolveRequest` below maps an absent module on this list to an EMPTY module, which
+ * lands on each caller's existing "peer absent" degradation path (no exports on the namespace →
+ * undefined → icon renders null / analytics field omitted / native review → store-URL fallback).
+ * It is a safety net that makes optionality independent of the host's Metro settings — not the
+ * only thing holding it up.
+ *
+ * ONLY modules read through a guarded require/import belong here. A module the kit imports
+ * STATICALLY at module scope (react-native-reanimated, expo-blur,
+ * @blazejkustra/react-native-onboarding) must NOT be stubbed: an empty module would turn a loud
+ * build-time "Unable to resolve module" into a silent `undefined` component that crashes at
+ * render, which is strictly worse. `test/canary/metroResolution.test.tsx` keeps this list in sync
+ * with the source by asking Metro which deps it marks optional.
+ */
+const OPTIONAL_MODULES = [
+  // src/icons/expoIcons.ts — icon families; absent → WireIcon renders null.
+  '@expo/vector-icons',
+  // src/device/appVersion.ts — analytics `by_app_version`; absent → field omitted.
+  'expo-constants',
+  'expo-application',
+  // src/device/deviceModel.ts — analytics iOS `by_model`; absent → field omitted.
+  'expo-device',
+  // src/reviews/storeReview.ts — native review prompt; absent → store-URL fallback.
+  'expo-store-review',
+  // src/haptics/haptics.ts — haptic feedback; absent → silent no-op.
+  'expo-haptics',
+];
+
 function withWireOnboarding(config, options) {
   const opts = options || {};
   const appRoot = opts.appRoot || process.cwd();
@@ -67,6 +123,29 @@ function withWireOnboarding(config, options) {
     ...existingExtra,
   };
 
+  // ── resolveRequest: make the optional modules genuinely optional ────────────
+  // Non-destructive by construction: the app's own resolveRequest (or Metro's default) is asked
+  // FIRST and keeps full authority. We only act on the failure of an OPTIONAL specifier, turning
+  // "Unable to resolve module" into an empty module so a host that never installed the peer still
+  // bundles. A non-optional module's failure is rethrown untouched — a missing `react` must stay
+  // a hard, loud build error.
+  const optional = new Set([...OPTIONAL_MODULES, ...(opts.optionalModules || [])]);
+  const upstream = resolver.resolveRequest;
+
+  resolver.resolveRequest = (context, moduleName, platform) => {
+    const resolve = upstream || context.resolveRequest;
+    try {
+      return resolve(context, moduleName, platform);
+    } catch (error) {
+      // Match the package root and its subpaths ('@expo/vector-icons/build/Icons').
+      const isOptional = [...optional].some(
+        (name) => moduleName === name || moduleName.startsWith(name + '/'),
+      );
+      if (!isOptional) throw error;
+      return { type: 'empty' };
+    }
+  };
+
   // ── watchFolders: add the kit source so Metro reads + HMRs it ───────────────
   if (source) {
     const existingWatch = Array.isArray(config.watchFolders) ? config.watchFolders : [];
@@ -81,3 +160,4 @@ function withWireOnboarding(config, options) {
 module.exports = { withWireOnboarding };
 module.exports.withWireOnboarding = withWireOnboarding;
 module.exports.SINGLE_INSTANCE_DEPS = SINGLE_INSTANCE_DEPS;
+module.exports.OPTIONAL_MODULES = OPTIONAL_MODULES;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wireai/activation",
-  "version": "0.8.0",
+  "version": "0.9.1",
   "private": false,
   "description": "Premium, fully-themable drop-in AI onboarding kit for React Native / Expo, on top of the open-source wireai-rn SDK.",
   "author": "Malik Chohra <malik@getwireai.com>",
@@ -143,7 +143,9 @@
   },
   "peerDependencies": {
     "@blazejkustra/react-native-onboarding": ">=1",
+    "@expo/vector-icons": ">=14",
     "expo-blur": ">=13",
+    "expo-haptics": ">=13",
     "expo-store-review": ">=6",
     "react": ">=18.0.0",
     "react-native": ">=0.73.0",
@@ -156,12 +158,18 @@
     "zod": {
       "optional": false
     },
+    "@expo/vector-icons": {
+      "optional": true
+    },
     "react-native-reanimated": {
       "optional": true
     },
     "expo-blur": {
       "optional": true
     },
+    "expo-haptics": {
+      "optional": true
+    },
     "expo-store-review": {
       "optional": true
     },

package/src/WireOnboarding.tsx

@@ -16,6 +16,7 @@ import { WireAIProvider, type LocalLLMConfig } from "wireai-rn";
 import { OnboardingThemeProvider } from "./theme/ThemeContext";
 import { IllustrationProvider } from "./components/Illustration";
 import { defaultIllustrations } from "./illustrations/defaultIllustrations";
+import { IconRegistryProvider } from "./icons/IconRegistry";
 import { LoadingScreen } from "./components/LoadingScreen";
 import { OnboardingFlow, DEFAULT_COPY } from "./OnboardingFlow";
 import { onboardingComponents } from "./cards";
@@ -36,6 +37,7 @@ export const WireOnboarding: React.FC<WireOnboardingProps> = ({
   theme,
   components,
   illustrations,
+  icons,
   validators,
   onComplete,
   onSkip,
@@ -229,6 +231,7 @@ export const WireOnboarding: React.FC<WireOnboardingProps> = ({
   return (
     <OnboardingThemeProvider theme={theme}>
       <IllustrationProvider registry={{ ...defaultIllustrations, ...illustrations }}>
+        <IconRegistryProvider registry={icons}>
         <WireAIProvider llm={llm} components={cards}>
           <OnboardingFlow
             validators={validators}
@@ -248,6 +251,7 @@ export const WireOnboarding: React.FC<WireOnboardingProps> = ({
             clientContext={clientContext}
           />
         </WireAIProvider>
+        </IconRegistryProvider>
       </IllustrationProvider>
     </OnboardingThemeProvider>
   );

package/src/analytics/currentSession.ts

@@ -6,7 +6,7 @@
  * post `app.session_started` with it — so the SERVER knows that id. But other client paths
  * (`identify`, host `app_event`s through the analytics façade) used to reference a DIFFERENT id
  * (a frozen per-instance id), which the server had never seen, so it back-filled a synthetic
- * `session_started` — inflating session counts (the Morrow/Myelino "phantom-session" bug).
+ * `session_started` — inflating session counts (the "phantom-session" bug).
  *
  * This registry is the single seam that lets those paths reuse the LIVE per-open session id the
  * server already ingested. `reportSessionStart` writes the current id here on every open; the façade