Skip to content

Improve message when evals isn't installed#687

Merged
EricGustin merged 2 commits into
mainfrom
ericgustin/err-msg
Dec 9, 2025
Merged

Improve message when evals isn't installed#687
EricGustin merged 2 commits into
mainfrom
ericgustin/err-msg

Conversation

@EricGustin

@EricGustin EricGustin commented Nov 14, 2025

Copy link
Copy Markdown
Member

New and improved error message


Note

Enhances dependency checks to display uv/pip install commands and updates evals command accordingly; bumps package version to 1.5.9.

  • CLI utils:
    • Update require_dependency to accept uv_install_command and pip_install_command and format error message with both install options.
  • Evals command (arcade_cli/main.py):
    • Update require_dependency calls for arcade_evals and arcade_tdk to provide uv/pip install commands.
  • Version:
    • Bump project.version in pyproject.toml from 1.5.8 to 1.5.9.

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

@EricGustin EricGustin requested a review from shubcodes November 14, 2025 18:49
@codecov

codecov Bot commented Nov 14, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
libs/arcade-cli/arcade_cli/utils.py 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@EricGustin EricGustin requested a review from evantahler December 3, 2025 01:00

@shubcodes shubcodes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good it was the uv tool install piece that had me hung up on this originally! thanks for updating

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on December 13

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.

command_name="evals",
install_command=r"pip install arcade-tdk",
uv_install_command=r"uv pip install arcade-tdk",
pip_install_command=r"pip install arcade-tdk",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Inconsistent uv install commands may fail in isolation

The install instructions use inconsistent uv commands: uv tool install for arcade-mcp[evals] and uv pip install for arcade-tdk. When uv tool install creates an isolated environment for the tool, packages installed via uv pip install go to a different environment. Users following both instructions would have arcade-tdk in their project environment rather than in the arcade-mcp tool's isolated environment, potentially causing the evals command to still fail when it needs to load evaluation code that imports from arcade_tdk.

Fix in Cursor Fix in Web

@EricGustin EricGustin merged commit 489e01e into main Dec 9, 2025
13 of 14 checks passed
@EricGustin EricGustin deleted the ericgustin/err-msg branch December 9, 2025 23:41
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.

3 participants