bug: ghFetchIssue logs a debug error when gh CLI is not installed, causing TestGhFetchIssue_GhUnavailable to fail

[claude]

The function ghFetchIssue in internal/project/workingmemory.go calls logFn unconditionally on any error from cmd.Output(), including the case where the gh binary is simply not installed.

Problem

// workingmemory.go:162
if err != nil {
    logFn("[debug] gh issue fetch failed (issue #%d): %v", issueNumber, err)
    return
}

When gh is not in $PATH, cmd.Output() returns an *exec.Error wrapping exec.ErrNotFound. The code logs this at debug level, which triggers the logFn callback.

The existing test TestGhFetchIssue_GhUnavailable in internal/project/workingmemory_test.go:115 explicitly asserts that logFn is not called when gh is unavailable:

// workingmemory_test.go:133
if warned {
    t.Error("logFn should not be called when gh is unavailable")
}

Since the code does call logFn, this test fails.

The rationale in the test comment is correct: gh is an optional dependency; its absence should be silently ignored rather than generating noise in debug logs.

Fix

Check for exec.ErrNotFound before logging, and silently return if the binary is simply missing:

if err != nil {
    var execErr *exec.Error
    if errors.As(err, &execErr) && errors.Is(execErr.Err, exec.ErrNotFound) {
        return // gh not installed — silently skip
    }
    logFn("[debug] gh issue fetch failed (issue #%d): %v", issueNumber, err)
    return
}

Location

• internal/project/workingmemory.go:162 (the unconditional logFn call)
• internal/project/workingmemory_test.go:133 (the assertion that is currently violated)

@claude please implement this

[claude]

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

---

Tasks

• Gather context and understand the request
• Read internal/project/workingmemory.go to understand the current code
• Apply the fix to skip logging when gh is not installed
• Verify the build passes (go build ./...)
• Commit and push changes
• Create PR

Summary

Fixed ghFetchIssue in internal/project/workingmemory.go to silently return when gh is not installed, rather than logging a debug message.

Changes:

• Added "errors" import
• Before calling logFn, check if the error is *exec.Error wrapping exec.ErrNotFound; if so, return without logging

This makes TestGhFetchIssue_GhUnavailable pass, since gh is an optional dependency and its absence should be silent.

PR: #481