Skip to content

feat: primary workspace switch with ^ alias - #67

Merged
eersnington merged 2 commits into
mainfrom
primary-workspace-switch
May 4, 2026
Merged

feat: primary workspace switch with ^ alias#67
eersnington merged 2 commits into
mainfrom
primary-workspace-switch

Conversation

@eersnington

Copy link
Copy Markdown
Owner
  • Add ^ as a navi switch target for the repo-primary workspace.
  • Resolve the primary workspace through validated repo-root discovery instead of assuming workspace name default.
  • Update CLI help, docs, and integration coverage for primary-workspace switching.

@eersnington eersnington added the release:minor Requires minor release label May 4, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 8 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/repo/paths.rs">

<violation number="1" location="src/repo/paths.rs:279">
P2: Don't derive the primary workspace root from `repo_storage_path` alone; absolute repo-storage pointers will make this point at the wrong directory and break `switch ^` in those repos.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Comment thread src/repo/paths.rs Outdated
@eersnington
eersnington merged commit fa9a62c into main May 4, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:minor Requires minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant