Skip to content

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

@miguelg719
Copy link
Collaborator

miguelg719 commented Jul 13, 2025

got it, I think we can move down to 13.7 instead? It seems simpler than having to do conditional logic for formatting
@filip-michalsky

@filip-michalsky
Copy link
Collaborator Author

got it, I think we can move down to 13.7 instead? It seems simpler than having to do conditional logic for formatting @filip-michalsky

yep, this has been done. Thank you for the review. Just need the green whenever you get a chance, ty

@filip-michalsky filip-michalsky merged commit ca22d08 into main Jul 14, 2025
12 checks passed
@filip-michalsky filip-michalsky deleted the fm/stg-552-update-stagehand-python-for-crewai branch July 14, 2025 10:00
@miguelg719 miguelg719 mentioned this pull request Jul 15, 2025
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