[Tracking] Deployed preview state — split-source, do not merge#94
Draft
alastairong1 wants to merge 5 commits intomainfrom
Draft
[Tracking] Deployed preview state — split-source, do not merge#94alastairong1 wants to merge 5 commits intomainfrom
alastairong1 wants to merge 5 commits intomainfrom
Conversation
Return the ABI-encoded order bytes in order list responses so the frontend can decode OrderV4 directly without a separate Raindex hydration round-trip. Matches albion.rest.api which already returns this. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…history Enables the frontend to fetch trades for a specific token instead of all trades globally. Uses GetOrdersTokenFilter to find orders where the token appears as input or output, then batch-fetches trades via DirectTradesFetcher with time-range filtering and pagination. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add max_output field to OrderSummary populated from the on-chain quote simulation's formattedMaxOutput. Falls back to output_vault_balance when no quote is available. This allows the frontend depth chart to show accurate per-epoch availability for DCA/strategy orders instead of the full vault balance. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The existing /health endpoint only returns {"status":"ok"} without
checking actual system health. The new GET /health/detailed endpoint
reports app DB connectivity, raindex DB connectivity, and per-orderbook
sync progress (last synced block, updated_at timestamp, latest trade
age). This enables diagnosing sync stalls like the Apr 17 stoppage.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit captures the working-tree state currently deployed to api.preview.st0x.io, prior to splitting it into single-topic PRs. Includes: - cache_warmer + supporting caches (block_number, limit_ratio, stale_price_skip, swap_quote) - market_calendar - ops tooling (docs/ops.md, scripts/smoke.sh, uptimerobot-setup.sh) - RPC override config - direct_trades_fetcher extensions - Health detailed endpoint additions for cache_warmer status - Various route + infra adjustments NOTE: lib/rain.orderbook submodule has local modifications that are not committed inside the submodule itself; the submodule pointer in this commit is unchanged. Those upstream changes need separate handling before any per-topic PR can build cleanly against main. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This was referenced Apr 28, 2026
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.
Purpose
This is a tracking PR, not for merge. It captures the full state currently deployed to https://api.preview.st0x.io so the per-topic PRs below can be reviewed against a single source of truth.
Once the per-topic PRs land in dependency order, this branch becomes redundant and the PR can be closed.
What's in the deployed branch
Built on top of
b1ebea3(the now-reverted direct push). Includes:Already committed (4 commits ahead of pre-revert main):
37596c4— order_bytes in OrderSummary → covered by Add order_bytes to OrderSummary API response #92 ✓ (force-pushed clean onto current main)99cd19a—GET /v1/trades/token/{addr}endpoint → blocked, see belowb3dba2f— simulated maxOutput from quote in OrderSummary → blocked, see below7a279a9— detailed health endpoint → covered by Add detailed health endpoint with raindex sync status #95 ✓ (clean cherry-pick)Snapshot commit on top (
e77b2a1):cache_warmerbackground task + supporting caches (block_number,limit_ratio,stale_price_skip,swap_quote)market_calendarfor NYSE-hours awarenessdocs/ops.md,scripts/smoke.sh,scripts/uptimerobot-setup.sh→ covered by Add ops cheat sheet and smoke/uptime monitoring scripts #96 ✓config/rest-api.toml,flake.nix,os.nix)direct_trades_fetcherextensions (~339 more lines)cache_warmerstatusPer-topic split — status
✅ Landed PRs (single-topic, clean against current main)
Add order_bytes to OrderSummary API response(force-pushed to a single commit)Add detailed health endpoint with raindex sync statusAdd ops cheat sheet and smoke/uptime monitoring scripts🟡 Already-open, in-flight PRs that need to land first (the reverted direct-push work)
Plus the upstream foundation already in flight by @findolor:
GET /v1/trades/tx/{tx_hash}endpointGET /v1/trades/{address}endpoint (depends on Implement GET /v1/trades/tx/{tx_hash} endpoint #50)🔴 Blocked work — needs the chain above to land before clean cherry-pick is possible
These commits exist on the deployed branch but cannot be cleanly cherry-picked onto current main because they depend on hundreds of lines of structural changes (caching foundation, trades module restructure, order endpoint refactors) carried by the in-flight PRs. Attempting cherry-pick produces large multi-file conflicts.
99cd19a→GET /v1/trades/token/{addr}— needs Implement GET /v1/trades/tx/{tx_hash} endpoint #50, Implement GET /v1/trades/{address} endpoint #51, Add in-process endpoint caching with moka #54, DirectTradesFetcher module (depends on #50, #51) #91, then trivial cherry-pickb3dba2f→ simulated maxOutput in OrderSummary — needs Order perf: SDK output-vault filter + quote timing #88, Order endpoint caching (depends on #54) #89, then trivial cherry-pickcache_warmer.rs+block_cache.rs+limit_cache.rs+stale_price_skip.rs+market_calendar.rs+ integrations intoorders/mod.rs/get_by_owner.rs/get_by_token.rs/main.rs/health.rs/types/health.rs) — needs Order perf: SDK output-vault filter + quote timing #88, Order endpoint caching (depends on #54) #89, Add in-process endpoint caching with moka #54, then split into ~3 PRs (cache-warmer-core, market-calendar, health-cache-status)routes/swap/quote.rscache integration +routes/swap/mod.rs) — needs Add in-process endpoint caching with moka #54config.rs,raindex/config.rs,config/rest-api.toml,flake.nixplumbing) — likely standalone, can be PR'd once a quiet moment existsdirect_trades.rsextensions (~339 additional lines beyond DirectTradesFetcher module (depends on #50, #51) #91) — needs DirectTradesFetcher module (depends on #50, #51) #91tokens.rs,admin.rs,db/pool.rs,test_helpers.rs) — small, can ride along with other PRslib/rain.orderbookhas local modifications not committed inside the submodule itself (~13 files modified, see snapshot commit message). The submodule pointer in this snapshot is unchanged at57253129e, so any per-topic PR depending on those upstream changes will fail to build until the submodule changes are landed (likely via a fork branch + pointer bump in this repo).Recommended merge order
cc @findolor for visibility and review.