More than 3 years have passed since last update.

Laravel Sailで作成したプロジェクトにPHP CS Fixerで自動整形できるようにする

Last updated at 2021-03-19Posted at 2021-03-18

PHP CS Fixerを利用するとPSR-1、PSR-2などで定義されているPHPコーディング規約に沿って自動整形することができます。

FriendsOfPHP/PHP-CS-Fixer: A tool to automatically fix PHP Coding Standards issues
https://github.com/FriendsOfPHP/PHP-CS-Fixer

これをLaravel Sailで作成したプロジェクトで利用できるようにパッケージのインストール、定義の作成、VSCodeの拡張機能を利用して自動整形できるようにしてみました。

前提

こちらの手順で手元にLaravel Sailで作成したプロジェクトがクローンされており、Dockerコンテナが実行されている環境となります。

今回の手順はGitHubリポジトリでも確認できます。

手順

設定ファイルを用意する

今回はLaravelのコーディング規約に合わせたいので公式ドキュメントを確認します。

Contribution Guide - Laravel - The PHP Framework For Web Artisans
https://laravel.com/docs/8.x/contributions#coding-style

Laravel follows the PSR-2 coding standard and the PSR-4 autoloading standard.
(日本語翻訳)LaravelはPSR-2コーディング標準とPSR-4自動読み込み標準に準拠しています。

PHP CS FixerにはLaravelのルールセットは含まれていません。

PHP-CS-Fixer/index.rst at 2.18 · FriendsOfPHP/PHP-CS-Fixer
https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/2.18/doc/ruleSets/index.rst

自前でLaravelのコーディング規約に沿って設定を用意することができますが、なかなかに設定が多くなるのと、メンテナンスが大変そうです。

Laravel PHP-CS-Fixer のルール設定 - Qiita
https://qiita.com/ucan-lab/items/7d4180462347a42009d5

今回は下記パッケージがLaravel用のルールセットを用意してくれていたので利用します。
こちらのパッケージをインストールするとPHP-CS-Fixerも合わせてインストールされます。

> cd <プロジェクトのディレクトリ>
> ./vendor/bin/sail composer require matt-allan/laravel-code-style --dev

パッケージがインストールできたらパッケージに含まれるPHP-CS-Fixerの設定ファイル.php_cs.dist をプロジェクトのディレクトリにコピーします。

> cp ./vendor/matt-allan/laravel-code-style/.php_cs.dist .

設定ファイル.php_cs.distを一部変更します。
@Laravel:risky は副作用のあるルールとなりますので、必要に応じて外してください。

.php_cs.dist

<?php

// 下記を削除(コメントアウト)する
// require __DIR__ . '/vendor/autoload.php';

$finder = PhpCsFixer\Finder::create()
    ->in(__DIR__);

// フィルターを追加する
return (new MattAllan\LaravelCodeStyle\Config())
        ->setFinder($finder
            ->exclude('bootstrap/cache')
            ->exclude('storage/framework')
        )

        ->setRules([
            '@Laravel' => true,
            '@Laravel:risky' => true,
        ])
        ->setRiskyAllowed(true);

PHP-CS-Fixerを実行すると.php_cs.cacheが作成されますが、Gitリポジトリにコミットする必要はないので、.gitignoreに設定を追加します。

> echo '.php_cs.cache' >> .gitignore

VSCodeの拡張機能のインストール

下記の拡張機能を利用して、VSCodeでソース保存またはコマンド実行で自動整形されるようにします。

PHP-CS-Fixer - Visual Studio Marketplace
https://marketplace.visualstudio.com/items?itemName=higoka.php-cs-fixer

プロジェクト直下に.vscode/settings.jsonファイルを作成することでワークスペースごとにVSCodeの設定が切り替えできるので、そのファイルに拡張機能の設定を追加します。

.vscode/settings.json

{
    "php-cs-fixer.executable": "vendor/bin/sail exec -T laravel.test vendor/bin/php-cs-fixer",
    "php-cs-fixer.allowRisky": true
}

VSCodeでプロジェクト単位の拡張機能を設定する - Qiita
https://qiita.com/k_bobchin/items/717c216ddc29e5fbcd43

`docker-compose.yml` の編集とコンテナ再起動

VSCodeの拡張機能から自動整形する場合、対象となるソースファイルを絶対パスでパラメータ指定されます。そのため、コンテナ内でPHP-CS-Fixerが絶対パスが参照できないと実行できないので、docker-compose.ymlを編集します。

docker-compose.yml(一部)

services:
    laravel.test:
        build:
            context: ./vendor/laravel/sail/runtimes/8.0
            dockerfile: Dockerfile
            args:
                WWWGROUP: '${WWWGROUP}'
        image: sail-8.0/app
        ports:
            - '${APP_PORT:-80}:80'
        environment:
            WWWUSER: '${WWWUSER}'
            LARAVEL_SAIL: 1
        volumes:
            - '.:/var/www/html'
+           - '$PWD:$PWD'
(略)

laravel.testコンテナのvolumesに'$PWD:$PWD' を追加して、絶対パスが参照できるようにしました。

docker-compose.ymlを編集したらコンテナを再起動します。sail restartコマンドだと追加した設定が反映されないのでstopしてからup -dします。

> ./vendor/bin/sail stop
> ./vendor/bin/sail up -d

動作確認

インストールや設定ができたら実際に自動整形されるか確認してみます。

コマンドで自動整形

コマンドでPHP-CS-Fixerを実行します。ここでは--dry-runオプションを付けて自動整形の対象をリストするだけにします。

> ./vendor/bin/sail exec laravel.test vendor/bin/php-cs-fixer fix --dry-run

Loaded config Laravel from "/var/www/html/.php_cs.dist".
Using cache file ".php_cs.cache".
   1) app/Models/User.php
   2) app/Providers/AuthServiceProvider.php
   3) server.php
   4) tests/Feature/ExampleTest.php

Checked all files in 2.434 seconds, 14.000 MB memory used

自動整形される内容を--diffオプションを付けて確認しています。

> ./vendor/bin/sail exec laravel.test vendor/bin/php-cs-fixer fix --dry-run --diff --diff-format udiff app/Models/User.php

Loaded config Laravel from "/var/www/html/.php_cs.dist".
Using cache file ".php_cs.cache".
Paths from configuration file have been overridden by paths provided as command arguments.
   1) app/Models/User.php
      ---------- begin diff ----------
--- Original
+++ New
@@ -2,7 +2,6 @@

 namespace App\Models;

-use Illuminate\Contracts\Auth\MustVerifyEmail;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Foundation\Auth\User as Authenticatable;
 use Illuminate\Notifications\Notifiable;

      ----------- end diff -----------


Checked all files in 0.053 seconds, 14.000 MB memory used

コマンドで実行できることが確認できました。

VSCodeで自動整形

自動整形対象となったソースファイルをVSCodeで自動整形できるか確認します。
VSCodeでプロジェクトのディレクトリを開きます。それ以外のディレクトリだと./vendor/bin/sailコマンドのパスが拡張機能から参照できず実行できないので注意が必要です。

> code <プロジェクトのディレクトリ>

VSCodeでも自動整形できることが確認できました。

注意点

自動整形されない(エラーになる)ケースとして、1.Dockerコンテナが起動していない、2.vendor/bin/sailがみつからないなどの原因があります。
エラー内容はVSCodeの「表示→出力」メニューから「ログ(ウィンドウ)」を表示するとエラー内容が確認できます。

2.vendor/bin/sailがみつからない場合、VSCodeアプリを再起動すると解消することは確認できています。

まとめ

せっかくLaravel SailでDockerの開発環境が構築できたので、できるだけDockerを活用したくて試してみたら条件付きですが、いい感じにできました。

参考