refactor(ui): consolidate bulk modals and sprint Primer shell

[fathiraz]

Summary

• Introduce a shared ModalShell and route bulk/sprint dialogs through the modal factory for consistent Primer layout
• Remove standalone bulk close/lock/open/pin/unpin modal components in favor of consolidated bulk action flows
• Refresh sprint modal, settings, progress, and table widget UI to align with GitHub-native Primer CSS patterns
• Extract sprint settings helpers with unit tests for field configuration logic

Changes

Modal infrastructure (src/ui/modal-shell.tsx, src/lib/modal-factory.tsx, src/lib/primer-css-helper.ts)

• Added reusable modal shell (header slot, flat panel styling, motion tokens)
• Updated modal factory to mount dialogs with shared shell semantics
• Extended Primer CSS helper utilities used by sprint and bulk surfaces

Bulk actions (src/features/bulk-actions-*.tsx, deleted bulk-*-modal.tsx)

• Wired bulk bar and modals to consolidated patterns; removed per-action modal files
• Adjusted bulk delete modal and edit flyout/relationship pane for the new shell
• Updated bulk edit flyout tests for the refactor

Sprint UI (src/features/sprint-*.tsx, src/features/sprint-settings-utils.ts)

• Reworked sprint panel modal layout and settings view structure
• Updated sprint injections, progress view, and group header widget
• Added sprint-settings-utils plus tests for settings derivation

Content script (src/entries/content.ts)

• Adjusted content entry wiring for sprint/bulk modal mounting

Test Plan

• pnpm typecheck passes locally
• Bulk bar: close, lock, open, pin, and unpin actions open the expected consolidated UI (no missing modals)
• Bulk delete modal renders and confirms destructive flow
• Bulk edit flyout still opens, edits fields, and passes existing unit tests
• Sprint panel opens from project view; settings, progress, and end-sprint steps navigate correctly
• Sprint group header widget shows loading/active/no-active states and opens the sprint panel
• Modal keyboard focus and Escape behavior remain correct (no input trapped in GitHub page)

---

Summary by cubic

Consolidated bulk action modals and rebuilt the sprint panel on a shared ModalShell for a consistent Primer look and correct focus/Escape behavior. Restores close-reason selection, streamlines sprint widgets, and removes duplicate components.

• Refactors

  • Added ModalShell and updated the modal factory to mount dialogs with shared Primer styling, motion, and Escape isolation.
  • Removed per-action bulk modals (BulkCloseModal, BulkLockModal, BulkOpenModal, BulkPinModal, BulkUnpinModal); routed remaining flows through the factory and ported close/delete to ModalShell.
  • Reworked sprint panel, settings, progress, and table widget to Primer patterns; added ProgressBar and updated header widget styling.
  • Extracted sprint-settings-utils with tests; extended primer-css-helper with chip/button presets; injected sprint UI via light DOM using ctx.

• Bug Fixes

  • Restored bulk close reason radio and set mark-menu “Close” to COMPLETED per ADR; re-check selection after async PAT validation before opening the close modal.
  • Isolated modal Escape from global shortcuts and added ModalShell tests to verify.
  • Pruned detached sprint header widgets and stabilized tooltip mounting.
  • Clarified bulk edit empty states and relationship pane messaging.

^{Written for commit 63bbd43. Summary will update on new commits.}

[github-actions]

Cursor auto review

No actionable issues found on changed lines.

No actionable issues found.

Generated automatically when this PR was submitted using Cursor CLI with --model auto.

[cubic-dev-ai]

3 issues found across 22 files

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[cubic-dev-ai]

2 issues found across 6 files (changes from recent commits).

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>