Skip to content

feat: workspace UI Enhancements and Workflow Improvements#102

Merged
sirily11 merged 2 commits into
mainfrom
workspace
Jun 8, 2026
Merged

feat: workspace UI Enhancements and Workflow Improvements#102
sirily11 merged 2 commits into
mainfrom
workspace

Conversation

@autopilot-project-manager

Copy link
Copy Markdown
Contributor

Changes

  • Removed code review and hooks from certain files when in plan mode or the plan is ready.
  • Added an "Automation" menu to the macOS menu bar.
  • Created standalone windows for Autopilot, Hooks, and Custom Context Menus.
  • Updated WorkspaceSwitcher.swift to support renaming and deleting workspaces.
  • Reevaluated workflow for smoother user transitions and testing integration.
  • Implemented renaming and deleting functionality in WorkspaceSwitcher.swift.

Introduce multi-workspace support (AppWorkspace model, per-workspace
defaults, switching/creation) and surface a WorkspaceSwitcher in the
sidebar's toolbar and the menu bar.

- Place the WorkspaceSwitcher in the sidebar column's toolbar (not the
  main content toolbar, and not inside the sidebar body).
- Lift the Create Workspace sheet out of the toolbar-hosted view to a
  stable parent (sidebar + menu bar) via a binding, fixing the
  present/dismiss flicker loop caused by toolbar re-evaluation.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
rxcode Ready Ready Preview, Comment Jun 8, 2026 2:46pm

Request Review

…s-test crash

PlanCardViewTests hosts PlanSheetView via ViewInspector's ViewHosting.
expel() removes the host but AppKit tears the backing NSHostingView down
asynchronously. PlanSheetView's PlanSheetInspection has a main-actor
back-deployed deinit, so the deferred teardown could hop onto the main
actor during a later, unrelated test (testDecidedPlan_doesNotSuppress in
the new PlanModeHookSuppressionTests suite) and double-free on the Swift
concurrency runtime (POINTER_BEING_FREED_WAS_NOT_ALLOCATED, SIGABRT),
aborting the whole RxCodeTests process and failing CI with exit 65.

Pump the main run loop in a synchronous tearDown so every hosted view is
fully deallocated within its own test, never leaking into a sibling suite.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sirily11 sirily11 enabled auto-merge (squash) June 8, 2026 14:47
@sirily11 sirily11 merged commit 30e88bb into main Jun 8, 2026
14 checks passed
@sirily11 sirily11 deleted the workspace branch June 8, 2026 15:05
@sirily11

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.15.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant