diff options
| author | Peter Stone <thepeterstone@gmail.com> | 2026-03-26 06:36:32 +0000 |
|---|---|---|
| committer | Peter Stone <thepeterstone@gmail.com> | 2026-03-26 06:36:32 +0000 |
| commit | 44521cc50e304b61c44b9a269a8239fd0fef49cd (patch) | |
| tree | f4c09def154fce8de6803ad376aca6676bebc05e /internal/api/stories_test.go | |
| parent | ba6a83ae5a62c3e93d7a119c5d8e6690bee7c099 (diff) | |
fix: story branch push to bare repo; drain at 3 consecutive failures
createStoryBranch was pushing to 'origin' which doesn't exist — branches
never landed in the bare repo so agents couldn't clone them. Now uses
the project's RemoteURL (bare repo path) directly for fetch and push.
Raise drain threshold from 2 to 3 consecutive failures to reduce false
positives from transient errors.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'internal/api/stories_test.go')
| -rw-r--r-- | internal/api/stories_test.go | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/internal/api/stories_test.go b/internal/api/stories_test.go index 53d15eb..342840b 100644 --- a/internal/api/stories_test.go +++ b/internal/api/stories_test.go @@ -212,7 +212,8 @@ func TestHandleStoryApprove_SetsRepositoryURL(t *testing.T) { ID: "proj-repo", Name: "claudomator", RemoteURL: "/site/git.terst.org/repos/claudomator.git", - LocalPath: "/workspace/claudomator", + // LocalPath intentionally empty: branch creation is a non-fatal side effect, + // omitting it keeps the test fast and free of real git operations. } if err := store.CreateProject(proj); err != nil { t.Fatalf("CreateProject: %v", err) |