Skip to content

fix: ensure fetch succeeds before branch setup#119

Merged
ejc3 merged 1 commit intoclaude-review-prompting-fixfrom
claude/fix-20727916151
Jan 5, 2026
Merged

fix: ensure fetch succeeds before branch setup#119
ejc3 merged 1 commit intoclaude-review-prompting-fixfrom
claude/fix-20727916151

Conversation

@claude-claude
Copy link
Copy Markdown
Contributor

@claude-claude claude-claude bot commented Jan 5, 2026

Auto-Fix for PR #110

Issues Fixed

  • [MEDIUM] Silent fetch failure - Changed git("fetch", ...) to gitOrFail("fetch", ...) to ensure the fetch operation succeeds before attempting to checkout/reset branches

Changes

Modified line 125 in scripts/claude-assistant/index.ts to use gitOrFail instead of git for the fetch operation. This prevents:

  • Silently using stale remote refs if fetch fails
  • Confusing error messages from subsequent checkout/reset operations
  • Reviewing outdated code due to stale remote branches

Generated by Claude | Review Run

Changes git("fetch") to gitOrFail("fetch") to prevent silent failures
that could lead to reviewing stale code or unclear errors during branch
checkout/reset operations.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@ejc3 ejc3 merged commit 932e8ef into claude-review-prompting-fix Jan 5, 2026
7 checks passed
@ejc3 ejc3 deleted the claude/fix-20727916151 branch January 17, 2026 03:36
ejc3 added a commit that referenced this pull request Mar 2, 2026
fix: ensure fetch succeeds before branch setup
ejc3 added a commit that referenced this pull request Mar 2, 2026
fix: ensure fetch succeeds before branch setup
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