Improved naming on tools and prompts. #38
Merged
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 pull request includes several changes to the CI workflow, Python version setup, and the removal of multiple unused coder classes and their associated files. The most important changes include updating the CI workflow to use
actions/setup-python
, setting a specific Python version, and removing theChangesDescriberCoder
andReplacerCoder
classes along with their associated models and prompts.CI Workflow Updates:
.github/workflows/ci.yml
: Updated the CI workflow to useactions/setup-python@v5
and set the Python version using.python-version
file. Also, changed the command to install the project dependencies touv sync --locked
instead ofuv sync --all-extras --dev
. [1] [2]Python Version Setup:
.python-version
: Added Python version3.12.5
.Removal of Unused Coder Classes:
daiv/automation/coders/change_describer/__init__.py
,daiv/automation/coders/change_describer/coder.py
,daiv/automation/coders/change_describer/models.py
,daiv/automation/coders/change_describer/prompts.py
: Removed theChangesDescriberCoder
class and its associated models and prompts. [1] [2] [3] [4]daiv/automation/coders/replacer/__init__.py
,daiv/automation/coders/replacer/coder.py
,daiv/automation/coders/replacer/prompts.py
,daiv/automation/coders/replacer/tools.py
: Removed theReplacerCoder
class and its associated models, prompts, and tools. [1] [2] [3] [4]daiv/automation/coders/review_fixer/coder.py
,daiv/automation/coders/review_fixer/models.py
,daiv/automation/coders/review_fixer/prompts.py
: Removed theReviewFixerCoder
class and its associated models and prompts. [1] [2] [3]Makefile Adjustments:
Makefile
: Removed thetest-ci
andunittests
targets and updated thetest
target to usepytest
. [1] [2]Agent Model Initialization:
daiv/automation/graphs/agents.py
: Updated the agent model initialization to useinit_chat_model
fromlangchain.chat_models
and added configurable fields to the model kwargs. [1] [2] [3]This pull request includes several changes to the CI workflow configuration and significant updates to thereview_adressor
agent's logic and prompts in thedaiv/automation/graphs
directory. The most important changes involve modifying the CI workflow to use locked dependencies and refactoring the agent's state management and prompts for better clarity and functionality.CI Workflow Changes:
.github/workflows/ci.yml
: Updated the project installation step to useuv sync --locked
instead ofuv sync --all-extras --dev
for more consistent dependency management. [1] [2]Agent Logic Refactoring:
daiv/automation/graphs/review_adressor/agent.py
: Refactored the agent's state management by replacingPlanExecute
withOverallState
and updated the workflow nodes and edges accordingly. Added new methods for assessment and renamed existing methods for clarity. [1] [2] [3] [4]Prompt Updates:
daiv/automation/graphs/review_adressor/prompts.py
: Introduced new prompt templates for assessment and response, and updated existing templates for planning and execution to improve clarity and specificity.