Skip to content

fix(plugin_source): give up on a git clone that outlasts its timeout - #98

Merged
marcossevilla merged 2 commits into
mainfrom
fix/clone-timeout
Oct 28, 2026
Merged

marcossevilla merged 2 commits into
mainfrom
fix/clone-timeout

Conversation

@ryzizub

@ryzizub ryzizub commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Description

Stack 1 of 5 for #68. Base: main.

The clone in GitPluginSource had no timeout, so an unreachable github.com left install and doctor waiting forever. HarnessDetector already bounds its own probe at five seconds.

The git process is not killed, but nothing waits on it and the half-filled temp directory is thrown away, so the next run starts clean. The source also takes an optional log sink, which carries the command line and git's stderr to --verbose; nothing passes it yet, that comes in stack 2.

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore
  • 🧪 Test

🤖 Generated with Claude Code

@ryzizub
ryzizub added this pull request to stack #103 October 21, 2026 12:19
@ryzizub
ryzizub marked this pull request as ready for review October 21, 2026 12:20
marcossevilla
marcossevilla previously approved these changes Oct 21, 2026

@marcossevilla marcossevilla left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit, but lgtm!

The clone had no timeout, so an unreachable github.com left install and doctor
waiting forever. HarnessDetector already bounds its own probe at five seconds.

The git process is not killed, but nothing waits on it and the half-filled
temp directory is thrown away, so the next run starts clean. GitPluginSource
also takes an optional log sink, which carries the command line and git's
stderr to --verbose.
@marcossevilla
marcossevilla merged commit 6456825 into main Oct 28, 2026
8 checks passed
@marcossevilla
marcossevilla deleted the fix/clone-timeout branch October 28, 2026 11:55
@vgvbot vgvbot mentioned this pull request Oct 29, 2026
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