feat: extract PDFExport IIFE module from JavaScript.html (Phase 1 PR7 — final)

[cheerc]

What

Extract printScheduleToPdf (~320 lines, the largest single method) from JavaScript.html into PDFExport.js.html IIFE module. This is the final PR of Phase 1 IIFE domain separation (#129).

After this PR, JavaScript.html contains only 2 methods: init and applyTagFilters.

Phase 1 Complete Summary (48 → 2 methods in JavaScript.html)

PR	Module	Methods	Status
PR1	UtilityFunctions.js.html	7	✅ Merged
PR2	LockManager.js.html	6	✅ Merged
PR3	DataCollection.js.html	14	✅ Merged
PR4	FilterEngine.js.html	8	✅ Merged
PR5	DataIO.js.html	5	✅ Merged
PR6	ScheduleManager.js.html	11	✅ PR #150
PR7	PDFExport.js.html	1	This PR

How

• Created PDFExport.js.html with IIFE pattern (function(App) { ... })(App)
• Converted all this.xxx → App.xxx references
• Added include in Index.html after ScheduleManager.js, before App.init()
• Updated 2 test files (appWiringContracts, asyncMethodsWiring)
• lifecycleRegression.test.js L242: no change needed (checks Interaction.js.html event binding, not method source)

Changed Files (5)

• PDFExport.js.html (new, ~340 lines)
• JavaScript.html (removed printScheduleToPdf, only init + applyTagFilters remain)
• Index.html (+1 include line)
• tests/unit/appWiringContracts.test.js (NOT_EXTRACTABLE 3→2, IIFE_EXTRACTED +1, count 3→2)
• tests/unit/asyncMethodsWiring.test.js (PDF_EXPORT_ASYNC_METHODS + resolveSource)

Verification

• npm test: 1147 tests passed on first try (36 files) ✅
• npx eslint --ext .html .: 0 errors (8 pre-existing warnings) ✅
• bash scripts/check-structure.sh: all present ✅
• Scope gate: only expected 5 files changed ✅

refs #129

[cheerc]

✅ VERIFIED

Reviewed HEAD: b61857f02933f2992e6632e2f8757999e079f118
Reviewer: cb-team-reviewer
Audit mode: fast_path (D1 — single self-contained leaf method, no cross-module callers)

Files Reviewed (5)

• PDFExport.js.html (new, +338) — IIFE module with printScheduleToPdf
• JavaScript.html (-319) — only init + applyTagFilters remain (2 methods)
• Index.html (+1) — include after ScheduleManager.js, before App.init()
• tests/unit/appWiringContracts.test.js — IIFE_EXTRACTED +1, NOT_EXTRACTABLE 3→2, count 3→2
• tests/unit/asyncMethodsWiring.test.js — PDF_EXPORT_ASYNC_METHODS, resolveSource

Fast Path Checklist

• Zero residual this. references in PDFExport.js.html
• IIFE pattern (function(App) { ... })(App) ✓
• JavaScript.html reduced to 2 methods: init, applyTagFilters ✓
• Count: 48 - 7 - 4 - 11 - 7 - 5 - 11 - 1 = 2 ✓
• NOT_EXTRACTABLE: init + applyTagFilters ✓
• Load order: ScheduleManager → PDFExport → App.init() ✓
• No conflict markers (5 files clean)

🎉 Phase 1 Milestone

This is the final PR of Phase 1 IIFE refactoring. JavaScript.html: 48→2 methods. All domain modules now IIFE-wrapped.

Evidence

• ran: grep this\\. PDFExport.js.html → 0 matches
• ran: grep remaining methods JavaScript.html → 2 (init, applyTagFilters)
• ran: conflict marker scan (5 files) → all 0
• cited: PDFExport.js.html:1 — \u003cscript\u003e (function(App) {
• cited: PDFExport.js.html:338 — })(App);\u003c/script\u003e
• cited: Index.html:482 — PDFExport.js after ScheduleManager.js
• cited: appWiringContracts.test.js:131 — expect(publicMethods.length).toBe(2)