sync: port duplicate workspace roots guard and record July 16 upstream ancestry (#3829) - #158
Conversation
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
@coderabbitai review |
✅ Action performedReview finished.
|
7b3a640 to
9f5a104
Compare
c24c2c1 to
5c6da4e
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9f5a104857
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
9f5a104 to
10a8aa4
Compare
5c6da4e to
34b0027
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 20a2420a45
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| yield* requireActiveProjectWorkspaceRootAbsent({ | ||
| readModel, | ||
| command, | ||
| workspaceRoot: command.workspaceRoot, | ||
| exceptProjectId: command.projectId, | ||
| }); |
There was a problem hiding this comment.
Handle normalized autobootstrap matches before create
When the web server auto-bootstrap path in serverRuntimeStartup.ts looks for an existing project, it uses ProjectionSnapshotQuery.getActiveProjectByWorkspaceRoot, which is an exact workspace_root = ... lookup. For an existing active row that differs only by formatting this guard now normalizes and rejects the project.create instead of letting auto-bootstrap reuse the existing project/thread, so databases with older /repo/ rows or Windows case variants fail to produce bootstrapped welcome targets. Normalize the lookup/reuse path or translate this duplicate into the existing project before dispatching create.
Useful? React with 👍 / 👎.
20a2420 to
4a940ef
Compare
34b0027 to
4025f28
Compare
pingdotgg#3978) (cherry picked from commit 2fdc704)
Co-authored-by: codex <codex@users.noreply.github.com> (cherry picked from commit b982337)
Co-authored-by: codex <codex@users.noreply.github.com> (cherry picked from commit 54f167e)
(cherry picked from commit 5e8b2c8)
(cherry picked from commit 0418301)
Cherry-pick of upstream d114e27 (Prevent duplicate project workspace roots). Adds requireActiveProjectWorkspaceRootAbsent invariant to the decider, normalizeProjectPathForComparison/normalizeProjectPathForDispatch moved to packages/shared/src/path.ts, and sidebar project-grouping dedupe-by-physical-key logic (prefers the primary environment's member, then freshest updatedAt/createdAt, then highest id). apps/server/src/provider/Layers/CursorAdapter.test.ts is intentionally left unchanged: upstream's hunk there fixes an unrelated ACP-stdio session/cancel test behavior that has no equivalent in the fork's SDK-based Cursor adapter test harness, and is unrelated to the workspace-root invariant this commit otherwise ports. Co-authored-by: codex <codex@users.noreply.github.com>
4a74aa1 to
7009cae
Compare
4025f28 to
1ced949
Compare

What changed
#3829).origin/mainis zero behindupstream/mainafter merge.Validation
vp check/vp run typecheck— passgit rev-list --left-right --count HEAD...upstream/main→911 0Stack: 4 of 4; depends on #157.