Skip to content

fix: show UNKNOWN in status when a PR's live state cannot be fetched - #120

Merged
vitali87 merged 4 commits into
mainfrom
fix/status-unverified-state
Aug 31, 2026
Merged

fix: show UNKNOWN in status when a PR's live state cannot be fetched#120
vitali87 merged 4 commits into
mainfrom
fix/status-unverified-state

Conversation

@vitali87

Copy link
Copy Markdown
Owner

Summary

status fell back to the recorded PRRecord.state whenever the live gh pr view lookup returned nothing. That recorded state is never written back after merge/clean, so it is always open — a merged, closed, or deleted PR (or any run with gh unauthenticated) was shown as a confident OPEN with no hint that the fetch had failed.

Now an unfetchable PR renders UNKNOWN and a yellow line after the table lists the affected PR numbers with a pointer to gh auth status. README's status paragraph documents this.

Test plan

  • test_unfetchable_state_is_unknown_not_open (one PR {}, one MERGED): UNKNOWN and MERGED shown, no OPEN, warning names #7 — fails on main with OPEN
  • test_live_state_and_review_are_shown: normal path unchanged, no warning
  • 446 tests pass, ruff clean
  • Local review: 5/5

@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 46 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 50eb59d9-7b90-4146-8987-37a7f0dab12f

📥 Commits

Reviewing files that changed from the base of the PR and between 7878a8d and 40367e4.

📒 Files selected for processing (3)
  • README.md
  • pr_split/cli.py
  • tests/test_cli_coverage.py

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.

@vitali87
vitali87 merged commit ec2f754 into main Aug 31, 2026
4 checks passed
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