mirror of
https://github.com/morluto/rea.git
synced 2026-11-05 06:45:22 +00:00
[GH-ISSUE #3172] [Bug] Passive capture comparison treats missing redirect response fields as known evidence #1108
Closed
opened 2026-10-15 18:31:08 +00:00 by bsakaguchi
·
2 comments
No Branch/Tag specified
main
codex/procedure-reference-stacked
release-please--branches--main--components--rea-agents
release-please--branches--main--components--rea-agents--release-notes
codex/kuna-independent-retry
codex/byte-search-overlay-coverage
codex/open-failure-target-receipt
codex/kuna-xref-evidence
codex/ghidra-ambiguous-procedure-recovery
codex/function-source-bundle
codex/trace-function-integration-prototype
codex/snapshot-source-closure
codex/recorded-evidence-array-pages
codex/evidence-recovery-redesign
codex/trace-feature-address-scope
codex/call-discovery-scope
codex/ghidra-literal-candidate-evidence
codex/decompile-directory-independent-retry
codex/retry-provenance-base
codex/value-flow-capability-experiment
codex/ghidra-containing-instruction
cursor/readme-discord-button-ef9f
dependabot/github_actions/actions/setup-node-7.1.0
dependabot/github_actions/actions/upload-artifact-7.0.2
dependabot/npm_and_yarn/npm-minor-and-patch-d8d97ce6df
codex/kuna-1989
fix/8559-native-ui-fixture-ownership
fix/8555-go-verifier-mcp-ownership
codex/provider-capability-recovery
fix/8642-rizin-target-stdin
codex/ghidra-memory-effect-scope
codex/ghidra-annotation-entry-selector
codex/ghidra-readonly-recovery
codex/ghidra-regex-verifier-recovery
codex/evidence-discovery-view-guidance
fix/8545-bulk-append-overflow
fix/8428-navigation-refusal-alternatives
codex/address-context-recovery
codex/matching-member-limitations
codex/unknown-scope-contract
codex/evidence-view-guidance
feat/8373-native-trace-views
codex/ghidra-snapshot-verifier-binding
codex/elf-export-visibility
codex/evidence-view-recovery
codex/native-reference-cli-help
codex/provider-candidate-observation-scope
codex/kuna-session-verification
codex/ghidra-switch-lifetime-guidance
codex/pseudocode-paging-discovery
codex/clarify-evidence-view-support
codex/kuna-xrefs-selector-recovery
codex/context-capacity-recovery
codex/evidence-closure-large-links
codex/same-function-evidence-identity
codex/ghidra-verifier-inspected-jdk
codex/audit-guided-workflows
codex/crash-prompt-retained-dossiers
codex/analysis-view-error-recovery
codex/kuna-verifier-target-selection
codex/ghidra-selected-project-call-coverage
codex/docs-native-analysis-scope
codex/ghidra-java-socket-boundary
codex/ghidra-retained-snapshot-verifier
codex/ghidra-search-inventory-facts
fix/8284-zero-argument-parameter-bindings
fix/8198-ui-action-handler-offset
fix/8257-elf-entry-label-identity
fix/8304-deferred-output-schema-projection
fix/8277-call-path-computed-target-uncertainty
fix/pr-8089-conflicts
fix/ci-static-duplicate-export
dependabot/npm_and_yarn/lint-staged-17.6.0
dependabot/github_actions/actions/download-artifact-8.0.2
dependabot/npm_and_yarn/write-file-atomic-8.0.0
dependabot/npm_and_yarn/canonicalize-5.1.0
dependabot/github_actions/actions/cache/restore-6.1.0
dependabot/npm_and_yarn/toon-format/toon-4.4.0
dependabot/github_actions/actions/deploy-pages-5.0.1
codex/review-fixes-pr-2403
codex/review-fixes-pr-2400
codex/process-terminal-width
codex/test-brittleness-cleanup
verify/7167-browser-connect-macos
fix/7167-browser-connect
feat/native-matching-allowance-setting
feat/browser-scenario-dry-run
test/managed-caller-parameters-20261024
codex/parser-fortythird-boundaries
fix/6926-pinvoke-platform-decorations-20261023
fix/6889-har-body-encoding-20261023
fix/6888-sqlite-schema-budget-20261023
fix/dependency-security-gates-6708
cursor/evidence-overwrite-rebase-b4ac
feat/4057-verify-native-exports
fix/windows-acl-observer-overhead-20261022
cursor/erlang-beam-restack-4476
cursor/pe-certificate-review-bf32
fix/6116-javascript-application-fs-errors
fix/6061-unmapped-runtime-script-unknown
feat/browser-scenario-summary
fix/pe-certificates
codex/verify-3407
cursor/rebase-5699
cursor/rebase-5352
cursor/rebase-5600
cursor/fix-cli-schema-omission-10a8
codex/audit-hopper-shutdown-reasons-20261021
fix/demangle-swift-omitted-record
fix/nib-boolean-value-types
cursor/cancellation-partial-observation-a5ac
fix/ib-object-limit-reasons
fix/electron-fuse-limit-advice
fix/kuna-doctor-dangling-cache
fix/fixed-budget-refusal-advice
fix/js-ipc-listener-dedup
fix/js-return-member-provenance
fix/dmg-integrity-retry-advice
fix/ida-doctor-readiness
fix/kuna-version-probe-output-drain
fix/legacy-swift-runtime-name-matching
fix/dylib-lazy-load-commands
fix/browser-scenario-startup-cancel
fix/swift-nested-punycode-classification
fix/compare-unselected-scopes
fix/objc-method-budget-coverage
fix/dmg-mounted-root-observations
audit/ci-tests-20261019-round2
release/6.7.0
fix/wabt-cleanup-retry-resolution
fix/electron-fuse-elf-pe-architecture
fix/native-plist-descriptor-ownership
fix/kuna-caller-cleanup-ownership
experiment/atomic-codemode-20261016
review/first-contributor-3776
fix/call-resolution-full-read-cycles
fix/member-mutation-selection-repeat
experiment/atomic-re-tools-20261015
perf/member-mutation-lazy-origins
perf/static-network-mutation-index
fix/hopper-startup-lifecycle
audit/3050-windows-proof-20261015
feat/windows-window-capture-2183
audit/3049-windows-proof-20261015
audit/3735-macos-fixtures-20261015
fix/batch-decompile-null-pseudocode
fix/kuna-inventory-honors-jobs
experiment/browser-codemode-tool-design-20261015
fix/hopper-clean-runtime-argv
codex/review-2049-27f1b367
review-fixes/pr-2787-routing
fix/3360-scoped-this-emitters
fix/3367-replaced-global-effects
fix/3838-compact-any-value-items
test/hopper-download-probe-20261015
codex/windows-runtime-creation-red-green-20261015
codex/windows-readonly-root-red-green-20261015
codex/windows-root-share-red-green-20261015
codex/exp-1685-win-mcp-startup
codex/bug-sweep-20261015
investigate/2190-windows-package-phases
investigate/3543-macos-cli-deadline
perf/3281-reverse-tools-feedback-quality
fix/3406-hopper-doctor-fixture
fix/3341-wrapper-require-test-context
fix/3301-owned-uninstall-fixtures
fix/3164-compiler-input-ownership
fix/pcap-cleanup-stdout-newline-20261014
perf/interface-builder-summary
codex/review-fixes-pr-1977
fix/ghidra-effective-jvm-platform
fix/docs-turkish-rust-link-20261013
fix/main-maintenance-ci-20261010
fix/recover-binary-session-process-effects
codex/review-fixes-pr-2457
codex/ci-required-cancellation-gate
fix/main-format-baseline-20261013
fix/setup-preserve-entry-fields
cursor/restore-main-checks-1ffc
fix/ghidra-bridge-hashmap-import
codex/shared-ci-jadx-json-default-20261013
codex/shared-ci-ghidra-test-groups-20261013
codex/shared-ci-java-fixture-format-20261013
codex/fix-intrinsic-invocation-boundaries
test/release-preflight-reusable-jobs
test/js-parameter-provenance-expectation
fix/linux-hopper-verification-current
test/2419-ghidra-elf-probe-size
followup/pr-1544-native-ida
codex/fold-pr-2230-contribution
docs/readme-responsible-use
fix/issue-2064
fix/issue-2050
fix/issue-2053
fix/issue-1968
fix/issue-1945
experiment/codemode-bridge-20261011
fix/tool-effects-negative-wording-1904-20261012
fix/1842-electron-interop-provenance
feat/recover-javascript-multi-input
refactor/website-i18n-prep
fix/nativeaot-pe-virtual-extent
fix/browser-verifier-ci-hang
fix/pe-resources-input-selection
docs/beginner-friendly-guides
docs/beginner-setup-guide
fix/javascript-analysis-worker-1563
fix/1649-ghidra-duplicate-names
fix/javascript-export-keys-integration-1369
test/maintenance-boundary-contracts-20261011
review/javascript-export-keys-1369
review/process-live-progress-1483
test/javascript-1495-growth-ratio
fix/ci-escape-timing
docs/normalized-terminal-geometry-1450
release/6.3.0
archive/release-6.3.0-pr1381
release/6.2.0
fix/electron-preload-dynamic-override
fix/javascript-self-referential-bundler-edges
release/6.1.0
cursor/grok-client-cherry-pick-0b56
feat/signature-posture
feat/dyld-shared-cache
feat/artifact-integrity-policy
feat/pkg-inventory
release/6.0.0
chore/sync-release-5.0.0
release/5.0.0
cursor/cloud-agent-node-787e
fix/javascript-xhr-open-endpoints
review/pr-711-realpath-fallback-wording
cleanup/replay-confinement
codex/test-suite-performance
codex/issue-foundations
codex/address-open-issues-2026-07-20
agent/javascript-vite-rollup-esbuild-adapters
agent/process-cleanup
agent/triage-integration
agent/unknown-commands-registry
feat/mcp-identity-metadata
morlvps/issue_cleanup
codex/installer-redesign
morl/roadmap-investigation-core
morl/roadmap-investigation-docs
release-please--branches--main--components--rea
rea-agents-6.8.0
rea-agents-6.7.0
rea-agents-6.6.0
experiment/atomic-codemode-wait-runtime-20261016
rea-agents-6.5.0
rea-agents-6.4.0
rea-agents-6.3.0
rea-agents-6.2.0
rea-agents-6.1.0
rea-agents-6.0.0
rea-agents-5.0.0
rea-agents-4.1.0
rea-agents-4.0.1
rea-agents-4.0.0
rea-agents-3.2.1
rea-agents-3.2.0
rea-agents-3.1.0
rea-agents-3.0.0
rea-agents-2.7.0
rea-agents-2.6.0
rea-agents-2.5.1
rea-agents-2.5.0
rea-agents-2.4.0
rea-agents-2.3.0
rea-agents-2.2.0
rea-agents-2.1.0
rea-agents-2.0.0
rea-agents-1.7.0
rea-agents-1.6.0
rea-agents-1.5.0
rea-agents-1.4.0
rea-agents-1.3.0
rea-agents-1.2.0
rea-agents-1.1.0
rea-agents-1.0.0
rea-agents-0.5.0
rea-agents-0.4.0
rea-agents-0.3.0
rea-0.2.1
rea-0.2.0
Labels
Clear labels
area: bridge
area: bridge
area: build
area: build
area: build
area: cli
area: domain
area: evidence
area: mcp
area: native
area: providers
area: runtime
area: server
area: setup
area: tests
bug
design
documentation
enhancement
enhancement
enhancement
experiment
help wanted
i18n
invalid
P0: critical
P1: high
P2: medium
P2: medium
P2: medium
P3: low
performance
provider-request
pull-request
Mirrored from GitHub Pull Request
question
roadmap
triage
No labels
area: bridge
area: bridge
area: build
area: build
area: build
area: cli
area: domain
area: evidence
area: mcp
area: native
area: providers
area: runtime
area: server
area: setup
area: tests
bug
design
documentation
enhancement
enhancement
enhancement
experiment
help wanted
i18n
invalid
P0: critical
P1: high
P2: medium
P2: medium
P2: medium
P3: low
performance
provider-request
pull-request
question
roadmap
triage
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".
No due date set.
Dependencies
No dependencies set.
Reference
morluto/rea#1108
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @coygeek on GitHub (Oct 14, 2026).
Original GitHub issue: https://github.com/morluto/rea/issues/3172
Before submitting
Area
Evidence, artifacts, or reconstruction
Observed behavior
compare-web-capturesreports a network modification when otherwise identical passive page captures differ only because a redirect hop'sstatus,mime_type, orencoded_data_lengthis unavailable on one side. Reversing the inputs gives the same false modification. Comparing two captures with the same unavailable hop field instead reports the network dimension unchanged. Both results give callers more certainty about redirect behavior than the observations support.The independent verification ran 15 offline CLI comparisons at the revision below, and all exited 0. Each missing-versus-known hop-field comparison returned
normalized_result.dimensions.network.status: "changed",total_changes: 1, onemodifiedchange, andreason: null. Each missing-versus-missing hop-field comparison returned networkstatus: "unchanged", zero changes, andreason: null. Equivalent unavailable fields on the final response correctly returned network unknown with zero changes and an incomplete-response-evidence reason. A known redirect status change from 302 to 307 correctly returned changed.A focused localhost CDP fake also reproduced the producer-to-comparison path through real
inspect-web-pageandcompare-web-capturesCLI calls. Independently omitting status, MIME type, or byte length from the simulated redirect response produced a retained hop null and the same false modification. Missing byte length additionally markednetwork_requestsunavailable; that marker did not prevent the false modification. Missing status and MIME type did not produce that unavailable-section marker. This verifies simulated protocol omissions, not their frequency in real Chrome.Reproduction
Use a clean checkout at
58300885207133422af235e0f2b10658cf958b03. The comparison commands below were executed against its prepared runtime. Dependency installation and the build are setup instructions and were not rerun during verification.Expected behavior
docs/browser-observation.md:22states that passive comparison does not treat missing or truncated evidence as equivalence. The returned comparison limitation defined atsrc/domain/webCaptureDiff.ts:40promises that changed proves an observed difference. The rootAGENTS.md, Architecture and Cleanup, also requires preserving omission and uncertainty across boundaries.When no other difference is proven, a hop response field that is null on either side must remain unknown. The network dimension should return
status: "unknown", zero changes, and a reason identifying incomplete response evidence for null-versus-known and null-versus-null observations. Apply this rule to status, MIME type, and byte count while preserving genuine observed differences. Complete identical hops must still compare unchanged, and known status 302 versus 307 must still compare changed.Impact and scope
P2. This bounded passive-capture comparison defect can mislead an analyst into treating unavailable redirect observations as a proven modification or as equivalence. The verifier observed no data mutation, security boundary crossing, or broader workflow failure. The inputs are schema-valid, and the simulated CDP producer generated the same nullable fields through the public inspection path.
Raw capture inspection exposes the null values, but a complete comparison workflow that preserves the required redirect result was not demonstrated. CLI help provides no passive comparison option to exclude unavailable hop response fields. Deleting redirects from the input would discard relevant evidence. Real Chrome omission frequency was not measured.
REA version
6.4.0 (main at
5830088520)Runtime and platform
Node.js v26.10.0 on macOS. The verifier did not record the OS version or architecture. Locked dependencies and the built runtime were already prepared; dependency installation and build were not rerun during verification.
Target and artifact context
Local normalized passive page-capture JSON containing one final HTTP response and one ordered redirect hop. Every inline fixture passes
captureSnapshotSchema.parse. The example URLs are observation strings and are never requested by the offline reproduction. Additional producer verification used a localhost CDP fake with simulated redirect events.Interface
CLI
Earliest failure stage
Transport or result projection
Relevant output or evidence
The network dimension is the relevant result. Unrecorded WebMCP discovery independently makes the overall result unknown for otherwise identical fixtures, including the complete identical-hop control. Missing-versus-known hop fields still make the overall result changed.
Typed outcome or recovery expected
Return network unknown with zero proven changes and an explanatory reason when unavailable redirect response evidence is the only discrepancy. Preserve that uncertainty for both comparison directions and missing-versus-missing fields, including byte-length omissions with incomplete network coverage marked. Keep any separately proven change visible even if another hop field is unavailable. Complete identical observations must remain unchanged, and known non-null redirect response differences must remain changed.
Analysis or suspected cause
Source-derived at baseline
58300885207133422af235e0f2b10658cf958b03:src/domain/webCaptureDiff.ts:257-271derives jointly available response fields from final requests only.src/domain/webCaptureDiff.ts:366-373derives response incompleteness from that field set.src/domain/webCaptureDiff.ts:391-397includes redirect response objects in the comparison digest, excluding only the two event timestamps. The source therefore does not apply the final-response availability handling to redirect hops. This explanation agrees with the independently executed comparisons.src/domain/browserObservationSchemas.ts:135-149accepts nullable redirect response fields.src/browser/CdpCaptureEventHandlers.ts:282-301retains missing status, MIME type, and byte length as null, and records incomplete network coverage for unavailable or invalid byte length.tests/boundary/browser/webCaptureDiff.test.ts:731-768requires unavailable final-response status and MIME evidence to remain unknown, including null-versus-null observations.Additional context
Related to #1830, which fixed unavailable final-response metadata; this report concerns redirect-hop metadata that still produces false changes and false equivalence after that fix. The P2 rating reflects incorrect certainty for bounded comparison inputs without demonstrated broader harm.
Related: #3173 reports a separate failure in the same area; the two have independent fixes.
@CassieFox23 commented on GitHub (Oct 14, 2026):
Heads up: this looks fixed on main in
2b60e097f. The change is insrc/domain/webCaptureDiff.ts. It is listed on tracker #3582. If it still happens on main, please say so here and I'll take it off the list.🤖 Generated with Claude Code (Sonnet 5.5, verified by Opus 5.5) 🦊
@morluto commented on GitHub (Oct 14, 2026):
Resolved on main by
2b60e097f.Missing redirect response fields remain unknown; they no longer establish false known network evidence.
Rechecked on main
b8510ee00in the current focused run: 20 test files passed, 225 tests passed and one platform-dependent test skipped. Relevant regression coverage: tests/boundary/browser/webCaptureDiff.test.ts.