Skip to content

Stop testing on error pages#55

Merged
DavertMik merged 2 commits into
mainfrom
fix/stop-testing-on-error-pages
Jun 16, 2026
Merged

Stop testing on error pages#55
DavertMik merged 2 commits into
mainfrom
fix/stop-testing-on-error-pages

Conversation

@DenysKuchma

Copy link
Copy Markdown
Collaborator

Summary

Stops exploration/test execution when the current page is detected as an HTTP error page. #54

Changes

  • Capture main document HTTP status from Playwright navigation responses
  • Store httpStatus in page state/action results.
  • Detect error pages by HTTP status before falling back to existing title/heading checks
  • Stop explore before planning when the current page is an error page
  • Stop test before AI execution when the start/current page is an error page
  • Clear stale testing activity after early abort.
  • Add unit coverage for error-page detection, explore guard, and tester abort flow

@DenysKuchma DenysKuchma requested a review from DavertMik June 15, 2026 20:38

@DavertMik DavertMik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

minor ideas

Comment thread src/ai/tester.ts
Comment thread src/commands/explore-command.ts Outdated
@DenysKuchma DenysKuchma requested a review from DavertMik June 16, 2026 09:29
@DavertMik DavertMik merged commit 29b998e into main Jun 16, 2026
2 checks passed
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.

2 participants