feat: CSV issue export from issues list and machine detail pages by timothyfroehlich · Pull Request #1159 · timothyfroehlich/PinPoint

timothyfroehlich · 2026-04-08T02:49:12Z

Summary

Add CSV export functionality for issues, accessible from two entry points:
- Issues list page: exports issues matching current filter state (next to View Options button)
- Machine detail page: exports all issues for that machine (in the issues expando header)
Server action generates CSV with 13 columns (Issue ID, Machine, Title, Description, Status, Severity, Priority, Frequency, Reporter, Assigned To, Created, Updated, Closed)
Description column uses first paragraph only, with formatting stripped
UTF-8 BOM for Excel compatibility, RFC 4180 compliant CSV
Loading spinner, empty state toast, error handling

Origin

Discord discussion (Eric E., Becca S., Neil Wilson, Tim) — 2026-03-16. Users wanted to identify which machines have the most issues and spot recurring problems for tournament bank decisions.

Test plan

Unit tests: CSV generation utility (6 tests)
Unit tests: First-paragraph ProseMirrorDoc extraction (9 tests)
810/810 existing tests still pass
TypeScript strict mode clean
E2E: Click export on issues list, verify download triggers
E2E: Click export on machine detail page, verify download triggers

🤖 Generated with Claude Code

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

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Tests click the export-csv-button on the issues list page and machine detail page, verifying a download event fires with the correct filename pattern. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

vercel · 2026-04-08T02:49:18Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
pin-point	Ready	Preview, Comment	Apr 8, 2026 2:53pm

supabase · 2026-04-08T02:49:25Z

Updates to Preview Branch (claude/modest-villani) ↗︎

Deployments	Status	Updated
Database	✅	Wed, 08 Apr 2026 14:52:42 UTC
Services	✅	Wed, 08 Apr 2026 14:52:42 UTC
APIs	✅	Wed, 08 Apr 2026 14:52:42 UTC

Tasks are run on every commit but only new migration files are pushed.
Close and reopen this PR if you want to apply changes from existing seed or migration files.

Tasks	Status	Updated
Configurations	✅	Wed, 08 Apr 2026 14:52:44 UTC
Migrations	✅	Wed, 08 Apr 2026 14:52:44 UTC
Seeding	✅	Wed, 08 Apr 2026 14:52:44 UTC
Edge Functions	✅	Wed, 08 Apr 2026 14:52:44 UTC

View logs for this Workflow Run ↗︎.
Learn more about Supabase for Git ↗︎.

Copilot

Pull request overview

Adds CSV export of issues from both the issues list (respecting current filters) and the machine detail page (machine-scoped), using a server action to generate an Excel-friendly CSV and a small client button to trigger downloads.

Changes:

Added CSV generation + first-paragraph extraction utilities with unit tests.
Added exportIssuesAction server action + Zod schemas to generate and return CSV/filename.
Added export UI entry points (issues list + machine detail expando) and Playwright smoke tests for downloads.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
src/lib/tiptap/first-paragraph.ts	New utility to extract first-paragraph plain text from ProseMirror JSON for CSV Description.
src/lib/tiptap/first-paragraph.test.ts	Unit tests for first-paragraph extraction behavior.
src/lib/export/csv.ts	New RFC 4180-style CSV generator with BOM + CRLF endings.
src/lib/export/csv.test.ts	Unit tests for quoting/escaping/BOM behavior.
src/components/issues/IssueList.tsx	Adds export button to issues list toolbar.
src/components/issues/ExportButton.tsx	Client “download CSV” button that calls the server action and triggers a Blob download.
src/app/(app)/m/[initials]/issues-expando.tsx	Adds export button to the machine detail issues expando header.
src/app/(app)/issues/export-schema.ts	Zod schemas for export inputs and permissive filter parsing.
src/app/(app)/issues/export-action.ts	Server action to authenticate, query issues, format rows, and generate CSV + filename.
e2e/smoke/machine-details-redesign.spec.ts	Smoke test that clicking export triggers a CSV download on machine detail.
e2e/smoke/issue-list.spec.ts	Smoke test that clicking export triggers a CSV download on issues list.
docs/superpowers/specs/2026-04-06-csv-export-design.md	Design spec documenting behavior, columns, and approach.
docs/superpowers/plans/2026-04-06-csv-export.md	Implementation plan documenting steps and test strategy.

src/app/(app)/issues/export-action.ts

src/app/(app)/issues/export-schema.ts

src/app/(app)/m/[initials]/issues-expando.tsx

src/lib/export/csv.ts

src/lib/tiptap/first-paragraph.ts

timothyfroehlich · 2026-04-08T02:55:22Z

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}

…hines, JSON validation - Add CSV formula injection protection (prefix =, +, -, @ with single quote) - Set includeInactiveMachines=true for machine-page exports so off-the-floor machines can still export their issues - Return VALIDATION error on malformed filter JSON instead of silently falling back to "export all" Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.

src/lib/export/csv.ts

src/lib/tiptap/first-paragraph.ts

The comment said "Recursively" but the function only iterates the given node array without recursion. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

# Conflicts: # src/app/(app)/m/[initials]/issues-expando.tsx

Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.

Copilot · 2026-04-08T14:59:16Z

src/app/(app)/issues/export-schema.ts

+export const exportFiltersSchema = z.object({
+  q: z.string().optional(),
+  status: z.array(z.enum(ISSUE_STATUS_VALUES)).optional(),
+  machine: z.array(z.string()).optional(),
+  severity: z
+    .array(z.enum(["cosmetic", "minor", "major", "unplayable"]))
+    .optional(),
+  priority: z.array(z.enum(["low", "medium", "high"])).optional(),
+  frequency: z
+    .array(z.enum(["intermittent", "frequent", "constant"]))
+    .optional(),
+  assignee: z.array(z.string()).optional(),
+  owner: z.array(z.string()).optional(),
+  reporter: z.array(z.string()).optional(),
+  watching: z.boolean().optional(),
+  includeInactiveMachines: z.boolean().optional(),
+  sort: z.string().optional(),
+});


exportFiltersSchema doesn’t include the date-range fields (createdFrom, createdTo, updatedFrom, updatedTo) that are part of IssueFilters and are used by buildWhereConditions(). Since ExportButton serializes the current IssueFilters, any active date filters will be silently dropped during parsing and the export won’t match what the user is viewing. Add these fields to the schema (ideally with coercion from ISO strings and a permissive fallback so a single bad date doesn’t cause all filters to be ignored).

Copilot · 2026-04-08T14:59:16Z

src/app/(app)/issues/export-action.ts

+    // 6. Generate CSV
+    const csv = generateCsv(CSV_HEADERS, rows);
+    const dateStr = format(new Date(), "yyyy-MM-dd");
+    const fileName = machineInitials
+      ? `pinpoint-${machineInitials.toUpperCase()}-issues-${dateStr}.csv`
+      : `pinpoint-issues-${dateStr}.csv`;
+
+    return ok({ csv, fileName });


The new export action currently has no integration/unit test that validates CSV headers/row formatting and that filters are applied correctly. The added Playwright checks only assert that a download starts, so regressions in column ordering, description extraction, or filter behavior (e.g., machineInitials export) won’t be caught. Please add at least one integration test around the export action’s query+CSV generation using the existing worker-scoped PGlite pattern.

timothyfroehlich and others added 10 commits April 6, 2026 22:29

docs: add CSV issue export design spec

20df822

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

docs: add CSV export implementation plan

80242e5

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

feat(export): add CSV string generation utility

1406c47

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(export): add first-paragraph extraction for ProseMirrorDoc

622abe3

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(export): add Zod validation schema for export action

7db6d7b

feat(export): add exportIssuesAction server action

cf466a0

feat(export): add ExportButton client component

d18796f

feat(export): add CSV export button to machine detail page

02a7115

feat(export): add CSV export button to issues list page

615ec8e

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

test(export): add E2E tests for CSV export buttons

91e1238

Tests click the export-csv-button on the issues list page and machine detail page, verifying a download event fires with the correct filename pattern. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copilot AI review requested due to automatic review settings April 8, 2026 02:49

Copilot started reviewing on behalf of timothyfroehlich April 8, 2026 02:49 View session

vercel bot deployed to Preview April 8, 2026 02:50 View deployment

Copilot AI reviewed Apr 8, 2026

View reviewed changes

vercel bot deployed to Preview April 8, 2026 03:08 View deployment

Merge remote-tracking branch 'origin/main' into claude/modest-villani

f063ac8

Copilot AI review requested due to automatic review settings April 8, 2026 03:11

Copilot started reviewing on behalf of timothyfroehlich April 8, 2026 03:12 View session

vercel bot deployed to Preview April 8, 2026 03:14 View deployment

Copilot AI reviewed Apr 8, 2026

View reviewed changes

src/lib/export/csv.ts Show resolved Hide resolved

src/lib/tiptap/first-paragraph.ts Show resolved Hide resolved

src/lib/tiptap/first-paragraph.ts Outdated Show resolved Hide resolved

fix(export): correct misleading comment in extractTextFromNodes

671f54b

The comment said "Recursively" but the function only iterates the given node array without recursion. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

vercel bot deployed to Preview April 8, 2026 03:26 View deployment

timothyfroehlich added the ready-for-review PR passed CI and has no unresolved review comments label Apr 8, 2026

Merge remote-tracking branch 'origin/main' into claude/modest-villani

9720342

# Conflicts: # src/app/(app)/m/[initials]/issues-expando.tsx

Copilot AI review requested due to automatic review settings April 8, 2026 14:52

Copilot started reviewing on behalf of timothyfroehlich April 8, 2026 14:52 View session

vercel bot deployed to Preview April 8, 2026 14:53 View deployment

Copilot AI reviewed Apr 8, 2026

View reviewed changes

Conversation

timothyfroehlich commented Apr 8, 2026

Summary

Origin

Test plan

Uh oh!

vercel bot commented Apr 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

supabase bot commented Apr 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

timothyfroehlich commented Apr 8, 2026

Code review

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vercel bot commented Apr 8, 2026 •

edited

Loading

supabase bot commented Apr 8, 2026 •

edited

Loading