From 7270480e0bd096276a825bf10f4b517fc32fc678 Mon Sep 17 00:00:00 2001 From: sneak Date: Fri, 14 Aug 2026 04:16:47 +0000 Subject: [PATCH] build: add ESLint to script/lint and containerize linting (closes #152) script/lint ran `prettier --check .`, byte for byte what script/fmt-check runs, so make check checked formatting twice and did no static analysis on a cryptocurrency wallet. Two used-but-not-imported crashes shipped past it. ESLint is pinned in package.json with @eslint/js recommended as the base and a flat config in eslint.config.js. no-undef and no-unused-vars are restated error-level so a future recommended-set change cannot downgrade them. Globals are declared per tree rather than globally, because a too-wide set hides the next unimported identifier: browser for the popup and content scripts, service worker for src/background/ and src/shared/, browser for the one documented POPUP ONLY module in src/shared/, jest for tests/, node for build.js, and both for the e2e harnesses, which carry the callbacks they ship into the page inline. Two rules new to the recommended set are off, and both would have cost something to satisfy. no-useless-assignment flags the `password = null` and `decryptedSecret = null` wipes in approval.js and confirmTx.js: those assignments are dead by construction, which is the point of them, and the rule's fix is to delete the wipe. preserve-caught-error would change what the wallet's error paths throw, which is a decision of its own. Every remaining violation is fixed: 41 unused bindings and 53 undefined identifiers. Unused catch bindings became `catch {`, which the repo already used; the shared init(ctx) view signature keeps its parameter as _ctx in the three views that do not read it. src/shared/uniswap.js keeps its unused V2_SWAP_EXACT_OUT decoder behind a scoped disable, because deleting it would widen the gap it represents rather than close it (#283). Linting is containerized. script/lint builds the Dockerfile's new lint stage so the ESLint deciding whether this repo is green is the pinned one and not whatever the host has; AUTISTMASK_LINT_NATIVE, set only in that image, is what makes make check inside the CI build lint in place instead of recursing into docker. The check stage takes a COPY --from=lint dependency so a lint failure fails the whole build early rather than racing it. No --fix anywhere in the lint path: make check remains non-mutating. The README claim that a used-but-not-imported identifier is invisible to make check, and the same claim in script/test-e2e, are no longer true and are corrected. --- Dockerfile | 19 +- README.md | 17 +- TODO.md | 28 +- build.js | 4 +- eslint.config.js | 144 ++++++++++ package.json | 5 +- script/lint | 34 ++- script/test-e2e | 5 +- src/content/inpage.js | 2 +- src/popup/index.js | 1 - src/popup/views/addWallet.js | 4 +- src/popup/views/addressDetail.js | 4 +- src/popup/views/addressToken.js | 8 +- src/popup/views/approval.js | 4 +- src/popup/views/confirmTx.js | 16 +- src/popup/views/deleteWallet.js | 2 +- src/popup/views/helpers.js | 6 +- src/popup/views/home.js | 1 - src/popup/views/receive.js | 2 +- src/popup/views/send.js | 3 +- src/popup/views/settings.js | 1 - src/popup/views/transactionDetail.js | 9 +- src/popup/views/txStatus.js | 6 +- src/shared/prices.js | 2 +- src/shared/txValidation.js | 2 +- src/shared/uniswap.js | 5 + src/shared/vault.js | 2 +- tests/e2e/firefox/driver.js | 4 +- tests/e2e/harness.js | 2 +- tests/e2e/run.js | 6 +- tests/uniswap.test.js | 2 +- yarn.lock | 380 ++++++++++++++++++++++++++- 32 files changed, 647 insertions(+), 83 deletions(-) create mode 100644 eslint.config.js diff --git a/Dockerfile b/Dockerfile index cdb2550..eee86fc 100644 --- a/Dockerfile +++ b/Dockerfile @@ -1,8 +1,13 @@ # node:22-slim (22.x LTS), 2026-02-24 -FROM node@sha256:5373f1906319b3a1f291da5d102f4ce5c77ccbe29eb637f072b6c7b70443fc36 +FROM node@sha256:5373f1906319b3a1f291da5d102f4ce5c77ccbe29eb637f072b6c7b70443fc36 AS base WORKDIR /app +# Marks "already inside the lint container" for script/lint, which otherwise +# shells out to docker to build the lint stage below. Nothing outside this +# image sets it. +ENV AUTISTMASK_LINT_NATIVE=1 + # script/bootstrap installs all prerequisites (make via apt here; node # is already in the base image, yarn comes via corepack) and runs # yarn install --frozen-lockfile. Dependency manifests are copied first @@ -13,5 +18,17 @@ RUN script/bootstrap COPY . . +# Lint stage — fail fast on static analysis and formatting, before the tests +# and the build. This is also the stage script/lint builds from a host, which +# is how linting stays on the pinned ESLint rather than the host's. +FROM base AS lint +RUN make lint + +# Full check and build. The COPY --from is a no-op file copy whose only job is +# to make BuildKit finish the lint stage before this one starts; without it the +# stages run in parallel and a lint failure would not fail the build early. +FROM base AS check +COPY --from=lint /app/package.json /dev/null + RUN make check RUN make build diff --git a/README.md b/README.md index cb2733a..12553f9 100644 --- a/README.md +++ b/README.md @@ -88,7 +88,13 @@ provide: - `script/test-e2e-firefox` — run the Firefox browser end-to-end suite (docker required; builds its own pinned image, see [End-to-End Tests](#end-to-end-tests)) -- `script/lint` — run the linter +- `script/lint` — run ESLint (`eslint.config.js`) and then `prettier --check`, + failing on either. It never writes: `--fix` is not in this path, so + `make check` stays non-mutating. Linting runs in the container — the script + builds the Dockerfile's `lint` stage — because an ESLint result that depends + on whichever ESLint the host happens to have is not a result. Docker is + therefore required to lint; inside that image `AUTISTMASK_LINT_NATIVE=1` makes + the same script lint in place instead of recursing. - `script/fmt` — format all files (writes) - `script/fmt-check` — check formatting (read-only) - `script/check` — run test, test-verify-build, lint, and fmt-check @@ -235,9 +241,12 @@ being intercepted, run with `E2E_TRACE_NETWORK=1` and every routed request is printed, tagged `[sw]` or `[page]`. **Any uncaught page error or `console.error` fails the run.** That is the point: -a `ReferenceError` from a used-but-not-imported identifier is invisible to -`make check` (`script/lint` is only `prettier --check`) but fatal in a browser, -and this suite exists because exactly that class of bug shipped twice. +this suite exists because a `ReferenceError` from a used-but-not-imported +identifier shipped twice, fatal in a browser and invisible to a `make check` +that was `prettier --check` only. ESLint's `no-undef` now catches that exact +class before a browser is involved, so this suite is no longer the only thing +standing between it and a release — but a static rule only sees identifiers, and +the runtime errors this suite catches are broader than one rule. ### Firefox (`make test-e2e-firefox`) diff --git a/TODO.md b/TODO.md index 86fd8b7..a4e8131 100644 --- a/TODO.md +++ b/TODO.md @@ -32,19 +32,30 @@ The backlog lives on the [Gitea tracker](https://git.eeqj.de/sneak/AutistMask/issues), which is authoritative; this file does not duplicate it. Full policy file set present. Real-browser end-to-end suites (`make test-e2e` for Chrome, -`make test-e2e-firefox` for Firefox) now sit alongside `make check`, which -cannot see a runtime `ReferenceError` in a popup view. +`make test-e2e-firefox` for Firefox) sit alongside `make check`, which now does +static analysis as well as formatting. # Next Step -Land [#152](https://git.eeqj.de/sneak/AutistMask/issues/152): add ESLint to -`script/lint`. `make check` is `prettier --check` only today and cannot catch -undefined identifiers, which is how -[#150](https://git.eeqj.de/sneak/AutistMask/issues/150) and -[#151](https://git.eeqj.de/sneak/AutistMask/issues/151) shipped. +Pre-1.0 security review of the extension (key handling, DEBUG mode policy, RPC +input validation) before any 1.0rc tag. Individual filed issues are parts of it, +but the review is broader than any of them. # Completed Steps +- 2026-08-14: `make check` does static analysis. `script/lint` ran + `prettier --check .`, byte-identical to `script/fmt-check`, so a wallet with + two shipped used-but-not-imported crashes behind it was green. ESLint is now + pinned in `package.json` with `@eslint/js` recommended as the base, flat + config in `eslint.config.js`, `no-undef` and `no-unused-vars` error-level, and + globals declared per tree — browser for the popup and content scripts, service + worker for `src/background/` and `src/shared/`, jest for `tests/`, node for + `build.js`. It found 41 unused bindings and 53 undefined identifiers; all are + fixed, and dropping a call to an unimported `foo()` into any `src/` file fails + `make lint`. Linting is also containerized now: `script/lint` builds the + Dockerfile's new `lint` stage, so the ESLint that decides whether this repo is + green is the pinned one and not the host's + ([#152](https://git.eeqj.de/sneak/AutistMask/issues/152)). - 2026-08-12: EIP-1193 error codes now reach the page. `src/content/inpage.js` rebuilt every failure as `new Error(error.message)`, so the code the background produced and the content script relayed intact was dropped in the @@ -365,9 +376,6 @@ undefined identifiers, which is how Only work that has no issue of its own belongs here; everything else is on the tracker. -- Pre-1.0 security review of the extension (key handling, DEBUG mode policy, RPC - input validation) before any 1.0rc tag. Individual filed issues are parts of - it, but the review is broader than any of them. - Decide whether docker-in-docker makes `make test-e2e` and `make test-e2e-firefox` runnable in the Gitea workflow. Extending the Chrome suite itself is tracked as diff --git a/build.js b/build.js index c5b3953..ec38667 100644 --- a/build.js +++ b/build.js @@ -63,7 +63,7 @@ function getBuildInfo() { commitHash = execSync("git rev-parse --short HEAD", { encoding: "utf8", }).trim(); - } catch (_) { + } catch { // not a git repo or git not available } let commitHashFull = "unknown"; @@ -71,7 +71,7 @@ function getBuildInfo() { commitHashFull = execSync("git rev-parse HEAD", { encoding: "utf8", }).trim(); - } catch (_) { + } catch { // not a git repo or git not available } return { diff --git a/eslint.config.js b/eslint.config.js new file mode 100644 index 0000000..a91457d --- /dev/null +++ b/eslint.config.js @@ -0,0 +1,144 @@ +// ESLint flat config. Static analysis for make check; formatting stays with +// prettier (script/fmt-check), so nothing here touches style. +// +// The sources are CommonJS and are bundled per entrypoint by build.js, so the +// globals differ by tree and are declared per tree below. Getting that wrong in +// either direction defeats the point: too few globals buries a real no-undef in +// false positives, too many hides the next unimported identifier. + +const js = require("@eslint/js"); +const globals = require("globals"); + +// The extension APIs. MV3 Chrome exposes `chrome`; Firefox exposes both, and +// the code feature-detects between them. +const extensionGlobals = { + chrome: "readonly", + browser: "readonly", +}; + +const commonjs = { + ecmaVersion: 2024, + sourceType: "commonjs", +}; + +module.exports = [ + { + ignores: [ + "dist/", + "node_modules/", + // Emitted by build.js, not authored here. + "src/popup/styles/", + ], + }, + + js.configs.recommended, + + { + rules: { + // The two rules this config exists for. Both are already + // error-level in the recommended set; restated so a future + // recommended-set change cannot silently downgrade them. + "no-undef": "error", + // `_`-prefixed arguments are the deliberate "present for the + // interface, unused here" marker: the popup views share one + // init(ctx) signature and three of the eight do not read ctx. + // An unused catch binding is written `catch {`, which the repo + // already does, so caught errors stay checked. + "no-unused-vars": ["error", { argsIgnorePattern: "^_" }], + + // Off: it flags `password = null` and `decryptedSecret = null` in + // approval.js and confirmTx.js, which are the best-effort wipes of + // decrypted key material after use. The assignments are dead by + // construction — that is the point of them — and satisfying the + // rule would mean deleting the wipes. + "no-useless-assignment": "off", + + // Off: it requires every rethrow to carry `{ cause }`. That is a + // change to what the wallet's error paths actually throw, which is + // not this config's business; adopting it is its own decision. + "preserve-caught-error": "off", + }, + }, + + // Popup and content scripts: page/window context. + { + files: ["src/popup/**/*.js", "src/content/**/*.js"], + languageOptions: { + ...commonjs, + globals: { ...globals.browser, ...extensionGlobals }, + }, + }, + + // MV3 background: a service worker, with no window and no document. + { + files: ["src/background/**/*.js"], + languageOptions: { + ...commonjs, + globals: { ...globals.serviceworker, ...extensionGlobals }, + }, + }, + + // src/shared is bundled into both, so it may only use what both provide: + // the service worker globals are the intersection, plus the extension APIs. + { + files: ["src/shared/**/*.js"], + languageOptions: { + ...commonjs, + globals: { ...globals.serviceworker, ...extensionGlobals }, + }, + }, + + // src/shared/ens.js is the documented exception to the line above: its own + // header says POPUP ONLY, it caches in localStorage, and only popup views + // require it. Linting it as a service worker would be wrong about the file. + { + files: ["src/shared/ens.js"], + languageOptions: { + ...commonjs, + globals: { ...globals.browser, ...extensionGlobals }, + }, + }, + + // Unit tests: jest on node. + { + files: ["tests/**/*.test.js"], + languageOptions: { + ...commonjs, + globals: { ...globals.node, ...globals.jest }, + }, + }, + + // The build script is a plain node program. + { + files: ["build.js"], + languageOptions: { + ...commonjs, + globals: { ...globals.node }, + }, + }, + + // The e2e harnesses are node programs that also carry, inline, the + // callbacks they ship into the browser via page.evaluate — so both + // contexts really are present in the same file and both sets of globals + // are in scope somewhere in it. + { + files: ["tests/e2e/**/*.js"], + languageOptions: { + ...commonjs, + globals: { + ...globals.node, + ...globals.browser, + ...extensionGlobals, + }, + }, + }, + + // This config file itself. + { + files: ["eslint.config.js"], + languageOptions: { + ...commonjs, + globals: { ...globals.node }, + }, + }, +]; diff --git a/package.json b/package.json index 45a6c80..df915e8 100644 --- a/package.json +++ b/package.json @@ -9,13 +9,16 @@ "test": "jest --forceExit", "test:verbose": "jest --forceExit --verbose", "build": "node build.js", - "lint": "prettier --check .", + "lint": "eslint . && prettier --check .", "fmt": "prettier --write .", "fmt-check": "prettier --check ." }, "devDependencies": { + "@eslint/js": "10.0.1", "@tailwindcss/cli": "^4.2.1", "esbuild": "^0.27.3", + "eslint": "10.8.1", + "globals": "17.11.0", "jest": "^30.2.0", "playwright-core": "1.56.0", "prettier": "^3.8.1", diff --git a/script/lint b/script/lint index 385ee2e..032e254 100755 --- a/script/lint +++ b/script/lint @@ -1,13 +1,41 @@ #!/bin/sh -# script/lint: run the linter. +# script/lint: run the linter (eslint, then prettier --check). +# +# Linting is containerized. ESLint results depend on the ESLint version, and +# the pinned one is the one in the image; a host's own install must not be +# able to decide whether this repo is green. From a host this therefore builds +# the Dockerfile's `lint` stage, which runs this same script inside the image. +# +# AUTISTMASK_LINT_NATIVE is set only in that image (see the Dockerfile) and is +# what stops the recursion, so `make check` inside the CI build lints in place +# instead of trying to reach a docker daemon it does not have. set -eu ROOT="$(cd "$(dirname "$0")/.." && pwd -P)" main() { cd "$ROOT" - echo "Linting..." - yarn run lint 2>&1 + + if [ "${AUTISTMASK_LINT_NATIVE:-}" = "1" ]; then + echo "Linting..." + yarn run lint 2>&1 + return 0 + fi + + if ! command -v docker >/dev/null 2>&1; then + echo "lint: docker is required; linting does not run on the host" >&2 + exit 1 + fi + + echo "Linting in the pinned container..." + # --progress=plain: the default progress renderer collapses the lint + # output on success, and a lint run whose output cannot be seen is not + # evidence that it ran. + # + # --output=type=cacheonly: the exit status is the whole result; exporting + # an image afterwards costs about ten times the lint itself. + docker build --progress=plain --target lint \ + --output=type=cacheonly . 2>&1 } main "$@" diff --git a/script/test-e2e b/script/test-e2e index ece9a5d..e3dbc0a 100755 --- a/script/test-e2e +++ b/script/test-e2e @@ -5,8 +5,9 @@ # # Deliberately NOT called by script/check or script/test: REPO_POLICIES.md # caps make test at 20 seconds and a browser suite does not fit. Run it -# yourself before touching popup views; it is the only check that can see -# a used-but-not-imported identifier blow up at runtime. +# yourself before touching popup views. ESLint's no-undef now catches a +# used-but-not-imported identifier in make check, but only this suite sees +# what a view actually does when it runs. set -eu ROOT="$(cd "$(dirname "$0")/.." && pwd -P)" diff --git a/src/content/inpage.js b/src/content/inpage.js index ee19d0d..be5606c 100644 --- a/src/content/inpage.js +++ b/src/content/inpage.js @@ -79,7 +79,7 @@ for (const cb of cbs) { try { cb(data); - } catch (e) { + } catch { // ignore listener errors } } diff --git a/src/popup/index.js b/src/popup/index.js index 4185491..fad1d6c 100644 --- a/src/popup/index.js +++ b/src/popup/index.js @@ -12,7 +12,6 @@ const { setBackRenderer, pushCurrentView, goBack, - clearViewStack, } = require("./views/helpers"); const { applyTheme } = require("./theme"); // Renders a view the popup lands on without having navigated to it forward: diff --git a/src/popup/views/addWallet.js b/src/popup/views/addWallet.js index dfb3953..bb83132 100644 --- a/src/popup/views/addWallet.js +++ b/src/popup/views/addWallet.js @@ -194,7 +194,7 @@ async function importPrivateKey(ctx) { let addr; try { addr = addressFromPrivateKey(key); - } catch (e) { + } catch { showFlash("Invalid private key."); return; } @@ -246,7 +246,7 @@ async function importXprvKey(ctx) { let result; try { result = hdWalletFromXprv(xprv); - } catch (e) { + } catch { showFlash( "That extended private key is not valid. Please check it and try again.", ); diff --git a/src/popup/views/addressDetail.js b/src/popup/views/addressDetail.js index 105c352..f059d30 100644 --- a/src/popup/views/addressDetail.js +++ b/src/popup/views/addressDetail.js @@ -12,7 +12,7 @@ const { goBack, pushCurrentView, } = require("./helpers"); -const { state, currentAddress, saveState } = require("../../shared/state"); +const { state, saveState } = require("../../shared/state"); const { formatUsd, getAddressValueUsd } = require("../../shared/prices"); const { fetchRecentTransactions, @@ -44,7 +44,6 @@ function show() { state.selectedToken = null; const wallet = state.wallets[state.selectedWallet]; const addr = wallet.addresses[state.selectedAddress]; - const wi = state.selectedWallet; const ai = state.selectedAddress; $("address-title").textContent = wallet.name + " \u2014 Address " + (ai + 1); @@ -246,7 +245,6 @@ function renderTransactions(txs) { row.addEventListener("click", () => { const idx = parseInt(row.dataset.tx, 10); const tx = loadedTxs[idx]; - const counterparty = tx.direction === "sent" ? tx.to : tx.from; tx.fromEns = ensNameMap.get(tx.from) || null; tx.toEns = ensNameMap.get(tx.to) || null; ctx.showTransactionDetail(tx); diff --git a/src/popup/views/addressToken.js b/src/popup/views/addressToken.js index 7dba24c..13fb3da 100644 --- a/src/popup/views/addressToken.js +++ b/src/popup/views/addressToken.js @@ -16,13 +16,9 @@ const { goBack, pushCurrentView, } = require("./helpers"); -const { state, currentAddress, saveState } = require("../../shared/state"); +const { state, saveState } = require("../../shared/state"); const { TOKEN_BY_ADDRESS, resolveSymbol } = require("../../shared/tokenList"); -const { - formatUsd, - getPrice, - getAddressValueUsd, -} = require("../../shared/prices"); +const { formatUsd, getPrice } = require("../../shared/prices"); const { fetchRecentTransactions, filterTransactions, diff --git a/src/popup/views/approval.js b/src/popup/views/approval.js index ebbc25c..ca83db2 100644 --- a/src/popup/views/approval.js +++ b/src/popup/views/approval.js @@ -9,7 +9,7 @@ const { attachCopyHandlers, onViewLeave, } = require("./helpers"); -const { state, saveState, currentNetwork } = require("../../shared/state"); +const { state, saveState } = require("../../shared/state"); const { networkByChainId } = require("../../shared/networks"); const { formatEther, @@ -537,7 +537,7 @@ function clearSignPassword() { hideError("approve-sign-error"); } -function init(ctx) { +function init(_ctx) { onViewLeave("approve-tx", clearTxPassword); onViewLeave("approve-sign", clearSignPassword); diff --git a/src/popup/views/confirmTx.js b/src/popup/views/confirmTx.js index 5d59734..cf66207 100644 --- a/src/popup/views/confirmTx.js +++ b/src/popup/views/confirmTx.js @@ -2,20 +2,12 @@ // Shows transaction details, warnings, errors. On Sign & Send, // reads inline password, decrypts secret, signs and broadcasts. -const { - parseEther, - parseUnits, - formatEther, - formatUnits, - Contract, -} = require("ethers"); +const { parseEther, parseUnits, formatEther, Contract } = require("ethers"); const { $, showError, hideError, showView, - showFlash, - flashCopyFeedback, addressTitle, escapeHtml, renderAddressHtml, @@ -23,7 +15,7 @@ const { goBack, onViewLeave, } = require("./helpers"); -const { state, currentNetwork } = require("../../shared/state"); +const { state } = require("../../shared/state"); const { getSignerForAddress } = require("../../shared/wallet"); const { decryptWithPassword } = require("../../shared/vault"); const { formatUsd, getPrice } = require("../../shared/prices"); @@ -399,7 +391,7 @@ function clearPassword() { hideError("confirm-tx-password-error"); } -function init(ctx) { +function init(_ctx) { onViewLeave("confirm-tx", clearPassword); $("btn-confirm-send").addEventListener("click", async () => { @@ -421,7 +413,7 @@ function init(ctx) { wallet.encryptedSecret, password, ); - } catch (e) { + } catch { showError( "confirm-tx-password-error", "That password is incorrect. Please try again.", diff --git a/src/popup/views/deleteWallet.js b/src/popup/views/deleteWallet.js index 685f937..cf68048 100644 --- a/src/popup/views/deleteWallet.js +++ b/src/popup/views/deleteWallet.js @@ -73,7 +73,7 @@ function init(_ctx) { // Verify password against the wallet's encrypted data try { await decryptWithPassword(wallet.encryptedSecret, pw); - } catch (_e) { + } catch { $("delete-wallet-flash").textContent = "That password is incorrect. Please try again."; $("delete-wallet-flash").style.visibility = "visible"; diff --git a/src/popup/views/helpers.js b/src/popup/views/helpers.js index d16885d..6b6b891 100644 --- a/src/popup/views/helpers.js +++ b/src/popup/views/helpers.js @@ -1,11 +1,7 @@ // Shared DOM helpers used by all views. const { isDebug } = require("../../shared/log"); -const { - formatUsd, - getPrice, - getAddressValueUsd, -} = require("../../shared/prices"); +const { formatUsd, getPrice } = require("../../shared/prices"); const { state, saveState, currentNetwork } = require("../../shared/state"); const { markViewRendered } = require("../viewRouter"); diff --git a/src/popup/views/home.js b/src/popup/views/home.js index bdf1e6f..5d10463 100644 --- a/src/popup/views/home.js +++ b/src/popup/views/home.js @@ -2,7 +2,6 @@ const { $, showView, showFlash, - flashCopyFeedback, balanceLinesForAddress, isoDate, timeAgo, diff --git a/src/popup/views/receive.js b/src/popup/views/receive.js index 25d71ae..35f276b 100644 --- a/src/popup/views/receive.js +++ b/src/popup/views/receive.js @@ -57,7 +57,7 @@ function show() { attachCopyHandlers("view-receive"); } -function init(ctx) { +function init(_ctx) { $("btn-receive-copy").addEventListener("click", () => { const addr = $("receive-address-block").dataset.full; if (addr) { diff --git a/src/popup/views/send.js b/src/popup/views/send.js index 85cadbf..9ab0fc3 100644 --- a/src/popup/views/send.js +++ b/src/popup/views/send.js @@ -4,7 +4,6 @@ const { $, showFlash, addressTitle, - escapeHtml, renderAddressHtml, attachCopyHandlers, goBack, @@ -210,7 +209,7 @@ function init(_ctx) { } resolvedTo = resolved; ensName = to; - } catch (e) { + } catch { showFlash("Failed to resolve ENS name."); return; } diff --git a/src/popup/views/settings.js b/src/popup/views/settings.js index 2d1baa3..2d51414 100644 --- a/src/popup/views/settings.js +++ b/src/popup/views/settings.js @@ -14,7 +14,6 @@ const { parseDustThresholdGwei, } = require("../dustThreshold"); const { state, saveState, currentNetwork } = require("../../shared/state"); -const { NETWORKS, SUPPORTED_CHAIN_IDS } = require("../../shared/networks"); const { onChainSwitch } = require("../../shared/chainSwitch"); const { log, debugFetch, setRuntimeDebug } = require("../../shared/log"); const deleteWallet = require("./deleteWallet"); diff --git a/src/popup/views/transactionDetail.js b/src/popup/views/transactionDetail.js index 5922ddb..3d669c5 100644 --- a/src/popup/views/transactionDetail.js +++ b/src/popup/views/transactionDetail.js @@ -23,8 +23,6 @@ const makeBlockie = require("ethereum-blockies-base64"); const { log, debugFetch } = require("../../shared/log"); const { decodeCalldata } = require("./approval"); -let ctx; - /** * Determine a human-readable transaction type string from tx fields. */ @@ -166,7 +164,7 @@ function render() { if (el) el.classList.add("hidden"); } - loadFullTxDetails(tx.hash, tx.to, tx.isContractCall); + loadFullTxDetails(tx.hash, tx.to); const isoStr = isoDate(tx.timestamp); $("tx-detail-time").innerHTML = @@ -274,7 +272,7 @@ function populateOnChainDetails(txData) { } } -async function loadFullTxDetails(txHash, toAddress, isContractCall) { +async function loadFullTxDetails(txHash, toAddress) { const section = $("tx-detail-calldata-section"); const actionEl = $("tx-detail-calldata-action"); const detailsEl = $("tx-detail-calldata-details"); @@ -349,8 +347,9 @@ async function loadFullTxDetails(txHash, toAddress, isContractCall) { } } +// The ctx this view is initialized with is unused: this module is the leaf of +// the navigation, and the other views reach it through their own ctx. function init(_ctx) { - ctx = _ctx; $("btn-tx-back").addEventListener("click", () => { goBack(); }); diff --git a/src/popup/views/txStatus.js b/src/popup/views/txStatus.js index 3a114da..8edd4ed 100644 --- a/src/popup/views/txStatus.js +++ b/src/popup/views/txStatus.js @@ -12,7 +12,7 @@ const { clearViewStack, } = require("./helpers"); const { TOKEN_BY_ADDRESS } = require("../../shared/tokenList"); -const { state, saveState, currentNetwork } = require("../../shared/state"); +const { state, currentNetwork } = require("../../shared/state"); const { getProvider } = require("../../shared/balances"); const { log } = require("../../shared/log"); @@ -229,10 +229,6 @@ function tokenLabel(address) { return t ? t.symbol : null; } -function etherscanTokenLink(address) { - return `${currentNetwork().explorerUrl}/token/${address}`; -} - function decodedDetailsHtml(decoded) { if (!decoded || !decoded.details) return ""; let html = `
`; diff --git a/src/shared/prices.js b/src/shared/prices.js index ac45dd1..ffd521b 100644 --- a/src/shared/prices.js +++ b/src/shared/prices.js @@ -21,7 +21,7 @@ async function refreshPrices() { const fetched = await getTopTokenPrices(25); Object.assign(prices, fetched); lastFetchedAt = now; - } catch (e) { + } catch { // prices stay stale on error } } diff --git a/src/shared/txValidation.js b/src/shared/txValidation.js index 9a2e3d5..c55c66b 100644 --- a/src/shared/txValidation.js +++ b/src/shared/txValidation.js @@ -82,7 +82,7 @@ function toFixedPoint(value) { if (text === "") return null; try { return parseUnits(text, SCALE_DECIMALS); - } catch (e) { + } catch { return null; } } diff --git a/src/shared/uniswap.js b/src/shared/uniswap.js index 55ecb2c..09f8c6a 100644 --- a/src/shared/uniswap.js +++ b/src/shared/uniswap.js @@ -102,6 +102,11 @@ function decodeV2SwapExactIn(input) { // Decode V2_SWAP_EXACT_OUT (command 0x09) input bytes. // ABI: (address recipient, uint256 amountOut, uint256 amountInMax, // address[] path, bool payerIsUser) +// +// Nothing calls this: decode() has no 0x09 arm, so a V2 exact-out swap gets +// its command name and no token or amount detail. Kept for the fix, which is +// https://git.eeqj.de/sneak/AutistMask/issues/283. +// eslint-disable-next-line no-unused-vars function decodeV2SwapExactOut(input) { try { const d = coder.decode( diff --git a/src/shared/vault.js b/src/shared/vault.js index b475915..e31d694 100644 --- a/src/shared/vault.js +++ b/src/shared/vault.js @@ -57,7 +57,7 @@ async function cryptoBackend() { try { await WebAssembly.compile(EMPTY_WASM_MODULE); return "wasm"; - } catch (_) { + } catch { return "asmjs"; } } diff --git a/tests/e2e/firefox/driver.js b/tests/e2e/firefox/driver.js index 6f2ee30..337ef5d 100644 --- a/tests/e2e/firefox/driver.js +++ b/tests/e2e/firefox/driver.js @@ -88,7 +88,7 @@ class Driver { let parsed; try { parsed = JSON.parse(text); - } catch (_) { + } catch { throw new Error( method + " " + path + ": non-JSON response: " + text, ); @@ -420,7 +420,7 @@ async function waitForDriverReady(base, timeoutMs) { const body = await res.json(); if (body && body.value && body.value.ready !== false) return; } - } catch (_) { + } catch { // not listening yet } if (Date.now() >= deadline) { diff --git a/tests/e2e/harness.js b/tests/e2e/harness.js index 6758c1c..af0fa9f 100644 --- a/tests/e2e/harness.js +++ b/tests/e2e/harness.js @@ -287,7 +287,7 @@ async function pageCompilesWasm(page) { try { await WebAssembly.compile(new Uint8Array(bytes)); return true; - } catch (_) { + } catch { return false; } }, EMPTY_WASM_MODULE); diff --git a/tests/e2e/run.js b/tests/e2e/run.js index 3998316..7734581 100644 --- a/tests/e2e/run.js +++ b/tests/e2e/run.js @@ -66,7 +66,11 @@ function withTimeout(promise, name) { const timeout = new Promise((_, reject) => { timer = setTimeout( () => - reject(new Error("timed out after " + TEST_TIMEOUT_MS + "ms")), + reject( + new Error( + name + ": timed out after " + TEST_TIMEOUT_MS + "ms", + ), + ), TEST_TIMEOUT_MS, ); }); diff --git a/tests/uniswap.test.js b/tests/uniswap.test.js index 17b9290..7f0c243 100644 --- a/tests/uniswap.test.js +++ b/tests/uniswap.test.js @@ -1,4 +1,4 @@ -const { AbiCoder, Interface, solidityPacked, getBytes } = require("ethers"); +const { AbiCoder, Interface, solidityPacked } = require("ethers"); const uniswap = require("../src/shared/uniswap"); const ROUTER_ADDR = "0x66a9893cc07d91d95644aedd05d03f95e1dba8af"; diff --git a/yarn.lock b/yarn.lock index 3602e67..73cbeee 100644 --- a/yarn.lock +++ b/yarn.lock @@ -427,6 +427,90 @@ resolved "https://registry.yarnpkg.com/@esbuild/win32-x64/-/win32-x64-0.27.3.tgz#0eaf705c941a218a43dba8e09f1df1d6cd2f1f17" integrity sha512-4uJGhsxuptu3OcpVAzli+/gWusVGwZZHTlS63hh++ehExkVT8SgiEf7/uC/PclrPPkLhZqGgCTjd0VWLo6xMqA== +"@eslint-community/eslint-utils@^4.8.0": + version "4.10.1" + resolved "https://registry.yarnpkg.com/@eslint-community/eslint-utils/-/eslint-utils-4.10.1.tgz#8911bd72b2c3640a543609e0400b8c4d2e7e7cb6" + integrity sha512-cuadcxVFE8sDK6iWJbs8Sn0av2Nrh2QSGQhVlBW9AaAHqHwjWsZHT8LJ4hFGPh7ASBV2deFdM7H/DPjulmh8rg== + dependencies: + eslint-visitor-keys "^3.4.3" + +"@eslint-community/regexpp@^4.12.2": + version "4.12.2" + resolved "https://registry.yarnpkg.com/@eslint-community/regexpp/-/regexpp-4.12.2.tgz#bccdf615bcf7b6e8db830ec0b8d21c9a25de597b" + integrity sha512-EriSTlt5OC9/7SXkRSCAhfSxxoSUgBm33OH+IkwbdpgoqsSsUg7y3uh+IICI/Qg4BBWr3U2i39RpmycbxMq4ew== + +"@eslint/config-array@^0.23.5": + version "0.23.5" + resolved "https://registry.yarnpkg.com/@eslint/config-array/-/config-array-0.23.5.tgz#56e86d243049195d8acc0c06a1b3dfdc3fa3de95" + integrity sha512-Y3kKLvC1dvTOT+oGlqNQ1XLqK6D1HU2YXPc52NmAlJZbMMWDzGYXMiPRJ8TYD39muD/OTjlZmNJ4ib7dvSrMBA== + dependencies: + "@eslint/object-schema" "^3.0.5" + debug "^4.3.1" + minimatch "^10.2.4" + +"@eslint/config-helpers@^0.7.0": + version "0.7.0" + resolved "https://registry.yarnpkg.com/@eslint/config-helpers/-/config-helpers-0.7.0.tgz#09ee4aa07b73f059ec2d4c74bf4b2ff02b322377" + integrity sha512-DObd/KKUsU+FaFv4PLxSRenpXfQWmPXXP3pPZ6/K1PCrMu2vQpMDMuQe/BqYeoLcz8ro0bVDF1RxOJgfVEdhUw== + dependencies: + "@eslint/core" "^1.2.1" + +"@eslint/core@^1.2.1": + version "1.2.1" + resolved "https://registry.yarnpkg.com/@eslint/core/-/core-1.2.1.tgz#c1da7cd1b82fa8787f98b5629fb811848a1b63ce" + integrity sha512-MwcE1P+AZ4C6DWlpin/OmOA54mmIZ/+xZuJiQd4SyB29oAJjN30UW9wkKNptW2ctp4cEsvhlLY/CsQ1uoHDloQ== + dependencies: + "@types/json-schema" "^7.0.15" + +"@eslint/js@10.0.1": + version "10.0.1" + resolved "https://registry.yarnpkg.com/@eslint/js/-/js-10.0.1.tgz#1e8a876f50117af8ab67e47d5ad94d38d6622583" + integrity sha512-zeR9k5pd4gxjZ0abRoIaxdc7I3nDktoXZk2qOv9gCNWx3mVwEn32VRhyLaRsDiJjTs0xq/T8mfPtyuXu7GWBcA== + +"@eslint/object-schema@^3.0.5": + version "3.0.5" + resolved "https://registry.yarnpkg.com/@eslint/object-schema/-/object-schema-3.0.5.tgz#88e9bf4d11d2b19c082e78ebe7ce88724a5eb091" + integrity sha512-vqTaUEgxzm+YDSdElad6PiRoX4t8VGDjCtt05zn4nU810UIx/uNEV7/lZJ6KwFThKZOzOxzXy48da+No7HZaMw== + +"@eslint/plugin-kit@^0.7.2": + version "0.7.2" + resolved "https://registry.yarnpkg.com/@eslint/plugin-kit/-/plugin-kit-0.7.2.tgz#4b0962f3f2c7ce8bc98b3ecfe34525c09d2cb729" + integrity sha512-+CNAzxglkrpNf/kKywqQfk74QjtceuOE7Qm+AF8miRvPF/wmmK5+OJOgVh3AVTT3RP2mH3+FOaxlE5v72owk0A== + dependencies: + "@eslint/core" "^1.2.1" + levn "^0.4.1" + +"@humanfs/core@^0.19.2": + version "0.19.2" + resolved "https://registry.yarnpkg.com/@humanfs/core/-/core-0.19.2.tgz#a8272ca03b2acf492670222b2320b6c421bfde60" + integrity sha512-UhXNm+CFMWcbChXywFwkmhqjs3PRCmcSa/hfBgLIb7oQ5HNb1wS0icWsGtSAUNgefHeI+eBrA8I1fxmbHsGdvA== + dependencies: + "@humanfs/types" "^0.15.0" + +"@humanfs/node@^0.16.6": + version "0.16.8" + resolved "https://registry.yarnpkg.com/@humanfs/node/-/node-0.16.8.tgz#8f800cccc13f4f8cd3116e2d9c0a94939da3e3ed" + integrity sha512-gE1eQNZ3R++kTzFUpdGlpmy8kDZD/MLyHqDwqjkVQI0JMdI1D51sy1H958PNXYkM2rAac7e5/CnIKZrHtPh3BQ== + dependencies: + "@humanfs/core" "^0.19.2" + "@humanfs/types" "^0.15.0" + "@humanwhocodes/retry" "^0.4.0" + +"@humanfs/types@^0.15.0": + version "0.15.0" + resolved "https://registry.yarnpkg.com/@humanfs/types/-/types-0.15.0.tgz#f2a09f62012390b2bff3fc6fb248ddec8c09a090" + integrity sha512-ZZ1w0aoQkwuUuC7Yf+7sdeaNfqQiiLcSRbfI08oAxqLtpXQr9AIVX7Ay7HLDuiLYAaFPu8oBYNq/QIi9URHJ3Q== + +"@humanwhocodes/module-importer@^1.0.1": + version "1.0.1" + resolved "https://registry.yarnpkg.com/@humanwhocodes/module-importer/-/module-importer-1.0.1.tgz#af5b2691a22b44be847b0ca81641c5fb6ad0172c" + integrity sha512-bxveV4V8v5Yb4ncFTT3rPSgZBOpCkjfK0y4oVVVJwIuDVBRMDXrPyXRL988i5ap9m9bnyEEjWfm5WkBmtffLfA== + +"@humanwhocodes/retry@^0.4.0", "@humanwhocodes/retry@^0.4.2": + version "0.4.3" + resolved "https://registry.yarnpkg.com/@humanwhocodes/retry/-/retry-0.4.3.tgz#c2b9d2e374ee62c586d3adbea87199b1d7a7a6ba" + integrity sha512-bV0Tgo9K4hfPCek+aMAn81RppFKv2ySDQeMoSZuvTASywNTnVJCArCZE2FWqpvIatKu7VMRLWlR1EazvVhDyhQ== + "@isaacs/cliui@^8.0.2": version "8.0.2" resolved "https://registry.npmjs.org/@isaacs/cliui/-/cliui-8.0.2.tgz" @@ -1008,6 +1092,16 @@ dependencies: "@babel/types" "^7.28.2" +"@types/esrecurse@^4.3.1": + version "4.3.1" + resolved "https://registry.yarnpkg.com/@types/esrecurse/-/esrecurse-4.3.1.tgz#6f636af962fbe6191b830bd676ba5986926bccec" + integrity sha512-xJBAbDifo5hpffDBuHl0Y8ywswbiAp/Wi7Y/GtAgSlZyIABppyurxVueOPE8LUQOxdlgi6Zqce7uoEpqNTeiUw== + +"@types/estree@^1.0.6", "@types/estree@^1.0.8": + version "1.0.9" + resolved "https://registry.yarnpkg.com/@types/estree/-/estree-1.0.9.tgz#cf3f0e876d7bee15a93ab925b82bf570a3904a24" + integrity sha512-GhdPgy1el4/ImP05X05Uw4cw2/M93BCUmnEvWZNStlCzEKME4Fkk+YpoA5OiHNQmoS7Cafb8Xa3Pya8m1Qrzeg== + "@types/istanbul-lib-coverage@*", "@types/istanbul-lib-coverage@^2.0.1", "@types/istanbul-lib-coverage@^2.0.6": version "2.0.6" resolved "https://registry.npmjs.org/@types/istanbul-lib-coverage/-/istanbul-lib-coverage-2.0.6.tgz" @@ -1027,6 +1121,11 @@ dependencies: "@types/istanbul-lib-report" "*" +"@types/json-schema@^7.0.15": + version "7.0.15" + resolved "https://registry.yarnpkg.com/@types/json-schema/-/json-schema-7.0.15.tgz#596a1747233694d50f6ad8a7869fcb6f56cf5841" + integrity sha512-5+fP8P8MFNC+AyZCDxrB2pkZFPGzqQWUzpSeuuVLvm8VMcorNYavBqoFcxK8bQz4Qsbn4oUEEem4wDLfcysGHA== + "@types/node@*", "@types/node@22.7.5": version "22.7.5" resolved "https://registry.npmjs.org/@types/node/-/node-22.7.5.tgz" @@ -1153,11 +1252,31 @@ resolved "https://registry.yarnpkg.com/@unrs/resolver-binding-win32-x64-msvc/-/resolver-binding-win32-x64-msvc-1.11.1.tgz#538b1e103bf8d9864e7b85cc96fa8d6fb6c40777" integrity sha512-lrW200hZdbfRtztbygyaq/6jP6AKE8qQN2KvPcJ+x7wiD038YtnYtZ82IMNJ69GJibV7bwL3y9FgK+5w/pYt6g== +acorn-jsx@^5.3.2: + version "5.3.2" + resolved "https://registry.yarnpkg.com/acorn-jsx/-/acorn-jsx-5.3.2.tgz#7ed5bb55908b3b2f1bc55c6af1653bada7f07937" + integrity sha512-rq9s+JNhf0IChjtDXxllJ7g41oZk5SlXtp0LHwyA5cejwn7vKmKp4pPri6YEePv2PU65sAsegbXtIinmDFDXgQ== + +acorn@^8.16.0: + version "8.18.0" + resolved "https://registry.yarnpkg.com/acorn/-/acorn-8.18.0.tgz#4faf01b2d6d326bfeed97aea1f52220b5f4c1940" + integrity sha512-lGq+9yr1/GuAWaVYIHRjvvySG5/4VfKIvC8EWxStPdcDh/Ka7FG3twP6v4d5BkravUilhIAsG4Qj83t02LWUPQ== + aes-js@4.0.0-beta.5: version "4.0.0-beta.5" resolved "https://registry.npmjs.org/aes-js/-/aes-js-4.0.0-beta.5.tgz" integrity sha512-G965FqalsNyrPqgEGON7nIx1e/OVENSgiEIzyC63haUMuvNnwIgIjMs52hlTCKhkBny7A2ORNlfY9Zu+jmGk1Q== +ajv@^6.14.0: + version "6.15.0" + resolved "https://registry.yarnpkg.com/ajv/-/ajv-6.15.0.tgz#07e982c74626167aa7a2495c53817892d7139492" + integrity sha512-fgFx7Hfoq60ytK2c7DhnF8jIvzYgOMxfugjLOSMHjLIPgenqa7S7oaagATUq99mV6IYvN2tRmC0wnTYX6iPbMw== + dependencies: + fast-deep-equal "^3.1.1" + fast-json-stable-stringify "^2.0.0" + json-schema-traverse "^0.4.1" + uri-js "^4.2.2" + ansi-escapes@^4.3.2: version "4.3.2" resolved "https://registry.npmjs.org/ansi-escapes/-/ansi-escapes-4.3.2.tgz" @@ -1297,6 +1416,13 @@ brace-expansion@^5.0.2: dependencies: balanced-match "^4.0.2" +brace-expansion@^5.0.8: + version "5.0.9" + resolved "https://registry.yarnpkg.com/brace-expansion/-/brace-expansion-5.0.9.tgz#7c72438809b5fa5babf54199a1f1c281a6984fcf" + integrity sha512-ScQ4IuvIEF1TMlP7Zt+vjJ//9zlPb2SDcxWxM3bk8s6t6GGdJ7KO1dCcTidOPJKePW30LE/2cT7wCyPho9/Wxg== + dependencies: + balanced-match "^4.0.2" + braces@^3.0.3: version "3.0.3" resolved "https://registry.npmjs.org/braces/-/braces-3.0.3.tgz" @@ -1429,7 +1555,7 @@ cross-spawn@^7.0.3, cross-spawn@^7.0.6: shebang-command "^2.0.0" which "^2.0.1" -debug@^4.1.0, debug@^4.1.1, debug@^4.3.1: +debug@^4.1.0, debug@^4.1.1, debug@^4.3.1, debug@^4.3.2: version "4.4.3" resolved "https://registry.npmjs.org/debug/-/debug-4.4.3.tgz" integrity sha512-RGwwWnwQvkVfavKVt22FGLw+xYSdzARwm0ru6DhTVA3umU5hZc28V3kO4stgYryrTlLpuvgI9GiijltAjNbcqA== @@ -1446,6 +1572,11 @@ dedent@^1.6.0: resolved "https://registry.npmjs.org/dedent/-/dedent-1.7.1.tgz" integrity sha512-9JmrhGZpOlEgOLdQgSm0zxFaYoQon408V1v49aqTWuXENVlnCuY9JBZcXZiCsZQWDjTm5Qf/nIvAy77mXDAjEg== +deep-is@^0.1.3: + version "0.1.4" + resolved "https://registry.yarnpkg.com/deep-is/-/deep-is-0.1.4.tgz#a6f2dce612fadd2ef1f519b73551f17e85199831" + integrity sha512-oIPzksmTg4/MriiaYGO+okXDT7ztn/w3Eptv/+gSIdMdKsJo0u4CfYNFJPy+4SKMuCqGw2wxnA+URMg3t8a/bQ== + deepmerge@^4.3.1: version "4.3.1" resolved "https://registry.npmjs.org/deepmerge/-/deepmerge-4.3.1.tgz" @@ -1548,11 +1679,105 @@ escape-string-regexp@^2.0.0: resolved "https://registry.npmjs.org/escape-string-regexp/-/escape-string-regexp-2.0.0.tgz" integrity sha512-UpzcLCXolUWcNu5HtVMHYdXJjArjsF9C0aNnquZYY4uW/Vu0miy5YoWvbV345HauVvcAUnpRuhMMcqTcGOY2+w== +escape-string-regexp@^4.0.0: + version "4.0.0" + resolved "https://registry.yarnpkg.com/escape-string-regexp/-/escape-string-regexp-4.0.0.tgz#14ba83a5d373e3d311e5afca29cf5bfad965bf34" + integrity sha512-TtpcNJ3XAzx3Gq8sWRzJaVajRs0uVxA2YAkdb1jm2YkPz4G6egUFAyA3n5vtEIZefPk5Wa4UXbKuS5fKkJWdgA== + +eslint-scope@^9.1.2: + version "9.1.2" + resolved "https://registry.yarnpkg.com/eslint-scope/-/eslint-scope-9.1.2.tgz#b9de6ace2fab1cff24d2e58d85b74c8fcea39802" + integrity sha512-xS90H51cKw0jltxmvmHy2Iai1LIqrfbw57b79w/J7MfvDfkIkFZ+kj6zC3BjtUwh150HsSSdxXZcsuv72miDFQ== + dependencies: + "@types/esrecurse" "^4.3.1" + "@types/estree" "^1.0.8" + esrecurse "^4.3.0" + estraverse "^5.2.0" + +eslint-visitor-keys@^3.4.3: + version "3.4.3" + resolved "https://registry.yarnpkg.com/eslint-visitor-keys/-/eslint-visitor-keys-3.4.3.tgz#0cd72fe8550e3c2eae156a96a4dddcd1c8ac5800" + integrity sha512-wpc+LXeiyiisxPlEkUzU6svyS1frIO3Mgxj1fdy7Pm8Ygzguax2N3Fa/D/ag1WqbOprdI+uY6wMUl8/a2G+iag== + +eslint-visitor-keys@^5.0.1: + version "5.0.1" + resolved "https://registry.yarnpkg.com/eslint-visitor-keys/-/eslint-visitor-keys-5.0.1.tgz#9e3c9489697824d2d4ce3a8ad12628f91e9f59be" + integrity sha512-tD40eHxA35h0PEIZNeIjkHoDR4YjjJp34biM0mDvplBe//mB+IHCqHDGV7pxF+7MklTvighcCPPZC7ynWyjdTA== + +eslint@10.8.1: + version "10.8.1" + resolved "https://registry.yarnpkg.com/eslint/-/eslint-10.8.1.tgz#fb37d514c19b6dd5b2d6b70169fd26fddfa97967" + integrity sha512-wqA7W2jbsC/BnV9Iv1UZpKVFkO1AdNoSmYW8NWG4HNOBbkAMvIqDZ27pI2f07dqn583NcIC44ckjAcOXDL1QbQ== + dependencies: + "@eslint-community/eslint-utils" "^4.8.0" + "@eslint-community/regexpp" "^4.12.2" + "@eslint/config-array" "^0.23.5" + "@eslint/config-helpers" "^0.7.0" + "@eslint/core" "^1.2.1" + "@eslint/plugin-kit" "^0.7.2" + "@humanfs/node" "^0.16.6" + "@humanwhocodes/module-importer" "^1.0.1" + "@humanwhocodes/retry" "^0.4.2" + "@types/estree" "^1.0.6" + ajv "^6.14.0" + cross-spawn "^7.0.6" + debug "^4.3.2" + escape-string-regexp "^4.0.0" + eslint-scope "^9.1.2" + eslint-visitor-keys "^5.0.1" + espree "^11.2.0" + esquery "^1.7.0" + esutils "^2.0.2" + fast-deep-equal "^3.1.3" + file-entry-cache "^8.0.0" + find-up "^5.0.0" + glob-parent "^6.0.2" + ignore "^5.2.0" + imurmurhash "^0.1.4" + is-glob "^4.0.0" + json-stable-stringify-without-jsonify "^1.0.1" + minimatch "^10.2.5" + natural-compare "^1.4.0" + optionator "^0.9.3" + +espree@^11.2.0: + version "11.2.0" + resolved "https://registry.yarnpkg.com/espree/-/espree-11.2.0.tgz#01d5e47dc332aaba3059008362454a8cc34ccaa5" + integrity sha512-7p3DrVEIopW1B1avAGLuCSh1jubc01H2JHc8B4qqGblmg5gI9yumBgACjWo4JlIc04ufug4xJ3SQI8HkS/Rgzw== + dependencies: + acorn "^8.16.0" + acorn-jsx "^5.3.2" + eslint-visitor-keys "^5.0.1" + esprima@^4.0.0: version "4.0.1" resolved "https://registry.npmjs.org/esprima/-/esprima-4.0.1.tgz" integrity sha512-eGuFFw7Upda+g4p+QHvnW0RyTX/SVeJBDM/gCtMARO0cLuT2HcEKnTPvhjV6aGeqrCB/sbNop0Kszm0jsaWU4A== +esquery@^1.7.0: + version "1.7.0" + resolved "https://registry.yarnpkg.com/esquery/-/esquery-1.7.0.tgz#08d048f261f0ddedb5bae95f46809463d9c9496d" + integrity sha512-Ap6G0WQwcU/LHsvLwON1fAQX9Zp0A2Y6Y/cJBl9r/JbW90Zyg4/zbG6zzKa2OTALELarYHmKu0GhpM5EO+7T0g== + dependencies: + estraverse "^5.1.0" + +esrecurse@^4.3.0: + version "4.3.0" + resolved "https://registry.yarnpkg.com/esrecurse/-/esrecurse-4.3.0.tgz#7ad7964d679abb28bee72cec63758b1c5d2c9921" + integrity sha512-KmfKL3b6G+RXvP8N1vr3Tq1kL/oCFgn2NYXEtqP8/L3pKapUA4G8cFVaoF3SU323CD4XypR/ffioHmkti6/Tag== + dependencies: + estraverse "^5.2.0" + +estraverse@^5.1.0, estraverse@^5.2.0: + version "5.3.0" + resolved "https://registry.yarnpkg.com/estraverse/-/estraverse-5.3.0.tgz#2eea5290702f26ab8fe5370370ff86c965d21123" + integrity sha512-MMdARuVEQziNTeJD8DgMqmhwR11BRQ/cBP+pLtYdSTnf3MIO8fFeiINEbX36ZdNlfU/7A9f3gUw49B3oQsvwBA== + +esutils@^2.0.2: + version "2.0.3" + resolved "https://registry.yarnpkg.com/esutils/-/esutils-2.0.3.tgz#74d2eb4de0b8da1293711910d50775b9b710ef64" + integrity sha512-kVscqXk4OCp68SZ0dkgEKVi6/8ij300KBWTJq32P/dYeWTSwK41WyTxalN1eRmA5Z9UU/LX9D7FWSmV9SAYx6g== + ethereum-blockies-base64@^1.0.2: version "1.0.2" resolved "https://registry.npmjs.org/ethereum-blockies-base64/-/ethereum-blockies-base64-1.0.2.tgz" @@ -1605,11 +1830,21 @@ expect@30.2.0: jest-mock "30.2.0" jest-util "30.2.0" -fast-json-stable-stringify@^2.1.0: +fast-deep-equal@^3.1.1, fast-deep-equal@^3.1.3: + version "3.1.3" + resolved "https://registry.yarnpkg.com/fast-deep-equal/-/fast-deep-equal-3.1.3.tgz#3a7d56b559d6cbc3eb512325244e619a65c6c525" + integrity sha512-f3qQ9oQy9j2AhBe/H9VC91wLmKBCCU/gDOnKNAYG5hswO7BLKj09Hc5HYNz9cGI++xlpDCIgDaitVs03ATR84Q== + +fast-json-stable-stringify@^2.0.0, fast-json-stable-stringify@^2.1.0: version "2.1.0" resolved "https://registry.npmjs.org/fast-json-stable-stringify/-/fast-json-stable-stringify-2.1.0.tgz" integrity sha512-lhd/wF+Lk98HZoTCtlVraHtfh5XYijIjalXck7saUtuanSDyLMxnHhSXEDJqHxD7msR8D0uCmqlkwjCV8xvwHw== +fast-levenshtein@^2.0.6: + version "2.0.6" + resolved "https://registry.yarnpkg.com/fast-levenshtein/-/fast-levenshtein-2.0.6.tgz#3d8a5c66883a16a30ca8643e851f19baa7797917" + integrity sha512-DCXu6Ifhqcks7TZKY3Hxp3y6qphY5SJZmrWMDrKcERSOXWQdMhU9Ig/PYrzyw/ul9jOIyh0N4M0tbC5hodg8dw== + fb-watchman@^2.0.2: version "2.0.2" resolved "https://registry.npmjs.org/fb-watchman/-/fb-watchman-2.0.2.tgz" @@ -1617,6 +1852,13 @@ fb-watchman@^2.0.2: dependencies: bser "2.1.1" +file-entry-cache@^8.0.0: + version "8.0.0" + resolved "https://registry.yarnpkg.com/file-entry-cache/-/file-entry-cache-8.0.0.tgz#7787bddcf1131bffb92636c69457bbc0edd6d81f" + integrity sha512-XXTUwCvisa5oacNGRP9SfNtYBNAMi+RPwBFmblZEF7N7swHYQS6/Zfk7SRwx4D5j3CH211YNRco1DEMNVfZCnQ== + dependencies: + flat-cache "^4.0.0" + fill-range@^7.1.1: version "7.1.1" resolved "https://registry.npmjs.org/fill-range/-/fill-range-7.1.1.tgz" @@ -1632,6 +1874,27 @@ find-up@^4.0.0, find-up@^4.1.0: locate-path "^5.0.0" path-exists "^4.0.0" +find-up@^5.0.0: + version "5.0.0" + resolved "https://registry.yarnpkg.com/find-up/-/find-up-5.0.0.tgz#4c92819ecb7083561e4f4a240a86be5198f536fc" + integrity sha512-78/PXT1wlLLDgTzDs7sjq9hzz0vXD+zn+7wypEe4fXQxCmdmqfGsEPQxmiCSQI3ajFV91bVSsvNtrJRiW6nGng== + dependencies: + locate-path "^6.0.0" + path-exists "^4.0.0" + +flat-cache@^4.0.0: + version "4.0.1" + resolved "https://registry.yarnpkg.com/flat-cache/-/flat-cache-4.0.1.tgz#0ece39fcb14ee012f4b0410bd33dd9c1f011127c" + integrity sha512-f7ccFPK3SXFHpx15UIGyRJ/FJQctuKZ0zVuN3frBo4HnK3cay9VEW0R6yPYFHC0AgqhukPzKjq22t5DmAyqGyw== + dependencies: + flatted "^3.2.9" + keyv "^4.5.4" + +flatted@^3.2.9: + version "3.4.4" + resolved "https://registry.yarnpkg.com/flatted/-/flatted-3.4.4.tgz#aeeca2a506303f0cee61c59e6c9f2a88d2f29fc6" + integrity sha512-5+ybhBZANEJxaH3X5evAFatUxLfEHSr7n6kYJ+1Qd0mUqr4eu9gIf6GDbWHf8RJijHrjjO8G+la14SlL2SeS1Q== + foreground-child@^3.1.0: version "3.3.1" resolved "https://registry.npmjs.org/foreground-child/-/foreground-child-3.3.1.tgz" @@ -1670,6 +1933,13 @@ get-stream@^6.0.0: resolved "https://registry.npmjs.org/get-stream/-/get-stream-6.0.1.tgz" integrity sha512-ts6Wi+2j3jQjqi70w5AlN8DFnkSwC+MqmxEzdEALB2qXZYV3X/b1CTfgPLGJNMeAWxdPfU8FO1ms3NUfaHCPYg== +glob-parent@^6.0.2: + version "6.0.2" + resolved "https://registry.yarnpkg.com/glob-parent/-/glob-parent-6.0.2.tgz#6d237d99083950c79290f24c7642a3de9a28f9e3" + integrity sha512-XxwI8EOhVQgWp6iDL+3b0r86f4d6AX6zSU55HfB4ydCEuXLXc5FcYeOu+nnGftS4TEju/11rt4KJPTMgbfmv4A== + dependencies: + is-glob "^4.0.3" + glob@^10.3.10: version "10.5.0" resolved "https://registry.npmjs.org/glob/-/glob-10.5.0.tgz" @@ -1694,6 +1964,11 @@ glob@^7.1.4: once "^1.3.0" path-is-absolute "^1.0.0" +globals@17.11.0: + version "17.11.0" + resolved "https://registry.yarnpkg.com/globals/-/globals-17.11.0.tgz#d643485bb30220d7751e511cf4f68c73d3870d87" + integrity sha512-Z2I8hM+PbJDXQDq3Icgpzv+mPdwr68iZUU9d5WW4FuXfDUQfkZaZuvjMv42/5crNyw154+9+VWXbYrUgDXbxNw== + graceful-fs@^4.2.11, graceful-fs@^4.2.4: version "4.2.11" resolved "https://registry.npmjs.org/graceful-fs/-/graceful-fs-4.2.11.tgz" @@ -1714,6 +1989,11 @@ human-signals@^2.1.0: resolved "https://registry.npmjs.org/human-signals/-/human-signals-2.1.0.tgz" integrity sha512-B4FFZ6q/T2jhhksgkbEW3HBvWIfDW85snkQgawt07S7J5QXTk6BkNV+0yAeZrM5QpMAdYlocGoljn0sJ/WQkFw== +ignore@^5.2.0: + version "5.3.2" + resolved "https://registry.yarnpkg.com/ignore/-/ignore-5.3.2.tgz#3cd40e729f3643fd87cb04e50bf0eb722bc596f5" + integrity sha512-hsBTNUqQTDwkWtcdYI2i06Y/nUBEsNEDJKjWdigLvegy8kDuJAS8uRlpkkcQpyEXL0Z/pjDy5HBmMjRCJ2gq+g== + import-local@^3.2.0: version "3.2.0" resolved "https://registry.npmjs.org/import-local/-/import-local-3.2.0.tgz" @@ -1760,7 +2040,7 @@ is-generator-fn@^2.1.0: resolved "https://registry.npmjs.org/is-generator-fn/-/is-generator-fn-2.1.0.tgz" integrity sha512-cTIB4yPYL/Grw0EaSzASzg6bBy9gqCofvWN8okThAYIxKJZC+udlRAmGbM0XLeniEJSs8uEgHPGuHSe1XsOLSQ== -is-glob@^4.0.3: +is-glob@^4.0.0, is-glob@^4.0.3: version "4.0.3" resolved "https://registry.npmjs.org/is-glob/-/is-glob-4.0.3.tgz" integrity sha512-xelSayHH36ZgE7ZWhli7pW34hNbNl8Ojv5KVmkJD4hBdD3th8Tfk9vYasLM+mXWOZhFkgZfxhLSnrwRr4elSSg== @@ -2212,21 +2492,51 @@ jsesc@^3.0.2: resolved "https://registry.npmjs.org/jsesc/-/jsesc-3.1.0.tgz" integrity sha512-/sM3dO2FOzXjKQhJuo0Q173wf2KOo8t4I8vHy6lF9poUp7bKT0/NHE8fPX23PwfhnykfqnC2xRxOnVw5XuGIaA== +json-buffer@3.0.1: + version "3.0.1" + resolved "https://registry.yarnpkg.com/json-buffer/-/json-buffer-3.0.1.tgz#9338802a30d3b6605fbe0613e094008ca8c05a13" + integrity sha512-4bV5BfR2mqfQTJm+V5tPPdf+ZpuhiIvTuAB5g8kcrXOZpTT/QwwVRWBywX1ozr6lEuPdbHxwaJlm9G6mI2sfSQ== + json-parse-even-better-errors@^2.3.0: version "2.3.1" resolved "https://registry.npmjs.org/json-parse-even-better-errors/-/json-parse-even-better-errors-2.3.1.tgz" integrity sha512-xyFwyhro/JEof6Ghe2iz2NcXoj2sloNsWr/XsERDK/oiPCfaNhl5ONfp+jQdAZRQQ0IJWNzH9zIZF7li91kh2w== +json-schema-traverse@^0.4.1: + version "0.4.1" + resolved "https://registry.yarnpkg.com/json-schema-traverse/-/json-schema-traverse-0.4.1.tgz#69f6a87d9513ab8bb8fe63bdb0979c448e684660" + integrity sha512-xbbCH5dCYU5T8LcEhhuh7HJ88HXuW3qsI3Y0zOZFKfZEHcpWiHU/Jxzk629Brsab/mMiHQti9wMP+845RPe3Vg== + +json-stable-stringify-without-jsonify@^1.0.1: + version "1.0.1" + resolved "https://registry.yarnpkg.com/json-stable-stringify-without-jsonify/-/json-stable-stringify-without-jsonify-1.0.1.tgz#9db7b59496ad3f3cfef30a75142d2d930ad72651" + integrity sha512-Bdboy+l7tA3OGW6FjyFHWkP5LuByj1Tk33Ljyq0axyzdk9//JSi2u3fP1QSmd1KNwq6VOKYGlAu87CisVir6Pw== + json5@^2.2.3: version "2.2.3" resolved "https://registry.npmjs.org/json5/-/json5-2.2.3.tgz" integrity sha512-XmOWe7eyHYH14cLdVPoyg+GOH3rYX++KpzrylJwSW98t3Nk+U8XOl8FWKOgwtzdb8lXGf6zYwDUzeHMWfxasyg== +keyv@^4.5.4: + version "4.5.4" + resolved "https://registry.yarnpkg.com/keyv/-/keyv-4.5.4.tgz#a879a99e29452f942439f2a405e3af8b31d4de93" + integrity sha512-oxVHkHR/EJf2CNXnWxRLW6mg7JyCCUcG0DtEGmL2ctUo1PNTin1PUil+r/+4r5MpVgC/fn1kjsx7mjSujKqIpw== + dependencies: + json-buffer "3.0.1" + leven@^3.1.0: version "3.1.0" resolved "https://registry.npmjs.org/leven/-/leven-3.1.0.tgz" integrity sha512-qsda+H8jTaUaN/x5vzW2rzc+8Rw4TAQ/4KjB46IwK5VH+IlVeeeje/EoZRpiXvIqjFgK84QffqPztGI3VBLG1A== +levn@^0.4.1: + version "0.4.1" + resolved "https://registry.yarnpkg.com/levn/-/levn-0.4.1.tgz#ae4562c007473b932a6200d403268dd2fffc6ade" + integrity sha512-+bT2uH4E5LGE7h/n3evcS/sQlJXCpIp6ym8OWJ5eV6+67Dsql/LaaT7qJBAt2rzfoa/5QBGBhxDix1dMt2kQKQ== + dependencies: + prelude-ls "^1.2.1" + type-check "~0.4.0" + libsodium-sumo@^0.8.0: version "0.8.2" resolved "https://registry.npmjs.org/libsodium-sumo/-/libsodium-sumo-0.8.2.tgz" @@ -2325,6 +2635,13 @@ locate-path@^5.0.0: dependencies: p-locate "^4.1.0" +locate-path@^6.0.0: + version "6.0.0" + resolved "https://registry.yarnpkg.com/locate-path/-/locate-path-6.0.0.tgz#55321eb309febbc59c4801d931a72452a681d286" + integrity sha512-iPZK6eYjbxRu3uB4/WZ3EsEIMJFMqAoopl3R+zuq0UjcAm/MO6KCweDgPfP3elTztoKP3KtnVHxTn2NHBSDVUw== + dependencies: + p-locate "^5.0.0" + lru-cache@^10.2.0: version "10.4.3" resolved "https://registry.npmjs.org/lru-cache/-/lru-cache-10.4.3.tgz" @@ -2376,6 +2693,13 @@ mimic-fn@^2.1.0: resolved "https://registry.npmjs.org/mimic-fn/-/mimic-fn-2.1.0.tgz" integrity sha512-OqbOk5oEQeAZ8WXWydlu9HJjz9WVdEIvamMCcXmuqUYjTknH/sqsWvhQ3vgwKFRR1HpjvNBKQ37nbJgYzGqGcg== +minimatch@^10.2.4, minimatch@^10.2.5: + version "10.2.6" + resolved "https://registry.yarnpkg.com/minimatch/-/minimatch-10.2.6.tgz#fd956bbe0b77241e9f15ac5dccb1c638060968ef" + integrity sha512-vpLQEs+VLCr1nU0BXS07maYoFwlDAH0gngQuuttxIwutDFEMHq2blX+8vpgxDdK3J1PwjCJiep77OitTZ4Ll1A== + dependencies: + brace-expansion "^5.0.8" + minimatch@^3.0.4, minimatch@^3.1.1: version "3.1.3" resolved "https://registry.npmjs.org/minimatch/-/minimatch-3.1.3.tgz" @@ -2456,6 +2780,18 @@ onetime@^5.1.2: dependencies: mimic-fn "^2.1.0" +optionator@^0.9.3: + version "0.9.4" + resolved "https://registry.yarnpkg.com/optionator/-/optionator-0.9.4.tgz#7ea1c1a5d91d764fb282139c88fe11e182a3a734" + integrity sha512-6IpQ7mKUxRcZNLIObR0hz7lxsapSSIYNZJwXPGeF0mTVqGKFIXj1DQcMoT22S3ROcLyY/rz0PWaWZ9ayWmad9g== + dependencies: + deep-is "^0.1.3" + fast-levenshtein "^2.0.6" + levn "^0.4.1" + prelude-ls "^1.2.1" + type-check "^0.4.0" + word-wrap "^1.2.5" + p-limit@^2.2.0: version "2.3.0" resolved "https://registry.npmjs.org/p-limit/-/p-limit-2.3.0.tgz" @@ -2463,7 +2799,7 @@ p-limit@^2.2.0: dependencies: p-try "^2.0.0" -p-limit@^3.1.0: +p-limit@^3.0.2, p-limit@^3.1.0: version "3.1.0" resolved "https://registry.npmjs.org/p-limit/-/p-limit-3.1.0.tgz" integrity sha512-TYOanM3wGwNGsZN2cVTYPArw454xnXj5qmWF1bEoAc4+cU/ol7GVh7odevjp1FNHduHc3KZMcFduxU5Xc6uJRQ== @@ -2477,6 +2813,13 @@ p-locate@^4.1.0: dependencies: p-limit "^2.2.0" +p-locate@^5.0.0: + version "5.0.0" + resolved "https://registry.yarnpkg.com/p-locate/-/p-locate-5.0.0.tgz#83c8315c6785005e3bd021839411c9e110e6d834" + integrity sha512-LaNjtRWUBY++zB5nE/NwcaoMylSPk+S+ZHNB1TzdbMJMny6dynpAGt7X/tl/QYq3TIeE6nxHppbo2LGymrG5Pw== + dependencies: + p-limit "^3.0.2" + p-try@^2.0.0: version "2.2.0" resolved "https://registry.npmjs.org/p-try/-/p-try-2.2.0.tgz" @@ -2562,6 +2905,11 @@ pnglib@0.0.1: resolved "https://registry.npmjs.org/pnglib/-/pnglib-0.0.1.tgz" integrity sha512-95ChzOoYLOPIyVmL+Y6X+abKGXUJlvOVLkB1QQkyXl7Uczc6FElUy/x01NS7r2GX6GRezloO/ecCX9h4U9KadA== +prelude-ls@^1.2.1: + version "1.2.1" + resolved "https://registry.yarnpkg.com/prelude-ls/-/prelude-ls-1.2.1.tgz#debc6489d7a6e6b0e7611888cec880337d316396" + integrity sha512-vkcDPrRZo1QZLbn5RLGPpg/WmIQ65qoWWhcGKf/b5eplkkarX0m9z8ppCat4mlOqUsWpyNuYgO3VRyrYHSzX5g== + prettier@^3.8.1: version "3.8.1" resolved "https://registry.npmjs.org/prettier/-/prettier-3.8.1.tgz" @@ -2576,6 +2924,11 @@ pretty-format@30.2.0: ansi-styles "^5.2.0" react-is "^18.3.1" +punycode@^2.1.0: + version "2.3.1" + resolved "https://registry.yarnpkg.com/punycode/-/punycode-2.3.1.tgz#027422e2faec0b25e1549c3e1bd8309b9133b6e5" + integrity sha512-vYt7UD1U9Wg6138shLtLOvdAu+8DsC/ilFtEVHcH+wydcSpNE20AfSOduf6MkRFahL5FY7X1oU7nKVZFtfq8Fg== + pure-rand@^7.0.0: version "7.0.1" resolved "https://registry.npmjs.org/pure-rand/-/pure-rand-7.0.1.tgz" @@ -2822,6 +3175,13 @@ tslib@^2.8.1: resolved "https://registry.yarnpkg.com/tslib/-/tslib-2.8.1.tgz#612efe4ed235d567e8aba5f2a5fab70280ade83f" integrity sha512-oJFu94HQb+KVduSUQL7wnpmqnfmLsOA/nAh6b6EH0wCEoK0/mPeXU6c3wKDV83MkOuHPRHtSXKKU99IBazS/2w== +type-check@^0.4.0, type-check@~0.4.0: + version "0.4.0" + resolved "https://registry.yarnpkg.com/type-check/-/type-check-0.4.0.tgz#07b8203bfa7056c0657050e3ccd2c37730bab8f1" + integrity sha512-XleUoc9uwGXqjWwXaUTZAmzMcFZ5858QA2vvx1Ur5xIcixXIP+8LnFDgRplU30us6teqdlskFfu+ae4K79Ooew== + dependencies: + prelude-ls "^1.2.1" + type-detect@4.0.8: version "4.0.8" resolved "https://registry.npmjs.org/type-detect/-/type-detect-4.0.8.tgz" @@ -2872,6 +3232,13 @@ update-browserslist-db@^1.2.0: escalade "^3.2.0" picocolors "^1.1.1" +uri-js@^4.2.2: + version "4.4.1" + resolved "https://registry.yarnpkg.com/uri-js/-/uri-js-4.4.1.tgz#9b1a52595225859e55f669d928f88c6c57f2a77e" + integrity sha512-7rKUyy33Q1yc98pQ1DAmLtwX109F7TIfWlW1Ydo8Wl1ii1SeHieeh0HHfPeL2fMXK6z0s8ecKs9frCuLJvndBg== + dependencies: + punycode "^2.1.0" + v8-to-istanbul@^9.0.1: version "9.3.0" resolved "https://registry.npmjs.org/v8-to-istanbul/-/v8-to-istanbul-9.3.0.tgz" @@ -2900,6 +3267,11 @@ which@^2.0.1: dependencies: isexe "^2.0.0" +word-wrap@^1.2.5: + version "1.2.5" + resolved "https://registry.yarnpkg.com/word-wrap/-/word-wrap-1.2.5.tgz#d2c45c6dd4fbce621a66f136cbe328afd0410b34" + integrity sha512-BN22B5eaMMI9UMtjrGd5g5eCYPpCPDUy0FJXbYsaT5zYxjFOckS53SQDE3pWkVoWpHXVb3BrYcEN4Twa55B5cA== + "wrap-ansi-cjs@npm:wrap-ansi@^7.0.0": version "7.0.0" resolved "https://registry.npmjs.org/wrap-ansi/-/wrap-ansi-7.0.0.tgz"