@symbiote-native/engine 1.2.0 → 1.3.1

package/build/surface.js

@@ -1,15 +1,5 @@
 // A surface is one mounted root: it owns the rootTag handed down by the native Fabric host and the
 // top-level nodes under it. Adapters mutate it and ask it to commit.
-//
-// A SURFACE IS ONE ORDINARY NODE, and that is the whole implementation rather than a trick. It holds
-// a real handle, its child ops are the ordinary append / insertBefore / removeChild, and `OP_COMMIT`
-// names that one handle — so the host needs no `rootTag -> node` map, which is what keeps it
-// stateless.
-//
-// The node is RN's AppContainer (`createSurfaceRoot` in `node.ts` — `flex: 1`, `box-none`), so what
-// reaches the root child set is one view with the app under it. An ANCHOR in the same position
-// hoists its children instead, and the host takes both through the same call, so nothing here is a
-// special case.
 import { dlog } from './debug.js';
 import { installEventHandler } from './events/index.js';
 import { detachAnimatedProps } from './animated/host-binding.js';
@@ -33,17 +23,10 @@ export class SymbioteSurface {
         this.rootTag = rootTag;
         this.node = createSurfaceRoot();
     }
-    /**
-     * Every OTHER live surface, so one commit names every root — see `commitSurfaceOps`.
-     *
-     * Static because it reads a sibling instance's private handle, which only a member of this class
-     * may do. One surface — the universal case — allocates nothing.
-     *
-     * `self` may already be OUT of the registry: a teardown unregisters and then commits, and that
-     * commit is the one carrying the removals. So the fast path cannot be a size check — with one
-     * live surface left, `size === 1` means either "only me" or "only the other one", and taking the
-     * shortcut on the second reading is how a surface loses the very batch that empties it.
-     */
+    // Every other live surface, so one commit names every root — see commitSurfaceOps. Static
+    // because it reads a sibling instance's private handle, which only a member of this class may do.
+    // self may already be out of the registry: a teardown unregisters and then commits, carrying the
+    // removals — so the fast path can't be a size check, since size===1 could mean either surface.
     static others(self) {
         let out;
         for (const other of surfaces.values()) {
@@ -54,13 +37,8 @@ export class SymbioteSurface {
         }
         return out ?? NO_CO_COMMITTERS;
     }
-    /**
-     * The top-level nodes, asked of the host.
-     *
-     * A getter rather than an array this class maintains: a second copy of a child list is the thing
-     * this design removes, and `nextSiblingOf` (host-access.ts) needs the same answer the host gives
-     * for a parented node.
-     */
+    // The top-level nodes, asked of the host. A getter rather than an array this class maintains: a
+    // second copy of a child list is the thing this design removes.
     get children() {
         return childrenOf(this.node);
     }
@@ -70,10 +48,8 @@ export class SymbioteSurface {
     insertBefore(child, beforeChild) {
         insertBefore(this.node, child, beforeChild);
     }
-    // Nomination for teardown rides on `node.ts`'s `removeChild`, and only NOMINATES for the reason
-    // stated there: a framework may spell a move as remove-then-reinsert, so the commit sweep
-    // decides. The surface is one ordinary node, so it is not a second removal path that could miss
-    // the sweep and leave a behavior — its timers included — outliving the surface.
+    // Nomination for teardown rides on node.ts's removeChild, and only nominates: a framework may
+    // spell a move as remove-then-reinsert, so the commit sweep decides.
     removeChild(child) {
         removeChild(this.node, child);
     }
@@ -81,63 +57,41 @@ export class SymbioteSurface {
         for (const child of this.children)
             removeChild(this.node, child);
     }
-    /**
-     * Release every host behavior still standing under this surface, at unmount.
-     *
-     * The sweep above cannot answer this: it only sees nodes a `removeChild` NOMINATED, and an
-     * unmount removes nothing — the adapter drops the whole surface. Without it every node keeps its
-     * `afterCommit` registration and its timers, and a restarted surface's commits drain the dead
-     * one's hooks forever.
-     */
+    // Release every host behavior still standing under this surface, at unmount. The sweep above
+    // can't answer this: it only sees nodes removeChild nominated, and an unmount removes nothing —
+    // the adapter drops the whole surface.
     teardown() {
-        // The nominations first: an adapter that empties the surface and disposes it without a commit
-        // in between (React's `clearContainer`) never reaches the commit sweep, and the walk below
-        // cannot see those nodes either — they already left the tree.
+        // The nominations first: an adapter that empties and disposes the surface without a commit in
+        // between never reaches the commit sweep, and the walk below can't see those nodes either.
         sweepDetachedBehaviors(this.children, detachAnimatedProps);
         teardownSubtree(this.node, detachAnimatedProps);
     }
     // Synchronous commit: used by React's resetAfterCommit, which already batches per logical update.
     commit() {
-        // A SUPERSEDED surface still flushes its ops and still names every other root — its teardown is
-        // what carries the removals — but it must not complete a root another surface now owns. Fast
-        // Refresh and the focus lifecycle re-mount the same rootTag, so the old surface's teardown
-        // commit lands AFTER the new one's mount commit and would hand Fabric the emptied tree.
-        // An UNREGISTERED surface is not superseded — nobody took the root, so its final emptied tree
-        // is still the truth for it. Only a live OTHER owner suppresses the op.
-        // The teardown half of the behavior lifecycle. It runs AFTER the ops are applied — it decides
-        // what really left by asking the host for a parent, and the host does not know about a removal
-        // it has not been handed — but BEFORE the root is completed, so a behavior's parting writes
-        // (ScrollView taking its forced `scrollEventThrottle` back) ride this commit instead of owing
-        // another one. `flushOps` is that split: apply, do not publish.
-        //
-        // `removeChild` only NOMINATES — a framework spells a move as remove-then-reinsert, so tearing
-        // down at the call would kill a machine that comes back in the same batch.
+        // A superseded surface still flushes its ops and names every other root (its teardown carries
+        // the removals), but must not complete a root another surface now owns — Fast Refresh and the
+        // focus lifecycle can re-mount the same rootTag while an old teardown commit is still pending.
+        // The behavior-teardown half runs after ops are applied (asking the host for a parent needs the
+        // removal already handed over) but before the root completes, so a behavior's parting writes
+        // ride this commit instead of owing another one. flushOps is that split: apply, don't publish.
         flushOps();
-        // GUARDED AT THE CALL SITE, not inside the sweep — `this.children` is a host read that builds
-        // the whole top-level list, and it would run on every commit for a sweep that had nothing to do.
+        // Guarded at the call site, not inside the sweep — this.children is a host read that builds the
+        // whole top-level list, and it would run on every commit for a sweep with nothing to do.
         if (hasDetachCandidates()) {
             sweepDetachedBehaviors(this.children, detachAnimatedProps);
         }
         const owner = surfaces.get(this.rootTag);
         const superseded = owner !== undefined && owner !== this;
         commitSurfaceOps(superseded ? undefined : this.rootTag, this.node, SymbioteSurface.others(this));
-        // Fresh Fabric handles are now assigned, so the three things that could not run before one
-        // existed all drain here — this is the moment the old `commitChildren` drained them too.
-        //
-        // `notifyCommitted` releases the imperative waiters (`whenCommitted`); `runPostCommitHooks` the
-        // consumers that needed a committed TAG and ran too early, which is the Animated native driver
-        // binding a props node under an async-batched commit; `runDeferredAttaches` the half of a host
-        // behavior whose setup needs a tag (a view command, an event attach). The predicate is passed in
-        // rather than imported by `host-behavior.ts`, keeping that dependency one-directional — a cycle
-        // there is a live hazard under Metro's `inlineRequires`.
+        // Fresh Fabric handles are now assigned, so the three things that couldn't run before one
+        // existed all drain here: notifyCommitted releases the imperative waiters, runPostCommitHooks
+        // the consumers that needed a committed tag, runDeferredAttaches the same for a host behavior.
         notifyCommitted();
         runPostCommitHooks();
         runDeferredAttaches(isNodeCommitted);
-        // After the setup half, never before it: a node carrying both hooks has `attachAfterCommit`
-        // seed the mirrors `afterCommit` then compares against. Unlike the two above it asks only
-        // "props were published", so it is NOT gated on the commit having made native calls — a fold
-        // that strips a prop makes its own commit byte-identical, and the hook that must react to the
-        // flip would be the one the flip cannot wake.
+        // After the setup half, never before: a node carrying both hooks has attachAfterCommit seed the
+        // mirrors afterCommit then compares against. Unlike the two above, this asks only "props were
+        // published", so it isn't gated on the commit having made native calls.
         runCommittedHooks(isNodeCommitted);
     }
     // Coalesced commit: for reactive frameworks that emit many mutations per tick. Collapses to a
@@ -152,9 +106,8 @@ export class SymbioteSurface {
         });
     }
 }
-// Every live surface, so the microtask flush in `imperative.ts` can commit the one a queued write
-// named. Registered from here rather than imported there: a surface owns its own commit, and
-// reaching into it from the imperative half would put back the cycle that split exists to remove.
+// Every live surface, so the microtask flush in imperative.ts can commit the one a queued write
+// named. Registered from here rather than imported there, keeping that dependency one-directional.
 const surfaces = new Map();
 registerSurfaceCommit(rootTag => {
     surfaces.get(rootTag)?.commit();

package/build/text-input-state.js

@@ -1,7 +1,5 @@
-// Mirrors RN's TextInput/TextInputState: the single currently-focused input, tracked
-// JS-side because native exposes no focus getter. TextInput reports focus/blur here so
-// Keyboard.dismiss can blur whatever holds focus without a ref, exactly how RN's
-// dismissKeyboard() works (blurTextInput(currentlyFocusedInput())).
+// Mirrors RN's TextInput/TextInputState: the single currently-focused input, tracked JS-side since
+// native exposes no focus getter. Lets Keyboard.dismiss blur whatever holds focus without a ref.
 import { dispatchViewCommand, propOf } from './imperative.js';
 import { dlog } from './debug.js';
 let currentlyFocused = null;
@@ -18,10 +16,8 @@ export function setInputBlurred(node) {
     if (currentlyFocused === node)
         currentlyFocused = null;
 }
-// Imperative blur: drive the native `blur` view command and drop the tracked focus.
-// Used by TextInput.blur() and Keyboard.dismiss(). A no-op if this node isn't the
-// currently-focused one — mirrors RN's TextInputState.blurTextInput, which guards the
-// same way so blurring an already-unfocused input never reaches native.
+// Imperative blur: drives the native `blur` command and drops tracked focus. A no-op if this
+// node isn't the currently-focused one, mirroring RN's TextInputState.blurTextInput guard.
 export function blurTextInput(node) {
     if (node === null || currentlyFocused !== node)
         return;

package/build/toast-android/index.android.d.ts

@@ -0,0 +1,10 @@
+export declare const ToastAndroid: {
+    SHORT: number;
+    LONG: number;
+    TOP: number;
+    BOTTOM: number;
+    CENTER: number;
+    show(message: string, duration: number): void;
+    showWithGravity(message: string, duration: number, gravity: number): void;
+    showWithGravityAndOffset(message: string, duration: number, gravity: number, xOffset: number, yOffset: number): void;
+};

package/build/toast-android/index.android.js

@@ -0,0 +1,108 @@
+// ToastAndroid module: pops a brief Android system toast from JS. Mirrors RN's
+// Libraries/Components/ToastAndroid/ToastAndroid.android.js: `show` takes a message
+// and a duration; `showWithGravity` adds a layout gravity; `showWithGravityAndOffset`
+// adds an x/y pixel offset. No Fabric view, pure JS->native.
+//
+// symbiote is iOS-first and this is an Android-only module. RN's iOS fallback
+// (ToastAndroid.ios.js) throws "not supported"; we prefer a no-op + dlog so a
+// cross-platform smoke never crashes when the native module is absent.
+//
+// The native contract is confirmed from RN's TurboModule spec at
+// specs_DEPRECATED/modules/INativeToastAndroid.js:
+//   getConstants(): { SHORT, LONG, TOP, BOTTOM, CENTER }
+//   show(message: string, duration: number)
+//   showWithGravity(message: string, duration: number, gravity: number)
+//   showWithGravityAndOffset(message, duration, gravity, xOffset, yOffset)
+import { dlog } from '../debug.js';
+import { getNativeModule } from '../native-modules/index.js';
+// The native module name RN registers this under. NOTE: this is the name the spec
+// resolves via `TurboModuleRegistry.getEnforcing<Spec>('ToastAndroid')`, NOT the
+// spec filename `INativeToastAndroid`. A module name is only provable on a real
+// host (a headless fake answers to any name); this Android name is confirmed from
+// RN source but not yet exercised on a device.
+const TOAST_MODULE = 'ToastAndroid';
+// Conventional RN values for SHORT/LONG/TOP/BOTTOM/CENTER. On a real device the
+// numbers come from native getConstants() (resolved below); these are the fallbacks
+// that keep the constants object populated headless and on a host without the module.
+const FALLBACK_CONSTANTS = {
+    SHORT: 0,
+    LONG: 1,
+    TOP: 48,
+    BOTTOM: 80,
+    CENTER: 17,
+};
+// Lazily resolved so importing this module has no native side effect.
+let toastModule;
+function getModule() {
+    if (toastModule === undefined) {
+        toastModule = getNativeModule(TOAST_MODULE);
+        dlog(`ToastAndroid: ToastAndroid module ${toastModule ? 'resolved' : 'NOT resolved (null)'}`);
+    }
+    return toastModule;
+}
+// Read a single numeric constant from the native getConstants() payload, narrowing
+// with a `typeof` guard (no cast) and falling back to the conventional RN value when
+// the module or the key is absent.
+function readConstant(raw, key) {
+    const value = raw[key];
+    return typeof value === 'number' ? value : FALLBACK_CONSTANTS[key];
+}
+// Resolve the constants once: real numbers from native on a device, fallbacks
+// otherwise. Always returns a fully-populated object so the constants are never
+// undefined, even headless.
+function resolveConstants() {
+    const module = getModule();
+    if (module === null) {
+        dlog('ToastAndroid: constants from fallbacks (native module unavailable)');
+        return { ...FALLBACK_CONSTANTS };
+    }
+    const raw = module.getConstants();
+    return {
+        SHORT: readConstant(raw, 'SHORT'),
+        LONG: readConstant(raw, 'LONG'),
+        TOP: readConstant(raw, 'TOP'),
+        BOTTOM: readConstant(raw, 'BOTTOM'),
+        CENTER: readConstant(raw, 'CENTER'),
+    };
+}
+const constants = resolveConstants();
+export const ToastAndroid = {
+    // Toast duration constants.
+    SHORT: constants.SHORT,
+    LONG: constants.LONG,
+    // Toast gravity constants.
+    TOP: constants.TOP,
+    BOTTOM: constants.BOTTOM,
+    CENTER: constants.CENTER,
+    // Show a toast with the given message and duration. Degrades to a no-op (logged)
+    // when the module is absent; a missing optional native module must never throw.
+    show(message, duration) {
+        const module = getModule();
+        if (module === null) {
+            dlog('ToastAndroid.show -> ToastAndroid native module unavailable, no-op');
+            return;
+        }
+        dlog(`ToastAndroid.show -> "${message}" (${duration})`);
+        module.show(message, duration);
+    },
+    // Show a toast at the given layout gravity (TOP / BOTTOM / CENTER).
+    showWithGravity(message, duration, gravity) {
+        const module = getModule();
+        if (module === null) {
+            dlog('ToastAndroid.showWithGravity -> ToastAndroid native module unavailable, no-op');
+            return;
+        }
+        dlog(`ToastAndroid.showWithGravity -> "${message}" (${duration}, gravity=${gravity})`);
+        module.showWithGravity(message, duration, gravity);
+    },
+    // Show a toast at the given gravity, offset by xOffset/yOffset pixels.
+    showWithGravityAndOffset(message, duration, gravity, xOffset, yOffset) {
+        const module = getModule();
+        if (module === null) {
+            dlog('ToastAndroid.showWithGravityAndOffset -> ToastAndroid native module unavailable, no-op');
+            return;
+        }
+        dlog(`ToastAndroid.showWithGravityAndOffset -> "${message}" (${duration}, gravity=${gravity}, ${xOffset}, ${yOffset})`);
+        module.showWithGravityAndOffset(message, duration, gravity, xOffset, yOffset);
+    },
+};

package/build/toast-android/index.d.ts

@@ -1,10 +1 @@
-export declare const ToastAndroid: {
-    SHORT: number;
-    LONG: number;
-    TOP: number;
-    BOTTOM: number;
-    CENTER: number;
-    show(message: string, duration: number): void;
-    showWithGravity(message: string, duration: number, gravity: number): void;
-    showWithGravityAndOffset(message: string, duration: number, gravity: number, xOffset: number, yOffset: number): void;
-};
+export * from './index.ios';

package/build/toast-android/index.ios.d.ts

@@ -0,0 +1,10 @@
+export declare const ToastAndroid: {
+    SHORT: number;
+    LONG: number;
+    TOP: number;
+    BOTTOM: number;
+    CENTER: number;
+    show(_message: string, _duration: number): void;
+    showWithGravity(_message: string, _duration: number, _gravity: number): void;
+    showWithGravityAndOffset(_message: string, _duration: number, _gravity: number, _xOffset: number, _yOffset: number): void;
+};

package/build/toast-android/index.ios.js

@@ -0,0 +1,19 @@
+// ToastAndroid off Android: RN's ToastAndroidFallback (ToastAndroid.ios.js). Zeroed constants and
+// RN's own user-facing warning on every call; nothing reaches native.
+const UNSUPPORTED = 'ToastAndroid is not supported on this platform.';
+export const ToastAndroid = {
+    SHORT: 0,
+    LONG: 0,
+    TOP: 0,
+    BOTTOM: 0,
+    CENTER: 0,
+    show(_message, _duration) {
+        console.warn(UNSUPPORTED);
+    },
+    showWithGravity(_message, _duration, _gravity) {
+        console.warn(UNSUPPORTED);
+    },
+    showWithGravityAndOffset(_message, _duration, _gravity, _xOffset, _yOffset) {
+        console.warn(UNSUPPORTED);
+    },
+};

package/build/toast-android/index.js

@@ -1,108 +1,3 @@
-// ToastAndroid module: pops a brief Android system toast from JS. Mirrors RN's
-// Libraries/Components/ToastAndroid/ToastAndroid.android.js: `show` takes a message
-// and a duration; `showWithGravity` adds a layout gravity; `showWithGravityAndOffset`
-// adds an x/y pixel offset. No Fabric view, pure JS->native.
-//
-// symbiote is iOS-first and this is an Android-only module. RN's iOS fallback
-// (ToastAndroid.ios.js) throws "not supported"; we prefer a no-op + dlog so a
-// cross-platform smoke never crashes when the native module is absent.
-//
-// The native contract is confirmed from RN's TurboModule spec at
-// specs_DEPRECATED/modules/INativeToastAndroid.js:
-//   getConstants(): { SHORT, LONG, TOP, BOTTOM, CENTER }
-//   show(message: string, duration: number)
-//   showWithGravity(message: string, duration: number, gravity: number)
-//   showWithGravityAndOffset(message, duration, gravity, xOffset, yOffset)
-import { dlog } from '../debug.js';
-import { getNativeModule } from '../native-modules/index.js';
-// The native module name RN registers this under. NOTE: this is the name the spec
-// resolves via `TurboModuleRegistry.getEnforcing<Spec>('ToastAndroid')`, NOT the
-// spec filename `INativeToastAndroid`. A module name is only provable on a real
-// host (a headless fake answers to any name); this Android name is confirmed from
-// RN source but not yet exercised on a device.
-const TOAST_MODULE = 'ToastAndroid';
-// Conventional RN values for SHORT/LONG/TOP/BOTTOM/CENTER. On a real device the
-// numbers come from native getConstants() (resolved below); these are the fallbacks
-// that keep the constants object populated headless and on a host without the module.
-const FALLBACK_CONSTANTS = {
-    SHORT: 0,
-    LONG: 1,
-    TOP: 48,
-    BOTTOM: 80,
-    CENTER: 17,
-};
-// Lazily resolved so importing this module has no native side effect.
-let toastModule;
-function getModule() {
-    if (toastModule === undefined) {
-        toastModule = getNativeModule(TOAST_MODULE);
-        dlog(`ToastAndroid: ToastAndroid module ${toastModule ? 'resolved' : 'NOT resolved (null)'}`);
-    }
-    return toastModule;
-}
-// Read a single numeric constant from the native getConstants() payload, narrowing
-// with a `typeof` guard (no cast) and falling back to the conventional RN value when
-// the module or the key is absent.
-function readConstant(raw, key) {
-    const value = raw[key];
-    return typeof value === 'number' ? value : FALLBACK_CONSTANTS[key];
-}
-// Resolve the constants once: real numbers from native on a device, fallbacks
-// otherwise. Always returns a fully-populated object so the constants are never
-// undefined, even headless.
-function resolveConstants() {
-    const module = getModule();
-    if (module === null) {
-        dlog('ToastAndroid: constants from fallbacks (native module unavailable)');
-        return { ...FALLBACK_CONSTANTS };
-    }
-    const raw = module.getConstants();
-    return {
-        SHORT: readConstant(raw, 'SHORT'),
-        LONG: readConstant(raw, 'LONG'),
-        TOP: readConstant(raw, 'TOP'),
-        BOTTOM: readConstant(raw, 'BOTTOM'),
-        CENTER: readConstant(raw, 'CENTER'),
-    };
-}
-const constants = resolveConstants();
-export const ToastAndroid = {
-    // Toast duration constants.
-    SHORT: constants.SHORT,
-    LONG: constants.LONG,
-    // Toast gravity constants.
-    TOP: constants.TOP,
-    BOTTOM: constants.BOTTOM,
-    CENTER: constants.CENTER,
-    // Show a toast with the given message and duration. Degrades to a no-op (logged)
-    // when the module is absent; a missing optional native module must never throw.
-    show(message, duration) {
-        const module = getModule();
-        if (module === null) {
-            dlog('ToastAndroid.show -> ToastAndroid native module unavailable, no-op');
-            return;
-        }
-        dlog(`ToastAndroid.show -> "${message}" (${duration})`);
-        module.show(message, duration);
-    },
-    // Show a toast at the given layout gravity (TOP / BOTTOM / CENTER).
-    showWithGravity(message, duration, gravity) {
-        const module = getModule();
-        if (module === null) {
-            dlog('ToastAndroid.showWithGravity -> ToastAndroid native module unavailable, no-op');
-            return;
-        }
-        dlog(`ToastAndroid.showWithGravity -> "${message}" (${duration}, gravity=${gravity})`);
-        module.showWithGravity(message, duration, gravity);
-    },
-    // Show a toast at the given gravity, offset by xOffset/yOffset pixels.
-    showWithGravityAndOffset(message, duration, gravity, xOffset, yOffset) {
-        const module = getModule();
-        if (module === null) {
-            dlog('ToastAndroid.showWithGravityAndOffset -> ToastAndroid native module unavailable, no-op');
-            return;
-        }
-        dlog(`ToastAndroid.showWithGravityAndOffset -> "${message}" (${duration}, gravity=${gravity}, ${xOffset}, ${yOffset})`);
-        module.showWithGravityAndOffset(message, duration, gravity, xOffset, yOffset);
-    },
-};
+// ToastAndroid: base / headless build. Metro picks index.android.ts / index.ios.ts on a device; off
+// those, RN's non-Android fallback is the default, as RN's own ToastAndroid.js is.
+export * from './index.ios.js';

package/build/touch-history.js

@@ -1,14 +1,8 @@
-// Per-touch position/time tracking, ported from RN's
-// react-native-renderer/.../legacy-events/ResponderTouchHistoryStore.js. PanResponder's
-// multitouch dx/vx math needs each touch's own previous->current delta (RN counts only
-// touches that moved since `_accountsForMovesUpTo`), which a grant-relative centroid of
-// ALL live touches cannot reconstruct. We maintain the bank as touches flow and ATTACH
-// `touchHistory` onto the nativeEvent reaching responder handlers, exactly how
-// ResponderEventPlugin.js sets `*.touchHistory`.
-//
-// events/index.ts consumes only this file's public surface (recordTouchTrack,
-// attachTouchHistory, resetTouchHistory, touchHistory); everything else here is a
-// private implementation detail of the bank.
+// Per-touch position/time tracking, ported from RN's ResponderTouchHistoryStore. PanResponder's
+// multitouch dx/vx math needs each touch's own previous->current delta, which a grant-relative
+// centroid of all live touches can't reconstruct; the bank attaches to responder events.
+// events/index.ts consumes only this file's public surface (recordTouchTrack, attachTouchHistory,
+// resetTouchHistory, touchHistory); everything else here is a private implementation detail.
 import { isRecord } from './type-guards.js';
 // RN's bank is indexed by touch identifier and warns above 20; we never warn (headless
 // events may carry larger or absent ids), we just skip anything out of a sane range.