feat: native books island, CSS theming, port alignment, and UI polish #14

Closed

mik-tf wants to merge 21 commits from development_books_native_island into development

pull from: development_books_native_island

merge into: lhumina_code:development

lhumina_code:development

lhumina_code:development_fix_tech_debt

lhumina_code:development_ci_working

lhumina_code:development_pdf_preview

Conversation 1 Commits 21 Files changed 282 +13018 -19741

mik-tf commented 2026-02-17 14:55:17 +00:00

Owner

Copy link

Summary

Consolidates PRs #8, #10, and #14 into a single PR covering:

Native Books Island (was PR #14)

• Replace iframe embed with full native Dioxus books UI calling hero_books JSON-RPC API
• 8 views: home, library browse, book detail, page viewer, search+AI summary, import, convert, settings
• AI-powered search with Groq/OpenRouter/SambaNova, voice input via Whisper
• Markdown rendering of AI summaries via pulldown-cmark
• PDF generation, version history, inline PDF viewer links
• Mermaid diagram rendering, code block copy buttons
• Breadcrumb navigation, prev/next page, sidebar table of contents

CSS Variable Theming (was PR #10)

• Replace ~119 files of hardcoded dark hex colors with CSS custom properties
• Enables light/dark theme switching from hero_os
• Core styles.rs and context.rs updated to CSS var() pattern
• hero_service island registered in new registry system

Port Alignment (was PRs #8 and #10)

• Standardize hero_os port to 8880 (was 8201)
• Align embed island ports with hero_ports registry

Infrastructure

• Replace old generated registry/router with new metadata-based registry system
• Consolidate calendar, identity, messaging sub-crates into unified crates
• 19 curl-based smoke tests for books API
• CI passing

Closes

• Closes #13 (Convert books island from iframe to native Dioxus UI)
• Closes #15 (Add AI settings view for API key configuration)
• Absorbs PR #8 (closed) and PR #10 (closed)

Dependencies

• hero_os PR #6 depends on this PR being merged first (CSS var theming)
• hero_books server required at port 8883 for books island functionality

Test Plan

• Local build with succeeds
• 19/19 smoke tests pass against hero_books API
• Books island renders correctly in Hero OS
• AI summary works and renders markdown
• PDF view/download works with correct URLs
• Settings save/load works via admin API with CORS
• CI passes

## Summary Consolidates PRs #8, #10, and #14 into a single PR covering: ### Native Books Island (was PR #14) - Replace iframe embed with full native Dioxus books UI calling hero_books JSON-RPC API - 8 views: home, library browse, book detail, page viewer, search+AI summary, import, convert, settings - AI-powered search with Groq/OpenRouter/SambaNova, voice input via Whisper - Markdown rendering of AI summaries via pulldown-cmark - PDF generation, version history, inline PDF viewer links - Mermaid diagram rendering, code block copy buttons - Breadcrumb navigation, prev/next page, sidebar table of contents ### CSS Variable Theming (was PR #10) - Replace ~119 files of hardcoded dark hex colors with CSS custom properties - Enables light/dark theme switching from hero_os - Core styles.rs and context.rs updated to CSS var() pattern - hero_service island registered in new registry system ### Port Alignment (was PRs #8 and #10) - Standardize hero_os port to 8880 (was 8201) - Align embed island ports with hero_ports registry ### Infrastructure - Replace old generated registry/router with new metadata-based registry system - Consolidate calendar, identity, messaging sub-crates into unified crates - 19 curl-based smoke tests for books API - CI passing ## Closes - Closes #13 (Convert books island from iframe to native Dioxus UI) - Closes #15 (Add AI settings view for API key configuration) - Absorbs PR #8 (closed) and PR #10 (closed) ## Dependencies - hero_os PR #6 depends on this PR being merged first (CSS var theming) - hero_books server required at port 8883 for books island functionality ## Test Plan - [x] Local build with succeeds - [x] 19/19 smoke tests pass against hero_books API - [x] Books island renders correctly in Hero OS - [x] AI summary works and renders markdown - [x] PDF view/download works with correct URLs - [x] Settings save/load works via admin API with CORS - [x] CI passes

mik-tf added 7 commits 2026-02-17 14:55:17 +00:00

docs: restructure and rewrite documentation for accuracy ... da41ad0def

- Move root ARCHITECTURE.md and STYLING_STANDARDIZATION_PLAN.md to docs/
- Delete duplicate docs (examples/os_components/ copies, obsolete journal prompt)
- Rewrite README.md: fix host attribute (was osis-url), update island list (9→30+),
  fix project structure, update build instructions, add full archipelago table
- Rewrite ARCHITECTURE.md: fix type names (IslandInfo vs IslandMetadata),
  fix trait signatures (Island returns IslandInfo, Archipelago returns ArchipelagoInfo),
  add IslandContext fields, add IslandCommand enum, fix IslandWrapper props
- Fix THEMING.md: clarify embedded/direct mode is current, Shadow DOM is legacy
- Fix WASM_OPTIMIZATION.md: correct paths (islands/ → archipelagos/files/,
  shared → core)
- Fix BUILD.md: update project structure tree with all archipelago groups
- Fix STYLING.md: remove "Plan" framing, remove stale line number references,
  consolidate hero_wasmos fixes into single section
- Fix VIEW_SYSTEM.md: update auto-discovery description
- Fix HERO_OS_SPECS.md: add roadmap disclaimer

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

docs: rewrite architecture and client integration for archipelago-centric model ... 5d707a21fe

Archipelagos are now the primary application unit that own layout, routing,
services, and data. Islands become pure view components receiving data structs
as props with zero SDK dependencies.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

refactor: flatten calendar, files, mycelium archipelagos and port contacts ... 6085e871fd

- Flatten calendar/calendar/ → calendar/ (single crate, one level up)
- Flatten files/filesystem/ → files/ (single crate, one level up)
- Flatten mycelium/mycelium/ → mycelium/ (single crate, one level up)
- Port contacts from identity/contacts/ → contacts/ (flat archipelago pattern)
- Update all workspace, server, and example path references
- Add core/src/sizing.rs (use_archipelago_size hook)
- Update core island traits and wrapper components
- Update server generated files, archipelagos.rs, island_display.rs

Part of: Unify and document archipelago framework (#12)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

refactor: merge 4 messaging crates into unified archipelago ... 8463de94fb

Merge chat, chats, chat_preview, and message into a single
hero_archipelagos_messaging crate following the flat archipelago
pattern (same as contacts). The archipelago owns routing, data
fetching, and responsive layout; islands are pure view components.

- Deduplicate get_initials() (was in 5 places), format_time helpers
- Deduplicate new_chat_view into ChatForm (pure view, receives props)
- Merge chat_service + chats_service into messaging_service
- Add archipelago.rs with Route enum, responsive sidebar layout
- Update server, examples, and workspace references

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

refactor: consolidate server metadata into single registry with Archipelago trait ... 1cbda44c35

Replace 5 overlapping metadata files (archipelagos.rs, generated/registry.rs,
generated/router.rs, generated/props_registry.rs) with a single registry.rs that
uses the Archipelago trait. Single-crate archipelagos implement the trait in their
own crates; multi-island archipelagos have trait impls defined in the server registry.
Strip build.rs down to only generate island_content.rs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat: convert books island from iframe embed to native Dioxus UI ... ac7813fd98

Replace the iframe wrapper (pointing at hero_books :8883) with a native
Dioxus component that calls the hero_books REST/JSON-RPC API directly
via gloo-net. Ports 5 views (home grid, book list, book detail, page
viewer, search+AI) using signal-based navigation and CSS custom property
theming, matching the established archipelago island pattern.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat: complete books island with all features — PDF gen, import, convert, library browse, search filters ... 9dfadc1352

Adds all remaining hero_books features to the native Dioxus island:
- Library-grouped home view with namespace browsing
- PDF generation via pdfservice RPC (generate_book, generate_page, convert_url)
- Version history dropdown with content-addressed hash links
- Collection import from git URL/path with live log streaming
- Markdown-to-PDF converter
- Search filters (library/book dropdowns via searchservice.get_filters)
- AI summary integration (Groq/OpenRouter via hero_books API)
- Moved books from "embed/Development" to "library/Library" archipelago
- 9/9 API smoketests passing

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-17 15:12:38 +00:00

feat: redesign search UX with sidebar filters, voice input, AI summary toggle, relevance bars ... fe823ea4aa

- Sidebar layout: Library, Topic (checkbox multi-select), Book filters on the left
- Typed SearchFilters struct with FilterItem (name + display_name) from API
- AI summary shown by default with collapsible toggle (arrow + header)
- Relevance bars (color-coded: purple >= 75%, amber >= 50%, gray < 50%)
- Topic badges on each result
- Voice search via Web Speech API (webkitSpeechRecognition) with mic button
- Empty state with search suggestions
- Larger search input with mic overlay

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-17 15:36:30 +00:00

feat: add Settings view for AI provider key configuration ... a7a2434fdb

- SettingsView with status grid showing per-provider key status (configured, masked, source)
- Password-masked input fields for Groq, OpenRouter, SambaNova keys + embedder URL
- Calls admin.settings.get/set RPC on hero_books admin endpoint
- Gear icon in home view header for quick access
- Registered as action view in metadata

Relates to hero_archipelagos #15, depends on hero_books #56

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-17 15:44:53 +00:00

feat: add friendly error messages for AI summary when API keys are missing ... c1ff76821d

Replaces generic error text with an AiErrorMessage component that detects
key-related errors and shows a helpful message with a "Configure API Keys"
button linking to Settings. Generic AI errors get a clean display with the
raw error in a monospace block.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-17 15:54:58 +00:00

feat: add persistent navbar and simplify HomeView to library index ... c2fe0d5871

- Add BooksNavbar component (Libraries, Search, Tools dropdown, Settings)
  that persists across all views, matching the original hero_books_ui flow
- Simplify HomeView to show library cards (name + book count) instead of
  inline book grids — clicking a library drills into LibraryView
- Replace per-view back buttons with breadcrumbs where appropriate
- Remove redundant toolbar buttons from individual views

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-17 16:02:54 +00:00

fix: library drill-down, scoped search, and API field mapping ... f82ef157e9

- Fix empty LibraryView: API returns {books: [...]} object, not array
- Use display_name for library cards, book_count from API response
- Map book "description" field to display title (matching API format)
- Make Search namespace-aware: BooksView::Search { namespace }
- Add "Search this library" button in LibraryView
- Pre-set namespace filter when entering Search from a library

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-17 16:15:32 +00:00

feat: scope search to library context with "Ask the Librarian" UX ... 14c00daaf5

- Remove global Search from navbar — search only within a library
- Make BooksView::Search require a namespace (always scoped)
- Add "Ask the Librarian" button in LibraryView
- Rename search UX to match original hero_books_ui: conversational
  "Ask a question about any topic..." placeholder, inviting empty state
- Remove Library filter from sidebar (namespace is fixed)
- Keep Topic and Book filters for refinement within the library

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-17 16:49:55 +00:00

fix: broken images, missing PDF links, tags, and content styling in books island ... eb4cd6f989

- Add rewrite_content_urls() to fix broken images (relative paths now point to books server)
- Add CONTENT_STYLE for proper HTML rendering (headings, code, tables, blockquotes, images)
- Replace RPC-based PDF generation with direct View/Download PDF links on both BookDetailView and PageContentView
- Add Regenerate as secondary action with version history on both views
- Add tags (up to 5) as pill badges on BookCard and LibraryView cards
- Add prev/next page navigation buttons to PageContentView
- Show page title in breadcrumb instead of raw page name
- Add page-level version history with content-addressed hash links
- Add "Ask the Librarian" button to BookDetailView

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-17 18:50:47 +00:00

fix: rewrite CI workflow — fix stale paths, wrong package name, add deps patching ... 40420632d8

The CI has never passed (all 130 runs failed) due to:

1. Stale island paths — messaging, calendar, contacts paths referenced
   pre-refactor subdirectories that no longer exist
2. Wrong server package name — `archipelagos-server` (hyphen) should be
   `archipelagos_server` (underscore)
3. No cross-repo dependency patching — missing deps.txt for hero_osis
   and hero_forge git dependencies

Fixes:
- Replace 50+ stale wasm-pack builds with cargo check --target wasm32
  for 12 key packages (faster, always up-to-date with workspace)
- Fix server package name to archipelagos_server
- Add deps patching step (same pattern as hero_books CI)
- Add .forgejo/deps.txt with hero_osis_sdk and heroforge_webdav_client
- Move clippy to end (non-blocking, continue-on-error)
- Remove wasm-pack install (not needed for cargo check)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf changed title from WIP: Convert books island from iframe to native Dioxus UI to Convert books island from iframe to native Dioxus UI 2026-02-17 19:21:13 +00:00

mik-tf added 1 commit 2026-02-17 19:56:22 +00:00

feat: add feature parity — mermaid, sidebar, tag filters, search context, convert history ... 05f8f713df

- Mermaid diagram rendering via CDN (auto-loads mermaid@10, dark theme)
- Page reader sidebar showing all pages in current book
- Search "View in context" button with highlight_line navigation
- Tag-based filtering in LibraryView with clickable pill badges
- BookListView "Group by Library" toggle for organized browsing
- ImportView namespace selector with quick-pick from existing libraries
- ConvertView conversion history (last 20 conversions)
- Copy-to-clipboard buttons on code blocks (hover to reveal)
- Hero banner on HomeView with gradient and description
- Tags shown in BookListView flat and grouped modes

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-18 00:59:07 +00:00

fix: polish books island — PDF URLs, nav, settings, search layout, markdown AI summary ... 1989582cf1

- Fix PDF page URLs (add missing /page/ segment) and ensure absolute URLs for generated PDFs
- Fix page navigation not updating content (add component key for re-render)
- Move search filters to right sidebar, results on left
- Unify Settings UI — merge status + input into single SettingsRow cards
- Tighten is_api_key_error() to avoid false positives on provider errors
- Render AI summary as markdown via pulldown-cmark
- Add Settings as proper nav item (replace gear icon)
- Remove duplicate "Books" branding from internal navbar
- Move prev/next page buttons to top-right breadcrumb row
- Add comprehensive curl-based smoke tests (19 tests)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 4 commits 2026-02-18 01:39:26 +00:00

fix: align all embed island ports with hero_ports registry ... 87eaa14dfc

Corrects default port assignments across all iframe embed islands and
core context to match the canonical hero_ports allocation:

- aibroker: 3751 → 3385
- shrimp: 3131 → 3384
- indexer: 9753 → 3386
- forge: 7365 → 3387
- forgejo: 3000 → 3393
- redis: 6379 → 3378
- compute: 3760 → 3389

Fixes both island.rs (props default) and lib.rs (standalone fallback)
for each embed island, plus core/src/context.rs port constants.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

fix: replace hardcoded dark-mode CSS colors with CSS custom properties ... fcd1f2808d

All islands and views now use var(--color-*) with dark-mode fallbacks
so they properly follow the host app's light/dark theme. Covers ~116
files across all archipelago groups: business, calendar, code, communication,
embed, files, identity, intelligence, library, messaging, mycelium,
productivity, and system.

Replacement patterns:
- background: #0a0a0f -> var(--color-background, #0a0a0f)
- background: #12121a -> var(--color-surface, #12121a)
- border: #1a1a25/#1e1e2e -> var(--color-border, ...)
- color: #f1f5f9/#e2e8f0 -> var(--color-text, ...)
- color: #94a3b8/#64748b -> var(--color-text-muted, ...)
- rgba(255,255,255,X) bg -> color-mix(in srgb, var(--color-text) X%, transparent)

Accent/status colors (#8b5cf6, #ef4444, #22c55e, etc.) and .colors()
metadata are intentionally unchanged.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

fix: replace hardcoded dark constants in core/styles.rs with CSS var() and register hero_service island ... 88efaca6bb

Completes light-mode support by converting CARD_BASE, CARD_HOVER, INPUT_BASE,
and TAB_INACTIVE constants to use CSS custom properties with dark fallbacks.
Also adds hero_service island to generated registry/router/content files.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

merge: consolidate PR #10 (theming, ports) and PR #8 (port 8880) into unified PR ... f7ed72a985

Absorbs changes from:
- PR #10 (development_hero_suite_fix): CSS var() theming across ~119 island files,
  port alignment with hero_ports registry, hero_service island registration
- PR #8 (development_heroports): standardize hero_os port to 8880

Conflict resolution:
- Modify/delete conflicts: kept our structural consolidation (deleted old sub-crate
  files for calendar, identity, messaging that were replaced by unified crates)
- Content conflicts: kept our structural code, adopted CSS var() theming from PR #10
- hero_service registration already present in our new registry system

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf referenced this pull request 2026-02-18 01:40:29 +00:00

fix: standardize hero_os port to 8880 #8

mik-tf referenced this pull request 2026-02-18 01:40:31 +00:00

fix: light/dark theme support, port alignment, and hero_service island #10

mik-tf changed title from Convert books island from iframe to native Dioxus UI to feat: native books island, CSS theming, port alignment, and UI polish 2026-02-18 01:40:57 +00:00

mik-tf added 1 commit 2026-02-18 01:54:14 +00:00

fix: cargo fmt + clippy fixes in core, build.rs formatting enforcement in CI ... 149afb2481

- Run cargo fmt --all to fix formatting across all crates
- Fix clippy errors in hero_archipelagos_core: derive Default, implement
  FromStr trait, fix clone-on-Copy, suppress dead_code for icon constants,
  remove useless format!, prefix unused field with _
- Fix push_str single-char in server/build.rs
- Make formatting a blocking CI step (was continue-on-error)
- Keep clippy as non-blocking due to ~200 pre-existing warnings across workspace

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf referenced this pull request 2026-02-18 01:56:59 +00:00

Convert books island from iframe to native Dioxus UI #13

mik-tf referenced this pull request 2026-02-18 01:59:22 +00:00

Add AI settings view for API key configuration #15

mik-tf added 1 commit 2026-02-18 01:59:57 +00:00

ci: re-trigger after infrastructure failure on run #135 6b01a47f69

mik-tf added 1 commit 2026-02-18 02:03:28 +00:00

fix: add rustfmt::skip to generated island_content.rs ... 085fea6fa7

build.rs generates code with long lines that rustfmt wants to reformat.
Adding #[rustfmt::skip] to the generated function prevents CI formatting
failures without requiring nightly rustfmt features.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-18 02:33:10 +00:00

fix: resolve all clippy warnings across workspace ... 7b14d1eebe

- Convert inherent from_str methods to FromStr trait (14 types)
- Gate standalone-only imports with #[cfg(feature = "standalone")]
- Fix unused variables, redundant closures, clone-on-Copy
- Add #[allow(dead_code)] for deserialization-only structs
- Fix Dioxus #[component] parameter naming (props not _props)
- Suppress too_many_arguments and type_complexity where appropriate

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf added 1 commit 2026-02-18 02:38:15 +00:00

fix: allow unused ts param in timestamp_to_date_string (cfg-gated) ... 54ff1da706

The parameter is used in the wasm32 cfg block but not in the native
fallback, so #[allow(unused_variables)] is needed for both targets.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mik-tf referenced this pull request from geomind_code/dev_docs 2026-02-18 02:54:54 +00:00

docs: comprehensive UI porting guide for archipelagos skill #19

mik-tf referenced this pull request 2026-02-19 03:01:47 +00:00

feat: native books island, CSS theming, port alignment, and UI polish #16

mik-tf commented 2026-02-19 03:01:48 +00:00

Author

Owner

Copy link

Superseded by #16 — consolidated to development_mik branch with latest development merged in.

Superseded by #16 — consolidated to `development_mik` branch with latest `development` merged in.

mik-tf closed this pull request 2026-02-19 03:01:50 +00:00

All checks were successful

Hide all checks

Build and Test / build (pull_request) Successful in 5m27s

Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No reviewers

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

Subscribe

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_archipelagos!14

No description provided.