Skip to content

Conversation

@mplatzer
Copy link
Contributor

@mplatzer mplatzer commented Nov 28, 2025

Note

Upgrades the engine to 2.3.1, adds createdAt to AssistantMessage, and disables the client CPU test job.

  • Dependencies:
    • Bump mostlyai-engine to 2.3.1 for both local and local-gpu extras in pyproject.toml.
    • Refresh uv.lock with numerous dependency updates (e.g., ray, transformers, accelerate, etc.).
  • SDK/API:
    • Extend AssistantMessage by adding created_at (alias createdAt) in mostlyai/sdk/domain.py.
  • CI:
    • Disable the run-test-cpu-client job in .github/workflows/run-tests-cpu.yaml (adds if: false).

Written by Cursor Bugbot for commit b73db8c. This will update automatically on new commits. Configure here.

@mplatzer mplatzer requested a review from a team as a code owner November 28, 2025 16:55
@mplatzer mplatzer requested a review from a team as a code owner November 28, 2025 17:45
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

uv run --no-sync pytest -vv tests/test_domain.py
run-test-cpu-client:
if: false
Copy link

Choose a reason for hiding this comment

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

Bug: CI test job disabled, may hide upgrade issues

The run-test-cpu-client job has been disabled by adding if: false. This completely skips the client mode end-to-end tests that validate API authentication and S3 integration. In a PR meant to upgrade the engine from 2.0.1 to 2.3.1, disabling these tests could mask breaking changes or regressions introduced by the new version. This appears to be debug code that was accidentally included in the commit.

Fix in Cursor Fix in Web

@mplatzer mplatzer merged commit f094d6c into main Nov 28, 2025
14 checks passed
@mplatzer mplatzer deleted the engine231 branch November 28, 2025 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants