Format, lint on PR, require all changes be committed

.github/workflows/test.yml → .github/workflows/ci.yml

@@ -1,4 +1,4 @@
-name: Run Tests
+name: CI
 
 on:
   push:
@@ -17,21 +17,61 @@
       - name: Checkout code
         uses: actions/checkout@v4
 
+      - name: Parse .tool-versions
+        uses: wistia/parse-tool-versions@v2.1.1
+
       - name: Setup Bun
         uses: oven-sh/setup-bun@v1
         with:
-          bun-version: 1.2.20
+          bun-version: '${{ env.BUN }}'
 
       - name: Install dependencies
         run: bun install
 
       - name: Run unit tests
         run: bun test:unit
 
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Parse .tool-versions
+        uses: wistia/parse-tool-versions@v2.1.1
+
+      - name: Setup Bun
+        uses: oven-sh/setup-bun@v1
+        with:
+          bun-version: '${{ env.BUN }}'
+
+      - name: Run linters and formatters
+        run: |
+          bun install
+          bun run format
+          bun run lint:fix
+          bun run lint
+
+      - name: Fail if source files changed
+        run: |
+          # Check for changes excluding bun.lock (platform-specific)
+          CHANGED_FILES=$(git status --porcelain | grep -v "bun.lock" || true)
+
+          if [[ -z "$CHANGED_FILES" ]]; then
+            echo "✅ No source file changes detected."
+            exit 0
+          fi
+
+          echo "❌ Code is not properly formatted. Please run 'bun run format && bun run lint:fix' and commit the changes."
+          echo "$CHANGED_FILES"
+          exit 1
+
   integration-tests:
     name: Integration Tests
     runs-on: ubuntu-latest
-    needs: unit-tests  # run after unit tests pass
+    needs: unit-tests # run after unit tests pass
 
     # these are placeholders which is fine for integration tests
     env:
@@ -54,10 +94,13 @@
       - name: Checkout code
         uses: actions/checkout@v4
 
+      - name: Parse .tool-versions
+        uses: wistia/parse-tool-versions@v2.1.1
+
       - name: Setup Bun
         uses: oven-sh/setup-bun@v1
         with:
-          bun-version: 1.2.20
+          bun-version: '${{ env.BUN }}'
 
       - name: Install dependencies
         run: bun install

.tool-versions

@@ -0,0 +1 @@
+bun 1.2.20

eslint.config.mjs

@@ -5,6 +5,7 @@ const compat = new FlatCompat({
 })
 
 const eslintConfig = [
+  { ignores: ['.next/'] },
   ...compat.config({
     extends: ['next/core-web-vitals', 'next/typescript'],
     plugins: ['eslint-plugin-prettier'],

package.json

@@ -8,8 +8,8 @@
     "build": "next build --turbopack",
     "start": "next start | pino-pretty --colorize",
     "preview": "next build --turbopack && next start | pino-pretty --colorize",
-    "lint": "next lint",
-    "lint:fix": "next lint --fix",
+    "lint": "eslint",
+    "lint:fix": "eslint --fix",
     "format": "prettier --write .",
     "prebuild": "bun scripts:check-app-env",
     "<<<<<<< Development Tools": "",