You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
sdk(ts): treat 429 as retryable to match the Go client #450
So a rate-limited request auto-retries (honoring Retry-After, capped at 30s) in Go and fails immediately in TypeScript. The Go behavior was added during PR #434's review rounds; the TS client was never updated to match.
Go is the better behavior here — 429 is the canonical "back off and try again" response, and Retry-After is most often set on exactly that status. Bringing TS in line keeps the cross-SDK parity claim honest.
Fix: add res.status === 429 to the TS retryable predicate, confirm the TS retry path honors Retry-After with the same 30s clamp the Go client uses (clients/go/http.go:20), and add a wire-conformance case (clients/go/testdata/wire_cases.json) so the two clients can't drift again.
Found during the PR #434 pre-push docs review. docs/src/content/docs/sdk/go/reference.md documents the Go 429 row as retryable; the TS reference table should stay as-is until the code changes.
Activity
coderabbitai commentedon Aug 11, 2026
🔗 Related PRs
#174 - refactor(table names): handle unsafe table names [closed]
#434 - feat(sdk): add Go client SDK with full API-tree parity [open]
#448 - fix(sdk): keep a baseURL path prefix instead of discarding it [open]
🧪 Issue enrichment is currently in open beta.
You can configure auto-planning by selecting labels in the issue_enrichment configuration.
To disable automatic issue enrichment, add the following to your
.coderabbit.yaml:💬 Have feedback or questions? Drop into our discord!
docs: address pre-push review round 12 — E2E lifecycle, lint install,…