[danielededo]

Summary

With many payslips, the list of files grew long on the page, and it showed every file whenever one needed a look. Now the page lists no file at all.

On the page

• One line under the buttons says how many files need a look, for example "1 file to look at ›", and only if any do.
• Under the report, "Files read (12) · Something read badly?" opens the files.

The sheet

• A <dialog>: full screen on a phone, a 480 px panel at the side of a wider screen. ✕, Escape and the browser's back button close it; it pushes a history entry so that back works.
• To look at: files with a problem, each with its reason. Files whose every payslip was read before collapse into one line, such as "2 files already read, ignored", which opens to list them.
• Payslips read: by month, the latest first. The month is the main text, and the file's name sits small below it, after the person when there are several. The first six show, with "Show all (n)" for the rest.
• Anonymized copy: each row has a link to it. The copy now opens inside the sheet, with "‹ Back to the files".

Core

• analyze now also returns, per document, latest (year×100+month, for sorting) and people.
• report.PersonLabel is exported for this.

Type of change

• New feature (non-breaking)

Checklist

• Commits follow the commit convention
• Tests added or updated for the change
• All tests and linters pass locally
• Documentation updated (README, docs/, comments) where behavior changed
• CHANGELOG.md updated under Unreleased for user-visible changes

Notes for the reviewer

e2e tests

• statuses() now reads every file's line in the sheet, in the order the files were picked, through data-i/data-note. The existing assertions on statuses are unchanged.
• New assertions cover:
  • the count on the page;
  • the repeated files grouped into one line;
  • the anonymized copy opened from the sheet;
  • closing the sheet with the back button.
• The agreement test expects the two new per-document fields.

Local checks

• On top of feat(report): four tabs, one question each, and a swipe that follows the finger #68 and chore: bump playwright from 1.63.0 to 1.64.1 in /web #67: gofmt, vet (including wasm), go test -race, piiscan, the web unit tests and the 13 e2e tests in Chromium all pass. WebKit runs in CI.