[plainprogrammer]

Summary

This PR implements spec 006 Collection Table View and Bulk Editing (v5.0.0), following its plan and data model.

• Table view. The collection can be shown as a table with one row per lot. Columns: name with thumbnail, set · number, language, finish, condition, quantity, price paid, and a "…" menu.
  • You can sort by name, set · number, condition (in scale order), quantity and price paid. Unspecified values sort last, and the grid's order breaks ties.
  • The filter, counts, empty states and 120-row pages are shared with the grid.
  • On phones, the optional columns fold into a line under the name.
  • Edit copy and Remove from a row return to the table, keeping its filter, sort and page.
• View switch. Grid and Table appear in the filter bar. The choice is saved per user, and Grid stays the default.
  • The switch's options are buttons in a GET form, so nothing is ever prefetched.
  • The server also never saves a view from a prefetch-marked request (Sec-Purpose, X-Sec-Purpose, X-Moz, Purpose).
• Bulk mode (Edit many).
  • The selection is stored on the server for each session. The database deletes it automatically when the session ends, including when a user's sessions are ended with delete_all.
  • You can tick rows across pages, or tick "Select all" to select every lot matching the filter. Unticking a row then keeps it as an exception.
  • The count is in copies ("12 of 4,812 selected").
  • Everything works without JavaScript: one PATCH form carries the ticks with whichever button you press (Filter, the sort headers, the pager, the actions, Done).
  • With JavaScript on, a Stimulus controller adds the live count, a mixed state on the "Select all" box, and Esc to leave bulk mode.
• Set condition… has its own page. Lots that become identical merge, all or nothing at the 9,999-copy cap.
• Remove asks for confirmation first, then offers a one-time Undo in the status message.
  • The Undo belongs to the session that removed. Any later removal in that session ends it.
  • Replaying an Undo answers 422; another session's or an unknown Undo answers 404.
• Design system. Six new pattern docs (SortHeader, StatusAction, BulkConfirmPage, ChoicePage, BulkForm, ViewSwitchForm), with token-only CSS in collector/additions.css.

Migrations (additive, reversible): users.collection_view (default grid), bulk_selections + bulk_selection_marks, and bulk_removals (a partial unique index allows one undoable removal per session).

Also in this branch

• Brakeman 8.1.0: bin/brakeman runs with --ensure-latest, so the upstream release had started failing CI.
• Status line: it now shows an alert ahead of a notice, and never shows Undo beside an alert. This applies to every page.

Screenshots

These were captured from the dev server with real Scryfall data, in headless Firefox. The phone shots come from a 390px frame.

Bulk-flow walkthrough: grid → Table → sort by condition → row menu → Edit many → tick → Select all with one untick → Set condition → Remove → Undo → Esc.

Grid with the view switch and Edit many	Table, one row per lot
Sorted by condition (scale order, unspecified last)	Row actions menu
Bulk mode, with the count in copies	Select all with one untick (mixed header; the bar stays pinned while scrolled)
Set condition page (51 items: the untick is honoured)	After Set condition
Remove confirmation	Removed, with a one-time Undo
Restored by Undo	Dark theme, table
Dark theme, bulk mode

Phone: table	Phone: bulk mode, actions menu

What the walkthrough caught. These are all fixed test-first in this PR:

• Selection mismatch (data-affecting). With scripting on, "Select all" then an untick showed 51 of 59 selected, but the action covered all 59. Spec v5.0.0 rewrote the FR-4 header rule so that rows changed against their rendered baseline are still applied after a header toggle. New AC-5.10 browser tests cover it (2746e20).
• Bulk bar hidden while scrolling. The bulk bar slid under the sticky app header and filter bar. Both bars now sit in one sticky block below the header (cd95320).
• Table card names in browser-default blue. They now use the design system's brand link colour (24ccf5b). Empty-state links from 004 get the same fix (7fcfb6f).

Process

• The spec went through three Fable spec reviews, the plan through a Fable plan review, and the implementation through two Fable implementation reviews (Mode B). Both implementation reviews said SPEC-ALIGNED.
• Every phase was test-first. Where implementers deviated from the plan, the commit bodies record it as Ruling: lines.

Performance (spec NFR, measured at 5,000 lots)

Measured on synthetic data in a rolled-back test-env transaction:

Step	Time	Budget
Table, any sort	12–35 ms	< 500 ms
Set condition on all 5,000	377–496 ms (was 8.5 s before 6e553c3)	< 2 s
Remove all 5,000	~1.8 s	< 2 s
Undo of 5,000	~0.85 s (was 7.4 s)	—

The queries for the table and the bulk table don't grow with the number of rows (spec/requests/query_counts_spec.rb).

Test plan

• bin/ci passes: RuboCop, Brakeman, bundler-audit, importmap audit and RSpec (480 examples, 0 failures)
• The three migrations roll back and re-apply cleanly, with db/schema.rb unchanged
• Automated screenshot walkthrough at 1280px and 390px, in light and dark themes (see Screenshots; it found and fixed three issues)
• Manual walkthrough at 1280px and 390px, in light and dark themes: switch views, sort, Edit many, tick across pages, Select all then untick one, Set condition, Remove then Undo, Esc with and without an open menu
• Repeat the bulk flow with JavaScript disabled, including Enter in the filter and paging with ticks kept
• Check focus rings, touch targets and status announcements on the view switch, sort headers, checkboxes, bulk bar and Undo

🤖 Generated with Claude Code