Skip to content

loosen rich dep and update example render #142

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

filip-michalsky
Copy link
Collaborator

why

Need to update deps to merge new pypi stagehand dep to CrewAI

what changed

Loosened rich - this is a non-core dep for example rendering, not in core lib

test plan

CI

@filip-michalsky
Copy link
Collaborator Author

filip-michalsky commented Jul 9, 2025

alpha published and smoke tested:

https://test.pypi.org/project/stagehand/0.4.0a1/

@miguelg719
Copy link
Collaborator

what if we just pin the rick dependency to 14 and update the examples to remove the conditionals? @filip-michalsky

@filip-michalsky
Copy link
Collaborator Author

filip-michalsky commented Jul 11, 2025

what if we just pin the rick dependency to 14 and update the examples to remove the conditionals? @filip-michalsky

This is unfortunately not possible as this PR is fighting the dependency hell between python stagehand and CrewAI tools. The point of this PR is to allow update of dependencies between CrewAI tools and our updated stagehand package.

See here:
crewAIInc/crewAI-tools#360

it does not build.

 And because stagehand==0.4.0 depends on rich>=14.0.0 and
      embedchain>=0.1.114 depends on rich>=13.7.0,<14.0.0, we can conclude
      that crewai-tools[stagehand] and embedchain>=0.1.114 are incompatible.

Hence we need rich max of 13.7.0 unless we update embedchain dependency in CrewAI tools package itself or tiktoken. (we could propose that too if you think thats better?)

I can update the example to not have the conditional there - that is just a rendering thing.

@miguelg719

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