lhumina_code/hero_demo
26
0
Fork 0
Code Issues 43 Pull requests Projects Releases 2 Packages Wiki Activity Actions

[nu-demo] hero_os_app: island-room (and possibly others) missing from web feature — WASM 404 at runtime #14

New issue
Open
opened 2026-04-28 12:20:05 +00:00 by mik-tf · 0 comments
mik-tf commented 2026-04-28 12:20:05 +00:00
Owner
Copy link

Symptom

Clicking Rooms (under Intelligence in the dock) shows:

Failed to load island WASM 'room': JsValue(Error: Island not found: room (HTTP 404))
at eval (eval at <anonymous> (https://heronu.gent01.grid.tf/hero_os/ui/wasm/hero_os_app.js:551:25), <anonymous>:4:23)

Root cause

hero_os/crates/hero_os_app/Cargo.toml:43 defines the feature:

island-room = ["dep:hero_archipelagos_room", "hero_archipelagos_room/web"]

But the web feature list (line 117+) does NOT include island-room:

web = [
    "core",
    "island-ai", "island-messaging", "island-call", "island-livekit",
    "island-calendar", "island-filesystem", "island-tasks",
    ...
]

The dock does route to the room island (registry.rs registers it under Intelligence archipelago), but the WASM bundle doesn't contain the code, so runtime dispatch 404s.

Same class of bug as the earlier island-videos regression (fixed in our rebuild by adding FEATURES="web,island-videos"). Suspect other features are also drift: island-knowledge, island-intelligence, island-services all appear in web-native but not web.

Demo workaround (planned)

Rebuild hero_os_app with FEATURES="web,island-videos,island-room" (and grep the registry/dock for any other referenced islands missing from web). One-time ~8 min.

Proper fix

  1. Audit: compare registry.rs dock/archipelago entries against the web feature list. Any referenced island missing from web = incomplete build.
  2. Make it impossible: have the build script (or a compile-time check) assert that every v.push(..._metadata()) call in registry.rs is behind a #[cfg(feature = "...")] whose feature is in the web list. Fail the build otherwise.
  3. Tidy: consider merging web and web-native into a single web feature with a sub-flag for native islands, so the two lists can't drift out of sync.

Related: native islands use web-native — some are already registered. Unifying would eliminate duplication.

Linked

  • Videos was the same class of bug — we rebuilt with island-videos after the fact.
  • home#122 — broader pattern of hardcoded defaults.

Filed 2026-04-23 nu-shell demo bring-up. Signed-off-by: mik-tf

Originally filed as home#123 on 2026-04-23 by mik-tf — moved to hero_demo as part of consolidating issue tracking.

## Symptom Clicking **Rooms** (under Intelligence in the dock) shows: ``` Failed to load island WASM 'room': JsValue(Error: Island not found: room (HTTP 404)) at eval (eval at <anonymous> (https://heronu.gent01.grid.tf/hero_os/ui/wasm/hero_os_app.js:551:25), <anonymous>:4:23) ``` ## Root cause `hero_os/crates/hero_os_app/Cargo.toml:43` defines the feature: ```toml island-room = ["dep:hero_archipelagos_room", "hero_archipelagos_room/web"] ``` But the `web` feature list (line 117+) does NOT include `island-room`: ```toml web = [ "core", "island-ai", "island-messaging", "island-call", "island-livekit", "island-calendar", "island-filesystem", "island-tasks", ... ] ``` The dock **does** route to the `room` island (registry.rs registers it under Intelligence archipelago), but the WASM bundle doesn't contain the code, so runtime dispatch 404s. Same class of bug as the earlier `island-videos` regression (fixed in our rebuild by adding `FEATURES="web,island-videos"`). Suspect other features are also drift: `island-knowledge`, `island-intelligence`, `island-services` all appear in `web-native` but not `web`. ## Demo workaround (planned) Rebuild `hero_os_app` with `FEATURES="web,island-videos,island-room"` (and grep the registry/dock for any other referenced islands missing from `web`). One-time ~8 min. ## Proper fix 1. **Audit**: compare `registry.rs` dock/archipelago entries against the `web` feature list. Any referenced island missing from `web` = incomplete build. 2. **Make it impossible**: have the build script (or a compile-time check) assert that every `v.push(..._metadata())` call in `registry.rs` is behind a `#[cfg(feature = "...")]` whose feature is in the `web` list. Fail the build otherwise. 3. **Tidy**: consider merging `web` and `web-native` into a single `web` feature with a sub-flag for native islands, so the two lists can't drift out of sync. Related: native islands use `web-native` — some are already registered. Unifying would eliminate duplication. ## Linked - Videos was the same class of bug — we rebuilt with `island-videos` after the fact. - [home#122](https://forge.ourworld.tf/lhumina_code/home/issues/122) — broader pattern of hardcoded defaults. Filed 2026-04-23 nu-shell demo bring-up. Signed-off-by: mik-tf --- *Originally filed as [home#123](https://forge.ourworld.tf/lhumina_code/home/issues/123) on 2026-04-23 by mik-tf — moved to hero_demo as part of consolidating issue tracking.*
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
development
development_mik2
v0.9.3-dev
v0.9.0-dev
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
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_demo#14
No description provided.