fix(console): select default option in ask component (#1139)

Author: brendt

src/Tempest/Console/src/Components/Concerns/OpensInEditor.php

@@ -22,9 +22,9 @@ private function supportsOpeningInEditor(): bool
         return is_subclass_of(static::class, CanOpenInEditor::class) && ((bool) $this->getEditorCommand());
     }
 
-    private function getEditorCommand(): string
+    private function getEditorCommand(): ?string
     {
-        return env('TEMPEST_EDITOR') ?? env('EDITOR');
+        return env('TEMPEST_EDITOR') ?? env('EDITOR') ?? null;
     }
 
     public function openInEditor(?string $text): string

src/Tempest/Console/src/Components/Interactive/SingleChoiceComponent.php

@@ -81,7 +81,7 @@ private function getControls(): array
         }
 
         if ($this->default !== null) {
-            $controls['alt+enter'] = 'default';
+            $controls['tab'] = 'select default';
         }
 
         return [
@@ -138,12 +138,10 @@ public function enter(): null|int|string|Stringable|UnitEnum
         return $this->options->getRawActiveOption($this->default);
     }
 
-    #[HandlesKey(Key::ALT_ENTER)]
-    public function altEnter(): null|int|string|Stringable|UnitEnum
+    #[HandlesKey(Key::TAB)]
+    public function tab(): void
     {
         $this->options->setActive($this->default);
-
-        return $this->default;
     }
 
     #[HandlesKey(Key::ESCAPE)]

src/Tempest/Console/src/Key.php

@@ -12,6 +12,7 @@ enum Key: string
     case CTRL_LEFT = "\eb";
     case RIGHT = "\e[C";
     case CTRL_RIGHT = "\ef";
+    case TAB = "\t";
     case ENTER = "\n";
     case ALT_ENTER = "\e\n";
     case BACKSPACE = "\x7F";

tests/Integration/Console/Components/SingleChoiceComponentTest.php

@@ -85,7 +85,9 @@ public function test_default(): void
             ->call(function (): void {
                 $component = new SingleChoiceComponent(label: 'Enter a name', options: ['Brent', 'Paul', 'Aidan', 'Roman'], default: 'Paul');
 
-                $this->assertSame('Paul', $component->altEnter());
+                $component->tab();
+
+                $this->assertSame('Paul', $component->enter());
             });
     }
 }

tests/Integration/Console/Fixtures/InteractiveCommand.php

@@ -66,8 +66,7 @@ public function single(): void
                 'b',
                 'c',
             ],
-            default: 1,
-            multiple: true,
+            default: 'b',
         );
 
         $result = json_encode($result);
@@ -93,6 +92,17 @@ public function single_without_default(): void
         $this->console->writeln("You picked <em>{$result}</em>");
     }
 
+    #[ConsoleCommand('interactive:multiline')]
+    public function multiline(): void
+    {
+        $result = $this->console->ask(
+            question: 'Write something',
+            multiline: true,
+        );
+
+        $this->console->writeln($result);
+    }
+
     #[ConsoleCommand('interactive:multiple')]
     public function multiple(): void
     {