PR State:
merged
PR:
#27
Commit:
93f34dff9d5c0b80d2ba3c0483de783cdc3480ee
[FIX] viin_brain: P8 remove duplicate imports in v2 extension seams test
P7 added side-effect imports for ai_block_states and property_types
registries followed by named imports from the same modules. The named
imports already trigger module side-effects on first import, so the
side-effect imports are redundant and trip eslint no-duplicate-imports.
Verified registries call registry.add() at module top level; named
import alone is sufficient.
Fixes 2 NEW eslint errors discovered by P8 full-suite run.
All other P8 test failures (11 TestBrainAclPortalShare + 4
TestCollabStepController + tour tests + 1 TestPyLint W0012) confirmed
pre-existing on baseline 17.0 — NOT caused by this hardening pipeline.
Verified via dedicated baseline runs on ~/git/ai17 at branch 17.0
(see ops/logs/runs/p0-baseline-* + dropdb cleanup).
Phase: P8 convergence
Pipeline: 20260503-brain-ui-hardening-v1
Plan: ~/.claude/plans/parsed-spinning-peacock.md
Branch:
17.0
| Subbuilds | Name | State | Detail |
|---|---|---|---|
| Build 381430 | Check the quality with Pylint: from test_lint to test_pylint 14m | Failed | Log |
| Build 381431 | At install tests: from account to website_twitter 124m | Succeed | Log |
| Build 381432 | Post install tests: step install all modules 64m | Succeed | Log |
| Build 381433 | Post install tests: from account to payment_alipay 165m | Succeed | Log |
| Build 381434 | Post install tests: from payment_aps to test_viin_web_cohort 195m | Succeed | Log |
| Build 381435 | Post install tests: from test_website to to_website_apps_store 80m | Succeed | Log |
| Build 381436 | Post install tests: from to_website_apps_store_loc to viin_estimate_sale 89m | Succeed | Log |
| Build 381437 | Post install tests: from viin_event_checkin_website to viin_project_gantt 35m | Succeed | Log |
| Build 381438 | Post install tests: from viin_project_progress to website_twitter 44m | Succeed | Log |
| Build 381439 | Test install all modules without demo data: from account to website_twitter 49m | Succeed | Log |
| Create Date | Level | Message |
|---|---|---|
| 05/03/2026 15:55:23 | INFO |
Updated repository Viindoo-odoo
|
| 05/03/2026 15:55:23 | INFO |
Updated repository Viindoo-tvtmaaddons
|
| 05/03/2026 15:55:23 | INFO |
Updated repository Viindoo-erponline-enterprise
|
| 05/03/2026 15:55:23 | INFO |
Updated repository Viindoo-branding
|
| 05/03/2026 15:55:23 | INFO |
Cloned repository Viindoo-ai
|
| 05/03/2026 16:10:34 | ERROR |
Subbuild # 381430:
odoo.addons.test_pylint.tests.test_eslint: FAIL: TestESLint.test_eslint
Traceback (most recent call last):
File "/data/build/Viindoo-tvtmaaddons-17.0/test_pylint/tests/test_eslint.py", line 67, in test_eslint
self.assertEqual(process.returncode, 0, msg=f"""
AssertionError: 1 != 0 :
stdout:
/data/build/rb-93f34df-221694/viin_brain/static/tests/seams/v2_extension_seams_tests.js
34:19 error Replace `⏎············mount.classList.contains("d-none"),⏎············"canvas·mount·is·hidden·by·default·(d-none)"⏎········` with `mount.classList.contains("d-none"),·"canvas·mount·is·hidden·by·default·(d-none)"` prettier/prettier
41:13 error Replace `"canvas·mount·declares·t-ref=\"canvasMount\""` with `'canvas·mount·declares·t-ref="canvasMount"'` prettier/prettier
56:16 error Replace `"Seam·4:·ShareDialog·template·has·empty·<t·t-slot=\"collabSection\"/>"` with `'Seam·4:·ShareDialog·template·has·empty·<t·t-slot="collabSection"/>'` prettier/prettier
60:45 error Replace `"exactly·one·<t·t-slot=\"collabSection\"/>·declared"` with `'exactly·one·<t·t-slot="collabSection"/>·declared'` prettier/prettier
83:16 error Replace `"Seam·6:·AI·block·HTML·output·carries·additive·o_brain_ai_state_<status>·class",` with `⏎········"Seam·6:·AI·block·HTML·output·carries·additive·o_brain_ai_state_<status>·class",⏎·······` prettier/prettier
84:1 error Replace `········` with `············` prettier/prettier
85:9 error Insert `····` prettier/prettier
86:1 error Replace `········` with `············` prettier/prettier
87:9 error Insert `····` prettier/prettier
88:1 error Insert `····` prettier/prettier
89:9 error Insert `····` prettier/prettier
90:1 error Insert `····` prettier/prettier
91:1 error Replace `········` with `············` prettier/prettier
92:1 error Insert `····` prettier/prettier
93:1 error Replace `············` with `················` prettier/prettier
94:9 error Insert `····` prettier/prettier
95:1 error Replace `········` with `············` prettier/prettier
96:9 error Insert `····` prettier/prettier
97:1 error Insert `····` prettier/prettier
98:13 error Insert `····` prettier/prettier
99:9 error Insert `····` prettier/prettier
100:1 error Replace `····}` with `········}⏎····` prettier/prettier
✖ 22 problems (22 errors, 0 warnings)
22 errors and 0 warnings potentially fixable with the `--fix` option.
Perhaps you might benefit from installing the tooling found at:
https://github.com/odoo/odoo/wiki/Javascript-coding-guidelines#use-a-linter
stderr:
|
| 05/03/2026 16:10:34 | ERROR |
Subbuild # 381430:
odoo.modules.loading: Module test_pylint: 1 failures, 0 errors of 7 tests
|
| 05/03/2026 16:10:34 | ERROR |
Subbuild # 381430:
odoo.modules.loading: At least one test failed when loading the modules.
|
| 05/03/2026 16:10:34 | ERROR |
Subbuild # 381430:
odoo.tests.result: 1 failed, 0 error(s) of 24 tests when loading database 'rb-93f34df-221694-381430'
|