Apply latest formatting/linting config

The main difference is the bump to prettier 3, which switches
`trailingComma` to `all` by default. This hits on the order of 3000
lines by adding trailing commas to all function invocations and param
lists.

.github/workflows/builds/detect-env-block.js

@@ -29,7 +29,7 @@ module.exports = async function detectEnvBlock({ github, context }) {
       context.issue?.number,
       "; status code was ",
       status,
-      ". "
+      ". ",
     )
     return ""
   }
@@ -46,15 +46,17 @@ module.exports = async function detectEnvBlock({ github, context }) {
         varName.startsWith("ENABLE_") ||
         varName.startsWith("USE_") ||
         varName.startsWith("SHOW_") ||
-        varName.startsWith("SUPPORT_")
+        varName.startsWith("SUPPORT_"),
     )
     .map((varPair) => varPair.join("="))
 
   const validEnvBlock = validEnvLines.join("\n")
 
   if (envLines.length !== validEnvLines.length) {
     console.warn(
-      `Filtered ${validEnvLines.length - envLines.length} disallowed env lines.`
+      `Filtered ${
+        validEnvLines.length - envLines.length
+      } disallowed env lines.`,
     )
   }
   console.log("Detected env block", validEnvBlock)

.github/workflows/builds/post-build-link.js

@@ -13,7 +13,7 @@ module.exports = async function postBuildLink({ github, context }) {
   const manualWorkFlowId = context?.inputs?.workflow_run_id
 
   const workflowRunId = Number(
-    context.payload?.workflow_run?.id ?? manualWorkFlowId
+    context.payload?.workflow_run?.id ?? manualWorkFlowId,
   )
 
   if (Number.isNaN(workflowRunId)) {
@@ -31,7 +31,7 @@ module.exports = async function postBuildLink({ github, context }) {
 
   if (workflowLookupStatus !== 200) {
     throw new Error(
-      `Failed to fetch workflow :( Status ${workflowLookupStatus}.`
+      `Failed to fetch workflow :( Status ${workflowLookupStatus}.`,
     )
   }
 
@@ -46,36 +46,36 @@ module.exports = async function postBuildLink({ github, context }) {
 
   if (artifactLookupStatus !== 200) {
     throw new Error(
-      `Failed to fetch workflow artifacts :( Status ${artifactLookupStatus}.`
+      `Failed to fetch workflow artifacts :( Status ${artifactLookupStatus}.`,
     )
   }
 
   const matchArtifact = allArtifacts.filter((artifact) =>
-    artifact.name.startsWith("extension-builds-")
+    artifact.name.startsWith("extension-builds-"),
   )[0]
 
   if (matchArtifact === undefined || matchArtifact === null) {
     throw new Error(
       `Failed to find extension artifact :( Artifacts were ${JSON.stringify(
-        allArtifacts
-      )}`
+        allArtifacts,
+      )}`,
     )
   }
 
   const prNumber = matchArtifact.name.match(/extension-builds-(.*)/)?.[1]
 
   if (prNumber === undefined) {
     throw new Error(
-      `Could not extract PR number from extension artifact filename (${matchArtifact.name}) :(`
+      `Could not extract PR number from extension artifact filename (${matchArtifact.name}) :(`,
     )
   } else if (prNumber.match(/^[a-f0-9]+$/) && !prNumber.match(/^[0-9]+$/)) {
     console.log(
-      "Workflow was for a merge commit rather than a PR, skipping build link."
+      "Workflow was for a merge commit rather than a PR, skipping build link.",
     )
     return
   } else if (!prNumber.match(/^[0-9]+$/)) {
     throw new Error(
-      `Could not extract PR number from extension artifact filename (${matchArtifact.name}) :(`
+      `Could not extract PR number from extension artifact filename (${matchArtifact.name}) :(`,
     )
   }
 
@@ -96,14 +96,14 @@ module.exports = async function postBuildLink({ github, context }) {
   const artifactUrl = `${baseUrl}/suites/${checkSuiteId}/artifacts/${matchArtifact.id}`
 
   console.log(
-    `Detected artifact ${matchArtifact.name} at ${artifactUrl}, posting...`
+    `Detected artifact ${matchArtifact.name} at ${artifactUrl}, posting...`,
   )
 
   const updatedBody = `${(body ?? "").replace(
     /\s+Latest build: [^\n]*/,
-    ""
+    "",
   )}\n\nLatest build: [${matchArtifact.name}](${artifactUrl}) (as of ${new Date(
-    workflowUpdatedAt
+    workflowUpdatedAt,
   ).toUTCString()}).`
 
   await github.rest.pulls.update({

.github/workflows/pledge-signer-sync/pledge-sync.js

@@ -43,7 +43,7 @@ const getAddresses = async () => {
   await signInWithEmailAndPassword(
     getAuth(app),
     FIRESTORE_USER,
-    FIRESTORE_PASSWORD
+    FIRESTORE_PASSWORD,
   )
 
   const db = getFirestore(app)
@@ -56,7 +56,7 @@ const getAddresses = async () => {
       dbCollection,
       orderBy("signedManifesto.timestamp", "desc"),
       limit(CHUNK_SIZE),
-      where("signedManifesto.timestamp", ">=", TARGET_DATE)
+      where("signedManifesto.timestamp", ">=", TARGET_DATE),
     )
 
     if (offset) {

__mocks__/@ethersproject/web.ts

@@ -4,7 +4,7 @@ import sinon from "sinon"
 
 const mock =
   jest.createMockFromModule<typeof import("@ethersproject/web")>(
-    "@ethersproject/web"
+    "@ethersproject/web",
   )
 
 const actual =

__mocks__/webextension-polyfill.ts

@@ -25,7 +25,7 @@ module.exports = {
     getCurrent: jest.fn(() =>
       // getCurrent can return undefined if there is no tab, and we act accordingly
       // in the code.
-      Promise.resolve(undefined as unknown as Tabs.Tab)
+      Promise.resolve(undefined as unknown as Tabs.Tab),
     ),
   },
   windows: {

background/abilities.ts

@@ -39,7 +39,7 @@ export const ABILITY_TYPES = [
   "misc",
 ] as const
 
-export type AbilityType = typeof ABILITY_TYPES[number]
+export type AbilityType = (typeof ABILITY_TYPES)[number]
 
 export type Ability = {
   type: AbilityType

background/assets.ts

@@ -220,7 +220,7 @@ export function isFungibleAsset(asset: AnyAsset): asset is FungibleAsset {
  * Type guard to check if an AnyAsset is actually a SmartContractFungibleAsset.
  */
 export function isSmartContractFungibleAsset<T extends AnyAsset>(
-  asset: T
+  asset: T,
 ): asset is T & SmartContractFungibleAsset {
   return "homeNetwork" in asset && isFungibleAsset(asset)
 }
@@ -232,7 +232,7 @@ export function isSmartContractFungibleAsset<T extends AnyAsset>(
  * FungibleAsset nature of the internal asset!
  */
 export function isFungibleAssetAmount(
-  assetAmount: AnyAssetAmount
+  assetAmount: AnyAssetAmount,
 ): assetAmount is FungibleAssetAmount {
   return isFungibleAsset(assetAmount.asset)
 }
@@ -279,7 +279,7 @@ export function flipPricePoint(pricePoint: PricePoint): PricePoint {
  */
 export function convertAssetAmountViaPricePoint<T extends AnyAssetAmount>(
   sourceAssetAmount: T,
-  assetPricePoint: PricePoint | undefined
+  assetPricePoint: PricePoint | undefined,
 ): FungibleAssetAmount | undefined {
   if (typeof assetPricePoint === "undefined") {
     return undefined
@@ -350,7 +350,7 @@ export function convertAssetAmountViaPricePoint<T extends AnyAssetAmount>(
  * is undefined.
  */
 export function unitPricePointForPricePoint(
-  assetPricePoint: PricePoint | undefined
+  assetPricePoint: PricePoint | undefined,
 ): UnitPricePoint<FungibleAsset> | undefined {
   if (typeof assetPricePoint === "undefined") {
     return undefined
@@ -366,7 +366,7 @@ export function unitPricePointForPricePoint(
           : 1n,
       asset: sourceAsset,
     },
-    assetPricePoint
+    assetPricePoint,
   )
 
   if (typeof unitPrice !== "undefined") {
@@ -394,7 +394,7 @@ export function unitPricePointForPricePoint(
  */
 export function assetAmountToDesiredDecimals(
   assetAmount: FungibleAssetAmount,
-  desiredDecimals: number
+  desiredDecimals: number,
 ): number {
   const {
     amount,

background/constants/coin-types.ts

@@ -16,4 +16,4 @@ export const coinTypesByAssetSymbol = {
  * All coin types known to the extension.
  */
 export type Slip44CoinType =
-  typeof coinTypesByAssetSymbol[keyof typeof coinTypesByAssetSymbol]
+  (typeof coinTypesByAssetSymbol)[keyof typeof coinTypesByAssetSymbol]

background/constants/currencies.ts

@@ -11,7 +11,7 @@ export const USD: FiatCurrency = {
 
 export const FIAT_CURRENCIES = [USD]
 export const FIAT_CURRENCIES_SYMBOL = FIAT_CURRENCIES.map(
-  (currency) => currency.symbol
+  (currency) => currency.symbol,
 )
 
 export const ETH_DATA = {

background/constants/networks.ts

@@ -107,12 +107,12 @@ export const DEFAULT_NETWORKS = [
 
 export function isBuiltInNetwork(network: EVMNetwork): boolean {
   return DEFAULT_NETWORKS.some(
-    (builtInNetwork) => builtInNetwork.chainID === network.chainID
+    (builtInNetwork) => builtInNetwork.chainID === network.chainID,
   )
 }
 
 export const DEFAULT_NETWORKS_BY_CHAIN_ID = new Set(
-  DEFAULT_NETWORKS.map((network) => network.chainID)
+  DEFAULT_NETWORKS.map((network) => network.chainID),
 )
 
 export const FORK: EVMNetwork = {
@@ -124,13 +124,13 @@ export const FORK: EVMNetwork = {
 }
 
 export const EIP_1559_COMPLIANT_CHAIN_IDS = new Set(
-  [ETHEREUM, POLYGON, GOERLI, AVALANCHE].map((network) => network.chainID)
+  [ETHEREUM, POLYGON, GOERLI, AVALANCHE].map((network) => network.chainID),
 )
 
 export const CHAINS_WITH_MEMPOOL = new Set(
   [ETHEREUM, POLYGON, AVALANCHE, GOERLI, BINANCE_SMART_CHAIN].map(
-    (network) => network.chainID
-  )
+    (network) => network.chainID,
+  ),
 )
 
 export const NETWORK_BY_CHAIN_ID = {
@@ -148,7 +148,7 @@ export const NETWORK_BY_CHAIN_ID = {
 }
 
 export const TEST_NETWORK_BY_CHAIN_ID = new Set(
-  [GOERLI].map((network) => network.chainID)
+  [GOERLI].map((network) => network.chainID),
 )
 
 export const NETWORK_FOR_LEDGER_SIGNING = [ETHEREUM, POLYGON]
@@ -168,13 +168,13 @@ export const CHAIN_ID_TO_0X_API_BASE: {
 }
 
 export const NETWORKS_SUPPORTING_SWAPS = new Set(
-  Object.keys(CHAIN_ID_TO_0X_API_BASE)
+  Object.keys(CHAIN_ID_TO_0X_API_BASE),
 )
 
 export const ALCHEMY_SUPPORTED_CHAIN_IDS = new Set(
   [ETHEREUM, POLYGON, ARBITRUM_ONE, OPTIMISM, GOERLI].map(
-    (network) => network.chainID
-  )
+    (network) => network.chainID,
+  ),
 )
 
 export const FLASHBOTS_SUPPORTED_CHAIN_IDS = new Set([ETHEREUM.chainID])
@@ -251,5 +251,7 @@ export const CHAIN_ID_TO_OPENSEA_CHAIN = {
 }
 
 export const NETWORKS_WITH_FEE_SETTINGS = new Set(
-  [ETHEREUM, POLYGON, ARBITRUM_ONE, AVALANCHE].map((network) => network.chainID)
+  [ETHEREUM, POLYGON, ARBITRUM_ONE, AVALANCHE].map(
+    (network) => network.chainID,
+  ),
 )

background/features.ts

@@ -58,7 +58,7 @@ export const FeatureFlags = Object.keys({
  */
 export function isEnabled(
   flagName: FeatureFlagType,
-  checkBrowserStorage: boolean = BuildTimeFlag.SWITCH_RUNTIME_FLAGS
+  checkBrowserStorage: boolean = BuildTimeFlag.SWITCH_RUNTIME_FLAGS,
 ): boolean {
   // Guard to narrow flag type
   const isBuildTimeFlag = (flag: string): flag is BuildTimeFlagType =>
@@ -82,7 +82,7 @@ export function isEnabled(
  */
 export function isDisabled(
   flagName: FeatureFlagType,
-  checkBrowserStorage: boolean = BuildTimeFlag.SWITCH_RUNTIME_FLAGS
+  checkBrowserStorage: boolean = BuildTimeFlag.SWITCH_RUNTIME_FLAGS,
 ): boolean {
   return !isEnabled(flagName, checkBrowserStorage)
 }
@@ -105,7 +105,7 @@ export function isDisabled(
  */
 export function wrapIfEnabled<T>(
   flag: FeatureFlagType,
-  valueToWrap: T
+  valueToWrap: T,
 ): [T] | [] {
   return isEnabled(flag) ? [valueToWrap] : []
 }
@@ -117,7 +117,7 @@ export function wrapIfEnabled<T>(
  */
 export function wrapIfDisabled<T>(
   flag: FeatureFlagType,
-  valueToWrap: T
+  valueToWrap: T,
 ): [T] | [] {
   return isDisabled(flag) ? [valueToWrap] : []
 }

background/generate-validators.ts

@@ -35,19 +35,19 @@ const ajvJSON = new AjvJSON({
 
 const jtdModuleCode = standaloneCode(ajvJTD).replace(
   '/*# sourceURL="https://uniswap.org/tokenlist.schema.json" */',
-  ""
+  "",
 )
 const jsonModuleCode = standaloneCode(ajvJSON).replace(
   '/*# sourceURL="https://uniswap.org/tokenlist.schema.json" */',
-  ""
+  "",
 )
 
 writeFileSync(
   path.join(__dirname, "/lib/validate/jtd-validators.js"),
-  jtdModuleCode
+  jtdModuleCode,
 )
 
 writeFileSync(
   path.join(__dirname, "/lib/validate/json-validators.js"),
-  jsonModuleCode
+  jsonModuleCode,
 )

background/index.ts

@@ -23,7 +23,7 @@ type BackgroundAsyncThunkDispatch<S, E, A extends Action> = {
   // script, thus only going through a simple browser port that is expected to
   // be reliable except in particularly unusual circumstances..
   <TypePrefix extends string, Returned>(
-    asyncAction: BackgroundAsyncThunkAction<TypePrefix, Returned>
+    asyncAction: BackgroundAsyncThunkAction<TypePrefix, Returned>,
   ): Promise<Returned>
   <T extends A>(action: T): Promise<T>
   <R>(asyncAction: ThunkAction<R, S, E, A>): R
@@ -70,7 +70,7 @@ export async function newProxyStore(): Promise<
       }
 
       const result = produce(oldObj, (draft) =>
-        patchDeepDiff(draft, patchesWrapper[0])
+        patchDeepDiff(draft, patchesWrapper[0]),
       )
 
       return result