[MIG] ai_oca_bridge: Migration to 19.0

- Domain handling migrated to `odoo.fields.Domain` (AND helpers, safe empty defaults) and user group access checks updated to new `group_ids` API.
- Async response URLs now built with `odoo.tools.urls.urljoin` and chatter tracking disabled in tests via a new mixin to avoid noisy metadata.
- JS/web tests now create bridge records dynamically (no hardcoded mock data) and fake test model registration updated for Odoo 19 registry APIs; inactive bridge uses `action_archive`.
- Message/action tests now share computed domains for clarity and resilience under the new Domain API.
* Change _ to self.env._, more info at odoo/odoo#174844
* Work around charget AttributeError
* Import ValidationError from odoo.exceptions instead of models
* Rename webRecord to record
* Use new way of popover
* Test for response content

Author: hailangvn

ai_oca_bridge/README.rst

@@ -21,13 +21,13 @@ AI OCA Bridge
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fai-lightgray.png?logo=github
-    :target: https://github.com/OCA/ai/tree/18.0/ai_oca_bridge
+    :target: https://github.com/OCA/ai/tree/19.0/ai_oca_bridge
     :alt: OCA/ai
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/ai-18-0/ai-18-0-ai_oca_bridge
+    :target: https://translation.odoo-community.org/projects/ai-19-0/ai-19-0-ai_oca_bridge
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
-    :target: https://runboat.odoo-community.org/builds?repo=OCA/ai&target_branch=18.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/ai&target_branch=19.0
     :alt: Try me on Runboat
 
 |badge1| |badge2| |badge3| |badge4| |badge5|
@@ -159,7 +159,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/ai/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/ai/issues/new?body=module:%20ai_oca_bridge%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/ai/issues/new?body=module:%20ai_oca_bridge%0Aversion:%2019.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -200,6 +200,6 @@ OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.
 
-This module is part of the `OCA/ai <https://github.com/OCA/ai/tree/18.0/ai_oca_bridge>`_ project on GitHub.
+This module is part of the `OCA/ai <https://github.com/OCA/ai/tree/19.0/ai_oca_bridge>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

ai_oca_bridge/__manifest__.py

@@ -6,13 +6,17 @@
     "summary": """
     Makes a basic configuration to be used as bridge with external AI systems
     """,
-    "version": "18.0.2.0.0",
+    "version": "19.0.1.0.0",
     "license": "AGPL-3",
     "author": "Dixmit,Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/ai",
     "category": "AI",
     "development_status": "Beta",
-    "depends": ["mail"],
+    "depends": [
+        "base",
+        "mail",
+        "web",
+    ],
     "data": [
         "data/ir_module_category.xml",
         "security/ir.model.access.csv",

ai_oca_bridge/controllers/ai.py

@@ -6,7 +6,6 @@
 from odoo import fields, http
 from odoo.http import request
 from odoo.tools import consteq
-from odoo.tools.translate import _
 
 
 class AIController(http.Controller):
@@ -22,24 +21,27 @@ class AIController(http.Controller):
     def ai_process_response(self, execution_id, token):
         execution = request.env["ai.bridge.execution"].sudo().browse(execution_id)
         if not execution.exists():
-            return request.make_response(_("Execution not found."), status=404)
+            return request.make_response(self.env._("Execution not found."), status=404)
         if not consteq(execution._generate_token(), token):
             return request.make_response(
-                _("Token is not allowed for this execution."), status=404
+                self.env._("Token is not allowed for this execution."),
+                status=404,
             )
         if (
             not execution.expiration_date
             or execution.expiration_date < fields.Datetime.now()
         ):
-            return request.make_response(_("Execution is expired."), status=404)
+            return request.make_response(
+                self.env._("Execution is expired."), status=404
+            )
+        try:
+            charset = request.httprequest.charset
+        except AttributeError:
+            charset = "utf-8"
         return request.make_response(
             json.dumps(
                 execution._process_response(
-                    json.loads(
-                        request.httprequest.get_data().decode(
-                            request.httprequest.charset
-                        )
-                    )
+                    json.loads(request.httprequest.get_data().decode(charset))
                 )
             ),
             headers=[

ai_oca_bridge/models/ai_bridge.py

@@ -6,7 +6,9 @@
 import logging
 from datetime import date, datetime
 
-from odoo import _, api, fields, models
+from odoo import api, fields, models
+from odoo.exceptions import ValidationError
+from odoo.fields import Domain
 from odoo.tools.safe_eval import safe_eval
 
 _logger = logging.getLogger(__name__)
@@ -172,8 +174,8 @@ def _compute_payload_type(self):
     def _check_payload_type_usage_compatibility(self):
         for record in self:
             if record.usage == "ai_thread_unlink" and record.payload_type != "none":
-                raise models.ValidationError(
-                    _(
+                raise ValidationError(
+                    self.env._(
                         "When usage is 'On Record Deleted', "
                         "the Payload Type must be 'No payload'."
                     )
@@ -235,11 +237,11 @@ def _get_info(self):
     def execute_ai_bridge(self, res_model, res_id):
         self.ensure_one()
         if not self.active or (
-            self.group_ids and not self.env.user.groups_id & self.group_ids
+            self.group_ids and not self.env.user.group_ids & self.group_ids
         ):
             return {
-                "body": _("%s is not active.", self.name),
-                "args": {"type": "warning", "title": _("AI Bridge Inactive")},
+                "body": self.env._("%s is not active.", self.name),
+                "args": {"type": "warning", "title": self.env._("AI Bridge Inactive")},
             }
         record = self.env[res_model].browse(res_id).exists()
         if record:
@@ -260,24 +262,31 @@ def execute_ai_bridge(self, res_model, res_id):
             if execution.state == "done":
                 return {
                     "notification": {
-                        "body": _("%s executed successfully.", self.name),
-                        "args": {"type": "success", "title": _("AI Bridge Executed")},
+                        "body": self.env._("%s executed successfully.", self.name),
+                        "args": {
+                            "type": "success",
+                            "title": self.env._("AI Bridge Executed"),
+                        },
                     }
                 }
             return {
                 "notification": {
-                    "body": _("%s failed.", self.name),
-                    "args": {"type": "danger", "title": _("AI Bridge Failed")},
+                    "body": self.env._("%s failed.", self.name),
+                    "args": {
+                        "type": "danger",
+                        "title": self.env._("AI Bridge Failed"),
+                    },
                 }
             }
 
     def _enabled_for(self, record):
         """Check if the bridge is enabled for the given record."""
         self.ensure_one()
-        domain = safe_eval(self.domain)
-        if self.group_ids and not self.env.user.groups_id & self.group_ids:
+        domain_list = safe_eval(self.domain or "[]")
+        domain = Domain(domain_list)
+        if self.group_ids and not self.env.user.group_ids & self.group_ids:
             return False
-        if domain:
+        if domain_list:
             return bool(record.filtered_domain(domain))
         return True
 

ai_oca_bridge/models/ai_bridge_execution.py

@@ -7,9 +7,9 @@
 from io import StringIO
 
 import requests
-from werkzeug import urls
 
-from odoo import _, api, fields, models, tools
+from odoo import api, fields, models, tools
+from odoo.tools import urls
 
 
 class AiBridgeExecution(models.Model):
@@ -92,7 +92,7 @@ def _add_extra_payload_fields(self, payload):
                 seconds=self.ai_bridge_id.async_timeout
             )
             token = self._generate_token()
-            payload["_response_url"] = urls.url_join(
+            payload["_response_url"] = urls.urljoin(
                 self.get_base_url(), f"/ai/response/{self.id}/{token}"
             )
         IrParamSudo = self.env["ir.config_parameter"].sudo()
@@ -163,7 +163,7 @@ def _get_auth(self):
                 self.ai_bridge_id.sudo().auth_password,
             )
         else:
-            raise ValueError(_("Unsupported authentication type."))
+            raise ValueError(self.env._("Unsupported authentication type."))
 
     def _get_headers(self):
         """Return headers for the request."""

ai_oca_bridge/models/base.py

@@ -5,6 +5,7 @@
 import logging
 
 from odoo import api, models
+from odoo.fields import Domain
 
 _logger = logging.getLogger(__name__)
 
@@ -46,7 +47,11 @@ def _execute_ai_bridges_for_records(self, records, usage, values=None):
         bridges = (
             self.env["ai.bridge"]
             .sudo()
-            .search([("model_id", "=", model_id), ("usage", "=", usage)])
+            .search(
+                Domain.AND(
+                    [Domain("model_id", "=", model_id), Domain("usage", "=", usage)]
+                )
+            )
         )
         for bridge in bridges:
             # If this is a write, and the bridge has trigger fields configured,
@@ -73,7 +78,12 @@ def _prepare_execution_ai_bridges_unlink(self, records):
 
         model_id = self.env["ir.model"]._get_id(records._name)
         bridges = self.env["ai.bridge"].search(
-            [("model_id", "=", model_id), ("usage", "=", "ai_thread_unlink")]
+            Domain.AND(
+                [
+                    Domain("model_id", "=", model_id),
+                    Domain("usage", "=", "ai_thread_unlink"),
+                ]
+            )
         )
 
         executions = self.env["ai.bridge.execution"]

ai_oca_bridge/models/mail_thread.py

@@ -4,6 +4,7 @@
 from lxml import etree
 
 from odoo import api, fields, models
+from odoo.fields import Domain
 from odoo.tools.misc import frozendict
 
 
@@ -21,10 +22,16 @@ def _compute_ai_bridge_info(self):
 
     def _get_ai_bridge_info(self):
         self.ensure_one()
-        model_id = self.env["ir.model"].sudo().search([("model", "=", self._name)]).id
+        model_id = (
+            self.env["ir.model"].sudo().search(Domain("model", "=", self._name)).id
+        )
         return (
             self.env["ai.bridge"]
-            .search([("model_id", "=", model_id), ("usage", "=", "thread")])
+            .search(
+                Domain.AND(
+                    [Domain("model_id", "=", model_id), Domain("usage", "=", "thread")]
+                )
+            )
             .filtered(lambda r: r._enabled_for(self))
         )
 

ai_oca_bridge/static/description/index.html

@@ -374,7 +374,7 @@ <h1>AI OCA Bridge</h1>
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! source digest: sha256:c702d03816331f686ec7f037a5bbef08e513f54645bfdb51d11b15bcd5db77f2
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/ai/tree/18.0/ai_oca_bridge"><img alt="OCA/ai" src="https://img.shields.io/badge/github-OCA%2Fai-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/ai-18-0/ai-18-0-ai_oca_bridge"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/ai&amp;target_branch=18.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/ai/tree/19.0/ai_oca_bridge"><img alt="OCA/ai" src="https://img.shields.io/badge/github-OCA%2Fai-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/ai-19-0/ai-19-0-ai_oca_bridge"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/ai&amp;target_branch=19.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module is used to create a bridge between Odoo and other AI systems
 like n8n.</p>
 <p><strong>Table of contents</strong></p>
@@ -512,7 +512,7 @@ <h2><a class="toc-backref" href="#toc-entry-13">Bug Tracker</a></h2>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/ai/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/ai/issues/new?body=module:%20ai_oca_bridge%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/ai/issues/new?body=module:%20ai_oca_bridge%0Aversion:%2019.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
@@ -550,7 +550,7 @@ <h3><a class="toc-backref" href="#toc-entry-17">Maintainers</a></h3>
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/ai/tree/18.0/ai_oca_bridge">OCA/ai</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/ai/tree/19.0/ai_oca_bridge">OCA/ai</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>

ai_oca_bridge/static/src/components/chatter/chatter.esm.js

@@ -5,9 +5,9 @@ patch(Chatter.prototype, {
     async onClickAiBridge(aiBridge) {
         let saved = true;
 
-        if (this.props.webRecord && this.props.webRecord.save) {
+        if (this.props.record && this.props.record.save) {
             try {
-                await this.props.webRecord.save();
+                await this.props.record.save();
             } catch (error) {
                 saved = false;
                 console.error("Error saving record:", error);
@@ -18,8 +18,8 @@ patch(Chatter.prototype, {
             return;
         }
 
-        const model = this.props.webRecord.resModel;
-        const id = this.props.webRecord.resId;
+        const model = this.props.record.resModel;
+        const id = this.props.record.resId;
 
         const result = await this.env.services.orm.call(
             "ai.bridge",

ai_oca_bridge/static/src/components/chatter_topbar/chatter_topbar.xml

@@ -8,7 +8,7 @@
         <xpath expr="//div/div/div/button" position="before">
             <ChatterAITopbar
                 record="this"
-                t-if="props.webRecord and props.webRecord.data.ai_bridge_info and props.webRecord.data.ai_bridge_info.length > 0"
+                t-if="props.record and props.record.data.ai_bridge_info and props.record.data.ai_bridge_info.length > 0"
             />
         </xpath>
     </t>