Skip to content

Drop pip install transformerlab from task templates#2000

Merged
aliasaria merged 4 commits into
mainfrom
fix/skill-cli-no-pip-install-transformerlab
May 6, 2026
Merged

Drop pip install transformerlab from task templates#2000
aliasaria merged 4 commits into
mainfrom
fix/skill-cli-no-pip-install-transformerlab

Conversation

@aliasaria
Copy link
Copy Markdown
Member

Summary

  • The transformerlab SDK is preinstalled in the task execution environment, so pip install transformerlab in setup: is redundant.
  • Updated the transformerlab-cli skill examples (task.yaml structure block and the hello-world example) to omit it and explicitly note it's preinstalled — agents kept re-adding the line because the skill modeled it.
  • Updated the lab task init template (cli/src/transformerlab_cli/templates/task_init/task.yaml) to comment out the setup: block with the same note, so freshly initialized tasks don't ship with the unnecessary install.

Test plan

  • cd cli && python -m pytest tests/ -k "init or template" — 11 passed
  • Spot-check lab task init output to confirm the new template renders as expected

The transformerlab SDK is preinstalled in the task environment, so
adding it to setup: is redundant. Update the CLI skill examples and the
lab task init template to omit it (and explicitly note that it's
preinstalled) so agents stop adding the line back.
@netlify
Copy link
Copy Markdown

netlify Bot commented May 6, 2026

Deploy Preview for transformerlab canceled.

Name Link
🔨 Latest commit 4e235a6
🔍 Latest deploy log https://app.netlify.com/projects/transformerlab/deploys/69fb6dfceb2b120008df64c7

@paragon-review
Copy link
Copy Markdown

paragon-review Bot commented May 6, 2026

Paragon Review Skipped

Hi @aliasaria! Your Polarity credit balance is insufficient to complete this review.

Please visit https://app.paragon.run to finish your review.

Copy link
Copy Markdown
Member

@deep1401 deep1401 left a comment

Choose a reason for hiding this comment

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

Thanks if you hold on, I will also change this in all the examples which still do it

Copy link
Copy Markdown
Member

@deep1401 deep1401 left a comment

Choose a reason for hiding this comment

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

Also removed all statements where installed transformerlab sdk in examples

@sentry
Copy link
Copy Markdown

sentry Bot commented May 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@aliasaria aliasaria merged commit be414b5 into main May 6, 2026
23 checks passed
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