Skip to content

MTL-127: Add PDF report download#83

Merged
LeonSilva15 merged 3 commits into
masterfrom
codex/MTL-127-pdf-report-download
May 27, 2026
Merged

MTL-127: Add PDF report download#83
LeonSilva15 merged 3 commits into
masterfrom
codex/MTL-127-pdf-report-download

Conversation

@LeonSilva15

@LeonSilva15 LeonSilva15 commented May 27, 2026

Copy link
Copy Markdown
Owner

Summary:

  • Add a ReportLab-backed PDF report download endpoint generated from the persisted EPD-style report DTO.
  • Add report page PDF download UI with loading, success, and error feedback.
  • Document PDF as an optional presentation export, not a verified EPD.
  • Preserve report context snapshots so historical JSON/PDF exports do not drift when current assessment, product, plant, manufacturer, inputs, factors, or findings change.

Validation:

  • uv run --project backend pytest backend/app/tests/test_assessment_report_api.py backend/app/tests/test_report_export_service.py backend/app/tests/test_calculation_run_persistence.py: passed, 28 tests
  • uv run --project backend pytest: passed, 312 tests
  • uv run --project backend ruff check .: passed
  • uv run --project backend ruff format backend/app/domain/calculation.py backend/app/domain/calculation_runs.py backend/app/domain/report_export.py backend/app/domain/report_pdf.py backend/app/tests/test_assessment_report_api.py backend/app/tests/test_calculation_run_persistence.py: passed, unchanged
  • uv run --project backend mypy app: passed
  • npm --prefix frontend run lint: passed
  • npm --prefix frontend run format:check: passed
  • npm --prefix frontend run build: passed
  • git diff --check: passed
  • git diff --check upstream/master..HEAD: passed

Review:

  • One subagent review completed.
  • Fixed accepted findings: historical report context drift and PDF determinism wording/byte stability.

Evidence:

Playwright Evidence

Committed evidence directory: skills/materialtrace-demo-evidence/MTL-127

pdf-report-download (unknown)

MaterialTrace ticket: MTL-127

@LeonSilva15 LeonSilva15 merged commit 2785e90 into master May 27, 2026
2 checks passed
@LeonSilva15 LeonSilva15 deleted the codex/MTL-127-pdf-report-download branch May 27, 2026 03:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant