@simon-says-build/demo-player 0.14.2 → 0.14.4

package/index.js

@@ -1,3 +1,3 @@
 export { DemoHost } from './src/DemoHost.js';
-export { demoConfig, findTarget, handlerFor, phoneSteps, scalePoint, playDemo, currentDemo, onDemo, reviewChangesOf, releaseOf, releaseReviewsOf, PLAYER_VERSION, tuckedDemo, announceOverlay } from './src/core.js';
+export { demoConfig, findTarget, handlerFor, phoneSteps, scalePoint, playDemo, currentDemo, onDemo, reviewChangesOf, releaseOf, releaseReviewsOf, PLAYER_VERSION, tuckedDemo, announceOverlay, setCurrentRoute, currentRoute } from './src/core.js';
 export { navigationRoute, navigationRouter, navigateArgs, pathOfState } from './src/navigation.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@simon-says-build/demo-player",
-  "version": "0.14.2",
+  "version": "0.14.4",
   "description": "SimonSays mobile demo player: walks a preview's viewer through a job's change, inside the app. Renders nothing unless the bundle carries a demo.",
   "repository": {
     "type": "git",

package/src/DemoHost.js

@@ -23,7 +23,7 @@ import { pageName } from './page-name.js';
 // app (box-none). An app without react-native-screens keeps the in-tree layer.
 let FullWindowOverlay = null;
 try { FullWindowOverlay = require('react-native-screens').FullWindowOverlay || null; } catch { FullWindowOverlay = null; }
-import { OVERLAY_ID, overlayLayer, chapterOf, cardBox, pinnedBox, setTuckedDemo, overlayAnnounced, tapInside, currentDemo, demoConfig, findTarget, handlerFor, introBody, introOf, onDemo, phoneSteps, playDemo, releaseOf, reviewChangesOf, scalePoint, rescueNote, screenCandidates, stepTitle, targetOfKey } from './core.js';
+import { OVERLAY_ID, overlayLayer, pressableHost, rowBox, alreadyOn, currentRoute, chapterOf, cardBox, pinnedBox, setTuckedDemo, overlayAnnounced, tapInside, currentDemo, demoConfig, findTarget, handlerFor, introBody, introOf, onDemo, phoneSteps, playDemo, releaseOf, reviewChangesOf, scalePoint, rescueNote, screenCandidates, stepTitle, targetOfKey } from './core.js';
 import { demoApi, tourOf } from './api.js';
 import { signInFirst, openInApp } from './signin.js';
 import { runEvent, fillStep } from './events.js';
@@ -157,7 +157,8 @@ function Player({ config, router, insets, onClose, children }) {
     const started = Date.now();
     const tick = async () => {
       const f = findTarget(fiberOf(rootRef.current), target);
-      const r = f ? await measure(f) : null;
+      const host = f ? pressableHost(f) : null;
+      const r = f ? rowBox(await measure(f), host !== f ? await measure(host) : null, screen) : null;
       if (!live) return;
       setFound(f);
       setRect(r);
@@ -217,6 +218,16 @@ function Player({ config, router, insets, onClose, children }) {
     advance();
   }, [steps, step, router, advance, flash]);
 
+  // A GOTO TO WHERE THE PERSON ALREADY IS (0.14.4) is passed straight over —
+  // only when arriving forward, never after Back (Back onto it must stay, or
+  // it would bounce forward again).
+  const lastIndex = useRef(-1);
+  useEffect(() => {
+    const forward = index > lastIndex.current;
+    lastIndex.current = index;
+    if (forward && step?.kind === 'goto' && step.path && !flash && alreadyOn(step.path, currentRoute())) advance();
+  }, [index]);
+
   useEffect(() => {
     if (!tried) return undefined;
     setFlash('✓ That\'s it');

package/src/core.js

@@ -10,7 +10,7 @@ import { bubbleBox } from './step-place.js';
 // What this player is, said when the app registers for "Ready to see"
 // (packages/ssb-overlay sends it; functions/in-app-notify.mjs clientOf). Kept
 // equal to package.json by test/core.test.mjs.
-export const PLAYER_VERSION = '0.14.2';
+export const PLAYER_VERSION = '0.14.4';
 
 export const HOST = 5;
 export const TEXT = 6;
@@ -144,6 +144,26 @@ export function handlerFor(f, name) {
   return null;
 }
 
+/**
+ * THE THING A PERSON TAPS (0.14.3, 2026-11-07): a target found by its words is
+ * often a Text inside a pressable row. Ringing the Text put a small ring on a
+ * label at the row's edge, and a tap anywhere else on the row — what a person
+ * and SSB's check both do — was not seen as the try: Connie Ho's "Open Yane"
+ * opened her profile and the card kept asking (demo 16D4S5HqugIonvUSdC5u).
+ * → the first host view of the nearest ancestor with an onPress (the row), or
+ * `f` when there is none. Pure (walks fibers: return / child, tag 5 = host).
+ */
+export function pressableHost(f) {
+  let p = f;
+  while (p && typeof p.memoizedProps?.onPress !== 'function') p = p.return;
+  if (!p) return f;
+  for (let q = p; q; q = q.child) if (q.tag === 5 && q.stateNode) return q;
+  return f;
+}
+/** Use the row's box only when it is a row, not a screen-sized wrapper. Pure. */
+export const rowBox = (text, row, screen) =>
+  row && text && row.width * row.height <= 0.4 * screen.width * screen.height ? row : text;
+
 /** A recorded point, scaled from the recording screen to `screen`. */
 export function scalePoint(point, screen, from = RECORDED_SCREEN) {
   return {
@@ -205,6 +225,21 @@ export function cardBox(place, at, screen, insets, card) {
 // pinned at the TOP stays left of it; one pinned at the bottom is full width.
 export const DEV_FAB_CLEAR = 88;
 
+// THE SCREEN THE PERSON IS ON (0.14.4, 2026-11-07): the in-app overlay knows
+// it (expo-router's usePathname, passed to <SsbHost route>) and tells the
+// player, so a `goto` to where the person already is isn't offered as a
+// destination ("Your walks. Take me there" on the Walks screen — Michael).
+let routeNow = null;
+export const setCurrentRoute = (route) => { routeNow = typeof route === 'string' ? route : null; };
+export const currentRoute = () => routeNow;
+/** "/(app)/(tabs)/walks/" → "/walks": expo-router's groups, a query and trailing slashes are not the screen. Pure. */
+export const screenOf = (path) => {
+  const p = String(path || '').split(/[?#]/)[0].split('/').filter((s) => s && !/^\(.*\)$/.test(s)).join('/');
+  return `/${p}`;
+};
+/** Is the person already on the screen this path opens? Unknown (no route told) is no. Pure. */
+export const alreadyOn = (path, route) => typeof route === 'string' && !!path && screenOf(path) === screenOf(route);
+
 // Where the overlay draws (0.14.2): iOS with react-native-screens → its own
 // window above sheets and modals (DemoHost, FullWindowOverlay); else in-tree.
 export const overlayLayer = (os, fwo) => (os === 'ios' && fwo ? 'window' : 'tree');