Skip to content

🥅 Improve Aegis suggestion errors#706

Merged
C-Valen merged 3 commits intomainfrom
task/AEGIS-272-improve-aegis-error-details
Jan 7, 2026
Merged

🥅 Improve Aegis suggestion errors#706
C-Valen merged 3 commits intomainfrom
task/AEGIS-272-improve-aegis-error-details

Conversation

@C-Valen
Copy link
Copy Markdown
Member

@C-Valen C-Valen commented Dec 30, 2025

AEGIS-272 Improve Aegis suggestion errors

Checklist:

  • Commits consolidated
  • Changelog updated
  • Test cases added/updated
  • [-] Integration tests updated
  • Jira ticket updated

Summary:

Improved Aegis errors handling to provide user-friendly, descriptive messages for network failures and backend errors instead of generic "Request failed" text.

Scenario Message
Network Unable to connect to AEGIS-AI server. Please check your network connection or try again later.
Backend AEGIS-AI server error (HTTP 500 Internal Server Error). Please try again or contact support.

Changes:

  • Removed toast calls from service layer
  • Added descriptive messages
  • Added test for network error handling

@C-Valen C-Valen requested a review from a team December 30, 2025 14:59
@C-Valen C-Valen self-assigned this Dec 30, 2025
@C-Valen C-Valen added the enhancement New feature or request label Dec 30, 2025
@C-Valen C-Valen force-pushed the task/AEGIS-272-improve-aegis-error-details branch from cdc8ca2 to 042fad5 Compare December 30, 2025 15:01
Copy link
Copy Markdown
Collaborator

@superbuggy superbuggy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice polish on the error messages

@C-Valen C-Valen force-pushed the task/AEGIS-272-improve-aegis-error-details branch from 042fad5 to d756b37 Compare January 7, 2026 11:26
@C-Valen C-Valen merged commit 54f9610 into main Jan 7, 2026
5 checks passed
@C-Valen C-Valen deleted the task/AEGIS-272-improve-aegis-error-details branch January 7, 2026 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants