Skip to content
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
set keywordsToIndicateExcluded to lowercase
  • Loading branch information
gauthierpetetin committed Sep 5, 2025
commit ab9cb7946a2b2dd8bbf2eeac4dfd2d52149668f0
4 changes: 3 additions & 1 deletion src/update-changelog.ts
Original file line number Diff line number Diff line change
Expand Up @@ -266,5 +266,7 @@ export function getCategory(description: string): ChangeCategory {
function checkIfDescriptionIndicatesExcluded(description: string): boolean {
const _description = description.toLowerCase();

return keywordsToIndicateExcluded.some((word) => _description.includes(word));
return keywordsToIndicateExcluded.some((word) =>
_description.includes(word.toLowerCase()),
);
Copy link

Choose a reason for hiding this comment

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

Bug: Redundant Lowercase Conversion Causes Performance Regression

The checkIfDescriptionIndicatesExcluded function now converts keywordsToIndicateExcluded to lowercase on every call. This repeated array mapping creates a new array each time, introducing a performance regression due to inefficient, redundant computation, particularly when processing many commits.

Fix in Cursor Fix in Web

}