Repository navigation
Conversation
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — Clarifies connection, pairing, host, and permission wording across mobile and web, including host-specific onboarding labels. Existing connection behavior, access controls, and product defaults are unchanged. You can add or adjust custom eligibility rules. Learn more. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the
Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Connection and host labels apps/mobile/src/features/connection/ConnectionsNewRouteScreen.tsx, apps/web/src/components/HostedEmptyState.tsx |
The T3 Connect heading and hosted environment messages use updated connection and host terminology. |
Pairing and permission guidance apps/mobile/src/features/settings/EnvironmentRoutesSection.tsx, apps/web/src/components/settings/AuthorizedClientsSettings.tsx, apps/web/src/components/settings/SessionPermissions.tsx |
Pairing instructions describe using a pairing code and server address. Permission notices describe the scope of the current connection. |
Access and mobile activity wording apps/web/src/components/cloud/ConnectOnboardingDialog.tsx, apps/web/src/components/settings/SavedEnvironmentRows.tsx, apps/web/src/components/settings/settingsSearch.ts |
The onboarding dialog, saved environment settings, and settings search use updated language for T3 Connect access and sending agent activity to mobile devices. |
Priority:
Estimated code review effort: 1 (Trivial) | ~5 minutes
Change: Other
Merge Risk: 
Minimal · up to 9594d
-
No actionable issue remains from this review; the change is mergeable after normal checks.
Pre-merge checks: 
4
-
Full details: Description check
-
Explanation
The description includes the required Problem, Change, Scope and approval, Verification, and UI evidence sections. It clearly describes the copy changes, approval basis, verification results, limitations, and before/after screenshots. It does not include a direct approval link or maintainer approval comment, and the screenshots cover only the pairing dialog, but the description is otherwise mostly complete.
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
Dismissing prior approval to re-evaluate 98aadca
Dismissing prior approval to re-evaluate 4e55003
4e55003 to
4be7f76
Compare
Dismissing prior approval to re-evaluate 4be7f76
Dismissing prior approval to re-evaluate 9594df3
|
Note Written by thanks @extoci. your copy makes disabled hosts, pairing, connection permissions, and mobile activity sharing clearer while preserving the existing actions. merged current main into your branch without rewriting its three commits. resolved the onboarding and settings-search conflicts, kept main's account/tunnel-limit controls and verification at
matching pairing-dialog captures using current main's copy and this head: |
|
Note Written by Verified in a real dev environmentverified unchanged head
Not verifiednative iOS/Android and Electron shell behavior were not exercised on this linux web pass. signed-in T3 Connect onboarding, host-name interpolation, tunnel/activity toggles and notifications require an account and other devices; those flows were not exercised. hosted no-host/disabled-host states, remote-route permission help, and the loopback QR explanation were not reached in this local-server setup. other providers and cross-network pairing were not exercised. the initial thread view did not display the completed provider response until reload/reopening; live reply rendering is therefore not verified as healthy. the response persisted and rendered afterward. this eight-file copy-only diff does not change that path, and no unrelated fix was made. uploaded video playback and seeking were checked in chrome. task dev servers are stopped after verification. |




Problem
Connection setup mixes host, backend, client, and publishing terminology. The instructions make users work out what they are connecting and who gets access.
Change
Use the approved host, pairing, permission, and T3 Connect wording in onboarding and Connections settings. Update the matching mobile pairing and permission text, plus the agent-activity setting and its accessibility label.
Copy before and after
Hosts and connection setup
Pairing and permissions
Agent activity on mobile devices
The setup step and remote-access help use the current host's name, for example, "Share MacBook Pro" and "Connect to MacBook Pro from other devices signed in to your account." The empty home screen has no connected host to name, so the highlighted sentence keeps "computer". Its other instructions use "host".
The updated switch label also appears in settings search. Desktop uses the same web copy.
Scope and approval
Copy-only maintenance. The exact wording and four-PR split were requested and approved in the originating T3 Code conversation. Runtime behavior and defaults are unchanged.
Verification
vp run check:changedpasses on this independent branch.Screenshots
Before and after: Create pairing link dialog
Before
After
Model: GPT-6.1-Sol. Harness: Codex in T3 Code.