Skip to content

Remove unused rate limiting config from providers#189

Merged
jeroenrinzema merged 6 commits intomainfrom
copilot/remove-rate-limiting-configurations
Mar 6, 2026
Merged

Remove unused rate limiting config from providers#189
jeroenrinzema merged 6 commits intomainfrom
copilot/remove-rate-limiting-configurations

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 6, 2026

Provider rate limiting (rate_limit, rate_interval) is not used. Remove it from the API spec, backend, and UI.

OAPI Spec & Generated Code

  • Removed rate_limit and rate_interval fields from Provider, CreateProvider, UpdateProvider schemas in resources.yml
  • Updated resources_gen.go — removed ProviderRateInterval, CreateProviderRateInterval, UpdateProviderRateInterval types and constants
  • Updated management.generated.ts — removed corresponding TypeScript fields

Backend

  • Removed RateLimit/RateInterval from Provider and ProviderUpdate structs
  • Stripped from all provider SQL queries (SELECT, INSERT, UPDATE) across providers.go and organizations.go
  • Removed rate limit handling in create/update controller handlers

UI

  • Removed rate limit and rate interval form fields from IntegrationModal.tsx
  • Cleaned up unused Select component imports
  • Removed rate_limit/rate_interval from TypeScript Provider interface and ProviderCreateParams
  • Removed provider-specific rate limit locale entries from en.json

Not touched

  • Balancer journey step rate limiting — separate feature, still in use
  • DB columns — left in place; no destructive migration
  • Top-level locale keys rate_limit/rate_interval — still used by Balancer step
Original prompt

Please remove the rate limiting configurations from the wasm modules they are not used at the moment. Please remove them from the ui as well if they are hard coded. Please check if rate limiting is referenced anywhere else.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits March 6, 2026 10:46
- Remove rate_limit and rate_interval from Provider, CreateProvider, and UpdateProvider schemas in resources.yml
- Update generated Go code (resources_gen.go) to remove rate limiting types and constants
- Remove RateLimit/RateInterval from Provider and ProviderUpdate structs in Go store
- Remove rate_limit/rate_interval from all provider SQL queries
- Remove rate_limit/rate_interval handling in Go controller (create/update)
- Remove rate_limit/rate_interval UI fields from IntegrationModal.tsx
- Remove rate_limit/rate_interval from TypeScript Provider type and ProviderCreateParams
- Remove provider-specific rate_limit locale entries from en.json settings section
- Update generated TypeScript (management.generated.ts) to remove provider rate limiting
- Keep Balancer journey step rate limiting (separate feature, still in use)

Co-authored-by: jeroenrinzema <3440116+jeroenrinzema@users.noreply.github.com>
Co-authored-by: jeroenrinzema <3440116+jeroenrinzema@users.noreply.github.com>
Co-authored-by: jeroenrinzema <3440116+jeroenrinzema@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove rate limiting configurations from wasm modules and UI Remove unused rate limiting config from providers Mar 6, 2026
@jeroenrinzema jeroenrinzema marked this pull request as ready for review March 6, 2026 12:48
@jeroenrinzema jeroenrinzema enabled auto-merge (squash) March 6, 2026 12:48
@jeroenrinzema jeroenrinzema disabled auto-merge March 6, 2026 12:48
@jeroenrinzema jeroenrinzema merged commit 496688b into main Mar 6, 2026
4 checks passed
@jeroenrinzema jeroenrinzema deleted the copilot/remove-rate-limiting-configurations branch March 6, 2026 12:48
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