Skip to content

fix: increase API request cleanup batch size to 10000 - #5823

Merged
chrarnoldus merged 1 commit into
mainfrom
increase-api-request-cleanup-batch-size
Sep 2, 2026
Merged

fix: increase API request cleanup batch size to 10000#5823
chrarnoldus merged 1 commit into
mainfrom
increase-api-request-cleanup-batch-size

Conversation

@chrarnoldus

@chrarnoldus chrarnoldus commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Increase the API request log cleanup batch size from 1,000 to 10,000 rows per request.
  • Update the existing cleanup tests to expect the new limit; retain seven-day retention and single-batch behavior.

Validation

  • All applicable CI checks passed, including tests, typecheck, lint, formatting, migration checks, builds, and extension E2E tests.
  • Automated Kilo Code Review completed with no issues found.
  • Tests ran only in CI as requested.
  • Local targeted formatting and git diff --check passed. Local focused lint and web typecheck exceeded the environment’s two-minute limit; both passed in CI.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus chrarnoldus self-assigned this Sep 1, 2026
@kilo-code-bot

kilo-code-bot Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/app/api/cron/cleanup-api-request-log/route.ts
  • apps/web/src/app/api/cron/cleanup-api-request-log/route.test.ts

Reviewed by grok-4.6 · Input: 99.1K · Output: 6K · Cached: 122.6K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus merged commit b7eebeb into main Sep 2, 2026
17 checks passed
@chrarnoldus
chrarnoldus deleted the increase-api-request-cleanup-batch-size branch September 2, 2026 09:40
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