Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion experiments/agentcompany/openhands/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ smmap==5.0.1
sniffio==1.3.1
soupsieve==2.5
stack-data==0.6.3
starlette==0.41.2
starlette==1.3.1

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P0 FastAPI constraint conflict

fastapi==0.115.4 in this same requirements file requires starlette>=0.40.0,<0.42.0, but this change pins starlette==1.3.1. With both pins present, pip cannot resolve the environment, so installing experiments/agentcompany/openhands/requirements.txt fails before the OpenHands experiment can run. If Starlette is forced past the resolver, FastAPI is still running against an unsupported major version of its Starlette dependency. Please either keep Starlette inside FastAPI 0.115.4's supported range or bump FastAPI to a release that supports Starlette 1.x.

Artifacts

PyPI metadata check showing fastapi 0.115.4 requires starlette<0.42.0

  • Keeps the command output available without making the summary code-heavy.

PyPI metadata check showing fastapi 0.115.4 requires starlette<0.42.0

  • Keeps the command output available without making the summary code-heavy.

View artifacts

T-Rex Ran code and verified through T-Rex

Prompt To Fix With AI
This is a comment left during a code review.
Path: experiments/agentcompany/openhands/requirements.txt
Line: 169

Comment:
**FastAPI constraint conflict**

`fastapi==0.115.4` in this same requirements file requires `starlette>=0.40.0,<0.42.0`, but this change pins `starlette==1.3.1`. With both pins present, pip cannot resolve the environment, so installing `experiments/agentcompany/openhands/requirements.txt` fails before the OpenHands experiment can run. If Starlette is forced past the resolver, FastAPI is still running against an unsupported major version of its Starlette dependency. Please either keep Starlette inside FastAPI 0.115.4's supported range or bump FastAPI to a release that supports Starlette 1.x.

How can I resolve this? If you propose a fix, please make it concise.

Fix in Cursor Fix in Claude Code Fix in Codex

sympy==1.13.1
terminado==0.18.1
threadpoolctl==3.5.0
Expand Down