Skip to content

Safely integrate Windows WeChat snapshot reader - #15

Merged
mcncarl merged 3 commits into
mainfrom
codex/merge-pr14-safely
Aug 26, 2026
Merged

Safely integrate Windows WeChat snapshot reader#15
mcncarl merged 3 commits into
mainfrom
codex/merge-pr14-safely

Conversation

@mcncarl

@mcncarl mcncarl commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Maintainer takeover of #14 to finish the contribution without another contributor revision round.

This branch preserves the original contributor-authored commit fe589d60d719a5304469b583cf1a48e863a8348d and adds a separate maintainer hardening commit.

Key changes:

  • fail closed on snapshot overlap, unsafe links/reparse points, SQLite sidecars, unsupported schemas, and non-UUIDv4 snapshot IDs
  • bound Zstandard window/output, decoded bytes, scanned rows, and result size
  • mark snapshot content as untrusted data and neutralize exported Markdown
  • bind Windows CI to the exact PR head and lock the CPython 3.12 Windows AMD64 dependency artifact
  • accurately document experimental schema compatibility and provenance

Local verification:

  • Python 3.12 synthetic tests: 21/21 passed
  • Skill quick validation passed
  • workflow YAML, SBOM JSON, diff check, and Windows wheel SHA-256 verification passed
  • independent final security diff review: complete, 0 findings

Merge only after the exact-head windows-2022 job succeeds. Use a merge commit so the contributor commit and authorship remain in history.

@mcncarl
mcncarl merged commit e18fb26 into main Aug 26, 2026
2 checks passed
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.

2 participants