mirror of
https://github.com/morluto/rea.git
synced 2026-11-05 06:45:22 +00:00
[GH-ISSUE #4677] [Bug] Page inspection loses COOP and COEP policy tokens with valid report-to parameters #2439
Closed
opened 2026-10-17 21:04:41 +00:00 by bsakaguchi
·
3 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#2439
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 17, 2026).
Original GitHub issue: https://github.com/morluto/rea/issues/4677
Before submitting
Area
Process capture or runtime observation
Observed behavior
rea inspect-web-pagesucceeds but reportspolicies.coop: nullandpolicies.coep: nullfor responses carryingCross-Origin-Opener-Policy: same-origin; report-to="coop"andCross-Origin-Embedder-Policy: require-corp; report-to="coep". The bare-token controls returnsame-originandrequire-corp. A separate real CDP session records the full valid declarations, and Chrome accepts the parameterized response as a cross-origin-isolated top-level document.Reproduction
npm ci, then runnpm run build:cachedusing the pinned development toolchain. For the published build, runnpm install --prefix ./published rea-agents@6.5.0in a disposable directory, thencd published/node_modules/rea-agents. Both builds use the same relative launcher path below.REA_BROWSER_EXECUTABLE, or use the browser installed for the locked Playwright version. From the repository root or installed package directory, run this fixture. It creates a loopback server, a dedicated headless browser profile, and a fresh HOME for the CLI, then removes them.samples["/params"]with/bare,/invalid, and/absent. In each tested build, all three captured parameterized responses lost both policy tokens. Each control also produced three responses with consistent results. Separate top-level navigation returnedcrossOriginIsolated: truefor the valid parameterized and bare cases, and false for the malformed and absent cases.Expected behavior
Retain the declared
same-originandrequire-corptokens for these valid structured headers. The HTML Standard defines both headers as structured-field items whose bare value is a token and whosereport-toparameter may contain an endpoint-name string. The parameter does not invalidate the bare token. This expectation concerns the declaration captured in response metadata, not an inferred browser default or the document's full effective policy. See the COOP headers, COEP headers, and structured-field Items.Impact and scope
Valid response declarations become indistinguishable in these projected fields from absent or malformed declarations. The failure affects the parameterized input form while the CLI and the browser's isolation behavior continue to work. No change to browser enforcement or security-boundary crossing was observed.
REA version
Published npm package 6.5.0, confirmed as the latest release in the registry and GitHub Releases; independently built current
mainate31aa72c9d. Both launchers reported the package version shown above.Runtime and platform
Node v24.18.0; Darwin arm64; real headless Chrome/145.0.7632.6 matching the repository's locked playwright-core 1.58.2. The source build used npm 11.16.0 and the repository lockfile. Each CLI invocation had a fresh HOME and a minimal environment containing only HOME and PATH.
Target and artifact context
A self-owned loopback HTTP server provides valid parameterized, bare, malformed quoted-parameter, and absent-header fixtures. One local app fetches those responses during the CLI observation. Each fixture is also opened separately as a top-level document in the dedicated browser context to check browser acceptance. No external target or credentials were used.
Interface
CLI
Earliest failure stage
Capture or observation
Relevant output or evidence
The inline fixture invokes this production command with its discovered local endpoint and target ID:
In both tested builds, the command exited 0 with empty stderr. The raw CDP response contained these exact header strings:
The observed result pairs were:
In both builds, all four top-level document controls reported
isSecureContext: true. The minimal fixture output records the number of responses captured for each case. The browser check establishes acceptance of the chosen valid declarations; it does not change the meaning of REA's response-declaration projection.Typed outcome or recovery expected
For the valid parameterized response, return
coop: "same-origin"andcoep: "require-corp". Preserve the bare-token control results and keep malformed quoted parameters distinguishable from a valid structured item. The observed malformed and absent controls should continue to return null for this fixture. This restores the same declared-token result with and without the valid reporting parameter.Analysis or suspected cause
Source-derived reasoning: both policy fields pass the complete header string to a helper that accepts only a restricted bare token. The semicolon-delimited parameter therefore causes the entire declaration to become null. The standards represent the bare token and its parameters separately. The schema's nullable strings do not establish that null uniquely means the header was absent.
Additional context
A search of open and closed issues and pull requests found no matching report using COOP, COEP, both full header names, require-corp, policies.coop, policies.coep, and broader browser-policy queries. The relevant matches concern worker capture and Referrer-Policy, rather than this input form. Repository documentation and tests cover bare COOP/COEP values but do not specify bare-token-only input handling. The documented malformed-token behavior for Referrer-Policy concerns that different field. This reproduction exercises
inspect-web-page; screenshot comparison was not exercised. Removingreport-tois a discriminating control and would change server reporting behavior. No last-known-good release was established.@morluto commented on GitHub (Oct 17, 2026):
Published in
c360cfcd6don main.The old whole-value bare-token helper could not represent a Structured Field Item. COOP/COEP now use a dedicated complete-item parser: preserve the exact token, validate every parameter, and return null for malformed/non-token items. Strings/escapes, tokens, flags, integer/decimal limits, byte sequences, dates and UTF-8 display strings are validated. Other policy fields retain their own grammar. No reporting endpoint values are retained. This follows the HTML COOP, COEP and RFC9651 parsing rules. Returned tokens describe declarations, not the effective browser policy.
Executed the report's real Chrome/145.0.7632.6 public CLI fixture on baseline
df2fc251eand candidate, Darwin arm64/Node24.18.0, 800ms observation window. Each run captured three responses per case. Parameterized declarations changed from null/null to same-origin/require-corp for all three responses; bare, malformed-quote and absent controls were unchanged. Both CLI commands succeeded with empty stderr. Separate navigations observed crossOriginIsolated=true for valid parameterized/bare controls and false for malformed/absent controls, all with isSecureContext=true. The fixture closed its browser/server and removed its dedicated workspace.Validation:
Other browser/platform combinations, policy-specific MCP runtime delivery, performance/full-task tokens and peak-family memory remain unmeasured. No efficiency claim is made. Repeated-field Link behavior from #4679 remains covered. The broader browser/codebase audit remains partial.
@coygeek commented on GitHub (Oct 18, 2026):
Closing as addressed on
mainby c360cfcd6. COOP and COEP now use complete Structured Field Item parsing, preserving valid declared tokens with report-to parameters while leaving malformed declarations unknown. I verified the committed source and regression coverage; I have not rerun the runtime reproduction.@matbrgz commented on GitHub (Oct 20, 2026):
Cross-referencing with sibling HTTP response header parsing defect #4679 (
[Bug] Page inspection drops repeated Link fields from Document responses). Both address header normalization and security token handling (COOP, COEP, Link) ininspect-web-page.Consolidated under Cluster AP (Web Page Inspection HTTP Header Parsing: Link, COOP, COEP) in tracker #4330.
trace_android_references: find APK callers of a framework or library method that the APK doesn't define #4647