bippy 0.6.1-dev.879f777 → 0.6.1-dev.94e5797

package/src/core.ts

@@ -70,7 +70,7 @@ export const isValidElement = (element: unknown): element is React.ReactElement
 /**
  * Returns `true` if object is a React Fiber.
  */
-export const isValidFiber = (fiber: unknown): fiber is Fiber =>
+export const isFiber = (fiber: unknown): fiber is Fiber =>
   typeof fiber === "object" &&
   fiber !== null &&
   "tag" in fiber &&
@@ -84,7 +84,7 @@ const isFiberRoot = (fiberRoot: unknown): fiberRoot is FiberRoot =>
   typeof fiberRoot === "object" &&
   fiberRoot !== null &&
   "current" in fiberRoot &&
-  isValidFiber(fiberRoot.current);
+  isFiber(fiberRoot.current);
 
 /**
  * Returns `true` if fiber is a host fiber. Host fibers are DOM nodes in react-dom, `View` in react-native, etc.
@@ -123,16 +123,6 @@ export const isCompositeFiber = (fiber: Fiber): boolean => {
   }
 };
 
-/**
- * Returns `true` if the object is a {@link Fiber}
- */
-export const isFiber = (maybeFiber: unknown): maybeFiber is Fiber => {
-  if (!maybeFiber || typeof maybeFiber !== "object") return false;
-  // this is a fast check. pendingProps will ALWAYS exist in fiber
-  // `containerInfo` is in FiberRootNode, not FiberNode
-  return "pendingProps" in maybeFiber && !("containerInfo" in maybeFiber);
-};
-
 /**
  * Returns `true` if the {@link Fiber} has rendered. Note that this does not mean the fiber has rendered in the current commit, just that it has rendered in the past.
  */
@@ -837,14 +827,14 @@ const getLegacyRootFiber = (hostInstance: object): Fiber | null => {
   const current = Reflect.get(internalRoot, "current");
   if (typeof current !== "object" || current === null) return null;
   const child = Reflect.get(current, "child");
-  return isValidFiber(child) ? child : null;
+  return isFiber(child) ? child : null;
 };
 
 const getInternalInstanceHandle = (hostInstance: object): Fiber | null => {
   const internalInstanceHandle =
     Reflect.get(hostInstance, "__internalInstanceHandle") ??
     Reflect.get(hostInstance, "_internalInstanceHandle");
-  return isValidFiber(internalInstanceHandle) ? internalInstanceHandle : null;
+  return isFiber(internalInstanceHandle) ? internalInstanceHandle : null;
 };
 
 const getPublicHostInstance = (stateNode: unknown): unknown => {
@@ -880,14 +870,14 @@ export const getFiber = <T>(hostInstance: T): Fiber | null => {
 
     for (const knownKey of knownFiberPropertyKeys) {
       const fiber = Reflect.get(hostInstance, knownKey);
-      if (isValidFiber(fiber)) return fiber;
+      if (isFiber(fiber)) return fiber;
     }
 
     for (const key of Object.keys(hostInstance)) {
       if (isFiberPropertyKey(key)) {
         knownFiberPropertyKeys.add(key);
         const fiber = Reflect.get(hostInstance, key);
-        if (isValidFiber(fiber)) return fiber;
+        if (isFiber(fiber)) return fiber;
       }
     }
   }

package/src/react.ts

@@ -1,6 +1,6 @@
 import "./install-hook-only.js";
 import * as React from "react";
-import { isValidFiber } from "./core.js";
+import { isFiber } from "./core.js";
 import type { Fiber } from "./react-internals/index.js";
 
 export type { Fiber } from "./react-internals/index.js";
@@ -18,7 +18,7 @@ const captureFiberFromHook = (useCaptureHook: () => void): Fiber | null => {
   const bindProxy = new Proxy(originalBind, {
     apply: (bind, functionToBind, boundArguments) => {
       const fiber = boundArguments[1];
-      if (!capturedFiber && isValidFiber(fiber)) {
+      if (!capturedFiber && isFiber(fiber)) {
         capturedFiber = fiber;
       }
       return Reflect.apply(bind, functionToBind, boundArguments);