fix(bin): prioritize bridge talk and photo controls - #98
Merged
Conversation
The captain often opens the phone page just for those utilities, so they need to sit above the fleet glance. Co-authored-by: Cursor <cursoragent@cursor.com>
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.
What Changed
Risk Assessment
✅ Low: The change only reorders existing authenticated-page sections and updates the matching documentation and ordering assertion; no broken DOM assumptions, call-site regressions, or material risks were found.
Testing
The focused bridge behavior suite passed, and an authenticated end-to-end phone render visibly confirmed Talk and Send photos appear first above every fleet-glance section; the worktree remained clean.
/var/folders/1g/hctp3vpn27b1zrlsn4nsfg680000gn/T/no-mistakes-evidence/01M0Q9E0XRBYKRTRTJ792W0CZS/bridge-phone.png)Evidence: Authenticated bridge page returned by the running server
Pipeline
Updates from git push no-mistakes
⏭️ **intent** - skipped
✅ No issues found.
✅ **Rebase** - passed
✅ No issues found.
✅ **Review** - passed
✅ No issues found.
✅ **Test** - passed
✅ No issues found.
Inspectedgit diff 9f158d3f25ca1d2182cc168ce806a7c3a94df00f..871c2655f03b68c887e45953a9554f08dfb97c7dto establish user intent and affected behaviorbin/fm-test-run.sh tests/fm-bridge-view.test.shStartedbin/fm-bridge-view.sh serve --host bridge.localhost --port 18766with an isolated evidence home, initialized authentication, logged in via HTTP request, and captured the authenticated page responseRendered the authenticated response at a 390×844 phone viewport and visually inspected/var/folders/1g/hctp3vpn27b1zrlsn4nsfg680000gn/T/no-mistakes-evidence/01M0Q9E0XRBYKRTRTJ792W0CZS/bridge-phone.pngVerifiedgit status --shortremained clean after testing✅ **Document** - passed
✅ No issues found.
✅ **Lint** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.