[clarka34]

PR summary

Closes #355

ewoks install with pixi always installed the default environment of the workspace, even when the requirements were gathered in another pixi environment. The requirements now record the name of the pixi environment, and the install uses it.

Recording the pixi environment name

• A new field, pixi_environment_name, was added to the pixi manager.
• _gather_extra_fields() was added to BaseManager: each package manager can add its own fields to its manager info if needed.

Installing the pixi environment

• files_environment() was added to BaseManager: each package manager can set the environment in which the files of the requirements are installed if needed. Ex: Pixi returns <workspace>/.pixi/envs/<pixi_environment_name> instead of <workspace>/.pixi/envs/default
• pixi install and pixi add now pass --environment.
• When installing the pixi files fails, the fallback still installs the python distributions in default.

Tests

• Tests added for gathering the pixi environment name and for selecting the environment to install.

AI Disclosure

• No AI used
• Claude used for reviewing and tests

[clarka34]

Adding this to put files at the end of the manager section so it's more readable.

[clarka34]

Allows pixi to override this and install into the environment named in the requirements instead of default.

[clarka34]

Allows pixi to override this and add an extra field, pixi_environment_name, to record the environment name.

[clarka34]

The CI issue will be fixed in: #357

[codecov]

Codecov Report

❌ Patch coverage is 86.44068% with 8 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/ewoks/_requirements/utils/base_manager.py	72.22%	5 Missing ⚠️
src/ewoks/_requirements/pixi.py	87.50%	2 Missing ⚠️
src/ewoks/_requirements/__init__.py	75.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[woutdenolf]

Claude found that the env name change (not necessarily called default) was not covered everywhere:

Location is unchanged, but the prefix is no longer always default. That breaks later lookups.

• ewoks execute --env <name> runs the wrong Python

  • command_in_environment (__main__.py) uses Environment.from_location().
  • It only scans ENVIRONMENT_SUBDIRS: "", .venv, .pixi/envs/default.
  • create_environment still builds default first, so for a workspace installed from env test, from_location returns the empty default env, not .pixi/envs/test.
  • Only install_requirements returns the right environment.
  • Suggestion: make the lookup find the real environment (e.g. scan .pixi/envs/*, or record the subdir in the workspace). A test with a non-default name through from_location would cover it.

• Empty default env is still created

  • create_environment runs pixi install on an empty manifest before the files are installed.
  • Wasteful, and it is what makes the point above silently pick the wrong env.

• _owns / is_native still assume default

  • They compare against ENVIRONMENT_SUBDIR, so they return False for the non-default env.
  • Fine today because is_native runs before files_environment, but fragile.

• pixi add --environment needs pixi >= 0.74

  • _add_ewoks now always passes --environment, also for default.
  • pixi add has no --environment option before 0.74, as far as I can tell: it came with inline environments (prefix-dev/pixi#6497).
  • So with an older pixi, ewoks install fails at its last step (adding ewoks), even for the default env.
  • Nothing checks the pixi version. I suggest passing --environment only when the name is not default. Otherwise, add a minimum version check like the one in poetry.py.

• Minor

  • PIXI_ENVIRONMENT_NAME takes precedence over sys.prefix; they can disagree if the interpreter is not the one pixi activated.

Related: #353 (install location). Both need one way to resolve "the environment of a location" instead of fixed subdirs. I can make a follow-up for both.