@buoy-gg/highlight-updates 4.0.1 → 4.0.2

package/lib/commonjs/sync/highlightUpdatesSyncAdapter.js

@@ -6,6 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.highlightUpdatesSyncAdapter = void 0;
 var _HighlightUpdatesController = _interopRequireDefault(require("../highlight-updates/utils/HighlightUpdatesController"));
 var _RenderTracker = require("../highlight-updates/utils/RenderTracker");
+var _screenElements = require("../highlight-updates/utils/screenElements");
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 function ensureInitialized() {
   if (!_HighlightUpdatesController.default.isInitialized()) {
@@ -78,6 +79,14 @@ const highlightUpdatesSyncAdapter = exports.highlightUpdatesSyncAdapter = {
     setSilentTracking: params => {
       ensureInitialized();
       _HighlightUpdatesController.default.setSilentTracking(params.enabled ?? false);
-    }
+    },
+    // Walk the live fiber tree and return the current screen's meaningful /
+    // interactive elements, each with normalized tap coordinates. Powers the
+    // MCP `describe_screen` reader — no screenshot, no silent-tracking needed.
+    describeScreen: () => (0, _screenElements.describeScreen)(),
+    // Tap an element purely in JS: resolve it (by nativeTag / testID / query),
+    // find the nearest onPress up its fiber chain, and invoke it with a
+    // synthetic press event. Powers the MCP `tap_element` action.
+    tapElement: params => (0, _screenElements.tapElement)(params ?? {})
   }
 };

package/lib/module/highlight-updates/utils/HighlightUpdatesController.js

@@ -1034,9 +1034,17 @@ function findScrollAncestor(stateNode) {
   while (fiber && guard++ < 200) {
     const inst = fiber.stateNode;
     if (inst && typeof inst.scrollTo === "function") {
+      // getInnerViewNode()/getScrollableNode() return a numeric node handle —
+      // only valid for the legacy UIManager.measureLayout fallback below.
+      // Fabric's ref.measureLayout requires an actual host-component ref
+      // (getNativeScrollRef()); passing it a number trips a dev warning
+      // ("ref.measureLayout must be called with a ref to a native component")
+      // and silently resolves neither callback.
+      const nativeRef = typeof inst.getNativeScrollRef === "function" && inst.getNativeScrollRef() || null;
       const innerNode = typeof inst.getInnerViewNode === "function" && inst.getInnerViewNode() || typeof inst.getScrollableNode === "function" && inst.getScrollableNode() || inst;
       return {
         instance: inst,
+        nativeRef,
         innerNode
       };
     }
@@ -1083,8 +1091,8 @@ async function scrollDescendantIntoView(stateNode, publicInstance, margin) {
         settle(null);
       }
     });
-    if (typeof inst.measureLayout === "function") {
-      top = await withTimeout(resolve => inst.measureLayout(ancestor.innerNode, (_left, t) => resolve(t), () => resolve(null)));
+    if (ancestor.nativeRef && typeof inst.measureLayout === "function") {
+      top = await withTimeout(resolve => inst.measureLayout(ancestor.nativeRef, (_left, t) => resolve(t), () => resolve(null)));
     }
     if (top == null) {
       const targetHandle = findNodeHandle(publicInstance);