GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||
GeezerVison is showing github.com — fix(api): restore legacy endpoint compatibility by TheodoreSpeaks · Pull Request #6552 · simstudioai/sim Conversation
Contributor
Summary
Type of Change
Testing
Checklist
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
|
There was an error while loading. Please reload this page.
PR SummaryMedium Risk Overview
Public workflow log pagination accepts legacy cursors that only contain Reviewed by Cursor Bugbot for commit ad665ca. Bugbot is set up for automated code reviews on this repo. Configure here. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Greptile SummaryRestores three legacy API behaviors without changing their established authentication boundaries.
Confidence Score: 5/5The PR appears safe to merge, with no actionable changed-code defects identified. The restored defaults and response fields match established legacy contracts, rate-limit status retrieval fails softly, explicit modern cursor order binding remains enforced, and focused tests cover the compatibility paths.
|
| Filename | Overview |
|---|---|
| apps/sim/app/api/table/[tableId]/groups/route.ts | Restores the legacy omitted-value default for the one-shot group auto-run input while preserving explicit false. |
| apps/sim/app/api/users/me/usage-limits/route.ts | Restores synchronous and asynchronous rate-limit status fields after authentication and validates the assembled legacy response. |
| apps/sim/lib/api/contracts/usage-limits.ts | Replaces the permissive usage-limits response contract with a strict schema matching the restored endpoint and SDK shape. |
| apps/sim/lib/logs/public-queries.ts | Accepts orderless legacy cursors by binding them to the request order while retaining mismatch rejection for modern order-bound cursors. |
Reviews (1): Last reviewed commit: "fix(api): restore legacy endpoint compat..." | Re-trigger Greptile
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
ad665cae
Deployed
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.