Skip to content

ref: Add backoff to summary fetch retries#1965

Merged
Aaron (btaaron) merged 8 commits into
mainfrom
aaron-summary-retry-backoff
May 20, 2026
Merged

ref: Add backoff to summary fetch retries#1965
Aaron (btaaron) merged 8 commits into
mainfrom
aaron-summary-retry-backoff

Conversation

@btaaron
Copy link
Copy Markdown
Contributor

@btaaron Aaron (btaaron) commented May 7, 2026

Summary

Adds exponential backoff between existing JS SDK get_json retries to help post run experiment.summarize() calls to /experiment-comparison2, where immediate retries can repeat during the same transient 504/timeout window. Keeps existing retry count and only spaces out retries for callers already opting into get_json retries

Test Plan

  • pnpm run build
  • pnpm exec vitest run src/logger.test.ts
  • pnpm exec vitest run tests/auto-instrumentations/loader-hook.test.ts
  • pnpm run test
  • pnpm run check:typings

CI note

integration-tests/e2e and e2e-canary are failing on the Copilot/canary e2e suite, which is also failing on main; JS checks for this PR are green

@btaaron Aaron (btaaron) enabled auto-merge (squash) May 7, 2026 20:47
@btaaron Aaron (btaaron) self-assigned this May 7, 2026
@btaaron Aaron (btaaron) changed the title Add backoff to summary fetch retries Back off summary retries May 8, 2026
@btaaron Aaron (btaaron) changed the title Back off summary retries Add backoff to summary fetch retries May 8, 2026
@lforst Luca Forstner (lforst) changed the title Add backoff to summary fetch retries ref: Add backoff to summary fetch retries May 20, 2026
@btaaron Aaron (btaaron) merged commit cc4d7e7 into main May 20, 2026
31 checks passed
@btaaron Aaron (btaaron) deleted the aaron-summary-retry-backoff branch May 20, 2026 18:44
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