chore: move all prettier formatting into individual packages

This PR makes a few related changes:

- Stops using `pre-commit` to run prettier anywhere in the monorepo, instead we
now run prettier from each package and trigger the runs using turborepo at the
root.

- Upgrade the prettier version used in all packages that were previously relying
on `pre-commit` to run prettier.

- Reformat files that need it due to the new prettier version.

- Cleans up the turborepo config so that it now runs some tasks in the root
package, and also automatically runs `pnpm install` when needed.

The main downside of this PR is that auto-formatting on pre-commit will no
longer happen, but for that I think we should help developers get their prettier
editor integration set up so it happens on file save.

Author: cprussin

.github/workflows/ci-turbo-test.yml

@@ -33,4 +33,4 @@ jobs:
       - name: Cache for Turbo
         uses: rharkor/[email protected]
       - name: Test
-        run: pnpm test:ci
+        run: pnpm turbo test

.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/turbo.json

@@ -3,27 +3,11 @@
   "extends": ["//"],
   "tasks": {
     "build": {
-      "dependsOn": ["pull:env", "^build"],
-      "outputs": [".next/**", "!.next/cache/**"],
       "env": [
         "WALLETCONNECT_PROJECT_ID",
         "AMPLITUDE_API_KEY",
         "GOOGLE_ANALYTICS_ID"
       ]
-    },
-    "pull:env": {
-      "outputs": [".env.local"],
-      "cache": false
-    },
-    "start:dev": {
-      "dependsOn": ["pull:env"],
-      "persistent": true,
-      "cache": false
-    },
-    "start:prod": {
-      "dependsOn": ["build"],
-      "persistent": true,
-      "cache": false
     }
   }
 }

apps/entropy-debugger/turbo.json

@@ -76,11 +76,12 @@ The [HermesClient](./src/examples/HermesClient.ts) example demonstrates both the
 examples above. To run the example:
 
 1. Clone [the Pyth monorepo](https://github.com/pyth-network/pyth-crosschain)
-2. In the root of the monorepo, run `pnpm example:hermes-client -- <args>`. For
-   example, to print BTC and ETH price feeds in the testnet network, run:
+2. In the root of the monorepo, run `pnpm turbo --filter
+   @pythnetwork/hermes-client example -- <args>`. For example, to print BTC and
+   ETH price feeds in the testnet network, run:
 
 ```bash
-pnpm example:hermes-client -- \
+pnpm turbo --filter @pythnetwork/hermes-client example -- \
   --endpoint https://hermes.pyth.network \
   --price-ids \
     0xe62df6c8b4a85fe1a67db44dc12de5db330f7ac66b72dc658afedf0f4a415b43 \

apps/hermes/client/js/README.md

@@ -3,22 +3,23 @@
   "extends": ["//"],
   "tasks": {
     "build:schemas": {
-      "dependsOn": ["pull:schema"],
+      "dependsOn": ["//#install:modules", "pull:schema"],
       "inputs": ["schema.json"],
       "outputs": ["src/zodSchemas.ts"]
     },
     "build:typescript": {
-      "dependsOn": ["build:schemas", "^build"],
+      "dependsOn": ["//#install:modules", "build:schemas", "^build"],
       "outputs": ["lib/**"]
     },
     "build": {
       "dependsOn": ["build:typescript", "build:schemas"]
     },
     "example": {
-      "dependsOn": ["build"],
+      "dependsOn": ["//#install:modules", "build"],
       "cache": false
     },
     "pull:schema": {
+      "dependsOn": ["//#install:modules"],
       "outputs": ["schema.json"],
       "cache": false
     }

apps/hermes/client/js/turbo.json

@@ -3,8 +3,6 @@
   "extends": ["//"],
   "tasks": {
     "build": {
-      "dependsOn": ["pull:env", "^build"],
-      "outputs": [".next/**", "!.next/cache/**"],
       "env": [
         "VERCEL_ENV",
         "GOOGLE_ANALYTICS_ID",
@@ -17,36 +15,28 @@
       ]
     },
     "fix:lint": {
-      "dependsOn": ["fix:lint:eslint", "fix:lint:stylelint"],
-      "cache": false
+      "dependsOn": [
+        "//#install:modules",
+        "fix:lint:eslint",
+        "fix:lint:stylelint"
+      ]
     },
     "fix:lint:eslint": {
-      "dependsOn": ["^build"],
+      "dependsOn": ["//#install:modules", "^build"],
       "cache": false
     },
     "fix:lint:stylelint": {
-      "cache": false
-    },
-    "pull:env": {
-      "outputs": [".env.local"],
-      "cache": false
-    },
-    "start:dev": {
-      "dependsOn": ["pull:env"],
-      "persistent": true,
-      "cache": false
-    },
-    "start:prod": {
-      "dependsOn": ["build"],
-      "persistent": true,
+      "dependsOn": ["//#install:modules"],
       "cache": false
     },
     "test:lint": {
       "dependsOn": ["test:lint:eslint", "test:lint:stylelint"]
     },
     "test:lint:eslint": {
-      "dependsOn": ["^build"]
+      "dependsOn": ["//#install:modules", "^build"]
     },
-    "test:lint:stylelint": {}
+    "test:lint:stylelint": {
+      "dependsOn": ["//#install:modules"]
+    }
   }
 }

apps/insights/turbo.json

@@ -3,8 +3,6 @@
   "extends": ["//"],
   "tasks": {
     "build": {
-      "dependsOn": ["pull:env", "^build"],
-      "outputs": [".next/**", "!.next/cache/**"],
       "env": [
         "IP_ALLOWLIST",
         "GOVERNANCE_ONLY_REGIONS",
@@ -17,23 +15,13 @@
         "GOOGLE_ANALYTICS_ID"
       ]
     },
-    "pull:env": {
-      "outputs": [".env.local"],
-      "cache": false
-    },
     "start:dev": {
       "dependsOn": [
+        "//#install:modules",
         "pull:env",
         "@pythnetwork/hermes-client#build",
         "@pythnetwork/solana-utils#build"
-      ],
-      "persistent": true,
-      "cache": false
-    },
-    "start:prod": {
-      "dependsOn": ["build"],
-      "persistent": true,
-      "cache": false
+      ]
     }
   }
 }

apps/staking/turbo.json

@@ -20,7 +20,7 @@ export const PROPOSAL_STATUSES = [
   'draft',
   'unkwown',
 ] as const
-export type ProposalStatus = typeof PROPOSAL_STATUSES[number]
+export type ProposalStatus = (typeof PROPOSAL_STATUSES)[number]
 
 export const getProposalStatus = (
   proposal: TransactionAccount | undefined,