Repository navigation
Conversation
Move `cloud/organizations.rs` to `cloud/organizations/mod.rs` unchanged, so blame follows the file into the new layout. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
#1157) Move the `org`, `member` and `invitation` command code out of `cloud/organizations/mod.rs` into one file per command family, each with the tests for its code: - `cli.rs`: clap definitions, write classification and parse tests - `client.rs`: `CloudClient` wrappers, in one `impl` block - `org.rs`: org list, get, update, quota, balance, notifications, notification types, prometheus and usage, with the private endpoint removal parser - `byoc.rs`: `org byoc` and its request builders - `roles.rs`: `org role` and its request builders - `members.rs`: `member` and `invitation` `mod.rs` keeps `PERMISSIONS`, the `run_*` dispatch and re-exports of the command enums. Only visibility and imports change; no behaviour change. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…1156) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
sdairs
force-pushed
the
claude/1157-split-cloud-organizations
branch
from
October 30, 2026 17:27
b2e9464 to
5a63f7d
Compare
This branch has not been deployed
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.
Part of #1157 (3 of 5:
organizations). Stacked on #1170.Summary
This splits
src/cloud/organizations.rsintosrc/cloud/organizations/. The seams follow the three top-level commands it dispatches (org,member,invitation). Behaviour is unchanged.mod.rsPERMISSIONS,run_*dispatch, re-exportscli.rsis_writeclient.rsimpl CloudClientorg.rsbyoc.rsroles.rsmembers.rsjoin_absentCommits
Pure
git mv, 100% rename.Moves only. The multiset check is clean: the only differences are
mod/uselines, test-module wrappers, and six rustfmt signature rewraps afterpub(super)was added..git-blame-ignore-revsentry. The SHA needs updating on rebase.Follow-ups from the stack review (Route Cloud Postgres handlers through CloudClient wrappers #1156, kept here so the split SHAs below don't change):
services/query_key.rsis fixed.Tests
--no-default-featurescheck, the Python classifier tests andgit diff --checkall pass.cargo docwarnings are unchanged from the base branch.Remaining for #1157
clickpipesandclickstackwait until the open drift stack (#1149–#1152) merges, because it edits both files.Merge with a merge commit, not a squash. The
.git-blame-ignore-revsentry names this PR's move commit. A squash merge would leave it pointing at a commit that isn't on main, and git would silently ignore it.🤖 Generated with Claude Code
Note
Low Risk
Mechanical file split with no intended runtime or API changes; only comment and blame-metadata updates outside the move.
Overview
Replaces the monolithic
cloud/organizations.rswith acloud/organizations/directory, grouped by command family (cli,client,org,byoc,roles,members) withmod.rsholding PERMISSIONS and therun_org/run_member/run_invitationdispatch. Public CLI surface and behavior stay the same; code is mostly moved and visibility adjusted (pub(super)), with tests split across the new modules.Also adds a
.git-blame-ignore-revsentry for the split commit and updates a stale comment inservices/query_key.rsto referenceservices/query.rsinstead ofservices.rs.Reviewed by Cursor Bugbot for commit 5a63f7d. Bugbot is set up for automated code reviews on this repo. Configure here.