browser removal + todo tab addition - #58
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove in-app browser, add Todo tab with history
Summary
browsermodule (binary/chrome/daemon),BrowserPane.tsx,lib/browser.ts, the bundledbrowserskill, and all related state, hooks, icons, and menu wiring.src/todo/TodoView.tsx) with full CRUD, two-phase completion (checkmark flash before archiving), and a collapsible History drawer for completed items with restore support.Worktree/TodoItemstate to persist todos and completion history across sessions.Why
The in-app browser added significant native surface complexity (daemon/chrome binary management, injected skill) for a feature that saw little use, and its removal simplifies the terminal/window surface logic. In its place, a lightweight Todo pane gives agents and users a simple way to track and restore task-level work without leaving the app.
Test plan
bun run tauri:dev— verify app launches with no references to the removed browser pane/skill.claude/skills/browseris no longer installed on launchtsc,cargo build)