@octane-xplat/patches 0.14.0 → 0.15.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@octane-xplat/patches",
-  "version": "0.14.0",
+  "version": "0.15.0",
   "description": "pnpm config dependency carrying the framework patch files",
   "keywords": [
     "framework",

package/patches/@nativescript-community__octane@0.2.4.patch

@@ -1,20 +1,195 @@
 diff --git a/dist/driver.js b/dist/driver.js
-index c88828e30337ee566bc1a919940e58b4b0227aba..52cde3f8f8acffcc05a1db8e0fcd90515efc70f6 100644
+index c88828e30337ee566bc1a919940e58b4b0227aba..5fcee7ffb92d8ddf937b65b27825cbb81944a195 100644
 --- a/dist/driver.js
 +++ b/dist/driver.js
-@@ -85,6 +85,11 @@ function setProp(node, view, name, value) {
+@@ -56,15 +56,34 @@ function applyProps(node, props) {
+     }
+ }
+ /**
+- * `update` carries the host's dynamic-prop snapshot; attributes that were
+- * static in the compiled plan arrive only with `create`. Merge instead of
+- * replace, or a first update would strip a node's static layout props.
++ * `update` carries the host's complete prop bag: a key present in the last
++ * applied bag but absent from the next is a removal, and the view state it
++ * installed returns to the prop's documented default. `undefined` never
++ * crosses the wire (the runtime elides it); `null` is "not supplied" for
++ * props without an explicit nullable contract — both reset through setProp.
+  */
+ function updateProps(node, next) {
++    const dropped = [];
++    for (const name in node.props) {
++        if (node.props[name] != null && !(name in next))
++            dropped.push(name);
++    }
+     applyProps(node, next);
+-    node.props = { ...node.props, ...next };
++    for (const name of dropped) {
++        resetProp(node, node.view, name);
++    }
++    node.props = next;
+ }
+ function setProp(node, view, name, value) {
++    // Keep authored placement separate from Grid's computed native coordinates.
++    // NativeScript does not retain a setter write equal to its default (row/col 0).
++    if (name === 'row' || name === 'col' || name === 'rowSpan' || name === 'colSpan') {
++        const key = Symbol.for('octane-xplat.grid-placement');
++        const placement = view[key] ?? (view[key] = {});
++        if (value == null || value === unsetValue) delete placement[name];
++        else placement[name] = value;
++        view.notify({ eventName: 'xplatGridPlacementChange', object: view });
++    }
+     if (name === 'children' || name === 'key' || name === 'ref')
+         return;
+     // Event props arrive as their own `event` commands.
+@@ -78,13 +97,28 @@ function setProp(node, view, name, value) {
+         return;
+     }
+     if (name === 'style') {
+-        applyStyle(view, value);
++        applyStyle(node, view, value);
+         return;
+     }
+     if (name === 'renderItem' && view instanceof ListView) {
          setRenderItem(view, value ?? null, createNativeScriptRoot);
          return;
      }
-+    // `undefined` means "prop not supplied" — skip the write. NativeScript
-+    // native setters coerce it (e.g. editable → ios.userInteractionEnabled =
-+    // NO) leaving dead UI that still has its JS listeners.
-+    if (value === undefined)
++    // `null`/`unsetValue` restore the prop's default rather than writing a
++    // literal null through a native setter. `undefined` is a removal: skip it
++    // when nothing applied (NativeScript setters coerce it badly) and reset
++    // when a real value did.
++    if (value == null || value === unsetValue) {
++        if (value === undefined && node.props[name] == null)
++            return;
++        resetProp(node, view, name);
 +        return;
++    }
++    // Capture the pre-prop value once so a plain (non-Property) field can be
++    // restored when the prop is removed later.
++    const defaults = (node.propDefaults ??= {});
++    if (!(name in defaults))
++        defaults[name] = view[name];
      // Core raises `<name>Change` for a script write exactly as for a user
      // edit; a controlled input must not see its own value come back as one.
      const echo = `${name}Change`;
-@@ -482,7 +487,7 @@ function applyCommand(container, command) {
+@@ -99,6 +133,44 @@ function setProp(node, view, name, value) {
+             node.muted.delete(echo);
+     }
+ }
++/** Restore a removed prop to the value it holds when never supplied. */
++function resetProp(node, view, name) {
++    if (view === null)
++        return;
++    if (name === 'children' || name === 'key' || name === 'ref')
++        return;
++    // `on*` handlers unbind through `event` commands, not props.
++    if (EVENT_PROP.test(name))
++        return;
++    if (name === 'className' || name === 'class') {
++        view.className = unsetValue;
++        return;
++    }
++    if (name === 'style') {
++        applyStyle(node, view, null);
++        return;
++    }
++    if (name === 'renderItem' && view instanceof ListView) {
++        setRenderItem(view, null, createNativeScriptRoot);
++        return;
++    }
++    const echo = `${name}Change`;
++    const listening = node.listeners.has(echo);
++    if (listening)
++        node.muted.add(echo);
++    try {
++        // Property/CssProperty setters consume the sentinel and restore the
++        // declared default. A plain field stores it raw — detect that and
++        // write back the captured pre-prop value instead.
++        view[name] = unsetValue;
++        if (view[name] === unsetValue)
++            view[name] = node.propDefaults?.[name];
++    }
++    finally {
++        if (listening)
++            node.muted.delete(echo);
++    }
++}
+ // `class`/`className` compose clsx-style on the DOM renderer; join arrays
+ // with spaces so NativeScript's class selectors can match each class.
+ function normalizeClass(value) {
+@@ -112,15 +184,67 @@ function normalizeClass(value) {
+             .join(' ');
+     return '';
+ }
+-function applyStyle(view, value) {
+-    if (value == null || value === unsetValue)
+-        return;
++/**
++ * The `style` prop replaces wholesale — a key dropped or nulled since the last
++ * apply resets what it installed (`unsetValue` clears a local CssProperty
++ * value so the cascade can re-apply). String styles are parsed for their
++ * declared names so a later object or removal can reset them.
++ */
++function applyStyle(node, view, value) {
++    const applied = node.styleKeys;
++    node.styleKeys = undefined;
+     if (typeof value === 'string') {
++        clearStyleKeys(view, applied);
+         view.setInlineStyle(value);
++        node.styleKeys = inlineStyleKeys(value);
++        return;
++    }
++    if (value == null || value === unsetValue || typeof value !== 'object') {
++        clearStyleKeys(view, applied);
+         return;
+     }
+-    if (typeof value === 'object')
+-        Object.assign(view.style, value);
++    if (applied !== undefined) {
++        for (const key of applied) {
++            if (!(key in value))
++                unsetStyleKey(view, key);
++        }
++    }
++    for (const [key, nextValue] of Object.entries(value)) {
++        if (nextValue == null)
++            unsetStyleKey(view, key);
++        else
++            view.style[key] = nextValue;
++    }
++    node.styleKeys = new Set(Object.keys(value));
++}
++function clearStyleKeys(view, keys) {
++    if (keys === undefined)
++        return;
++    for (const key of keys)
++        unsetStyleKey(view, key);
++}
++function unsetStyleKey(view, name) {
++    const style = view.style;
++    if (name.startsWith('--')) {
++        style.setUnscopedCssVariable?.(name, unsetValue);
++        return;
++    }
++    style[name] = unsetValue;
++    // A style field outside the CssProperty registry stores the sentinel raw.
++    if (style[name] === unsetValue)
++        style[name] = undefined;
++}
++function inlineStyleKeys(style) {
++    const keys = new Set();
++    for (const declaration of style.split(';')) {
++        const colon = declaration.indexOf(':');
++        if (colon === -1)
++            continue;
++        const name = declaration.slice(0, colon).trim();
++        if (name !== '')
++            keys.add(name.replace(/-([a-z])/g, (_, letter) => letter.toUpperCase()));
++    }
++    return keys;
+ }
+ /** Recompute a text host's `text` from its `#text` children. */
+ function syncText(parent) {
+@@ -482,7 +606,7 @@ function applyCommand(container, command) {
  }
  export const nativeScriptDriver = {
      id: NATIVESCRIPT_RENDERER_ID,

package/patches/@nativescript-community__octane@0.2.5.patch

@@ -0,0 +1,225 @@
+diff --git a/dist/driver.js b/dist/driver.js
+index 8bc563add4cb675a12651184b63805b10224274d..a10f12af74fd32a630b71811c3f10f9338d63a50 100644
+--- a/dist/driver.js
++++ b/dist/driver.js
+@@ -91,15 +91,34 @@
+     }
+ }
+ /**
+- * `update` carries the host's dynamic-prop snapshot; attributes that were
+- * static in the compiled plan arrive only with `create`. Merge instead of
+- * replace, or a first update would strip a node's static layout props.
++ * `update` carries the host's complete prop bag: a key present in the last
++ * applied bag but absent from the next is a removal, and the view state it
++ * installed returns to the prop's documented default. `undefined` never
++ * crosses the wire (the runtime elides it); `null` is "not supplied" for
++ * props without an explicit nullable contract — both reset through setProp.
+  */
+ function updateProps(node, next) {
++    const dropped = [];
++    for (const name in node.props) {
++        if (node.props[name] != null && !(name in next))
++            dropped.push(name);
++    }
+     applyProps(node, next);
+-    node.props = { ...node.props, ...next };
++    for (const name of dropped) {
++        resetProp(node, node.view, name);
++    }
++    node.props = next;
+ }
+ function setProp(node, view, name, value) {
++    // Keep authored placement separate from Grid's computed native coordinates.
++    // NativeScript does not retain a setter write equal to its default (row/col 0).
++    if (name === 'row' || name === 'col' || name === 'rowSpan' || name === 'colSpan') {
++        const key = Symbol.for('octane-xplat.grid-placement');
++        const placement = view[key] ?? (view[key] = {});
++        if (value == null || value === unsetValue) delete placement[name];
++        else placement[name] = value;
++        view.notify({ eventName: 'xplatGridPlacementChange', object: view });
++    }
+     if (name === 'children' || name === 'key' || name === 'ref')
+         return;
+     // Event props arrive as their own `event` commands.
+@@ -113,13 +132,28 @@
+         return;
+     }
+     if (name === 'style') {
+-        applyStyle(view, value);
++        applyStyle(node, view, value);
+         return;
+     }
+     if (name === 'renderItem' && view instanceof ListView) {
+         setRenderItem(view, value ?? null, createNativeScriptRoot);
+         return;
+     }
++    // `null`/`unsetValue` restore the prop's default rather than writing a
++    // literal null through a native setter. `undefined` is a removal: skip it
++    // when nothing applied (NativeScript setters coerce it badly) and reset
++    // when a real value did.
++    if (value == null || value === unsetValue) {
++        if (value === undefined && node.props[name] == null)
++            return;
++        resetProp(node, view, name);
++        return;
++    }
++    // Capture the pre-prop value once so a plain (non-Property) field can be
++    // restored when the prop is removed later.
++    const defaults = (node.propDefaults ??= {});
++    if (!(name in defaults))
++        defaults[name] = view[name];
+     // Core raises `<name>Change` for a script write exactly as for a user
+     // edit; a controlled input must not see its own value come back as one.
+     const echo = `${name}Change`;
+@@ -128,6 +162,44 @@
+         node.muted.add(echo);
+     try {
+         view[name] = value;
++    }
++    finally {
++        if (listening)
++            node.muted.delete(echo);
++    }
++}
++/** Restore a removed prop to the value it holds when never supplied. */
++function resetProp(node, view, name) {
++    if (view === null)
++        return;
++    if (name === 'children' || name === 'key' || name === 'ref')
++        return;
++    // `on*` handlers unbind through `event` commands, not props.
++    if (EVENT_PROP.test(name))
++        return;
++    if (name === 'className' || name === 'class') {
++        view.className = unsetValue;
++        return;
++    }
++    if (name === 'style') {
++        applyStyle(node, view, null);
++        return;
++    }
++    if (name === 'renderItem' && view instanceof ListView) {
++        setRenderItem(view, null, createNativeScriptRoot);
++        return;
++    }
++    const echo = `${name}Change`;
++    const listening = node.listeners.has(echo);
++    if (listening)
++        node.muted.add(echo);
++    try {
++        // Property/CssProperty setters consume the sentinel and restore the
++        // declared default. A plain field stores it raw — detect that and
++        // write back the captured pre-prop value instead.
++        view[name] = unsetValue;
++        if (view[name] === unsetValue)
++            view[name] = node.propDefaults?.[name];
+     }
+     finally {
+         if (listening)
+@@ -147,15 +219,67 @@
+             .join(' ');
+     return '';
+ }
+-function applyStyle(view, value) {
+-    if (value == null || value === unsetValue)
+-        return;
++/**
++ * The `style` prop replaces wholesale — a key dropped or nulled since the last
++ * apply resets what it installed (`unsetValue` clears a local CssProperty
++ * value so the cascade can re-apply). String styles are parsed for their
++ * declared names so a later object or removal can reset them.
++ */
++function applyStyle(node, view, value) {
++    const applied = node.styleKeys;
++    node.styleKeys = undefined;
+     if (typeof value === 'string') {
++        clearStyleKeys(view, applied);
+         view.setInlineStyle(value);
++        node.styleKeys = inlineStyleKeys(value);
+         return;
+     }
+-    if (typeof value === 'object')
+-        Object.assign(view.style, value);
++    if (value == null || value === unsetValue || typeof value !== 'object') {
++        clearStyleKeys(view, applied);
++        return;
++    }
++    if (applied !== undefined) {
++        for (const key of applied) {
++            if (!(key in value))
++                unsetStyleKey(view, key);
++        }
++    }
++    for (const [key, nextValue] of Object.entries(value)) {
++        if (nextValue == null)
++            unsetStyleKey(view, key);
++        else
++            view.style[key] = nextValue;
++    }
++    node.styleKeys = new Set(Object.keys(value));
++}
++function clearStyleKeys(view, keys) {
++    if (keys === undefined)
++        return;
++    for (const key of keys)
++        unsetStyleKey(view, key);
++}
++function unsetStyleKey(view, name) {
++    const style = view.style;
++    if (name.startsWith('--')) {
++        style.setUnscopedCssVariable?.(name, unsetValue);
++        return;
++    }
++    style[name] = unsetValue;
++    // A style field outside the CssProperty registry stores the sentinel raw.
++    if (style[name] === unsetValue)
++        style[name] = undefined;
++}
++function inlineStyleKeys(style) {
++    const keys = new Set();
++    for (const declaration of style.split(';')) {
++        const colon = declaration.indexOf(':');
++        if (colon === -1)
++            continue;
++        const name = declaration.slice(0, colon).trim();
++        if (name !== '')
++            keys.add(name.replace(/-([a-z])/g, (_, letter) => letter.toUpperCase()));
++    }
++    return keys;
+ }
+ /** Recompute a text host's `text` from its `#text` children. */
+ function syncText(parent) {
+@@ -641,7 +765,7 @@
+ }
+ export const nativeScriptDriver = {
+     id: NATIVESCRIPT_RENDERER_ID,
+-    capabilities: { text: 'host' },
++    capabilities: { text: 'host', visibility: true },
+     events: {
+         classify(name) {
+             if (!EVENT_PROP.test(name))
+diff --git a/dist/intrinsics.d.ts b/dist/intrinsics.d.ts
+index 45a47cbbce199df9635852ca4cdb2849e9bbc83d..d555b50c08276e925132b2be9bb62302fe0a4228 100644
+--- a/dist/intrinsics.d.ts
++++ b/dist/intrinsics.d.ts
+@@ -24,9 +24,9 @@
+  */
+ import type * as NS from '@nativescript/core';
+ export type NativeScriptNode = unknown;
+-type Ref<TInstance> = ((instance: TInstance | null) => void) | {
++type Ref<TInstance> = ((instance: TInstance | null) => void | (() => void)) | {
+     current: TInstance | null;
+-} | null;
++} | readonly Ref<TInstance>[] | null | undefined;
+ /**
+  * The driver assigns every non-event prop onto the view instance, so a JSX
+  * attribute is a NativeScript view property rather than a DOM attribute.
+@@ -117,7 +117,7 @@ export interface ListViewAttributes {
+ }
+ export interface OctaneAttributes<TInstance> {
+     key?: string | number;
+-    ref?: Ref<TInstance> | readonly Ref<TInstance>[];
++    ref?: Ref<TInstance>;
+     children?: NativeScriptNode;
+ }
+ type Overridden = keyof CommonAttributes | keyof OctaneAttributes<unknown> | keyof CommonEvents<unknown>;

package/patches/octane@0.11.0.patch

@@ -1,5 +1,5 @@
 diff --git a/dist/cjs/runtime.cjs b/dist/cjs/runtime.cjs
-index a035f05..9912e51 100644
+index a035f05..e1c9ca1 100644
 --- a/dist/cjs/runtime.cjs
 +++ b/dist/cjs/runtime.cjs
 @@ -365,6 +365,7 @@ var import_stream_protocol = require("./stream-protocol.cjs");
@@ -18,6 +18,15 @@ index a035f05..9912e51 100644
  function useState(initial, slot) {
    const s = readStateHook(initial, slot, arguments.length === 1);
    return [s.value, s.setter];
+@@ -9659,6 +9661,8 @@ function renderClientContextProvider(context, props, renderScope) {
+       ensureHooks(scope).set(CHILDREN_DIALECT_SLOT, dialect);
+     }
+     renderSharedBody(childrenAsBody(children), void 0, scope, void 0);
++  } else {
++    resetScopeChildren(scope);
+   }
+ }
+ let SHARED_BODY_SCOPE = null;
 diff --git a/dist/cjs/signals/client.cjs b/dist/cjs/signals/client.cjs
 index 28e2844..af40518 100644
 --- a/dist/cjs/signals/client.cjs
@@ -453,7 +462,7 @@ index 90848a6..474c2bb 100644
 +		let specifiers = signalHelperImports.get(request);
 +		if (specifiers === undefined) signalHelperImports.set(request, (specifiers = []));
 diff --git a/dist/node/runtime.js b/dist/node/runtime.js
-index 927d060..cef6b5b 100644
+index 927d060..d311ac9 100644
 --- a/dist/node/runtime.js
 +++ b/dist/node/runtime.js
 @@ -185,6 +185,7 @@ import {
@@ -472,6 +481,15 @@ index 927d060..cef6b5b 100644
  function useState(initial, slot) {
    const s = readStateHook(initial, slot, arguments.length === 1);
    return [s.value, s.setter];
+@@ -9532,6 +9534,8 @@ function renderClientContextProvider(context, props, renderScope) {
+       ensureHooks(scope).set(CHILDREN_DIALECT_SLOT, dialect);
+     }
+     renderSharedBody(childrenAsBody(children), void 0, scope, void 0);
++  } else {
++    resetScopeChildren(scope);
+   }
+ }
+ let SHARED_BODY_SCOPE = null;
 diff --git a/dist/node/signals/client.js b/dist/node/signals/client.js
 index 21a39c3..3f85700 100644
 --- a/dist/node/signals/client.js
@@ -1034,7 +1052,7 @@ index 6427a01..8049b78 100644
    createContext,
    createSubSlot,
 diff --git a/dist/runtime.js b/dist/runtime.js
-index 67ace09..ef61a91 100644
+index 67ace09..afeb8ea 100644
 --- a/dist/runtime.js
 +++ b/dist/runtime.js
 @@ -184,6 +184,7 @@ import {
@@ -1053,6 +1071,15 @@ index 67ace09..ef61a91 100644
  function useState(initial, slot) {
    const s = readStateHook(initial, slot, arguments.length === 1);
    return [s.value, s.setter];
+@@ -9531,6 +9533,8 @@ function renderClientContextProvider(context, props, renderScope) {
+       ensureHooks(scope).set(CHILDREN_DIALECT_SLOT, dialect);
+     }
+     renderSharedBody(childrenAsBody(children), void 0, scope, void 0);
++  } else {
++    resetScopeChildren(scope);
+   }
+ }
+ let SHARED_BODY_SCOPE = null;
 diff --git a/dist/signals/client.d.ts b/dist/signals/client.d.ts
 index 6388c2f..f9e9862 100644
 --- a/dist/signals/client.d.ts