Skip to content

feat(perps): report provider trigger order capabilities - #10629

Closed
abretonc7s wants to merge 11 commits into
fix/lighter-device-key-recoveryfrom
feat/lighter-advanced-orders
Closed

abretonc7s wants to merge 11 commits into
fix/lighter-device-key-recoveryfrom
feat/lighter-advanced-orders

Conversation

@abretonc7s

@abretonc7s abretonc7s commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Explanation

Clients currently infer stop-loss and take-profit support from the protocol name.

Add supportedTriggerOrderTypes to provider capabilities so clients can use the provider's declaration. Hyperliquid reports its four existing types; an omitted declaration means support is unknown.

Capability tests, public API checks and build pass. Lighter support follows in #10638.

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and prepared draft consumer PRs

@abretonc7s
abretonc7s force-pushed the feat/lighter-advanced-orders branch from 568b2c8 to 64e799f Compare September 30, 2026 17:49
@abretonc7s
abretonc7s marked this pull request as ready for review September 30, 2026 18:43
@abretonc7s
abretonc7s requested review from a team as code owners September 30, 2026 18:43
@abretonc7s
abretonc7s force-pushed the fix/lighter-device-key-recovery branch from 797e923 to fc726f1 Compare October 1, 2026 06:42
@abretonc7s
abretonc7s force-pushed the feat/lighter-advanced-orders branch from 10e9a4e to 4e2f2cd Compare October 1, 2026 06:42
@abretonc7s
abretonc7s force-pushed the feat/lighter-advanced-orders branch from 4e2f2cd to 18ae053 Compare October 1, 2026 10:33

@geositta geositta left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approve. HL four declared types match its existing order building paths.

@abretonc7s
abretonc7s requested a review from geositta October 2, 2026 06:34
@abretonc7s

Copy link
Copy Markdown
Contributor Author

Consolidated into #10638 to simplify review. The combined branch includes all changes from this PR and depends directly on recovery #10618.

@abretonc7s abretonc7s closed this Oct 2, 2026
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.

2 participants