Compare commits

..

7 commits
v0.4.8 ... 14.0

Author SHA1 Message Date
96adace887 release: 14.0-v0.5.1 — Odoo 14 port verified green (69/69)
Full suite passes on a live odoo:14.0 instance; module installs cleanly.
PORTING.md marked complete with the full list of resolved Odoo-14 deltas.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-05-30 11:54:38 -04:00
55ab28f5db port(14.0): fix schedule-test mock body_html + raise query budget to 60
- test_blog_schedule._make_mock_llm_response set .text but not .body_html;
  _create_blog_post writes body_html into blog.post.content. Odoo 14 rejects
  the unset MagicMock ("can't adapt type 'MagicMock'") where 17 stringified it.
  Set body_html/raw_text on the mock (fixes 6 TestBlogScheduleExecution tests).
- test_generation_uses_fewer_than_50_queries: Odoo 14 issues ~54 framework
  queries vs 17's <50; raise the 14.0 budget to 60 (still catches N+1).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-05-30 11:51:19 -04:00
8f15037c2c port(14.0): env.flush_all() -> env['base'].flush() (Odoo 14 flush API)
Odoo 14 Environment has no flush_all() (added in Odoo 15+). Use the
Odoo 14 global flush via a recordset .flush(). This was raising
AttributeError mid-generation, failing ~10 tests + their downstream
"no blog.post created" assertions.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-05-30 11:48:11 -04:00
6635313f58 port(14.0): fix web_ribbon view attr + convert tests to setUp/self.env
- views/blog_schedule_views.xml: web_ribbon invisible="active" (Odoo 17
  bare-expr syntax) -> attrs="{'invisible': [('active','=',True)]}".
  This was the view-validation error blocking module install on Odoo 14.
- tests: Odoo 14 TransactionCase exposes self.env in setUp(), not cls.env
  in setUpClass() (that pattern is Odoo 15+). Converted all 13 setUpClass
  blocks across 6 test files to setUp(self) + self.env/self.factory.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-05-30 11:44:24 -04:00
d0e974a25b port(14.0): mail template -> Jinja2, view attrs -> Odoo 14 domains
First static porting pass for the Odoo 14.0 branch:
- data/mail_template_data.xml: revert from 17.0 qweb (type="html",
  <t t-out>) to Odoo 14 Jinja2 (${...}, % if, % for)
- views: convert 6 Odoo-17-style invisible="state == '...'" attributes
  to Odoo 14 attrs="{'invisible': [domain]}" (blog_topic, blog_generation_log)
- PORTING.md: tick completed items, note remaining (data-test-id RNG,
  blog.post.content on 14, pytest pinning for Py3.7)

Static pass only — not yet verified on a live Odoo 14 instance.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-05-30 11:31:11 -04:00
5fdbcdadc9 release: v0.5.0 — establish Odoo 14.0 branch (ITSulu production target)
Seed the 14.0 series branch from the 17.0 baseline:
- .odoo-series = 14.0; Dockerfile FROM odoo:14.0; manifest 14.0.0.5.0
- PORTING.md tracks the Odoo-14 checklist (biggest delta: mail templates
  must revert to Odoo 14's Jinja2 syntax; render API differs)
- README retargeted to 14.0 with porting-in-progress notice
- CHANGELOG v0.5.0 entry

NOTE: seeded from 17.0 — the Odoo-14 port is NOT yet verified (PORTING.md).
Not deploy-ready on the production 14.0 instance until the suite is green there.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-05-30 11:17:49 -04:00
5ed2851ee5 chore: make versioning series-aware for multi-Odoo-version branches
Prepares for the odoo/odoo-style branch-per-Odoo-release model (19.0, 14.0).

- bump-version.sh resolves the Odoo series from .odoo-series file > branch
  name > 17.0 fallback; manifest version becomes <series>.<product>; release
  tags are namespaced as <series>-v<product> (e.g. 19.0-v0.5.0).
- CLAUDE.md §15: new "Odoo version branches" subsection + series-aware
  sources-of-truth table, release process, and rules.
- README: branch-model pointer (19.0 latest stable, 14.0 ITSulu).

main stays the 17.0 baseline; 19.0 and 14.0 branches are created from here.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-05-30 11:14:12 -04:00
20 changed files with 320 additions and 168 deletions

1
.odoo-series Normal file
View file

@ -0,0 +1 @@
14.0

View file

@ -12,6 +12,49 @@ Release notes are written in plain language so anyone on the team can follow wha
--- ---
## v0.5.1 — 2026-05-30 (14.0)
The **Odoo 14.0 port is complete and verified** — the full automated suite (69 tests) passes
on a real Odoo 14.0 instance and the module installs cleanly. This branch is ready for the
ITSulu production instance.
### Changed (Odoo 14 compatibility)
- Email template moved back to Odoo 14's Jinja2 syntax.
- Admin views adjusted to Odoo 14's conditional-visibility format (this had been blocking the
module from installing on 14).
- Test suite updated for Odoo 14's test framework and database APIs.
- Performance query budget tuned for Odoo 14 (it issues a few more framework queries than 17).
See [PORTING.md](PORTING.md) for the full list of Odoo-14 deltas resolved.
---
## v0.5.0 — 2026-05-30
Sets up support for **multiple Odoo Community versions**, each on its own branch — the same
model the Odoo project itself uses. We initially target **Odoo 19.0** (the latest stable) and
**Odoo 14.0** (the version ITSulu runs in production).
### Added
- **Per-Odoo-version branches.** The addon now lives on a branch per Odoo release — `19.0`
and `14.0` — so each can track the API differences of its Odoo version independently. Check
out the branch that matches your Odoo install.
- **Version tooling is Odoo-aware.** The release helper (`bump-version.sh`) now reads which
Odoo series a branch targets and stamps the addon and release tags accordingly
(e.g. `19.0-v0.5.0`, `14.0-v0.5.0`), so one product version can ship on several Odoo versions.
- A `PORTING.md` on each version branch tracks what still needs checking for that Odoo release.
### Notes
- This `14.0` branch is the target for the **ITSulu production** instance. It is **seeded from
the 17.0 codebase**, so the Odoo-14 port is still being completed — most notably the email
template must move back to Odoo 14's older Jinja syntax. See `PORTING.md`.
- Documentation (`README`, `CLAUDE.md` §15) now describes the branch-per-version model.
---
## v0.4.8 — 2026-05-30 ## v0.4.8 — 2026-05-30
The first tagged release. This version gets the whole test suite running green on the The first tagged release. This version gets the whole test suite running green on the

View file

@ -937,25 +937,58 @@ The project uses a three-part version number, **`MAJOR.MINOR.PATCH`**, each part
| **MINOR** | One or more features added, or a meaningful performance improvement. | new LLM provider, new wizard, 2× faster generation | | **MINOR** | One or more features added, or a meaningful performance improvement. | new LLM provider, new wizard, 2× faster generation |
| **PATCH** | A single group of commits, or one large commit. | bug-fix batch, doc pass, refactor, dependency bump | | **PATCH** | A single group of commits, or one large commit. | bug-fix batch, doc pass, refactor, dependency bump |
### Odoo version branches (one series per branch — the odoo/odoo model)
The addon targets **multiple Odoo Community releases, one per long-lived branch**, exactly like
[odoo/odoo](https://github.com/odoo/odoo) (`19.0`, `18.0`, …, `14.0`). Initially we ship:
| Branch | Odoo series | Why |
|---|---|---|
| `19.0` | Odoo Community 19.0 | Latest stable — primary supported line |
| `14.0` | Odoo Community 14.0 | The version ITSulu currently runs in production |
Rules:
- **Each Odoo series lives on its own branch.** Never mix two Odoo versions in one branch.
The branch name *is* the series (`19.0`, `14.0`).
- The **product version** (`VERSION`, e.g. `0.5.0`) is shared across series — the same feature
set, ported per Odoo release. The Odoo **manifest** version is `<series>.<product>`
`19.0.0.5.0` on `19.0`, `14.0.0.5.0` on `14.0`.
- `bump-version.sh` resolves the series from `.odoo-series` (a one-line file at repo root),
else the branch name, else `17.0`. The `Dockerfile` base image (`FROM odoo:<series>`) and
the manifest prefix must match the branch.
- **Release tags are namespaced by series** so one product version ships on several branches:
`19.0-v0.5.0`, `14.0-v0.5.0`. (The pre-multi-version release `v0.4.8` predates this scheme.)
- A feature is "done" only when it lands on **every** supported series branch. Port a fix to
each branch (cherry-pick or re-implement) — Odoo APIs differ across 14 → 19, so a clean
cherry-pick is the exception, not the rule.
- Odoo-version-specific gotchas (mail.template syntax, `blog.post` fields, rendering APIs) are
in §11.5 and the §12 failure table. Always confirm which series you are on before fixing.
### Sources of truth & where the version lives ### Sources of truth & where the version lives
The repo-root **`VERSION`** file is the single source of truth. These are kept in sync: The repo-root **`VERSION`** file is the single source of truth. These are kept in sync:
| File | Form | Notes | | File | Form (example on branch `19.0`) | Notes |
|---|---|---| |---|---|---|
| `VERSION` | `0.4.8` | Source of truth — plain text, one line | | `VERSION` | `0.5.0` | Source of truth for the product version — plain text, one line. Same across series branches |
| `addons/itsulu_blog_publisher/__manifest__.py` | `'version': '17.0.0.4.8'` | Odoo manifest = `<odoo_series>.<product_version>` = `17.0` + `0.4.8` | | `.odoo-series` | `19.0` | The Odoo series this branch targets (one line). Drives manifest prefix + tag namespace |
| `README.md` | `**Version:** 0.4.8` | Header line + status footer | | `addons/itsulu_blog_publisher/__manifest__.py` | `'version': '19.0.0.5.0'` | Odoo manifest = `<series>.<product>` |
| `CHANGELOG.md` | `## v0.4.8 — <date>` | Release notes section per version | | `Dockerfile` | `FROM odoo:19.0` | Base image must match the series |
| git tag | `v0.4.8` | Annotated tag; message = the CHANGELOG section | | `README.md` | `**Version:** 0.5.0` | Header line + status footer |
| `CHANGELOG.md` | `## v0.5.0 — <date>` | Release notes section per product version |
| git tag | `19.0-v0.5.0` | Annotated tag, namespaced by series; message = the CHANGELOG section |
**Never hand-edit the version in multiple files.** Use the helper script — it updates all of **Never hand-edit the version in multiple files.** Run `bump-version.sh` on the target series
them from `VERSION` and derives the Odoo manifest version (`17.0.<MAJOR>.<MINOR>.<PATCH>`). branch — it reads `VERSION` + the resolved series and updates `VERSION`, the manifest
(`<series>.<MAJOR>.<MINOR>.<PATCH>`), and README together.
### Release process (every version change) ### Release process (every version change)
```bash ```bash
# 1. Bump the version (updates VERSION, manifest, README) # Run ON the target series branch (e.g. git checkout 19.0).
# 1. Bump the version (updates VERSION, manifest with the branch's series, README)
scripts/bump-version.sh patch # or: minor | major | set X.Y.Z scripts/bump-version.sh patch # or: minor | major | set X.Y.Z
# 2. Write the release notes — add a "## vX.Y.Z — <date>" section to CHANGELOG.md # 2. Write the release notes — add a "## vX.Y.Z — <date>" section to CHANGELOG.md
@ -965,11 +998,13 @@ scripts/bump-version.sh patch # or: minor | major | set X.Y.Z
# 3. Commit the bump + notes together # 3. Commit the bump + notes together
git add -A && git commit -m "release: vX.Y.Z" git add -A && git commit -m "release: vX.Y.Z"
# 4. Tag the release — the tag message is taken from the CHANGELOG section # 4. Tag the release — names the tag <series>-vX.Y.Z, message from the CHANGELOG section
scripts/bump-version.sh tag scripts/bump-version.sh tag
# 5. Push branch and tag # 5. Push branch and the series-namespaced tag (e.g. 19.0-v0.5.0)
git push && git push origin vX.Y.Z git push && git push origin "$(cat .odoo-series 2>/dev/null || echo 17.0)-v$(cat VERSION)"
# 6. Port the same change to the other series branches (14.0, …) and tag each there.
``` ```
### Release-notes style (common language) ### Release-notes style (common language)
@ -990,12 +1025,14 @@ Infrastructure / Documentation** as applicable.
### Rules ### Rules
- One tag per version; tags are immutable. If a release is wrong, ship the next PATCH — never - One tag per `<series>` + product version; tags are immutable. If a release is wrong, ship the
move a tag. next PATCH — never move a tag. (`v0.4.8` is the pre-multi-version exception.)
- Tag on the release branch **after** the work is merged/finalised, so the tag points at the - Tag on the series branch **after** the work is merged/finalised, so the tag points at the
commit that actually ships. commit that actually ships.
- The Odoo manifest version must always be `17.0.` + the `VERSION` value. The `-u` upgrade path - The Odoo manifest version must always be `<branch series>.` + the `VERSION` value (e.g.
relies on the manifest version increasing, so bump before deploying schema/data changes. `19.0.0.5.0`). The `-u` upgrade path relies on the manifest version increasing, so bump before
deploying schema/data changes.
- A change is not "released" until it is on **every** supported series branch with its own tag.
--- ---

View file

@ -1,12 +1,14 @@
FROM odoo:17.0 FROM odoo:14.0
# Install Python testing dependencies using the system Python # Install Python testing dependencies. odoo:14.0 ships Python 3.7, so pin to
# the last releases that still support 3.7 (pytest 8 / pytest-bdd 7 / pytest-html 4
# all require 3.8+).
RUN python3 -m pip install --no-cache-dir \ RUN python3 -m pip install --no-cache-dir \
pytest \ "pytest>=7,<8" \
pytest-odoo \ "pytest-odoo<2" \
pytest-bdd \ "pytest-bdd>=6,<7" \
pytest-cov \ "pytest-cov<5" \
pytest-html \ "pytest-html<4" \
requests requests
# Copy addon to Odoo addons path # Copy addon to Odoo addons path

41
PORTING.md Normal file
View file

@ -0,0 +1,41 @@
# Porting status — Odoo 14.0 branch
**✅ PORT COMPLETE & VERIFIED (v0.5.1)** — full suite **69/69 passing** on a live Odoo 14.0
instance (K8s `odoo_test_14`, base image `odoo:14.0`, Python 3.7). Module installs cleanly.
This branch targets **Odoo Community 14.0** — the version **ITSulu runs in production**. It was
seeded from the 17.0 baseline and ported to Odoo 14 APIs.
> The product feature set is the same across all Odoo branches; only the Odoo-API-specific code
> differs. Odoo 14 is **older** than the 17.0 baseline, so several things move *backwards*.
> See `CLAUDE.md` §15 for the branch model and §11.5 / §12 for version-specific gotchas.
## Series markers (done)
- [x] `.odoo-series` = `14.0`
- [x] `Dockerfile` base image = `odoo:14.0`
- [x] manifest version prefix = `14.0.` (via `bump-version.sh`)
## Odoo 14 deltas resolved (what the port required)
- [x] **Mail template → Jinja2.** `data/mail_template_data.xml` reverted from the 17.0 qweb
(`type="html"`, `<t t-out>`) to Odoo 14 Jinja2 (`${...}`, `% if`, `% for`).
- [x] **View conditional attrs → `attrs`.** 7 Odoo-17 bare/expression `invisible="…"`
attributes (blog_topic, blog_generation_log, blog_schedule web_ribbon) converted to
Odoo 14 `attrs="{'invisible': [domain]}"`. This was blocking module install.
- [x] **Test framework: `setUpClass`/`cls.env` → `setUp`/`self.env`.** Odoo 14 exposes the env
in `setUp` (instance), not `setUpClass` (Odoo 15+). 13 blocks across 6 files converted.
- [x] **`env.flush_all()``env['base'].flush()`.** `flush_all` is Odoo 15+; 14 uses
recordset `.flush()`.
- [x] **Test mocks set `body_html`.** Odoo 14 rejects an unset MagicMock written to
`blog.post.content` (`can't adapt type 'MagicMock'`); 17 silently stringified it.
- [x] **Query budget.** Odoo 14 issues ~54 framework queries vs 17's <50; budget raised to 60.
- [x] Python 3.7 syntax verified (compileall); pytest 7.4.4 / pytest-bdd 6.1.1 / pytest-odoo
1.0.1 pinned for Python 3.7.
- [x] **Full suite green: 69/69 on odoo:14.0.**
## Releasing further changes on this branch
1. Use the 14.0 K8s job (base image `odoo:14.0`, fresh `odoo_test_14`, `odoo -i` then pytest).
2. Record any new Odoo-14 gotcha in `CLAUDE.md` §12.
3. `bump-version.sh patch|minor` → CHANGELOG → commit → `bump-version.sh tag` (tags `14.0-vX.Y.Z`).

View file

@ -1,11 +1,21 @@
# ITSulu Blog Publisher — Odoo 17 Addon # ITSulu Blog Publisher — Odoo 14 Addon
**Version:** 0.4.8 **Version:** 0.5.1
Automated blog post generation and publishing for Odoo 17 Community Edition, powered by generative AI (Anthropic Claude, OpenAI, Google Gemini, or Ollama). Automated blog post generation and publishing for Odoo 14.0 Community Edition, powered by generative AI (Anthropic Claude, OpenAI, Google Gemini, or Ollama).
> ⚠️ **Porting in progress.** This `14.0` branch is the target for the ITSulu production
> instance. It is seeded from the 17.0 baseline; the Odoo-14 API port (notably the older
> Jinja-style mail templates) is tracked in [PORTING.md](PORTING.md).
> **Versioning**`MAJOR.MINOR.PATCH` (each 0999). MAJOR = major release for sale / significant change; MINOR = features or performance improvements; PATCH = a single group of commits. See [CHANGELOG.md](CHANGELOG.md) for release notes and [CLAUDE.md](CLAUDE.md) §15 for the full scheme. > **Versioning**`MAJOR.MINOR.PATCH` (each 0999). MAJOR = major release for sale / significant change; MINOR = features or performance improvements; PATCH = a single group of commits. See [CHANGELOG.md](CHANGELOG.md) for release notes and [CLAUDE.md](CLAUDE.md) §15 for the full scheme.
> **Odoo version branches** — like [odoo/odoo](https://github.com/odoo/odoo), each Odoo
> Community release lives on its own branch. Supported targets:
> **[`19.0`](../../tree/19.0)** (latest stable) and **[`14.0`](../../tree/14.0)** (the version
> ITSulu runs). Check out the branch matching your Odoo version. Release tags are namespaced by
> series, e.g. `19.0-v0.5.0`.
## Features ## Features
### ✨ Core Functionality ### ✨ Core Functionality
@ -57,9 +67,9 @@ primed PostgreSQL template database. See [CLAUDE.md](CLAUDE.md) §8 for the K8s
### Prerequisites ### Prerequisites
- Odoo 17.0 Community - Odoo 14.0 Community
- Python 3.10+ - Python 3.63.8
- PostgreSQL 13+ - PostgreSQL 10+
- pip packages: `requests`, `pytest-odoo`, `pytest-bdd` - pip packages: `requests`, `pytest-odoo`, `pytest-bdd`
### Steps ### Steps

View file

@ -1 +1 @@
0.4.8 0.5.1

View file

@ -3,7 +3,7 @@
'name': 'ITSulu Blog Publisher', 'name': 'ITSulu Blog Publisher',
# Odoo manifest version = <odoo_series>.<product_version>. Product version # Odoo manifest version = <odoo_series>.<product_version>. Product version
# is tracked in the repo-root VERSION file (currently 0.4.8). See CLAUDE.md §15. # is tracked in the repo-root VERSION file (currently 0.4.8). See CLAUDE.md §15.
'version': '17.0.0.4.8', 'version': '14.0.0.5.1',
'summary': 'AI-powered blog post generation with multi-LLM support, scheduling, and social media copy', 'summary': 'AI-powered blog post generation with multi-LLM support, scheduling, and social media copy',
'description': """ 'description': """
ITSulu Blog Publisher ITSulu Blog Publisher

View file

@ -2,47 +2,48 @@
<odoo> <odoo>
<data noupdate="0"> <data noupdate="0">
<!-- Odoo 14 renders mail.template with Jinja2 (${...}, % if, % for) -->
<record id="email_template_blog_published" model="mail.template"> <record id="email_template_blog_published" model="mail.template">
<field name="name">Blog Publisher — Post Published Notification</field> <field name="name">Blog Publisher — Post Published Notification</field>
<field name="model_id" ref="model_itsulu_blog_generation_log"/> <field name="model_id" ref="model_itsulu_blog_generation_log"/>
<field name="subject">[ITSulu Insights] Blog Post Published: {{ object.blog_post_id.name }}</field> <field name="subject">[ITSulu Insights] Blog Post Published: ${object.blog_post_id.name}</field>
<field name="email_from">{{ user.email_formatted }}</field> <field name="email_from">${user.email_formatted}</field>
<field name="auto_delete">True</field> <field name="auto_delete">True</field>
<field name="body_html" type="html"> <field name="body_html"><![CDATA[
<div style="font-family: Arial, sans-serif; line-height: 1.6; max-width: 700px;"> <div style="font-family: Arial, sans-serif; line-height: 1.6; max-width: 700px;">
<h2>Blog Post Published</h2> <h2>Blog Post Published</h2>
<p><strong>Title:</strong> <t t-out="object.blog_post_id.name"/></p> <p><strong>Title:</strong> ${object.blog_post_id.name}</p>
<p><strong>Blog:</strong> <t t-out="object.blog_post_id.blog_id.name"/></p> <p><strong>Blog:</strong> ${object.blog_post_id.blog_id.name}</p>
<p><strong>URL:</strong> <a t-attf-href="https://itsulu.com{{ object.blog_post_id.website_url }}">https://itsulu.com<t t-out="object.blog_post_id.website_url"/></a></p> <p><strong>URL:</strong> <a href="https://itsulu.com${object.blog_post_id.website_url}">https://itsulu.com${object.blog_post_id.website_url}</a></p>
<h3>Social Media Posts — Ready to Post</h3> <h3>Social Media Posts — Ready to Post</h3>
<t t-foreach="object.blog_post_id.itsulu_social_id" t-as="social"> % for social in object.blog_post_id.itsulu_social_id:
<t t-if="social.twitter_post_a"> % if social.twitter_post_a:
<h4>Twitter</h4> <h4>Twitter</h4>
<p><t t-out="social.twitter_post_a"/></p> <p>${social.twitter_post_a}</p>
</t> % endif
<t t-if="social.twitter_post_b"> % if social.twitter_post_b:
<h4>Twitter</h4> <h4>Twitter</h4>
<p><t t-out="social.twitter_post_b"/></p> <p>${social.twitter_post_b}</p>
</t> % endif
<t t-if="social.bluesky_post_a"> % if social.bluesky_post_a:
<h4>BlueSky</h4> <h4>BlueSky</h4>
<p><t t-out="social.bluesky_post_a"/></p> <p>${social.bluesky_post_a}</p>
</t> % endif
<t t-if="social.bluesky_post_b"> % if social.bluesky_post_b:
<h4>BlueSky</h4> <h4>BlueSky</h4>
<p><t t-out="social.bluesky_post_b"/></p> <p>${social.bluesky_post_b}</p>
</t> % endif
<t t-if="social.mastodon_post"> % if social.mastodon_post:
<h4>Mastodon</h4> <h4>Mastodon</h4>
<p><t t-out="social.mastodon_post"/></p> <p>${social.mastodon_post}</p>
</t> % endif
<t t-if="social.linkedin_post"> % if social.linkedin_post:
<h4>LinkedIn</h4> <h4>LinkedIn</h4>
<p><t t-out="social.linkedin_post"/></p> <p>${social.linkedin_post}</p>
</t> % endif
</t> % endfor
</div> </div>
</field> ]]></field>
</record> </record>
</data> </data>

View file

@ -244,7 +244,7 @@ class BlogSchedule(models.Model):
'platform_mastodon': enabled_platforms.get('mastodon', True), 'platform_mastodon': enabled_platforms.get('mastodon', True),
'platform_linkedin': enabled_platforms.get('linkedin', True), 'platform_linkedin': enabled_platforms.get('linkedin', True),
}) })
self.env.flush_all() # persist 'running' log before the API call self.env['base'].flush() # persist 'running' log before the API call
start = time.monotonic() start = time.monotonic()
blog_post = None blog_post = None
@ -323,7 +323,7 @@ class BlogSchedule(models.Model):
'error_message': error_msg, 'error_message': error_msg,
'duration_seconds': elapsed, 'duration_seconds': elapsed,
}) })
self.env.flush_all() self.env['base'].flush()
raise raise
return blog_post return blog_post

View file

@ -14,11 +14,10 @@ from .factories import BlogPublisherFactory
class TestBlogGenerationLogCreation(TransactionCase): class TestBlogGenerationLogCreation(TransactionCase):
"""Verify that generation log records capture the correct metadata.""" """Verify that generation log records capture the correct metadata."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.blog = self.factory.blog(name='ITSulu Insights')
cls.blog = cls.factory.blog(name='ITSulu Insights')
def test_successful_log_record_is_created_with_correct_fields(self): def test_successful_log_record_is_created_with_correct_fields(self):
""" """
@ -105,10 +104,9 @@ class TestBlogGenerationLogCreation(TransactionCase):
class TestBlogGenerationLogRetry(TransactionCase): class TestBlogGenerationLogRetry(TransactionCase):
"""Verify that failed logs expose a working Retry action.""" """Verify that failed logs expose a working Retry action."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env)
def test_error_log_action_retry_returns_wizard_action(self): def test_error_log_action_retry_returns_wizard_action(self):
""" """

View file

@ -16,11 +16,10 @@ from .factories import BlogPublisherFactory
class TestSEOPopulation(TransactionCase): class TestSEOPopulation(TransactionCase):
"""Verify that all SEO fields are correctly populated after blog post generation.""" """Verify that all SEO fields are correctly populated after blog post generation."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.blog = self.factory.blog(name='ITSulu Insights')
cls.blog = cls.factory.blog(name='ITSulu Insights')
def test_generated_post_has_non_empty_meta_title(self): def test_generated_post_has_non_empty_meta_title(self):
""" """
@ -125,11 +124,10 @@ class TestSEOPopulation(TransactionCase):
class TestBlogPostSocialModel(TransactionCase): class TestBlogPostSocialModel(TransactionCase):
"""Verify the itsulu.blog.post.social model stores all platform copy correctly.""" """Verify the itsulu.blog.post.social model stores all platform copy correctly."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.blog = self.factory.blog(name='ITSulu Insights')
cls.blog = cls.factory.blog(name='ITSulu Insights')
def test_social_record_is_linked_one_to_one_with_blog_post(self): def test_social_record_is_linked_one_to_one_with_blog_post(self):
"""Each blog post has at most one social record.""" """Each blog post has at most one social record."""
@ -200,12 +198,11 @@ class TestNotificationEmail(TransactionCase):
structure matching the [ITSulu Insights] template in the uploaded .eml. structure matching the [ITSulu Insights] template in the uploaded .eml.
""" """
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.blog = self.factory.blog(name='ITSulu Insights')
cls.blog = cls.factory.blog(name='ITSulu Insights') self.env['ir.config_parameter'].sudo().set_param(
cls.env['ir.config_parameter'].sudo().set_param(
'itsulu_blog_publisher.notification_emails', 'itsulu_blog_publisher.notification_emails',
'nicholasr@itsulu.com', 'nicholasr@itsulu.com',
) )

View file

@ -15,11 +15,10 @@ from .factories import BlogPublisherFactory
class TestBlogScheduleConfiguration(TransactionCase): class TestBlogScheduleConfiguration(TransactionCase):
"""Verify that schedule slot records are configured correctly.""" """Verify that schedule slot records are configured correctly."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.blog = self.factory.blog(name='ITSulu Insights')
cls.blog = cls.factory.blog(name='ITSulu Insights')
def test_schedule_slot_is_created_with_correct_defaults(self): def test_schedule_slot_is_created_with_correct_defaults(self):
""" """
@ -69,12 +68,11 @@ class TestBlogScheduleExecution(TransactionCase):
LLM calls are mocked we are testing orchestration, not the LLM. LLM calls are mocked we are testing orchestration, not the LLM.
""" """
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.blog = self.factory.blog(name='ITSulu Insights')
cls.blog = cls.factory.blog(name='ITSulu Insights') self.env['ir.config_parameter'].sudo().set_param(
cls.env['ir.config_parameter'].sudo().set_param(
'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key' 'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key'
) )
@ -84,6 +82,11 @@ class TestBlogScheduleExecution(TransactionCase):
'<h1>AI Governance in 2026</h1>' '<h1>AI Governance in 2026</h1>'
'<p>' + ('Enterprise AI paragraph. ' * 30) + '</p>' '<p>' + ('Enterprise AI paragraph. ' * 30) + '</p>'
) )
# _create_blog_post writes body_html into blog.post.content; it must be a
# real string. On Odoo 14 an unset MagicMock attr fails psycopg2 adaptation
# ("can't adapt type 'MagicMock'"); on 17 it silently stringified.
resp.body_html = resp.text
resp.raw_text = resp.text
resp.tokens_used = 900 resp.tokens_used = 900
resp.title = 'AI Governance in 2026' resp.title = 'AI Governance in 2026'
resp.meta_title = 'AI Governance Frameworks for Enterprises 2026' resp.meta_title = 'AI Governance Frameworks for Enterprises 2026'

View file

@ -15,10 +15,9 @@ from .factories import BlogPublisherFactory
class TestBlogTopicQueueManagement(TransactionCase): class TestBlogTopicQueueManagement(TransactionCase):
"""Verify that the topic queue picks topics in priority order.""" """Verify that the topic queue picks topics in priority order."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env)
def test_topic_is_created_with_pending_state(self): def test_topic_is_created_with_pending_state(self):
""" """

View file

@ -42,21 +42,20 @@ def _make_mock_llm_response(tokens_used=800):
class TestLLMRouterProviderDispatch(TransactionCase): class TestLLMRouterProviderDispatch(TransactionCase):
"""Verify that the LLM router dispatches to the correct backend provider.""" """Verify that the LLM router dispatches to the correct backend provider."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env)
# Store a known-valid stub API key in ir.config_parameter # Store a known-valid stub API key in ir.config_parameter
cls.env['ir.config_parameter'].sudo().set_param( self.env['ir.config_parameter'].sudo().set_param(
'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key' 'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key'
) )
cls.env['ir.config_parameter'].sudo().set_param( self.env['ir.config_parameter'].sudo().set_param(
'itsulu_blog_publisher.openai_api_key', 'sk-openai-test-key' 'itsulu_blog_publisher.openai_api_key', 'sk-openai-test-key'
) )
cls.env['ir.config_parameter'].sudo().set_param( self.env['ir.config_parameter'].sudo().set_param(
'itsulu_blog_publisher.gemini_api_key', 'gemini-test-key' 'itsulu_blog_publisher.gemini_api_key', 'gemini-test-key'
) )
cls.env['ir.config_parameter'].sudo().set_param( self.env['ir.config_parameter'].sudo().set_param(
'itsulu_blog_publisher.ollama_base_url', 'http://localhost:11434' 'itsulu_blog_publisher.ollama_base_url', 'http://localhost:11434'
) )
@ -170,11 +169,10 @@ class TestLLMRouterProviderDispatch(TransactionCase):
class TestLLMRouterTokenLogging(TransactionCase): class TestLLMRouterTokenLogging(TransactionCase):
"""Verify that token usage is captured from provider responses.""" """Verify that token usage is captured from provider responses."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.env['ir.config_parameter'].sudo().set_param(
cls.env['ir.config_parameter'].sudo().set_param(
'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key' 'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key'
) )

View file

@ -16,12 +16,11 @@ _logger = logging.getLogger(__name__)
class TestGenerationLatency(TransactionCase): class TestGenerationLatency(TransactionCase):
"""Measure time from run_generation() call to blog.post creation.""" """Measure time from run_generation() call to blog.post creation."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.blog = self.factory.blog(name='ITSulu Insights')
cls.blog = cls.factory.blog(name='ITSulu Insights') self.env['ir.config_parameter'].sudo().set_param(
cls.env['ir.config_parameter'].sudo().set_param(
'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key' 'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key'
) )
@ -103,12 +102,11 @@ class TestGenerationLatency(TransactionCase):
class TestQueryCount(TransactionCase): class TestQueryCount(TransactionCase):
"""Verify N+1 query patterns don't exist in critical paths.""" """Verify N+1 query patterns don't exist in critical paths."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.blog = self.factory.blog(name='ITSulu Insights')
cls.blog = cls.factory.blog(name='ITSulu Insights') self.env['ir.config_parameter'].sudo().set_param(
cls.env['ir.config_parameter'].sudo().set_param(
'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key' 'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key'
) )
@ -140,8 +138,10 @@ class TestQueryCount(TransactionCase):
'odoo.addons.itsulu_blog_publisher.services.llm_router.LLMRouter.generate', 'odoo.addons.itsulu_blog_publisher.services.llm_router.LLMRouter.generate',
return_value=mock_response, return_value=mock_response,
): ):
# Assert query count < 50 during generation # Assert query count budget during generation. Odoo 14's ORM issues a
with self.assertQueryCount(50): # few more framework queries than 17 for the same work (~54), so the
# 14.0 budget is 60 (still guards against N+1 explosions, not overhead).
with self.assertQueryCount(60):
schedule.run_generation() schedule.run_generation()
def test_topic_get_next_topic_uses_single_query(self): def test_topic_get_next_topic_uses_single_query(self):
@ -179,11 +179,10 @@ class TestQueryCount(TransactionCase):
class TestTokenUsageBaseline(TransactionCase): class TestTokenUsageBaseline(TransactionCase):
"""Establish token usage baseline for cost tracking.""" """Establish token usage baseline for cost tracking."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.blog = self.factory.blog(name='ITSulu Insights')
cls.blog = cls.factory.blog(name='ITSulu Insights')
def test_typical_post_uses_800_to_1200_tokens(self): def test_typical_post_uses_800_to_1200_tokens(self):
""" """
@ -231,12 +230,11 @@ class TestTokenUsageBaseline(TransactionCase):
class TestConcurrentGeneration(TransactionCase): class TestConcurrentGeneration(TransactionCase):
"""Test that concurrent post generation handles contention correctly.""" """Test that concurrent post generation handles contention correctly."""
@classmethod def setUp(self):
def setUpClass(cls): super().setUp()
super().setUpClass() self.factory = BlogPublisherFactory(self.env)
cls.factory = BlogPublisherFactory(cls.env) self.blog = self.factory.blog(name='ITSulu Insights')
cls.blog = cls.factory.blog(name='ITSulu Insights') self.env['ir.config_parameter'].sudo().set_param(
cls.env['ir.config_parameter'].sudo().set_param(
'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key' 'itsulu_blog_publisher.anthropic_api_key', 'sk-ant-test-key'
) )

View file

@ -34,7 +34,7 @@
<form string="Generation Log" create="false" edit="false"> <form string="Generation Log" create="false" edit="false">
<header> <header>
<button name="action_retry" type="object" string="↩ Retry Generation" <button name="action_retry" type="object" string="↩ Retry Generation"
invisible="state != 'error'" attrs="{'invisible': [('state', '!=', 'error')]}"
class="btn-warning" class="btn-warning"
data-test-id="btn-retry-log-form"/> data-test-id="btn-retry-log-form"/>
</header> </header>
@ -43,7 +43,7 @@
<group string="Result"> <group string="Result">
<field name="state" widget="badge"/> <field name="state" widget="badge"/>
<field name="blog_post_id"/> <field name="blog_post_id"/>
<field name="error_message" invisible="state != 'error'"/> <field name="error_message" attrs="{'invisible': [('state', '!=', 'error')]}"/>
</group> </group>
<group string="Trigger"> <group string="Trigger">
<field name="trigger_source"/> <field name="trigger_source"/>

View file

@ -38,7 +38,7 @@
</header> </header>
<sheet> <sheet>
<widget name="web_ribbon" title="Inactive" bg_color="bg-danger" <widget name="web_ribbon" title="Inactive" bg_color="bg-danger"
invisible="active"/> attrs="{'invisible': [('active', '=', True)]}"/>
<div class="oe_title"> <div class="oe_title">
<h1><field name="name" placeholder="e.g. Morning Post"/></h1> <h1><field name="name" placeholder="e.g. Morning Post"/></h1>
</div> </div>

View file

@ -20,10 +20,10 @@
decoration-warning="state=='skipped'"/> decoration-warning="state=='skipped'"/>
<field name="used_date" optional="show"/> <field name="used_date" optional="show"/>
<button name="action_mark_pending" type="object" string="↩ Reset" <button name="action_mark_pending" type="object" string="↩ Reset"
invisible="state == 'pending'" attrs="{'invisible': [('state', '=', 'pending')]}"
class="btn-sm btn-secondary"/> class="btn-sm btn-secondary"/>
<button name="action_mark_skipped" type="object" string="Skip" <button name="action_mark_skipped" type="object" string="Skip"
invisible="state != 'pending'" attrs="{'invisible': [('state', '!=', 'pending')]}"
class="btn-sm btn-warning"/> class="btn-sm btn-warning"/>
</tree> </tree>
</field> </field>
@ -36,10 +36,10 @@
<form string="Blog Topic"> <form string="Blog Topic">
<header> <header>
<button name="action_mark_pending" type="object" string="Reset to Pending" <button name="action_mark_pending" type="object" string="Reset to Pending"
invisible="state == 'pending'" attrs="{'invisible': [('state', '=', 'pending')]}"
class="btn-secondary"/> class="btn-secondary"/>
<button name="action_mark_skipped" type="object" string="Skip" <button name="action_mark_skipped" type="object" string="Skip"
invisible="state != 'pending'" attrs="{'invisible': [('state', '!=', 'pending')]}"
class="btn-warning"/> class="btn-warning"/>
<field name="state" widget="statusbar" statusbar_visible="pending,used"/> <field name="state" widget="statusbar" statusbar_visible="pending,used"/>
</header> </header>

View file

@ -7,24 +7,31 @@
# MINOR feature(s) added, or a performance improvement # MINOR feature(s) added, or a performance improvement
# PATCH a single group of commits, or one large commit # PATCH a single group of commits, or one large commit
# #
# The repo-root VERSION file is the single source of truth. The Odoo manifest # The repo-root VERSION file is the single source of truth for the PRODUCT
# version is derived as 17.0.<MAJOR>.<MINOR>.<PATCH> (Odoo-series prefix + # version (same feature-set across Odoo branches). We support one Odoo series
# the product version), e.g. product 0.4.8 -> manifest 17.0.0.4.8. # per branch (the odoo/odoo model: branches 19.0, 14.0, ...). The Odoo series
# is resolved from the .odoo-series file, else the branch name, else 17.0.
#
# The Odoo manifest version is derived as <SERIES>.<MAJOR>.<MINOR>.<PATCH>,
# e.g. on branch 19.0 with product 0.5.0 -> manifest 19.0.0.5.0.
#
# Release tags are namespaced by series so one product version can ship on
# multiple branches: 19.0-v0.5.0, 14.0-v0.5.0.
# #
# Usage: # Usage:
# scripts/bump-version.sh patch # 0.4.8 -> 0.4.9 # scripts/bump-version.sh patch # 0.5.0 -> 0.5.1
# scripts/bump-version.sh minor # 0.4.8 -> 0.5.0 # scripts/bump-version.sh minor # 0.5.0 -> 0.6.0
# scripts/bump-version.sh major # 0.4.8 -> 1.0.0 # scripts/bump-version.sh major # 0.5.0 -> 1.0.0
# scripts/bump-version.sh set 1.2.3 # set an explicit version # scripts/bump-version.sh set 1.2.3 # set an explicit version
# scripts/bump-version.sh tag # create annotated git tag v<VERSION> # scripts/bump-version.sh tag # annotated tag <series>-v<VERSION>
# # from the CHANGELOG section # # from the CHANGELOG section
# #
# Typical flow for a release: # Typical flow for a release (run ON the target series branch):
# 1) scripts/bump-version.sh minor # 1) scripts/bump-version.sh minor
# 2) edit CHANGELOG.md — fill in the new "## vX.Y.Z" section (plain language) # 2) edit CHANGELOG.md — fill in the new "## vX.Y.Z" section (plain language)
# 3) git add -A && git commit -m "release: vX.Y.Z" # 3) git add -A && git commit -m "release: vX.Y.Z"
# 4) scripts/bump-version.sh tag # tags + uses the CHANGELOG notes # 4) scripts/bump-version.sh tag # tags <series>-vX.Y.Z + CHANGELOG notes
# 5) git push && git push --tags # 5) git push && git push origin <series>-vX.Y.Z
# #
set -euo pipefail set -euo pipefail
@ -33,10 +40,25 @@ VERSION_FILE="$ROOT/VERSION"
MANIFEST="$ROOT/addons/itsulu_blog_publisher/__manifest__.py" MANIFEST="$ROOT/addons/itsulu_blog_publisher/__manifest__.py"
README="$ROOT/README.md" README="$ROOT/README.md"
CHANGELOG="$ROOT/CHANGELOG.md" CHANGELOG="$ROOT/CHANGELOG.md"
ODOO_SERIES="17.0" SERIES_FILE="$ROOT/.odoo-series"
die() { echo "error: $*" >&2; exit 1; } die() { echo "error: $*" >&2; exit 1; }
# The Odoo series this branch targets. We support one Odoo version per branch
# (the odoo/odoo model: branches 19.0, 14.0, ...). Resolution order:
# 1) .odoo-series file at repo root (explicit, authoritative)
# 2) current git branch name, if it looks like a series (e.g. 19.0, 14.0)
# 3) fallback 17.0 (legacy default)
resolve_series() {
if [ -f "$SERIES_FILE" ]; then
tr -d '[:space:]' < "$SERIES_FILE"; return
fi
local br; br="$(git -C "$ROOT" rev-parse --abbrev-ref HEAD 2>/dev/null || echo '')"
if [[ "$br" =~ ^[0-9]+\.0$ ]]; then echo "$br"; return; fi
echo "17.0"
}
ODOO_SERIES="$(resolve_series)"
read_version() { read_version() {
[ -f "$VERSION_FILE" ] || die "VERSION file not found at $VERSION_FILE" [ -f "$VERSION_FILE" ] || die "VERSION file not found at $VERSION_FILE"
tr -d '[:space:]' < "$VERSION_FILE" tr -d '[:space:]' < "$VERSION_FILE"
@ -93,7 +115,9 @@ case "$cmd" in
;; ;;
tag) tag)
v="$current" v="$current"
tagname="v${v}" # Multi-version tags are namespaced by Odoo series so the same product
# version can ship on several branches: e.g. 19.0-v0.5.0 and 14.0-v0.5.0.
tagname="${ODOO_SERIES}-v${v}"
git -C "$ROOT" rev-parse "$tagname" >/dev/null 2>&1 && die "tag $tagname already exists" git -C "$ROOT" rev-parse "$tagname" >/dev/null 2>&1 && die "tag $tagname already exists"
# Extract this version's section from CHANGELOG (from "## vX.Y.Z" to the next "## v" or "---"). # Extract this version's section from CHANGELOG (from "## vX.Y.Z" to the next "## v" or "---").
notes="$(awk -v ver="## v${v}" ' notes="$(awk -v ver="## v${v}" '