Skip to content

Conversation

@prestwich
Copy link
Member

Summary

  • Add bundle types (EthBundle, EthCall, CallBundle) matching the Rust SDK for cross-implementation compatibility
  • Add feasibility checking utilities (checkOrderFeasibility, checkFillFeasibility) for validating orders and fills
  • Add enhanced nonce utilities (splitNonce, joinNonce, incrementNonce)
  • Add Permit2 ABI for token approval handling
  • Add comprehensive test coverage including test vectors generated from the Rust SDK

Test plan

  • All 174 unit tests passing
  • ESLint clean
  • TypeScript type check clean
  • Bundle serialization matches Rust SDK test vectors

🤖 Generated with Claude Code

@prestwich prestwich requested a review from rswanson as a code owner February 3, 2026 21:14
@prestwich prestwich force-pushed the feature/sdk-gap-analysis branch 4 times, most recently from 3a2679e to 6a9ac37 Compare February 3, 2026 21:44
Add bundle types and serialization:
- EthBundle, EthCall, CallBundle types matching Rust SDK
- JSON serialization functions for bundle interchange
- Test vectors generated from Rust SDK for cross-impl verification

Add feasibility checking utilities:
- checkOrderFeasibility for basic order validation
- checkFillFeasibility for fill validation against orders
- Detailed error types for each validation failure

Additional improvements:
- Enhanced nonce utilities (splitNonce, joinNonce, incrementNonce)
- Builder pattern tests for order and fill construction
- Edge case tests for validation and encoding
- Permit2 ABI for token approval handling
- Expanded Anvil integration tests

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@prestwich prestwich force-pushed the feature/sdk-gap-analysis branch from 6a9ac37 to b58f8d0 Compare February 4, 2026 17:43
On-chain operation wrappers that every Signet dApp needs:
- passage: enter() and enterToken() for Host to Signet bridging
- weth: wrapEth() and unwrapEth() for WETH deposit/withdraw
- permit2: getPermit2Allowance(), approvePermit2(), ensurePermit2Approval()
  with USDT-style reset-to-zero handling

Also adds USDC/USDT addresses to Parmigiana token registry.
@prestwich prestwich enabled auto-merge (squash) February 5, 2026 14:55
@prestwich prestwich disabled auto-merge February 5, 2026 14:55
@prestwich prestwich merged commit 1caf84c into main Feb 5, 2026
5 checks passed
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