feat(framework): Queue "Needs you" list — open PRs across projects (#632) - #633
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #632. Part of #624 (the Queue).
The first slice of Queue A — the cross-project "Needs you" list. Rom's design (#624): proposals and finished work are both just PRs, so the bulk of "needs a human" is the set of open PRs across the registered projects, merge to confirm / close to reject.
Server
buildInterventions(projects)— rolls up every registered project's open, non-draft PRs viagh pr list, newest first. Deduped by PR url (a monorepo registered at root + a subdir would otherwise list the same PR twice). Degrades to empty when there's no remote / gh is missing / on the relay (no checkout). Injectable PR lister, unit-tested off disk.onInterventions()read RPC, wired likeonOverview.Dashboard (client, bundled)
ghpoll (slow cadence — PRs change rarely).Verification
onInterventionsreturned the real open PR chore: version packages #285 once (draft spike(#610): WebDriver PoC on the routines fire API #623 correctly excluded).Next (follow-ups on #624): the second Queue-A source — a run paused at an await gate ("awaiting your answer") — then #627 notifications firing off this same set.