fix(ai): install AI test deps via depends() so engine constraints apply

[asclearuc]

Summary

• ai:test installed test requirements with plain pip. pip is never handed the engine's compiled constraints file, so unpinned deps (e.g. pillow) resolved to the latest version instead of the engine-pinned one.
• On Windows this broke test collection. pillow was pulled to a newer version, then depends() tried to downgrade it at import time and could not overwrite the already-loaded _imaging*.pyd (file locked). That left a half-written PIL package, so tests failed with ModuleNotFoundError: No module named 'PIL'.
• Install the test requirements through depends() instead of pip. It runs uv pip install with the constraints file, so every dep (and each nested -r include) resolves to its pinned version. pillow now stays at the pinned 10.4.0.

Type

fix (test/build tooling)

Testing

• Tests added or updated
• Tested locally
• ./builder test passes

The four previously-failing test files collect and pass (23 tests), and pillow resolves to the pinned 10.4.0. Full ./builder test not run.

Checklist

• Commit messages follow conventional commits
• No secrets or credentials included
• Wiki updated (if applicable)
• Breaking changes documented (if applicable)

Linked Issue

Fixes #

Summary by CodeRabbit

• Chores
  • Updated the test setup to install Python dependencies through the standard engine-managed flow, helping keep test environments consistent and avoid unexpected dependency upgrades.

[github-actions]

🤖 Internal: Discord sync marker

Auto-managed by the Discord notification workflow. Stores the linked Discord message ID. Do not edit or delete.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 7dec2c5e-f213-4a23-b87d-1bb438e3a0bc

📥 Commits

Reviewing files that changed from the base of the PR and between a430793 and 7bd741a.

📒 Files selected for processing (1)

• packages/ai/scripts/tasks.js

---

📝 Walkthrough

Walkthrough

The ai:run-pytest action in packages/ai/scripts/tasks.js now installs requirements by invoking the engine's depends() function via a Python snippet instead of calling pip install directly, with comments explaining the change ensures engine constraints apply and avoids unpinned dependency resolution.

Changes

Dependency Installation Change

Layer / File(s)	Summary
Engine-based dependency install for pytest task packages/ai/scripts/tasks.js	Replaced direct pip install -r requirements.txt with an engine-executed Python snippet calling depends(requirementsPath), adding comments on constraint behavior.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

• rocketride-org/rocketride-server#1466: Same file and flow modified to install AI test dependencies via engine's depends() instead of direct pip install.

Suggested labels: `ci/cd`

Suggested reviewers: `jmaionchi`, `Rod-Christensen`, `stepmikhaylov`

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly matches the main change: installing AI test dependencies through depends() so engine constraints are applied.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/ai-tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed: private package registry requires authentication. Disable ESLint in CodeRabbit settings or use public packages.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}