feat(gastown): build 'Needs Your Attention' section for Merge Queue page#1365
Merged
jrf0110 merged 5 commits intoconvoy/redesign-merge-queue-needs-attention-ref/e0d20455/headfrom Mar 21, 2026
Conversation
…1299) * feat(gastown): configurable terminal orientation and drag-to-resize (#1237) Terminal bar can be positioned at bottom/top/left/right with drag-to-resize. Position and size persisted to localStorage. Collapsed state only reserves the tab bar strip. Dynamic page padding replaces static pb-[340px]. DrawerStack offsets when terminal is on the right. * fix(gastown): clamp size on orientation switch, add vertical close button Re-clamp persisted size when switching position so horizontal minimum (100px) doesn't persist as a too-small vertical width (min 200px). Show close button on agent tabs in vertical mode via absolute positioning on hover. * fix(gastown): fix terminal resize handle and position picker for left/right orientations - Make resize handle a flex child instead of absolute positioned to avoid z-index conflicts with framer-motion stacking contexts - Add visible hover indicator pill on resize handle for discoverability - Remove width/height from CSS transition so drag resize is immediate - Use fixed positioning with viewport clamping for position picker popup to prevent overflow on left/right orientations
…eue-needs-attention-ref/e0d20455/gt/toast/c4ed4587' into convoy/redesign-merge-queue-needs-attention-ref/e0d20455/gt/toast/d93f38f3
…5/gt/toast/d93f38f3' of https://github.com/Kilo-Org/cloud into convoy/redesign-merge-queue-needs-attention-ref/e0d20455/gt/toast/d93f38f3 # Conflicts: # cloudflare-gastown/src/dos/town/review-queue.ts
Add NeedsAttention component with: - Three attention categories: open PRs, failed reviews, stale PRs - Convoy grouping with header cards showing progress, branch, merge mode - Action buttons: Open PR, Retry Review, Fail Bead, View Bead - DrawerStack integration for bead and convoy drawer opening - Confirmation dialogs for destructive actions - Empty state when no items need attention Rewrite MergesPageClient to use getMergeQueueData tRPC procedure with 5s polling interval.
jrf0110
merged commit 3df7843
into
convoy/redesign-merge-queue-needs-attention-ref/e0d20455/head
2 checks passed
jrf0110
deleted the
convoy/redesign-merge-queue-needs-attention-ref/e0d20455/gt/toast/d93f38f3
branch
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
| (SELECT ${bead_events.metadata} | ||
| FROM ${bead_events} | ||
| WHERE ${bead_events.bead_id} = ${beads.bead_id} | ||
| AND ${bead_events.event_type} IN ('review_completed', 'pr_creation_failed') |
Contributor
There was a problem hiding this comment.
WARNING: This can hide the actual failure reason for PR creation failures
pr_creation_failed events currently log { reason: 'invalid_url' } without a message, while completeReviewWithResult() logs the human-readable failure text on the preceding review_completed event. Because this subquery picks the newest of either event type, invalid-URL failures end up with failure_event_metadata.message === undefined, so the new UI renders no reason for that class of failure.
|
|
||
| // Fail bead mutation: use adminForceFailBead | ||
| const failMutation = useMutation( | ||
| trpc.gastown.adminForceFailBead.mutationOptions({ |
Contributor
There was a problem hiding this comment.
WARNING: This action is wired to an admin-only mutation
adminForceFailBead is protected by adminProcedure, but the merge queue page itself is town-scoped and available to non-admin Gastown users. For those users the new "Fail Bead" button will always 403 even though the UI presents it as a normal action.
Contributor
Code Review SummaryStatus: 2 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
Other Observations (not in diff)N/A Files Reviewed (9 files)
Fix these issues in Kilo Cloud Reviewed by gpt-5.4-20260305 · 1,301,752 tokens |
jrf0110
added a commit
that referenced
this pull request
Mar 23, 2026
… log (#1378) * feat(gastown): add getMergeQueueData tRPC procedure for merge queue page Add a dedicated tRPC query that returns structured merge queue data: - needsAttention section with openPRs, failedReviews, and stalePRs - activityLog with enriched review-related bead events - Full JOINs to review_metadata, source beads, convoy_metadata, agents, rigs - Input params: townId (required), rigId, limit, since for filtering/polling - Zod schemas with rpcSafe wrappers and type declarations for frontend * feat(gastown): build 'Needs Your Attention' section for Merge Queue page (#1365) * feat(gastown): configurable terminal orientation with drag-to-resize (#1299) * feat(gastown): configurable terminal orientation and drag-to-resize (#1237) Terminal bar can be positioned at bottom/top/left/right with drag-to-resize. Position and size persisted to localStorage. Collapsed state only reserves the tab bar strip. Dynamic page padding replaces static pb-[340px]. DrawerStack offsets when terminal is on the right. * fix(gastown): clamp size on orientation switch, add vertical close button Re-clamp persisted size when switching position so horizontal minimum (100px) doesn't persist as a too-small vertical width (min 200px). Show close button on agent tabs in vertical mode via absolute positioning on hover. * fix(gastown): fix terminal resize handle and position picker for left/right orientations - Make resize handle a flex child instead of absolute positioned to avoid z-index conflicts with framer-motion stacking contexts - Add visible hover indicator pill on resize handle for discoverability - Remove width/height from CSS transition so drag resize is immediate - Use fixed positioning with viewport clamping for position picker popup to prevent overflow on left/right orientations * chore: cherry-pick getMergeQueueData types from bead 0 * feat(gastown): build 'Needs Your Attention' section for Merge Queue page Add NeedsAttention component with: - Three attention categories: open PRs, failed reviews, stale PRs - Convoy grouping with header cards showing progress, branch, merge mode - Action buttons: Open PR, Retry Review, Fail Bead, View Bead - DrawerStack integration for bead and convoy drawer opening - Confirmation dialogs for destructive actions - Empty state when no items need attention Rewrite MergesPageClient to use getMergeQueueData tRPC procedure with 5s polling interval. * feat(gastown): add convoy grouping and per-rig filtering to Merge Queue page (#1366) * feat(gastown): add convoy grouping and per-rig filtering to Merge Queue page - Add RefineryActivityLog with convoy grouping: entries grouped by convoy with header cards showing title, branch, progress, clickable to open convoy drawer - Add per-rig filter dropdown using listRigs query and shadcn Select, passes rigId to getMergeQueueData for server-side filtering - Include status_changed ActionType with type guard (no unsafe 'as' cast) - Polish layout: page title, rig filter, Needs Attention, Activity Log sections with consistent headers and empty states * fix(gastown): move hooks above early returns and paginate by convoy groups - Move all useMemo hooks above isLoading/empty early returns to prevent React crash when entries transition from 0 to non-zero (hooks must be called in the same order on every render) - Replace flat-entry pagination with group-based pagination: convoy groups (sorted by most recent activity) are kept whole, and standalone entries fill remaining page budget. This ensures recently active convoys appear on page 1 regardless of other convoys' sizes. * fix(gastown): interleave convoy groups and standalone entries by recency in pagination The previous pagination showed all convoy groups before any standalone entries regardless of timestamps. A standalone entry more recent than all convoy groups would still appear below them on page 1. Merge convoy groups and standalone entries into a unified DisplayItem list sorted by most-recent timestamp, then paginate over that list. Convoy groups use their latestTimestamp as sort key; standalone entries use their event.created_at. This ensures the most recently active items appear first on page 1 regardless of whether they are convoy-grouped or standalone. Hooks were already correctly placed before early returns (no conditional hooks issue in the current code). * fix(gastown): gate fail button behind admin check and fix pagination remaining count Hide the admin-only 'Fail Bead' button for non-admin users by checking isAdmin from the session. Fix the activity log pagination to compute hasMore and remaining count based on actual visible entries rather than the budget, preventing incorrect 'Show more' state when a convoy group exceeds the page budget.
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
1 participant
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.
Summary
getMergeQueueDatatRPC procedure with 5s polling, replacing the old event-based approach. Includes loading, error, and empty states.review-queue.tsdata layer withfailure_event_metadataon MR beads and proper source bead resolution viabead_dependenciesJOIN for the activity log.Verification
pnpm typecheck— passes cleanly across all workspace packagesVisual Changes
N/A (no screenshots — backend data layer + UI components for merge queue page)
Reviewer Notes
feat(gastown): configurable terminal orientation with drag-to-resize (#1299)which was already merged to main but not yet in the convoy base branch. These are brought in via cherry-pick commit0f867653.6dc2e09a. Thereview-queue.tsbackend enhancements (failure metadata, source bead resolution) support the NeedsAttention UI's failure reason display and bead title resolution.