bippy 0.7.3-dev.8ba0705 → 0.7.3-dev.9f2850b

package/src/source/get-source.ts

@@ -1,26 +1,17 @@
-import type { Fiber } from "../react-internals/index.js";
-import { getDisplayName } from "../core.js";
+import type { Fiber } from "../react-internals/index";
+import { getDisplayName } from "../core";
 
-import type { FiberSource } from "./types.js";
-import {
-  SCHEME_REGEX,
-  INTERNAL_SCHEME_PREFIXES,
-  ABOUT_REACT_PREFIX,
-  ANONYMOUS_FILE_PATTERNS,
-  SOURCE_FILE_EXTENSION_REGEX,
-  BUNDLED_FILE_PATTERN_REGEX,
-  QUERY_PARAMETER_PATTERN_REGEX,
-} from "./constants.js";
-import { getDefinitionFrameFromOwnedChild, getParentStack, hasDebugStack } from "./owner-stack.js";
-import { parseDebugStack } from "./parse-debug-stack.js";
-import { parseStack, type StackFrame } from "./parse-stack.js";
+import type { FiberSource } from "./types";
+import { getDefinitionFrameFromOwnedChild, getParentStack, hasDebugStack } from "./owner-stack";
+import { parseDebugStack } from "./parse-debug-stack";
+import { parseStack, type StackFrame } from "./parse-stack";
 import {
   getSourceFromSourceMapByFunctionName,
   getSourceMap,
   symbolicateStack,
   type SourceFetch,
   type SourceMapRequestOptions,
-} from "./symbolication.js";
+} from "./symbolication";
 
 export const hasDebugSource = (
   fiber: Fiber,
@@ -165,130 +156,4 @@ export const getSource = async (
   return getSourceByComponentName(fiber, shouldUseCache, sourceFetch, requestOptions);
 };
 
-const getPathSegmentCount = (path: string): number => path.split("/").filter(Boolean).length;
-
-const stripSingleBasePathPrefix = (path: string): string => {
-  const firstSlashIndex = path.indexOf("/", 1);
-  if (firstSlashIndex === -1) {
-    return path;
-  }
-
-  const basePath = path.slice(0, firstSlashIndex);
-  if (getPathSegmentCount(basePath) !== 1) {
-    return path;
-  }
-
-  const remainderPath = path.slice(firstSlashIndex);
-  if (!SOURCE_FILE_EXTENSION_REGEX.test(remainderPath)) {
-    return path;
-  }
-
-  const remainderSegments = remainderPath.split("/").filter(Boolean);
-  if (remainderSegments.length < 2) {
-    return path;
-  }
-
-  const firstRemainderSegment = remainderSegments[0];
-  if (firstRemainderSegment.startsWith("@")) {
-    return path;
-  }
-
-  if (firstRemainderSegment.length > 4) {
-    return path;
-  }
-
-  return remainderPath;
-};
-
-export const normalizeFileName = (fileName: string): string => {
-  if (!fileName) {
-    return "";
-  }
-
-  if (ANONYMOUS_FILE_PATTERNS.some((pattern) => pattern === fileName)) {
-    return "";
-  }
-
-  let normalizedFileName = fileName;
-  const isWindowsDrivePath = /^[a-zA-Z]:[\\/]/.test(normalizedFileName);
-
-  const isHttpUrl =
-    normalizedFileName.startsWith("http://") || normalizedFileName.startsWith("https://");
-  if (isHttpUrl) {
-    try {
-      const parsedUrl = new URL(normalizedFileName);
-      normalizedFileName = parsedUrl.pathname;
-    } catch {}
-    normalizedFileName = stripSingleBasePathPrefix(normalizedFileName);
-  }
-
-  if (normalizedFileName.startsWith(ABOUT_REACT_PREFIX)) {
-    const remainder = normalizedFileName.slice(ABOUT_REACT_PREFIX.length);
-    const slashIndex = remainder.indexOf("/");
-    const colonIndex = remainder.indexOf(":");
-
-    if (slashIndex !== -1 && (colonIndex === -1 || slashIndex < colonIndex)) {
-      normalizedFileName = remainder.slice(slashIndex + 1);
-    } else {
-      normalizedFileName = remainder;
-    }
-  }
-
-  let didStripPrefix = true;
-  while (didStripPrefix) {
-    didStripPrefix = false;
-    for (const prefix of INTERNAL_SCHEME_PREFIXES) {
-      if (normalizedFileName.startsWith(prefix)) {
-        normalizedFileName = normalizedFileName.slice(prefix.length);
-
-        if (prefix === "file:///") {
-          normalizedFileName = `/${normalizedFileName.replace(/^\/+/, "")}`;
-        }
-
-        didStripPrefix = true;
-        break;
-      }
-    }
-  }
-
-  if (!isWindowsDrivePath) {
-    const schemeMatch = normalizedFileName.match(SCHEME_REGEX);
-    if (schemeMatch) {
-      normalizedFileName = normalizedFileName.slice(schemeMatch[0].length);
-    }
-  }
-
-  if (normalizedFileName.startsWith("//")) {
-    const firstPathSlashIndex = normalizedFileName.indexOf("/", 2);
-    normalizedFileName =
-      firstPathSlashIndex === -1 ? "" : normalizedFileName.slice(firstPathSlashIndex);
-  }
-
-  const queryParameterIndex = normalizedFileName.indexOf("?");
-  if (queryParameterIndex !== -1) {
-    const potentialQueryParameters = normalizedFileName.slice(queryParameterIndex);
-    if (QUERY_PARAMETER_PATTERN_REGEX.test(potentialQueryParameters)) {
-      normalizedFileName = normalizedFileName.slice(0, queryParameterIndex);
-    }
-  }
-
-  return normalizedFileName;
-};
-
-export const isSourceFile = (fileName: string): boolean => {
-  const normalizedFileName = normalizeFileName(fileName);
-
-  if (!normalizedFileName) {
-    return false;
-  }
-
-  if (!SOURCE_FILE_EXTENSION_REGEX.test(normalizedFileName)) {
-    return false;
-  }
-
-  if (BUNDLED_FILE_PATTERN_REGEX.test(normalizedFileName)) {
-    return false;
-  }
-
-  return true;
-};
+export { isSourceFile, normalizeFileName } from "./normalize-file-name";

package/src/source/index.ts

@@ -6,8 +6,8 @@ export {
   getParentStack,
   getRawOwnerStack,
   hasDebugStack,
-} from "./owner-stack.js";
-export { getRawSource, getSource, isSourceFile, normalizeFileName } from "./get-source.js";
+} from "./owner-stack";
+export { getRawSource, getSource, isSourceFile, normalizeFileName } from "./get-source";
 export {
   getSourceContentFromSourceMap,
   getSourceFromSourceMap,
@@ -20,22 +20,22 @@ export {
   type SourceMap,
   type SourceMapRequestOptions,
   type StandardSourceMap,
-} from "./symbolication.js";
-export type { FiberSource } from "./types.js";
-export { parseStack, type ParseOptions, type StackFrame } from "./parse-stack.js";
-export { getDisplayNameFromSource } from "./get-display-name-from-source.js";
+} from "./symbolication";
+export type { FiberSource } from "./types";
+export { parseStack, type ParseOptions, type StackFrame } from "./parse-stack";
+export { getDisplayNameFromSource } from "./get-display-name-from-source";
 export {
   BippyError,
   BippyHookInspectionError,
   BippyHookRenderError,
   BippySourceMapError,
   BippyUnsupportedHookError,
-} from "../errors.js";
+} from "../errors";
 export {
   getFiberHooks,
   inspectHooks,
   type HookSource,
   type HooksNode,
   type HooksTree,
-} from "./inspect-hooks.js";
-export { parseHookNames, type HookNames } from "./parse-hook-names.js";
+} from "./inspect-hooks";
+export { parseHookNames, type HookNames } from "./parse-hook-names";

package/src/source/inspect-hooks.ts

@@ -1,5 +1,5 @@
-import { getRenderer } from "../core.js";
-import type { ReactDevToolsTarget } from "../rdt-hook.js";
+import { getRenderer } from "../core";
+import type { ReactDevToolsTarget } from "../rdt-hook";
 import type {
   Fiber,
   ContextDependency,
@@ -7,19 +7,15 @@ import type {
   ReactContext,
   ReactDebugInfo,
   RendererDispatcherRef,
-} from "../react-internals/index.js";
+} from "../react-internals/index";
 import {
   BippyHookInspectionError,
   BippyHookRenderError,
   BippyUnsupportedHookError,
-} from "../errors.js";
-import { getReactWorkTagsForFiber } from "../react-internals/index.js";
-import { parseStack, type StackFrame } from "./parse-stack.js";
-import {
-  getRendererDispatcherRefs,
-  readDispatcher,
-  writeDispatcher,
-} from "./renderer-dispatchers.js";
+} from "../errors";
+import { getReactWorkTagsForFiber } from "../react-internals/index";
+import { createStackParser, parseStack, type StackFrame } from "./parse-stack";
+import { getRendererDispatcherRefs, readDispatcher, writeDispatcher } from "./renderer-dispatchers";
 
 const REACT_CONTEXT_TYPE = Symbol.for("react.context");
 const REACT_MEMO_CACHE_SENTINEL = Symbol.for("react.memo_cache_sentinel");
@@ -87,6 +83,16 @@ let currentHook: MemoizedState | null = null;
 let currentContextDependency: ContextDependency<unknown> | null = null;
 let currentThenableIndex = 0;
 let currentThenableState: unknown[] | null = null;
+let currentMemoCacheIndex = 0;
+let isInspectingHooks = false;
+
+const assertNotInspectingHooks = (): void => {
+  if (isInspectingHooks) {
+    throw new BippyHookInspectionError(
+      "Hook inspection cannot be called during another inspection.",
+    );
+  }
+};
 
 const SuspenseException: unknown = new Error(
   "Suspense interrupted this render. This error is an internal implementation detail of `use`.",
@@ -323,16 +329,8 @@ const dispatcherUseMemoCache = (size: number): unknown[] => {
   const memoCache = fiber.updateQueue?.memoCache;
   if (memoCache === null || memoCache === undefined) return [];
 
-  let memoCacheSlots = memoCache.data[memoCache.index];
-  if (memoCacheSlots === undefined) {
-    memoCacheSlots = memoCache.data[memoCache.index] = Array.from(
-      { length: size },
-      () => REACT_MEMO_CACHE_SENTINEL,
-    );
-  }
-
-  memoCache.index++;
-  return memoCacheSlots;
+  const memoCacheSlots = memoCache.data[currentMemoCacheIndex++];
+  return memoCacheSlots?.slice() ?? Array.from({ length: size }, () => REACT_MEMO_CACHE_SENTINEL);
 };
 
 const dispatcherUseOptimistic = (passthrough: unknown): [unknown, () => void] => {
@@ -602,8 +600,9 @@ const findPrimitiveIndex = (hookStack: StackFrame[], hook: HookLogEntry): number
 const parseTrimmedStack = (
   rootStack: StackFrame[],
   hook: HookLogEntry,
+  parseHookStack: (stack: string) => StackFrame[],
 ): [StackFrame | null, StackFrame[] | null] => {
-  const hookStack = parseErrorStack(hook.stackError);
+  const hookStack = parseHookStack(hook.stackError.stack || "");
   const rootIndex = findCommonAncestorIndex(rootStack, hookStack);
   const primitiveIndex = findPrimitiveIndex(hookStack, hook);
   if (rootIndex === -1 || primitiveIndex === -1 || rootIndex - primitiveIndex < 2) {
@@ -624,13 +623,14 @@ const NON_ID_HOOK_PRIMITIVES = new Set([
 
 const buildTree = (rootStack: StackFrame[], capturedHookLog: HookLogEntry[]): HooksTree => {
   const rootChildren: HooksNode[] = [];
+  const parseHookStack = createStackParser();
   let previousStack: StackFrame[] | null = null;
   let levelChildren = rootChildren;
   let nativeHookID = 0;
   const childrenStack: HooksNode[][] = [];
 
   for (const hook of capturedHookLog) {
-    const [primitiveFrame, stack] = parseTrimmedStack(rootStack, hook);
+    const [primitiveFrame, stack] = parseTrimmedStack(rootStack, hook, parseHookStack);
     let displayName = hook.displayName;
     if (displayName === null && primitiveFrame !== null) {
       const primitiveName = parseHookName(primitiveFrame.functionName);
@@ -834,6 +834,7 @@ const performDispatcherInspection = <TArgs extends unknown[]>(
 ): HooksTree => {
   const previousDispatcher = readDispatcher(dispatcherRef);
   writeDispatcher(dispatcherRef, dispatcherProxy);
+  isInspectingHooks = true;
 
   let capturedHookLog: HookLogEntry[] = [];
   let ancestorStackError: Error | undefined;
@@ -846,6 +847,7 @@ const performDispatcherInspection = <TArgs extends unknown[]>(
   } finally {
     capturedHookLog = hookLog;
     hookLog = [];
+    isInspectingHooks = false;
     writeDispatcher(dispatcherRef, previousDispatcher);
   }
 
@@ -938,6 +940,7 @@ export const inspectHooks = (
   props: Record<string, unknown>,
   target: ReactDevToolsTarget = globalThis,
 ): HooksTree => {
+  assertNotInspectingHooks();
   const dispatcherRef = requireDispatcherRef(undefined, target);
   getPrimitiveStackCache();
   currentHook = null;
@@ -961,6 +964,7 @@ export const inspectHooks = (
 };
 
 export const getFiberHooks = (fiber: Fiber): HooksTree => {
+  assertNotInspectingHooks();
   const dispatcherRef = requireDispatcherRef(fiber);
   const workTags = getReactWorkTagsForFiber(fiber);
 
@@ -976,6 +980,7 @@ export const getFiberHooks = (fiber: Fiber): HooksTree => {
 
   currentHook = fiber.memoizedState;
   currentFiber = fiber;
+  currentMemoCacheIndex = 0;
 
   const debugThenableState = fiber.dependencies?._debugThenableState;
   const usedThenables = Array.isArray(debugThenableState)
@@ -984,18 +989,16 @@ export const getFiberHooks = (fiber: Fiber): HooksTree => {
   currentThenableState = Array.isArray(usedThenables) ? usedThenables : null;
   currentThenableIndex = 0;
 
-  resolveContextDependency(fiber);
-
-  const type = fiber.type;
-  let props = fiber.memoizedProps;
-  if (type !== fiber.elementType) {
-    props = resolveDefaultProps(type, props);
-  }
-
   const originalConsoleMethods = suppressConsole();
   const contextMap = new Map<ReactContext<unknown>, unknown>();
 
   try {
+    resolveContextDependency(fiber);
+    const type = fiber.type;
+    const props =
+      type !== fiber.elementType
+        ? resolveDefaultProps(type, fiber.memoizedProps)
+        : fiber.memoizedProps;
     if (
       currentContextDependency !== null &&
       !Object.hasOwn(currentContextDependency, "memoizedValue")
@@ -1030,6 +1033,7 @@ export const getFiberHooks = (fiber: Fiber): HooksTree => {
     currentContextDependency = null;
     currentThenableState = null;
     currentThenableIndex = 0;
+    currentMemoCacheIndex = 0;
     restoreContexts(contextMap);
     restoreConsole(originalConsoleMethods);
   }