v2: Add left-hook pre-push (#1003)

Co-authored-by: Felix Weinberger <3823880+felixweinberger@users.noreply.github.com>

Author: KKonstantinov

.gitignore

@@ -52,3 +52,6 @@ dist/
 
 # Conformance test results
 results/
+
+# Ignore local lefthook configuration
+lefthook-local.yml

lefthook-local.example.yml

@@ -0,0 +1,23 @@
+# Optional local lefthook configuration
+# To enable this:
+#  cp lefthook-local.example.yml lefthook-local.yml
+
+pre-commit:
+    parallel: true
+    jobs:
+        - name: 'Typecheck'
+          run: pnpm typecheck:all
+
+        - name: 'Lint Fix & Format'
+          run: pnpm lint:fix:all
+          stage_fixed: true
+
+post-checkout:
+    jobs:
+        - name: 'Install Dependencies'
+          run: pnpm install
+
+post-merge:
+    jobs:
+        - name: 'Install Dependencies'
+          run: pnpm install

lefthook.yml

@@ -0,0 +1,37 @@
+# lefthook.yml
+# Configuration reference: https://lefthook.dev/configuration/
+
+assert_lefthook_installed: true
+
+output:
+    - meta # Print lefthook version
+    - summary # Print summary block (successful and failed steps)
+    - empty_summary # Print summary heading when there are no steps to run
+    - success # Print successful steps
+    - failure # Print failed steps printing
+    - execution # Print any execution logs (but prints if the execution failed)
+    - execution_out # Print execution output (but still prints failed commands output)
+    - execution_info # Print `EXECUTE > ...` logging
+    - skips # Print "skip" (i.e. no files matched)
+
+pre-push:
+    follow: true
+    parallel: true
+    jobs:
+        - name: 'Typecheck'
+          run: pnpm run typecheck:all
+          fail_text: |
+              💡 To catch typechecking issues earlier, enable the pre-commit hook:
+              cp lefthook-local.example.yml lefthook-local.yml
+
+        - name: 'Lint'
+          run: pnpm run lint:all
+          fail_text: |
+              💡 To catch linting issues earlier, enable the pre-commit hook:
+              cp lefthook-local.example.yml lefthook-local.yml
+
+        - name: 'Build'
+          run: pnpm run build:all
+          fail_text: |
+              💡 To catch build issues earlier, enable the pre-commit hook:
+              cp lefthook-local.example.yml lefthook-local.yml

package.json

@@ -34,6 +34,7 @@
         "lint:fix:all": "pnpm sync:snippets && pnpm -r lint:fix",
         "check:all": "pnpm -r typecheck && pnpm -r lint && pnpm run docs:check",
         "test:all": "pnpm -r test",
+        "prepare": "npx --no-install lefthook install",
         "test:conformance:client": "pnpm --filter @modelcontextprotocol/test-conformance run test:conformance:client",
         "test:conformance:client:all": "pnpm --filter @modelcontextprotocol/test-conformance run test:conformance:client:all",
         "test:conformance:client:run": "pnpm --filter @modelcontextprotocol/test-conformance run test:conformance:client:run",
@@ -43,6 +44,7 @@
         "test:conformance:all": "pnpm run test:conformance:client:all && pnpm run test:conformance:server:all"
     },
     "devDependencies": {
+        "lefthook": "^2.0.16",
         "@cfworker/json-schema": "catalog:runtimeShared",
         "@changesets/changelog-github": "^0.5.2",
         "@changesets/cli": "^2.29.8",