Skip to content

Add EVM TypeScript vault depositor SDK#3

Merged
ihsraham merged 9 commits into
masterfrom
feat/evm-ts-deposit-spec
Jun 22, 2026
Merged

Add EVM TypeScript vault depositor SDK#3
ihsraham merged 9 commits into
masterfrom
feat/evm-ts-deposit-spec

Conversation

@ihsraham

@ihsraham ihsraham commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

As decided, this PR is EVM-only. It adds the TypeScript SDK surface for EVM vault deposits and deposit verification, with native ETH and ERC-20 flows.

Changes

  • Added the ESM TypeScript package entrypoint, shared result/error types, and EVM vault depositor implementation.
  • Added native ETH and ERC-20 deposit handling with strict input validation, chain checks, receipt handling, and stable error codes.
  • Added unit tests, Anvil integration tests, TypeScript CI wiring, and npm package/build checks.
  • Added a local EVM deposit demo UI for wallet-based submit and verify flows.

Demo

ezgif-472631484026e943

Checks

  • npm run typecheck
  • npm test
  • npm run build
  • npm --workspace @yellow-org/evm-deposit-demo run build
  • npm run test:integration:evm
  • npm audit --audit-level=low
  • npm pack --dry-run
  • git diff --check

@ihsraham ihsraham marked this pull request as ready for review June 18, 2026 15:22
@ihsraham ihsraham requested review from dimast-x and philanton June 18, 2026 15:23

@philanton philanton 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.

I think it'd be better to have everything related to ts under sdk/ts like in nitrolite

@ihsraham

Copy link
Copy Markdown
Contributor Author

Addressed in 127f318: moved the TS SDK package, tests, lockfile, and demo under sdk/ts, with CI updated to run from there.

Comment thread sdk/ts/README.md Outdated
@ihsraham ihsraham force-pushed the feat/evm-ts-deposit-spec branch from 56ebd42 to f2b6f7b Compare June 19, 2026 11:19
Comment thread sdk/ts/README.md Outdated
@dimast-x

Copy link
Copy Markdown
Collaborator

Feel free to merge once @philanton approves

Comment thread .github/workflows/test-ts.yml Outdated
Comment thread sdk/ts/package.json Outdated
Comment thread Makefile Outdated
Comment thread sdk/ts/README.md Outdated
```sh
# From the repository root:
make devnet
npm --prefix sdk/ts ci

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.

you can add it as dependency to Make file as well)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated in 37ceff2: make integration now depends on ts-deps, which runs npm --prefix sdk/ts ci, and the README no longer asks users to run it separately.

@ihsraham ihsraham merged commit 10572b8 into master Jun 22, 2026
3 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.

3 participants