@symbiote-native/angular 3.1.0 → 3.1.2

package/src/primitives/index.ts

@@ -21,21 +21,8 @@ export {
 })
 export class ViewHost extends SymbiotePrimitiveHost {}
 
-/**
- * Text is an ORDINARY primitive host as of 2026-09-18, and the two `@Input()`s it used to declare
- * are gone with the code that needed them.
- *
- * They existed for one stated reason: a default can only be applied by code that can SEE whether the
- * caller supplied a value, and a pass-through host binding is invisible to the component — so the
- * defaults had to be applied where the inputs were readable. That argument was sound and it is now
- * answered one layer down. The payload builder reads the AUTHORED bag, so it can tell an absent
- * `ellipsizeMode` from an explicit `clip` without anyone declaring anything; the rule is keyed on the
- * component (`foldTextDefaults`, `SymbioteFabricProps.cpp`) and reaches every `RCTText` however it
- * was spelled.
- *
- * The pass-through is therefore the CORRECT path for both props now, and `text-defaults.test.ts`'s
- * "never overwrites a value the caller supplied" is what proves the authored value still arrives.
- */
+// Text is an ordinary primitive host — no `@Input()`s for defaults; `foldTextDefaults` (C++)
+// reads the authored bag directly, so an absent value never needs declaring here.
 @Component({
   selector: 'text',
   standalone: true,
@@ -99,10 +86,9 @@ export class MultilineTextInputHost extends SymbiotePrimitiveHost {}
 })
 export class SwitchHost extends SymbiotePrimitiveHost {}
 
-// The centering RCTView RN wraps the spinner in (ActivityIndicator.js:112), not the spinner —
-// which is what this tag resolved to until 2026-09-09. The native view moved to
-// `activity-indicator-spinner`, built by the engine's ActivityIndicator behavior, so nothing writes
-// it in a template and it needs no host here.
+// The centering RCTView RN wraps the spinner in (ActivityIndicator.js:112) is
+// `activity-indicator-spinner`, built by the engine's ActivityIndicator behavior — nothing writes
+// it in a template, so it needs no host here.
 @Component({
   selector: 'activity-indicator',
   standalone: true,

package/src/primitives/shared.ts

@@ -287,12 +287,9 @@ export class SymbioteHostPropsDirective {
     return this.elementRef.nativeElement;
   }
 
-  // MEASURED 2026-08-20, headless 1000-row create: writing every key of every bag cost 104 000
-  // engine setProp calls where Solid's identical screen cost 12 000 — and 90 000 of them carried
-  // `undefined`. A composed component's bag is a FIXED-SHAPE literal (Pressable's is ~48 keys,
-  // most of them unset on any given instance), so the first push of a fresh node spends most of
-  // its work deleting keys the node never had. Diffing per key removes exactly that: an unset key
-  // is skipped on mount and still written the moment it changes to or from a real value.
+  // A composed component's bag is a FIXED-SHAPE literal, most keys unset on any given instance, so
+  // writing every key on the first push spends most of its work on keys the node never had.
+  // Diffing per key skips an unset one on mount and writes it only when it gains a real value.
   set symbioteHostProps(props: Record<string, unknown>) {
     const pushed = this.pushed;
     // Only keys the node actually carries, so the vanished-key sweep below walks a handful rather

package/src/renderer/index.ts

@@ -62,17 +62,9 @@ function isRecord(value: unknown): value is Record<string, unknown> {
   return typeof value === 'object' && value !== null;
 }
 
-// RN's TWO TEXT DEFAULTS LEFT THIS RENDERER ENTIRELY ON 2026-09-18, in two steps a fortnight apart.
-// The SEED went first, to the payload builder — writing them as props cost a crossing every time an
-// app authored the same value (6 000 per 1 000-row create, `writesOfUnchanged`). What stayed was a
-// resolver for the clear-back path: a write of `undefined` looked up the default instead of clearing.
-//
-// That is gone too, and for the reason the seed was: `applyTextDefaults` (and its C++ twin) runs on
-// EVERY commit of every `RCTText`, so a cleared key is absent for exactly as long as it takes the
-// payload builder to supply the default again. The resolver was answering a question nothing asks.
-//
-// `PROP_ALIASES` (`id` -> `nativeID`) left this renderer on 2026-09-18 — `routeProp` resolves it
-// for every adapter now, so every path that can set a prop still reaches it.
+// RN's two TEXT DEFAULTS and `PROP_ALIASES` (`id` -> `nativeID`) are NOT resolved here: text
+// defaults run once per commit in `applyTextDefaults` (C++ twin included), and `routeProp` resolves
+// aliases for every adapter — every path that can set a prop still reaches both.
 // Angular's two-way sugar `[(value)]` compiles to a `(valueChange)` binding; the engine knows the
 // same fold as the function prop `onValueChange`. See `listen()`. The two names live in a leaf
 // module so `elements.ts`'s ControlValueAccessor can name them without importing this cyclic file.
@@ -636,19 +628,9 @@ export class SymbioteRenderer implements Renderer2 {
     this.surface.requestCommit();
   }
 
-  // FlatList/VirtualizedList cells are content projected into a component host (our
-  // ANCHOR_HOST_COMPONENTS, e.g. ScrollView) — Angular's own addLViewToLContainer
-  // (.vendors/angular node_manipulation.ts/container.ts) treats a null parent here as "defer —
-  // the child component's own <ng-content>/ɵɵprojection will place this once its structure
-  // resolves" (e.g. ScrollView's `@if(isHorizontal)` branch). Renderer2's contract types this
-  // return as nullable for exactly that reason; returning `this.surface` as a non-null fallback
-  // defeated that defer check and caused premature top-level insertion (2026-07: FlatList cells
-  // rendered outside their ScrollView).
-  //
-  // Safe only because appendChild/insertBefore above now also treat a null parent as "skip, wait
-  // for projection" — a second Angular call site (`insertAnchorNode`, hit whenever a directive
-  // does `inject(ViewContainerRef)`, e.g. VListOutletDirective) forwards this null straight into
-  // insertBefore without checking it; without that guard it crashed on-device.
+  // FlatList/VirtualizedList cells project into a component host (ANCHOR_HOST_COMPONENTS, e.g.
+  // ScrollView); Angular defers insertion on a null parent until projection resolves, so a
+  // non-null fallback here breaks that. appendChild/insertBefore/insertAnchorNode skip null too.
   parentNode(node: IHostNode): IHostElement | null {
     return parentOf(node) ?? null;
   }