-
Notifications
You must be signed in to change notification settings - Fork 338
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(amazonq): incorrect project scan size limit #4924
Conversation
@@ -69,7 +69,8 @@ async function filterOutGitignoredFiles(rootPath: string, files: Uri[]): Promise | |||
export async function collectFiles( | |||
sourcePaths: string[], | |||
workspaceFolders: CurrentWsFolders, | |||
respectGitIgnore: boolean = true | |||
respectGitIgnore: boolean = true, | |||
maxSize = maxRepoSizeBytes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we have an opportunity to improve some quality in the codebase which was probably left over from re-invent. It looks like codewhisperer is importing from the featureDev part of the codebase. If this is the case, can we pull collectFiles
out to a different place?
Generic utils (like collectFiles) used by multiple q products shouldn't be in individual products part of the codebase.
Also, was this issue caused by a lack of tests? Because we should be adding more tests to know that this issue is fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for lifting collectFiles
into a shared module!
Problem
collectFiles
method uses a different max payload size value than security scans.Solution
collectFiles
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.