coop/rose-ash
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Pass all values as &key args to ~root-header/~root-mobile

Browse Source 
Nested component calls in _aser are serialized without body expansion,
so free variables inside ~root-header would be sent unresolved to the
client. Fix by making ~root-header/~root-mobile take all values as
&key params, and having parent layout defcomps pass them explicitly.
The parent layout bodies ARE expanded (via async_eval_slot_to_sx),
so their free variables resolve correctly during that expansion.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
giles
2026-03-04 14:50:14 +00:00
parent e4bfd46c48
commit a30e7228d8
4 changed files with 55 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
account/sx/layouts.sx
View File

@@ -1,9 +1,12 @@
;; Account layout defcomps — read ctx values from env free variables.
;; Registered via register_sx_layout("account", ...) in __init__.py.
;; Free variables come from _ctx_to_env() in shared/sx/helpers.py.
;; Full page: root header + auth header row in header-child
(defcomp ~account-layout-full ()
(<> (~root-header)
(<> (~root-header :cart-mini cart-mini :blog-url blog-url :site-title site-title
:app-label app-label :nav-tree nav-tree :auth-menu auth-menu
:nav-panel nav-panel :settings-url settings-url :is-admin is-admin)
(~header-child-sx
:inner (~auth-header-row :account-url account-url
:select-colours select-colours
@@ -15,7 +18,10 @@
:select-colours select-colours
:account-nav account-nav
:oob true)
(~root-header :oob true)))
(~root-header :cart-mini cart-mini :blog-url blog-url :site-title site-title
:app-label app-label :nav-tree nav-tree :auth-menu auth-menu
:nav-panel nav-panel :settings-url settings-url :is-admin is-admin
:oob true)))
;; Mobile menu: auth section + root nav
(defcomp ~account-layout-mobile ()
@@ -24,4 +30,4 @@
:items (~auth-nav-items :account-url account-url
:select-colours select-colours
:account-nav account-nav))
(~root-mobile)))
(~root-mobile :nav-tree nav-tree :auth-menu auth-menu)))