fix: report invalid HAR archives cleanly

[sean0407]

Summary

extract_har
  open ZIP archive
+ catch BadZipFile
+ show a Click error and exit nonzero
  inspect HAR and extract matching files

Invalid input such as a plain text file currently raises an uncaught zipfile.BadZipFile from ZipFile(...). Convert it to a clear CLI error and add a regression test. Fixes #321.

Evidence

• Before: The regression test fails against main; invalid input produces no user-facing Click error.
• After: The same test passes: the command exits with status 1, prints Invalid ZIP archive, and includes no traceback.
• How to test: python3 -m unittest -v test_extract_har from the python directory.

Merge Danger

Door: two-way

Blast Radius: narrow

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-10-15T19:59:48.517845Z	d39acd4	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d39acd4acb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Catch BadZipFile raised while reading har.har

When a ZIP's central directory is valid but its har.har bytes fail CRC validation, ZipFile(harzip) succeeds and this new try ends, then zf.read("har.har") raises zipfile.BadZipFile outside the handler. Running the CLI on such a damaged HAR still produces an uncaught traceback and no Invalid ZIP archive message, so the clean-error handling needs to cover the initial member read as well.

Useful? React with 👍 / 👎.