fix(deps): mirror hax-sdk + python-dotenv into Docker requirements#68
Merged
Conversation
PR #66 added hax-sdk to runtime requirements.txt but left requirements-docker.txt unchanged. The Dockerfile installs from requirements-docker.txt, so the deployed image was missing hax-sdk and the HITL approval gate failed at runtime with "No module named 'hax'" whenever HAX_API_KEY was set (swe_af/app.py:622). Also bumps agentfield floor to >=0.1.77 to match the runtime list (drift left over from PR #62) and adds python-dotenv>=1.0 which was likewise missing from the docker variant. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
requirements-docker.txt, but that file had drifted fromrequirements.txtand was missinghax-sdk(added in feat(approval): rebase HITL plan-approval gate (supersedes #50) #66) andpython-dotenv.haxmodule. The HITL approval gate atswe_af/app.py:622(from hax import HaxClient) blew up at runtime withNo module named 'hax'wheneverHAX_API_KEYwas set.agentfieldfloor in the Docker list from>=0.1.73to>=0.1.77to match runtimerequirements.txt(left-over drift from chore(deps): bump agentfield floor to >=0.1.77 #62).Repro
Failing run on the test deployment:
run_1778185367193_ab50c1cd(build reasoner, executionexec_20260507_202248_rtahrqx2). Error in internal logs:Test plan
app.call("swe-planner.build", ...)withHAX_API_KEYset on the test deployment and confirm the approval gate creates a hax request instead of crashing.HAX_API_KEYpath is unchanged (the import is gated behind that env var, so the failure mode and the fix only affect the approval flow).Notes
The two requirements files are easy to drift. Worth a follow-up to either consolidate (single file + extras) or wire a CI check that diffs them.
🤖 Generated with Claude Code