ProjectOpenSea / opensea-js / 16659921530

88%

master: 30%

push

github

feat: add validateOrderOnchain method for Seaport integration (#1729)

* feat: add validateOrderOnchain method for Seaport integration

This adds a new method to validate orders onchain using Seaport's validate() method, which makes orders cheaper to fulfill since signature verification is skipped during fulfillment.

- Add validateOrderOnchain method to OpenSeaSDK
- Include comprehensive JSDoc documentation explaining benefits
- Add integration test for Polygon network using existing test infrastructure
- Bump package version to 7.2.1

The method submits orders onchain and pre-validates them using Seaport. While not strictly required (orders can be submitted to the API for free), this approach reduces gas costs during fulfillment at the expense of upfront validation gas costs.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* fix: handle domain parameter properly in validateOrderOnchain

Fixes TypeError when domain parameter is undefined by conditionally
calling the validate method with or without the domain parameter.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* feat: add validateOrderOnchain method for Seaport integration

- Add validateOrderOnchain method to SDK for onchain order validation
- Method calls seaport.validate() to submit orders onchain for pre-validation
- Add integration test for Polygon network with proper test flow
- Remove domain parameter and signature validation per requirements
- Bump package version to 7.2.1

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* fix: type casting for OrderWithCounter to OrderV2 conversion

* refactor: DRY order building logic and add convenience methods

- Remove submitOrder parameter from createListing/createOffer
- Extract order building logic into reusable _buildListingOrder/_buildOfferOrder helpers
- Update validateOrderO... (continued)

108 of 377 branches covered (28.65%)

Branch coverage included in aggregate %.

4 of 44 new or added lines in 1 file covered. (9.09%)

611 of 1070 relevant lines covered (57.1%)

1.68 hits per line