add flagged queue (#423)

Author: Cybdo

app/controllers/admin/application_controller.rb

@@ -30,6 +30,7 @@ def admin_stats_payload
         pending_reviews_tier_3: tier_counts["tier_3"].to_i,
         pending_reviews_tier_4: tier_counts["tier_4"].to_i,
         pending_requirements_checks: Project.kept.where(status: :pending).requirements_unchecked.count,
+        flagged_projects: Project.kept.flagged_for_review.count,
         projects: Project.kept.count,
         users: User.kept.count,
         pending_orders: Order.pending.count,

app/controllers/admin/reviews_controller.rb

@@ -4,6 +4,7 @@ class Admin::ReviewsController < Admin::ApplicationController
   before_action :set_project, only: [ :show, :skip, :track, :claim ]
 
   REQUIREMENTS_QUEUE = "requirements".freeze
+  FLAGGED_QUEUE = "flagged".freeze
 
   TRACKABLE_BUTTONS = %w[
     end_session skip next_project view_project
@@ -35,6 +36,13 @@ def requirements
     render_queue(policy_scope(Project).kept.where(status: :pending).requirements_unchecked, REQUIREMENTS_QUEUE)
   end
 
+  def flagged
+    base = policy_scope(Project).kept.flagged_for_review
+    base = base.where(status: params[:status]) if Project.statuses.key?(params[:status].to_s)
+
+    render_queue(base, FLAGGED_QUEUE, already_flagged: true)
+  end
+
   LEADERBOARD_DECISION_ACTIONS = %w[project.approved project.returned project.rejected project.pitch_approved].freeze
   LEADERBOARD_WEEKLY_TARGET = 5
 
@@ -230,12 +238,14 @@ def serialize_review_queue_row(project, sessions)
       name: project.name,
       user_id: project.user_id,
       user_display_name: project.user.display_name,
+      status: project.status,
       tier: project.tier,
       is_build_review: project.build_review,
       waiting_since_iso: (project.submitted_at || project.created_at).iso8601,
       claimed_by: holder ? { name: holder.reviewer.display_name, avatar: holder.reviewer.avatar } : nil,
       flagged: project.flagged_for_review?,
       flag_reason: project.flag_reason,
+      flagged_by_name: project.flagged_by&.display_name,
       requirements_checked_by: project.requirements_checked_by&.display_name
     }
   end
@@ -251,17 +261,17 @@ def require_review_access!
     raise ActionController::RoutingError, "Not Found"
   end
 
-  def render_queue(base, queue_key)
-    queue = base.not_flagged_for_review
+  def render_queue(base, queue_key, already_flagged: false)
+    queue = already_flagged ? base : base.not_flagged_for_review
 
     scope =
       case params[:filter]
-      when "flagged" then base.flagged_for_review
+      when "flagged" then already_flagged ? base : base.flagged_for_review
       when "design" then queue.where(build_review: false)
       when "build" then queue.where(build_review: true)
       else queue
       end
-    scope = scope.includes(:user, :requirements_checked_by)
+    scope = scope.includes(:user, :requirements_checked_by, :flagged_by)
     scope = scope.search(params[:query]) if params[:query].present?
 
     @pagy, @projects = pagy(scope.order(Arel.sql("COALESCE(submitted_at, created_at) ASC")))
@@ -276,6 +286,7 @@ def render_queue(base, queue_key)
       pagy: pagy_props(@pagy),
       query: params[:query].to_s,
       filter: params[:filter].to_s,
+      status: params[:status].to_s,
       tier: queue_key,
       allowed_tiers: available_queues,
       metrics: ReviewQueueMetrics.new(queue).as_json,

app/javascript/components/admin/AdminSidebar.tsx

@@ -13,6 +13,7 @@ import {
   Sparkles,
   Bot,
   Flag,
+  FlagTriangleRight,
   LifeBuoy,
   Database,
   Activity,
@@ -42,6 +43,7 @@ interface AdminStats {
   pending_reviews_tier_3: number
   pending_reviews_tier_4: number
   pending_requirements_checks: number
+  flagged_projects: number
   projects: number
   users: number
   pending_orders: number
@@ -129,6 +131,13 @@ function buildSections(): { items: NavItem[] }[] {
           statKey: 'pending_pitches',
           permission: 'pending_reviews',
         },
+        {
+          label: 'Flagged Projects',
+          href: '/admin/reviews/flagged',
+          icon: FlagTriangleRight,
+          statKey: 'flagged_projects',
+          permission: 'pending_reviews',
+        },
         { label: 'Leaderboard', href: '/admin/reviews/leaderboard', icon: Trophy, permission: 'pending_reviews' },
       ],
     },

app/javascript/pages/Admin/Reviews/Queue.tsx

@@ -15,17 +15,22 @@ interface ReviewQueueRow {
   name: string
   user_id: number
   user_display_name: string
+  status: string
   tier: ProjectTier
   is_build_review: boolean
   waiting_since_iso: string
   claimed_by: { name: string; avatar: string } | null
+  flagged_by_name: string | null
   requirements_checked_by: string | null
 }
 
 const REQUIREMENTS_QUEUE = 'requirements'
+const FLAGGED_QUEUE = 'flagged'
 
 function queueLabel(key: string) {
-  return key === REQUIREMENTS_QUEUE ? 'REQS' : key.toUpperCase()
+  if (key === REQUIREMENTS_QUEUE) return 'REQS'
+  if (key === FLAGGED_QUEUE) return 'Flagged'
+  return key.toUpperCase()
 }
 
 const FILTERS = [
@@ -34,6 +39,34 @@ const FILTERS = [
   { key: 'build', label: 'Build' },
 ]
 
+const STATUS_FILTERS = [
+  { key: '', label: 'All statuses' },
+  { key: 'draft', label: 'Draft' },
+  { key: 'pending', label: 'Pending' },
+  { key: 'approved', label: 'Approved' },
+  { key: 'returned', label: 'Returned' },
+  { key: 'rejected', label: 'Rejected' },
+  { key: 'pitch_pending', label: 'Pitch Pending' },
+  { key: 'pitch_approved', label: 'Pitch Approved' },
+]
+
+const STATUS_BADGE_VARIANT: Record<string, 'outline' | 'success' | 'warning' | 'destructive' | 'secondary'> = {
+  draft: 'secondary',
+  pending: 'outline',
+  approved: 'success',
+  returned: 'warning',
+  rejected: 'destructive',
+  pitch_pending: 'outline',
+  pitch_approved: 'success',
+}
+
+function statusLabel(status: string) {
+  return status
+    .split('_')
+    .map((w) => w[0].toUpperCase() + w.slice(1))
+    .join(' ')
+}
+
 function waitBadgeVariant(seconds: number, slaHours: number): 'outline' | 'warning' | 'destructive' {
   const hours = seconds / 3600
   if (hours >= slaHours) return 'destructive'
@@ -46,6 +79,7 @@ export default function AdminReviewsQueue({
   pagy,
   query,
   filter,
+  status,
   tier,
   allowed_tiers,
   metrics,
@@ -55,13 +89,15 @@ export default function AdminReviewsQueue({
   pagy: PagyProps
   query: string
   filter: string
+  status: string
   tier: string
   allowed_tiers: string[]
   metrics: QueueMetrics
   first_pending_id: number | null
 }) {
   const [searchQuery, setSearchQuery] = useState(query)
   const [now, setNow] = useState(() => Date.now())
+  const isFlaggedQueue = tier === FLAGGED_QUEUE
 
   useEffect(() => {
     const t = setInterval(() => setNow(Date.now()), 30000)
@@ -70,13 +106,25 @@ export default function AdminReviewsQueue({
 
   function submitSearch(e: React.FormEvent) {
     e.preventDefault()
-    router.get(`/admin/reviews/${tier}`, { query: searchQuery, filter: filter || undefined }, { preserveState: true })
+    router.get(
+      `/admin/reviews/${tier}`,
+      { query: searchQuery, filter: filter || undefined, status: status || undefined },
+      { preserveState: true },
+    )
   }
 
   function applyFilter(key: string) {
     router.get(
       `/admin/reviews/${tier}`,
-      { query: searchQuery || undefined, filter: key || undefined },
+      { query: searchQuery || undefined, filter: key || undefined, status: status || undefined },
+      { preserveState: true },
+    )
+  }
+
+  function applyStatus(key: string) {
+    router.get(
+      `/admin/reviews/${tier}`,
+      { query: searchQuery || undefined, filter: filter || undefined, status: key || undefined },
       { preserveState: true },
     )
   }
@@ -86,12 +134,18 @@ export default function AdminReviewsQueue({
       <div className="flex flex-col md:flex-row md:items-center md:justify-between gap-4">
         <div>
           <h1 className="text-2xl font-semibold tracking-tight">
-            {tier === REQUIREMENTS_QUEUE ? 'Requirements Check Queue' : `Tier ${tier.slice(1)} Review Queue`}
+            {tier === REQUIREMENTS_QUEUE
+              ? 'Requirements Check Queue'
+              : tier === FLAGGED_QUEUE
+                ? 'Flagged Projects'
+                : `Tier ${tier.slice(1)} Review Queue`}
           </h1>
           <p className="text-sm text-muted-foreground mt-1">
             {tier === REQUIREMENTS_QUEUE
               ? 'Projects waiting on a requirements check. Work the checklist, then pass it on or return it to the builder.'
-              : 'Oldest submissions first. Open one to start a timed review session — keyboard shortcuts make it fast.'}
+              : tier === FLAGGED_QUEUE
+                ? 'Projects a reviewer has pulled aside for a second look. Resolve the flag from inside the review.'
+                : 'Oldest submissions first. Open one to start a timed review session — keyboard shortcuts make it fast.'}
           </p>
           {allowed_tiers.length > 1 && (
             <div className="flex gap-2 mt-3">
@@ -144,6 +198,28 @@ export default function AdminReviewsQueue({
               </button>
             )
           })}
+          {isFlaggedQueue && (
+            <>
+              <span className="w-px bg-border mx-1" />
+              {STATUS_FILTERS.map((f) => {
+                const isActive = (status || '') === f.key
+                return (
+                  <button
+                    key={f.key}
+                    onClick={() => applyStatus(f.key)}
+                    className={cn(
+                      'inline-flex items-center px-3 py-1.5 text-xs font-medium rounded-md border transition-colors cursor-pointer',
+                      isActive
+                        ? 'bg-primary text-primary-foreground border-primary'
+                        : 'border-border bg-background text-muted-foreground hover:bg-accent hover:text-foreground',
+                    )}
+                  >
+                    {f.label}
+                  </button>
+                )
+              })}
+            </>
+          )}
         </div>
         <form onSubmit={submitSearch} className="flex gap-2 items-center">
           <div className="relative">
@@ -164,16 +240,19 @@ export default function AdminReviewsQueue({
 
       <div className="rounded-md border border-border bg-card overflow-hidden">
         {projects.length === 0 ? (
-          <p className="text-sm text-muted-foreground py-12 text-center">Nothing pending — queue is clear. 🎉</p>
+          <p className="text-sm text-muted-foreground py-12 text-center">
+            {isFlaggedQueue ? 'No flagged projects — queue is clear. 🎉' : 'Nothing pending — queue is clear. 🎉'}
+          </p>
         ) : (
           <Table>
             <TableHeader>
               <TableRow>
                 <TableHead>Project</TableHead>
                 <TableHead>Author</TableHead>
                 <TableHead>Type</TableHead>
+                {isFlaggedQueue && <TableHead>Status</TableHead>}
                 <TableHead>Waiting</TableHead>
-                <TableHead>Claimed</TableHead>
+                <TableHead>{isFlaggedQueue ? 'Flagged by' : 'Claimed'}</TableHead>
                 <TableHead className="w-24"></TableHead>
               </TableRow>
             </TableHeader>
@@ -206,14 +285,25 @@ export default function AdminReviewsQueue({
                         <span className="text-xs text-muted-foreground capitalize">{p.tier.replace('_', ' ')}</span>
                       )}
                     </TableCell>
+                    {isFlaggedQueue && (
+                      <TableCell>
+                        <Badge variant={STATUS_BADGE_VARIANT[p.status] ?? 'outline'}>{statusLabel(p.status)}</Badge>
+                      </TableCell>
+                    )}
                     <TableCell>
                       <Badge variant={waitBadgeVariant(waitSeconds, metrics.sla_hours)}>
                         {formatDuration(waitSeconds)}
                         {waitSeconds / 3600 >= metrics.sla_hours && ' overdue'}
                       </Badge>
                     </TableCell>
                     <TableCell>
-                      {p.claimed_by ? (
+                      {isFlaggedQueue ? (
+                        p.flagged_by_name ? (
+                          <span className="text-xs text-muted-foreground">{p.flagged_by_name}</span>
+                        ) : (
+                          <span className="text-xs text-muted-foreground/50">—</span>
+                        )
+                      ) : p.claimed_by ? (
                         <span className="flex items-center gap-1.5 text-xs text-muted-foreground">
                           <img src={p.claimed_by.avatar} alt="" className="size-4 rounded-full" />
                           {p.claimed_by.name}

config/routes.rb

@@ -300,6 +300,7 @@
       get "reviews/:tier" => "reviews#index", as: :tier_reviews, constraints: { tier: /t[1-4]/ }
       get "reviews/leaderboard" => "reviews#leaderboard", as: :reviews_leaderboard
       get "reviews/requirements" => "reviews#requirements", as: :requirements_reviews
+      get "reviews/flagged" => "reviews#flagged", as: :flagged_reviews
       get "reviews/:id" => "reviews#show", as: :review
       post "reviews/:id/skip" => "reviews#skip", as: :skip_review
       post "reviews/:id/track" => "reviews#track", as: :track_review