This repository was archived by the owner on Nov 29, 2025. It is now read-only.
fix: enhance browser tool reliability and minimize outputs #92
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…stness - Introduce retry mechanism for `act` and `observe` methods to handle intermittent protocol errors during navigation. - Refactor `interaction_context_capture` to return a callable. - Suppress verbose logging for `LiteLLM` to improve tool output clarity. - Replace direct calls to page methods with newly added `act` and `observe` wrappers for consistency.
…eck to non-redirects - Skip response body size checks for redirects to avoid unnecessary processing. - Add error handling with logging for potential failures during response body retrieval.
…evance - Separate important and skip headers to enhance readability and maintainability. - Exclude less relevant security headers like `x-frame-options` and `x-xss-protection`. - Introduce improved filtering for `x-` headers, skipping `x-amz-` while capturing others.
- Refactor network call summarization to return HAR file details instead of verbose stringified outputs. - Replace dictionary-based `logs` and `dialogs` with `TypedDict` for improved type safety and clarity. - Update `simplify_metadata_for_llm` to include keyword-only arguments for better readability.
Author
|
@westonbrown this should reduce token usage during browser calls. I'm now only returning a reference to a har file and prompting the agent to use I'm not seeing any degradation in behaviour. I tested before and after with same vulnerable target and it found the issue within similar time frames. Old approach
New approach
|
Owner
|
@niranjan94 this is great! Tested today and awesome work. Thanks for the contribution. Working on addition conversation manager refactor that should further improve ctx management. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Follow up to #84 (based on further testing against different targets)
actandobservemethods to handle intermittent protocol errors during navigation