pkg/hub/server.go, line 4780-4784 (link)

sendNextQueuedMessage duplicates the busy-check inline instead of calling the new isBusyLocked() helper. If the predicate ever changes (e.g. a third field is added to the busy state), this copy would silently diverge.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

pkg/hub/pr_watcher.go, line 613-640 (link)

delivered flag and "delivered" log are misleading in the failure branch

delivered = true is set unconditionally in both the happy path and the error path (after re-queuing a failed write). The subsequent log.Printf("[pr-watcher] delivered injected message …") then fires even though the message was only prepended to the retry queue. During incident investigation, a log that says "delivered" when the message was actually queued-after-failure can send debugging in the wrong direction.