Repository navigation
feat(workspaces): admins see every workspace, open one only by joining (D6) - #495
Merged
Merged
Conversation
Contributor
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: be5beecc-3caa-47ca-9406-7445b20867c4) |
Contributor
Test IDsAdded (10)
|
huyplb
force-pushed
the
feat/workspace-invites
branch
from
October 10, 2026 04:32
8497bc3 to
cd337da
Compare
…ning it D6 of the workspaces plan. - GET /api/workspaces/all (admins): every workspace, private ones included, with owners and size — never what it holds. - Admin → Access control → Workspaces: Join (the admin appears in the member list, added by itself), Open once in it, Members (hand ownership on), Archive shared workspaces, show archived. - Nothing changes for acting in a workspace: admins, like everyone, must be members of the workspace a request acts in. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
huyplb
force-pushed
the
feat/workspaces-admin
branch
from
October 10, 2026 04:36
64361a5 to
922cc7d
Compare
Contributor
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: 9373b424-4a29-452e-b197-78de4fd4d84b) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
D6 of the workspaces plan. Stacked on #494 (D4) → #493 → #492 → #491.
What changes
GET /api/workspaces/all(admins only): every workspace, private ones included: name, visibility, owners, member count, personal owner, archived, and whether the asking admin is in it. Never its connections, history or secrets. Contract table 133 → 134.X-Fox-Workspace/select for a workspace it is not in answers 404.Tests
AdminWorkspacesPanel.test.tsx(5): list text, Join only (no Open) before joining with the "members will see you" confirm, Open switches, archive shared only, show archived. A/B: offering Open without joining fails.workspaces.test.ts7/7 against the dev server: the admin finds a throwaway account's personal workspace in the tab with Join and no Open, joins, and the account sees the admin in its member list; the admin then leaves.Review
Fixed: a cross-feature import of the workspaces API's internals (architecture test). Left: deactivated accounts' workspaces aren't marked in the list; Join always makes the admin an owner.
🤖 Generated with Claude Code
Note
Medium Risk
Expands admin visibility across all workspaces and adds join-as-owner flows that affect member lists; membership gates still block silent data access, but mis-clicks or policy gaps around admin
addMembercould have broader impact than normal user actions.Overview
Adds admin-wide workspace directory (D6): admins can see every workspace on the install (including private and personal) with metadata only—names, owners, member counts, archived state—not connections or secrets.
Backend: New
GET /api/workspaces/all(?archived=1optional) andlistAllon the workspace directory, restricted to app admins. Acting in a workspace still requires membership; select/access for non-member workspaces stays 404.Admin UI: New Workspaces tab under Access control lists all workspaces with Join (confirm warns members will see the admin), Open after joining, Members (reuses workspace settings, with optional
knownNamewhen the admin isn’t in their workspace list), Archive for shared workspaces, and Show archived.Tests: Directory unit tests,
AdminWorkspacesPanelcomponent tests, e2e coverage for admin join visibility, and HTTP contract bump (133 → 134 routes).Reviewed by Cursor Bugbot for commit 922cc7d. Bugbot is set up for automated code reviews on this repo. Configure here.