Skip to content

feat: Christmas tooling update 🎅🏼 #2295

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
0xKheops merged 76 commits into from
Jan 13, 2026
Merged

feat: Christmas tooling update 🎅🏼 #2295

0xKheops merged 76 commits into from
Jan 13, 2026

Conversation

@0xKheops
Copy link
Contributor

@0xKheops 0xKheops commented Jan 9, 2026 •
edited
Loading

This PR replaces our test/build pipeline:

  • Before: preconstruct, turbo, webpack (+x plugins), jest, prettier (+ x plugins), eslint (+ x plugins)
  • After: tsup, wxt, vitest, biome

Important:

  • WXT dev mode does not support firefox MV3 yet, so MV3 is only used for prod builds, while MV2 is used for firefox dev builds.
  • Biome scripts have the --changed flag to process only the flles that are changed compared to dev branch. this is to avoid a big bang PR where all files would be reformatted. this can be done later when there are no other pending PRs in the repo

⚠️ Because we need ES Modules support:

  • new minimum chrome version: 121 (January 23, 2024)
  • new minimum firefox version: 128 (July 9, 2024)

Note: hot reloading (rebuilding) the background script is pretty long for now, like 10+ seconds. it can be improved in another "big bang refactor" PR after this one is merged. it will require separating code that is background-only and code that needs to be shared by frontend and background, in 2 separate packages, so wxt can treeshake background script more efficiently.

Misc:

  • installed knip and removed 35 unused dependencies and 40 unused files with it. we will be able to integrate knip checks in the CI if we wish to, in the future.
  • updated CI to run on node 24 instead of 18
  • updated caniuse list which was 15 months old
  • added a global pnpm typecheck script, which is now included in CI
  • added a global pnpm check that performs biome checks on files that are different from target branch
  • packages dont need to be built anymore, except when publishing them

@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026 •
edited
Loading

🦋 Changeset(s) detected

This PR includes changeset(s) for the following changed packages:

"@talismn/balances"
"@talismn/balances-react"
"@talismn/chain-connectors"
"@talismn/chaindata-provider"
"@talismn/connection-meta"
"@talismn/crypto"
"@talismn/icons"
"@talismn/keyring"
"@talismn/on-chain-id"
"@talismn/orb"
"@talismn/sapi"
"@talismn/scale"
"@talismn/solana"
"@talismn/token-rates"
"@talismn/util"

Not sure what this means? Click here to learn what changesets are.

@0xKheops 0xKheops marked this pull request as draft January 9, 2026 14:50
@socket-security
Copy link

socket-security bot commented Jan 9, 2026 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Updated@​types/​pako@​2.0.3 ⏵ 2.0.410010071 +181100
Added@​vitest/​coverage-v8@​3.2.4991007299100
Added@​types/​sinon-chrome@​2.2.16821007277100
Addeddexie-react-hooks@​1.1.71001007487100
Addeddate-fns@​4.1.0981009280100
Added@​vitest/​ui@​3.2.4991008299100
Addedcolor@​4.2.310010010084100
Updated@​rollup/​plugin-replace@​2.4.2 ⏵ 6.0.3100 +110010085100
Addeddompurify@​3.2.6991001008690
Added@​sentry/​vite-plugin@​4.6.110010010089100
Addeddexie@​4.0.910010010090100
Added@​vitejs/​plugin-react@​4.7.010010010092100
Updated@​biomejs/​biome@​1.9.4 ⏵ 2.3.11100 +710010098100

View full report

@socket-security
Copy link

socket-security bot commented Jan 9, 2026 •
edited
Loading

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

@UrbanWill UrbanWill self-requested a review January 12, 2026 06:04
UrbanWill
UrbanWill requested changes Jan 12, 2026
View reviewed changes
Copy link
Contributor

@UrbanWill UrbanWill left a comment

Choose a reason for hiding this comment

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

Ho Ho Ho Talisman engineering xmas is finally here! 🎅

I am very happy to see these changes going in, code looks good and everything works as expected.

Only one minor thing, import warnings. Eg: Libs like "@talismn/util" or "@talismn/icons" and some type imports here and there. Perhaps a biome config missing?

UrbanWill
UrbanWill previously approved these changes Jan 12, 2026
View reviewed changes
Copy link
Contributor

@UrbanWill UrbanWill left a comment

Choose a reason for hiding this comment

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

Previous issues fixed by running pnpm build:packages

0xKheops added 20 commits January 12, 2026 18:49
@0xKheops 0xKheops merged commit 250839f into dev Jan 13, 2026
7 checks passed
@0xKheops 0xKheops deleted the feat/wxt-vite branch January 13, 2026 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@UrbanWill UrbanWill UrbanWill left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@0xKheops @UrbanWill