Repository navigation
feat(proofshot): Harden isolated multi-session evidence - #7
Merged
Merged
Conversation
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Redact nested authentication routes and structured diagnostic secrets without losing safe sibling fields. Reject inherited global JSON output so ProofShot controls structured browser calls explicitly. Co-authored-by: Codex <noreply@openai.com>
Consume full unquoted authorization fields and recognize escaped JSON key/value quotes so nested credentials cannot survive sanitization. Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Consume complete unquoted and multiline sensitive fields so diagnostic continuations cannot escape canonical evidence redaction. Co-authored-by: Codex <noreply@openai.com>
Treat indented diagnostic continuations as part of their sensitive field while preserving later unindented evidence. Co-authored-by: Codex <noreply@openai.com>
Carry sensitive free-form fields across empty folded lines while preserving later unindented diagnostics. Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Use locale-independent PowerShell listener queries on Windows and fail explicitly when Unix lsof inspection is unavailable. Preserve exact owned-tree cleanup and expose the prerequisite through doctor and user documentation. Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Cover lowercase secretkey and secretkeys in the bounded redaction grammar across public evidence surfaces. Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
This was referenced Aug 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
Build / Unit / Integration / E2E
npm cinpm run build— build and DTS output passed under Node 24node dist/bin/proofshot.js --helpnode dist/bin/proofshot.js --versionnpm pack --dry-runnpm test— 548/548 tests across 34 filesnpm run test:ffmpeg— real FFmpeg integration passedUser testing
c4dcf97, CLI SHA-256804c33caca90cb000a78f9de6d8134554c48fa7a69ab5b76ba323513594e0be3, ffmpeg availability, a healthy doctor result, and an empty session inventory.Revert risk
High: this changes core session lifecycle, process ownership, browser containment, evidence redaction, artifact creation, and the minimum Node runtime.
Notes
npm cireported eight dependency vulnerabilities (3 moderate, 4 high, 1 critical). Their triage remains a separate follow-up.