refactor(Concerns): Rename WithRunable to WithRunnable

guanguans · guanguans · commit 0887d3ca7c58 · 2026-03-19T14:29:57.000+08:00
- Correct the spelling of the trait and its references across files.
- Update the JSON configuration to reflect the correct spelling.
- Ensures consistency and clarity in the codebase.

Signed-off-by: guanguans &lt;ityaozm@gmail.com&gt;
diff --git a/composer-dependency-analyser.php b/composer-dependency-analyser.php
@@ -37,11 +37,6 @@
         ],
         [ErrorType::SHADOW_DEPENDENCY],
     )
-    ->ignoreErrorsOnPackageAndPath(
-        'illuminate/collections',
-        __DIR__.'/src/Support/helpers.php',
-        [ErrorType::SHADOW_DEPENDENCY]
-    )
     ->ignoreErrorsOnPackageAndPath(
         'symfony/var-dumper',
         __DIR__.'/src/Concerns/WithDumpable.php',
diff --git a/peck.json b/peck.json
@@ -5,7 +5,7 @@
             "sudo",
             "makeable",
             "dumpable",
-            "runable"
+            "runnable"
         ],
         "paths": []
     }
diff --git a/src/Concerns/WithRunnable.php b/src/Concerns/WithRunnable.php
@@ -18,7 +18,7 @@
 /**
  * @mixin \Guanguans\SoarPHP\Soar
  */
-trait WithRunable
+trait WithRunnable
 {
     /** @var null|\Closure(Process): Process */
     protected ?\Closure $pipe = null;
diff --git a/src/Soar.php b/src/Soar.php
@@ -20,7 +20,7 @@
 use Guanguans\SoarPHP\Concerns\HasSudoPassword;
 use Guanguans\SoarPHP\Concerns\Makeable;
 use Guanguans\SoarPHP\Concerns\WithDumpable;
-use Guanguans\SoarPHP\Concerns\WithRunable;
+use Guanguans\SoarPHP\Concerns\WithRunnable;
 use Symfony\Component\Process\Process;
 
 class Soar implements \ArrayAccess, \Stringable, Contracts\Soar
@@ -32,7 +32,7 @@ class Soar implements \ArrayAccess, \Stringable, Contracts\Soar
     use HasSudoPassword;
     use Makeable;
     use WithDumpable;
-    use WithRunable;
+    use WithRunnable;
 
     public function __construct(array $options = [], ?string $binary = null)
     {
diff --git a/src/Support/helpers.php b/src/Support/helpers.php
@@ -13,59 +13,6 @@
 
 namespace Guanguans\SoarPHP\Support;
 
-use Composer\Autoload\ClassLoader;
-use Illuminate\Support\Collection;
-
-if (!\function_exists('Guanguans\SoarPHP\Support\classes')) {
-    /**
-     * @see https://github.com/illuminate/collections
-     * @see https://github.com/alekitto/class-finder
-     * @see https://github.com/ergebnis/classy
-     * @see https://gitlab.com/hpierce1102/ClassFinder
-     * @see https://packagist.org/packages/haydenpierce/class-finder
-     * @see \get_declared_classes()
-     * @see \get_declared_interfaces()
-     * @see \get_declared_traits()
-     * @see \DG\BypassFinals::enable()
-     * @see \Composer\Util\ErrorHandler
-     * @see \Monolog\ErrorHandler
-     * @see \PhpCsFixer\ExecutorWithoutErrorHandler
-     * @see \Phrity\Util\ErrorHandler
-     *
-     * @template TObject of object
-     *
-     * @internal
-     *
-     * @param null|(callable(class-string<TObject>, string): bool) $filter
-     *
-     * @return \Illuminate\Support\Collection<class-string<TObject>, \ReflectionClass<TObject>|\Throwable>
-     *
-     * @noinspection PhpUndefinedNamespaceInspection
-     */
-    function classes(?callable $filter = null): Collection
-    {
-        $filter ??= static fn (string $_, string $__): bool => true;
-
-        /** @var null|\Illuminate\Support\Collection $classes */
-        static $classes;
-        $classes ??= collect(spl_autoload_functions())->flatMap(
-            static fn (callable $loader): array => \is_array($loader) && $loader[0] instanceof ClassLoader
-                ? $loader[0]->getClassMap()
-                : []
-        );
-
-        return $classes
-            ->filter(static fn (string $file, string $class): bool => $filter($class, $file))
-            ->mapWithKeys(static function (string $_, string $class): array {
-                try {
-                    return [$class => new \ReflectionClass($class)];
-                } catch (\Throwable $throwable) {
-                    return [$class => $throwable];
-                }
-            });
-    }
-}
-
 if (!\function_exists('Guanguans\SoarPHP\Support\str_snake')) {
     function str_snake(string $value, string $delimiter = '-'): string
     {
diff --git a/tests/Concerns/WithRunnableTest.php b/tests/Concerns/WithRunnableTest.php
diff --git a/tests/Support/HelpersTest.php b/tests/Support/HelpersTest.php
@@ -19,17 +19,8 @@
  * @see https://github.com/guanguans/soar-php
  */
 
-use Illuminate\Support\Collection;
-use function Guanguans\SoarPHP\Support\classes;
 use function Guanguans\SoarPHP\Support\str_snake;
 
-it('can get classes', function (): void {
-    expect(classes(fn (string $class): bool => str($class)->startsWith('Illuminate\Support')))
-        ->toBeInstanceOf(Collection::class)
-        ->groupBy(fn (object $object): bool => $object instanceof ReflectionClass)
-        ->toHaveCount(2);
-})->group(__DIR__, __FILE__);
-
 it('can snake string', function (): void {
     expect(str_snake(__FILE__))->toBe(str(__FILE__)->snake('-')->toString());
 })->group(__DIR__, __FILE__);

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`	`"sudo",`
`6`	`6`	`"makeable",`
`7`	`7`	`"dumpable",`
`8`		`- "runable"`
	`8`	`+ "runnable"`
`9`	`9`	`],`
`10`	`10`	`"paths": []`
`11`	`11`	`}`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@`
`18`	`18`	`/**`
`19`	`19`	`* @mixin \Guanguans\SoarPHP\Soar`
`20`	`20`	`*/`
`21`		`-trait WithRunable`
	`21`	`+trait WithRunnable`
`22`	`22`	`{`
`23`	`23`	`/** @var null\|\Closure(Process): Process */`
`24`	`24`	`protected ?\Closure $pipe = null;`