feat: Create Financial Assistant agent with Docker and API #120
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.
This commit introduces the "Financial Assistant" agent, forked from the "Financial Advisor" agent.
Key changes include:
python/agents/financial-assistant/
.README.md
with:Dockerfile
for containerized execution.fastapi
anduvicorn
topyproject.toml
.financial_advisor/main.py
with a/invoke
endpoint.Challenges Encountered:
The primary challenge was with building the Docker image due to issues with
poetry lock
andpoetry install
in the provided environment. Thepoetry lock
command consistently failed with a "Could not parse version constraint: ==*" error when processing the full set of project dependencies for Poetry version 1.7.1.Attempts to resolve this included:
pyproject.toml
was correct ("financial-assistant").poetry lock
andpoetry install
strategies.poetry export
to arequirements.txt
and thenpip install -r requirements.txt
.Unfortunately, the
poetry export
command also relies on a valid lock file for full dependency resolution, so I could not fully overcome these issues in the current environment. I was unable to successfully complete testing the Dockerized agent due to these persistent Docker build failures related to Poetry.Further work would require a more flexible environment for Poetry or manual adjustments to the dependency list to isolate the problematic constraint. The current submission includes all code changes up to the point of the Docker build issues.