Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: browser kill after first task completion caused by context reset #14

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

michiosw
Copy link
Contributor

No description provided.

- Create isolated browser contexts for each task to prevent conflicts
- Replace global browser context with task-specific instances
- Add proper resource cleanup in finally blocks to prevent memory leaks
- Extract hardcoded values as constants for better maintainability
- Add comprehensive type hints throughout the codebase
- Improve error handling and logging
- Reorganize imports and improve code structure
- Update Dockerfile to use Playwright's bundled browser instead of custom Chrome
- Ensure each task runs in its own isolated environment with unique debugging ports
- Fix potential race conditions when multiple tasks access the same context
@michiosw
Copy link
Contributor Author

feat: Improve browser task isolation and resource management

  • Create isolated browser contexts for each task to prevent conflicts
  • Replace global browser context with task-specific instances
  • Add proper resource cleanup in finally blocks to prevent memory leaks
  • Extract hardcoded values as constants for better maintainability
  • Add comprehensive type hints throughout the codebase
  • Improve error handling and logging
  • Reorganize imports and improve code structure
  • Update Dockerfile to use Playwright's bundled browser instead of custom Chrome
  • Ensure each task runs in its own isolated environment with unique debugging ports
  • Fix potential race conditions when multiple tasks access the same context

@michiosw michiosw requested a review from maceip March 15, 2025 21:51
@michiosw michiosw self-assigned this Mar 15, 2025
@michiosw michiosw added the enhancement New feature or request label Mar 15, 2025
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.

1 participant