chore: sync with upstream pingdotgg/t3code main - #12
Conversation
…gdotgg#4905) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
pingdotgg#4897) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…ngdotgg#4882) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…it (pingdotgg#4949) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…ingdotgg#4867) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…gdotgg#4964) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…oring (pingdotgg#4967) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…ingdotgg#4975) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…g#4807) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…hy (pingdotgg#4988) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…tgg#4855) Co-authored-by: Claude Fable 5 <noreply@anthropic.com> Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
actions/checkout implements sparse-checkout with --filter=blob:none, so the runner repo is a partial clone missing the .repos/ blobs. eas-cli archives the project via `git clone --depth 1 file://<workspace>`, which upload-pack cannot serve from a partial clone and exits 128 (pingdotgg#4802 broke both EAS build workflows this way). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…pstream-20260730-1 Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com> # Conflicts: # apps/server/src/persistence/Migrations.ts # apps/server/src/server.test.ts
|
Important Review skippedToo many files! This PR contains 221 files, which is 121 over the limit of 100. To get a review, narrow the scope: Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (233)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
PR SummaryMedium Risk Overview CI adds workflow concurrency (cancel in-flight PR runs), sparse checkouts excluding Marketing/README reflect mobile on App Store and Google Play (footer, download page, hero links), refreshed doc index paths ( Mobile (1.0.1) includes message-body thread search with snippets in list rows, connection wakeups (short background → probe, ≥10s → reconnect) plus network refresh on foreground, pre–Liquid Glass iOS home/archive headers (pull-down search + bottom toolbar), composer image drag-and-drop on iOS, favicon load cache with supersession handling, thread feed scroll/layout fixes ( Reviewed by Cursor Bugbot for commit 4d151ab. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
Reviewed by Cursor Bugbot for commit 4d151ab. Configure here.
| setPathInput(getAddProjectInitialQuery(environmentBaseDirectory)); | ||
| } | ||
| }, [environment, setPathInput]); | ||
| }, [environmentBaseDirectory, environmentId, setPathInput]); |
There was a problem hiding this comment.
Browse path ignores late base directory
Medium Severity
useBrowsePathInput only refreshes the path when environmentId changes, and the ref is initialized to the current id, so the effect does not run on mount. If baseDirectory arrives later from server config for that same environment, the input stays on the ~/ default instead of the configured add-project root.
Reviewed by Cursor Bugbot for commit 4d151ab. Configure here.


ProjectionThreadTitleRegenerationmigration from id 35 to 36: the fork already shipped ids 33-35 (ProjectionThreadParentplus the previously slottedSettled/Snoozedmigrations), and the migrator only remembers the highest id that ran, so reusing 35 would make existing fork installs silently skip the new migration. Its test's id pins moved with it.orchestrationEngineLayer/projectionSnapshotQueryLayermock variables inserver.test.ts(they are shared with the subagent MCP toolkit dependencies from divergence 0003) instead of upstream's inline mocks, and folded in upstream's newsearchThreadsmock so the content-search feature stays covered..repos/regeneration needed: upstream's lockfile changes did not bump the vendoredeffect-smol/alchemy-effectdependencies.