Skip to content

Use git-harness v0.3.1 for non-interactive network git - #19

Merged
bschellenberger2600 merged 3 commits into
mainfrom
fix/non-interactive-git-network
Jun 16, 2026
Merged

Use git-harness v0.3.1 for non-interactive network git#19
bschellenberger2600 merged 3 commits into
mainfrom
fix/non-interactive-git-network

Conversation

@bschellenberger2600

Copy link
Copy Markdown
Member

Summary

  • Pin github.com/git-fire/git-harness v0.3.1 and delegate PrepareNetworkGit to harness
  • Apply non-interactive fetch behavior on default fetch --all, mainline fetch, and full sync paths
  • Migrate .goreleaser.stable.yaml from deprecated brewshomebrew_casks
  • Fix release workflow gate to verify Casks/git-rain.rb (not Formula/) after Goreleaser publish

Why

Parallel git-rain runs were leaking HTTPS credential prompts when remotes lacked cached credentials. The fix belongs in git-harness (shared with git-fire); git-rain now consumes it.

Supersedes #16 (reverted in #18 after premature merge while Bugbot Autofix was in flight).

Related PRs

Test plan

  • go test -race -count=1 ./...
  • CI green (including Validate GoReleaser Configs)
  • Bugbot review clean
  • Human review before merge

Made with Cursor

bschellenberger2600 and others added 3 commits June 16, 2026 00:00
Apply GIT_TERMINAL_PROMPT=0 to all network fetch paths so parallel git-rain
runs fail fast with a frozen auth message instead of interleaved TTY prompts.

Co-authored-by: Cursor <cursoragent@cursor.com>
…aser CI.

Delegate PrepareNetworkGit to git-harness instead of duplicating the helper,
pin github.com/git-fire/git-harness v0.3.1, and migrate stable releases from
deprecated brews to homebrew_casks so release config validation passes.

Co-authored-by: Cursor <cursoragent@cursor.com>
GoReleaser stable config publishes homebrew_casks under Casks/, so the
release workflow must wait on Casks/git-rain.rb instead of Formula/.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@bschellenberger2600, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 28 minutes and 55 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5e6da329-322f-440e-8825-7f2d1dc5ae5a

📥 Commits

Reviewing files that changed from the base of the PR and between 96af267 and d4b3b77.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (9)
  • .github/workflows/release.yml
  • .goreleaser.stable.yaml
  • cmd/root.go
  • cmd/root_test.go
  • go.mod
  • internal/git/command.go
  • internal/git/command_test.go
  • internal/git/fetch_mainline.go
  • internal/git/rain.go
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/non-interactive-git-network

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@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.

Did not approve. Cursor Bugbot check passed but no BUGBOT_REVIEW comment was posted, so the required automated review signal is missing. Human review is still needed before merge; no reviewers assigned (solo maintainer repo).

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Approver

@bschellenberger2600
bschellenberger2600 merged commit 241c851 into main Jun 16, 2026
12 checks passed
bschellenberger2600 added a commit that referenced this pull request Jun 16, 2026
Follow-up to #19: remove duplicated operations helpers and internal/safety in favor of git-harness v0.3.1.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

1 participant