Repository navigation
bring bench 001 on dev up to what production runs - #2808
Merged
Merged
Conversation
Eight PRs of bench 001 work went straight to main this week on "mets live
now", and the fast-forward back to dev that AGENTS.md prescribes after a
hotfix was never done. The two branches have since diverged, so a
fast-forward is no longer possible: dev carries its own merges of the same
work under different SHAs plus commits main does not have.
This ports the one bench, by file, rather than reconciling twenty commits.
Every line it removes from dev is a line main deliberately replaced:
Timestamp string dev still types the OKX millis field as a
string, which is the bug #2795 fixed: the
WebSocket sends a JSON number and 136 of 136
frames were rejected
plain websocket.Dialer replaced by the pinned Chrome ClientHello in
#2802, without which Cloudflare 403s every
dial from Railway
the 403 retry loop #2799 gave it a circuit breaker after it
hammered a third party from three regions
TestOKXDoesNotCoverBNB re-derived as a coupling in #2805
The spec comes with it, which is the operational reason this cannot wait:
the materialize worker clones dev at BUILD time, so until this lands a
worker rebuild would pick up a spec that still says OKX and Birdeye are
measured on Base and Solana only.
Verified on dev's base, not main's: go build ./... and
go test ./cmd/script/ -run 'Birdeye|OKX|Reference|Chain' pass, pnpm validate
reports 235 specs valid.
The wider divergence is untouched and still needs a decision.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Flotapponnier
added a commit
that referenced
this pull request
Oct 4, 2026
Same port as #2808, for the same reason: the materialize worker clones dev at BUILD time, so the two new methodology bullets only reach the page from here. Carries the harness change with the spec, so dev does not end up describing an eligibility rule its own race.go does not apply. Verified on dev's base: go build ./..., the full go test ./cmd/script/ suite, and pnpm validate (235 specs valid). Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.
Prerequisite for the materialize worker rebuild: it clones
devat BUILD time, so until this lands a rebuild picks up a spec that still says OKX and Birdeye are measured on Base and Solana only.Why this is a file port and not a fast-forward
Eight PRs of bench 001 work went straight to
mainthis week, and the fast-forward back todevthat AGENTS.md prescribes after a hotfix was never done. The branches have since diverged in both directions, so a fast-forward is no longer available:devcarries its own merges of the same work under different SHAs (#2791, #2797, #2792) plus commitsmainlacks (#2803, #2793, #2796).Rather than reconcile twenty commits, this ports one bench by file.
Every line it removes from dev is one main deliberately replaced
Checked before porting, not assumed:
Timestamp stringwebsocket.DialerTestOKXDoesNotCoverBNBNothing on
devis lost: no file in the port exists only there, andmainis strictly ahead on all six.Verified on dev's base, not main's
go build ./...andgo test ./cmd/script/ -run 'Birdeye|OKX|Reference|Chain'pass;pnpm validatereports 235 specs valid.Still open
The wider
dev/maindivergence is untouched and needs a decision: reconcile it, or accept thatmainleads and port by file each time.🤖 Generated with Claude Code