fix: sidecar dylib loading, ModelSetup UX, AI prompts#10
Conversation
…l info, app pages - Expand project structure tree to list every component, store, page, lib, and Rust module - Add App Pages table (Home, Project, Templates, Settings) - Add Deploy section explaining each provider and local export - Add AI Model section with model sizes and hardware recommendations - Update Tech Stack to include SQLite details and local HTML export - Update sidecar version reference to b7472 - Replace ASCII box-drawing with portable characters Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Fix llama-server crash: copy shared libraries (.dylib) alongside binary, fix @rpath -> @loader_path, add build.rs to copy libs to target dir - Restyle ModelSetup for 300px sidebar: compact layout, show errors from aiStore - Fix chat showing raw HTML: stream status indicator instead of raw tokens, save friendly summary messages instead of HTML code - Rewrite AI system prompts: detailed styling instructions, no empty images, realistic content, extractHtml() strips markdown fences and artifacts - Add comprehensive debug logging with [AI], [ChatPanel], [ModelSetup] prefixes - Add Cargo.lock to .gitignore Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (11)
WalkthroughThe PR updates documentation with expanded feature descriptions and architecture details, adds sidecar library management to the Rust build pipeline, improves the setup-sidecar.sh script for cross-platform binary and library handling, and refactors the AI streaming state from Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~75 minutes Possibly related PRs
Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
llama-serverneeded 8 shared libraries (.dylib) that weren't being copied. Updatedsetup-sidecar.shto extract all dylibs, fix@rpath→@loader_path, and addedbuild.rslogic to copy libs totarget/debug/extractHtml()strips markdown fences and artifacts from preview[AI],[ChatPanel],[ModelSetup]prefixes throughout model download/sidecar flowdocs/and deployed to GitHub PagesTest plan
bash scripts/setup-sidecar.sh— verify dylibs are copied and binary runspnpm tauri dev— verify sidecar starts (check[AI] Sidecar running on portin console)🤖 Generated with Claude Code
Summary by CodeRabbit
Release Notes
New Features
Documentation
Improvements