Skip to content

feat(tempo): add burnAt token role - #497

Merged
jxom merged 2 commits into
wevm:mainfrom
decofe:tempo-burn-at-role
Sep 30, 2026
Merged

jxom merged 2 commits into
wevm:mainfrom
decofe:tempo-burn-at-role

Conversation

@decofe

@decofe decofe commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Adds the TIP-1006 burnAt role to TokenRole. The role maps to BURN_AT_ROLE and activates in T12 (tempoxyz/tempo#7935).

Holders of BURN_AT_ROLE can burn from any unprotected account without a transfer-policy check. Once this role is part of the TokenRole union, viem's token.grantRoles, hasRole and related actions can accept 'burnAt'.

Also fixes the core test job for pull requests from forks. GitHub doesn't pass repository secrets to fork PRs, so VITE_ANVIL_FORK_URL arrives as an empty string. getEnv treated that as set, so anvil started without a usable fork URL and the AbiEvent behavior: network tests failed with an HTTP 400. getEnv now treats an empty value as unset and falls back to the public RPC.

Validation: pnpm test --project tempo-unit src/tempo/TokenRole.test.ts, vp check, and pnpm check:types all pass.

Prompted by: @max-digi

Co-authored-by: decofe <256792747+decofe@users.noreply.github.com>
@vercel

vercel Bot commented Sep 30, 2026

Copy link
Copy Markdown

@max-digi is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

@pkg-pr-new

pkg-pr-new Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/ox@497

commit: cd0ec2d

Co-authored-by: decofe <256792747+decofe@users.noreply.github.com>
@jxom
jxom merged commit ee2bce9 into wevm:main Sep 30, 2026
13 of 14 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