Skip to content

fix: remove unused dependencies from Python template pyproject.toml f…#328

Open
notgitika wants to merge 2 commits intomainfrom
fix/dependency-bloat
Open

fix: remove unused dependencies from Python template pyproject.toml f…#328
notgitika wants to merge 2 commits intomainfrom
fix/dependency-bloat

Conversation

@notgitika
Copy link
Contributor

…iles

Description

  • Remove python-dotenv dependency from all 6 Python framework template pyproject.toml files (autogen, crewai, googleadk, langchain_langgraph, openaiagents, strands) — it is listed as a dependency but never imported in any template code (.env loading is handled by agentcore dev)
  • Make provider-specific SDK dependencies conditional in the Strands template: anthropic, google-genai, and openai are now only included when the corresponding modelProvider is selected, instead of always installing all three

Related Issue

Closes #327

Documentation PR

N/a

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

How have you tested the change?

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

I also ran the CLI with the changes to see that it works!

Checklist

Yes to all

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the
terms of your choice.

@notgitika notgitika requested a review from a team February 17, 2026 22:59
@github-actions
Copy link
Contributor

github-actions bot commented Feb 17, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 28.26% 1807 / 6393
🔵 Statements 27.59% 1875 / 6794
🔵 Functions 27.36% 356 / 1301
🔵 Branches 24.52% 918 / 3743
Generated in workflow #407 for commit 41bfd78 by the Vitest Coverage Report Action

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.

Python templates install unused and provider-irrelevant dependencies

1 participant