Fix element action triggers on Craft 4 element indexes

.agents/skills/artisan-command-authoring/SKILL.md

@@ -0,0 +1,64 @@
+---
+name: artisan-command-authoring
+description: Write and review Craft CMS Laravel Artisan commands using repository conventions. Use when adding or refactoring command classes in `src/**/Commands`, porting legacy Yii console controllers, or addressing review feedback on command signatures, aliases, prompts, output formatting, and Laravel service/facade usage.
+---
+
+# Artisan Command Authoring
+
+## Overview
+Implement or refactor commands so they match Craft CMS 6 Laravel patterns and pass common review checks the first time.
+
+This skill now includes a Laravel 13 Prompts reference in `references/laravel-prompts-reference.md`.
+
+## Workflow
+1. Inspect neighboring command classes in the same domain and the domain service provider registration.
+2. Define signature, aliases, arguments/options, injected services, and interactive behavior.
+3. If the command is interactive, load `references/laravel-prompts-reference.md` and choose from the full prompt surface, not just basic text/confirm/select helpers.
+4. Implement using Laravel-first APIs (services/facades/components/prompts), then preserve required backward-compatible aliases.
+5. Run format + targeted tests and address findings.
+
+## Command Construction Rules
+- Use `final class ... extends Command` plus `use CraftCommand;`.
+- Define `protected $signature`, `protected $description`, and `protected $aliases` explicitly.
+- Read CLI inputs via `$this->argument()` / `$this->option()` inside `handle()`.
+- Inject services into `handle()` (or constructor when needed) instead of pulling from globals.
+- Prefer Laravel APIs over legacy Yii/Craft app APIs where equivalent exists.
+- Keep compatibility aliases when replacing legacy commands.
+
+## Output And Prompt Rules
+- Use Laravel Prompts for interactive input and interactive command UX, including `search()`, `multisearch()`, `autocomplete()`, `form()`, `progress()`, `task()`, `stream()`, `title()`, and `clear()` when they fit.
+- Prefer `$this->components->info|warn|error|success|task` for conventional Artisan status output in non-interactive or mixed flows.
+- Prompt-side informational helpers (`info()`, `warning()`, `error()`, `alert()`, `note()`, `intro()`, `outro()`, `table()`) are valid when the command is intentionally using a Prompts-driven experience; avoid mixing both styles line-by-line without a reason.
+- Avoid ad-hoc `output->write()`/`line()` for status messaging when components or prompt helpers fit.
+- Gate interactive prompts with `$this->input->isInteractive()` and provide non-interactive fallbacks.
+- Normalize prompt input with `transform:` before `validate:` when trimming or coercion would otherwise leak into validation logic.
+- Keep labels, option text, and validation messages short enough for narrow terminals.
+- In task closures, do not add unnecessary `return true;` values.
+- Do not add manual blank-line spacing after component calls unless behavior requires it.
+
+## When To Load References
+- You need any prompt type beyond `text()`, `confirm()`, `select()`, or `multiselect()`.
+- You need prompt validation/transform behavior or searchable prompt patterns.
+- You need form builder chaining (`form()->...->submit()`) or conditional form steps.
+- You need long-running interactive UX (`spin()`, `progress()`, `task()`, `stream()`) or terminal helpers (`title()`, `clear()`).
+- You need prompt-specific testing expectations or fallback behavior details.
+
+## Interactive Command Guidance
+- Prefer `search()` / `multisearch()` over large static selects when options come from the database or are too numerous to scan.
+- Prefer `autocomplete()` or `suggest()` when users benefit from completion but may still need freeform values.
+- Prefer `form()` for grouped setup flows where users may need to revisit earlier answers (`CTRL + U` in supported terminals).
+- Prefer `task()` for long-running work that benefits from a live log area, status messages, dynamic labels, or partial streamed output.
+- Prefer `progress()` for bounded loops and `spin()` for single opaque operations.
+- Assume unsupported environments and non-interactive runs still exist even when using Prompts; Laravel configures fallbacks automatically, but command behavior must remain safe and predictable.
+
+## Testing And Fallbacks
+- When prompt helpers produce informational output, assert it with prompt-aware expectations such as `expectsPromptsInfo()`, `expectsPromptsWarning()`, `expectsPromptsError()`, `expectsPromptsAlert()`, `expectsPromptsIntro()`, `expectsPromptsOutro()`, and `expectsPromptsTable()`.
+- If a command has both interactive and non-interactive branches, test both paths.
+- If reviewing custom prompt fallback behavior, prefer Laravel's built-in fallbacks unless the command genuinely needs `Prompt::fallbackWhen(...)` or prompt-class `fallbackUsing(...)` customization.
+
+## Validation
+Run at minimum:
+```bash
+./vendor/bin/pint <touched files>
+./vendor/bin/pest --compact <relevant tests>
+```

.agents/skills/artisan-command-authoring/references/laravel-prompts-reference.md

@@ -0,0 +1,260 @@
+# Laravel Prompts Reference
+
+## Source And Scope
+
+- Official Laravel 13 docs: <https://laravel.com/docs/13.x/prompts>
+- Prompt helper signatures: <https://github.com/laravel/prompts/blob/main/src/helpers.php>
+- Form builder signatures: <https://github.com/laravel/prompts/blob/main/src/FormBuilder.php>
+- Prompt test expectations in Artisan tests: `vendor/laravel/framework/src/Illuminate/Testing/PendingCommand.php`
+- Last synced: 2026-03-24
+
+This reference is for authoring and reviewing interactive Artisan commands in Craft CMS.
+
+## Import Patterns
+
+Import only the helpers you use:
+
+```php
+use function Laravel\Prompts\confirm;
+use function Laravel\Prompts\multiselect;
+use function Laravel\Prompts\search;
+use function Laravel\Prompts\text;
+```
+
+Or import many in one statement:
+
+```php
+use function Laravel\Prompts\{confirm, error, info, multiselect, search, text};
+```
+
+## Global Prompt Helpers
+
+### Input Helpers
+
+| Helper | Return | Key arguments | Notes |
+| --- | --- | --- | --- |
+| `text()` | `string` | `label`, `placeholder`, `default`, `required`, `validate`, `hint`, `transform` | Single line text input. |
+| `textarea()` | `string` | `label`, `placeholder`, `default`, `required`, `validate`, `hint`, `rows`, `transform` | Multiline input. |
+| `number()` | `int|string` | `label`, `placeholder`, `default`, `required`, `validate`, `hint`, `min`, `max`, `step` | Numeric input with arrow key adjustments. |
+| `password()` | `string` | `label`, `placeholder`, `required`, `validate`, `hint`, `transform` | Input is masked. |
+| `confirm()` | `bool` | `label`, `default`, `yes`, `no`, `required`, `validate`, `hint`, `transform` | Yes/no selection. |
+| `select()` | `int|string` | `label`, `options`, `default`, `scroll`, `validate`, `hint`, `required`, `transform` | Single option from fixed list. |
+| `multiselect()` | `array<int|string>` | `label`, `options`, `default`, `scroll`, `required`, `validate`, `hint`, `transform` | Multi option selection with space bar. |
+| `suggest()` | `string` | `label`, `options` (array or closure), `placeholder`, `default`, `scroll`, `required`, `validate`, `hint`, `transform` | Autocomplete, freeform value still allowed. |
+| `search()` | `int|string` | `label`, `options` (closure), `placeholder`, `scroll`, `validate`, `hint`, `required`, `transform` | Search first, then select one option. |
+| `multisearch()` | `array<int|string>` | `label`, `options` (closure), `placeholder`, `scroll`, `required`, `validate`, `hint`, `transform` | Search first, then select multiple options. |
+| `pause()` | `bool` | `message` | Waits for enter/return confirmation. |
+| `autocomplete()` | `string` | `label`, `options` (array or closure), `placeholder`, `default`, `required`, `validate`, `hint` | Inline type-ahead completion via tab/right arrow. |
+
+### Output And Utility Helpers
+
+| Helper | Return | Purpose |
+| --- | --- | --- |
+| `info()` | `void` | Informational message. |
+| `warning()` | `void` | Warning message. |
+| `error()` | `void` | Error message. |
+| `alert()` | `void` | Alert message. |
+| `note()` | `void` | Generic note style. |
+| `intro()` | `void` | Introductory message. |
+| `outro()` | `void` | Closing message. |
+| `table()` | `void` | Render rows and headers as a table. |
+| `grid()` | `void` | Render items in a grid. |
+| `spin()` | `mixed` | Spinner while callback executes; returns callback result. |
+| `progress()` | `array|Progress` | Progress bar for iterable or fixed step count. |
+| `task()` | `mixed` | Spinner + scrolling live log area for long-running work. |
+| `stream()` | `Stream` | Stream incremental text into the terminal. |
+| `title()` | `void` | Update the terminal window/tab title. |
+| `clear()` | `void` | Clear terminal. |
+| `form()` | `FormBuilder` | Build a multi-step prompt flow with backtracking. |
+
+## Shared Argument Behavior
+
+- `required`:
+  - `false` means optional.
+  - `true` enforces input.
+  - `string` enforces input with a custom message.
+- `validate`:
+  - Closure returning `null` or an error message.
+  - For many text-like prompts, validation rule arrays are also accepted (for example `['name' => 'required|max:255']`).
+- `transform`:
+  - Runs before validation.
+  - Use for normalization, such as `trim`, lowercase conversion, or casting.
+- `hint`:
+  - Help text shown under the prompt.
+- `scroll`:
+  - Visible option count before scrolling.
+- `options` return value:
+  - Associative arrays return keys.
+  - Indexed arrays return values.
+
+## Search Prompt Notes
+
+- `search()` and `multisearch()` expect an `options` closure that receives current input and returns options.
+- For value-based filtering, ensure the array is reindexed (`values()->all()` or `array_values(...)`) so it is not treated as associative.
+
+## Form Builder Reference
+
+Use `form()` when prompts are a sequence and users may need to go back (`CTRL + U` in supported terminals).
+
+- `submit()` returns indexed responses unless prompt steps are given `name:`.
+- Named steps return associative responses.
+- Use `add(...)` when later prompts depend on earlier responses.
+- `FormBuilder` also supports conditional chaining such as `when(...)` because it is conditionable.
+
+### Core Methods
+
+| Method | Purpose |
+| --- | --- |
+| `add(Closure $step, ?string $name = null, bool $ignoreWhenReverting = false)` | Add custom step closure. |
+| `addIf(Closure|bool $condition, Closure $step, ?string $name = null, bool $ignoreWhenReverting = false)` | Add conditional custom step. |
+| `submit(): array` | Execute steps and return responses. |
+
+### Built-in Prompt Methods On `FormBuilder`
+
+- `text(...)`
+- `textarea(...)`
+- `password(...)`
+- `confirm(...)`
+- `select(...)`
+- `multiselect(...)`
+- `suggest(...)`
+- `search(...)`
+- `multisearch(...)`
+- `pause(...)`
+- `spin(...)`
+- `note(...)`
+- `info(...)`
+- `warning(...)`
+- `error(...)`
+- `alert(...)`
+- `intro(...)`
+- `outro(...)`
+- `table(...)`
+- `progress(...)`
+
+Most form methods accept `name: 'key'` so `submit()` returns named responses.
+For numeric input in a form flow, use `add(...)` and call `number(...)` inside the closure.
+
+## Transform Before Validation
+
+Use `transform` when users might include formatting you do not want to validate directly:
+
+```php
+$slug = text(
+    label: 'Slug',
+    transform: fn (string $value) => trim(strtolower($value)),
+    validate: ['slug' => 'required|alpha_dash']
+);
+```
+
+## Informational Messages In Commands
+
+Prefer one style per command path and keep message noise low:
+
+- Prompt helpers for rich, prompt-themed messaging (`info()`, `warning()`, etc.).
+- `$this->components->...` for conventional Artisan status output.
+
+Use one approach intentionally and avoid mixing styles in every line unless there is a reason.
+
+## Task And Stream Patterns
+
+Use `task()` when a command should show live status while work is running.
+
+- `task()` callback receives a logger.
+- Logger methods include `line()`, `success()`, `warning()`, `error()`, `label()`, `partial()`, and `commitPartial()`.
+- Use `limit:` to control how many scrolling log lines remain visible.
+- `task()` and `spin()` animate when `ext-pcntl` is available; otherwise they degrade to static output.
+
+Use `stream()` when output itself should arrive incrementally rather than as task log lines:
+
+```php
+use function Laravel\Prompts\stream;
+
+$stream = stream();
+
+foreach ($chunks as $chunk) {
+    $stream->append($chunk);
+}
+
+$stream->close();
+```
+
+Use `title()` for long-running commands where terminal/tab context helps, and reset with `title('')` when appropriate.
+
+## Progress Patterns
+
+Map-style progress:
+
+```php
+use function Laravel\Prompts\progress;
+
+$results = progress(
+    label: 'Processing users',
+    steps: $users,
+    callback: fn ($user, $progress) => handleUser($user),
+);
+```
+
+Manual progress:
+
+```php
+$progress = progress(label: 'Processing users', steps: count($users));
+$progress->start();
+
+foreach ($users as $user) {
+    handleUser($user);
+    $progress->advance();
+}
+
+$progress->finish();
+```
+
+## Fallbacks And Unsupported Environments
+
+- Laravel Prompts supports macOS, Linux, and Windows under WSL.
+- Laravel framework configures fallbacks automatically in unsupported environments.
+- For non-Laravel or custom behavior:
+  - `Laravel\Prompts\Prompt::fallbackWhen(bool)`
+  - Per prompt class: `SomePrompt::fallbackUsing(Closure $fallback)`
+
+## Terminal Constraints
+
+- Keep labels/options/validation messages short enough for narrow terminals.
+- A safe target is about 74 characters for 80-column terminals.
+- Prompts with `scroll` automatically clamp to terminal height.
+
+## Testing Prompt Output In Artisan Tests
+
+Available prompt message assertions on `PendingCommand`:
+
+- `expectsPromptsInfo(string $message)`
+- `expectsPromptsWarning(string $message)`
+- `expectsPromptsError(string $message)`
+- `expectsPromptsAlert(string $message)`
+- `expectsPromptsIntro(string $message)`
+- `expectsPromptsOutro(string $message)`
+- `expectsPromptsTable(array|Collection $headers, array|Collection|null $rows)`
+
+## Craft Command Integration Pattern
+
+Always gate prompts in case the command is running non-interactively:
+
+```php
+use function Laravel\Prompts\confirm;
+use function Laravel\Prompts\text;
+
+$name = $this->argument('name');
+
+if (!$name) {
+    if (!$this->input->isInteractive()) {
+        $this->components->error('The name argument is required in non-interactive mode.');
+        return self::FAILURE;
+    }
+
+    $name = text(label: 'Name', required: true);
+}
+
+$shouldProceed = $this->input->isInteractive()
+    ? confirm('Continue?', default: true)
+    : (bool)$this->option('force');
+```