feat: add stdin support with dash and stdin-filename option

- Replace --stdin flag with Unix-standard dash (-) for stdin input
- Add --stdin-filename option for editor integration and context
- Support both 'pint -' and 'pint --stdin-filename' patterns
- Add comprehensive tests for stdin formatting scenarios

Author: A909M

app/Commands/DefaultCommand.php

@@ -51,7 +51,7 @@ protected function configure()
                     new InputOption('cache-file', '', InputArgument::OPTIONAL, 'The path to the cache file'),
                     new InputOption('parallel', 'p', InputOption::VALUE_NONE, 'Runs the linter in parallel (Experimental)'),
                     new InputOption('max-processes', null, InputOption::VALUE_REQUIRED, 'The number of processes to spawn when using parallel execution'),
-                    new InputOption('stdin', null, InputOption::VALUE_NONE, 'Read and format code from standard input'),
+                    new InputOption('stdin-filename', null, InputOption::VALUE_REQUIRED, 'The path to use for stdin input (for config resolution and exclusions)'),
                 ],
             );
     }
@@ -76,12 +76,13 @@ public function handle($fixCode, $elaborateSummary)
 
     /**
      * Fix the code sent to Pint on stdin and output to stdout.
+     *
+     * The stdin-filename option provides context for config resolution and
+     * exclusion rules. Falls back to 'stdin.php' if not provided.
      */
     protected function fixStdinInput(FixCode $fixCode): int
     {
-        $paths = $this->argument('path');
-
-        $contextPath = ! empty($paths) ? $paths[0] : 'stdin.php';
+        $contextPath = $this->option('stdin-filename') ?: 'stdin.php';
         $tempFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.'pint_stdin_'.uniqid().'.php';
 
         $this->input->setArgument('path', [$tempFile]);
@@ -106,9 +107,18 @@ protected function fixStdinInput(FixCode $fixCode): int
 
     /**
      * Determine if there is input available on stdin.
+     *
+     * Stdin mode is triggered by either:
+     * - Passing '-' as the path argument (Unix convention like Black, cat)
+     * - Providing the --stdin-filename option (editor-friendly like Prettier)
      */
     protected function hasStdinInput(): bool
     {
-        return $this->option('stdin');
+        $paths = $this->argument('path');
+
+        $hasStdinPlaceholder = ! empty($paths) && $paths[0] === '__STDIN_PLACEHOLDER__';
+        $hasStdinFilename = ! empty($this->option('stdin-filename'));
+
+        return $hasStdinPlaceholder || $hasStdinFilename;
     }
 }

pint

@@ -40,6 +40,27 @@ $app = require_once __DIR__.'/bootstrap/app.php';
 
 $kernel = $app->make(Illuminate\Contracts\Console\Kernel::class);
 
+/*
+|--------------------------------------------------------------------------
+| Handle Stdin Mode
+|--------------------------------------------------------------------------
+|
+| When using '-' to indicate stdin input (following Unix convention like
+| Black, cat, etc.), Symfony Console's ArgvInput parser fails because it
+| treats '-' as a malformed option. We work around this by replacing '-'
+| with a placeholder before the input is parsed. The DefaultCommand then
+| detects this placeholder to enable stdin mode.
+|
+*/
+
+if (isset($_SERVER['argv'])) {
+    $stdinIndex = array_search('-', $_SERVER['argv'], true);
+
+    if ($stdinIndex !== false) {
+        $_SERVER['argv'][$stdinIndex] = '__STDIN_PLACEHOLDER__';
+    }
+}
+
 $status = $kernel->handle(
     $input = new Symfony\Component\Console\Input\ArgvInput,
     new Symfony\Component\Console\Output\ConsoleOutput

tests/Feature/StdinTest.php

@@ -3,65 +3,132 @@
 use Illuminate\Support\Facades\Process;
 
 it('formats code from stdin', function (string $input, ?string $expected) {
-    $result = Process::input($input)->run('php pint app/Test.php --stdin')->throw();
+    $result = Process::input($input)
+        ->run('php pint - --stdin-filename=app/Test.php')
+        ->throw();
 
     expect($result)
-        ->output()->toBe($expected ?? $input)
-        ->errorOutput()->toBe('');
+        ->output()
+        ->toBe($expected ?? $input)
+        ->errorOutput()
+        ->toBe('');
 })->with([
     'basic array and conditional' => [
         <<<'PHP'
-            <?php
-            $array = array("a","b");
-            if($condition==true){
-                echo "test";
-            }
-            PHP,
+        <?php
+        $array = array("a","b");
+        if($condition==true){
+            echo "test";
+        }
+        PHP
+        ,
         <<<'PHP'
-            <?php
+        <?php
 
-            $array = ['a', 'b'];
-            if ($condition == true) {
-                echo 'test';
-            }
+        $array = ['a', 'b'];
+        if ($condition == true) {
+            echo 'test';
+        }
 
-            PHP,
+        PHP
+        ,
     ],
     'class with method' => [
         <<<'PHP'
-            <?php
-            class Test{
-            public function method(){
-            return array("key"=>"value");
-            }
-            }
-            PHP,
+        <?php
+        class Test{
+        public function method(){
+        return array("key"=>"value");
+        }
+        }
+        PHP
+        ,
         <<<'PHP'
-            <?php
+        <?php
 
-            class Test
+        class Test
+        {
+            public function method()
             {
-                public function method()
-                {
-                    return ['key' => 'value'];
-                }
+                return ['key' => 'value'];
             }
+        }
 
-            PHP,
+        PHP
+        ,
     ],
     'already formatted code' => [
         <<<'PHP'
-            <?php
+        <?php
 
-            class AlreadyFormatted
+        class AlreadyFormatted
+        {
+            public function method()
             {
-                public function method()
-                {
-                    return ['key' => 'value'];
-                }
+                return ['key' => 'value'];
             }
+        }
 
-            PHP,
+        PHP
+        ,
         null,
     ],
 ]);
+
+it('formats code from stdin without filename', function () {
+    $input = <<<'PHP'
+    <?php
+    $array = array("a","b");
+    PHP;
+
+    $expected = <<<'PHP'
+    <?php
+
+    $array = ['a', 'b'];
+
+    PHP;
+
+    $result = Process::input($input)->run('php pint -')->throw();
+
+    expect($result)->output()->toBe($expected)->errorOutput()->toBe('');
+});
+
+it('uses stdin-filename for context', function () {
+    $input = <<<'PHP'
+    <?php
+    $array = array("test");
+    PHP;
+
+    $expected = <<<'PHP'
+    <?php
+
+    $array = ['test'];
+
+    PHP;
+
+    $result = Process::input($input)
+        ->run('php pint - --stdin-filename=app/Models/User.php')
+        ->throw();
+
+    expect($result)->output()->toBe($expected)->errorOutput()->toBe('');
+});
+
+it('formats code from stdin using only stdin-filename option', function () {
+    $input = <<<'PHP'
+    <?php
+    $array = array("foo","bar");
+    PHP;
+
+    $expected = <<<'PHP'
+    <?php
+
+    $array = ['foo', 'bar'];
+
+    PHP;
+
+    $result = Process::input($input)
+        ->run('php pint --stdin-filename=app/Models/Example.php')
+        ->throw();
+
+    expect($result)->output()->toBe($expected)->errorOutput()->toBe('');
+});