-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[STG-1045] Add logging at all 4 levels: agent.execute, agent.act/observe/extract, CLICK/HOVER/SCROLL, and CDP #1283
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
Merged
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
00aae08
add new flow logger
pirate 9c7d138
hide unused extract args
pirate 06472d6
fix the lint errors
pirate 9d06f31
fix unused label var
pirate b788e4d
Write flow logs to configurable file paths instead of stdout
github-actions[bot] b289fa8
simplify flow logging
pirate 6b632cc
improve flow logging timestamp prefixes
pirate aa02157
more accurate flow logging context var tracking
pirate d69fe15
fix id scoping for spans where no task is running, use uuidv7 for eveβ¦
pirate 121f463
move logging and formatting into flowLogger.ts and add logging for CUβ¦
pirate cc03642
refactor to use pino for logging
pirate b2af3fe
better prompr preview for CUA llm calls
pirate ccd0a96
code cleanups
pirate b651f85
make truncation consistent across all lines
pirate 5f34408
disable logging by default unless BROWSERBASE_CONFIG_DIR is set
pirate f05434e
use decorator for cleaner logging, add skip arrows back
pirate bc1c239
fix no-op screenshot logging in v3cua
pirate 9005489
fix imports from bad rebase
pirate 9453a31
reduce diff in page.ts by using decorator
pirate 86a5484
reduce diff in v3.ts by using decorator
pirate 3b62ab2
add safety guards to prevent errors in main logic
pirate a7863f9
fix indentation diffs
pirate 4cb2a69
fix lint issues
pirate 081c201
remove uneeded pkg
pirate 1a4ba75
also log llm response even in the case of parsing error
pirate 2f47d8a
log errors in the case of llm response parsing issue
pirate 558aef1
fix missing SessionFileLogger.logAgentTaskCompleted in agent stream mode
pirate d8cd98b
bump lockfiles
pirate 74f4567
dont modify root lockfiles
pirate 705b957
comment why Page.setViewportSize is disabled
pirate 23aa321
bump lockfiles
pirate 81621b1
Update packages/core/lib/v3/flowLogger.ts
pirate File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note for the future, we might want to keep track of other params (non-sensitive) in model based on the
ModelConfigurationtypeThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note for compactness we can also store the default model config in
sessions/{id}/session.jsondir, and then only add extra to the individual loglines if they are different from the session-wide defaults.