docs(edge): /api/stats states its own scope — per-isolate key availability (LAB-1618) - #14
docs(edge): /api/stats states its own scope — per-isolate key availability (LAB-1618)#1427Bslash6 wants to merge 4 commits into
Conversation
…bility, not SDK L1 (LAB-1618) The payload now carries a scope field saying counters are per-isolate and reset on isolate recycle, and that hit_rate is aggregate-key availability at this isolate — not an SDK L1 rate, not the end-user rate (POP cache hits are served before the worker runs). Dashboard tile and README copy match. Additive-only: existing field names unchanged (verify.sh, tests, dashboard).
This comment has been minimized.
This comment has been minimized.
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 20 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe ChangesStatistics scope
Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@edge/test/handler.test.ts`:
- Around line 187-198: Add assertions in the test case “scope names the reset
semantics and disclaims SDK-L1 / end-user readings” to verify body.scope
explicitly excludes the end-user rate and POP-cached readings, while preserving
the existing reset, aggregate-key availability, and SDK L1 assertions.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: e684a4dd-8651-4977-9178-575eab1dcf4b
📒 Files selected for processing (4)
edge/README.mdedge/public/index.htmledge/src/handler.tsedge/test/handler.test.ts
… contract The qa check failed on prettier column padding in the README API table after the longer /api/stats row landed. Also close the CodeRabbit gap: the scope test's description promised end-user/POP disclaimers its assertions never checked.
Kody Review CompleteGreat news! 🎉 Keep up the excellent work! 🚀 Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
|
@coderabbitai review |
|
# Conflicts: # edge/public/index.html
b5b260e
|
Resolved conflict in edge/public/index.html (took main's extracted dashboard.js structure and ported this PR's LAB-1618 tile copy into edge/public/dashboard.js + its test assertion); auto-rebased onto main; CI will re-run. |
Skyline slice of LAB-1618 (honest cache-layer reporting). Copy-only in
edge/per coordination with the LAB-1615 renderer rewrite./api/statspayload gains ascopefield stating its own semantics: per-isolate counters that reset on isolate recycle, andhit_rate= aggregate-key availability at this isolate — not an SDK L1 rate (the edge reads the backend directly; the publisher runs L1-disabled by design) and not the end-user rate (POP cache hits are served before the worker runs). Additive-only — existing field names untouched (verify.sh, dashboard, tests).Companion PRs: cachekit-io/saas#310 (analytics contract), docs (metric scopes).
Verification:
npm test(26 passed) +tsc --noEmitclean.