Commit ba18d86
authored
fix(chat): Fixing bug in read files and in accepting permissions (#7055)
## Problem
- Message should be provided as part of header body but right now its
passed as body directly.

- In existing IDE experience, If user asks a second question in the same
chat tab, IDE overrides the first workflow read files with second
workflow read files because `session.setMessageIdToUpdate` is
unchanged(not defined to undefined after completion of first request.)
## Solution
- Fixed in this as passing as part of header body.

- Cleaning the `session.setMessageIdToUpdate` value so for second
request IDE won't update the first request read fileTree.

---
- Treat all work as PUBLIC. Private `feature/x` branches will not be
squash-merged at release time.
- Your code changes must meet the guidelines in
[CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines).
- License: I confirm that my contribution is made under the terms of the
Apache 2.0 license.1 parent 28168cb commit ba18d86
File tree
2 files changed
+13
-7
lines changed- packages/core/src/codewhispererChat/controllers/chat
- messenger
2 files changed
+13
-7
lines changedLines changed: 11 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1286 | 1286 | | |
1287 | 1287 | | |
1288 | 1288 | | |
| 1289 | + | |
| 1290 | + | |
| 1291 | + | |
| 1292 | + | |
| 1293 | + | |
| 1294 | + | |
| 1295 | + | |
| 1296 | + | |
| 1297 | + | |
| 1298 | + | |
1289 | 1299 | | |
1290 | 1300 | | |
1291 | 1301 | | |
1292 | 1302 | | |
1293 | 1303 | | |
1294 | 1304 | | |
1295 | 1305 | | |
1296 | | - | |
1297 | | - | |
1298 | | - | |
1299 | | - | |
1300 | | - | |
1301 | | - | |
1302 | | - | |
| 1306 | + | |
1303 | 1307 | | |
1304 | 1308 | | |
1305 | 1309 | | |
| |||
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
844 | 844 | | |
845 | 845 | | |
846 | 846 | | |
| 847 | + | |
847 | 848 | | |
| 849 | + | |
848 | 850 | | |
849 | 851 | | |
850 | 852 | | |
| |||
0 commit comments