Skip to content

feat(reporting): Add PDF/HTML Scan Report Exporter with Visual Charts and Remediation Roadmap#925

Closed
Rafiaminhaj wants to merge 9 commits into
utksh1:mainfrom
Rafiaminhaj:feature/report-exporter
Closed

feat(reporting): Add PDF/HTML Scan Report Exporter with Visual Charts and Remediation Roadmap#925
Rafiaminhaj wants to merge 9 commits into
utksh1:mainfrom
Rafiaminhaj:feature/report-exporter

Conversation

@Rafiaminhaj

@Rafiaminhaj Rafiaminhaj commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Closes #916
Closes #944

Changes Implemented:

  1. Visual Scan Summaries: Generated PIL-based horizontal severity distribution charts embedded as base64 PNGs for high compatibility in both browser HTML and PDF exports.
  2. Remediation Roadmap Checklist: Prioritized findings by severity and difficulty with an interactive step-by-step checkbox UI.
  3. Format Options: Modern tab switcher to toggle between 'Executive Summary Sheet' (summaries, chart, AI Summary, roadmap) and 'Developer-Focused Logs' (full technical findings and CLI parameters).
  4. Usability Polish: Integrated a clean 'Print / Save PDF' action utilizing print styles to hide controls on export.

@utksh1 utksh1 added level:advanced 55 pts difficulty label for advanced contributor PRs type:feature Feature work category bonus label area:backend Backend API, database, or service work area:docs Documentation or contributor guide work labels Jun 14, 2026

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the report exporter work. This cannot merge while required CI is failing.

Current failing checks include frontend-checks and backend-unit. Please fix those first. This is also a large reporting change, so once CI is green it needs focused coverage for HTML/PDF generation, escaping of report-controlled content, and the remediation-roadmap output.

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update. This remains too broad to merge safely: it combines backend report-generation changes, embedded chart generation, interactive HTML report behavior, frontend TaskDetails export/print changes, global print CSS, and audit-config conflict resolution. Please split this into smaller PRs, keep each diff focused, and add targeted tests for the backend report output separately from frontend export UI.

@Rafiaminhaj

Copy link
Copy Markdown
Contributor Author

Closing this monolithic PR since I have split it into three separate, focused pull requests as requested by @utksh1:

Please review the separate PRs instead. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:backend Backend API, database, or service work area:docs Documentation or contributor guide work level:advanced 55 pts difficulty label for advanced contributor PRs type:feature Feature work category bonus label

Projects

None yet

2 participants