Skip to content

feat(github): support blobless clone and configurable clone timeout - #1870

Open
fy2ne wants to merge 2 commits into
getsentry:mainfrom
fy2ne:feat/github-clone-timeout
Open

fy2ne wants to merge 2 commits into
getsentry:mainfrom
fy2ne:feat/github-clone-timeout

Conversation

@fy2ne

@fy2ne fy2ne commented Sep 16, 2026

Copy link
Copy Markdown

Summary

Addresses #1547.

In \clone-repository.ts, \cloneRepository\ previously hardcoded a 120-second timeout (\2 * 60_000). For large repositories (e.g. \getsentry/static-sites, ~2.1 GB at HEAD) transferring across egress proxies, clones frequently hit 112–115s and trigger an abort coin flip.

This change:

  1. Adds \ imeoutSeconds\ (up to 900s, defaulting to 300s / 5 minutes) to \inputSchema\ so callers can configure the clone deadline for heavy repositories.
  2. Adds \�lobless\ boolean option (--filter=blob:none) to allow callers to drastically reduce transfer payloads on large repositories.
  3. Adds unit test coverage in \clone-repository.test.ts\ for both \ imeoutSeconds\ and \�lobless.

Fixes #1547

@vercel

vercel Bot commented Sep 16, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the Sentry Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the risk: low PR risk score: low label Sep 16, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5608eb2. Configure here.

Comment thread packages/junior-github/src/tools/clone-repository.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: low PR risk score: low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

junior-github: cloneRepository 120s timeout is hardcoded and discards the partial clone

1 participant