react-render-detective 0.4.0 → 0.6.0

package/dist/index.js

@@ -1,11 +1,45 @@
-import { explainEvents, formatExplanation } from './chunk-HOTY7J3X.js';
-export { explainEvents, formatExplanation } from './chunk-HOTY7J3X.js';
-import { getDetective, inspect, shallowEqual, diffProps, formatInspected } from './chunk-QOGTEVBP.js';
+import { explainEvents, formatExplanation } from './chunk-D7M4UC3Y.js';
+export { explainEvents, formatExplanation } from './chunk-D7M4UC3Y.js';
 import { profileFromEvents } from './chunk-LILK23YH.js';
+import { getDetective, shallowEqual, inspect, diffProps, formatInspected } from './chunk-3TZNM7JO.js';
 import * as React from 'react';
-import { createContext, useContext, useState, useRef, useEffect, useLayoutEffect, useCallback, Profiler } from 'react';
+import { createContext, useContext, useRef, useState, useEffect, useLayoutEffect, useCallback, Profiler } from 'react';
 import { jsx } from 'react/jsx-runtime';
 
+// src/console/style.ts
+var PALETTE = {
+  plain: "",
+  dim: "color:#8a94a6",
+  strong: "font-weight:600",
+  good: "color:#1f9d5b",
+  warn: "color:#c2820a",
+  bad: "color:#d1493f;font-weight:600"
+};
+var supportsStyling = () => {
+  try {
+    return typeof window !== "undefined" && typeof document !== "undefined";
+  } catch {
+    return false;
+  }
+};
+function styled(segments) {
+  if (!supportsStyling()) {
+    return [segments.map((s) => typeof s === "string" ? s : s[0]).join("")];
+  }
+  let format = "";
+  const styles = [];
+  for (const segment of segments) {
+    if (typeof segment === "string") {
+      format += segment.replace(/%/g, "%%");
+      continue;
+    }
+    const [text, tone] = segment;
+    format += `%c${text.replace(/%/g, "%%")}%c`;
+    styles.push(PALETTE[tone], "");
+  }
+  return [format, ...styles];
+}
+
 // src/console/reporter.ts
 var ICON = {
   normal: "\xB7",
@@ -14,30 +48,102 @@ var ICON = {
   "very-slow": "\u25B2\u25B2",
   critical: "\u25A0"
 };
+var BATCH_WINDOW_MS = 400;
 function attachConsoleReporter(detective) {
-  return detective.subscribe((event) => {
-    const mode = detective.config.mode;
-    if (mode === "silent") return;
+  let pending = [];
+  let timer;
+  const flush = () => {
+    timer = void 0;
+    const batch = pending;
+    pending = [];
+    if (batch.length === 0) return;
     try {
-      if (mode === "verbose") printVerbose(event);
-      else printConcise(event, detective.config.slowRenderThreshold);
+      printBatch(batch, detective.config.slowRenderThreshold);
     } catch {
     }
+  };
+  const unsubscribe = detective.subscribe((event) => {
+    const mode = detective.config.mode;
+    if (mode === "silent") return;
+    if (mode === "verbose") {
+      try {
+        printVerbose(event);
+      } catch {
+      }
+      return;
+    }
+    pending.push(event);
+    if (timer === void 0) {
+      timer = setTimeout(flush, BATCH_WINDOW_MS);
+      timer.unref?.();
+    }
   });
+  return () => {
+    unsubscribe();
+    if (timer !== void 0) clearTimeout(timer);
+  };
 }
-function printConcise(event, slowThreshold) {
-  const { diagnosis, timings, component } = event;
-  const changed = event.changedProps.map((c) => c.key).join(", ");
-  const parts = [
-    `[RRD] ${component.name} #${event.renderNumber}`,
-    `Reason: ${label(event)}`,
-    changed ? `Changed: ${changed}` : void 0,
-    timings.subtreeDuration > 0 ? `Duration: ${timings.selfDuration.toFixed(1)}ms` : void 0
-  ].filter(Boolean);
-  const line = `${ICON[diagnosis.severity] ?? "\xB7"} ${parts.join("  ")}`;
-  const slow = timings.selfDuration >= slowThreshold;
-  if (slow) console.warn(line);
-  else console.log(line);
+function printBatch(batch, slowThreshold) {
+  const byComponent = /* @__PURE__ */ new Map();
+  let totalMs = 0;
+  let avoidable = 0;
+  for (const event of batch) {
+    const key = event.component.name;
+    const entry = byComponent.get(key) ?? {
+      name: key,
+      source: event.component.source,
+      count: 0,
+      totalMs: 0,
+      reasons: /* @__PURE__ */ new Map(),
+      notableReasons: /* @__PURE__ */ new Map(),
+      avoidable: 0,
+      remount: false,
+      slow: false
+    };
+    entry.count++;
+    entry.totalMs += event.timings.selfDuration;
+    entry.reasons.set(event.diagnosis.reason, (entry.reasons.get(event.diagnosis.reason) ?? 0) + 1);
+    if (event.diagnosis.potentiallyAvoidable || event.timings.selfDuration >= slowThreshold) {
+      entry.notableReasons.set(event.diagnosis.reason, (entry.notableReasons.get(event.diagnosis.reason) ?? 0) + 1);
+    }
+    if (event.diagnosis.potentiallyAvoidable) entry.avoidable++;
+    if (event.diagnosis.summary.includes("rebuilt")) entry.remount = true;
+    if (event.timings.selfDuration >= slowThreshold) entry.slow = true;
+    if (!entry.worst || event.timings.selfDuration > entry.worst.timings.selfDuration) entry.worst = event;
+    byComponent.set(key, entry);
+    totalMs += event.timings.selfDuration;
+    if (event.diagnosis.potentiallyAvoidable) avoidable++;
+  }
+  const notable = [...byComponent.values()].filter((a) => a.avoidable > 0 || a.slow || a.remount);
+  if (notable.length === 0) return;
+  const ranked = [...notable].sort((a, b) => b.totalMs - a.totalMs);
+  const anySlow = ranked.some((a) => a.slow);
+  const segments = [
+    ["[RRD] ", "dim"],
+    [`${batch.length} render${batch.length === 1 ? "" : "s"}`, "strong"],
+    [` \xB7 ${totalMs.toFixed(1)}ms`, "dim"]
+  ];
+  if (avoidable > 0) segments.push([` \xB7 ${avoidable} potentially avoidable`, "warn"]);
+  for (const a of ranked.slice(0, 8)) {
+    const reasonSource = a.notableReasons.size > 0 ? a.notableReasons : a.reasons;
+    const reason = [...reasonSource.entries()].sort((x, y) => y[1] - x[1])[0]?.[0] ?? "unknown";
+    const tone = a.remount ? "bad" : a.slow ? "warn" : a.avoidable > 0 ? "warn" : "good";
+    const label2 = `  ${ICON[a.slow ? "slow" : "normal"]} ${a.name}${a.count > 1 ? ` \xD7${a.count}` : ""}`;
+    segments.push("\n", [label2.padEnd(30), tone], [`${a.totalMs.toFixed(1)}ms`, "plain"], [`  ${reason}`, "dim"]);
+    if (a.remount) segments.push(["  rebuilt", "bad"]);
+    if (a.avoidable > 0) segments.push([`  ${a.avoidable} avoidable`, "warn"]);
+    if (a.source) segments.push(["\n      " + a.source, "dim"]);
+  }
+  const worst = ranked[0]?.worst;
+  segments.push(
+    "\n",
+    [
+      worst?.diagnosis.suggestion ? `  \u2192 ${worst.diagnosis.suggestion}` : "  \u2192 rrd.printOpportunities() to rank everything by recoverable time",
+      "dim"
+    ]
+  );
+  const log = anySlow ? console.warn : console.log;
+  log(...styled(segments));
 }
 function printVerbose(event) {
   const { diagnosis, timings } = event;
@@ -60,9 +166,7 @@ function printVerbose(event) {
       for (const c of event.changedProps) console.log(propLine(c));
       console.groupEnd();
     }
-    if (event.unchangedProps.length > 0) {
-      console.log(`Props same:  ${event.unchangedProps.join(", ")}`);
-    }
+    if (event.unchangedProps.length > 0) console.log(`Props same:  ${event.unchangedProps.join(", ")}`);
     if (event.trackedState.length > 0) {
       console.log(
         `State:       ${event.trackedState.map((s) => `${s.name}: ${formatInspected(s.previous)} \u2192 ${formatInspected(s.current)}`).join(", ")}`
@@ -113,6 +217,8 @@ function label(event) {
       return "context update";
     case "state":
       return "state changed";
+    case "store":
+      return "store update";
     case "state-or-external":
       return "state or external store";
     case "unknown":
@@ -120,194 +226,6 @@ function label(event) {
   }
 }
 
-// src/core/interactions.ts
-var COMMIT_SLACK_MS = 100;
-var FALLBACK_CLOSE_MS = 50;
-var InteractionTracker = class {
-  constructor(capacity = 50) {
-    this.capacity = capacity;
-    this.records = [];
-    this.nextId = 0;
-  }
-  /** Returns false when the browser cannot report event timing. */
-  start() {
-    if (this.observer) return true;
-    const PO = globalThis.PerformanceObserver;
-    const supported = PO?.supportedEntryTypes?.includes("event");
-    if (!PO || !supported) return false;
-    try {
-      const observer = new PO((list) => {
-        for (const entry of list.getEntries()) {
-          const timing = entry;
-          this.record({
-            name: timing.name,
-            startTime: timing.startTime,
-            duration: timing.duration,
-            target: describeTarget(timing.target)
-          });
-        }
-      });
-      observer.observe({ type: "event", buffered: true, durationThreshold: 16 });
-      this.observer = observer;
-      return true;
-    } catch {
-      return false;
-    }
-  }
-  stop() {
-    this.observer?.disconnect();
-    this.observer = void 0;
-  }
-  /** Is the automatic path available in this browser? */
-  get automatic() {
-    return this.observer !== void 0;
-  }
-  /**
-   * Time an interaction by hand.
-   *
-   * The automatic path depends on the Event Timing API, which Safari only
-   * gained in 16.4 and which does not fire for synthetic input at all — so
-   * anything driven by a test harness records nothing. This measures a specific
-   * action instead, up to the paint that follows it, and needs no browser
-   * support beyond `performance.now`.
-   */
-  measure(label2, action) {
-    const startTime = now();
-    let handlerMs = 0;
-    let finished = false;
-    const finish = () => {
-      if (finished) return;
-      finished = true;
-      this.record({ name: label2, startTime, duration: now() - startTime, handlerMs });
-    };
-    let result;
-    try {
-      result = action();
-      handlerMs = now() - startTime;
-    } catch (error) {
-      handlerMs = now() - startTime;
-      finish();
-      throw error;
-    }
-    const raf = globalThis.requestAnimationFrame;
-    if (raf) raf(() => raf(finish));
-    setTimeout(finish, FALLBACK_CLOSE_MS);
-    return result;
-  }
-  /** Exposed for tests and for `measure`. */
-  record(timing) {
-    const record = {
-      id: `interaction_${++this.nextId}`,
-      type: timing.name,
-      target: timing.target,
-      startTime: timing.startTime,
-      durationMs: timing.duration,
-      handlerMs: timing.handlerMs,
-      renders: [],
-      renderTimeMs: 0,
-      avoidableRenderTimeMs: 0
-    };
-    this.records.push(record);
-    if (this.records.length > this.capacity) this.records.shift();
-    return record;
-  }
-  clear() {
-    this.records.length = 0;
-  }
-  /**
-   * Joins render events to interactions by commit time. A render belongs to an
-   * interaction when it committed between the event starting and shortly after
-   * it finished — React commits just after the event handler returns.
-   */
-  attribute(events) {
-    for (const record of this.records) {
-      const from = record.startTime;
-      const to = record.startTime + record.durationMs + COMMIT_SLACK_MS;
-      record.renders = events.filter((e) => e.timings.commitTime >= from && e.timings.commitTime <= to);
-      record.renderTimeMs = record.renders.reduce((a, e) => a + e.timings.selfDuration, 0);
-      record.avoidableRenderTimeMs = record.renders.filter((e) => e.diagnosis.potentiallyAvoidable).reduce((a, e) => a + e.timings.selfDuration, 0);
-    }
-    return [...this.records].sort((a, b) => b.durationMs - a.durationMs);
-  }
-};
-function summarise(record) {
-  const byComponent = /* @__PURE__ */ new Map();
-  for (const event of record.renders) {
-    const entry = byComponent.get(event.component.name) ?? {
-      renders: 0,
-      totalMs: 0,
-      source: event.component.source,
-      cause: event.diagnosis.reason
-    };
-    entry.renders++;
-    entry.totalMs += event.timings.selfDuration;
-    byComponent.set(event.component.name, entry);
-  }
-  const contributors = [...byComponent.entries()].map(([component, v]) => ({ component, source: v.source, renders: v.renders, totalMs: v.totalMs, cause: v.cause })).sort((a, b) => b.totalMs - a.totalMs);
-  const top = contributors[0];
-  const accounted = (record.handlerMs ?? record.durationMs) + record.renderTimeMs;
-  const idleWindow = record.handlerMs !== void 0 && record.durationMs > accounted * 3 && record.durationMs - accounted > 100;
-  const effectiveMs = idleWindow ? accounted : record.durationMs;
-  const share = effectiveMs > 0 ? record.renderTimeMs / effectiveMs : 0;
-  let headline;
-  let nextStep;
-  let confidence = "medium";
-  if (idleWindow) {
-    headline = `${record.type}: ${fmt(record.handlerMs ?? 0)} in the handler and ${fmt(record.renderTimeMs)} rendering. The measured window was ${fmt(record.durationMs)}, but most of that was the page waiting for a frame \u2014 ignore it.`;
-    nextStep = record.renderTimeMs > (record.handlerMs ?? 0) ? `Rendering dominates the real work${top ? `; start with ${top.component}` : ""}.` : "The handler itself costs more than rendering. Profile the handler, not React.";
-    return { interaction: record, contributors, headline, nextStep, confidence: "medium" };
-  }
-  if (record.renders.length === 0) {
-    headline = `${record.type} took ${fmt(record.durationMs)}, and no instrumented component rendered inside it.`;
-    nextStep = "The cost is somewhere other than React rendering \u2014 an event handler, a layout, or an uninstrumented component. Instrument more of the tree to narrow it down.";
-    confidence = "low";
-  } else if (share >= 0.4 && record.avoidableRenderTimeMs > 0) {
-    headline = `${record.type} took ${fmt(record.durationMs)}; ${fmt(record.renderTimeMs)} of it was rendering, and ${fmt(record.avoidableRenderTimeMs)} of that had no input change to explain it.`;
-    nextStep = top ? `Start with ${top.component}${top.source ? ` (${top.source})` : ""} \u2014 ${fmt(top.totalMs)} across ${top.renders} render${top.renders === 1 ? "" : "s"}.` : "Look at the top contributor below.";
-    confidence = "high";
-  } else if (share >= 0.4) {
-    headline = `${record.type} took ${fmt(record.durationMs)}; ${fmt(record.renderTimeMs)} of it was rendering, all of it explained by real input changes.`;
-    nextStep = "This is genuine work. Make the renders cheaper rather than fewer \u2014 or do less of it per interaction.";
-    confidence = "high";
-  } else {
-    headline = `${record.type} took ${fmt(record.durationMs)}, but only ${fmt(record.renderTimeMs)} was React rendering.`;
-    nextStep = "Most of the cost is outside rendering \u2014 event handlers, layout or paint. A browser profile will show more than this tool can.";
-    confidence = "medium";
-  }
-  return { interaction: record, contributors, headline, nextStep, confidence };
-}
-function formatInteraction(summary) {
-  const { interaction: i } = summary;
-  const lines = [
-    `${i.type}${i.target ? ` on ${i.target}` : ""}   ${fmt(i.durationMs)}`,
-    "",
-    summary.headline
-  ];
-  if (summary.contributors.length > 0) {
-    lines.push("", "Rendering inside this interaction");
-    for (const c of summary.contributors.slice(0, 8)) {
-      lines.push(
-        `  ${c.component.padEnd(22)} ${String(c.renders).padStart(4)} render(s)  ${fmt(c.totalMs).padStart(8)}  ${c.cause}`
-      );
-    }
-  }
-  lines.push("", `Next step`, `  ${summary.nextStep}`, "", `Confidence: ${summary.confidence}`);
-  return lines.join("\n");
-}
-function describeTarget(target) {
-  if (!target || typeof target !== "object") return void 0;
-  const el = target;
-  if (!el.tagName) return void 0;
-  const tag = el.tagName.toLowerCase();
-  if (el.id) return `${tag}#${el.id}`;
-  const className = typeof el.className === "string" ? el.className.trim().split(/\s+/)[0] : void 0;
-  if (className) return `${tag}.${className}`;
-  const text = el.textContent?.trim().slice(0, 20);
-  return text ? `${tag} "${text}"` : tag;
-}
-var fmt = (ms) => `${ms.toFixed(1)}ms`;
-var now = () => typeof performance !== "undefined" && typeof performance.now === "function" ? performance.now() : Date.now();
-
 // src/core/opportunities.ts
 var DEFAULT_MIN_SAVING_MS = 1;
 function rankOpportunities({ events, lifecycles, minSavingMs = DEFAULT_MIN_SAVING_MS }) {
@@ -426,6 +344,38 @@ function RenderDetective({ name, children }) {
   const { node, onRender } = useInstrumentedNode(name, { children });
   return renderInstrumented(node, onRender, children);
 }
+var useOwnerId = typeof React.useId === "function" ? React.useId : () => "";
+function trackSelector(value, options = {}) {
+  const detective = getDetective();
+  const node = useContext(AncestryContext);
+  const ownerId = useOwnerId();
+  const previous = useRef(void 0);
+  if (!detective.enabled) return value;
+  if (node && node.stateOwner === void 0) node.stateOwner = ownerId;
+  const owns = !node || node.stateOwner === ownerId;
+  const prior = previous.current;
+  previous.current = { value };
+  if (!prior || !node || !owns) return value;
+  if (Object.is(prior.value, value)) return value;
+  try {
+    const equal = shallowEqual(prior.value, value, detective.config);
+    detective.recordSelectorChange(node, {
+      name: options.name ?? options.source ?? "selector",
+      source: options.source,
+      referenceOnly: equal === true,
+      previous: inspect(prior.value, detective.config.inspection),
+      current: inspect(value, detective.config.inspection)
+    });
+  } catch {
+  }
+  return value;
+}
+function createTrackedSelectorHook(hook, options = {}) {
+  return function useTrackedSelector(...args) {
+    const value = hook(...args);
+    return trackSelector(value, options);
+  };
+}
 function useRenderDiagnostics(name, props) {
   const detective = getDetective();
   const parent = useContext(AncestryContext);
@@ -460,11 +410,11 @@ function useRenderDiagnostics(name, props) {
   }
   return last.current;
 }
-var useOwnerId = typeof React.useId === "function" ? React.useId : () => "";
+var useOwnerId2 = typeof React.useId === "function" ? React.useId : () => "";
 function useTrackedState(name, initial) {
   const detective = getDetective();
   const node = useContext(AncestryContext);
-  const ownerId = useOwnerId();
+  const ownerId = useOwnerId2();
   const [state, setState] = useState(initial);
   const previous = useRef(state);
   if (node && node.stateOwner === void 0) node.stateOwner = ownerId;
@@ -525,21 +475,12 @@ function isRecord(v) {
 
 // src/index.ts
 var REPORTER = /* @__PURE__ */ Symbol.for("react-render-detective.reporter");
-var INTERACTIONS = /* @__PURE__ */ Symbol.for("react-render-detective.interactions");
-function tracker() {
-  const g = globalThis;
-  if (!g[INTERACTIONS]) g[INTERACTIONS] = new InteractionTracker();
-  return g[INTERACTIONS];
-}
 function init(options = {}) {
   const detective = getDetective();
   detective.init(options);
   const g = globalThis;
   g[REPORTER]?.();
   g[REPORTER] = void 0;
-  if (detective.enabled) {
-    tracker().start();
-  }
   if (detective.enabled && detective.config.mode !== "silent") {
     g[REPORTER] = attachConsoleReporter(detective);
   } else if (!detective.enabled) {
@@ -571,14 +512,11 @@ function subscribe(listener) {
 }
 function clear() {
   getDetective().clear();
-  tracker().clear();
 }
 function reset() {
   const g = globalThis;
   g[REPORTER]?.();
   g[REPORTER] = void 0;
-  g[INTERACTIONS]?.stop();
-  g[INTERACTIONS] = void 0;
   getDetective().reset();
 }
 function explain(componentName2) {
@@ -593,31 +531,6 @@ function getRenderProfile(scenario) {
   for (const stats of getDetective().getComponentStats()) remounts[stats.name] = stats.remountCount;
   return profileFromEvents(scenario, getEvents(), remounts);
 }
-function getInteractions() {
-  return tracker().attribute(getEvents());
-}
-function explainInteractionStructured(id) {
-  const records = getInteractions();
-  const record = id ? records.find((r) => r.id === id) : records[0];
-  return record ? summarise(record) : void 0;
-}
-function explainInteraction(id) {
-  const summary = explainInteractionStructured(id);
-  return summary ? formatInteraction(summary) : void 0;
-}
-function printInteractions(limit = 5) {
-  const records = getInteractions().slice(0, limit);
-  if (records.length === 0) {
-    console.log(
-      tracker().automatic ? "No interactions recorded yet. Event timing is working \u2014 nothing has taken longer than 16ms.\nSynthetic clicks from a test harness never produce these entries; use measureInteraction() there." : "This browser does not report event timing (Safari before 16.4, jsdom).\nUse measureInteraction(label, fn) to time interactions by hand."
-    );
-    return;
-  }
-  console.log(records.map((r) => formatInteraction(summarise(r))).join("\n\n"));
-}
-function measureInteraction(label2, action) {
-  return tracker().measure(label2, action);
-}
 function getOpportunities(limit = 10) {
   const detective = getDetective();
   const lifecycles = /* @__PURE__ */ new Map();
@@ -676,15 +589,10 @@ var ReactRenderDetective = {
   explainStructured,
   getOpportunities,
   printOpportunities,
-  getInteractions,
-  explainInteraction,
-  explainInteractionStructured,
-  printInteractions,
-  measureInteraction,
   getRenderProfile,
   printStats
 };
 
-export { InteractionTracker, ReactRenderDetective, RenderDetective, clear, configure, explain, explainInteraction, explainInteractionStructured, explainStructured, formatInteraction, formatOpportunities, getComponentStats, getConfig, getEvents, getInteractions, getOpportunities, getRenderProfile, getStats, init, isEnabled, measureInteraction, printInteractions, printOpportunities, printStats, rankOpportunities, reset, subscribe, summarise as summariseInteraction, useRenderDiagnostics, useTrackedContextValue, useTrackedEffect, useTrackedState, withRenderDetective };
+export { ReactRenderDetective, RenderDetective, clear, configure, createTrackedSelectorHook, explain, explainStructured, formatOpportunities, getComponentStats, getConfig, getEvents, getOpportunities, getRenderProfile, getStats, init, isEnabled, printOpportunities, printStats, rankOpportunities, reset, subscribe, trackSelector, useRenderDiagnostics, useTrackedContextValue, useTrackedEffect, useTrackedState, withRenderDetective };
 //# sourceMappingURL=index.js.map
 //# sourceMappingURL=index.js.map