Repository navigation
feat(android): use icon buttons for back and settings - #303
Merged
Merged
Conversation
Replace the Back text buttons on the agent, chat and settings screens and the Settings text button on the task list with icon buttons, and add a plus icon to the New task button. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EezzT287BK4Co1sTK9W1Aq
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EezzT287BK4Co1sTK9W1Aq
The phone's New task screen now chooses the project, agent and model from dropdowns instead of a list of radio cards. The desktop serves GET /api/mobile/agents with the available agents and the models each offers (Claude aliases, Codex models from its CLI cache), and POST /api/mobile/tasks accepts optional agentId and model. The renderer checks the choice against the same list and applies the model with the existing reviewer model override. Older phones and desktops keep working: without a choice the desktop default agent runs, and the app hides the agent pickers when a desktop lacks the route. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EezzT287BK4Co1sTK9W1Aq
…tasks Creating a task makes the desktop build a git worktree before it answers, and it allows itself 120 s for that. The app's HTTP client kept OkHttp's default 10 s read timeout, so a slower creation failed on the phone with "Could not reach your computer" even though the desktop went on to create the task. Merge and close wait on git the same way. These requests now get a read timeout just above the desktop's, and a timeout on them says the computer did not answer in time instead of calling it unreachable. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EezzT287BK4Co1sTK9W1Aq
- Treat 403 as an older desktop without the agents route; that is what it answers a paired token, not 404. - Report a malformed agents reply as an error instead of crashing. - Give the diff and merge readiness the longer timeout too: the desktop runs git for both. - Don't flash the older-desktop agent note while agents load. - Keep new declarations below the doc comments they had displaced. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EezzT287BK4Co1sTK9W1Aq
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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
2 participants
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.
Replace the Back text buttons on the agent, chat and settings screens and
the Settings text button on the task list with icon buttons, and add a
plus icon to the New task button.
Co-Authored-By: Claude Opus 5.5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01EezzT287BK4Co1sTK9W1Aq