Skip to content

feat: show clearer analysis error messages for API failures - #110

Merged
vvatelot merged 1 commit into
mainfrom
feat/analysis-error-messages
Aug 18, 2026
Merged

vvatelot merged 1 commit into
mainfrom
feat/analysis-error-messages

Conversation

@vvatelot

Copy link
Copy Markdown
Member

Summary

  • Map POST errors 400, 401 and 403 (including excluded hosts) to dedicated French messages in the popup
  • Clarify GET analysis failures 500, 520 and 521 so users know when the page is unreachable, not HTML, or not returning HTTP 200
  • Stop treating a failed POST JSON body as a task id, and keep the technical API payload in the expandable error detail

Test plan

  • Trigger a POST 400 with an unreachable URL and check the popup title/detail
  • Trigger a POST 401 on a login-protected page
  • Trigger a POST 403 on an antibot-protected page, and on an excluded host if possible
  • Trigger analysis failures 500 / 520 (non-HTML resource) / 521 (non-200 page) via GET task polling
  • Confirm a successful analysis still displays the result as before

Made with Cursor

Map POST 400/401/403 and analysis 500/520/521 to dedicated French copy so users get an actionable explanation instead of raw API errors.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vvatelot
vvatelot merged commit ad94bc8 into main Aug 18, 2026
1 check passed
@vvatelot
vvatelot deleted the feat/analysis-error-messages branch August 18, 2026 13:17
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