Skip to content

Conversation

@agustif
Copy link

@agustif agustif commented Jan 2, 2026

Summary

  • add --install-target {prime, project, system} to prime env install and prime eval (default: prime)
  • manage a Prime venv at ~/.prime/venvs/default, or a project .venv, or use uv pip install --system
  • ensure install, environment checks, and uv run all use the same uv context

Rationale

uv pip install requires a target environment. Defaulting to a managed venv prevents accidental system installs while keeping quick‑start smooth; users can opt into project or system installs explicitly.

Fixes PrimeIntellect-ai/prime-environments#455

Notes

  • --project lets users point to a specific project root for the .venv target

None,
"--project",
help="Project directory to use with --install-target project",
),
Copy link

Choose a reason for hiding this comment

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

Uninstall command ignores install-target and project options

The uninstall command adds install_target and project parameters (lines 1775-1787), but these are never used in the function body. The uninstall logic (lines 1809-1830) always runs uv pip uninstall in the current environment without considering the target. This means packages installed with --install-target prime or --install-target project cannot be properly uninstalled because the uninstall command will attempt to remove from the wrong environment.

Additional Locations (1)

Fix in Cursor Fix in Web


use_system, uv_env, uv_cwd, venv_path = _resolve_uv_install_context(
install_target, project
)
Copy link

Choose a reason for hiding this comment

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

Missing exception handling for install context resolution in run_eval

The run_eval function calls _resolve_uv_install_context without a try/except block, unlike the install command which catches ValueError and displays a friendly error message. When install_target is "project" but no pyproject.toml or uv.toml is found in the directory tree, _resolve_uv_install_context raises a ValueError that would propagate as an uncaught exception with an ugly stack trace, rather than the clean CLI error shown in install.

Fix in Cursor Fix in Web

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.

Fix environments with git dependencies not installing from the hub

1 participant