Skip to content

PAB-54: improve palette feedback workflow#131

Merged
pablopunk merged 2 commits into
mainfrom
agent/orchestrator/406a9f1a
Jul 17, 2026
Merged

PAB-54: improve palette feedback workflow#131
pablopunk merged 2 commits into
mainfrom
agent/orchestrator/406a9f1a

Conversation

@pablopunk

@pablopunk pablopunk commented Jul 17, 2026

Copy link
Copy Markdown
Owner

This PR improves the feedback workflow for command palette actions by replacing ad‑hoc error and status views with a consistent, accessible list‑based feedback pattern. The changes deliver clearer user messaging, better accessibility, and richer toast notifications while protecting users from internal error details.

What changed

  • Standardized feedback views
    Errors, warnings, and success states in the palette now use a structured list view (feedbackView) instead of raw markdown previews. Each feedback view shows:

    • A clear title and message, color‑coded and icon‑annotated by tone (error, success, info).
    • Optional contextual detail items (e.g., which files failed to uninstall).
    • A default “Back” action or custom recovery actions, making it easy to navigate away or retry.
  • Richer toast notifications
    Toast messages now support info and success tones in addition to default and error, allowing more precise feedback for completed tasks (e.g., “1 item moved to Trash” shown as a success toast). Error toasts use role="alert" for screen readers.

  • Safer user‑facing error messages
    System‑returned errors (such as app uninstall failures) are now replaced with concise, user‑friendly messages that never expose internal paths or raw system output.

  • Accessibility enhancements

    • The new disabled item property prevents non‑selectable items from receiving keyboard focus, keeping navigation streamlined.
    • Form fields now include aria-invalid and aria-describedby to associate error messages with their controls.
    • The loading indicator gains an accessible status role.
  • Extension API updates
    Extensions can now set disabled: true on palette items and use info/success toast tones. The ui.error() helper returns a feedback view instead of a plain markdown preview, aligning with the new pattern.

These changes make the overall palette feedback more predictable, visually scannable, and accessible while reducing the risk of leaking sensitive information through error messages.

Co-authored-by: multica-agent <github@multica.ai>
@vercel

vercel Bot commented Jul 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nvm Ready Ready Preview, Comment Jul 17, 2026 3:48pm

Co-authored-by: multica-agent <github@multica.ai>
@pablopunk
pablopunk marked this pull request as ready for review July 17, 2026 15:59
@kody-ai

kody-ai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug
Performance
Security
Business Logic

Access your configuration settings here.

@pablopunk
pablopunk merged commit 5b2d707 into main Jul 17, 2026
10 checks passed
@pablopunk
pablopunk deleted the agent/orchestrator/406a9f1a branch July 17, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant