Skip to content

feat: add token resource for quick l1-l2 mapping, assetId querying, chain info#22

Merged
dutterbutter merged 9 commits intomainfrom
db/add-token-resource
Jan 5, 2026
Merged

feat: add token resource for quick l1-l2 mapping, assetId querying, chain info#22
dutterbutter merged 9 commits intomainfrom
db/add-token-resource

Conversation

@dutterbutter
Copy link
Contributor

What 💻

  • Adds token resource

Why ✋

  • Improves usability of token information relevant to bridging

Evidence 📷

Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

@github-actions
Copy link

github-actions bot commented Jan 5, 2026

ci-run-tests LCOV of commit 57f52dd during ci-run-tests #58

Summary coverage rate:
  lines......: 85.1% (10419 of 12248 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                    |Lines       |Functions  |Branches    
  Filename                                                          |Rate     Num|Rate    Num|Rate     Num
  ========================================================================================================
  src/adapters/ethers/e2e/erc20.e2e.ts                              | 0.0%    139|    -     0|    -      0
  src/adapters/ethers/resources/deposits/context.ts                 | 0.0%     37|    -     0|    -      0
  src/adapters/ethers/resources/deposits/index.ts                   | 0.0%    171|    -     0|    -      0
  src/adapters/ethers/resources/deposits/routes/erc20-base.ts       | 0.0%    111|    -     0|    -      0
  src/adapters/ethers/resources/deposits/routes/erc20-nonbase.ts    | 0.0%    156|    -     0|    -      0
  src/adapters/ethers/resources/deposits/routes/eth-nonbase.ts      | 0.0%     14|    -     0|    -      0
  src/adapters/ethers/resources/deposits/services/gas.ts            | 0.0%     46|    -     0|    -      0
  src/adapters/ethers/resources/tokens/index.ts                     | 0.0%      1|    -     0|    -      0
  src/adapters/ethers/resources/tokens/tokens.ts                    | 0.0%    189|    -     0|    -      0
  src/adapters/ethers/resources/utils.ts                            | 0.0%     46|    -     0|    -      0
  src/adapters/ethers/resources/withdrawals/context.ts              | 0.0%     40|    -     0|    -      0
  src/adapters/ethers/resources/withdrawals/index.ts                | 0.0%    227|    -     0|    -      0
  src/adapters/ethers/resources/withdrawals/routes/erc20-nonbase.ts | 0.0%    120|    -     0|    -      0
  src/adapters/ethers/sdk.ts                                        | 0.0%     14|    -     0|    -      0
  src/adapters/viem/resources/utils.ts                              | 0.0%     58|    -     0|    -      0
  src/adapters/viem/resources/withdrawals/context.ts                | 0.0%     48|    -     0|    -      0
  src/core/codec/ntv.ts                                             | 0.0%     10|    -     0|    -      0
  src/core/types/errors.ts                                          | 0.0%    127|    -     0|    -      0

Full coverage report

Copy link
Contributor

@sarahschwartz sarahschwartz left a comment

Choose a reason for hiding this comment

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

Nice! It looks like viem doesn't have the same token resources added, why is that?

@dutterbutter
Copy link
Contributor Author

Nice! It looks like viem doesn't have the same token resources added, why is that?

Will be added right after this PR is merged. Wanted to get these changes reviewed before duplicating for viem.

@dutterbutter dutterbutter enabled auto-merge (squash) January 5, 2026 22:15
@dutterbutter dutterbutter merged commit 09ee772 into main Jan 5, 2026
13 checks passed
@dutterbutter dutterbutter deleted the db/add-token-resource branch January 5, 2026 22:50
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