Skip to content

Conversation

@juusticeTG
Copy link
Contributor

@juusticeTG juusticeTG commented Oct 25, 2024

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • [ x ] New feature (non-breaking change which adds functionality)

Description

  • Decrease requestDelay to 5000ms from 10000ms to provide user with more frequent agent code change updates
  • Increased pollCount to 360 from 180 to accommodate for twice as frequent polling
  • Removed forced read-only view on before diff of agent code changes. This is to enable users to edit on the left-side of the diff which represents the their workspace, allowing a more fluid code acceptance and manipulation experience.

Checklist

  • [ x ] My code follows the code style of this project
  • [ x ] I have added tests to cover my changes (Current tests are sufficient)
  • [ x ] A short description of the change has been added to the CHANGELOG if the change is customer-facing in the IDE.
  • I have added metrics for my changes (if required)

License

I confirm that my contribution is made under the terms of the Apache 2.0 license.

@juusticeTG juusticeTG changed the title feat(Amazonq): Remove forced read-only view from before diff on agent code changes feat(Amazonq): Remove forced read-only view from before diff on agent code changes + Increase polling rate for Q agent code summary Oct 28, 2024
@juusticeTG juusticeTG marked this pull request as ready for review October 28, 2024 16:32
@juusticeTG juusticeTG requested review from a team as code owners October 28, 2024 16:32
@@ -0,0 +1,4 @@
{
"type" : "feature",
"description" : "Remove read-only mode on before diff of code changes generated by agent"
Copy link
Contributor

@tverney tverney Oct 28, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: can't we combine these both changelogs into one? Also, toolkit team requires to specify which team this change is going into.

"Amazon Q /dev: Remove read-only mode on proposed code changes and increase pooling rate for code generation status"

@rli
Copy link
Contributor

rli commented Oct 28, 2024

unclear why changes in #5007 are in here

@manodnyab manodnyab enabled auto-merge (squash) October 29, 2024 00:23
@manodnyab
Copy link
Contributor

unclear why changes in #5007 are in here

Addressed internally

@manodnyab manodnyab merged commit 343be58 into aws:main Oct 29, 2024
10 of 11 checks passed
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.

5 participants