Skip to content

Add PDF report export#3

Merged
eugen-goebel merged 3 commits into
mainfrom
feature/pdf-export
Apr 1, 2026
Merged

Add PDF report export#3
eugen-goebel merged 3 commits into
mainfrom
feature/pdf-export

Conversation

@eugen-goebel
Copy link
Copy Markdown
Owner

Summary

  • Add PDFReportGenerator class using fpdf2 for PDF threat analysis reports
  • Support all report sections: cover page, executive summary, traffic overview, protocol analysis, threat summary, threat details, timeline, recommendations, methodology
  • Add --format pdf CLI flag to select PDF output instead of default DOCX
  • Unicode sanitization for latin-1 compatible PDF output

Closes #2

Test plan

  • 11 new tests pass (file creation, PDF header, empty threats, nested dirs, special characters, sanitization)
  • Full suite: 85 tests pass
  • No AI traces in generated files

eugen-goebel and others added 3 commits April 1, 2026 13:11
Implement PDFReportGenerator class that produces threat analysis reports
as PDF documents. Includes cover page, executive summary, traffic overview,
protocol analysis, threat summary table with severity color-coding,
detailed threat breakdowns, timeline analysis, recommendations, and
methodology sections. Unicode text is sanitized to latin-1 for
compatibility with the Helvetica font.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Wire up the PDF generator through the orchestrator and CLI. Users can
now pass --format pdf to produce PDF reports instead of the default DOCX.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
11 tests covering file creation, PDF header validation, empty threats,
nested output directories, special character handling, and the Unicode
sanitization function.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@eugen-goebel eugen-goebel merged commit 89fbd0d into main Apr 1, 2026
3 checks passed
@eugen-goebel eugen-goebel deleted the feature/pdf-export branch April 1, 2026 11:12
eugen-goebel added a commit that referenced this pull request Jun 2, 2026
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.

Export threat report as PDF

1 participant