[PR #93] [MERGED] feat(process): add deterministic capture v3 #1832

New issue

Closed

opened 2026-10-16 21:11:34 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-16 21:11:34 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/morluto/rea/pull/93
Author: @morluto
Created: 7/13/2026
Status: ✅ Merged
Merged: 7/13/2026
Merged by: @morluto

Base: main ← Head: codex/process-capture-v3

📝 Commits (5)

a504342 fix(artifacts): diagnose unpacked ASAR integrity failures
5a361a6 feat(process): add deterministic capture v3
50f4a29 docs: document process capture v3
ba15d16 test(process): use deterministic hang fixture
841990b docs(process): preserve capture invariants

📊 Changes

551 files changed (+4877 additions, -1981 deletions)

View changed files

📝 README.md (+20 -1)
📝 docs/api/assets/hierarchy.js (+1 -1)
📝 docs/api/assets/navigation.js (+1 -1)
📝 docs/api/assets/search.js (+1 -1)
📝 docs/api/classes/application_BinarySession.BinarySession.html (+3 -3)
📝 docs/api/classes/application_CompositeProvider.CompositeProvider.html (+3 -3)
📝 docs/api/classes/application_EnhancedTools.EnhancedTools.html (+3 -3)
📝 docs/api/classes/application_EvidenceLedger.EvidenceLedger.html (+3 -3)
➕ docs/api/classes/application_ProcessCheckpoints.ProcessCheckpoints.html (+152 -0)
📝 docs/api/classes/application_ProcessHarness.ProcessCaptureError.html (+6 -6)
➕ docs/api/classes/application_TerminalRenderer.TerminalRenderer.html (+147 -0)
📝 docs/api/classes/domain_errors.AnalysisCancelledError.html (+3 -3)
📝 docs/api/classes/domain_errors.AnalysisCapabilityUnavailableError.html (+3 -3)
📝 docs/api/classes/domain_errors.AnalysisError.html (+3 -3)
📝 docs/api/classes/domain_errors.AnalysisInputError.html (+3 -3)
📝 docs/api/classes/domain_errors.AnalysisOutputError.html (+3 -3)
📝 docs/api/classes/domain_errors.AnalysisProtocolError.html (+3 -3)
📝 docs/api/classes/domain_errors.AnalysisTimeoutError.html (+3 -3)
📝 docs/api/classes/domain_errors.ArtifactOperationError.html (+13 -6)
📝 docs/api/classes/domain_errors.BinaryTargetError.html (+8 -8)

...and 80 more files

📄 Description

Summary

migrate process capture evidence to v3 with exact PTY rendering, scripted interactions, lifecycle filesystem checkpoints, bounded command shims, and ownership-aware process sampling and cleanup
add first-divergence comparison plus CLI parity for capture and comparison workflows
improve ASAR unpacked-file inventory and return structured integrity mismatch diagnostics
repair the upstream node-pty macOS spawn-helper mode during package installation without adding another dependency

Contract and behavior changes

Process Capture evidence predicate is now rea.process-capture/v3
Process comparison evidence predicate is now rea.process-comparison/v2
adds capture-process and compare-process-captures CLI commands
preserves the 68-tool MCP inventory

Verification

npm run check
npm run verify:package
npm run knip
350 tests across 56 files
clean install exercised the node-pty permission repair and native PTY scenarios

Real Hopper verification was not performed; this change does not alter Hopper analysis behavior.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/morluto/rea/pull/93 **Author:** [@morluto](https://github.com/morluto) **Created:** 7/13/2026 **Status:** ✅ Merged **Merged:** 7/13/2026 **Merged by:** [@morluto](https://github.com/morluto) **Base:** `main` ← **Head:** `codex/process-capture-v3` --- ### 📝 Commits (5) - [`a504342`](https://github.com/morluto/rea/commit/a504342d2b7e5ed87620d449f78102282054e063) fix(artifacts): diagnose unpacked ASAR integrity failures - [`5a361a6`](https://github.com/morluto/rea/commit/5a361a67258e1f89507497eafd5fc487e54d5820) feat(process): add deterministic capture v3 - [`50f4a29`](https://github.com/morluto/rea/commit/50f4a298f44bf56dc78d5717cb23d69db5df4265) docs: document process capture v3 - [`ba15d16`](https://github.com/morluto/rea/commit/ba15d16c5869ef29e85a0e9bacff3ac07edda3a2) test(process): use deterministic hang fixture - [`841990b`](https://github.com/morluto/rea/commit/841990bde158a996be65feabcef4aa5e18fc3a4b) docs(process): preserve capture invariants ### 📊 Changes **551 files changed** (+4877 additions, -1981 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+20 -1) 📝 `docs/api/assets/hierarchy.js` (+1 -1) 📝 `docs/api/assets/navigation.js` (+1 -1) 📝 `docs/api/assets/search.js` (+1 -1) 📝 `docs/api/classes/application_BinarySession.BinarySession.html` (+3 -3) 📝 `docs/api/classes/application_CompositeProvider.CompositeProvider.html` (+3 -3) 📝 `docs/api/classes/application_EnhancedTools.EnhancedTools.html` (+3 -3) 📝 `docs/api/classes/application_EvidenceLedger.EvidenceLedger.html` (+3 -3) ➕ `docs/api/classes/application_ProcessCheckpoints.ProcessCheckpoints.html` (+152 -0) 📝 `docs/api/classes/application_ProcessHarness.ProcessCaptureError.html` (+6 -6) ➕ `docs/api/classes/application_TerminalRenderer.TerminalRenderer.html` (+147 -0) 📝 `docs/api/classes/domain_errors.AnalysisCancelledError.html` (+3 -3) 📝 `docs/api/classes/domain_errors.AnalysisCapabilityUnavailableError.html` (+3 -3) 📝 `docs/api/classes/domain_errors.AnalysisError.html` (+3 -3) 📝 `docs/api/classes/domain_errors.AnalysisInputError.html` (+3 -3) 📝 `docs/api/classes/domain_errors.AnalysisOutputError.html` (+3 -3) 📝 `docs/api/classes/domain_errors.AnalysisProtocolError.html` (+3 -3) 📝 `docs/api/classes/domain_errors.AnalysisTimeoutError.html` (+3 -3) 📝 `docs/api/classes/domain_errors.ArtifactOperationError.html` (+13 -6) 📝 `docs/api/classes/domain_errors.BinaryTargetError.html` (+8 -8) _...and 80 more files_ </details> ### 📄 Description ## Summary - migrate process capture evidence to v3 with exact PTY rendering, scripted interactions, lifecycle filesystem checkpoints, bounded command shims, and ownership-aware process sampling and cleanup - add first-divergence comparison plus CLI parity for capture and comparison workflows - improve ASAR unpacked-file inventory and return structured integrity mismatch diagnostics - repair the upstream node-pty macOS spawn-helper mode during package installation without adding another dependency ## Contract and behavior changes - Process Capture evidence predicate is now `rea.process-capture/v3` - Process comparison evidence predicate is now `rea.process-comparison/v2` - adds `capture-process` and `compare-process-captures` CLI commands - preserves the 68-tool MCP inventory ## Verification - `npm run check` - `npm run verify:package` - `npm run knip` - 350 tests across 56 files - clean install exercised the node-pty permission repair and native PTY scenarios Real Hopper verification was not performed; this change does not alter Hopper analysis behavior. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-16 21:11:34 +00:00