Gnome toolbar for Tangled code review.

fix(verdicts)!: show PRs still being checked as no verdict, not grey "pending" #28

Merged permadeath.com
targeting main from azurite-frogmouth.test.pds.did.bot/gnome-shell-extension-tangled-reviewer:claude/verdict-states
Merged opened by and assigned to permadeath.com
targeting main from azurite-frogmouth.test.pds.did.bot/gnome-shell-extension-tangled-reviewer:claude/verdict-states

A PR whose checks haven't answered yet no longer counts as grey "pending". That's why the panel showed 30 grey while loading, then 21 green, 5 grey and 4 amber. Grey now means only waiting on CI: a run in progress, or one awaiting approval. Two new verdicts cover the rest. checking is a merge check or CI lookup that hasn't answered yet. unknown means there's no telling: no revision on Tangled, a failed lookup, cancelled CI, or nothing that passed. Both show in an outlined pill with no color, as do the not-set-up, loading and error marks. In the menu, checking shows "…" and unknown shows "?". A known problem still wins while other checks run: failing CI or a conflict shows straight away.

Before and after. The fixture now also has one PR still checking and one with no revision.

Panel before Panel after

Menu before Menu after

This adds two values to the verdict that the helper sends the panel (Verdict in types/protocol.d.ts), so it changes a state the extension keeps; please review it as such. The protocol-mode test (tests/driver/protocol.test.js) reproduces the load you saw. Its in-between states now read "checking" for every PR, and the one with no revision ends as "unknown". The rule's tests cover each case, and 80 tests pass.

🤖 Generated with Claude Code

Commits 0