Skip to content
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.

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

[PRE-RELEASE] - Update wagmi/viem/react-query versions and fix issues #100

Merged
merged 20 commits into from
Sep 3, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
832e7bd
fix(wagmi-connectors): remove custom PstlWeb3AuthConnector in favour …
W3stside Sep 3, 2024
9c518e5
fix(web3-modal): fixes from viem/wagmi update
W3stside Sep 3, 2024
f7e9605
fix(skillforge-widget): fixes from viem/wagmi update
W3stside Sep 3, 2024
a1bbdf0
fix(forge-web3): fixes from viem/wagmi update
W3stside Sep 3, 2024
e321492
fix(skillforge-ui): viem/wagmi update fixes
W3stside Sep 3, 2024
d754e0e
chore(root,skillforge-ui,forge-web3,skillforge-widget,wagmi-connector…
W3stside Sep 3, 2024
9eadcdb
chore: update yarn.lock
W3stside Sep 3, 2024
c9f8dd7
fix(skillforge-ui): update craco and rewire overrides
W3stside Sep 3, 2024
8f86861
fix(skillforge-ui): fix fixture
W3stside Sep 3, 2024
917a45f
fix(web3-modal): disable focus lock
W3stside Sep 3, 2024
11dfc7c
fix(web3-modal): fix fixed import path
W3stside Sep 3, 2024
5355282
styles(web3-modal): mobile updates
W3stside Sep 3, 2024
200c392
chore(skillforge-ui): set satisfying type to config obj
W3stside Sep 3, 2024
ed1ebe4
styles(web3-modal): remove magic number
W3stside Sep 3, 2024
f038330
chore(skillforge-widget): fix cosmos test env webpack
W3stside Sep 3, 2024
d259a2f
fix(web3-modal): disable structural sharing [to be reverted]
W3stside Sep 3, 2024
2931036
chore(forge-web3,web3-modal,skillforge-ui,root): fix webpack config f…
W3stside Sep 3, 2024
7ecb6f6
chore(forge-web3,wagmi-connectors): lint
W3stside Sep 3, 2024
5afb19b
chore: update yarn.lock
W3stside Sep 3, 2024
ee9835b
chore: revert @tanstack/react-query update
W3stside Sep 3, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions apps/skillforge-ui/config-overrides.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ module.exports = {
http: require.resolve('stream-http'),
os: require.resolve('os-browserify'),
url: require.resolve('url'),
vm: require.resolve("vm-browserify"),
zlib: require.resolve('browserify-zlib')
})
config.resolve.fallback = fallback
Expand All @@ -35,6 +36,14 @@ module.exports = {
}
})

// fixes "Uncaught TypeError: (0 , superstruct_1.boolean) is not a function" error
config.module.rules.map(rule => {
if (rule.oneOf instanceof Array) {
rule.oneOf[rule.oneOf.length - 1].exclude = [/\.(js|mjs|jsx|cjs|ts|tsx)$/, /\.html$/, /\.json$/];
}
return rule;
});

// Bundle size warning bypass
config.plugins.forEach((plugin) => {
if ('config' in plugin) {
Expand Down
1 change: 1 addition & 0 deletions apps/skillforge-ui/craco.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ module.exports = {
http: require.resolve('stream-http'),
os: require.resolve('os-browserify'),
url: require.resolve('url'),
vm: require.resolve("vm-browserify"),
zlib: require.resolve('browserify-zlib')
})
config.resolve.fallback = fallback
Expand Down
14 changes: 7 additions & 7 deletions apps/skillforge-ui/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,10 @@
},
"resolutions": {
"**/*/@tanstack/react-query": "^5.x",
"**/*/viem": "^2.x",
"viem": "^2.x",
"**/*/wagmi": "^2.x",
"wagmi": "^2.x",
"**/*/viem": ">=2.10.0 < 3",
"viem": ">=2.10.0 < 3",
"**/*/wagmi": ">=2.5.5 < 3",
"wagmi": ">=2.5.5 < 3",
"styled-components": "^5.x"
},
"dependencies": {
Expand All @@ -50,11 +50,10 @@
"@past3lle/theme": "^2.1.0",
"@past3lle/utils": "^2.1.0",
"@tanstack/react-query": "^5.17.15",
"@web3auth/torus-wallet-connector-plugin": "7.2.1",
"react-helmet-async": "^1.3.0",
"styled-components": "^5.3.5",
"viem": "^2.3.1",
"wagmi": "^2.2.1"
"viem": "^2.21.1",
"wagmi": "^2.12.8"
},
"devDependencies": {
"@babel/plugin-proposal-nullish-coalescing-operator": "^7.13.8",
Expand Down Expand Up @@ -92,6 +91,7 @@
"source-map-explorer": "^2.5.3",
"stream-browserify": "^3.0.0",
"stream-http": "^3.2.0",
"vm-browserify": "^1.1.2",
"url": "^0.11.0",
"web-vitals": "^3.1.0",
"workbox-webpack-plugin": "^7.0.0"
Expand Down
29 changes: 1 addition & 28 deletions apps/skillforge-ui/src/web3/config/chains.ts
Original file line number Diff line number Diff line change
@@ -1,31 +1,4 @@
import { polygon, sepolia } from 'viem/chains'
import { defineChain } from 'viem/utils'

// TODO: remove when viem adds amoy polygon
const amoy = defineChain({
id: 80002,
name: 'Amoy',
nativeCurrency: {
decimals: 18,
name: 'Matic',
symbol: 'MATIC'
},
rpcUrls: {
default: {
http: ['https://rpc-amoy.polygon.technology/'],
webSocket: ['wss://rpc-amoy.polygon.technology/']
}
},
blockExplorers: {
default: { name: 'Explorer', url: 'https://amoy.polygonscan.com/' }
},
contracts: {
multicall3: {
address: '0xcA11bde05977b3631167028862bE2a173976CA11',
blockCreated: 3127388
}
}
})
import { polygon, sepolia, polygonAmoy as amoy } from 'viem/chains'

// Sepolia, Polygon, Amoy Polygon (test)
export type SupportedChainsDevelop = 11155111 | 137 | 80002
Expand Down
14 changes: 7 additions & 7 deletions apps/skillforge-ui/src/web3/connectors/index.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import METAMASK_ICON from '../../assets/png/metamask.png'
import { ledgerHid, ledgerLive, pstlWeb3Auth, PstlWeb3AuthParameters } from '@past3lle/wagmi-connectors'
import { ledgerHid, ledgerLive, web3Auth, Web3AuthParameters } from '@past3lle/wagmi-connectors'
import { PstlWeb3ModalProps } from '@past3lle/web3-modal'
import { ASSETS_MAP } from 'assets'
import GOOGLE_APPLE_LOGO from 'assets/png/google-apple.png'
import { skillforgeTheme } from 'theme/skillforge'
Expand All @@ -8,7 +9,7 @@ import { injected } from 'wagmi/connectors'

const IS_SERVER = typeof globalThis?.window === 'undefined'

function _getWhitelistTheme(): PstlWeb3AuthParameters['uiConfig'] {
function _getWhitelistTheme(): Web3AuthParameters['uiConfig'] {
if (!JSON.parse(process.env.REACT_APP_WEB3AUTH_WHITELIST_ENABLED || 'false')) return

return {
Expand Down Expand Up @@ -48,14 +49,13 @@ const connectors = {
}
}
}),
pstlWeb3Auth({
network: process.env.REACT_APP_WEB3_AUTH_NETWORK as PstlWeb3AuthParameters['network'],
web3Auth({
network: process.env.REACT_APP_WEB3_AUTH_NETWORK as Web3AuthParameters['network'],
projectId: process.env.REACT_APP_WEB3AUTH_ID as string,
storageKey: 'session',
preset: 'DISALLOW_EXTERNAL_WALLETS',
mfaLevel: 'none',
uxMode: 'popup',
..._getWhitelistTheme()
uiConfig: _getWhitelistTheme()
})
],
overrides: {
Expand All @@ -82,6 +82,6 @@ const connectors = {
rank: 0
}
}
}
} satisfies PstlWeb3ModalProps['connectors']
const frameConnectors = [ledgerLive()]
export { connectors, frameConnectors }
25 changes: 0 additions & 25 deletions apps/skillforge-ui/src/web3/connectors/web3authPlugins.ts

This file was deleted.

16 changes: 8 additions & 8 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,14 +40,6 @@
"version:ci": "yarn version:cc --yes",
"version:update": "node scripts/updateVersions.mjs"
},
"resolutions": {
"@tanstack/react-query": "^5.x",
"**/*/@tanstack/react-query": "^5.x",
"**/*/viem": "^2.x",
"viem": "^2.x",
"**/*/wagmi": ">=2.5.5 < 3",
"wagmi": ">=2.5.5 < 3"
},
"devDependencies": {
"@babel/preset-typescript": "^7.18.6",
"@trivago/prettier-plugin-sort-imports": "^4.0.0",
Expand All @@ -68,6 +60,14 @@
"@babel/preset-env": "^7.20.2"
},
"dependencies": {},
"resolutions": {
"@tanstack/react-query": "^5.x",
"**/*/@tanstack/react-query": "^5.x",
"**/*/viem": ">=2.10.0 < 3",
"viem": ">=2.10.0 < 3",
"**/*/wagmi": ">=2.5.5 < 3",
"wagmi": ">=2.5.5 < 3"
},
"workspaces": [
"packages/*",
"apps/*"
Expand Down
24 changes: 11 additions & 13 deletions packages/forge-web3/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -66,21 +66,12 @@
"bugs": {
"url": "https://github.com/PAST3LLE/past3lle-monorepo/issues"
},
"resolutions": {
"@wagmi/core": ">2 < 3",
"**/*/@wagmi/core": ">2 < 3",
"**/*/@tanstack/react-query": "^5.x",
"**/*/viem": "^2.x",
"viem": "^2.x",
"**/*/wagmi": "^2.x",
"wagmi": "^2.x"
},
"peerDependencies": {
"@tanstack/react-query": "^5.x",
"react": ">=16",
"styled-components": ">=5",
"viem": "^2.x",
"wagmi": "^2.x"
"viem": ">=2.10.0 <3",
"wagmi": ">=2.5.5 < 3"
},
"dependencies": {
"@past3lle/hooks": "^2.1.0",
Expand All @@ -101,8 +92,15 @@
"react-cosmos": "^5.7.2",
"react-dom": "^18.2.0",
"styled-components": "^5.3.6",
"viem": "^2.3.1",
"wagmi": "^2.2.1"
"viem": "^2.21.1",
"wagmi": "^2.12.8"
},
"resolutions": {
"**/*/@tanstack/react-query": "^5.x",
"**/*/viem": ">=2.10.0 < 3",
"viem": ">=2.10.0 < 3",
"**/*/wagmi": ">=2.5.5 < 3",
"wagmi": ">=2.5.5 < 3"
},
"browserslist": {
"production": [
Expand Down
31 changes: 2 additions & 29 deletions packages/forge-web3/src/constants/chains.ts
Original file line number Diff line number Diff line change
@@ -1,30 +1,3 @@
import { defineChain } from 'viem'
import { Chain, polygon, sepolia } from 'viem/chains'
import { Chain, polygon, polygonAmoy, sepolia } from 'viem/chains'

// TODO: remove when viem adds amoy polygon
export const amoy = defineChain({
id: 80002,
name: 'Amoy',
nativeCurrency: {
decimals: 18,
name: 'Matic',
symbol: 'MATIC'
},
rpcUrls: {
default: {
http: ['https://rpc-amoy.polygon.technology/'],
webSocket: ['wss://rpc-amoy.polygon.technology/']
}
},
blockExplorers: {
default: { name: 'Explorer', url: 'https://amoy.polygonscan.com/' }
},
contracts: {
multicall3: {
address: '0xcA11bde05977b3631167028862bE2a173976CA11',
blockCreated: 3127388
}
}
})

export const FORGE_SUPPORTED_CHAINS = [sepolia, polygon, amoy] as const satisfies readonly [Chain, ...Chain[]]
export const FORGE_SUPPORTED_CHAINS = [sepolia, polygon, polygonAmoy] as const satisfies readonly [Chain, ...Chain[]]
6 changes: 4 additions & 2 deletions packages/forge-web3/src/fixtures/chains.ts
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
import { sepolia } from 'viem/chains'
import { polygonAmoy, sepolia } from 'viem/chains'

export const chains = [sepolia] as const
import { ForgeChainsMinimum } from '..'

export const chains = [sepolia, polygonAmoy] as const satisfies ForgeChainsMinimum
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ export function useForgeClaimLockedSkillCallback() {
address: mergeManager,
abi: MergeManager__factory.abi,
functionName: 'claimLockedSkill',
args: [formattedArgs]
args: [formattedArgs as never]
}).then((hash) => {
addPending(hash as Hash, { metadata: { forgeSkillId: skill?.properties.id, forgeTransactionType: 'claim' } })
return hash
Expand Down
2 changes: 1 addition & 1 deletion packages/forge-web3/src/types/addresses.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { Address } from 'abitype'
import { Address } from 'viem'

import { ForgeChainsMinimum } from './appConfig'

Expand Down
1 change: 1 addition & 0 deletions packages/forge-web3/webpack.override.js
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ module.exports = (webpackConfig) => (
https: require.resolve("https-browserify"),
os: require.resolve("os-browserify"),
url: require.resolve("url"),
vm: require.resolve("vm-browserify"),
zlib: require.resolve("browserify-zlib")
},
},
Expand Down
23 changes: 12 additions & 11 deletions packages/skillforge-widget/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -62,20 +62,13 @@
"<rootDir>src/**/__tests__/*.{ts,tsx}"
]
},
"resolutions": {
"**/*/@tanstack/react-query": "^5.x",
"**/*/viem": "^2.x",
"viem": "^2.x",
"**/*/wagmi": "^2.x",
"wagmi": "^2.x"
},
"peerDependencies": {
"@tanstack/react-query": "^5.x",
"jotai": ">=2",
"react": ">=16",
"styled-components": ">=5",
"viem": "^2.x",
"wagmi": "^2.x"
"viem": ">=2.10.0 < 3",
"wagmi": ">=2.5.5 < 3"
},
"devDependencies": {
"@babel/plugin-proposal-nullish-coalescing-operator": "^7.13.8",
Expand All @@ -95,8 +88,8 @@
"react-cosmos": "^5.7.2",
"react-dom": "^18.2.0",
"styled-components": "^5.3.5",
"viem": "^2.3.1",
"wagmi": "^2.2.1"
"viem": "^2.21.1",
"wagmi": "^2.12.8"
},
"dependencies": {
"@past3lle/components": "^2.1.0",
Expand All @@ -108,6 +101,14 @@
"jotai": "^2.6.0",
"polished": "^4.2.2"
},
"resolutions": {
"**/*/@tanstack/react-query": "^5.x",
"**/*/viem": ">=2.10.0 < 3",
"viem": ">=2.10.0 < 3",
"**/*/wagmi": ">=2.5.5 < 3",
"wagmi": ">=2.5.5 < 3"
},

"bugs": {
"url": "https://github.com/PAST3LLE/past3lle-monorepo/issues"
},
Expand Down
Loading
Loading