Skip to content

Commit 71d41bd

Browse files
committed
Skiped first note on issue addressor human feedback.
1 parent bad3bbf commit 71d41bd

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
2929
- Fixed index updates to branches that don't exist anymore, like when a branch is marked to delete after a merge request is merged: #153.
3030
- `SnippetReplacerAgent` was replacing multiple snippets when only one was expected. Now it will return an error if multiple snippets are found to instruct the llm to provide a more specific original snippet.
3131
- `PipelineFixerAgent` was trying to fix multiple jobs from the same stage at the same time, causing multiple fixes being applied simultaneously to the same files which could lead to conflicts or a job being fixed with outdated code. Now it will fix one job at a time. #164
32+
- Human feedback now is sent without the first note which is the bot note to the issue addressor agent.
3233

3334
### Removed
3435

daiv/codebase/managers/issue_addressor.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,9 @@ def _process_issue(self, issue: Issue, should_reset_plan: bool):
9999
):
100100
# TODO: Improve discovery of the last discussion awaiting for approval
101101
issue_addressor_agent.update_state(
102-
config, {"messages": notes_to_messages(discussions[-1].notes, self.client.current_user.id)}
102+
config,
103+
# Skip first note because it's the bot note
104+
{"messages": notes_to_messages(discussions[-1].notes[1:], self.client.current_user.id)},
103105
)
104106

105107
for chunk in issue_addressor_agent.stream(None, config, stream_mode="updates"):

0 commit comments

Comments
 (0)