Skip to content

Fix/external logs#7

Merged
kalil0321 merged 2 commits into
mainfrom
fix/external-logs
Dec 26, 2025
Merged

Fix/external logs#7
kalil0321 merged 2 commits into
mainfrom
fix/external-logs

Conversation

@kalil0321
Copy link
Copy Markdown
Owner

@kalil0321 kalil0321 commented Dec 26, 2025

Greptile Summary

This PR removes external library logging noise and fixes hardcoded version strings. The main changes include:

  • Simplified logging suppression for browser-use by replacing complex MemoryLogHandler with direct logger configuration
  • Added module-level logging suppression for claude_agent_sdk in engineer.py and utils.py
  • Replaced hardcoded version strings with dynamic __version__ imports across browser.py, opencode_ui.py
  • Improved result display formatting to hide generic "Task completed" messages
  • Version bumped to 0.2.3

The refactoring significantly simplifies the codebase by removing ~70 lines of complex log handler code in favor of straightforward logger suppression. The changes are well-focused on the stated goal of removing external logs.

Confidence Score: 4/5

  • Safe to merge with minor style improvements recommended
  • The changes are focused and accomplish the goal of suppressing external logs. The replacement of the complex MemoryLogHandler with simpler suppression logic is a good refactor. Minor concerns include redundant suppress_browser_use_logs() calls and potential environment variable timing issues, but these don't affect correctness.
  • No files require special attention - all changes are straightforward logging and version string updates

Important Files Changed

Filename Overview
src/reverse_api/init.py Version bumped from 0.2.2 to 0.2.3
src/reverse_api/browser.py Replaced complex memory log handler with simpler logging suppression, updated to use __version__ import, added environment variable for browser-use logging, and improved result display formatting
src/reverse_api/engineer.py Added logging suppression for claude_agent_sdk at module level
src/reverse_api/opencode_ui.py Updated to dynamically import and display __version__ instead of hardcoded version string
src/reverse_api/utils.py Added logging suppression for claude_agent_sdk in async folder name generation function

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@kalil0321
Copy link
Copy Markdown
Owner Author

@greptile

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

5 files reviewed, 3 comments

Edit Code Review Agent Settings | Greptile

Comment thread src/reverse_api/browser.py
Comment thread src/reverse_api/browser.py
Comment thread src/reverse_api/browser.py
@kalil0321 kalil0321 merged commit 2be6d16 into main Dec 26, 2025
2 checks passed
@kalil0321 kalil0321 mentioned this pull request May 5, 2026
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.

1 participant