feat(office): wire hero_archipelagos_office crate in place of inline iframes #88

Merged
zaelgohary merged 5 commits from development_wire_office_crate into development 2026-04-23 13:28:21 +00:00
Member

Summary

Replace the inline OnlyOffice iframe + per-island metadata wiring with the new hero_archipelagos_office crate. Collapses 5 match arms + metadata calls into a single DocType-driven pair.

#89

Depends on hero_archipelagos#146 (PR #145) — CI will go green once that merges and cargo update -p hero_archipelagos_office picks up the new development HEAD.

Changes

  • crates/hero_os_app/src/island_content.rs — office routes unified behind DocType::from_island_id
  • crates/hero_os_app/src/registry.rs — single metadata import for office
  • crates/hero_os_app/Cargo.toml — add hero_archipelagos_office + island-office feature
  • Cargo.toml[patch] entry for hero_archipelagos_core + commented local-path entry for hero_archipelagos_office

Test Results

cargo check --workspace ✓, cargo fmt --check -p hero_os_app ✓, cargo clippy -p hero_os_app -- -D warnings ✓.

Manual verification

End-to-end on herodev: opened Hello.xlsx, Hello.docx, and openpyxl-generated Analysis.xlsx in the editor — all render. Upload flow PUTs a valid xlsx into hero_foundry and the file appears in the list.

## Summary Replace the inline OnlyOffice iframe + per-island metadata wiring with the new `hero_archipelagos_office` crate. Collapses 5 match arms + metadata calls into a single `DocType`-driven pair. ## Related Issue #89 Depends on hero_archipelagos#146 (PR #145) — CI will go green once that merges and `cargo update -p hero_archipelagos_office` picks up the new `development` HEAD. ## Changes - `crates/hero_os_app/src/island_content.rs` — office routes unified behind `DocType::from_island_id` - `crates/hero_os_app/src/registry.rs` — single metadata import for office - `crates/hero_os_app/Cargo.toml` — add `hero_archipelagos_office` + `island-office` feature - `Cargo.toml` — `[patch]` entry for `hero_archipelagos_core` + commented local-path entry for `hero_archipelagos_office` ## Test Results `cargo check --workspace` ✓, `cargo fmt --check -p hero_os_app` ✓, `cargo clippy -p hero_os_app -- -D warnings` ✓. ## Manual verification End-to-end on herodev: opened Hello.xlsx, Hello.docx, and openpyxl-generated Analysis.xlsx in the editor — all render. Upload flow PUTs a valid xlsx into hero_foundry and the file appears in the list.
chore(deps): bump hero_archipelagos pin to fcc0bca2 (office crate merged)
Some checks failed
Build and Test / build (pull_request) Failing after 5s
72e1c29140
zaelgohary merged commit 50d1e8ed73 into development 2026-04-23 13:28:21 +00:00
zaelgohary deleted branch development_wire_office_crate 2026-04-23 13:28:22 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
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_os!88
No description provided.