bug: runGit and ghFetchIssue have no timeout, can hang indefinitely

[claude]

Bug

runGit (internal/project/project.go:79-87) and ghFetchIssue (internal/project/workingmemory.go:128-147) execute shell commands with no timeout:

// internal/project/project.go:79-87
func runGit(dir string, args ...string) (string, error) {
    cmd := exec.Command("git", args...)
    cmd.Dir = dir
    out, err := cmd.Output()   // blocks forever if git hangs
    ...
}

// internal/project/workingmemory.go:128-131
func ghFetchIssue(wm *WorkingMemory, issueNumber int) {
    cmd := exec.Command("gh", "issue", "view", ...)
    out, err := cmd.Output()   // blocks forever if gh hangs
    ...
}

GetWorkingMemory is called synchronously in runHandler (before the API fetch) and pregenHandler. Git commands can block indefinitely on network operations (e.g. git remote get-url with a slow/broken network, or git log on a pathological repo). gh issue view can also block waiting for the GitHub API. A hung git/gh call silently hangs the entire uncompact run hook.

Fix

Pass a context.Context with a short deadline (e.g. 5 s) to both helpers via exec.CommandContext, so they are cancelled if the operation takes too long. The callers (GetWorkingMemory, Detect) should accept and thread a context through.

Affected files:

• internal/project/project.go lines 79-87
• internal/project/workingmemory.go lines 128-147
• Callers: GetWorkingMemory, Detect, defaultBranch

@claude please implement this

[claude]

Claude finished @claude[bot]'s task in 3m 40s —— View job • claude/issue-47-20260227-0529 • [Create PR ➔](https://github.com/supermodeltools/Uncompact/compare/main...claude/issue-47-20260227-0529?quick_pull=1&title=Issue%20%2347%3A%20Changes%20from%20Claude&body=This%20PR%20addresses%20issue%20%2347%0A%0AGenerated%20with%20%5BClaude%20Code%5D(https%3A%2F%2Fclaude.ai%2Fcode)

---

Implementation Complete

• Read affected files to understand current structure
• Modify runGit to use exec.CommandContext with timeout
• Modify ghFetchIssue and defaultBranch to accept and thread context
• Update GetWorkingMemory and Detect to accept context
• Update callers in cmd/run.go, cmd/pregen.go, cmd/status.go
• Verify build passes (go build ./... and go vet ./... — clean)
• Commit and push changes
• Create PR

Changes made

internal/project/project.go — runGit now takes context.Context as its first argument and uses exec.CommandContext; Detect signature updated to accept and thread context.

internal/project/workingmemory.go — ghFetchIssue, defaultBranch, and GetWorkingMemory all updated to accept and thread context.Context; ghFetchIssue uses exec.CommandContext.

cmd/run.go, cmd/pregen.go, cmd/status.go — each call site creates a context.WithTimeout(context.Background(), 5*time.Second) before calling project.Detect / project.GetWorkingMemory, ensuring git and gh subprocesses are killed if they don't complete within 5 seconds.

Pull request #51

[coderabbitai]

🔗 Related PRs

#18 - feat: show circular dependency warning in context bomb [merged]

---

📝 Issue Planner

_{Check the box below or use the @coderabbitai plan command to generate an implementation plan and prompts that you can use with your favorite coding assistant.}

• Create Plan

---

🧪 Issue enrichment is currently in open beta.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!