chore: move prettier to individual packages & upgrade prettier

This commit makes the following connected changes:

- Removes prettier from `pre-commit` and instead moves it to be run by `turbo`,
  hooking into `test:format` and `fix:format`
- Fixes some of the linting scripts where they were incorrectly configured for
  turbo (i.e. renames the scripts to `fix:lint` and `test:lint` as needed)
- Upgrades prettier in all packgaes

Author: cprussin

.pre-commit-config.yaml

@@ -23,16 +23,6 @@ repos:
             target_chains/sui/vendor/|
             patches/
           )
-  # Hook to format many type of files in the repo
-  # including solidity contracts.
-  - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: "v2.7.1"
-    hooks:
-      - id: prettier
-        additional_dependencies:
-          - "[email protected]"
-          - "[email protected]"
-        exclude: target_chains/sui/vendor/
   - repo: local
     hooks:
       # Hooks for the remote executor

.prettierignore

@@ -1,25 +1,39 @@
 pnpm-lock.yaml
 patches/
 .turbo/
+target_chains/sui/vendor/
 
-# These apps have their own prettier config that uses a later version of
-# prettier
-#
-# TODO(cprussin): eventually I'll figure out how to upgrade prettier everywhere
-# and hook it in to pre-commit.  For now, I don't want to downgrade prettier in
-# the packages that are using the later version, and pre-commit doesn't support
-# later versions of prettier directly.
-#
-# Ideally, we should probably hook up a pre-commit script to run auto-fixes in a
-# generic way that packages can hook into by defining lifecycle scripts, or by
-# using the nx task graph.  Then, packages can use their own formatters /
-# formatter versions and can also hook up other auto-fixes like eslint, etc.
-#
-# I'll explore doing this when I get around to spending some time on our nx
-# build graph config.
-apps/api-reference
-apps/staking
-apps/insights
-apps/entropy-debugger
-governance/pyth_staking_sdk
+# This should stay in sync with the list of workspaces from
+# `pnpm-workspace.yaml`
+apps/*
 packages/*
+apps/hermes/client/js
+governance/xc_admin/packages/*
+governance/pyth_staking_sdk
+price_service/sdk/js
+price_service/client/js
+pythnet/message_buffer
+target_chains/aptos/cli
+target_chains/aptos/sdk/js
+target_chains/cosmwasm/tools
+target_chains/cosmwasm/deploy-scripts
+target_chains/ethereum/contracts
+target_chains/ethereum/abi_generator
+target_chains/ethereum/entropy_sdk/solidity
+target_chains/ethereum/sdk/js
+target_chains/ethereum/sdk/solidity
+target_chains/ethereum/sdk/stylus/pyth-mock-solidity
+target_chains/ethereum/examples/coin_flip/app
+target_chains/fuel/sdk/js
+target_chains/starknet/sdk/js
+target_chains/sui/sdk/js
+target_chains/sui/sdk/js-iota
+target_chains/sui/cli
+target_chains/sui/cli-iota
+target_chains/solana/sdk/js/solana_utils
+target_chains/solana/sdk/js/pyth_solana_receiver
+target_chains/ton/contracts
+target_chains/ton/sdk/js
+contract_manager
+lazer/contracts/solana
+lazer/sdk/js

apps/api-reference/package.json

@@ -60,7 +60,7 @@
     "eslint": "^9.5.0",
     "jest": "^29.7.0",
     "postcss": "^8.4.38",
-    "prettier": "^3.3.2",
+    "prettier": "^3.5.3",
     "tailwindcss": "^3.4.4",
     "typescript": "^5.5.2",
     "vercel": "^34.2.7"

apps/hermes/client/js/package.json

@@ -24,8 +24,10 @@
     "build:schemas": "openapi-zod-client ./schema.json --output src/zodSchemas.ts",
     "pull:schema": "curl -o schema.json -z schema.json https://hermes.pyth.network/docs/openapi.json",
     "example": "node lib/examples/HermesClient.js",
-    "format": "prettier --write \"src/**/*.ts\"",
+    "fix:lint": "eslint src/ --fix --max-warnings 0",
+    "fix:format": "prettier --write \"src/**/*.ts\"",
     "test:lint": "eslint src/ --max-warnings 0",
+    "test:format": "prettier --check \"src/**/*.ts\"",
     "prepublishOnly": "pnpm run build:typescript && pnpm run test:lint",
     "preversion": "pnpm run test:lint",
     "version": "pnpm run format && git add -A src"
@@ -44,7 +46,7 @@
     "eslint": "^8.14.0",
     "jest": "^29.4.0",
     "openapi-zod-client": "^1.18.1",
-    "prettier": "^2.6.2",
+    "prettier": "^3.5.3",
     "ts-jest": "^29.0.5",
     "typescript": "catalog:",
     "yargs": "^17.4.1"

apps/price_pusher/package.json

@@ -21,7 +21,9 @@
   },
   "scripts": {
     "build": "tsc",
-    "format": "prettier --write \"src/**/*.ts\"",
+    "fix:format": "prettier --write \"src/**/*.ts\"",
+    "fix:lint": "eslint src/ --fix --max-warnings 0",
+    "test:format": "prettier --check \"src/**/*.ts\"",
     "test:lint": "eslint src/ --max-warnings 0",
     "start": "node lib/index.js",
     "dev": "ts-node src/index.ts",
@@ -50,7 +52,7 @@
     "eslint": "^8.13.0",
     "jest": "^29.7.0",
     "pino-pretty": "^11.2.1",
-    "prettier": "^2.6.2",
+    "prettier": "^3.5.3",
     "ts-jest": "^29.1.1",
     "ts-node": "^10.9.1",
     "typescript": "^5.3.3"

apps/staking/package.json

@@ -68,7 +68,7 @@
     "eslint": "^9.8.0",
     "jest": "^29.7.0",
     "postcss": "^8.4.40",
-    "prettier": "^3.3.2",
+    "prettier": "^3.5.3",
     "tailwindcss": "^3.4.7",
     "tailwindcss-animate": "^1.0.7",
     "tailwindcss-react-aria-components": "^1.1.5",

contract_manager/package.json

@@ -11,8 +11,10 @@
   "scripts": {
     "build": "tsc",
     "shell": "ts-node ./src/shell.ts",
+    "fix:lint": "eslint src/ scripts/ --fix --max-warnings 0",
+    "fix:format": "prettier --write \"src/**/*.ts\" \"scripts/**/*.ts\"",
     "test:lint": "eslint src/ scripts/ --max-warnings 0",
-    "format": "prettier --write \"src/**/*.ts\" \"scripts/**/*.ts\""
+    "test:format": "prettier --check \"src/**/*.ts\" \"scripts/**/*.ts\""
   },
   "author": "",
   "license": "Apache-2.0",
@@ -34,10 +36,10 @@
     "@pythnetwork/hermes-client": "workspace:*",
     "@pythnetwork/price-service-client": "workspace:*",
     "@pythnetwork/pyth-fuel-js": "workspace:*",
+    "@pythnetwork/pyth-iota-js": "workspace:*",
     "@pythnetwork/pyth-sdk-solidity": "workspace:^",
     "@pythnetwork/pyth-starknet-js": "^0.2.1",
     "@pythnetwork/pyth-sui-js": "workspace:*",
-    "@pythnetwork/pyth-iota-js": "workspace:*",
     "@pythnetwork/pyth-ton": "workspace:*",
     "@pythnetwork/pyth-ton-js": "workspace:*",
     "@pythnetwork/solana-utils": "workspace:^",
@@ -67,7 +69,7 @@
   "devDependencies": {
     "@types/web3": "^1.2.2",
     "eslint": "^8.0.0",
-    "prettier": "^2.6.2",
+    "prettier": "^3.5.3",
     "typedoc": "^0.25.7"
   }
 }

governance/pyth_staking_sdk/package.json

@@ -30,7 +30,7 @@
     "@typescript-eslint/parser": "^8.3.0",
     "eslint": "^9.8.0",
     "jest": "^29.7.0",
-    "prettier": "^3.3.2",
+    "prettier": "^3.5.3",
     "typescript": "^5.5.4"
   },
   "dependencies": {

governance/xc_admin/packages/crank_executor/package.json

@@ -17,7 +17,8 @@
   "scripts": {
     "build": "tsc",
     "start": "node lib/index.js",
-    "format": "prettier --write \"src/**/*.ts\""
+    "fix:format": "prettier --write \"src/**/*.ts\"",
+    "test:format": "prettier --check \"src/**/*.ts\""
   },
   "dependencies": {
     "@certusone/wormhole-sdk": "^0.10.15",

governance/xc_admin/packages/crank_pythnet_relayer/package.json

@@ -17,7 +17,8 @@
   "scripts": {
     "build": "tsc",
     "start": "node lib/index.js",
-    "format": "prettier --write \"src/**/*.ts\""
+    "fix:format": "prettier --write \"src/**/*.ts\"",
+    "test:format": "prettier --check \"src/**/*.ts\""
   },
   "dependencies": {
     "@certusone/wormhole-sdk": "^0.10.15",