Skip to content

Fix RPI depth snapshot bridging - #298

Merged
jackhai9 merged 1 commit into
mainfrom
codex/fix-rpi-depth-bridge
Sep 21, 2026
Merged

jackhai9 merged 1 commit into
mainfrom
codex/fix-rpi-depth-bridge

Conversation

@jackhai9

Copy link
Copy Markdown
Owner

Summary

  • accept the first RPI depth event when its pu exactly names the observed snapshot update ID
  • preserve standard U..u snapshot coverage and strict pu === previous.u checks for later non-stale events
  • reject descending update ID ranges before mutating book state
  • bump the orderbook userscript to 2.7.213 and rebuild the install artifact

Testing

  • npm test — 2087 passed on Node.js 24.16.0
  • npm run test:ui — 372 passed on Node.js 24.16.0
  • npm run lint:tests
  • npm run test:binance-orderbook-trade — 855 passed
  • npm run build:binance-orderbook-trade
  • npm run check:binance-orderbook-trade
  • npm run inspect:binance-orderbook-trade
  • git diff --check

An initial Playwright run used Node.js 26.8.2 and the seven coverage-merge cases were rejected by the repositories explicit Node.js 24.16.0 version guard. The complete suite then passed under the pinned runtime.

@jackhai9
jackhai9 merged commit d89445d into main Sep 21, 2026
0 of 2 checks passed
@jackhai9
jackhai9 deleted the codex/fix-rpi-depth-bridge branch September 21, 2026 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant