Skip to content

Conversation

@laileni-aws
Copy link
Contributor

@laileni-aws laileni-aws commented Feb 11, 2025

Types of changes

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

Description

Problem

  • If user gives user input prompt more than 4096 characters, this fails at RTS as model supports only 4096 input characters.
    Value at 'userInput' failed to satisfy constraint: Member must have length less than or equal to 4096
  • Ideally we should not get more than 4096 char input from user input but still features like /test, /dev were able to get the message length more than 4096.

This recording is for VSCode but similar experience for JB too.

Trim.User.prompt.issue.mov

Solution

  • To resolve this issue, we are adding a truncation logic to continue the test generation workflow instead of failing at RTS.
  • No new tests were added as the functionality is not impacted.

Checklist

  • My code follows the code style of this project
  • I have added tests to cover my changes
  • 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.

@laileni-aws laileni-aws marked this pull request as ready for review February 11, 2025 19:12
@laileni-aws laileni-aws requested review from a team as code owners February 11, 2025 19:12
@laileni-aws laileni-aws enabled auto-merge (squash) February 11, 2025 19:36
@laileni-aws laileni-aws merged commit 0c88e67 into aws:main Feb 11, 2025
11 of 13 checks passed
samgst-amazon pushed a commit that referenced this pull request Feb 12, 2025
* Adding truncation logic to testGen user prompt
gandhi-21 pushed a commit to gandhi-21/aws-toolkit-jetbrains that referenced this pull request Feb 26, 2025
)

* Adding truncation logic to testGen user prompt
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.

4 participants