Add TSM, CEG feeds.#8
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughThis PR removes unused TWAP functionality from the Hermes client, enhances the price listener with startup synchronization and improved freshness validation, improves EVM gas handling, containerizes the price-pusher application via Docker, expands the price configuration, and adjusts development environment setup for cross-platform compatibility. ChangesPrice Pusher Docker Deployment and Improvements
Sequence Diagram(s)sequenceDiagram
participant Startup as EVM Startup
participant Listener as PythPriceListener
participant Hermes as Hermes Stream
participant EVM as EVM Client
participant ContractSim as Contract Simulation
Startup->>Listener: start()
Listener->>Listener: await startListening()
Listener->>Hermes: subscribe to price updates
Hermes-->>Listener: onMessage(priceUpdate)
Listener->>Listener: validate publish_time (24hr freshness)
Listener->>Listener: store in latestPriceInfo
Startup->>Listener: waitForFirstPriceUpdate(15s)
Listener-->>Startup: true (got first update)
Startup->>EVM: initialize client
Startup->>ContractSim: begin updatePriceFeed loop
ContractSim->>ContractSim: compute gasPrice, gasLimitToUse
ContractSim->>ContractSim: log Initial/Final gas price
ContractSim-->>ContractSim: simulation complete
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
Add TSM, CEG feeds.
Summary by CodeRabbit
Breaking Changes
Improvements