lhumina_code/hero_os
5
0
Fork 0
Code Issues 30 Pull requests Projects 1 Releases Packages Wiki Activity Actions

fix(os): preserve seeded contexts when reloading from OSIS #126

Merged
zaelgohary merged 1 commit from development_fix_context_dropdown_drop into development 2026-05-04 12:13:42 +00:00
Conversation 0 Commits 1 Files changed 1 +22 -2
zaelgohary commented 2026-05-04 12:04:21 +00:00
Member
Copy link

Summary

reload_contexts_from_backend was replacing the contexts signal with whatever hero_osis_base returned. Because geomind is a hardcoded WASM seed (not registered in OSIS), creating any other context in OSIS would silently drop geomind from the Switch Context dropdown — leaving the user unable to navigate back to it from the UI.

Replace with merge: backend wins for matching names, locally-known contexts (the seeded geomind) survive if the backend has not registered them yet. Same fix applied to the mobile reload path.

Related Issue

N/A (caught live while triaging contexts).

Changes

  • Merge OSIS-fetched contexts into the signal instead of replacing it.

Test Results

Playwright in browser via tunnel (release build deployed to herodev):

  • on audit-ctx, Switch Context dropdown now lists Geomind + audit-ctx (was: audit-ctx only)
  • clicking Geomind switches the URL to /space/geomind/voice and the toolbar updates to Geomind
## Summary `reload_contexts_from_backend` was replacing the `contexts` signal with whatever `hero_osis_base` returned. Because `geomind` is a hardcoded WASM seed (not registered in OSIS), creating any other context in OSIS would silently drop `geomind` from the Switch Context dropdown — leaving the user unable to navigate back to it from the UI. Replace with merge: backend wins for matching names, locally-known contexts (the seeded `geomind`) survive if the backend has not registered them yet. Same fix applied to the mobile reload path. ## Related Issue N/A (caught live while triaging contexts). ## Changes - Merge OSIS-fetched contexts into the signal instead of replacing it. ## Test Results Playwright in browser via tunnel (release build deployed to herodev): - on `audit-ctx`, Switch Context dropdown now lists `Geomind` + `audit-ctx` (was: `audit-ctx` only) - clicking `Geomind` switches the URL to `/space/geomind/voice` and the toolbar updates to `Geomind`
fix(os): preserve seeded contexts when reloading from OSIS backend
Some checks failed
Build and Test / build (pull_request) Failing after 1m27s
Details
83a0e8b937
zaelgohary force-pushed development_fix_context_dropdown_drop from 83a0e8b937
Some checks failed
Build and Test / build (pull_request) Failing after 1m27s
Details
to 9b598f0e7c
All checks were successful
Build and Test / build (pull_request) Successful in 3m11s
Details
2026-05-04 12:10:14 +00:00 Compare
zaelgohary merged commit 4ced8c6cd7 into development 2026-05-04 12:13:42 +00:00
zaelgohary deleted branch development_fix_context_dropdown_drop 2026-05-04 12:13:42 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
prio_critical

   
prio_low

   
type_bug

   
type_contact

   
type_issue

   
type_lead

   
type_question

   
type_story

   
type_task
No labels
prio_critical
prio_low
type_bug
type_contact
type_issue
type_lead
type_question
type_story
type_task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
zaelgohary
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_os!126
No description provided.