@mobileai/react-native 0.9.4 → 0.9.5

package/lib/module/tools/typeTool.js

@@ -3,64 +3,41 @@
 /**
  * Type Tool — Reliable multi-strategy text input.
  *
- * Strategy chain (tried in order, first success wins):
+ * For uncontrolled TextInputs (defaultValue, no onChangeText):
+ *   1. Walk fiber subtree (BFS) to find the native host view whose
+ *      memoizedProps has `onChange` — this is RN's internal `_onChange`.
+ *   2. ALSO walk to find the native stateNode with `setNativeProps` to
+ *      update the VISUAL text in the native view.
  *
- * 1. onChangeText — developer's controlled component wired React state.
- *    → calls element.props.onChangeText(text) directly.
+ * Both steps are needed:
+ *   - `onChange` → updates React's internal lastNativeText state
+ *   - `setNativeProps` → updates what the user sees on screen
  *
- * 2. Fiber subtree walk — for uncontrolled TextInputs (defaultValue only).
- *    React Native's TextInput.js always creates an internal `_onChange` handler
- *    and passes it as `onChange` to the native host view (RCTTextInputView /
- *    AndroidTextInput). That `_onChange` internally calls `props.onChangeText`
- *    and updates native text state.
- *
- *    We walk the fiber subtree (behavior-based, no tag numbers) to find any
- *    descendant node whose memoizedProps.onChange is a function. Then we call
- *    it with a synthetic TextInputChangeEvent. This triggers RN's internal
- *    `_onChange` which calls `setLastNativeText` and updates the native view.
- *
- * Why this is robust:
- *  - No hardcoded fiber tag numbers (host component tags can change across versions)
- *  - No dependency on setNativeProps (deprecated in New Architecture / Fabric)
- *  - Uses the same code path as a real user typing — RN's own _onChange handler
+ * No hardcoded fiber tag numbers. Detection is purely behavior-based.
  */
 
 import { walkFiberTree } from "../core/FiberTreeWalker.js";
-/**
- * Walk a fiber subtree to find the first descendant whose memoizedProps
- * has an `onChange` function. This is behavior-based — we don't check
- * fiber tags, component names, or any React internal that could change.
- * We just look for the prop that RN's TextInput always wires.
- */
-function findNativeOnChange(fiber, maxDepth = 8) {
-  if (!fiber) return null;
-  let current = fiber.child;
-
-  // BFS-style walk through the fiber children
-  const queue = [];
-  if (current) queue.push({
-    node: current,
+/** BFS through fiber children. Returns first node where predicate matches. */
+function findFiberNode(rootFiber, predicate, maxDepth = 10) {
+  if (!rootFiber?.child) return null;
+  const queue = [{
+    node: rootFiber.child,
     depth: 0
-  });
+  }];
   while (queue.length > 0) {
     const {
       node,
-      depth: d
+      depth
     } = queue.shift();
-    if (d > maxDepth) continue;
-    const props = node.memoizedProps;
-    if (props && typeof props.onChange === 'function') {
-      return props.onChange;
-    }
-
-    // Add child and sibling
+    if (!node || depth > maxDepth) continue;
+    if (predicate(node)) return node;
     if (node.child) queue.push({
       node: node.child,
-      depth: d + 1
+      depth: depth + 1
     });
     if (node.sibling) queue.push({
       node: node.sibling,
-      depth: d + 1
+      depth: depth + 1
     });
   }
   return null;
@@ -91,9 +68,9 @@ export function createTypeTool(context) {
       }
       const props = element.props;
       const label = element.label || `[${element.type}]`;
+      const fiberNode = element.fiberNode;
 
       // ── Strategy 1: controlled via onChangeText ───────────────────────────
-      // App developer wired onChangeText={setText} — most reliable.
       if (typeof props.onChangeText === 'function') {
         try {
           props.onChangeText(args.text);
@@ -104,33 +81,46 @@ export function createTypeTool(context) {
         }
       }
 
-      // ── Strategy 2: fiber subtree walk for native onChange ─────────────────
-      // For uncontrolled TextInputs (defaultValue only, no onChangeText).
-      // RN's TextInput.js creates _onChange internally and passes it as
-      // onChange to the native host view. We find that handler and call it
-      // with a synthetic TextInputChangeEvent.
-      const fiberNode = element.fiberNode;
+      // ── Strategy 2: uncontrolled — find native onChange + setNativeProps ──
+      // For TextInputs with defaultValue only.
+      // We need BOTH:
+      //   a) find the onChange handler (RN's internal _onChange) to update React state
+      //   b) find the native stateNode to call setNativeProps to update visual display
       if (fiberNode) {
-        // First check the matched fiber itself
-        const directOnChange = fiberNode.memoizedProps?.onChange;
-        const onChange = typeof directOnChange === 'function' ? directOnChange : findNativeOnChange(fiberNode);
-        if (onChange) {
+        // Find native onChange handler (behavior-based, no tag numbers)
+        const nativeOnChangeFiber = fiberNode.memoizedProps?.onChange ? fiberNode : findFiberNode(fiberNode, n => typeof n.memoizedProps?.onChange === 'function');
+
+        // Find native stateNode (host component with setNativeProps)
+        const nativeStateFiber = fiberNode.stateNode?.setNativeProps ? fiberNode : findFiberNode(fiberNode, n => n.stateNode && typeof n.stateNode.setNativeProps === 'function');
+        const onChange = nativeOnChangeFiber?.memoizedProps?.onChange;
+        const nativeInstance = nativeStateFiber?.stateNode;
+        if (onChange || nativeInstance) {
           try {
-            onChange({
-              nativeEvent: {
-                text: args.text,
-                eventCount: 0,
-                target: 0
-              }
-            });
+            // Step 1: Update visual text in native view
+            if (nativeInstance && typeof nativeInstance.setNativeProps === 'function') {
+              nativeInstance.setNativeProps({
+                text: args.text
+              });
+            }
+
+            // Step 2: Notify React's internal onChange so lastNativeText stays in sync
+            if (typeof onChange === 'function') {
+              onChange({
+                nativeEvent: {
+                  text: args.text,
+                  eventCount: 1,
+                  target: 0
+                }
+              });
+            }
             await new Promise(resolve => setTimeout(resolve, 300));
             return `✅ Typed "${args.text}" into [${args.index}] "${label}"`;
           } catch (err) {
-            return `❌ onChange failed: ${err.message}`;
+            return `❌ Type failed: ${err.message}`;
           }
         }
       }
-      return `❌ Element [${args.index}] "${label}" is not a typeable text input. No onChangeText or onChange handler found in fiber tree.`;
+      return `❌ Element [${args.index}] "${label}" is not a typeable text input. No onChange or native stateNode found in fiber tree.`;
     }
   };
 }

package/lib/module/tools/typeTool.js.map

@@ -1 +1 @@
-{"version":3,"names":["walkFiberTree","findNativeOnChange","fiber","maxDepth","current","child","queue","push","node","depth","length","d","shift","props","memoizedProps","onChange","sibling","createTypeTool","context","name","description","parameters","index","type","required","text","execute","args","interactives","elements","getRootRef","getWalkConfig","element","find","el","label","onChangeText","Promise","resolve","setTimeout","err","message","fiberNode","directOnChange","nativeEvent","eventCount","target"],"sourceRoot":"../../../src","sources":["tools/typeTool.ts"],"mappings":";;AAAA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;;AAEA,SAASA,aAAa,QAAQ,4BAAyB;AAGvD;AACA;AACA;AACA;AACA;AACA;AACA,SAASC,kBAAkBA,CAACC,KAAU,EAAEC,QAAQ,GAAG,CAAC,EAAiC;EACnF,IAAI,CAACD,KAAK,EAAE,OAAO,IAAI;EAEvB,IAAIE,OAAY,GAAGF,KAAK,CAACG,KAAK;;EAE9B;EACA,MAAMC,KAAqC,GAAG,EAAE;EAChD,IAAIF,OAAO,EAAEE,KAAK,CAACC,IAAI,CAAC;IAAEC,IAAI,EAAEJ,OAAO;IAAEK,KAAK,EAAE;EAAE,CAAC,CAAC;EAEpD,OAAOH,KAAK,CAACI,MAAM,GAAG,CAAC,EAAE;IACvB,MAAM;MAAEF,IAAI;MAAEC,KAAK,EAAEE;IAAE,CAAC,GAAGL,KAAK,CAACM,KAAK,CAAC,CAAE;IACzC,IAAID,CAAC,GAAGR,QAAQ,EAAE;IAElB,MAAMU,KAAK,GAAGL,IAAI,CAACM,aAAa;IAChC,IAAID,KAAK,IAAI,OAAOA,KAAK,CAACE,QAAQ,KAAK,UAAU,EAAE;MACjD,OAAOF,KAAK,CAACE,QAAQ;IACvB;;IAEA;IACA,IAAIP,IAAI,CAACH,KAAK,EAAEC,KAAK,CAACC,IAAI,CAAC;MAAEC,IAAI,EAAEA,IAAI,CAACH,KAAK;MAAEI,KAAK,EAAEE,CAAC,GAAG;IAAE,CAAC,CAAC;IAC9D,IAAIH,IAAI,CAACQ,OAAO,EAAEV,KAAK,CAACC,IAAI,CAAC;MAAEC,IAAI,EAAEA,IAAI,CAACQ,OAAO;MAAEP,KAAK,EAAEE,CAAC,GAAG;IAAE,CAAC,CAAC;EACpE;EAEA,OAAO,IAAI;AACb;AAEA,OAAO,SAASM,cAAcA,CAACC,OAAoB,EAAa;EAC9D,OAAO;IACLC,IAAI,EAAE,MAAM;IACZC,WAAW,EAAE,mDAAmD;IAChEC,UAAU,EAAE;MACVC,KAAK,EAAE;QAAEC,IAAI,EAAE,QAAQ;QAAEH,WAAW,EAAE,qCAAqC;QAAEI,QAAQ,EAAE;MAAK,CAAC;MAC7FC,IAAI,EAAE;QAAEF,IAAI,EAAE,QAAQ;QAAEH,WAAW,EAAE,kBAAkB;QAAEI,QAAQ,EAAE;MAAK;IAC1E,CAAC;IACDE,OAAO,EAAE,MAAOC,IAAI,IAAK;MACvB,MAAM;QAAEC,YAAY,EAAEC;MAAS,CAAC,GAAG7B,aAAa,CAACkB,OAAO,CAACY,UAAU,CAAC,CAAC,EAAEZ,OAAO,CAACa,aAAa,CAAC,CAAC,CAAC;MAC/F,MAAMC,OAAO,GAAGH,QAAQ,CAACI,IAAI,CAACC,EAAE,IAAIA,EAAE,CAACZ,KAAK,KAAKK,IAAI,CAACL,KAAK,CAAC;MAE5D,IAAI,CAACU,OAAO,EAAE;QACZ,OAAO,wBAAwBL,IAAI,CAACL,KAAK,aAAa;MACxD;MAEA,MAAMT,KAAK,GAAGmB,OAAO,CAACnB,KAAK;MAC3B,MAAMsB,KAAK,GAAGH,OAAO,CAACG,KAAK,IAAI,IAAIH,OAAO,CAACT,IAAI,GAAG;;MAElD;MACA;MACA,IAAI,OAAOV,KAAK,CAACuB,YAAY,KAAK,UAAU,EAAE;QAC5C,IAAI;UACFvB,KAAK,CAACuB,YAAY,CAACT,IAAI,CAACF,IAAI,CAAC;UAC7B,MAAM,IAAIY,OAAO,CAACC,OAAO,IAAIC,UAAU,CAACD,OAAO,EAAE,GAAG,CAAC,CAAC;UACtD,OAAO,YAAYX,IAAI,CAACF,IAAI,WAAWE,IAAI,CAACL,KAAK,MAAMa,KAAK,GAAG;QACjE,CAAC,CAAC,OAAOK,GAAQ,EAAE;UACjB,OAAO,0BAA0BA,GAAG,CAACC,OAAO,EAAE;QAChD;MACF;;MAEA;MACA;MACA;MACA;MACA;MACA,MAAMC,SAAS,GAAGV,OAAO,CAACU,SAAS;MACnC,IAAIA,SAAS,EAAE;QACb;QACA,MAAMC,cAAc,GAAGD,SAAS,CAAC5B,aAAa,EAAEC,QAAQ;QACxD,MAAMA,QAAQ,GAAG,OAAO4B,cAAc,KAAK,UAAU,GACjDA,cAAc,GACd1C,kBAAkB,CAACyC,SAAS,CAAC;QAEjC,IAAI3B,QAAQ,EAAE;UACZ,IAAI;YACFA,QAAQ,CAAC;cACP6B,WAAW,EAAE;gBACXnB,IAAI,EAAEE,IAAI,CAACF,IAAI;gBACfoB,UAAU,EAAE,CAAC;gBACbC,MAAM,EAAE;cACV;YACF,CAAC,CAAC;YACF,MAAM,IAAIT,OAAO,CAACC,OAAO,IAAIC,UAAU,CAACD,OAAO,EAAE,GAAG,CAAC,CAAC;YACtD,OAAO,YAAYX,IAAI,CAACF,IAAI,WAAWE,IAAI,CAACL,KAAK,MAAMa,KAAK,GAAG;UACjE,CAAC,CAAC,OAAOK,GAAQ,EAAE;YACjB,OAAO,sBAAsBA,GAAG,CAACC,OAAO,EAAE;UAC5C;QACF;MACF;MAEA,OAAO,cAAcd,IAAI,CAACL,KAAK,MAAMa,KAAK,0FAA0F;IACtI;EACF,CAAC;AACH","ignoreList":[]}
+{"version":3,"names":["walkFiberTree","findFiberNode","rootFiber","predicate","maxDepth","child","queue","node","depth","length","shift","push","sibling","createTypeTool","context","name","description","parameters","index","type","required","text","execute","args","interactives","elements","getRootRef","getWalkConfig","element","find","el","props","label","fiberNode","onChangeText","Promise","resolve","setTimeout","err","message","nativeOnChangeFiber","memoizedProps","onChange","n","nativeStateFiber","stateNode","setNativeProps","nativeInstance","nativeEvent","eventCount","target"],"sourceRoot":"../../../src","sources":["tools/typeTool.ts"],"mappings":";;AAAA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;;AAEA,SAASA,aAAa,QAAQ,4BAAyB;AAGvD;AACA,SAASC,aAAaA,CAACC,SAAc,EAAEC,SAAiC,EAAEC,QAAQ,GAAG,EAAE,EAAc;EACnG,IAAI,CAACF,SAAS,EAAEG,KAAK,EAAE,OAAO,IAAI;EAClC,MAAMC,KAAqC,GAAG,CAAC;IAAEC,IAAI,EAAEL,SAAS,CAACG,KAAK;IAAEG,KAAK,EAAE;EAAE,CAAC,CAAC;EACnF,OAAOF,KAAK,CAACG,MAAM,GAAG,CAAC,EAAE;IACvB,MAAM;MAAEF,IAAI;MAAEC;IAAM,CAAC,GAAGF,KAAK,CAACI,KAAK,CAAC,CAAE;IACtC,IAAI,CAACH,IAAI,IAAIC,KAAK,GAAGJ,QAAQ,EAAE;IAC/B,IAAID,SAAS,CAACI,IAAI,CAAC,EAAE,OAAOA,IAAI;IAChC,IAAIA,IAAI,CAACF,KAAK,EAAEC,KAAK,CAACK,IAAI,CAAC;MAAEJ,IAAI,EAAEA,IAAI,CAACF,KAAK;MAAEG,KAAK,EAAEA,KAAK,GAAG;IAAE,CAAC,CAAC;IAClE,IAAID,IAAI,CAACK,OAAO,EAAEN,KAAK,CAACK,IAAI,CAAC;MAAEJ,IAAI,EAAEA,IAAI,CAACK,OAAO;MAAEJ,KAAK,EAAEA,KAAK,GAAG;IAAE,CAAC,CAAC;EACxE;EACA,OAAO,IAAI;AACb;AAEA,OAAO,SAASK,cAAcA,CAACC,OAAoB,EAAa;EAC9D,OAAO;IACLC,IAAI,EAAE,MAAM;IACZC,WAAW,EAAE,mDAAmD;IAChEC,UAAU,EAAE;MACVC,KAAK,EAAE;QAAEC,IAAI,EAAE,QAAQ;QAAEH,WAAW,EAAE,qCAAqC;QAAEI,QAAQ,EAAE;MAAK,CAAC;MAC7FC,IAAI,EAAE;QAAEF,IAAI,EAAE,QAAQ;QAAEH,WAAW,EAAE,kBAAkB;QAAEI,QAAQ,EAAE;MAAK;IAC1E,CAAC;IACDE,OAAO,EAAE,MAAOC,IAAI,IAAK;MACvB,MAAM;QAAEC,YAAY,EAAEC;MAAS,CAAC,GAAGzB,aAAa,CAACc,OAAO,CAACY,UAAU,CAAC,CAAC,EAAEZ,OAAO,CAACa,aAAa,CAAC,CAAC,CAAC;MAC/F,MAAMC,OAAO,GAAGH,QAAQ,CAACI,IAAI,CAACC,EAAE,IAAIA,EAAE,CAACZ,KAAK,KAAKK,IAAI,CAACL,KAAK,CAAC;MAE5D,IAAI,CAACU,OAAO,EAAE;QACZ,OAAO,wBAAwBL,IAAI,CAACL,KAAK,aAAa;MACxD;MAEA,MAAMa,KAAK,GAAGH,OAAO,CAACG,KAAK;MAC3B,MAAMC,KAAK,GAAGJ,OAAO,CAACI,KAAK,IAAI,IAAIJ,OAAO,CAACT,IAAI,GAAG;MAClD,MAAMc,SAAS,GAAGL,OAAO,CAACK,SAAS;;MAEnC;MACA,IAAI,OAAOF,KAAK,CAACG,YAAY,KAAK,UAAU,EAAE;QAC5C,IAAI;UACFH,KAAK,CAACG,YAAY,CAACX,IAAI,CAACF,IAAI,CAAC;UAC7B,MAAM,IAAIc,OAAO,CAACC,OAAO,IAAIC,UAAU,CAACD,OAAO,EAAE,GAAG,CAAC,CAAC;UACtD,OAAO,YAAYb,IAAI,CAACF,IAAI,WAAWE,IAAI,CAACL,KAAK,MAAMc,KAAK,GAAG;QACjE,CAAC,CAAC,OAAOM,GAAQ,EAAE;UACjB,OAAO,0BAA0BA,GAAG,CAACC,OAAO,EAAE;QAChD;MACF;;MAEA;MACA;MACA;MACA;MACA;MACA,IAAIN,SAAS,EAAE;QACb;QACA,MAAMO,mBAAmB,GAAGP,SAAS,CAACQ,aAAa,EAAEC,QAAQ,GACzDT,SAAS,GACThC,aAAa,CAACgC,SAAS,EAAEU,CAAC,IAAI,OAAOA,CAAC,CAACF,aAAa,EAAEC,QAAQ,KAAK,UAAU,CAAC;;QAElF;QACA,MAAME,gBAAgB,GAAGX,SAAS,CAACY,SAAS,EAAEC,cAAc,GACxDb,SAAS,GACThC,aAAa,CAACgC,SAAS,EAAEU,CAAC,IAAIA,CAAC,CAACE,SAAS,IAAI,OAAOF,CAAC,CAACE,SAAS,CAACC,cAAc,KAAK,UAAU,CAAC;QAElG,MAAMJ,QAAQ,GAAGF,mBAAmB,EAAEC,aAAa,EAAEC,QAAQ;QAC7D,MAAMK,cAAc,GAAGH,gBAAgB,EAAEC,SAAS;QAElD,IAAIH,QAAQ,IAAIK,cAAc,EAAE;UAC9B,IAAI;YACF;YACA,IAAIA,cAAc,IAAI,OAAOA,cAAc,CAACD,cAAc,KAAK,UAAU,EAAE;cACzEC,cAAc,CAACD,cAAc,CAAC;gBAAEzB,IAAI,EAAEE,IAAI,CAACF;cAAK,CAAC,CAAC;YACpD;;YAEA;YACA,IAAI,OAAOqB,QAAQ,KAAK,UAAU,EAAE;cAClCA,QAAQ,CAAC;gBACPM,WAAW,EAAE;kBACX3B,IAAI,EAAEE,IAAI,CAACF,IAAI;kBACf4B,UAAU,EAAE,CAAC;kBACbC,MAAM,EAAE;gBACV;cACF,CAAC,CAAC;YACJ;YAEA,MAAM,IAAIf,OAAO,CAACC,OAAO,IAAIC,UAAU,CAACD,OAAO,EAAE,GAAG,CAAC,CAAC;YACtD,OAAO,YAAYb,IAAI,CAACF,IAAI,WAAWE,IAAI,CAACL,KAAK,MAAMc,KAAK,GAAG;UACjE,CAAC,CAAC,OAAOM,GAAQ,EAAE;YACjB,OAAO,kBAAkBA,GAAG,CAACC,OAAO,EAAE;UACxC;QACF;MACF;MAEA,OAAO,cAAchB,IAAI,CAACL,KAAK,MAAMc,KAAK,sFAAsF;IAClI;EACF,CAAC;AACH","ignoreList":[]}

package/lib/typescript/src/tools/typeTool.d.ts

@@ -1,26 +1,17 @@
 /**
  * Type Tool — Reliable multi-strategy text input.
  *
- * Strategy chain (tried in order, first success wins):
+ * For uncontrolled TextInputs (defaultValue, no onChangeText):
+ *   1. Walk fiber subtree (BFS) to find the native host view whose
+ *      memoizedProps has `onChange` — this is RN's internal `_onChange`.
+ *   2. ALSO walk to find the native stateNode with `setNativeProps` to
+ *      update the VISUAL text in the native view.
  *
- * 1. onChangeText — developer's controlled component wired React state.
- *    → calls element.props.onChangeText(text) directly.
+ * Both steps are needed:
+ *   - `onChange` → updates React's internal lastNativeText state
+ *   - `setNativeProps` → updates what the user sees on screen
  *
- * 2. Fiber subtree walk — for uncontrolled TextInputs (defaultValue only).
- *    React Native's TextInput.js always creates an internal `_onChange` handler
- *    and passes it as `onChange` to the native host view (RCTTextInputView /
- *    AndroidTextInput). That `_onChange` internally calls `props.onChangeText`
- *    and updates native text state.
- *
- *    We walk the fiber subtree (behavior-based, no tag numbers) to find any
- *    descendant node whose memoizedProps.onChange is a function. Then we call
- *    it with a synthetic TextInputChangeEvent. This triggers RN's internal
- *    `_onChange` which calls `setLastNativeText` and updates the native view.
- *
- * Why this is robust:
- *  - No hardcoded fiber tag numbers (host component tags can change across versions)
- *  - No dependency on setNativeProps (deprecated in New Architecture / Fabric)
- *  - Uses the same code path as a real user typing — RN's own _onChange handler
+ * No hardcoded fiber tag numbers. Detection is purely behavior-based.
  */
 import type { AgentTool, ToolContext } from './types';
 export declare function createTypeTool(context: ToolContext): AgentTool;

package/lib/typescript/src/tools/typeTool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"typeTool.d.ts","sourceRoot":"","sources":["../../../../src/tools/typeTool.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AAGH,OAAO,KAAK,EAAE,SAAS,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AAkCtD,wBAAgB,cAAc,CAAC,OAAO,EAAE,WAAW,GAAG,SAAS,CAgE9D"}
+{"version":3,"file":"typeTool.d.ts","sourceRoot":"","sources":["../../../../src/tools/typeTool.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAGH,OAAO,KAAK,EAAE,SAAS,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AAgBtD,wBAAgB,cAAc,CAAC,OAAO,EAAE,WAAW,GAAG,SAAS,CA+E9D"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mobileai/react-native",
-  "version": "0.9.4",
+  "version": "0.9.5",
   "description": "Build autonomous AI agents for React Native and Expo apps. Provides AI-native UI traversal, tool calling, and structured reasoning.",
   "main": "./lib/module/index.js",
   "source": "./src/index.ts",

package/src/tools/typeTool.ts

@@ -1,60 +1,33 @@
 /**
  * Type Tool — Reliable multi-strategy text input.
  *
- * Strategy chain (tried in order, first success wins):
+ * For uncontrolled TextInputs (defaultValue, no onChangeText):
+ *   1. Walk fiber subtree (BFS) to find the native host view whose
+ *      memoizedProps has `onChange` — this is RN's internal `_onChange`.
+ *   2. ALSO walk to find the native stateNode with `setNativeProps` to
+ *      update the VISUAL text in the native view.
  *
- * 1. onChangeText — developer's controlled component wired React state.
- *    → calls element.props.onChangeText(text) directly.
+ * Both steps are needed:
+ *   - `onChange` → updates React's internal lastNativeText state
+ *   - `setNativeProps` → updates what the user sees on screen
  *
- * 2. Fiber subtree walk — for uncontrolled TextInputs (defaultValue only).
- *    React Native's TextInput.js always creates an internal `_onChange` handler
- *    and passes it as `onChange` to the native host view (RCTTextInputView /
- *    AndroidTextInput). That `_onChange` internally calls `props.onChangeText`
- *    and updates native text state.
- *
- *    We walk the fiber subtree (behavior-based, no tag numbers) to find any
- *    descendant node whose memoizedProps.onChange is a function. Then we call
- *    it with a synthetic TextInputChangeEvent. This triggers RN's internal
- *    `_onChange` which calls `setLastNativeText` and updates the native view.
- *
- * Why this is robust:
- *  - No hardcoded fiber tag numbers (host component tags can change across versions)
- *  - No dependency on setNativeProps (deprecated in New Architecture / Fabric)
- *  - Uses the same code path as a real user typing — RN's own _onChange handler
+ * No hardcoded fiber tag numbers. Detection is purely behavior-based.
  */
 
 import { walkFiberTree } from '../core/FiberTreeWalker';
 import type { AgentTool, ToolContext } from './types';
 
-/**
- * Walk a fiber subtree to find the first descendant whose memoizedProps
- * has an `onChange` function. This is behavior-based — we don't check
- * fiber tags, component names, or any React internal that could change.
- * We just look for the prop that RN's TextInput always wires.
- */
-function findNativeOnChange(fiber: any, maxDepth = 8): ((event: any) => void) | null {
-  if (!fiber) return null;
-
-  let current: any = fiber.child;
-
-  // BFS-style walk through the fiber children
-  const queue: { node: any; depth: number }[] = [];
-  if (current) queue.push({ node: current, depth: 0 });
-
+/** BFS through fiber children. Returns first node where predicate matches. */
+function findFiberNode(rootFiber: any, predicate: (node: any) => boolean, maxDepth = 10): any | null {
+  if (!rootFiber?.child) return null;
+  const queue: { node: any; depth: number }[] = [{ node: rootFiber.child, depth: 0 }];
   while (queue.length > 0) {
-    const { node, depth: d } = queue.shift()!;
-    if (d > maxDepth) continue;
-
-    const props = node.memoizedProps;
-    if (props && typeof props.onChange === 'function') {
-      return props.onChange;
-    }
-
-    // Add child and sibling
-    if (node.child) queue.push({ node: node.child, depth: d + 1 });
-    if (node.sibling) queue.push({ node: node.sibling, depth: d + 1 });
+    const { node, depth } = queue.shift()!;
+    if (!node || depth > maxDepth) continue;
+    if (predicate(node)) return node;
+    if (node.child) queue.push({ node: node.child, depth: depth + 1 });
+    if (node.sibling) queue.push({ node: node.sibling, depth: depth + 1 });
   }
-
   return null;
 }
 
@@ -76,9 +49,9 @@ export function createTypeTool(context: ToolContext): AgentTool {
 
       const props = element.props;
       const label = element.label || `[${element.type}]`;
+      const fiberNode = element.fiberNode;
 
       // ── Strategy 1: controlled via onChangeText ───────────────────────────
-      // App developer wired onChangeText={setText} — most reliable.
       if (typeof props.onChangeText === 'function') {
         try {
           props.onChangeText(args.text);
@@ -89,37 +62,52 @@ export function createTypeTool(context: ToolContext): AgentTool {
         }
       }
 
-      // ── Strategy 2: fiber subtree walk for native onChange ─────────────────
-      // For uncontrolled TextInputs (defaultValue only, no onChangeText).
-      // RN's TextInput.js creates _onChange internally and passes it as
-      // onChange to the native host view. We find that handler and call it
-      // with a synthetic TextInputChangeEvent.
-      const fiberNode = element.fiberNode;
+      // ── Strategy 2: uncontrolled — find native onChange + setNativeProps ──
+      // For TextInputs with defaultValue only.
+      // We need BOTH:
+      //   a) find the onChange handler (RN's internal _onChange) to update React state
+      //   b) find the native stateNode to call setNativeProps to update visual display
       if (fiberNode) {
-        // First check the matched fiber itself
-        const directOnChange = fiberNode.memoizedProps?.onChange;
-        const onChange = typeof directOnChange === 'function'
-          ? directOnChange
-          : findNativeOnChange(fiberNode);
+        // Find native onChange handler (behavior-based, no tag numbers)
+        const nativeOnChangeFiber = fiberNode.memoizedProps?.onChange
+          ? fiberNode
+          : findFiberNode(fiberNode, n => typeof n.memoizedProps?.onChange === 'function');
 
-        if (onChange) {
+        // Find native stateNode (host component with setNativeProps)
+        const nativeStateFiber = fiberNode.stateNode?.setNativeProps
+          ? fiberNode
+          : findFiberNode(fiberNode, n => n.stateNode && typeof n.stateNode.setNativeProps === 'function');
+
+        const onChange = nativeOnChangeFiber?.memoizedProps?.onChange;
+        const nativeInstance = nativeStateFiber?.stateNode;
+
+        if (onChange || nativeInstance) {
           try {
-            onChange({
-              nativeEvent: {
-                text: args.text,
-                eventCount: 0,
-                target: 0,
-              },
-            });
+            // Step 1: Update visual text in native view
+            if (nativeInstance && typeof nativeInstance.setNativeProps === 'function') {
+              nativeInstance.setNativeProps({ text: args.text });
+            }
+
+            // Step 2: Notify React's internal onChange so lastNativeText stays in sync
+            if (typeof onChange === 'function') {
+              onChange({
+                nativeEvent: {
+                  text: args.text,
+                  eventCount: 1,
+                  target: 0,
+                },
+              });
+            }
+
             await new Promise(resolve => setTimeout(resolve, 300));
             return `✅ Typed "${args.text}" into [${args.index}] "${label}"`;
           } catch (err: any) {
-            return `❌ onChange failed: ${err.message}`;
+            return `❌ Type failed: ${err.message}`;
           }
         }
       }
 
-      return `❌ Element [${args.index}] "${label}" is not a typeable text input. No onChangeText or onChange handler found in fiber tree.`;
+      return `❌ Element [${args.index}] "${label}" is not a typeable text input. No onChange or native stateNode found in fiber tree.`;
     },
   };
 }