chore: sync with upstream pingdotgg/t3code main - #59
Conversation
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…gg#12635) Co-authored-by: Julius Marminge <51714798+juliusmarminge@users.noreply.github.com> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…art (pingdotgg#12625) Co-authored-by: Julius Marminge <51714798+juliusmarminge@users.noreply.github.com> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…12438) Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
…10831) Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
…tgg#12657) Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
…ion (pingdotgg#10098) Co-authored-by: shivamhwp <91240327+shivamhwp@users.noreply.github.com>
Co-authored-by: Julius Marminge <51714798+juliusmarminge@users.noreply.github.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com> Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: codex <codex@users.noreply.github.com>
…case screenshots (pingdotgg#13316) Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com> Co-authored-by: Exotic <118054752+extoci@users.noreply.github.com>
…otgg#11580) Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-authored-by: Simone <185146821+Lucenx9@users.noreply.github.com>
…3355) Co-authored-by: Yordis Prieto <yordis.prieto@gmail.com> Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
…pstream-20260924-1 Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com> # Conflicts: # .github/workflows/release.yml # apps/desktop/src/app/DesktopObservability.ts # apps/desktop/src/backend/DesktopBackendConfiguration.test.ts # apps/desktop/src/backend/DesktopBackendConfiguration.ts # apps/server/src/bin.test.ts # apps/server/src/cli/config.test.ts # apps/server/src/cli/config.ts # apps/server/src/cli/pair.ts # apps/server/src/config.ts # apps/server/src/environment/ServerEnvironment.test.ts # apps/server/src/observability/Layers/Observability.ts # apps/server/src/provider/Layers/ProviderRegistry.ts # apps/server/src/server.test.ts # apps/server/src/serverLogger.test.ts # apps/server/src/serverLogger.ts # apps/server/src/ws.ts # apps/web/src/components/ChatView.tsx # apps/web/src/components/Sidebar.tsx # packages/shared/src/otelEnvironment.test.ts # packages/shared/src/otelEnvironment.ts
…ovider and bootstrap services Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
…checks Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
PR SummaryMedium Risk Overview Mobile native review diff gains optional word wrap with Unicode-aware layout on Android and iOS (plus Robolectric/Catalyst tests), and Android agent notifications expose a showcase path for staged screenshots. Snap Shot on Linux Wayland improves AT-SPI window matching for Flatpak proxies and GTK4 unnamed frames; the desktop renderer CSP allows Release CI runs tests in separate sharded jobs (mirroring main CI) and blocks CLI publish on them; mobile EAS preview propagates Android Google Services config into the job environment after Reviewed by Cursor Bugbot for commit 1ae75b2. Bugbot is set up for automated code reviews on this repo. Configure here. |
| const tab = (yield* SynchronizedRef.get(tabsRef)).get(tabId); | ||
| yield* attempt({ operation: "recording.cursor", tabId, webContentsId: wc.id }, () => | ||
| wc.send(RECORDING_CURSOR_CHANNEL, true, options, tab?.controller), | ||
| ); |
There was a problem hiding this comment.
Recording options can be dropped
Medium Severity
startRecording marks the tab as recording before it stores recordingInputOptions, and the guest installs the cursor on the first preview:recording-cursor message via ??=. A dom-ready restore (or another sender) that lands in that window installs with the default flags (showKeyPresses/showMousePresses both false) and later sends with the real settings are ignored, so key and mouse overlays stay off for that page.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit 8a98267. Configure here.
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: Scenario and decoded snapshot size10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
…eads Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 1ae75b2. Configure here.
| : DEFAULT_RECORDING_INPUT_OPTIONS; | ||
| recordingCursor ??= installRecordingCursor(document, window, options, (input) => | ||
| ipcRenderer.send(RECORDING_INPUT_CHANNEL, input), | ||
| ); |
There was a problem hiding this comment.
Recording overlays keep default options
Medium Severity
The guest installs the recording cursor only once and never reapplies showKeyPresses or showMousePresses. startRecording adds the recording consumer before those options are stored, while dom-ready can send the cursor with undefined options. The later send with the real settings is ignored, so key and mouse decorations stay off for that take.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit 1ae75b2. Configure here.


Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.