refactor: Adopt 'internalId' for Stable DOM Keying (List & Data Views)

Input from Gemini 3 Pro:

✦ Fix: Helix/Gallery Navigation, VNode Crash & Store Identity

Critical Fixes:

1. Store Identity Race Condition (src/data/Store.mjs):

   • Issue: itemFactory was assigned after super.construct, causing initial data loads (e.g. ComboBox data, Helix loadData timing) to bypass internalId assignment.
   • Fix: Moved itemFactory assignment to the config object passed to super.construct.
   • Enhancement: Added "Lazy Assignment" to getInternalId. If an item lacks an ID (e.g. race condition), it is assigned on first access. This is a fail-safe.

2. Helix & Gallery Navigation (src/selection/HelixModel.mjs, GalleryModel.mjs):

   • Issue: store.indexOf(selected) returned -1 because selected is an internalId but indexOf expects a PK/Record.
   • Fix: Refactored onNavKey* to first resolve the record from the internalId (using robust lookup), then find its index.
   • Fix: Updated logic to respect view.maxItems, preventing navigation into "ghost" items (loaded but not rendered).

3. VNode ID Crash (src/component/Helix.mjs):

   • Issue: expandItem created a new VDOM node without an ID, causing the VDOM Worker to reject the update.
   • Fix: Explicitly assigned a unique ID to the new node.
   • Fix: Updated expandItem and destroyClones to use robust record lookup (store.items.find fallback).

4. ComboBox Value Update (src/form/field/ComboBox.mjs):

   • Issue: Selection change event passed internalId, but store.get failed.
   • Fix: Updated onListItemSelectionChange to use the robust record lookup pattern.

Refactor:

• Updated List.Base, ListModel, Gallery.mjs to consistently use getRecordId and robust lookups.

Outcome: Helix/Gallery navigation, camera movement, expansion, and ComboBox selection are fully restored and robust against ID mismatches.

Input from Gemini 3 Pro:

✦ Correction: Helix Fully Restored, Others WIP

To clarify the previous status update:

✅ Helix Component (Fixed & Verified):

• Navigation: Arrow key navigation is fully restored and respects maxItems.
• Expansion: Enter to expand/collapse works correctly (fixed VNode ID crash and destroyClones crash).
• Camera: Movement is synced with selection.
• Bonus: Fixed two pre-existing bugs related to VDOM ID generation and null pointer exceptions during collapse.

⚠️ Work in Progress (ComboBox & Gallery): While logic fixes were applied for ComboBox (value update) and Gallery (camera/selection), they require further verification to be considered "fully restored". The Store race condition fix is expected to resolve the underlying ID availability issues for these components, but they remain in a WIP state until confirmed.

Input from Gemini 3 Pro:

✦ I have pushed the fix for ComboBox to the dev branch.

Changes:

1. src/form/field/ComboBox.mjs:
   • Fix: Updated onListItemSelectionChange to parse the selected item's DOM ID using list.getItemRecordId() before attempting to retrieve the record from the store. This ensures compatibility with the new internalId system used by List.
2. Tests:
   • Added test/playwright/unit/form/field/ComboBoxInternalId.spec.mjs to verify that ComboBox selection works correctly when the underlying List uses internalId.

Outcome: Selecting an item from the ComboBox picker now correctly updates the field's value. The mismatch between DOM IDs (prefixed) and Store IDs (raw) has been resolved.