-
Notifications
You must be signed in to change notification settings - Fork 0
refactor: ♻️ Uncouple HyperLiquid & Associated code #69
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR removes Hyperliquid and CCTP integration from the oracle system to simplify the codebase while preserving the ability to restore this functionality later. A comprehensive checklist guides future re-enablement.
- Removed
hyperliquid-python-sdkdependency and associated configuration fields - Disabled Hyperliquid and CCTP adapters with runtime checks that provide clear error messages
- Updated CLI, documentation, and example configs to reflect the removal
Reviewed Changes
Copilot reviewed 27 out of 29 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
uv.lock, pyproject.toml |
Removed hyperliquid-python-sdk dependency |
src/tq_oracle/settings.py |
Removed Hyperliquid/CCTP environment toggles, RPC endpoints, and derived value logic |
src/tq_oracle/main.py |
Replaced HL-specific CLI flags with vault-focused options; added ignore flags for pre-checks |
src/tq_oracle/adapters/asset_adapters/hyperliquid.py |
Added optional dependency handling with runtime checks instead of hard import |
src/tq_oracle/adapters/asset_adapters/idle_balances.py |
Updated to use getattr() for accessing removed config fields with fallback error messages |
src/tq_oracle/adapters/asset_adapters/__init__.py |
Removed Hyperliquid adapter from registry |
src/tq_oracle/adapters/check_adapters/cctp_bridge.py |
Added graceful skipping when HL config fields are absent |
src/tq_oracle/pipeline/assets.py |
Changed create_adapter_task to return None for disabled adapters, filtering them out |
tq-oracle.toml.example |
Removed Hyperliquid configuration examples |
tests/**/*.py |
Removed Hyperliquid-related test parameters and disabled HL/CCTP test suites |
docs/hyperliquid-cctp-restore-checklist.md |
Added comprehensive re-enablement checklist |
README.md, ARCHITECTURE.md |
Updated documentation to reflect removal |
.gitignore |
Changed to exclude all .toml files except pyproject.toml |
.github/workflows/CI.yml |
Added integration test and live-run jobs; renamed unittest to unit-test |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 32 out of 34 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 32 out of 34 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
As hyperliquid won't be connected to the first version of the vault, to simplify operations and audit we should uncouple all mentions of hyperliquid.
This refactor reduces the amount of cli options, config, and associated checks.
We have also included a checklist to run through to re-enable when it is time to do so.
New CI's for testing have also been added