fix(shared): normalize a bare Windows drive root the same as C:\ / C:/

[arhxam]

What Changed

isRootPath (packages/shared/src/path.ts) now requires the drive separator (^[a-zA-Z]:[/\\]$ instead of [/\\]?), so a bare C: normalizes to the drive root like C:\ and C:/. Added a test.

Why

With the separator optional, a bare C: was treated as already-canonical and trimTrailingPathSeparators returned it unchanged as C:, while C:\ and C:/ normalize to the drive root. The canonical project-path-identity helpers normalizeProjectPathForComparison / normalizeProjectPathForDispatch — used for project matching and preference-key dedup across the server, client-runtime, and web — therefore treated C: and C:\ as different locations, so a project rooted at a bare drive silently failed identity/dedup checks against the same drive expressed as C:\.

Requiring the separator makes a bare C: fall through to the existing canonicalizer, which appends the separator, so C:, C:\, and C:/ all normalize to the same root. This is also more correct semantically: bare C: in Windows means "current directory on C:", not the drive root. Non-root paths and the / / \ roots are unaffected.

Verified: the new test fails on current code and passes with the change (reverted-source re-run); full packages/shared toolchain green (vp test, tsgo, vp lint, vp fmt).

Checklist

• This PR is small and focused
• I explained what changed and why
• No UI changes

---

Note

Low Risk
Small, localized path-normalization fix with a regression test; behavior change only affects edge-case Windows drive-root strings used for project identity.

Overview
Windows drive-root normalization in packages/shared/src/path.ts is tightened so C:, C:\, and C:/ compare as the same project location.

isRootPath now only treats drive roots that include a separator (^[a-zA-Z]:[/\\]$), not a bare C:. That lets bare C: go through the existing trim/canonicalize path (which appends \), aligning with how normalizeProjectPathForComparison and normalizeProjectPathForDispatch are used for project matching and dedup on server, client-runtime, and web. Non-root paths and / / \ roots are unchanged.

A focused unit test covers dispatch/comparison equality and confirms non-root trailing-separator trimming still works.

^{Reviewed by Cursor Bugbot for commit 487219f. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix isRootPath to treat bare Windows drive C: as non-root

A bare drive specifier like C: was incorrectly treated as a Windows drive root. The fix tightens the regex in isRootPath (path.ts) to require a trailing separator, so only C:\ or C:/ match as roots. As a result, normalizeProjectPathForDispatch("C:") now returns "C:\\" and normalizeProjectPathForComparison("C:") returns "c:\\"

^{Macroscope summarized 487219f.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 2b6a41a1-7976-4a8b-94e2-9648e6b62ad1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Approvability

Verdict: Approved 487219f

Straightforward bug fix that corrects Windows drive root path normalization. The regex change ensures C: (current directory on C:) is normalized to C:\ for consistent comparison with C:\ and C:/. Small, focused change with comprehensive test coverage.

^{You can customize Macroscope's approvability policy. Learn more.}