test: validate Polymarket v2 end-to-end environments - #67
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 46889d0042
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| exit 1 | ||
| fi | ||
| done < <( | ||
| jq -r '.contracts | to_entries[] | select(.value | type == "object" and has("repository")) | [ |
There was a problem hiding this comment.
Propagate jq failures from provenance extraction
When jq is unavailable or cannot parse a manifest, it runs inside a process substitution whose failure is not propagated by the surrounding while loop, even with set -euo pipefail. All three loops can therefore process zero records and the script still exits successfully with “provenance verified,” leaving the advertised source and ABI checks bypassed. Capture and validate each jq result before iterating, or otherwise explicitly wait for and check the producer status.
Useful? React with 👍 / 👎.
Summary
This targets
unaudited-pm-v2-oo-reporterbecause the fixtures validate the post-audit integration currently proposed in #65.Validation
forge test --no-match-contract '.*Fork.*' -vv— 83 passedPolymarketV2ManifestTest— 1 passedPolymarketV2VNetWiringForkTest— 3 passedPolymarketV2EndToEndForkTest— 2 passedPolymarketV2AmoyWiringForkTestat the pinned snapshot — 3 passedSafety and environment scope
All lifecycle mutations run inside Forge's disposable local EVM. The suite contains no broadcast path, signer, private key, service credential, or remote impersonation/write method.
Amoy passes the exact wiring and authorization subset. Its automatic lifecycle remains blocked because the deployed reporter is the base
OOReporterandOOReporterModulehas no selected operator.Linear: FRO-111