Skip to content

FEATURE: [exchange][hyperliquid] init hyperliquid exchange#2259

Draft
anywhy wants to merge 16 commits intoc9s:mainfrom
anywhy:feat_exchange_hyperliquid
Draft

FEATURE: [exchange][hyperliquid] init hyperliquid exchange#2259
anywhy wants to merge 16 commits intoc9s:mainfrom
anywhy:feat_exchange_hyperliquid

Conversation

@anywhy
Copy link
Contributor

@anywhy anywhy commented Oct 12, 2025

No description provided.

@anywhy anywhy requested a review from c9s as a code owner October 12, 2025 12:50
@anywhy anywhy marked this pull request as draft October 12, 2025 12:50
@anywhy anywhy changed the title FUTURE: [exchange][hyperliquid] init hyperliquid exchange WIP:FUTURE: [exchange][hyperliquid] init hyperliquid exchange Oct 12, 2025
@anywhy anywhy changed the title WIP:FUTURE: [exchange][hyperliquid] init hyperliquid exchange WIP:FEATURE: [exchange][hyperliquid] init hyperliquid exchange Oct 12, 2025
@codecov
Copy link

codecov bot commented Oct 12, 2025

Codecov Report

❌ Patch coverage is 51.41243% with 344 lines in your changes missing coverage. Please review.
✅ Project coverage is 20.03%. Comparing base (5b0275f) to head (96b1c35).
⚠️ Report is 160 commits behind head on main.

Files with missing lines Patch % Lines
pkg/exchange/hyperliquid/exchange.go 36.51% 136 Missing and 17 partials ⚠️
pkg/exchange/hyperliquid/hyperapi/client.go 70.52% 33 Missing and 18 partials ⚠️
pkg/exchange/hyperliquid/convert.go 82.19% 24 Missing and 2 partials ⚠️
pkg/exchange/hyperliquid/hyperapi/types.go 0.00% 22 Missing ⚠️
...pi/futures_get_meta_and_asset_ctxs_request copy.go 0.00% 15 Missing ⚠️
...d/hyperapi/spot_get_meta_and_asset_ctxs_request.go 0.00% 15 Missing ⚠️
pkg/testutil/auth.go 0.00% 9 Missing ⚠️
pkg/exchange/hyperliquid/futures.go 82.85% 3 Missing and 3 partials ⚠️
...hange/hyperliquid/hyperapi/cancel_order_request.go 0.00% 4 Missing ⚠️
...id/hyperapi/futures_get_account_balance_request.go 0.00% 4 Missing ⚠️
... and 10 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2259      +/-   ##
==========================================
+ Coverage   19.25%   20.03%   +0.78%     
==========================================
  Files        1070     1093      +23     
  Lines       71451    73247    +1796     
==========================================
+ Hits        13759    14677     +918     
- Misses      56415    57201     +786     
- Partials     1277     1369      +92     
Files with missing lines Coverage Δ
pkg/exchange/hyperliquid/types.go 100.00% <100.00%> (ø)
pkg/types/exchange.go 52.38% <ø> (ø)
pkg/types/interval.go 54.02% <ø> (ø)
pkg/types/order.go 1.02% <ø> (-0.05%) ⬇️
pkg/exchange/factory.go 0.00% <0.00%> (ø)
...hange/hyperliquid/hyperapi/cancel_order_request.go 0.00% <0.00%> (ø)
...id/hyperapi/futures_get_account_balance_request.go 0.00% <0.00%> (ø)
...e/hyperliquid/hyperapi/futures_get_meta_request.go 0.00% <0.00%> (ø)
...yperliquid/hyperapi/get_account_balance_request.go 0.00% <0.00%> (ø)
...change/hyperliquid/hyperapi/get_candles_request.go 0.00% <0.00%> (ø)
... and 14 more

... and 43 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 75b9bc5...96b1c35. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@anywhy anywhy force-pushed the feat_exchange_hyperliquid branch from 4fbb076 to ac09965 Compare October 14, 2025 05:29
@anywhy anywhy changed the title WIP:FEATURE: [exchange][hyperliquid] init hyperliquid exchange FEATURE: [exchange][hyperliquid] init hyperliquid exchange Oct 14, 2025
@anywhy anywhy marked this pull request as ready for review October 14, 2025 05:32
@anywhy anywhy force-pushed the feat_exchange_hyperliquid branch from ac09965 to 9ab74a6 Compare October 14, 2025 06:22
size string `param:"s"`
price string `param:"p"`
reduceOnly bool `param:"r"`
clientOrderID *string `param:"c"`
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is not supported -- param tag inside the field with slice

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, it really doesn’t support that. Any idea how to deal with this kind of param?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wrap it with a struct, and add String() method to marshal the internal payload into json string? I am not sure

is it a nested json data in the request?

@anywhy anywhy force-pushed the feat_exchange_hyperliquid branch 7 times, most recently from 0b17089 to 6dbf5f4 Compare October 15, 2025 16:59
@c9s c9s added exchange exchange related issues feature feature means adding new functionality for users from all perspective, e.g., web, cli, deployment labels Oct 27, 2025
@anywhy anywhy force-pushed the feat_exchange_hyperliquid branch 3 times, most recently from b281b82 to 5b1a689 Compare October 29, 2025 15:10
@anywhy anywhy force-pushed the feat_exchange_hyperliquid branch 2 times, most recently from cdfe99f to f54631d Compare November 9, 2025 03:56
@anywhy anywhy force-pushed the feat_exchange_hyperliquid branch from f54631d to d76bef3 Compare November 9, 2025 08:12
@anywhy anywhy force-pushed the feat_exchange_hyperliquid branch 3 times, most recently from bf7b6e9 to d98867d Compare November 14, 2025 14:21
@anywhy anywhy force-pushed the feat_exchange_hyperliquid branch from 4eecddb to 4d96cbd Compare November 17, 2025 13:50
@c9s
Copy link
Owner

c9s commented Jan 11, 2026

@anywhy please ping me on telegram when you think it's ready

@anywhy
Copy link
Contributor Author

anywhy commented Jan 12, 2026

@anywhy please ping me on telegram when you think it's ready

Okay, I might be late

@anywhy anywhy marked this pull request as draft January 12, 2026 09:27
@anywhy anywhy force-pushed the feat_exchange_hyperliquid branch 2 times, most recently from 9c8c691 to d4fd4d5 Compare January 18, 2026 14:44
@anywhy anywhy force-pushed the feat_exchange_hyperliquid branch from d4fd4d5 to 96b1c35 Compare March 7, 2026 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exchange exchange related issues feature feature means adding new functionality for users from all perspective, e.g., web, cli, deployment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants