Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: append file to command request only #2580

Merged
merged 2 commits into from
Jan 5, 2024

Conversation

abeatrix
Copy link
Contributor

@abeatrix abeatrix commented Jan 5, 2024

CLOSE #2360
Address issue described in #2575 (comment)

Test plan

run the agent test

@abeatrix abeatrix changed the base branch from main to revert-2575-olafurpg/revert-regression January 5, 2024 14:53
@abeatrix abeatrix merged commit 1020c8c into revert-2575-olafurpg/revert-regression Jan 5, 2024
4 checks passed
@abeatrix abeatrix deleted the bee/fix-agent-test branch January 5, 2024 14:54
abeatrix added a commit that referenced this pull request Jan 5, 2024
…th new fixes (#2583)

Close #2588

- Reverts #2575 with updated fixes from
#2580 that I've confirmed with
@Gedochao on call.

- Merged main and updated the recording to confirm there are no errors
caused by the changes in this branch in all covered tests:


![image](https://github.com/sourcegraph/cody/assets/68532117/e54d893a-0798-48af-a838-061056fbc889)

- Also confirmed this change also passed the new rate limit test
introduced in #2535

- Fixed upresponsive stop button when error is presented


https://github.com/sourcegraph/cody/assets/68532117/6d9263b0-3e9c-419e-939d-20a2ea011824

## Test Plan

- run `pnpm run test:unit` to confirm all the tests are passing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: Name of open file appears appended to chat message
1 participant