fix: replace bare except with except Exception in analysis.py

[srpatcha]

Summary

Replaces bare \�xcept:\ with \�xcept Exception:\ in \skills/publish-to-pages/scripts/convert-pdf.py\ analysis function to avoid catching \SystemExit\ and \KeyboardInterrupt.

Re-targeted to \staged\ branch as requested in #1500.

Signed-off-by: Srikanth Patchava spatchava@meta.com

[github-actions]

🔍 Skill Validator Results

✅ All checks passed

Scope	Checked
Skills	1
Agents	1
Total	2
Severity	Count
---	---:
❌ Errors	0
⚠️ Warnings	0
ℹ️ Advisories	0

Summary

Level	Finding
ℹ️	Found 1 skill(s)
ℹ️	[datanalysis-credit-risk] 📊 datanalysis-credit-risk: 1,365 BPE tokens [chars/4: 1,546] (detailed ✓), 16 sections, 1 code blocks
ℹ️	✅ All checks passed (1 skill(s))

Full validator output

```text Found 1 skill(s) [datanalysis-credit-risk] 📊 datanalysis-credit-risk: 1,365 BPE tokens [chars/4: 1,546] (detailed ✓), 16 sections, 1 code blocks ✅ All checks passed (1 skill(s)) ```

[aaronpowell]

The test-plugins scripts are unrelated to the PR description, and we already have scripts in the repo related to validating plugins.

[srpatcha]

Hi @aaronpowell, thank you for the feedback! You're right — the test-plugins scripts are unrelated to the bare except fix. I'll remove those commits and keep this PR focused on the except fix only. I can submit the validation framework as a separate PR if there's interest.

[aaronpowell]

We already have validation in eng/validate-plugins.mjs script, plus we have an external tool that runs validations as part of CI.

[srpatcha]

Hi @aaronpowell, I've restructured the PR to keep only the original bare-except fix in \scripts/eval/analysis.py. The validation framework and unrelated commits have been removed. The PR now contains exactly one focused change. Ready for re-review!