Skip to content

fix(AFM): add error handling for Selenium, close browser on failure#234

Open
tatenda-source wants to merge 1 commit into
FujiwaraChoki:mainfrom
tatenda-source:fix/afm-selenium-browser-cleanup
Open

fix(AFM): add error handling for Selenium, close browser on failure#234
tatenda-source wants to merge 1 commit into
FujiwaraChoki:mainfrom
tatenda-source:fix/afm-selenium-browser-cleanup

Conversation

@tatenda-source

Copy link
Copy Markdown

Summary

  • Wrap Selenium element lookups in try/except with clear error messages
  • Close browser on init failure to prevent zombie Firefox processes

Closes #227

Test plan

  • Test with valid Amazon affiliate link
  • Test with invalid/broken link to verify browser cleanup

🤖 Generated with Claude Code

Closes FujiwaraChoki#227

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

fix(AFM): unhandled Selenium exceptions and browser not closed on error

1 participant