Conversation
Only `odoo/addons` is required as other base modules are loaded by the framework. Explicitly adding that path in the addons-paths of `odoo-bin` introduces errors when running an upgrade locally.
Instead, only load table definitions and constraints when the table is created. This only impacts Odev's database, used for internal operations. Performance gains: - Framework loaded: 5ms (1.3%) - Framework started: 48ms (6.4%) - Execution completed: 46ms (5.5%)
Avoid some costly operations when update is not needed. Performance gains: - Framework loaded: 6ms (1.7%) - Framework started: 48ms (5.8%) - Execution completed: 51ms (6.5%)
Plugins where installed at every run of odev, taking a substantial amount of time. Also check for missing requirements before running `pip install`, as the operation is generally 50 times faster and can prevent running a useless install. Performance gains: - Framework loaded: 25ms (8.1%) - Framework started: 211ms (42.3%) - Execution completed: 184ms (31.1%)
Move some imports to lazy-load features only when required. Performance gains: minimal
We noticed the implematation of `functools.ttl_cache` was not fit with our needs and was failing to cache some subprocesses results.
This had an unnecessary toll on performances when listing databases.
Allow choosing a value for the bypass (do or do not).
Only run the subprocess once for all databases instead of once per database. This makes it more efficient if coupled with proper caching.
Stop filtering detached and version-mismatched worktrees in the shared iterator so commands can decide how to handle each case themselves. This avoids false "worktree does not exist" errors when a command needs to inspect detached worktrees before applying its own logic.
`warnings.deprecated` exists only from Python 3.13 onward; add `odev.common.deprecation` with a pre-3.13 shim and use it from `console` so CI on 3.10–3.12 can import the stack again. Stop shadowing the `upgrade` argument in `Odev.update()` and run the post- update path when the repo or plugins reported changes (or `upgrade=True`). Fix `install_requirements` treating the `missing_requirements` generator as always truthy, and skip cleanly when there is no `requirements.txt` so test plugins can install. Make `test_04_restart_on_update` force `__should_update_now` so `start()` exercises update reliably. Add 3.13 to the unit-test matrix and ignore `.cursor/` and `.agents/` for local tooling.
brinkflew
changed the title
odoo/odoo/addons in addons-paths
* fix: only process requested version worktrees in GitCommand * fix: respect ODEV_NO_SSH_AGENT and handle decryption failure gracefully * fix: correctly handle detached HEAD and missing upstream in worktrees * fix: resolve pre-commit issues and improve robustness * chore: bump version to 4.26.0
…134) * [REF] core: support AI CLI sandboxed execution * fix(rest): add ConnectionResetError to retryable exceptions The REST connector now catches ConnectionResetError in addition to RequestsConnectionError during requests. This ensures that the built-in retry logic is triggered when a connection is reset by the peer, improving resilience against transient network issues. * fix(test): improve debugger detection regex * [IMP] rest: improve cookie handling and fix caching order
- Add head method to RESTConnector - Fix virtualenv check in LocalDatabase
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.
Description
Only
odoo/addonsis required as other base modules are loaded by the framework. Explicitly adding that path in the addons-paths ofodoo-binintroduces errors when running an upgrade locally.Compliance
docsdirectoryrequirements.txtfile, if any