refactor(WithRunable): rename processTapper to tap for clarity

guanguans · guanguans · commit 31f316ab2eae · 2025-11-27T22:44:53.000+08:00
- Renamed the `processTapper` property to `tap` for improved clarity.
- Updated the setter method from `setProcessTapper` to `setTap` to reflect the new naming.
- Adjusted the usage of the callable in the process execution to match the new property name.

Signed-off-by: guanguans &lt;ityaozm@gmail.com&gt;
diff --git a/src/Concerns/WithRunable.php b/src/Concerns/WithRunable.php
@@ -20,12 +20,12 @@
  */
 trait WithRunable
 {
-    /** @var null|callable */
-    protected $processTapper;
+    /** @var null|callable(Process):void */
+    protected $tap;
 
-    public function setProcessTapper(?callable $processTapper): self
+    public function setTap(?callable $tap): self
     {
-        $this->processTapper = $processTapper;
+        $this->tap = $tap;
 
         return $this;
     }
@@ -52,8 +52,8 @@ protected function toProcess(): Process
             )
             : new Process([$this->soarBinary, ...$normalizedOptions]);
 
-        if (\is_callable($this->processTapper)) {
-            ($this->processTapper)($process);
+        if (\is_callable($this->tap)) {
+            ($this->tap)($process);
         }
 
         return $process;
diff --git a/tests/Concerns/WithRunableTest.php b/tests/Concerns/WithRunableTest.php
@@ -61,7 +61,7 @@ protected function shouldApplySudoPassword(): bool
 it('can run soar process with tapper', function (): void {
     expect(Soar::make())
         ->withVersion(true)
-        ->setProcessTapper(static function (Process $process): void {
+        ->setTap(static function (Process $process): void {
             $process->setTimeout(3);
         })
         ->run(static function (string $type, string $line): void {

Original file line number	Diff line number	Diff line change
`@@ -20,12 +20,12 @@`
`20`	`20`	`*/`
`21`	`21`	`trait WithRunable`
`22`	`22`	`{`
`23`		`- /** @var null\|callable */`
`24`		`- protected $processTapper;`
	`23`	`+ /** @var null\|callable(Process):void */`
	`24`	`+ protected $tap;`
`25`	`25`
`26`		`- public function setProcessTapper(?callable $processTapper): self`
	`26`	`+ public function setTap(?callable $tap): self`
`27`	`27`	`{`
`28`		`- $this->processTapper = $processTapper;`
	`28`	`+ $this->tap = $tap;`
`29`	`29`
`30`	`30`	`return $this;`
`31`	`31`	`}`
`@@ -52,8 +52,8 @@ protected function toProcess(): Process`
`52`	`52`	`)`
`53`	`53`	`: new Process([$this->soarBinary, ...$normalizedOptions]);`
`54`	`54`
`55`		`- if (\is_callable($this->processTapper)) {`
`56`		`- ($this->processTapper)($process);`
	`55`	`+ if (\is_callable($this->tap)) {`
	`56`	`+ ($this->tap)($process);`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`return $process;`