[IMP] various: implement sandbox log filtering and enhance stability#150
Merged
Conversation
sea-odoo
force-pushed
the
feature/sandbox-log-filtering
branch
from
845d272 to
24f59f9
Compare
sea-odoo
changed the title
sea-odoo
force-pushed
the
feature/sandbox-log-filtering
branch
3 times, most recently
from
114137b to
845d272
Compare
Add a token-efficient log filtering mechanism for the AI sandbox to suppress noisy HTTP logs and strip redundant metadata. This improves AI context clarity and reduces token usage. Also, fix RepositoriesSection.get_date to return a default epoch for new versions to ensure the initial pull logic works correctly. Assisted-by: gemini-3-flash <noreply@google.com>
- Robust ODOO_LOG_REGEX to handle aligned logs (multiple spaces). - Suppress odoo-bin command and info messages in AI sandbox to save tokens. - Remove redundant database name from pretty-printed logs. Assisted-by: gemini-3-flash <noreply@google.com>
Fixed a bug where comma-separated lists (captured by 'addons' argument) were not correctly rescued from unknown flags, causing Odoo to misinterpret flags following '-i' as module names. Assisted-by: gemini-3-flash <noreply@google.com>
Assisted-by: Gemini 3 Flash <noreply@google.com>
sea-odoo
force-pushed
the
feature/sandbox-log-filtering
branch
from
845d272 to
012983d
Compare
brinkflew
previously approved these changes
Apr 27, 2026
- Fix duplicate test name test_16 in test_database.py - Mock GitConnector in TestGitCommands to avoid real cloning and hangs - Fix LOG_WERKZEUG_REGEX to handle standard Werkzeug logs - Restore HOME_PATH in OdevTestCase.tearDownClass for better test isolation Assisted-by: antigravity <noreply@google.com>
…class - Add odoobin_process_class property to Odev with getter/setter - Add should_skip_update() hook method to Odev (replaces direct AI_SANDBOX checks) - Replace OdoobinProcess.run AI_SANDBOX checks with odev.should_skip_update() - Add OdoobinProcess.get_stream_filter() as an overridable hook (returns None) - Remove _ai_sandbox_filter from core OdoobinProcess (moved to plugin) - Update all OdoobinProcess instantiation sites to use odev.odoobin_process_class Assisted-by: claude-sonnet-4-5 <noreply@google.com>
…e() in OdoobinProcess Remove the odev.should_skip_update() call from OdoobinProcess.run — core stays completely unaware of sandbox concepts. Instead, add an overridable _print_run_info() that always prints by default; AI_OdoobinProcess overrides it to be a no-op to save tokens inside the sandbox. Assisted-by: claude-sonnet-4-5 <noreply@google.com>
brinkflew
requested changes
Apr 27, 2026
…m core Core odev now relies solely on ODEV_SKIP_GIT_UPDATE for update suppression, leaving AI-specific behavior entirely to the plugin. Assisted-by: claude-sonnet-4-5 <noreply@google.com>
- Revert check_release and rename __should_update_now to _should_update_now - Implement native stream filtering in run_script (supporting None to skip) - Move AI-specific sandbox logic entirely to the plugin via overrides - Rename progress to stream_filter and update all call sites Assisted-by: claude-sonnet-4-5 <noreply@google.com>
sea-odoo
force-pushed
the
feature/sandbox-log-filtering
branch
from
47e3e55 to
d61d155
Compare
brinkflew
reviewed
Apr 27, 2026
brinkflew
previously approved these changes
Apr 27, 2026
…n loading - Add `_odoobin_process_class` to `OdevFrameworkMixin` for flexible Odoo process specialization. - Refactor plugin loading to use direct imports and ensure `odev.plugins` namespace is correctly populated. - Minor improvements to `OdoobinProcess` representation and output formatting. Assisted-by: gemini-3-flash <noreply@google.com>
- Revert plugin path fallback logic in `plugins` property. - Remove extra newline in `_print_run_info` to keep output compact. Assisted-by: gemini-3-flash <noreply@google.com>
- Use richer console output for the Odev run message. - Restore the extra newline after the command for better readability. Assisted-by: gemini-3-flash <noreply@google.com>
brinkflew
reviewed
Apr 28, 2026
Contributor
|
And the tests are failing again |
…re logger.info for info_message to ensure tests can capture it\n- Remove custom rich styling and extra spacing\n\nAssisted-by: gemini-3-flash <noreply@google.com>
Contributor
Author
fixed |
brinkflew
approved these changes
Apr 28, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR consolidates several improvements for the odev framework, focusing on AI sandbox stability and core utility testing.
Key changes: