Skip to content

fix: resolved a bug in mobile connectivity by updating Swift and Kotlin code files for iOS and Android#97

Merged
sirily11 merged 9 commits into
mainfrom
bugfix
Jun 5, 2026
Merged

fix: resolved a bug in mobile connectivity by updating Swift and Kotlin code files for iOS and Android#97
sirily11 merged 9 commits into
mainfrom
bugfix

Conversation

@autopilot-project-manager

Copy link
Copy Markdown
Contributor
  • Updated Swift and Kotlin code files for iOS and Android.
  • Implemented relay normalization and composite ID construction for iOS.
  • Implemented companion object and relay-aware pairing logic for Android.
  • Test builds verified successfully for both platforms.
  • SourceKit diagnostic issues and UI tests remain unresolved.

sirily11 and others added 4 commits June 5, 2026 03:19
The sidebar review dot was driven by AppState.reviewPassedBySession, an
in-memory map that reset on launch. Persist the verdict on the ChatThread
row and rehydrate the map at startup.

- ChatThread: add persisted reviewPassed Bool? field (defaulted for migration)
- ThreadStore: setReviewPassed(sessionId:passed:) + loadReviewVerdicts()
- AppStateHookController: persist verdict alongside in-memory write
- AppState+Lifecycle: rehydrate reviewPassedBySession at launch

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Introduce a Codable + SwiftUI-View MenuItem so desktop and mobile render the
same context menus. Migrate the manual project/thread actions (code review,
commit, create PR) and the autopilot setup items (secrets, docs, release, CI)
to hook-built MenuItems. The desktop renders them natively and dispatches
commands locally; mobile fetches the same hook-built JSON menu from the desktop
over the relay and executes returned commands, opening deep links for on-device
sheets.

- RxCodeCore: MenuItem/MenuAction/MenuActionCommand + menuActionHandler env,
  MenuDeepLink helper; Hook context-menu methods now return [MenuItem];
  ProjectContextMenuPayload carries optional branch.
- App: ActionsMenuHook (one hook for project + thread menus), AppState menu
  dispatcher with branch-aware code review / PR, branch-aware open-PR check
  (only .open hides Create PR); desktop views render MenuItemsView and navigate
  to spawned threads.
- Transport: menuForProject/menuForThread/menuExecuteCommand ops + DTOs.
- Mobile: prefetch + render hook menus (invalidated on git/PR/file-edit state)
  across thread list, project sidebar, and briefing cards; shared deep-link
  router.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Strengthen the reviewer prompts (hook + manual thread/branch reviews) to
instruct the reviewer to write a complete, self-contained final message,
since that text is what gets folded onto the parent thread and fed into
any auto-fix turn.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
rxcode Ready Ready Preview, Comment Jun 5, 2026 6:47am

Request Review

@sirily11 sirily11 changed the title fix: resolved a bug in mobile connectivity by updating Swift and Kotlin code files for iOS and Android, including implementing relay normalization and composite ID construction for iOS and companion object and relay-aware pairing logic for Android. Test builds verified successfully for both platforms. ## Changes fix: resolved a bug in mobile connectivity by updating Swift and Kotlin code files for iOS and Android Jun 5, 2026
@autopilot-project-manager autopilot-project-manager Bot added the bug Something isn't working label Jun 5, 2026
Fold the reviewer's final assistant text into the parent-thread hook card
instead of the full thread transcript, so the card shows exactly the same
self-contained verdict that's fed into any auto-fix turn — no intermediate
review chatter. Falls back to the transcript only when the final message
is empty (cancelled/partial review).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
sirily11 and others added 2 commits June 5, 2026 13:22
Add a 60s debounce before an automatic code review starts. The
CodeReviewHook inserts an interactive countdown card (ReviewCountdownCard
contract + ReviewCountdownCardView) and waits on a ReviewScheduler gate so
a follow-up message, the Stop button, or a "Stop Code Review" context-menu
item cancels the review, while "Start it now" skips the wait. Skip reviews
for turns already superseded by newer thread activity, and surface
cancelled/stopped reviews with reason-aware card copy.

Also consolidate CIUpdateMenuHook into CIUpdateHook (banner + context-menu
item in one hook) and hide "Set Up CI Update" for repos already watched for
CI auto-updates via projectHasCIUpdates.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Move the threadLabel chip (e.g. "Commit") into the trailing
icon/time cluster so it aligns with other rows' status icons
instead of hugging the title.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…ition checks

Nested child rows in the sidebar hardcoded a "Review N" title and hid the
label chip for every child, mislabeling Send Message hook threads. Number
only code-review children; show other linked children's own title and label.

Also surface the Send Message hook's condition gate, which ran as a silent
background classifier, by inserting a card with the condition and its
YES/NO/error verdict so NO/error skips are no longer invisible.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Gate the Stop Review / Start it now buttons at compile time (#if os(macOS))
instead of on a non-nil reviewCountdownHandler binding, so they reliably
render during the countdown on desktop. Read the handler at tap time rather
than binding it in body. iOS still compiles out the buttons (read-only).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sirily11 sirily11 enabled auto-merge (squash) June 5, 2026 06:47
@sirily11 sirily11 merged commit c27f3d3 into main Jun 5, 2026
14 checks passed
@sirily11 sirily11 deleted the bugfix branch June 5, 2026 07:06
@sirily11

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.15.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant