anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-07-22 02:05:26 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

[PR #13835] fix(github): add null check for headRepository when handling PRs #14832

New Issue
Closed
opened 2026-02-16 18:19:35 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 18:19:35 -05:00
Owner
Copy Link
Copy Source

Original Pull Request: https://github.com/anomalyco/opencode/pull/13835

State: closed
Merged: No

Summary

Fixes a crash that occurs when OpenCode is configured to respond to PRs from forks, where headRepository can be null or undefined.

The Bug

Error message:

null is not an object (evaluating 'prData.headRepository.nameWithOwner')

This occurs when:

  • OpenCode is configured on a private repo via GitHub Actions
  • A PR is opened from a fork
  • GitHub's API doesn't always populate headRepository for fork PRs

The crash happens immediately when trying to process the PR, making OpenCode unusable for fork workflows.

The Fix

Added null checks in two locations:

  1. Line 565 - When determining if PR is from local branch:
if (
  prData.headRepository &&
  prData.baseRepository &&
  prData.headRepository.nameWithOwner === prData.baseRepository.nameWithOwner
) {
  1. Line 1044 - When checking out fork branch:
if (!pr.headRepository) {
  throw new Error("PR headRepository is null - unable to checkout fork branch")
}

Impact

✅ Fork PRs no longer cause crashes
✅ Clear error message when headRepository is unavailable
✅ Backward compatible - existing behavior preserved for normal PRs

Test plan

  • Added null checks before accessing headRepository.nameWithOwner
  • Added null check for baseRepository as safety measure
  • Added explicit error with clear message in checkoutForkBranch
  • Minimal change - only adds safety checks

Fixes #13812

🤖 Generated with Claude Code

**Original Pull Request:** https://github.com/anomalyco/opencode/pull/13835 **State:** closed **Merged:** No --- ## Summary Fixes a crash that occurs when OpenCode is configured to respond to PRs from forks, where `headRepository` can be null or undefined. ## The Bug Error message: ``` null is not an object (evaluating 'prData.headRepository.nameWithOwner') ``` This occurs when: - OpenCode is configured on a private repo via GitHub Actions - A PR is opened from a fork - GitHub's API doesn't always populate `headRepository` for fork PRs The crash happens immediately when trying to process the PR, making OpenCode unusable for fork workflows. ## The Fix **Added null checks** in two locations: 1. **Line 565** - When determining if PR is from local branch: ```typescript if ( prData.headRepository && prData.baseRepository && prData.headRepository.nameWithOwner === prData.baseRepository.nameWithOwner ) { ``` 2. **Line 1044** - When checking out fork branch: ```typescript if (!pr.headRepository) { throw new Error("PR headRepository is null - unable to checkout fork branch") } ``` ## Impact ✅ Fork PRs no longer cause crashes ✅ Clear error message when headRepository is unavailable ✅ Backward compatible - existing behavior preserved for normal PRs ## Test plan - [x] Added null checks before accessing `headRepository.nameWithOwner` - [x] Added null check for `baseRepository` as safety measure - [x] Added explicit error with clear message in `checkoutForkBranch` - [x] Minimal change - only adds safety checks Fixes #13812 🤖 Generated with [Claude Code](https://claude.com/claude-code)
yindo added the pull-request label 2026-02-16 18:19:35 -05:00
yindo closed this issue 2026-02-16 18:19:35 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
acp
bad
bug
discussion
docs
documentation
enhancement
extension
good first issue
growth
help-wanted
mcp
model-problem
needs:compliance
nix
opentui
perf
pull-request
Mirrored from GitHub Pull Request
 tui
web
windows
zen
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#14832
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?