fix: #3613457 Remove the em dash from the mode selector label and show "Prompt only" for prompt-only modes

Rajab Natshahrequested to merge
issue/ai_agent_modes-3613457:3613457-remove-em-dash into 1.0.x

Fixes #3613457.

Two user interface fixes, both verified in a real browser on a Varbase 11 site (8 shipped modes) before this MR was opened.

1. The default selector option no longer uses an em dash. It read All — let the assistant decide and now reads All, let the assistant decide. Verified through the live options endpoint the JavaScript consumes, /ai-agent-modes/options/canvas_ai_orchestrator.

2. The Modes listing no longer says "None" for prompt-only modes. A mode with an empty sub_agents list and a non-empty system_prompt_addition is still restrictive (ScopePayload::isRestrictive()), so it does steer the assistant, but the column read None as if nothing happened. It now reads Prompt only, and None is kept only for a mode with neither. On the test site the three Drupal Canvas modes that steer by directive (Build a page from Figma, Connect pages to the site, Review and improve a page) went from None to Prompt only; the five with real sub-agents are unchanged.

The remaining em dashes in code comments, the README and two test files are removed so the codebase is consistent. No configuration schema, stored data or API changes.

Checkpoints

  • File an issue
  • Addition/Change/Update/Fix
  • Reviewed by a human
  • Code review by maintainers
Edited by Rajab Natshah

Merge request reports