Merge pull request #2 from laravel/actions-updates

Actions updates

Author: tnylea

.github/workflows/lint.yml

@@ -0,0 +1,46 @@
+name: linter
+
+on:
+  push:
+    branches:
+      - develop
+      - main
+  pull_request:
+    branches:
+      - develop
+      - main
+
+jobs:
+  quality:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.2'
+
+      - name: Install Dependencies
+        run: |
+          composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
+          npm install
+
+      - name: Run Pint
+        run: vendor/bin/pint
+
+      - name: Frontend Format Check
+        run: npm run format
+
+      - name: Frontend Lint
+        run: npm run lint
+
+      - name: Commit changes
+        uses: stefanzweifel/git-auto-commit-action@v5
+        with:
+          commit_message: fix code style
+          commit_options: '--no-verify'
+
+      # We need to run PHPStan after commiting changes as it does not auto-fix errors.
+      - name: PHPStan
+        run: ./vendor/bin/phpstan

.prettierignore

@@ -0,0 +1,42 @@
+# Dependencies
+node_modules/
+vendor/
+
+# Build outputs
+public/
+bootstrap/
+dist/
+.output/
+.nuxt/
+
+# Generated files
+*.cache
+.env
+.env.*
+!.env.example
+
+# Config files
+*.config.js
+*.config.ts
+
+# ShadCN Vue components (as per project rules)
+resources/js/components/ui/
+
+# IDE and Editor files
+.idea/
+.vscode/
+*.sublime-project
+*.sublime-workspace
+
+# System files
+.DS_Store
+Thumbs.db
+
+# Logs
+*.log
+npm-debug.log*
+yarn-debug.log*
+yarn-error.log*
+
+# Test coverage
+coverage/

app/Http/Controllers/Auth/VerifyEmailController.php

@@ -19,7 +19,9 @@ public function __invoke(EmailVerificationRequest $request): RedirectResponse
         }
 
         if ($request->user()->markEmailAsVerified()) {
-            event(new Verified($request->user()));
+            /** @var \Illuminate\Contracts\Auth\MustVerifyEmail $user */
+            $user = $request->user();
+            event(new Verified($user));
         }
 
         return redirect()->intended(route('dashboard', absolute: false).'?verified=1');

composer.json

@@ -75,4 +75,4 @@
     },
     "minimum-stability": "dev",
     "prefer-stable": true
-}
+}

eslint.config.js

@@ -1,68 +1,19 @@
-import js from '@eslint/js';
 import prettier from 'eslint-config-prettier';
-import globals from 'globals';
-import typescript from 'typescript-eslint';
 import vue from 'eslint-plugin-vue';
 
-/** @type {import('eslint').Linter.Config[]} */
-export default [
-    js.configs.recommended,
-    ...typescript.configs.recommended,
+import { defineConfigWithVueTs, vueTsConfigs } from '@vue/eslint-config-typescript';
+
+export default defineConfigWithVueTs(
+    vue.configs['flat/essential'],
+    vueTsConfigs.recommended,
     {
-        ...vue.configs.flat.recommended,
-        ...vue.configs["flat/strongly-recommended"],
-        ...vue.configs["flat/essential"],
-        languageOptions: {
-            globals: {
-                ...globals.browser,
-            },
-        },
-        rules: {
-            "vue/match-component-import-name": "warn",
-            "vue/match-component-file-name": [
-                "error",
-                {
-                    extensions: ["vue"],
-                    shouldMatchCase: true,
-                },
-            ],
-            "vue/component-definition-name-casing": ["error", "PascalCase"],
-            "vue/block-tag-newline": [
-                "warn",
-                {
-                    singleline: "always",
-                    multiline: "always",
-                    maxEmptyLines: 0,
-                },
-            ],
-            "vue/html-self-closing": [
-                "error",
-                {
-                    html: {
-                        void: "always",
-                        normal: "never",
-                        component: "always",
-                    },
-                    svg: "always",
-                    math: "always",
-                },
-            ],
-            "vue/require-default-prop": "off",
-        },
+        ignores: ['vendor', 'node_modules', 'public', 'bootstrap/ssr', 'tailwind.config.js', 'resources/js/components/ui/*'],
     },
     {
-        plugins: {
-            "@typescript-eslint": tseslint.plugin,
-        },
-        languageOptions: {
-            parser: tseslint.parser,
-            parserOptions: {
-                project: true,
-            },
+        rules: {
+            'vue/multi-word-component-names': 'off',
+            '@typescript-eslint/no-explicit-any': 'off',
         },
     },
-    {
-        ignores: ['vendor', 'node_modules', 'public', 'bootstrap/ssr', 'tailwind.config.js'],
-    },
-    prettier, // Turn off all rules that might conflict with Prettier
-];
+    prettier,
+);

package-lock.json

@@ -34,6 +34,7 @@
     },
     "dependencies": {
         "@types/ziggy-js": "^1.3.3",
+        "@vue/eslint-config-typescript": "^14.3.0",
         "@vueuse/core": "^12.0.0",
         "class-variance-authority": "^0.7.1",
         "clsx": "^2.1.1",

package.json

@@ -0,0 +1,10 @@
+includes:
+    - vendor/larastan/larastan/extension.neon
+
+parameters:
+
+    paths:
+        - app/
+
+    # Level 9 is the highest level
+    level: 5

phpstan.neon

@@ -83,4 +83,4 @@
     body {
         @apply bg-background text-foreground;
     }
-}
+}

resources/css/app.css

@@ -2,8 +2,8 @@ import '../css/app.css';
 
 import { createInertiaApp } from '@inertiajs/vue3';
 import { resolvePageComponent } from 'laravel-vite-plugin/inertia-helpers';
-import { createApp, h } from 'vue';
 import type { DefineComponent } from 'vue';
+import { createApp, h } from 'vue';
 import { ZiggyVue } from '../../vendor/tightenco/ziggy';
 import { initializeTheme } from './composables/useAppearance';