Skip to content

fix(types): add optional delta to RateLimitOptions - #8144

Open
jeffa5 wants to merge 1 commit into
cloudflare:mainfrom
jeffa5:ajeffery/ratelimit-delta-types
Open

jeffa5 wants to merge 1 commit into
cloudflare:mainfrom
jeffa5:ajeffery/ratelimit-delta-types

Conversation

@jeffa5

@jeffa5 jeffa5 commented Nov 2, 2026 •

Copy link
Copy Markdown

The Rate Limiting binding accepts an optional delta in limit() to add a weighted amount to the counter instead of 1, but RateLimitOptions only declares key. This adds delta?: number with a JSDoc comment, updates the generated snapshots, and adds a type test.

Local dev support is in cloudflare/workers-sdk#16650.

🤖 Generated with Claude Code

The Rate Limiting binding accepts an optional `delta` in `limit()` to add
a weighted amount to the counter instead of 1. Expose it in the types and
add a type test.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@jeffa5
jeffa5 requested review from a team as code owners November 2, 2026 11:05
@github-actions

github-actions Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@clanky-the-clanker

Copy link
Copy Markdown

This pull request was not reviewed automatically because its author does not have write access to this repository. A maintainer with write access can start a review by commenting clanky review (or clanker review or @clanky-the-clanker review), optionally followed by instructions (e.g. clanky review focus on the retry logic).

@jeffa5

jeffa5 commented Nov 2, 2026

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@petebacondarwin
petebacondarwin removed the request for review from dario-piotrowicz November 2, 2026 19:52
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.

2 participants