…nded-issue closures) (#75) DISCLOSURE: LLM-GENERATED TEXT ## Why The quick-launch bar could only complete command names: once a gameplay command needed an argument (static choices or a dynamic provider), the palette was a dead end — T10's last user-facing remainder. ## What - Palette queries run the shell's `TryComplete` first: when the input resolves to an eligible command with an active argument stage, rows show that command's completion candidates (static choices, dynamic providers, staged chaining, subcommand routing via the shared shell path). Tab and row activation apply the selected candidate at the active token's replacement range (override ranges honored, space-containing values quoted when the token is not already quoted, caret parked like the terminal's); Enter runs the assembled line; failures keep the existing feedback flow. - Arrows never auto-load completion rows (it would corrupt the command text being assembled); ineligible commands (e.g. Editor-only, viewed in Play Mode) fall back to command-name filtering; a provider that answers with nothing collapses the results instead of falling back. - `TerminalUI.QuoteInsertionIfNeeded` moved verbatim to `CommandTokenizer.QuoteInsertionIfNeeded` so both surfaces share one insertion-quoting rule. - Row activation is index-driven (`RowActivated`): Unity 6000.4's dispatcher drops synthetic pointer events (re-probed this session; same blocked-parity status as the terminal's pointer tests). - Caret for completion is derived at change time (append -> new end, else live caret) because the TextField's cursorIndex lags programmatic value writes at ChangeEvent time — a stale live read produced wrong-stage completions in the first red run. - 10 new PlayMode tests: data-driven candidate listing (incl. quoted prefix), zero-candidate collapse, name fallback, eligibility, Tab apply + quoting, Tab inside an open quote, stage chaining + post-apply cycling, Enter runs the completed command, row activation applies without running, caret continuation after a command-name apply. ## Evidence - PlayMode 349/349 on Unity 6000.4.6f1 (full suite, committed code), plus 348/348 under the `.NET Framework` API level. - Generator 32/32; node tooling 155/155; package-content validator 842 entries green (6.7 MB packed / 14.4 MB unpacked); csharpier + all five C# linters + meta/llm lints clean. - One-off caret-poll flake observed under full-suite sequences and an attempted fix that regressed another caret test were reverted and recorded in #74 (diagnostics session recommended), together with the pre-existing terminal-side eligibility gap surfaced by review. ## Issue sweep (same session, recorded per GOAL) Closed with evidence comments after verifying the tree: #18 (.NET Framework RCA + live drill: package compiles and passes the full suite under `NET_Unity_4_8`, settings restored; README compatibility table updated), #52 (font-payload live smoke test: `list-fonts`/`list-themes`/valid + invalid `set-font`/`set-random-font` all behave), #64, #56, #20, #39, #47, #48, #50, #51, #54, #55. Left open deliberately: #22 (awaiting the reporter's confirmation), #40/#72/#66 (wallstop decisions pending), #38 (T05 remainder), #21/#19/#16 (out-of-scope boundaries), #74 (new follow-up). Refs: #74 (new), T10 remainder (palette dynamic argument completion), #17.  --- > [!NOTE] > **Medium Risk** > Touches interactive input, caret timing, and shell completion on a user-facing surface; behavior is covered by new PlayMode tests but shares complexity with terminal completion. > > **Overview** > The quick-launch **`CommandPaletteUI`** now completes **command arguments**, not just command names: when the input resolves to an eligible command with an active argument stage, result rows come from **`CommandShell.TryComplete`** (static/dynamic/staged providers, same path as the terminal). **Tab** and row activation insert the selected candidate at the active token (with optional quoting); **Enter** still runs the full line. Arrow navigation no longer auto-fills the input in completion mode; ineligible commands (e.g. Editor-only in Play Mode) and empty provider results collapse or fall back to name search. Caret position for completion is derived on input change so appended text completes at the line end despite UITK `cursorIndex` lag. > > **`QuoteInsertionIfNeeded`** is centralized on **`CommandTokenizer`**; **`TerminalUI`** calls that shared helper instead of a private duplicate. > > **README** documents **.NET Standard 2.0** and **.NET Framework 4.x** API compatibility; **CHANGELOG** and Quick-Launch Bar docs describe palette argument completion. **PlayMode tests** cover listing, Tab/quoting, staging, eligibility, Enter vs row-apply, and caret continuation. > > <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit a39b70d. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup>

chore: CHANGELOG Should Only be user-facing changes #54

Description

Activity

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions