Repository navigation
fix: separate source import quota usage - #50
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reachedNext included review available in 51 minutes. View limit detailsLimit details: You’ve used the included review currently available. Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (1)
ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (6)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe change scopes daily usage counters by quota key, updates ChangesDaily quota scoping
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant Client
participant handleRequest
participant consume_daily_quota
participant daily_usage_counters
Client->>handleRequest: Send authenticated import request
handleRequest->>consume_daily_quota: Request import_source reservation
consume_daily_quota->>daily_usage_counters: Atomically insert or increment counter
daily_usage_counters-->>consume_daily_quota: Return allowed status and count
consume_daily_quota-->>handleRequest: Return quota decision
handleRequest-->>Client: Return HTTP response
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Essentials by visiting https://app.coderabbit.ai/settings/billing. Comment |
|
@coderabbitai review |
|
Summary
quota_key, defaulting existing callers and rows toaiimport_sourcewithout changing AI or import quota valuesVerification
PATH=/home/codex/.nvm/versions/node/v22.23.2/bin:$PATH npm ciPATH=/home/codex/.nvm/versions/node/v22.23.2/bin:$PATH npm run test:edge(186 passed)PATH=/home/codex/.nvm/versions/node/v22.23.2/bin:$PATH npm run verify(1,061 Vitest, 186 Edge, build, 179 Chromium passed, 1 expected skip)git diff --checkNo staging or production migration/function deployment is included or authorized by this PR.
Summary by CodeRabbit
New Features
Bug Fixes
Tests