Skip to content
This repository was archived by the owner on Mar 5, 2026. It is now read-only.

chore: update install.sh and frontend assets for improved installation experience#12

Merged
andersonleal merged 2 commits intomainfrom
feat/fix-streams
Feb 13, 2026
Merged

chore: update install.sh and frontend assets for improved installation experience#12
andersonleal merged 2 commits intomainfrom
feat/fix-streams

Conversation

@andersonleal
Copy link
Copy Markdown
Contributor

@andersonleal andersonleal commented Feb 13, 2026 •
edited by coderabbitai bot
Loading

chore: update install.sh and frontend assets for improved installation experience

Summary

This PR improves the iii-console installation experience and fixes stream listing in the bridge. Key changes:

  1. Stream listing fix — Corrects the bridge response parsing for stream.listAll to use stream instead of streams, matching the engine's actual response format. This fixes the streams view in the console.

  2. Install script simplification — Removes GITHUB_TOKEN support and authenticated download paths from install.sh. The script now uses only public browser_download_url for downloads, simplifying maintenance and reducing complexity. Private repo support and rate-limit workarounds are removed.

  3. Favicon and PWA assets — Adds a complete favicon set and web manifest for the console frontend:

    • favicon.ico, favicon-16x16.png, favicon-32x32.png
    • apple-touch-icon.png (180x180)
    • android-chrome-192x192.png, android-chrome-512x512.png
    • site.webmanifest with theme colors (#0A0A0A)

  4. Version bump — Cargo.lock updated to iii-console 0.1.5.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • Enhancement (non-breaking change that improves functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • Code follows project style guidelines (Biome for frontend, cargo fmt for Rust)
  • Self-review performed
  • Documentation updated (if applicable)
  • No new warnings introduced
  • Changes build successfully

Additional Context

  • Stream fix: The iii engine's stream.listAll returns { "stream": [...] }, not { "streams": [...] }. The bridge was incorrectly looking for streams, causing the streams list to always appear empty.
  • Install script: GITHUB_TOKEN and asset_id-based authenticated downloads were removed. The console repo is public, so unauthenticated downloads suffice. This reduces script size and maintenance surface.
  • Favicons: The new assets follow standard PWA/favicon conventions for better branding when the console is bookmarked or added to home screen.

Summary by CodeRabbit

  • Documentation

    • Added comprehensive installation instructions including curl-based installer and manual downloads for multiple platforms.
    • Added macOS Gatekeeper bypass guidance.

  • New Features

    • Added favicon and Progressive Web App support to the console.

  • Improvements

    • Simplified installer to work with public URLs, removing authentication token requirement.

@andersonleal
chore: update install.sh and frontend assets for improved installatio…
7d4417b 
…n experience

Removed GITHUB_TOKEN references from install.sh, simplifying the download process. Enhanced index.html with favicon links and added new favicon assets for better branding. Updated version in Cargo.lock to 0.1.5.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 13, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

Updated project documentation with installation instructions via installer script and manual downloads, simplified the installer by removing GitHub token-based authentication support, enhanced console frontend with favicon and PWA manifest configuration, and fixed a data key reference in the Rust bridge.

Changes

Cohort / File(s) Summary
Documentation and Installation
README.md, install.sh		 Added comprehensive installation sections (script-based and manual) to README; simplified install.sh by removing GITHUB_TOKEN environment variable, asset_id handling, and authenticated API paths, relying solely on public URLs.
Frontend Configuration
packages/console-frontend/index.html, packages/console-frontend/public/favicon/site.webmanifest		 Updated HTML head with favicon links (ICO, PNG variants, Apple touch icon), site manifest reference, and adjusted console title casing; added new PWA manifest with app metadata including icons, theme colors, and standalone display mode.
Rust Bridge Fix
packages/console-rust/src/bridge/functions.rs		 Corrected data key reference in handle_streams_list from "streams" to "stream" when reading engine response.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Suggested reviewers

  • sergiofilhowz

Poem

🐰 With manifests and favicons so bright,
The installer now flows, clean and light,
No tokens needed, just public paths shine,
The console springs forth, looking fine—
A polish, a fix, all in one fine line! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title mentions 'install.sh and frontend assets' which are real parts of the changeset, but omits the key fix (stream listing parsing) and README updates, making it only partially aligned with the main changes.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/fix-streams

No actionable comments were generated in the recent review. 🎉

Comment @coderabbitai help to get the list of available commands and usage tips.

@andersonleal
docs: update README with installation instructions and script details
2165779 
Added a recommended install script section to the README, including commands for installing a specific version and to a custom directory. Removed the optional checksum verification section to streamline the installation process.
@andersonleal andersonleal merged commit c86f7c9 into main Feb 13, 2026
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@ytallo ytallo ytallo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@andersonleal @ytallo