improve: filter out generic keywords from smart clustering#49
Merged
Conversation
- Add comprehensive list of generic keywords that don't provide meaningful cluster themes - Improve theme generation to find meaningful keywords from item titles when needed - Filter out generic shared keywords from cluster connections - This eliminates useless cluster names like 'Opencode & Analyzing' and meaningless connections Fixes the issue where all clusters were named 'Jvalentini & Worklog' due to generic terms in PR titles.
…r names - Added comprehensive filtering of tool/framework names (npm, bun, react, etc.) - Added filtering of generic adjectives/adverbs (new, good, better, quick, etc.) - Improved theme generation to prefer longer, more specific words - Enhanced fallback logic to find meaningful terms from content analysis - More aggressive filtering to eliminate meaningless cluster names This should produce cluster names like 'Productivity Analysis' instead of 'Jvalentini & Worklog'.
- Renamed conflicting 'allText' variable to 'fullContent' - Fixes TypeScript compilation error
- Fixed type issues with regex match results that could be undefined - Added explicit type guard for array filtering - Ensures fallbackTokens is properly typed as string[]
- Fixed Biome linting error for arrow function parentheses - Added explicit null checks for array access to satisfy TypeScript
- Add special handling for PR events with same PR number (90% similarity) - Add special handling for events with same commit hash (95% similarity) - This prevents duplicate clusters for PR opened/merged events - Ensures related GitHub events are grouped together properly
- Added explicit null checks for array element access - Ensures TypeScript compiler accepts array indexing operations
Add parentheses around arrow function parameters to satisfy Biome formatter requirements.
jvalentini
force-pushed
the
improve-smart-clustering
branch
from
January 6, 2026 15:59
dfabaff to
d33c68d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Smart clustering was generating useless cluster names like:
And meaningless connections showing shared generic keywords like "opencode" across all clusters.
Solution
Changes
Testing