fix: restrict nobody@mozilla.org to backlog column only

Bugzilla requires a real assignee for non-backlog statuses (ASSIGNED,
IN_PROGRESS, RESOLVED). Filter out nobody@mozilla.org from the assignee
picker in all columns except backlog to prevent invalid status changes.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: msujaws

src/components/Board/Column.test.tsx

@@ -43,15 +43,18 @@ vi.mock('./Card', () => ({
     bug,
     isSelected,
     isGrabbed,
+    allAssignees,
   }: {
     bug: BugzillaBug
     isSelected?: boolean
     isGrabbed?: boolean
+    allAssignees?: Array<{ email: string; displayName: string }>
   }) => (
     <div
       data-testid={`card-${bug.id.toString()}`}
       data-selected={isSelected}
       data-grabbed={isGrabbed}
+      data-assignees={allAssignees ? allAssignees.map((a) => a.email).join(',') : ''}
     >
       Card: {bug.summary}
     </div>
@@ -330,4 +333,60 @@ describe('Column', () => {
       expect(card2).toHaveAttribute('data-grabbed', 'true')
     })
   })
+
+  describe('assignee filtering', () => {
+    const assigneesWithNobody = [
+      { email: 'nobody@mozilla.org', displayName: 'Nobody' },
+      { email: 'dev@mozilla.com', displayName: 'Developer' },
+      { email: 'qa@mozilla.com', displayName: 'QA Engineer' },
+    ]
+
+    it('should include nobody@mozilla.org in backlog column', () => {
+      render(<Column {...defaultProps} column="backlog" allAssignees={assigneesWithNobody} />)
+
+      const card = screen.getByTestId('card-12345')
+      expect(card).toHaveAttribute('data-assignees', expect.stringContaining('nobody@mozilla.org'))
+    })
+
+    it('should exclude nobody@mozilla.org in todo column', () => {
+      render(<Column {...defaultProps} column="todo" allAssignees={assigneesWithNobody} />)
+
+      const card = screen.getByTestId('card-12345')
+      expect(card).toHaveAttribute(
+        'data-assignees',
+        expect.not.stringContaining('nobody@mozilla.org'),
+      )
+      expect(card).toHaveAttribute('data-assignees', expect.stringContaining('dev@mozilla.com'))
+    })
+
+    it('should exclude nobody@mozilla.org in in-progress column', () => {
+      render(<Column {...defaultProps} column="in-progress" allAssignees={assigneesWithNobody} />)
+
+      const card = screen.getByTestId('card-12345')
+      expect(card).toHaveAttribute(
+        'data-assignees',
+        expect.not.stringContaining('nobody@mozilla.org'),
+      )
+    })
+
+    it('should exclude nobody@mozilla.org in in-review column', () => {
+      render(<Column {...defaultProps} column="in-review" allAssignees={assigneesWithNobody} />)
+
+      const card = screen.getByTestId('card-12345')
+      expect(card).toHaveAttribute(
+        'data-assignees',
+        expect.not.stringContaining('nobody@mozilla.org'),
+      )
+    })
+
+    it('should exclude nobody@mozilla.org in done column', () => {
+      render(<Column {...defaultProps} column="done" allAssignees={assigneesWithNobody} />)
+
+      const card = screen.getByTestId('card-12345')
+      expect(card).toHaveAttribute(
+        'data-assignees',
+        expect.not.stringContaining('nobody@mozilla.org'),
+      )
+    })
+  })
 })

src/components/Board/Column.tsx

@@ -1,10 +1,13 @@
+import { useMemo } from 'react'
 import { useDroppable } from '@dnd-kit/core'
 import { Card } from './Card'
 import type { BugzillaBug } from '@/lib/bugzilla/types'
 import type { KanbanColumn } from '@/lib/bugzilla/status-mapper'
 import type { Assignee } from '@/hooks/use-board-assignees'
 import { COLUMN_NAMES } from '@/types'
 
+const NOBODY_EMAIL = 'nobody@mozilla.org'
+
 interface ColumnProps {
   column: KanbanColumn
   bugs: BugzillaBug[]
@@ -49,6 +52,14 @@ export function Column({
   const stagedCount = bugs.filter((bug) => stagedBugIds.has(bug.id)).length
   const countId = `${column}-count`
 
+  // Filter out nobody@mozilla.org for non-backlog columns
+  // Bugzilla requires a real assignee for non-backlog statuses (ASSIGNED, IN_PROGRESS, etc.)
+  const filteredAssignees = useMemo(() => {
+    if (!allAssignees) return
+    if (column === 'backlog') return allAssignees
+    return allAssignees.filter((assignee) => assignee.email !== NOBODY_EMAIL)
+  }, [allAssignees, column])
+
   // Determine column styling based on state
   const getColumnClassName = () => {
     const base = 'flex min-h-[500px] w-72 flex-shrink-0 flex-col rounded-lg p-4 transition-colors'
@@ -128,7 +139,7 @@ export function Column({
               stagedAssignee={stagedAssignees?.get(bug.id)}
               isSelected={selectedIndex === index}
               isGrabbed={selectedIndex === index && isGrabbing}
-              allAssignees={allAssignees}
+              allAssignees={filteredAssignees}
               onAssigneeChange={onAssigneeChange}
             />
           ))}

src/store/slices/staged-slice.test.ts

@@ -430,6 +430,24 @@ describe('StagedSlice', () => {
         { id: 123, status: 'ASSIGNED', assigned_to: 'new@example.com' },
       ])
     })
+
+    it('should include both status and assignee when assignee changed first then column', async () => {
+      mockBatchUpdateBugs.mockResolvedValueOnce({
+        successful: [123],
+        failed: [],
+      })
+
+      const { stageChange, stageAssigneeChange, applyChanges } = useStore.getState()
+
+      // User first changes assignee, then drags to new column
+      stageAssigneeChange(123, 'old@example.com', 'new@example.com')
+      stageChange(123, 'backlog', 'todo')
+      await applyChanges(testApiKey)
+
+      expect(mockBatchUpdateBugs).toHaveBeenCalledWith([
+        { id: 123, status: 'ASSIGNED', assigned_to: 'new@example.com' },
+      ])
+    })
   })
 
   describe('applyChanges with resolution', () => {