pingdotgg/t3code. All authors. Drafts included. Default branch main. 570 issue assessments and 925 PR assessments. Initial inventory: 570 open issues and 924 open PRs. Current assessed open inventory: 570 issues and 924 PRs. Final reconciliation: 2026-09-01T11:57:41.491617+00:00.
Request. Add the iOS Keychain entitlement needed by SecureStore during pairing.
Audit finding. Main still uses SecureStore and does not declare this access group. Signing and Expo configuration have changed since the reported simulator failure, and the diff uses variant.iosBundleIdentifier rather than the final iosBundleIdentifier used by current builds. The body claims a config test, but the diff contains only the entitlement.
Recommendation. Keep open: retest. Reproduce SecureStore pairing on the current SDK57 build and use the resolved bundle identifier if the entitlement is still needed.
Confidence medium. Release: Not applicable. PR readiness: Needs small changes.
Request. Keep a grouped project new-thread target independent of the active search.
Audit finding. Main still derives lastActiveProject from matchingThreads, so a search can change the chosen environment. The PR fixes that source of instability, but main now also resolves stale project IDs to a canonical project path. Applying the old project-ID lookup unchanged would discard that later fix.
Recommendation. Keep open: work remains. Update the full-history lookup to retain canonical project resolution and test a multi-environment search.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Select an installed compatible Node version when the remote login shell exposes an incompatible Node.
Audit finding. Main still skips version-manager fallback when an incompatible Node exists on PATH, despite the merged fix for silent CLI startup. The diff repairs selection, but its final remote_node_ready call suppresses the useful version/range error as the review notes. Keep the fallback change and preserve that final diagnostic.
Recommendation. Keep open: work remains. Keep the final engine check unsilenced and add a two-version Node selection test.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Keep dev desktop web requests on the LAN host when its UI is opened in a remote browser.
Audit finding. The discussion narrows the reproduction to a browser opening the network URL of dev:desktop, not the browser dev runner. The merged single-origin change fixes dev and dev:web, but the desktop branch still deliberately bakes loopback HTTP and WebSocket URLs. This PR therefore retains a distinct desktop-dev LAN case and should not be closed as fixed.
Recommendation. Keep open: partial fix. Update this PR for remote-browser access to dev:desktop while preserving desktop bridge precedence.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Merged pr: PR #4556. Fixes only browser dev single-origin routing.
Comment: Discussion comment. Author gives a dev:desktop LAN-browser reproduction and confirms preview URL work did not fix it.
Independent closure check. The full discussion identifies a distinct dev:desktop LAN-browser reproduction, not normal browser development. The complete diff addresses configured loopback HTTP and WebSocket targets while preserving desktop bridge precedence. Pinned dev-runner still sets loopback origins specifically for desktop mode, and a remote browser has no desktop bridge to override them. The browser-only single-origin merge therefore does not cover the whole report. The cohort withdrew its earlier closure while this independent review was in progress.
Request. Avoid recursive settings and keybinding watcher expansion in Android proot.
Audit finding. Main still passes both configuration directories to Effect FileSystem.watch and has no PROOT_L2S_DIR-specific watcher. The PR adds a non-recursive node:fs watcher only for proot and keeps filename filtering and debounce at both callers. Its reported reduction from about 900 watcher handles to two is author evidence, not a reproduced measurement here.
Recommendation. Keep open: work remains. Add focused watcher cleanup and file-reload coverage, then verify handle counts under proot.
Confidence medium. Release: Not applicable. PR readiness: Needs small changes.
Request. Remove a deleted active thread from another connected client.
Audit finding. Main still excludes thread.deleted from the thread-detail event filter, while the shared reducer already handles that event. The web route still only redirects a missing thread if another thread exists, so deleting the last thread remains a separate gap. The patch covers both web defects, but its new server test waits on a timed sleep and mobile navigation is explicitly outside its scope.
Recommendation. Keep open: work remains. Update the server test to use an explicit stream handshake and review the two-client deletion fix.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Canonical current-head results are missing for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports conflicts with the target branch, so recorded checks do not validate a rebased integration.
Request. Keep completed assistant copy controls visible after the pointer leaves the message.
Audit finding. Main still hides the completed assistant footer outside hover or focus. The PR removes that restriction, but its regression regex searches for opacity-0 after the data attribute even though React emits class first, so the test cannot detect the regression. Review also identifies the unused group/assistant marker left by the change.
Recommendation. Keep open: work remains. Revise the footer patch to remove the unused hover group and test the selection-and-pointer interaction.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Restore composer DOM focus after browser automation sends a key press.
Audit finding. Main still calls bridge.automation.press directly and does not restore the active renderer element. The complete diff adds a small finally-based focus helper and tests success, failure, and detached elements. New keyboard-isolation work overlaps this path, but it is not on main.
Recommendation. Keep open: work remains. Review the focus helper against the keyboard-isolation branch before choosing one implementation.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Pr: PR #8494. Unmerged keyboard-isolation work changes the same input path.
Limits. Native focus behavior was not reproduced. The collected latest head has no results for these required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports merge conflicts with the current base.
Request. Start Claude OAuth login from the thread and provider error banners.
Audit finding. Main has no reauthentication descriptor or banner action. The diff preserves the failed session instance and custom config directory, but pastes a space-joined executable into a shell and does not exclude CLAUDE_CODE_OAUTH_TOKEN. Custom binary paths with spaces and explicit OAuth-token instances therefore need correction, and mobile has no recovery action. The auth matcher also accepts API-key failure wording that OAuth login cannot fix.
Recommendation. Keep open: work remains. Fix command execution for paths with spaces and explicit OAuth-token instances before client review.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Pr: PR #4151. The new launcher uses an unquoted command string and covers web and desktop only.
Comment: Discussion comment. The current matcher accepts Authentication failed: API key is invalid.
Limits. No interactive OAuth flow was run in this read-only audit. Latest-head required checks are not reported: Test, Check, Mobile Native Static Analysis, Release Smoke.
Request. Disable tools and slash commands in Claude background text generation.
Audit finding. Main still starts Claude metadata generation with --dangerously-skip-permissions and no empty tool set. The diff removes that flag and disables tools and slash commands, but it predates the portable fake-CLI tests now on main. The newer title-isolation proposal overlaps this change and needs a shared scope decision.
Recommendation. Keep open: work remains. Combine this capability restriction with the title-isolation work in the current portable test harness.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Use Tab to complete folder paths in the command palette.
Audit finding. Main has no browse-mode Tab completion and still offers only the existing directory navigation. The PR adds one resolver and tests highlighted rows, exact matches, stale rows, and an empty filter. The fix belongs in the shared add-project and clone-destination command-palette path.
Recommendation. Keep open: work remains. Review and update the small command-palette patch on current main.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. The collected latest head has no results for these required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports merge conflicts with the current base.
Request. Search projects by name or path in the new-thread headline picker.
Audit finding. The draft headline still uses an unfiltered MenuRadioGroup on main. The merged sidebar project-filter work does not change this entry point. The patch adds incremental search, IME handling and a New project action, but it must retain the newer carryComposerContent behavior when rebased.
Recommendation. Keep open: work remains. Rebase the picker change while preserving draft text and attachments when changing projects.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Canonical current-head results are missing for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports conflicts with the target branch, so recorded checks do not validate a rebased integration.
Request. Limit streaming Markdown parsing and rendering to a 50-millisecond cadence.
Audit finding. Main still passes each text update directly to ReactMarkdown. The latest server streaming changes reduce database reads and tool frames, not this client parser work. The diff needs to retain the newer extraRemarkPlugins, image, and citation paths when rebased.
Recommendation. Keep open: work remains. Rebase the coalescing hook onto current ChatMarkdown and verify final-flush and custom-plugin behavior.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Show remote environment labels on command-palette project search and new-thread rows.
Audit finding. The merged project-location change covers the new-thread submenu, but root project search still lacks an environment description. The diff also labels WSL desktop-local groups as remote because it ignores allRemoteMembersAreDesktopLocal, which matches the unresolved review finding. Narrow this to root search and reuse the current local/remote location logic.
Recommendation. Keep open: partial fix. Reduce this PR to environment labels in root project search using the existing location renderer.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Resolve the bare Opus alias within the installed Claude model family.
Audit finding. Main still resolves a missing normalized model to null without trying older Opus models, so the Opus 5 version gate can send older CLIs to an unrelated default. The diff adds a fixed fallback order but still accepts a custom entry whose slug exactly matches one of those canonical names. The author confirmed that narrow case and left it for maintainer review.
Recommendation. Keep open: work remains. Decide whether canonical custom-name collisions can satisfy the Opus family fallback.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Commit: Commit 41a430a88e8d. Opus 5 changed the alias target and added the CLI version gate.
Comment: Discussion comment. The author confirms and explains the remaining canonical custom-slug case.
Limits. Latest-head required checks are not reported: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports merge conflicts against the target branch.
Request. Compress packaged web assets and cache both payloads and immutable browser resources.
Audit finding. The packaged HTTP route on main still serves raw static bytes without asset cache policy. The patch adds bounded compression caching and hashes SPA-fallback HTML, but direct index.html and other unhashed file responses still use only path, mtime and size as their compression key. Replacing such a file with equal size and preserved mtime can therefore reuse old compressed bytes.
Recommendation. Keep open: work remains. Use a content-derived compression key for direct unhashed assets and test a same-size, same-mtime rebuild.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Issue: Issue #4398. The original slow remote-startup report.
Comment: Discussion comment. The fallback race is addressed, but the direct-file metadata-key case remains.
Limits. Canonical current-head results are missing for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports conflicts with the target branch, so recorded checks do not validate a rebased integration.
Request. Show unavailable-environment reasons in project browsing and path mentions instead of an empty result.
Audit finding. Main now blocks Add, Enter, clone submission, and direct folder-picker creation while disconnected through the merged add-project fix. The browse query and composer path menu still do not show the connection failure, and the menu can still say no matching files. The remaining UI explanation and existing-project reopen behavior need to be compared with the current guards.
Recommendation. Keep open: partial fix. Retain the missing browse and mention error states without duplicating the current creation guards.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Merged pr: PR #4834. Landed the add-project and clone guards.
Limits. Current-head required checks were not reported: Check, Mobile Native Static Analysis, Release Smoke, Test. GitHub reports merge conflicts with main on the collected head.
Request. Restart the environment session when a durable subscription loses its transport but the connection still looks usable.
Audit finding. Main still logs the transport failure and drains the individual stream without requesting a new session. Recent reconnect-backoff and server-disconnect fixes do not restart a stream that fails alone, and the later input-construction and status-display PRs address different gaps. The per-session recovery guard is useful, but the new tests use repeated scheduler yields instead of an explicit completion signal. Main now replaces a session when its owned config source terminates. Other durable streams still only log and drain RpcClientError, so the general per-session recovery guard remains needed.
Recommendation. Keep open: work remains. Keep the recovery change, replace yield-count waits with deterministic signals, and review it with the other subscription fixes.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Latest main change. Main now replaces a session when its owned config source terminates. Other durable streams still only log and drain RpcClientError, so the general per-session recovery guard remains needed. Keep the current disposition, add the config-only exception, and preserve session-managed config dispatch during rebase.
Limits. A stream-only transport failure was not reproduced against a running client. Required checks absent from the canonical latest-commit rollup: Check, Test, Mobile Native Static Analysis, Release Smoke.
Request. Restore all default keybindings while preserving project-script shortcuts.
Audit finding. The keybinding service still exposes only upsert and single-rule removal, and the current settings UI has no reset-all request. This patch adds the typed RPC, authorization, atomic rewrite and confirmation, including malformed-file recovery. Its reset path intentionally drops excess script rules at the config limit, which the confirmation should explain before this destructive action ships.
Recommendation. Keep open: work remains. Update the confirmation for script-rule truncation and port the reset control to the current settings layout.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Canonical current-head results are missing for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports conflicts with the target branch, so recorded checks do not validate a rebased integration.
Request. Remove deleted custom models from the composer before provider snapshots refresh.
Audit finding. Both model-option builders on main still start with all snapshot models, including removed custom entries. The settings-list cleanup does not change that composer path. This patch fixes immediate web membership, while the separate server cache patch remains needed to stop resurrection after restart and on other clients.
Recommendation. Keep open: work remains. Review this patch together with the server cache fix and preserve current OpenCode unavailable-selection handling.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Pr: PR #5230. Separate open fix for server cache resurrection.
Limits. Canonical current-head results are missing for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports conflicts with the target branch, so recorded checks do not validate a rebased integration.
Request. Desktop restarts should reuse a local authorization session instead of creating duplicate authorized-client entries.
Audit finding. Main still keeps the desktop bearer token only in memory, so restarts create new sessions. The PR adds encrypted reuse and expiry handling, but its token store treats invalid base64 or unrecoverable ciphertext like a transient read failure and never clears it. That leaves authentication permanently broken after key loss, as the unresolved review finding and current diff show.
Recommendation. Keep open: work remains. Distinguish unrecoverable saved ciphertext from transient storage failures and test session replacement.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Preserve POSIX case when deciding whether a Markdown file link is inside the workspace.
Audit finding. Both current containment and display helpers lowercase POSIX paths before comparison. Later file-link routing and directory-label fixes retain that behavior. The PR shares one case-aware containment result between click routing and display, so the wrong-case sibling defect remains.
Recommendation. Keep open: work remains. Review the shared containment helper with POSIX siblings, Windows drives, and UNC roots.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Keep the iPad Threads toolbar within the safe area after window resizing.
Audit finding. SDK57 moved the patch to react-native-screens 4.26.2, but the mail toolbar still uses the required resolvedWidth constraint and minimum width. The split-view Home route also still does not clear inherited bottom toolbar items. The native fix remains relevant and must be ported without dropping newer toolbar reuse and header fixes.
Recommendation. Keep open: work remains. Port the width and split-view reset changes to the current screens patch.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Persist the branch shown by the draft picker and hide default-branch labels consistently.
Audit finding. ChatView still initializes thread metadata with activeThreadBranch instead of resolving the displayed checkout branch. Main also lacks resolveSidebarBranchLabel and the proposed default-branch check. Later dedicated-worktree branch synchronization does not address untouched local drafts or this sidebar rule.
Recommendation. Keep open: work remains. Rebase the two metadata and label changes onto the current sidebar and confirm the default-branch display rule.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Reject corrupted CLI Connect state and PKCE fragments before starting browser authorization.
Audit finding. Main validates that state and challenge are nonempty but does not check their fixed base64url length or alphabet. The merged sign-in redirect fixes preserve OAuth parameters, but still forward malformed values into the same authorization flow. The PR adds early validation and tests valid CLI-generated requests as well as corrupted copies.
Recommendation. Keep open: work remains. Rebase the fragment validation onto the current sign-in redirect flow and review the parser boundary.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Merged pr: PR #5133. Fixes in-modal redirect, not malformed fragment validation.
Limits. Current-head required checks were not reported: Check, Mobile Native Static Analysis, Release Smoke, Test. GitHub reports merge conflicts with main on the collected head.
Request. Show one loading indicator when the mobile file browser first opens.
Audit finding. Main still sets RefreshControl.refreshing and the empty-list ActivityIndicator from isPending. The complete diff adds isInitialLoad at both mobile callers, so the refresh indicator starts only after data has loaded. Recent mobile theme changes did not change this loading logic. An unresolved review asks whether refreshing an empty workspace can still trigger the iOS indicator defect.
Recommendation. Keep open: work remains. Verify initial load and refresh on both empty and populated iOS workspaces.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. The linked interaction videos were not played and no simulator was used. The collected latest head has no results for these required checks: Test, Check, Mobile Native Static Analysis, Release Smoke.
Request. Show Done for mobile completions that arrived after the last visit.
Audit finding. Both current mobile status models still return an idle timestamp after completion and have no per-thread read marker. The PR adds focus-aware, device-local visit timestamps and preserves active, error, and approval priority. PR #5962 introduces another read-marker system for Mark unread, so their storage and completion rules must be combined before either lands.
Recommendation. Keep open: work remains. Choose one read-marker implementation shared with PR #5962.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Download arbitrary workspace files from the Files panel through signed environment URLs.
Audit finding. Main still offers only Copy mention and Add to chat in the file-tree context menu. Generic chat attachment downloads do not expose arbitrary workspace files, and the workspace-file contract has no download disposition. This remote-ready Files action is therefore not covered by the new attachment features.
Recommendation. Keep open: work remains. Rebase the Files download action onto the current asset response headers and verify a binary file from a remote environment.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Pass the effective account name to loginctl so WSL service installation does not require a logind session.
Audit finding. The current systemd manager still calls loginctl enable-linger without a username after the service-manager refactor. This PR addresses that headless and WSL failure, but eagerly calls os.userInfo while constructing the whole service. The unresolved review correctly notes that an account without a passwd entry can then break status and uninstall, which do not need the username.
Recommendation. Keep open: work remains. Resolve the username only during systemd installation and move the fix into the current manager.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Generated Linux systemd services cannot find provider executables installed in user-local binary directories.
Audit finding. The current Linux systemd unit still sets no PATH, and the later macOS launchd PATH fix does not cover it. This PR adds user-local directories by replacing PATH with a fixed list, which would remove custom manager entries. The unresolved review identifies that regression, but its suggested literal ${PATH} is not a valid systemd Environment expansion.
Recommendation. Keep open: work remains. Add user-local directories while preserving the installation PATH in the generated systemd unit.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Report a missing or invalid workspace directory before provider startup fails as a binary spawn error.
Audit finding. Main still sends the effective cwd directly to the provider adapter without checking whether it is a directory. The PR now uses stat, including the reviewed regular-file case, and exposes its typed message through the common command reactor for all providers. It does not relocate a moved project or repair missing worktrees.
Recommendation. Keep open: work remains. Review the common preflight and clean up the new temporary fixture directory after the test suite.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Show project names in slim settled and snoozed sidebar rows.
Audit finding. Main still renders the favicon directly beside the title in slim rows, with no project label. The patch adds the label, but its content-sized flex basis can take space from the flex-1 thread title, and it uses a different secondary-text token from the rest of the sidebar. Both open review findings match the diff, so the feature needs a small layout correction.
Recommendation. Keep open: work remains. Limit the project-label width and use the sidebar secondary-label token before checking narrow rows.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Treat externally signal-killed Claude subprocesses as interrupted turns.
Audit finding. Current interruption matching still omits the SDK SIGTERM and SIGINT exit messages. The merged stop fixes handle intentional stop cleanup and aborted SDK results, which do not cover an OS signal arriving before stopSession. The proposed classification is still needed, but its test relies on repeated scheduler yields instead of a completion receipt.
Recommendation. Keep open: work remains. Replace scheduler-yield waits with a deterministic completion signal and test both supported signals.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. macOS Tailscale discovery and Serve commands need to find the application executable when no CLI is on PATH.
Audit finding. Both current Tailscale command paths still spawn only the bare tailscale command. Merged PR #7116 catches spawn defects in desktop endpoint discovery, but it does not find the application executable or enable Serve. The PR supplies a macOS-only missing-executable fallback for both paths, with focused status-path tests.
Recommendation. Keep open: work remains. Rebase the fallback and add a Serve-command fallback test before maintainer review.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. A tall terminal drawer must leave enough room for the composer and its actionable banners.
Audit finding. Main still gives the drawer a fixed non-shrinking height while the chat column can shrink to zero, and it has no composer-height reserve. The PR measures the composer, permits drawer shrinkage, and preserves the stored height during constrained resizing. Recent composer-banner changes require the measuring targets and banner reserve to be updated before reuse.
Recommendation. Keep open: work remains. Rebase the height reserve on current composer banners and verify a tall draft plus drawer at a short window height.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Open the configured preview after a project action successfully starts.
Audit finding. The persistence fix saves previewUrl and autoOpenPreview, but main runProjectScript still never reads them. The complete PR restores auto-open after a successful write, but passes the raw URL to a helper that does not resolve environment ports. The review correctly identifies that this differs from manual opening of the same configured URL on remote environments.
Recommendation. Keep open: partial fix. Resolve configured action URLs through the environment-port path before restoring auto-open.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Commit: Commit 63b6b44627d6. The earlier merged persistence fix does not restore the run path.
Comment: Discussion comment. The raw-URL mismatch is confirmed against openUrlInPreview.
Limits. No interaction verification was performed. The collected latest head has no results for these required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports merge conflicts with the current base.
Request. Reduce resource-monitor process scans and share idle UI timers.
Audit finding. The five-second idle sample interval is now in main through the idle CPU fix. Native sampling still refreshes memory, disk, and command details for all processes, and the web hooks still create a timer per consumer. Those remaining changes are not replaced by the interval fix.
Recommendation. Keep open: partial fix. Drop the landed sample-interval hunk and review the tracked-process refresh and shared-timer changes.
Confidence high. Release: In nightly source. PR readiness: Needs small changes.
Request. Refresh Add Project folder listings when users reopen or navigate the picker.
Audit finding. Stable PR 8236 already adds a safer refresh option to executeAtomQuery, refreshing only settled values after mount. The web and mobile query runners and folder-picker callers still do not forward or enable it, so new external folders can remain hidden inside the 30-second cache window. Keep those client changes without replacing the newer shared helper.
Recommendation. Keep open: partial fix. Rebase onto the existing query refresh option and retain only the web and mobile folder-picker wiring.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Make the host-runtime lint rule catch platform and architecture reads through node:process imports.
Audit finding. Main still tracks node:os imports only, so the three NodeProcess reads in mobile-showcase bypass the rule. The final diff adds namespace/default process tracking and fixes both reviewed shadowing defects, including destructured parameters. Its hand-written scope walk does not cover every lexical form, and named process imports remain explicitly out of scope.
Recommendation. Keep open: work remains. Review the scope walker with loop-local and nested var bindings before enabling the broader error rule.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Loop-scope and nested function-scope var shadowing were not tested by the added cases. Required checks absent from the canonical latest-commit rollup: Check, Test, Mobile Native Static Analysis, Release Smoke.
Request. Show Grok context fill and window size in the existing composer meter.
Audit finding. Main has no TokenUsageUpdated parsing or Grok token-usage event emission on these ACP paths. The Grok usage merge reads transcript cost and token records for the Usage page, which does not supply the live composer context meter. This diff still adds distinct streaming metadata, prompt-result, and model-window mappings.
Recommendation. Keep open: work remains. Rebase and verify the context meter mappings against the current Grok ACP events.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. The PR has no before and after meter images. Required checks have no results on this head: Test, Check, Mobile Native Static Analysis, Release Smoke.
Request. Detach long-running Effect roots from inherited spans and suppress idle trace noise.
Audit finding. Main still forks parked roots with ambient context and traces both secret reads and idle PortDiscovery ticks. The recent provider and snapshot fixes do not change those paths. The helper still accepts effects that explicitly require ParentSpan even though it removes that service, and the planned long idle-memory run remains unreported.
Recommendation. Keep open: work remains. Constrain the helper's ParentSpan contract and run the planned idle-memory comparison.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Comment: Discussion comment. Changing the return type to Exclude does not prevent passing an effect that directly requires the removed service.
Limits. The proposed multi-hour idle-memory verification is not present in the PR evidence yet. Required CI has no result on the reviewed head: Test, Check, Mobile Native Static Analysis, Release Smoke.
Request. Expose Grok compact in the slash menu and report completed compaction.
Audit finding. Main does not advertise Grok compact or translate auto_compact_completed into a compacted activity. The Grok reliability merge added skills, plans, and transcript usage but did not add these paths. The proposed command and xAI notification handler therefore remain new behavior.
Recommendation. Keep open: work remains. Review the compact notification handler with current Grok session ownership and client feedback.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. The PR has no before and after slash-menu images. Required checks have no results on this head: Test, Check, Mobile Native Static Analysis, Release Smoke.
Request. Stop the active provider turn after the user cancels an approval request.
Audit finding. Main still forwards approval responses without a follow-up interrupt. The patch distinguishes cancel from decline, interrupts only after a successful response and records interrupt failures, matching the resolved review finding. Required checks passed on the recorded branch, but the new tests still poll and the patch predates current approval handling.
Recommendation. Keep open: work remains. Port the cancellation flow to current main and replace polling with reactor receipts and drains.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Keep URL text when a clipboard paste also contains an image attachment.
Audit finding. The current attachment-paste helper always claims image pastes before considering plain text, and ChatComposer prevents default without reinserting that text. The newer general-file attachment work therefore does not fix this link-plus-image case. The patch limits reinsertion to HTTP URLs, but needs porting to the current attachment path and behavior coverage for image-only copies that also carry a URL.
Recommendation. Keep open: work remains. Port the URL-preserving paste change and test link previews, image copies and selection insertion.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Canonical current-head results are missing for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports conflicts with the target branch, so recorded checks do not validate a rebased integration.
Request. Measure provider session idleness from turn and task activity.
Audit finding. Main still only publishes runtime activity events without touching the persisted lastSeenAt value. The merged reaper fix now protects live background work, covering that part of the reported symptom, but a completed provider-started turn can still leave an old idle timestamp. The proposed targeted touch remains separate and its tests need the current receipt-based wait pattern.
Recommendation. Keep open: partial fix. Retain the timestamp fix and replace clock-delay waits with event receipts.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Parse a full pairing URL pasted into the mobile host field without losing its token.
Audit finding. Main recognizes hosted links in parsePairingUrl, but the host input still goes straight to buildPairingUrl with a separate code value. The merged bare-IP fix only selects HTTP for IP literals and does not split pasted fields or normalize schemeless pairing links. The PR remains useful but must use the current connectAndClose submit path.
Recommendation. Keep open: work remains. Port the paste normalization into the current mobile submit handler.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Merged pr: PR #4990. Only changes the default protocol for IP addresses.
Limits. Current unresolved review threads were checked, but the long prior discussion and resolved review history were not fully read. Current-head required checks were not reported: Check, Mobile Native Static Analysis, Release Smoke, Test. GitHub reports merge conflicts with main on the collected head.
Request. Give Git command failures useful categories without returning secret-bearing stderr.
Audit finding. Main GitCommandError still has only a generic detail and output lengths on the core execution path. The current PR diff adds authentication and not-found categories with fixed messages, not the stderrTail described in its opening body. The capability remains absent, but the body must match the reviewed implementation.
Recommendation. Keep open: work remains. Update the PR description to the classified-error design and review the two core execution paths.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Keep provider-qualified pricing rates separate and make aliases deterministic.
Audit finding. Merged #8806 already preserves qualified rates and avoids ambiguous unequal-rate aliases. Main still overwrites normalized spelling collisions in document order, stores aliases in the same map and reports rates.size as the model count. Those remaining parts of this PR are not covered by the merged pricing fix.
Recommendation. Keep open: partial fix. Reduce the patch to normalized-key collision handling and accurate model counts, preserving the merged alias rules.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Expose environment-owned settings where the selected client can save them.
Audit finding. Text-generation model controls still live in the primary-scoped General panel, and their resolver still trusts stale provider enabled state. The patch moves that setting to per-environment Providers and hides an unusable hosted Source Control section. It explicitly leaves other primary-only General settings unchanged, so it does not resolve the broader environment-settings request.
Recommendation. Keep open: work remains. Port per-environment text-generation settings through the current settings availability and search code.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Canonical current-head results are missing for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports conflicts with the target branch, so recorded checks do not validate a rebased integration.
Request. Avoid sending private chat-link hosts to the third-party favicon service.
Audit finding. Web and mobile chat still build Google favicon URLs without a private-host guard. The browser-tab helper is already guarded by isPublicFaviconHost, so the body overstates the current diff by claiming a third call-site change. The proposed predicate still misses the reserved home.arpa apex, and the mobile guard must follow the current host rather than initial component state.
Recommendation. Keep open: partial fix. Reuse the current shared host rules or cover home.arpa, and evaluate the mobile guard for each current host.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Remove editor line and column suffixes before a File Manager launch without changing real colon-number filenames.
Audit finding. The current launcher still sends input.cwd unchanged to the normal File Manager action. The newer reveal action does not normalize the primary editor-launch path either. The PR checks the exact filename first and strips a position only when its base file exists, so this remains an unfixed path-handling case.
Recommendation. Keep open: work remains. Apply the position normalization to the current normal File Manager launch while preserving the separate reveal behavior.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Grow the native composer with a long prompt before enabling internal scroll.
Audit finding. Main still gives the expanded editor a minimum and maximum height without consuming native content-size events. Both iOS and Android wrappers omit onComposerContentSizeChange, so SDK57 did not add the proposed JS sizing behavior. The patch needs to use the current 72-point minimum and current composer structure.
Recommendation. Keep open: work remains. Refresh the sizing patch and verify grow, cap, and shrink on iOS and Android.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Restore the composer DOM element after preview keyboard automation temporarily takes focus.
Audit finding. Current performAutomationPress restores only the previously focused WebContents, not its active DOM element. The PR adds per-call element storage and restores it during cleanup, which addresses the specific cause in issue 5792. Recent preview refactoring moved this code into a shared helper but did not add DOM-focus restoration.
Recommendation. Keep open: work remains. Port the small change to performAutomationPress and verify same-thread and background-thread composer focus.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. No desktop focus interaction was reproduced. Required checks absent from the canonical latest-commit rollup: Check, Test, Mobile Native Static Analysis, Release Smoke.
Request. Hide inactive agent rows with a reversible per-thread cutoff.
Audit finding. Main still renders every agent row and has no Clear inactive or Show all control. The patch preserves working workflows and restores rows when later results arrive, but its out-of-order task.started path still assigns an older updatedAt directly. That can hide a restored result again under the cutoff, so the remaining review finding is valid.
Recommendation. Keep open: work remains. Make all relevant agent timestamp updates monotonic and test a late start after a cleared result reappears.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Comment: Discussion comment. Verified remaining late-start timestamp regression in the final patch.
Limits. Canonical current-head results are missing for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports conflicts with the target branch, so recorded checks do not validate a rebased integration.
Request. Use OpenCode's configured default agent instead of always selecting build.
Audit finding. Main still selects build in inferDefaultAgent and loadOpenCodeInventory never reads config.default_agent. The PR adds the missing config read and filters the configured value against selectable agents. Its old fixtures and inventory tuple need to retain the skills field added since its base.
Recommendation. Keep open: work remains. Rebase the default-agent change while retaining HTTP skill inventory and add current provider tests.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Canonical latest-commit checks do not include Check, Test, Mobile Native Static Analysis, Release Smoke. GitHub reports a merge conflict with current main.
Request. Show an empty-inbox message and New thread action above settled and snoozed history.
Audit finding. Main only renders its empty state when active, pinned, snoozed and settled groups are all empty. The patch adds the inbox-only row on web, phone and tablet and protects native loading and search states. The author confirms the web path still lacks a loading guard, so it can claim all work is handled before thread data arrives.
Recommendation. Keep open: work remains. Add the web hydration guard before reviewing the cross-client empty-inbox change.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Comment: Discussion comment. Author identifies the remaining web loading-state limit.
Limits. Canonical current-head results are missing for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports conflicts with the target branch, so recorded checks do not validate a rebased integration.
Request. Reserve Android bottom safe-area space for menus, lists, and terminal controls.
Audit finding. Current file-tree and review lists still use eight-point bottom padding, and the terminal keyboard button still sits at bottom 16. Earlier Android safe-area fixes did not cover these sites. The patch adds the missing insets, but its new global lint rule treats any bottomInset identifier as proof and cannot establish whether a nested list actually owns a screen edge.
Recommendation. Keep open: work remains. Review the targeted inset fixes separately from the proposed global lint rule.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Disambiguate default WSL environment labels from the Windows host and other WSL distributions.
Audit finding. Main still returns an unmarked hostname when friendly-name lookup fails. The PR adds the requested kernel-based WSL detection and fallback-only suffix, with two tests and an independent WSL verification report. The unresolved no-tests comment is stale, and the separate manual-rename proposal does not replace automatic WSL naming.
Recommendation. Keep open: work remains. Review the fallback-only WSL label change and refresh the tests and formatting.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Issue: Issue #4820. Original request explicitly limits the suffix to the plain-hostname fallback.
Limits. Existing saved labels are not migrated by the patch. Required checks absent from the canonical latest-commit rollup: Check, Test, Mobile Native Static Analysis, Release Smoke.
Request. Add an explicit Hide Terminal control without closing terminal sessions.
Audit finding. Main already toggles terminal visibility from the chat header and keybinding, but neither terminal toolbar has the proposed hide button. Later close-confirmation work still closes a session and does not replace this control. The body cites item 123, which GitHub identifies as an unrelated closed mobile-client PR, so that fixes reference is wrong.
Recommendation. Keep open: work remains. Correct the linked issue and review the extra hide control against the existing header toggle.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Validate popup URLs before loading them in the desktop preview.
Audit finding. Main now supports OAuth popup windows, but its same-tab fallback still calls loadURL(details.url) without the preview URL allowlist. The complete patch closes that path on its older base. It must be adapted to preserve the new OAuth-popup branch instead of restoring the old handler wholesale.
Recommendation. Keep open: work remains. Apply URL validation to the current popup handler and test both native OAuth popups and same-tab navigation.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Commit: Commit 0e2905eb783f. Merged OAuth popup support changes the handler this PR must preserve.
Limits. The collected latest head has no results for these required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports merge conflicts with the current base.
Request. Allow project removal when its only threads are archived.
Audit finding. Project settings on main counts only live thread shells before deciding whether to force deletion. The patch loads archived snapshots and scopes failures to the environments being removed, while the legacy sidebar already has a force-delete route. The body says deletion always forces, but the actual patch still sets force only when its combined count is nonzero.
Recommendation. Keep open: work remains. Update the PR description and rebase archived-thread counting while preserving current attachment cleanup.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Show the mobile bottom arrow only after leaving the end of the thread.
Audit finding. Main still bases the arrow on the live-follow latch alone, which is disabled when a drag starts at the bottom. The at-end state change remains useful, but the button moved into FloatingWorkingControl in merged PR #8793, so the old fixed -top-24 placement no longer matches the client. The disclosure-follow work in open PR #9013 changes a related latch path, not this visibility rule.
Recommendation. Keep open: work remains. Keep the at-end visibility fix and adapt it to FloatingWorkingControl.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Give the annotation picker keyboard focus immediately so Escape works without a page click.
Audit finding. Main still calls wc.focus only when wc.isFocused returns false. The PR removes that unreliable guard and covers the case where Electron reports focus but the renderer owns the keyboard. Keyboard-isolation work overlaps focus handling but is still unmerged.
Recommendation. Keep open: work remains. Verify the unconditional picker focus handoff alongside the keyboard-isolation changes.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Pr: PR #8494. Open keyboard-isolation work needs compatible picker focus behavior.
Limits. Native focus behavior was not reproduced. The collected latest head has no results for these required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports merge conflicts with the current base.
Request. Revoking a client or pairing link should show an error or remove its stale access row.
Audit finding. Main still ignores the revoked flag returned by individual revoke calls and relies on the access stream to remove rows. The PR adds result checks and local row hiding, but raises the two new inactive-result failures as plain Error objects unlike the surrounding tagged errors. The two unresolved convention findings match the current diff.
Recommendation. Keep open: work remains. Use structured inactive-client and inactive-link errors, then verify row removal against the access snapshot.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Open file references that include a range of lines and highlight the range.
Audit finding. Main splitPathAndPosition still supports only line and optional column, and file-panel state has no end line. The inspected production changes add both range syntaxes, carry the end through the panel, and reduce editor launches to the first line. Later file-link routing fixes do not provide range parsing.
Recommendation. Keep open: work remains. Review both range syntaxes through chat preview and external editors while preserving single-line references.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Open a mentioned file from its composer chip before sending the message.
Audit finding. ComposerMentionDecorator on main still renders an inert span. Message-file navigation and new generic attachment handling do not add a composer-chip handler. The PR provides click and keyboard actions, but its separate path resolver needs review alongside the existing shared path rules.
Recommendation. Keep open: work remains. Review chip activation and workspace containment in the current composer, including hosts without a file panel.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Stop task-checkbox edits from replacing a large Markdown file with its truncated preview.
Audit finding. Main selects rendered Markdown before the truncated read-only branch. That renderer still sends its preview contents to the whole-file save coordinator when a checkbox changes. The PR disables that handler for truncated reads and adds a second mutation guard, so the data-loss path is not fixed on main.
Recommendation. Keep open: work remains. Prioritize review of the truncated-Markdown write guard and verify that the complete file remains unchanged.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Keep the macOS renderer and its preview host alive when the last window closes.
Audit finding. The current window close handler only persists bounds, so it still lets a normal close destroy the renderer. The complete PR hides the window unless DesktopState.quitting is set and tests both branches. Recent hidden-preview power changes do not preserve a destroyed renderer. The open review also identifies hide during native fullscreen as a macOS case not handled by the patch.
Recommendation. Keep open: work remains. Handle native fullscreen before hide, then verify close, reopen, Quit, and updater shutdown.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Pr: PR #8981. The open background-automation branch overlaps this lifecycle.
Limits. No packaged macOS lifecycle test was run. The collected latest head has no results for these required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports merge conflicts with the current base.
Request. Include installed Claude plugin skills in the skill picker.
Audit finding. Current discovery reads user skills and the two workspace skill directories but never installed_plugins.json. The merged workspace-skill fix does not discover plugin manifests or namespaced plugin skills. The diff covers plugin layouts and workspace filters, but its new hand-built JSON fixtures insert native paths without JSON escaping and need correction for Windows.
Recommendation. Keep open: work remains. Use correctly escaped plugin-manifest fixtures and rerun discovery tests on Windows.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Register and populate iOS Home Screen Agent Activity widgets.
Audit finding. Main still registers only createLiveActivity and has no createWidget snapshot path or widget-family layout. SDK57 changed the dependency version but did not add the missing widget registration in app code. The diff adds independent widget refresh and sign-out clearing with stale-response guards, but needs native SDK57 rendering proof because JS serialization tests do not establish widget rendering.
Recommendation. Keep open: work remains. Verify the widget on a current SDK57 native build with Live Activities disabled and after sign-out.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Check: GitHub check. Current head 3d9dc9bf: Test=SUCCESS, Check=SUCCESS, Mobile Native Static Analysis=SKIPPED, Release Smoke=SUCCESS.
Limits. The body contains screenshots but also says a native after-state was not captured; exact-head SDK57 evidence is unclear. Mobile Native Static Analysis was skipped on the current head; no passing native static result is claimed.
Request. Make explicit file refresh rebuild the server workspace index before reloading the list.
Audit finding. Main still re-runs listEntries through the query refresh function without a projects.refreshEntries command. The recent open-file and agent-mutation refresh changes revalidate client reads but do not expose WorkspaceEntries.refresh to an explicit user action. The full diff fixes this distinct server-index path for web and mobile.
Recommendation. Keep open: work remains. Review the explicit rescan RPC and verify external create, rename, and delete through both client refresh controls.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Keep chat at the live edge when a sidebar resize changes row heights and a completed turn leaves its anchor armed.
Audit finding. Main releases the send anchor when the user returns to the live edge, through merged PR 6519. It still does not release a collapsed anchor when the turn settles, and MessagesTimeline has no full-viewport end-maintenance threshold. The resize and automatic-release parts of this patch remain missing.
Recommendation. Keep open: partial fix. Rebase the resize and turn-completion fix on the merged manual-return anchor handling.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Keep cumulative child-agent token usage out of the parent context meter.
Audit finding. Main still calls normalizeClaudeTaskProgressTokenUsage for task progress and completion, taking the maximum of child totals and the parent snapshot. The merged change that stops getContextUsage after a turn does not remove these updates. The diff fixes these child totals and total-only results, but overlaps the broader parent-context proposals.
Recommendation. Keep open: work remains. Choose one parent-context fix with coverage for task totals and total-only results.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Commit: Commit c131f289268d. The merged change removes only the post-turn SDK context query.
Limits. Latest-head required checks are not reported: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports merge conflicts against the target branch.
Request. Prevent managed SSH tunnels from inheriting user-configured port and socket forwards.
Audit finding. Main disables SSH connection sharing but still inherits user forwarding directives. The PR filters them into a generated config, but preserves canonicalizehostname and includes normal configs for names other than the original alias, allowing canonical-name reparsing to restore inherited forwards. The unresolved review matches that generated-config path, so this is not ready as written.
Recommendation. Keep open: work remains. Handle canonical-name reparsing without restoring user forwarding directives.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. The desktop update-downloaded toast should offer a restart action.
Audit finding. Main still provides only a release-notes link in that toast. The PR calls installUpdate directly from a new Restart action, but the existing sidebar install path first shows the platform-specific update confirmation. The toast needs the same confirmation and duplicate-action protection before it becomes a second install entry point.
Recommendation. Keep open: work remains. Route the toast action through the existing confirmed install flow and test cancellation.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Make Android source-file Word break wrap long lines instead of clipping them.
Audit finding. Main always chooses the native review view when it exists, without considering codeWordBreak. Its JavaScript line renderer also still applies flex-1 directly to Text. The PR changes both and adds refresh and scroll retry behavior, so theme changes alone have not fixed this path.
Recommendation. Keep open: work remains. Verify Android wrapped source text, pull-to-refresh, and jump-to-line after applying the current renderer change.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Use linear framing for both Codex protocol input and stderr.
Audit finding. Merged PR 8605 already fixes the protocol input accumulator with fragmented buffering and framing tests. CodexSessionRuntime still concatenates and splits the entire pending stderr remainder on each chunk. This PR has material stderr scope beyond the landed input fix, so closing the whole PR as fixed would lose that work.
Recommendation. Keep open: partial fix. Reduce this PR to the remaining stderr-framing change and its tests.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Report a missing workspace directory before starting a provider process.
Audit finding. Main recreates a missing thread worktree when its branch is available through the merged worktree recovery fix. It still passes the resolved workspace path to startSession without this directory check, so a moved project root or failed recovery can produce a provider error. The missing-workspace diagnostic remains useful, but its worktree guidance must match current recovery.
Recommendation. Keep open: partial fix. Rebase the guard and keep the moved-project-root case as the focused test.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Normalize typed mobile and web pairing codes without changing tokens received in URLs.
Audit finding. Current code fields still store the typed value unchanged, while issued codes use a 12-character uppercase alphabet. This diff keeps the normalizer at typing handlers and leaves scanned and URL-derived tokens unchanged. The body explicitly excludes the reported timeout, so its Fixes reference would close more of the linked issue than it handles.
Recommendation. Keep open: work remains. Replace the automatic issue-closing reference with a partial-fix reference before review.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Make file breadcrumbs reveal the selected project, folder, or file in the file tree.
Audit finding. Main now wraps breadcrumb labels in styled tooltips, but the labels remain spans without click actions. The PR adds one-shot reveal requests, flattened-directory handling, and root scrolling. The reveal behavior is not present in the later expand-all and tooltip changes.
Recommendation. Keep open: work remains. Rebase the breadcrumb buttons onto the current tooltip markup and review flattened-directory reveal behavior.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Let mobile users refresh providers from each connected environment row.
Audit finding. The merged OpenCode work now adds a provider-catalog refresh in mobile thread settings using the same environment-scoped RPC. Connection rows, environment settings, and onboarding still have no refresh action. This PR therefore has remaining entry-point scope, though it should reuse current refresh handling and replace its static-render tests.
Recommendation. Keep open: partial fix. Keep the missing environment-row entry points and reuse the current catalog refresh helper.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Show a work-log warning when Claude pauses for a usage limit and label Codex quota failures.
Audit finding. Main forwards Claude rate_limit_event only as account telemetry, so it still gives the thread no pause explanation. The diff adds active-turn and overage guards, deduplicates limit windows, and changes Codex usageLimitExceeded to a warning while retaining the separate failed-turn event. It leaves parked turns running by design and does not solve general stuck turns.
Recommendation. Keep open: work remains. Review the quota-warning behavior on both providers and replace scheduler-yield waits in the Claude tests.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Preserve the last discovered Claude commands when a capability probe fails.
Audit finding. Main has no last-good command cache and still replaces the command list when a probe fails. Since the merged compaction feature always inserts compact, a failed probe now returns a nonempty list. The proposed empty-list condition therefore misses the current failure path and would discard previously discovered commands.
Recommendation. Keep open: work remains. Retain commands using explicit probe success rather than an empty slash-command list.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Avoid scanning the OS-home projects directory when default Claude transcripts are absent.
Audit finding. Current UsageService still falls back from the missing default Claude transcript directory to the OS-home projects directory. The recent incremental transcript reader reduces repeat-read cost but leaves this root selection unchanged. The diff limits that fallback to an explicit home setting, but tests only the path helper rather than the scan root used by UsageService.
Recommendation. Keep open: work remains. Add a UsageService test that records scan roots when the default Claude transcript directory is absent.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Prevent an archived or deleted thread ID from being reused for a new project draft.
Audit finding. The background-draft merge now rejects already-promoted drafts on the initial reuse path. Main still does not retire the project draft mapping on archive or consult cached archived shells, and the asynchronous race path checks only live shells. The archived-ID failure therefore remains outside the merged guard.
Recommendation. Keep open: partial fix. Rebase the archive retirement and all reuse paths onto the existing promoted-draft guard.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Persist late Claude tool results without reopening the completed turn.
Audit finding. Main clears every in-flight tool during completeTurn and later ignores results whose tool ID is no longer present. This diff retains up to 64 finalized tools, persists terminal updates, and merges late rows on web and mobile by toolCallId. It is not covered by the existing payload-size fixes and changes lifecycle persistence across all clients, so it needs an integrated ordering review.
Recommendation. Keep open: work remains. Review late-result persistence and row merging with multiple tools and an intervening assistant reply.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Commit: Commit 3da315e7b5c4. The merged MCP payload fix reduces transport size rather than retaining late tool results.
Limits. Latest-head required checks are not reported: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports merge conflicts against the target branch.
Request. Label nested Claude tasks only with a model named by the spawn or serving response.
Audit finding. Main now preserves an early authoritative assistant snapshot when it arrives before task_started. It still falls back to the parent session model when no launch model is present, and it does not use message_start to refine the nested task. The remaining change must retain the newer snapshot-race buffer.
Recommendation. Keep open: partial fix. Rebase the explicit-model logic while preserving pendingTaskModels handling.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Ctrl+A and Ctrl+E should produce terminal control keys under non-Latin keyboard layouts.
Audit finding. Main still uses layout-derived Ctrl key text, while the PR adds a physical A/E override at the Ghostty encoder boundary. That override also changes valid Latin-layout chords, such as AZERTY Ctrl+Q at physical KeyA, into Ctrl+A. The unresolved review is correct, and the later Shift-modifier fix does not address this control-key mapping.
Recommendation. Keep open: work remains. Restrict the physical A/E fallback to non-Latin keys and test AZERTY or Colemak alongside Cyrillic.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Keep the last usable provider snapshot when a status probe times out.
Audit finding. Main provider probes still turn deadline misses into error snapshots and the managed provider publishes them as fresh status. Configurable probe budgets and opt-in defaults reduce probes but do not preserve known auth and models after a timeout. The PR adds a typed timeout and invalidates old enrichment before carrying forward a usable snapshot.
Recommendation. Keep open: work remains. Review timeout recovery with current provider enrichment and cache behavior.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Build OpenCode Go model slugs from catalog keys and send bare runtime model IDs.
Audit finding. Main still builds each slug from provider.id plus model.id and only splits that slug at the first slash. HTTP inventory fixed catalog discovery but does not normalize an already-namespaced model.id or persisted duplicated prefix. The PR handles catalog keys and normalizes only hosted OpenCode providers, preserving legitimate slashes for other providers.
Recommendation. Keep open: work remains. Rebase the catalog-key and runtime normalization changes onto HTTP inventory and verify send, resume, and text generation.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Canonical latest-commit checks do not include Check, Test, Mobile Native Static Analysis, Release Smoke. GitHub reports a merge conflict with current main. The branch-head rollup separately showed cancelled fork checks, not successful upstream verification.
Request. Print the desktop version and exit before Electron initializes a display.
Audit finding. Current main.ts starts DesktopApp.program without a version fast path, and DesktopPreReadyPlatform has no version predicate. The PR correctly places synchronous output and process exit before runtime-layer acquisition, leaving normal GUI startup unchanged. Its tests check token matching and output, but the author explicitly did not run a packaged Linux binary without DISPLAY.
Recommendation. Keep open: work remains. Verify the packaged Linux binary returns a version and exit 0 without X11 or Wayland.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Render proposed plans as copyable mobile thread entries.
Audit finding. Main still does not add thread.proposedPlans to buildThreadFeed or render a plan card. The patch keeps plans outside settled-turn folds and adds a small web title-stripping correction. It needs the current semantic theme tokens and current Markdown image props rather than the older hard-coded neutral card styles.
Recommendation. Keep open: work remains. Refresh the plan entry and card for the current mobile renderer and theme system.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Terminal context menus should act on the path or URL under the pointer and keep its highlight visible.
Audit finding. Main still drops the detected link from onContextMenu and offers only selection actions plus Paste. Merged PR #7488 improves hover and click feedback but does not add link-specific menu actions. The PR handles drawer and panel insertion, copy, and preview paths, but now needs to account for the file-manager reveal feature that has landed.
Recommendation. Keep open: work remains. Rebase the link-menu change and include the current file-manager reveal action for paths.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Filter the archive by a logical project, including projects with only archived threads.
Audit finding. The current archive route has no project scope and mounts the unfiltered archive panel. The patch adds a persistent URL key and archive-derived picker, with All as the way to clear it. Its latest version intentionally hides scoped results while any archive source refreshes or fails, even if earlier snapshots remain available.
Recommendation. Keep open: work remains. Review the intentional empty filtered state during refreshes and failures before approving the archive project scope.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. No current-head result for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke. An older related reference could not be retrieved from GitHub. It is excluded from verified dependencies and fix evidence.
Request. Recover the controlled mobile editor when a thread switch or send replaces its document.
Audit finding. Main still matches controlled values against native revision snapshots without a document identity. The reported empty-draft rejection can therefore persist after a replacement even though ordinary stale-typing protection works. The PR covers both native wrappers and new-task drafts, but its sendEpoch changes before awaiting send completion, so the clearing sequence needs a test at the actual async send boundary.
Recommendation. Keep open: work remains. Test document replacement through the current async send path and update the editor keying.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Reopen an existing web draft when returning to the home route.
Audit finding. Main still creates a draft in the most recently active project on index entry. Existing drafts are preserved, so this changes navigation preference rather than preventing data loss. The patch chooses creation time, not last edit time, and a newest draft whose project is gone prevents the helper from considering older valid drafts.
Recommendation. Keep open: decision needed. Confirm the resume rule and select among valid projects before choosing the newest draft.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Check: PR #7422. Current head 326f2d5c: required checks are absent from canonical latest-commit metadata: Test, Check, Mobile Native Static Analysis, Release Smoke.
Limits. Required CI checks are not present on the current head in canonical check metadata.
Request. Explain that a delegated-task wait timeout stops the parent wait, not the child.
Audit finding. The focused commit changes only tool wording, schema annotations, and text tests. Current V2 parent source still says wait blocks until completion or timeout without the explicit no-cancellation guidance, while pinned main does not expose this API at all. This does not fix polling CPU cost or change wait and spawn policy.
Recommendation. Keep open: work remains. Restack the wording commit onto the current V2 orchestrator branch.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Observed GitHub metadata. GitHub state OPEN. Author mwolson. Updated 2026-09-01T05:36:36Z. Draft no. Target t3code/codex-turn-mapping. Head 99fab47fe50a8c9b64707caf5f2fdc46f1c28e86. Branch fix/delegate-task-wait-timeout. Size +181700 / -85511, 886 files, 241 commits. Mergeability CONFLICTING. Merge state DIRTY. Target is not main. Metadata snapshot 2026-09-01T11:57:41.491617+00:00.
Pr: PR #2829. The open orchestrator branch is the required parent.
Check: GitHub check. Canonical latest-head checks: Check success, Test success, Mobile Native Static Analysis success, Release Smoke success.
Limits. The 45-line wording change was fully read, but the submitted 287650-line inherited diff was not. The reported live wait behavior was not reproduced.
Request. Use providers on a connected WSL or remote environment for the General text-generation model setting.
Audit finding. Main still reads primaryServerProvidersAtom and writes textGenerationModelSelection through primary settings. The PR scopes discovery, writes, and restore-default behavior to a selected usable environment, which addresses the empty Windows-primary picker. Its automatic first-usable target and null-primary cast need review so reconnects and client-only sessions cannot direct settings writes to an unintended host.
Recommendation. Keep open: work remains. Review target selection and the no-environment case before accepting the environment-scoped picker.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Current-head required checks were not reported: Check, Mobile Native Static Analysis, Release Smoke, Test. GitHub reports merge conflicts with main on the collected head.
Request. Add an explicit Hide keyboard control to the SwiftUI thread composer.
Audit finding. The experimental parent already has drag-based dismissal, but no visible footer button. This diff now adds only an opt-in button that calls the existing dismiss handler and keeps nonempty drafts expanded. It is therefore only partly related to the older drag and sizing work and is not fully replaced by it.
Recommendation. Keep open: work remains. Review the remaining footer button and require current-head interaction evidence.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Observed GitHub metadata. GitHub state OPEN. Author saphid. Updated 2026-08-29T13:44:49Z. Draft no. Target t3code/rebuild-mobile-app-swift. Head 7fa7bd12463e263f9ea39d5627dbe2588d68b00d. Branch agent/issue125-swiftui-keyboard-dismiss. Size +80 / -0, 3 files, 2 commits. Mergeability MERGEABLE. Merge state CLEAN. Target is not main. Metadata snapshot 2026-09-01T11:57:41.491617+00:00.
Check: GitHub check. Current head 7fa7bd12: Test=SUCCESS, Check=SUCCESS, Mobile Native Static Analysis=SKIPPED, Release Smoke=SUCCESS. SwiftUI contract fixtures and native tests passed.
Limits. Current-head native CI passed, but current-head screenshots and the long-draft interaction video remain pending in the body. Mobile Native Static Analysis was skipped on the current head; no passing native static result is claimed.
Audit finding. Main still sends theme strings through Color.parseColor, which cannot parse rgba values. The patch adds clamped CSS rgba conversion before the existing fallback, so the default skill theme has a concrete unresolved Android bug. The reported lint run skipped all native analyzers.
Recommendation. Keep open: work remains. Run Kotlin compile and native checks for the color parser change.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Check: PR #7527. Current head 0261e278: required checks are absent from canonical latest-commit metadata: Test, Check, Mobile Native Static Analysis, Release Smoke.
Limits. No native compile or device check was available in the reported verification. Required CI checks are not present on the current head in canonical check metadata.
Request. Register the MCP servers declared by enabled Claude plugins.
Audit finding. Main only adds the T3 MCP server at session creation and has no plugin-manifest discovery or dynamic registration. The proposed loader preserves plugin server names, but ignores installation scope and projectPath and takes the first install with a manifest. That can load another workspace installation or the wrong scoped version, while the related skill PR already handles this distinction.
Recommendation. Keep open: work remains. Apply workspace-aware plugin install selection and share it with plugin skill discovery.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Record modified Tab shortcuts in Settings without capturing plain Tab focus navigation.
Audit finding. Both current keybinding recorder paths still return for every Tab press. The recent-thread switcher explicitly depends on this patch and includes its commit, so it is a stacked prerequisite rather than an obsolete duplicate. Existing thread traversal can use these chords without adopting recent-thread ordering.
Recommendation. Keep open: work remains. Land the recorder fix first, then rebase its recent-thread-switcher dependent.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Use the sidebar project filter as the destination for its New thread button.
Audit finding. The current palette-open event carries no preferred project, so the sidebar filter cannot reach the project picker through that path. The searchable-filter merge changes the filter control, not the new-thread destination. This patch deliberately leaves keyboard and command-palette entry points tied to the viewed thread.
Recommendation. Keep open: work remains. Review filtered-project behavior for the sidebar button and Shift-click while preserving other entry points.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Populate the composer file picker when the user types @ without a search string.
Audit finding. Main still rejects an empty composer path query and hides its loading state until text is entered. The diff enables the existing empty-query option for the composer. The latest Check failure is specifically the two committed PR-only SVG files, which the CI log lists before exiting with code 1.
Recommendation. Keep open: work remains. Remove the committed screenshot assets and rerun the failed Check job.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Identify the upstream provider beside selected model labels on web and mobile.
Audit finding. The mobile model-source merge now distinguishes picker entries with sub-provider subtitles. Web selected labels still return only the stripped model name, and mobile selected controls still use a width-capped name, so the cross-client selected-value scope remains. Preserve the new mobile grouping while applying any remaining label and width changes.
Recommendation. Keep open: partial fix. Rebase only the remaining selected-label work on the current mobile model-source picker.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Open the existing model-options picker with a configurable keyboard shortcut.
Audit finding. Main registers and handles modelPicker.toggle but has no modelOptionsPicker.toggle command or composer handle method. The patch supplies keyboard access for both compact and expanded composer controls without adding reasoning cycling. The reasoning-cycle proposal is separate and not a landed replacement.
Recommendation. Keep open: work remains. Review keyboard opening, selected-option focus, and composer focus restoration in both composer layouts.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Mark user messages that are waiting behind an unsettled turn as Queued in the web timeline.
Audit finding. Main has no queued field or visible queued label in its message-row derivation. The provider runtime still accepts queued Codex follow-ups, so the display gap remains despite newer work-log folding changes. This patch adds a timestamp-derived flag, but its PR-only screenshot files should be removed before merge.
Recommendation. Keep open: work remains. Review queued-to-active label clearing on current turn events and remove PR-only screenshot files.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. The committed screenshot binaries were not visually inspected. No current-head result for required checks: Test, Check, Mobile Native Static Analysis, Release Smoke.
Request. Ignore Claude busy-only heartbeats that arrive after their turn ends.
Audit finding. Main still emits running or waiting for every status and api_retry message, even with no active turn. Removing the post-turn context query can reduce this traffic but does not protect manual compaction or late retry messages. The two proposed guards address that remaining path, while the focused test should wait for a deterministic event drain.
Recommendation. Keep open: work remains. Replace the scheduler-yield wait and review the post-compaction heartbeat guard.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Show a pointer cursor on raw pull-request buttons.
Audit finding. Current PR check links, reviewer rows, and review-thread toggles still lack the proposed pointer classes. Recent PR list layout and rendering fixes did not address these controls. The diff is styling-only, but its new static-markup assertion does not verify a hover behavior and the UI pass is unchecked.
Recommendation. Keep open: work remains. Remove the markup-only test and provide hover evidence for the changed controls.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Keep a first-send bootstrap error visible when the failed server thread returns to draft state.
Audit finding. ChatView on main still chooses only the draft-keyed error after returning to a draft. The fresh-ID retry and server cleanup fixes allow retries but do not merge the draft and server error timestamps. This patch addresses error visibility, not the cause of a failed fetch.
Recommendation. Keep open: work remains. Rebase the timestamp-aware error resolver onto the current bootstrap retry flow.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Open T3 Connect onboarding after Clerk sign-in reloads the desktop renderer.
Audit finding. Main still ignores the initial signed-in snapshot after Clerk reloads the renderer, so onboarding is skipped. The diff records a pending marker, but unguarded sessionStorage access can throw before openSignIn and block authentication, as the unresolved review reports. The marker must remain optional to the auth flow.
Recommendation. Keep open: work remains. Make marker read, write, and removal best effort so storage failure cannot block sign-in.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Offer only skills and slash commands that the selected insertion syntax can invoke.
Audit finding. Main Claude skill discovery still ignores invocation flags and skillOverrides, reports every discovered skill enabled, and uses frontmatter names. The latest composer deduplication merge removes repeated names but does not correct those rules or slash-command insertion. This branch addresses the reported identity, flag, precedence, and mobile-filtering findings, but its rebase must preserve current deduplication. The later request for Codex explicit-only metadata and a new menu filter is separate scope.
Recommendation. Keep open: work remains. Rebase the reviewed Claude invocation changes on the deduplicated menus, keeping the new Codex filtering request separate.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Use the thread model for source-control writing when no usable dedicated writer is configured.
Audit finding. GitManager still selects the global text-generation model when the dedicated writer is unset. The recent repository-instruction merge changes prompts, not this selection or the missing modelSelection RPC field. This PR wires web and mobile selections end to end but adds only a missing-instance test for a much broader fallback order.
Recommendation. Keep open: work remains. Add focused tests for dedicated-writer and thread-model precedence before review.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Stop and explain repeated desktop backend crashes before readiness.
Audit finding. Main still restarts every unexpected spawned-child exit while desiredRunning is true, with no crash-count limit. The patch adds bounded output capture, a five-crash limit, a primary dialog, and a WSL error row. Its new documentation incorrectly implies default desktop startup always conflicts on 3773, although main scans higher ports.
Recommendation. Keep open: work remains. Correct the port-conflict documentation and verify the primary and WSL crash-loop cases.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Native error dialogs and WSL behavior were not reproduced. Required CI checks are absent at the reviewed head: Test, Check, Mobile Native Static Analysis, Release Smoke.
Request. List projects and threads in an explicitly selected T3 state directory without writing to it.
Audit finding. Main has a general SQLite query script but no thread:list command. The proposed listing keeps threads from soft-deleted projects even though it removes those projects from the returned project list. Its read-only connection also lacks the existing five-second busy timeout, so the new command needs these fixes before review.
Recommendation. Keep open: work remains. Filter threads by live parent projects and use the existing read-only database timeout and structured error conventions.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Shortcut hints should use platform-correct key labels and the configured stash binding.
Audit finding. The current settings key pill still formats modifiers manually and the diff comment hint still combines Mac and Ctrl labels. The composer stash now receives stashShortcutLabel and already uses that configured label in its empty state, so that portion is covered. The formatter and diff hint remain useful, while the old stash edit should be dropped.
Recommendation. Keep open: partial fix. Rebase the platform-aware formatter and diff hint while dropping or adapting the old stash empty-state edit.
Confidence high. Release: In stable source. PR readiness: Needs small changes.
Request. Resolve provider transcript model names to the corresponding usage pricing aliases.
Audit finding. Main still performs an exact normalized rate lookup without the two alias fallbacks proposed here. Newer pricing code preserves provider-qualified names, so this patch must retain that behavior when rebased instead of restoring the old bare-name lookup. Existing transcript identifiers do not need a migration.
Recommendation. Keep open: work remains. Rebase the alias fallback on the current provider-qualified rate lookup.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Use a static AppImage runtime so Linux systems without libfuse2 can launch the desktop app.
Audit finding. Main still uses electron-builder 26.15.6 without an AppImage toolset override. The two-file patch selects the static 1.0.3 runtime only for Linux, but its test contains formatting errors and its report does not verify an in-place AppImage update. It does not address the separate Ubuntu sandbox or instance-lock failures.
Recommendation. Keep open: work remains. Fix the test formatting and provide a Linux in-place update result for the static runtime.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Static runtime launch and updater behavior were not reproduced in this audit. Required Check, Test, Release Smoke, Mobile Native Static Analysis checks are absent on the collected head.
Request. Dismiss hovered chat tooltips on timeline scroll while preserving keyboard-focused tooltips.
Audit finding. The current tooltip wrapper still delegates directly to Base UI and the timeline has no hover-dismiss controller. Earlier tooltip merges change stacking, not scroll ownership. The patch keeps dismissal off React state updates and limits it to timeline hover tooltips, so the original scope remains missing.
Recommendation. Keep open: work remains. Review delayed-hover cancellation and focused-tooltip preservation on the current timeline.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Show checkbox state for multi-select questions on web and mobile.
Audit finding. Main still renders web options with only a trailing selected check and mobile options without checkbox semantics. This patch adds independent checkbox state and keeps web number hints, but it predates the shared banner and mobile theme rewrites. It also commits four PR-only image assets, which should be uploaded rather than kept in the code change.
Recommendation. Keep open: work remains. Port the checkbox markup to current components and remove the PR-only assets from the branch.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. The four embedded-image SVG evidence files were structurally checked but not visually viewed. Canonical latest-commit metadata has no results for these required checks: Test, Check, Mobile Native Static Analysis, Release Smoke.
Request. Allow SSH readiness responses that take longer than one second.
Audit finding. Main still applies a one-second per-request limit to desktop-side tunnel probes. The diff raises the initial tunnel probe to eight seconds but leaves the existing-tunnel readiness check at the one-second default, so reusing a healthy slow tunnel can still fail. The earlier cold-start change extended remote startup time, not this reuse path.
Recommendation. Keep open: work remains. Apply the tunnel probe limit to the existing-tunnel check and test slow-link reuse.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Map Grok Build to an ACP model identifier accepted by Grok.
Audit finding. Main still normalizes the default to grok-build and passes that identifier to setSessionModel when it differs from the CLI current model. The Grok reliability and in-session model switching merges did not add the proposed alias. The PR also fixes the mock catalog that currently accepts the product name, so a real-CLI model check remains needed.
Recommendation. Keep open: work remains. Confirm the current Grok CLI model catalog and review the alias against it.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Open a new right-panel terminal with a direct configurable keyboard command.
Audit finding. Main has bindable panel toggle and maximize commands, but terminal.new selects the right panel only when focus is already there. The patch adds rightPanel.openTerminal through the contract, default bindings, ChatView handler, and user docs. This covers web and desktop keyboard entry without changing the bottom drawer or mobile.
Recommendation. Keep open: work remains. Verify the new binding from composer focus and from both terminal placements, including a custom binding.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Check: github.com/pingdotgg/t3code/pull/7835/checks. Head 002f817 has no complete required CI result. Missing: Check, Test, Release Smoke, Mobile Native Static Analysis. The unrelated Vercel marketing preview requires authorization.
Limits. No client keyboard interaction was run. Required Check, Test, Release Smoke, Mobile Native Static Analysis checks are absent on the collected head. The current head conflicts with main, and post-rebase checks are not available.
Request. Ignore launcher stdout preambles only before Codex protocol traffic begins.
Audit finding. Main still treats every nonempty stdout line as JSON-RPC, so a mise launcher preamble can fail initialization. The diff enables preamble tolerance for child-process clients and restores strict parsing after the first protocol message. It still logs the full raw line before deciding to ignore it, which conflicts with its claim that ignored diagnostics retain only byte length.
Recommendation. Keep open: work remains. Move ignored-preamble handling before raw payload logging and retain the strict post-start test.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Latest-head required check results are absent for Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports a conflict with main. Any passing checks apply to the existing head, not a rebased result.
Request. Record unexpected provider process exits as errors when no runtime.error arrives.
Audit finding. Main clears activeTurnId on session.exited but still sets the session to stopped and does not copy the exit reason into lastError. The patch addresses that remaining error-classification gap. Its added test uses the older polling harness and needs to follow current receipt-based test patterns. The required Check job is failing at this head.
Recommendation. Keep open: work remains. Update the ingestion test to use current receipts and rerun the focused tests and failing Check job.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Show a raw mobile patch when the native review view cannot be resolved.
Audit finding. Main still non-null-asserts the native review view, so the missing-native-module crash path remains. This diff adds a raw-patch fallback, but file-navigation controls still target the missing native ref and iOS has no fallback refresh action. The crash guard is useful; the fallback controls need to work or be hidden.
Recommendation. Keep open: work remains. Finish the raw fallback's file-navigation and iOS refresh behavior, then run required CI.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Let separate server instances on one machine advertise different environment labels.
Audit finding. Main derives each descriptor label from the OS computer name or hostname and has no environment-label override. The diff adds a trimmed flag and environment variable with precedence tests. It does not refresh descriptors already cached by paired clients, so existing connections can need a new pairing.
Recommendation. Keep open: work remains. Add user documentation for the override and the existing-client label refresh limit.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Display a successful Codex MCP result as completed when its outer status says failed.
Audit finding. Main still trusts failed or declined item status in both CodexAdapter and ActivityPayloadProjection. The patch changes both paths, but treats every non-null result without item.error as success, so an MCP result carrying isError needs an explicit case. The merged mixed-tool display fix does not change this provider classification.
Recommendation. Keep open: work remains. Add coverage for MCP error results before accepting the status override.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. The upstream contradictory-result shape was not reproduced against a live Codex process. Required CI checks are absent at the reviewed head: Test, Check, Mobile Native Static Analysis, Release Smoke.
Request. Delay loading the Claude SDK until a Claude query or capability probe needs it.
Audit finding. Main imports the SDK query functions at module load in both the adapter and provider, so non-Claude startup still evaluates the SDK. The diff replaces those runtime imports and adds tests for module isolation and interrupted loading. Recent probe isolation changes do not change this import behavior.
Recommendation. Keep open: work remains. Review the lazy-import cleanup path and run the adapter and capability-probe tests.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Commit: Commit f86c5e8c8700. The newer probe change only changes IDE detection settings.
Limits. Latest-head required checks are not reported: Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports merge conflicts against the target branch.
Request. Prevent Files tree rows from sitting beneath the header on older iPadOS.
Audit finding. Main applies automatic header insets only when native Liquid Glass is supported. The diff adds manual top padding for every other platform, which includes Android as well as older iOS. The reported proof covers two iPadOS versions, so the wider layout change needs a platform-specific decision.
Recommendation. Keep open: work remains. Restrict the manual header inset to the affected iOS layout and check Android spacing.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Limits. Latest-commit metadata has no required runs for Test, Check, Mobile Native Static Analysis, Release Smoke. GitHub reports conflicts with the target branch.
Request. Keep a PR link opened from another PR in the owning thread panel.
Audit finding. Main PR markdown passes an environment but no owning thread into the link handler. Recent GitHub-reference autolinks add clickable links without restoring that thread context. This diff separates PR navigation scope from file workspace scope, but its tests mostly assert prop wiring rather than navigation behavior.
Recommendation. Keep open: work remains. Test a nested PR link across same-project and foreign-project thread tabs using the current markdown context.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Stop retrying a thread subscription when the server explicitly says the thread no longer exists.
Audit finding. Main still retries expected thread-subscription failures every 250 milliseconds. The diff adds a typed not-found stop, but completes only the current inner stream, so foreground wakeups still resubscribe to the missing thread. It also reuses the existing deletion path whose queued persistence write can recreate a removed cache entry. The new config stream does not add a terminal missing-thread error or remove the generic expected-error retry loop.
Recommendation. Keep open: work remains. Gate foreground resubscription for a thread already marked deleted and test that behavior.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Latest main change. The new config stream does not add a terminal missing-thread error or remove the generic expected-error retry loop. Keep the disposition and preserve config dispatch while rebasing terminal-error handling.
Limits. Latest-commit metadata has no required runs for Test, Check, Mobile Native Static Analysis, Release Smoke.
Request. Use the Azure repository short name consistently when opening PRs from chat links and thread badges.
Audit finding. Main selects Azure repositories by short name on the server but sends displayName from web links. The diff shares the selector and repairs stored linked-PR targets, while the larger Files-tree PR 7443 includes much of the same change. Its linked-project lookup still matches project ID without checking environment, so duplicate IDs across environments need coverage.
Recommendation. Keep open: work remains. Scope the linked-project lookup by environment and keep this fix separate from the Files-tree feature.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Reuse an existing mobile project before trying to clone into its non-empty directory.
Audit finding. Main checks duplicate local folders after selection but attempts the clone before checking its destination. The patch moves that check earlier and removes the duplicate alert, but the unresolved tilde-path review is valid: resolveProjectPathForDispatch leaves ~/ paths intact while stored roots are absolute. The default destination can therefore still miss the existing project.
Recommendation. Keep open: work remains. Resolve the destination through the environment before the duplicate lookup and test a tilde-based default path.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Comment: Discussion comment. Review discussion checked against the source.
Check: github.com/pingdotgg/t3code/pull/8229/checks. Head 84ceb13 has no complete required CI result. Missing: Check, Test, Release Smoke, Mobile Native Static Analysis. The unrelated Vercel marketing preview requires authorization.
Limits. The reported iPhone verification was not rerun. Required Check, Test, Release Smoke, Mobile Native Static Analysis checks are absent on the collected head.
Request. Tell Grok and Cursor to use the in-app browser when T3 browser tools are attached.
Audit finding. Only Codex currently receives the dedicated browser preference instructions. The PR moves that block into shared code and adds a first-new-session prompt prefix for Grok and Cursor, while preserving the existing opt-out through absent MCP credentials. Recent browser result and permission fixes do not add this provider guidance.
Recommendation. Keep open: work remains. Rebase the first-prompt prefix on current Grok and Cursor lifecycle code.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Isolate desktop preview cookies and cache by environment and project.
Audit finding. Main still derives preview partitions from environmentId and clears every loaded preview session. The diff adds project identity through IPC, configuration lookup, clear actions, and guest remount keys, so the reported cross-project sharing remains unfixed. It intentionally starts fresh per-project cookie stores and needs a two-project desktop check, including linked threads and remote environments.
Recommendation. Keep open: work remains. Rebase onto current main and complete the focused client verification before maintainer review.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Number project-picker jump shortcuts after filtering and ranking the visible results.
Audit finding. Main still calls enumerateCommandPaletteItems while constructing projectThreadItems, before filterCommandPaletteGroups ranks the matches. The patch moves enumeration into the filter for both new-thread project views, so a lone filtered project can receive jump shortcut 1. The searchable sidebar project filter is a different entry point and does not fix the command palette.
Recommendation. Keep open: work remains. Add focused filtered and ranked shortcut coverage to the existing palette logic tests, then review the patch.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Use the instance CLI authentication result and show an instance-specific login hint.
Audit finding. Main still treats a capability object as proof of authentication. This diff adds CLI auth status and account labels, but its login hint ignores a configured binary and an environment-only CLAUDE_CONFIG_DIR. Its fallback also keeps the optimistic SDK verdict when the CLI probe cannot answer, unlike the competing auth-status proposal.
Recommendation. Keep open: work remains. Combine the auth-status proposals and make login hints use the resolved binary and config directory.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.
Request. Keep valid settings and a recovery copy when one provider instance fails schema decoding.
Audit finding. Main still falls back to default settings after a whole-file decode error without preserving the original file. The PR correctly salvages valid instance entries, but sets quarantinedThisEpisode before the backup write succeeds, so one failed write prevents backup retries for that error episode. The author confirms the watcher-driven backup guard has no test. Keep the recovery fix open and repair that failure path.
Recommendation. Keep open: work remains. Set the quarantine-complete flag only after a successful write and test a failed first backup.
Confidence high. Release: Not applicable. PR readiness: Needs small changes.