Skip to content

feat(cli): add --ri-account-scope flag - #2286

Merged
cristim merged 2 commits into
mainfrom
feat/2181-ri-account-scope-v2
Oct 20, 2026
Merged

cristim merged 2 commits into
mainfrom
feat/2181-ri-account-scope-v2

Conversation

@cristim

@cristim cristim commented Oct 20, 2026

Copy link
Copy Markdown
Member

Summary

Adds the --ri-account-scope flag to the CLI (rebuilt on main after the dependency bump merged).

Supersedes the stacked branch feat/2181-ri-account-scope, now that its base (#2284) is squash-merged.

Verification

go vet, go test -count=1 ./..., golangci-lint and scripts/test-git-secrets-allowlist.sh all pass.

Closes #2181

🤖 Generated with Claude Code

Add --ri-account-scope (linked default, payer) mirroring --sp-account-scope,
validated the same way and passed into RecommendationParams.RIAccountScope.

Closes #2181

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

  • Run on-demand review

This review includes 6 billable files and costs up to $1.50.

  • Ask an admin to make reviews automatic

Open in CodeRabbit

Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing.

Or wait 32 minutes for your next included review.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 87 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: LeanerCloud/cloud-commitments-cli/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: a81fc155-3eb8-4e24-bee1-6eb18910c045
📥 Commits

Reviewing files that changed from the base of the PR and between ce16689 and 9d6c193.

📒 Files selected for processing: 6
  • CHANGELOG.md
  • cmd/main.go
  • cmd/multi_service_helpers.go
  • cmd/ri_account_scope_test.go
  • cmd/validators.go
  • docs/cli/README.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@cristim

cristim commented Oct 20, 2026

Copy link
Copy Markdown
Member Author

Merge gate verdict (claude-cc-libs)

Every gate holds at head 9d6c193:

  • Two independent reviews approve the reviewed head b0b6f56. The branch was then updated from main, and the PR diff is byte-identical to the reviewed diff (compared mechanically).
  • CI is green on 9d6c193, which is the reviewed change on top of current main.
  • 0 unresolved threads; MERGEABLE/CLEAN.

CodeRabbit is waived as a merge gate under the 2026-10-19 decision. Squash-merging.

@cristim
cristim merged commit ab05b6a into main Oct 20, 2026
12 checks passed
@cristim
cristim deleted the feat/2181-ri-account-scope-v2 branch October 20, 2026 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(ri): consider PAYER account scope for RI recommendations

1 participant