Skip to content

docs: scope the vault guarantee to the gateway and state the standalone limit - #10

Merged
asafgolombek merged 1 commit into
mainfrom
docs/review-followup
Nov 12, 2026
Merged

asafgolombek merged 1 commit into
mainfrom
docs/review-followup

Conversation

@asafgolombek

Copy link
Copy Markdown
Contributor

Follow-up to #9, which merged before its one review thread was read. The reviewer was right that two statements disagreed: the non-negotiables were said to hold across every repository, while the connectors were listed as runnable standalone, where they take credentials from the environment and have no vault, sandbox or ledger.

The non-negotiables are now scoped to the gateway and what runs through it, the standalone limit is stated beside the connectors' row in both files, and the profile's credentials bullet says in the gateway.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JmSjrFKyPDdZkezWFQzqTs

…ne limit

Addresses the review thread left open when the previous pull request merged.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JmSjrFKyPDdZkezWFQzqTs
@coderabbitai

coderabbitai Bot commented Nov 12, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

Next included review available in 53 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 61 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

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Essentials
  • Run ID: f34348e1-42b8-4493-a61c-a29edad13dd0
Commits

Reviewing files that changed from the base of the PR and between eb3ec47 and 0b674e1.

Files selected for processing: 2

  • ECOSYSTEM.md
  • profile/README.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

@sonarqubecloud

Copy link
Copy Markdown

@asafgolombek
asafgolombek merged commit 6e2fdfe into main Nov 12, 2026
3 checks passed
@asafgolombek
asafgolombek deleted the docs/review-followup branch November 12, 2026 19:30
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.

1 participant