Skip to content

[18.0] [MIG] ai_automation -> ai_server_action#81

Open
angelmoya wants to merge 3 commits into
OCA:18.0from
angelmoya:18.0-mig-ai_automation
Open

[18.0] [MIG] ai_automation -> ai_server_action#81
angelmoya wants to merge 3 commits into
OCA:18.0from
angelmoya:18.0-mig-ai_automation

Conversation

@angelmoya

@angelmoya angelmoya commented Jun 23, 2026

Copy link
Copy Markdown
Member

[MIG] ai_automation -> ai_server_action: migration to 18.0

  • Renamed module from ai_automation to ai_server_action
  • Removed ai_connection model (now in base ai_connection module)
  • Added output modes: post_message, update_record, store_variable, none
  • Added markdown rendering support for HTML fields
  • Added template variable support via Qweb
  • Rewrote tests using FakeModelLoader pattern (no real API calls)

Depends on #77

@angelmoya angelmoya mentioned this pull request Jun 23, 2026
@angelmoya
angelmoya force-pushed the 18.0-mig-ai_automation branch from 6cfe6d3 to 622b6a2 Compare June 23, 2026 10:04
@pedrobaeza

This comment was marked as resolved.

@angelmoya

This comment was marked as resolved.

@angelmoya

Copy link
Copy Markdown
Member Author

Additionally, regarding the commit order, it might be better to split it. We could first do the migration and adaptation work to version 18 relying on the connection, and then handle the name change in a separate, subsequent commit.

This way, the actual changes between versions would be much easier to track and review.

@angelmoya
angelmoya force-pushed the 18.0-mig-ai_automation branch 2 times, most recently from 29ffca5 to f5876eb Compare July 11, 2026 09:38
@angelmoya

Copy link
Copy Markdown
Member Author

@etobella @pedrobaeza I have updated the PR and split the work into three clean commits: migration, refactoring, and module renaming.

If we decide not to rename the module, I can easily revert to the previous commit. Ready for review.

@angelmoya
angelmoya force-pushed the 18.0-mig-ai_automation branch 3 times, most recently from 931c718 to 523e637 Compare July 11, 2026 10:17
@etobella

Copy link
Copy Markdown
Member

@angelmoya Rename is not handled this way. I requires to modify previous commits because, otherwise, on the next migration everything will be lost...

@etobella etobella left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, remake history and just leave a migration commit.

The code is something like:

git filter-branch -f --index-filter '
git ls-files --stage | \
sed "s:ai_automation/:ai_server_action/:" | \
GIT_INDEX_FILE=$GIT_INDEX_FILE.new \
git update-index --index-info && \
mv "$GIT_INDEX_FILE.new" "$GIT_INDEX_FILE"
' HEAD

Also, there is some comments to be attended

Comment thread ai_automation/__manifest__.py Outdated
Comment thread ai_automation/__manifest__.py Outdated
@angelmoya
angelmoya force-pushed the 18.0-mig-ai_automation branch 2 times, most recently from 89dbc9e to 2012e85 Compare July 13, 2026 11:03
@angelmoya
angelmoya force-pushed the 18.0-mig-ai_automation branch from 2012e85 to 71660cb Compare July 13, 2026 11:11
@angelmoya
angelmoya force-pushed the 18.0-mig-ai_automation branch from 71660cb to 69bcfb1 Compare July 20, 2026 07:38
@angelmoya
angelmoya requested a review from etobella July 20, 2026 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants