feat(intelligence): add Duplicate Entity Detection#2953
Open
aashir-athar wants to merge 2 commits into
Open
Conversation
Contributor
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughAdds deterministic duplicate-entity clustering, RPC loaders, a presentational panel and container tab, Intelligence page wiring for a new "Duplicates" tab, tests, and i18n strings across locales. ChangesDuplicate Entity Detection in Knowledge Graphs
Sequence Diagram(s)sequenceDiagram
participant UI as EntityDuplicatesTab
participant Panel as EntityDuplicatesPanel
participant API as entityDuplicatesApi
participant RPC as memoryGraphQuery/memoryListNamespaces
UI->>API: loadNamespaces()
API->>RPC: memoryListNamespaces()
RPC-->>API: namespaces[]
API-->>UI: namespaces[]
UI->>API: loadEntityDuplicates(namespace)
API->>RPC: memoryGraphQuery(namespace)
RPC-->>API: triples[]
API->>API: computeEntityDuplicates(triples)
API-->>UI: DuplicateReport
UI->>Panel: render(report, loading, error)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches⚔️ Resolve merge conflicts
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
coderabbitai
Bot
added
feature
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@app/src/components/intelligence/EntityDuplicatesPanel.tsx`:
- Around line 95-167: The component EntityDuplicatesPanel.tsx uses i18n keys
entityDuplicates.namespaceAll and entityDuplicates.namespaceLabel that are
missing; add these two keys with appropriate English strings into the English
i18n source (app/src/lib/i18n/en.ts) and then add the same keys to every locale
chunk file under app/src/lib/i18n/chunks (the en-*.ts chunk files and
corresponding other locales) so all locales include
entityDuplicates.namespaceAll and entityDuplicates.namespaceLabel; ensure the
keys match exactly the strings used in EntityDuplicatesPanel.tsx and follow the
existing translation object structure.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: ba3323b2-9b8d-4407-96c6-98cd5a258a9c
📒 Files selected for processing (23)
app/src/components/intelligence/EntityDuplicatesPanel.test.tsxapp/src/components/intelligence/EntityDuplicatesPanel.tsxapp/src/components/intelligence/EntityDuplicatesTab.test.tsxapp/src/components/intelligence/EntityDuplicatesTab.tsxapp/src/lib/i18n/chunks/ar-1.tsapp/src/lib/i18n/chunks/bn-1.tsapp/src/lib/i18n/chunks/de-1.tsapp/src/lib/i18n/chunks/en-1.tsapp/src/lib/i18n/chunks/es-1.tsapp/src/lib/i18n/chunks/fr-1.tsapp/src/lib/i18n/chunks/hi-1.tsapp/src/lib/i18n/chunks/id-1.tsapp/src/lib/i18n/chunks/it-1.tsapp/src/lib/i18n/chunks/ko-1.tsapp/src/lib/i18n/chunks/pt-1.tsapp/src/lib/i18n/chunks/ru-1.tsapp/src/lib/i18n/chunks/zh-CN-1.tsapp/src/lib/i18n/en.tsapp/src/lib/memory/entityDuplicates.test.tsapp/src/lib/memory/entityDuplicates.tsapp/src/pages/Intelligence.tsxapp/src/services/api/entityDuplicatesApi.test.tsapp/src/services/api/entityDuplicatesApi.ts
Contributor
Author
|
Good catch @coderabbitai — fixed. Added entityDuplicates.namespaceLabel and entityDuplicates.namespaceAll to en.ts and mirrored across all locale chunks (i18n coverage gate EXIT 0). The namespace selector only renders when namespaces exist, which is why the empty-namespace unit tests didn't surface it. (The Rust Core Coverage lane is a TS-only-PR infra flake — re-triggered.) |
Contributor
|
Tip For best results, initiate chat on the files or code changes.
🐇 ✨ |
aashir-athar
force-pushed
the
feat/entity-duplicates
branch
from
480e43e to
4df2724
Compare
aashir-athar
force-pushed
the
feat/entity-duplicates
branch
2 times, most recently
from
db22c57 to
4ac1ea4
Compare
A new read-only "Duplicates" tab that surfaces entities recorded under different spellings (case / whitespace / punctuation) as likely duplicates — the natural companion to the centrality lens, which deliberately keeps variants distinct. Helps the user spot fragmentation in their knowledge graph. - Pure deterministic engine (lib/memory/entityDuplicates.ts): groups entities by a normalized key (trim, collapse whitespace, lowercase) and keeps clusters with >= 2 distinct raw spellings; each variant carries its distinct-neighbour degree so the most-connected (likely-canonical) spelling is visible. No clock, no RNG; deterministic variant + cluster ordering. Never mutates stored names. - Zero new core surface: reuses memoryGraphQuery + memoryListNamespaces. Read-only, recomputed from the live graph. - Container guards the load with a request token; namespace selector; summary tiles + cluster list of variant chips (with degree), a distinct "all clean" state when no duplicates, and a truncation note. i18n across all 13 locales. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
aashir-athar
force-pushed
the
feat/entity-duplicates
branch
from
4ac1ea4 to
c81c627
Compare
Address CodeRabbit: EntityDuplicatesTab references entityDuplicates.namespaceLabel / namespaceAll but they were never defined. Add both to en.ts + the locale chunks. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
aashir-athar
force-pushed
the
feat/entity-duplicates
branch
from
c81c627 to
d200ed8
Compare
Contributor
There was a problem hiding this comment.
Actionable comments posted: 6
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@app/src/lib/i18n/bn.ts`:
- Around line 4138-4155: Replace the English strings for the new i18n keys in
bn.ts with correct Bengali translations: update each key
entityDuplicates.allClean, blankEntity, empty, emptyHint, errorPrefix, heading,
intro, loading, metricAffected, metricClusters, metricEntities, namespaceAll,
namespaceLabel, retry, title, truncated, variantTitle and memory.tab.duplicates
with accurate Bengali text (not English), preserving placeholders like {shown},
{total}, and {degree}; keep punctuation/formatting consistent and ensure the
translations read naturally in Bengali for the entity duplicates UI.
In `@app/src/lib/i18n/es.ts`:
- Around line 4215-4232: The Spanish locale file contains English literals for
the new keys; replace each `entityDuplicates.*` and `memory.tab.duplicates`
value with proper Spanish translations, preserving placeholders and punctuation
(e.g., keep "{shown}", "{total}", "{degree}") and key names:
entityDuplicates.allClean, entityDuplicates.blankEntity, entityDuplicates.empty,
entityDuplicates.emptyHint, entityDuplicates.errorPrefix,
entityDuplicates.heading, entityDuplicates.intro, entityDuplicates.loading,
entityDuplicates.metricAffected, entityDuplicates.metricClusters,
entityDuplicates.metricEntities, entityDuplicates.namespaceAll,
entityDuplicates.namespaceLabel, entityDuplicates.retry, entityDuplicates.title,
entityDuplicates.truncated, entityDuplicates.variantTitle, and
memory.tab.duplicates — update the strings to fluent Spanish equivalents
matching the English intent.
In `@app/src/lib/i18n/fr.ts`:
- Around line 4232-4249: Replace the English literals for the i18n keys
introduced: all keys under entityDuplicates (entityDuplicates.allClean,
blankEntity, empty, emptyHint, errorPrefix, heading, intro, loading,
metricAffected, metricClusters, metricEntities, namespaceAll, namespaceLabel,
retry, title, truncated, variantTitle) and memory.tab.duplicates with proper
French translations in fr.ts; keep interpolation placeholders exactly as-is
(e.g., {shown}, {total}, {degree}) and preserve punctuation/ellipsis/ellipsis
characters used in strings (e.g., "…") while matching the style of surrounding
translations.
In `@app/src/lib/i18n/it.ts`:
- Around line 4210-4227: The Italian locale contains English placeholders for
the entityDuplicates.* keys and memory.tab.duplicates; replace each value for
'entityDuplicates.allClean', 'entityDuplicates.blankEntity',
'entityDuplicates.empty', 'entityDuplicates.emptyHint',
'entityDuplicates.errorPrefix', 'entityDuplicates.heading',
'entityDuplicates.intro', 'entityDuplicates.loading',
'entityDuplicates.metricAffected', 'entityDuplicates.metricClusters',
'entityDuplicates.metricEntities', 'entityDuplicates.namespaceAll',
'entityDuplicates.namespaceLabel', 'entityDuplicates.retry',
'entityDuplicates.title', 'entityDuplicates.truncated',
'entityDuplicates.variantTitle', and 'memory.tab.duplicates' with correct
Italian translations (natural phrasing, preserve interpolation tokens like
{shown}, {total}, {degree}), ensuring punctuation and capitalization match the
UI style.
In `@app/src/lib/i18n/ru.ts`:
- Around line 4179-4196: The i18n entries for Russian are still in English;
replace each English string with a correct Russian translation for the keys
'entityDuplicates.allClean', 'entityDuplicates.blankEntity',
'entityDuplicates.empty', 'entityDuplicates.emptyHint',
'entityDuplicates.errorPrefix', 'entityDuplicates.heading',
'entityDuplicates.intro', 'entityDuplicates.loading',
'entityDuplicates.metricAffected', 'entityDuplicates.metricClusters',
'entityDuplicates.metricEntities', 'entityDuplicates.namespaceAll',
'entityDuplicates.namespaceLabel', 'entityDuplicates.retry',
'entityDuplicates.title', 'entityDuplicates.truncated',
'entityDuplicates.variantTitle', and 'memory.tab.duplicates'—keeping
placeholders like {shown}, {total}, and {degree} intact and grammar-appropriate
in Russian. Ensure translations are natural, match existing locale style, and
follow pluralization/placeholders exactly before committing.
In `@app/src/lib/i18n/zh-CN.ts`:
- Around line 3941-3958: The listed i18n keys (entityDuplicates.allClean,
entityDuplicates.blankEntity, entityDuplicates.empty,
entityDuplicates.emptyHint, entityDuplicates.errorPrefix,
entityDuplicates.heading, entityDuplicates.intro, entityDuplicates.loading,
entityDuplicates.metricAffected, entityDuplicates.metricClusters,
entityDuplicates.metricEntities, entityDuplicates.namespaceAll,
entityDuplicates.namespaceLabel, entityDuplicates.retry, entityDuplicates.title,
entityDuplicates.truncated, entityDuplicates.variantTitle, and
memory.tab.duplicates) are still in English in zh-CN.ts—replace each value with
a correct Simplified Chinese translation while preserving placeholders like
{shown}, {total}, and {degree} and keeping punctuation/ellipsis semantics
exactly; ensure translations are natural, concise, and match the tone of the
other zh-CN entries.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: a87d7ead-8751-49e1-971e-8f26c2c317d0
📒 Files selected for processing (15)
app/src/lib/i18n/ar.tsapp/src/lib/i18n/bn.tsapp/src/lib/i18n/de.tsapp/src/lib/i18n/en.tsapp/src/lib/i18n/es.tsapp/src/lib/i18n/fr.tsapp/src/lib/i18n/hi.tsapp/src/lib/i18n/id.tsapp/src/lib/i18n/it.tsapp/src/lib/i18n/ko.tsapp/src/lib/i18n/pl.tsapp/src/lib/i18n/pt.tsapp/src/lib/i18n/ru.tsapp/src/lib/i18n/zh-CN.tsapp/src/pages/Intelligence.tsx
✅ Files skipped from review due to trivial changes (7)
- app/src/lib/i18n/ko.ts
- app/src/lib/i18n/pl.ts
- app/src/lib/i18n/id.ts
- app/src/lib/i18n/de.ts
- app/src/lib/i18n/ar.ts
- app/src/lib/i18n/hi.ts
- app/src/lib/i18n/pt.ts
🚧 Files skipped from review as they are similar to previous changes (2)
- app/src/pages/Intelligence.tsx
- app/src/lib/i18n/en.ts
Comment on lines
+4138
to
+4155
| 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.', | ||
| 'entityDuplicates.blankEntity': '(blank)', | ||
| 'entityDuplicates.empty': 'No knowledge graph yet.', | ||
| 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.', | ||
| 'entityDuplicates.errorPrefix': 'Could not load the graph:', | ||
| 'entityDuplicates.heading': 'Likely duplicate entities', | ||
| 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.', | ||
| 'entityDuplicates.loading': 'Scanning for duplicates…', | ||
| 'entityDuplicates.metricAffected': 'Affected', | ||
| 'entityDuplicates.metricClusters': 'Duplicate sets', | ||
| 'entityDuplicates.metricEntities': 'Entities', | ||
| 'entityDuplicates.namespaceAll': 'All namespaces', | ||
| 'entityDuplicates.namespaceLabel': 'Namespace', | ||
| 'entityDuplicates.retry': 'Retry', | ||
| 'entityDuplicates.title': 'Duplicate Entities', | ||
| 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.', | ||
| 'entityDuplicates.variantTitle': '{degree} connections', | ||
| 'memory.tab.duplicates': 'Duplicates', |
Contributor
There was a problem hiding this comment.
Translate new entityDuplicates.* and memory.tab.duplicates values to Bengali.
These newly added bn.ts strings are English, so Bengali users will see untranslated copy in the new tab/panel. Please replace them with real Bengali translations for all added keys in this range.
As per coding guidelines: “When adding or changing i18n keys in app/src/lib/i18n/en.ts, add the same key with a real, correct translation (not English) to every non-English locale file.”
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@app/src/lib/i18n/bn.ts` around lines 4138 - 4155, Replace the English strings
for the new i18n keys in bn.ts with correct Bengali translations: update each
key entityDuplicates.allClean, blankEntity, empty, emptyHint, errorPrefix,
heading, intro, loading, metricAffected, metricClusters, metricEntities,
namespaceAll, namespaceLabel, retry, title, truncated, variantTitle and
memory.tab.duplicates with accurate Bengali text (not English), preserving
placeholders like {shown}, {total}, and {degree}; keep punctuation/formatting
consistent and ensure the translations read naturally in Bengali for the entity
duplicates UI.
Comment on lines
+4215
to
+4232
| 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.', | ||
| 'entityDuplicates.blankEntity': '(blank)', | ||
| 'entityDuplicates.empty': 'No knowledge graph yet.', | ||
| 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.', | ||
| 'entityDuplicates.errorPrefix': 'Could not load the graph:', | ||
| 'entityDuplicates.heading': 'Likely duplicate entities', | ||
| 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.', | ||
| 'entityDuplicates.loading': 'Scanning for duplicates…', | ||
| 'entityDuplicates.metricAffected': 'Affected', | ||
| 'entityDuplicates.metricClusters': 'Duplicate sets', | ||
| 'entityDuplicates.metricEntities': 'Entities', | ||
| 'entityDuplicates.namespaceAll': 'All namespaces', | ||
| 'entityDuplicates.namespaceLabel': 'Namespace', | ||
| 'entityDuplicates.retry': 'Retry', | ||
| 'entityDuplicates.title': 'Duplicate Entities', | ||
| 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.', | ||
| 'entityDuplicates.variantTitle': '{degree} connections', | ||
| 'memory.tab.duplicates': 'Duplicates', |
Contributor
There was a problem hiding this comment.
Translate new entityDuplicates.* and memory.tab.duplicates strings to Spanish.
These newly added values are English literals in es.ts, so Spanish users will see mixed-language UI in this panel/tab. Please replace all of them with proper Spanish translations in this file.
As per coding guidelines: “When adding or changing i18n keys in app/src/lib/i18n/en.ts, add the same key with a real, correct translation (not English) to every non-English locale file.”
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@app/src/lib/i18n/es.ts` around lines 4215 - 4232, The Spanish locale file
contains English literals for the new keys; replace each `entityDuplicates.*`
and `memory.tab.duplicates` value with proper Spanish translations, preserving
placeholders and punctuation (e.g., keep "{shown}", "{total}", "{degree}") and
key names: entityDuplicates.allClean, entityDuplicates.blankEntity,
entityDuplicates.empty, entityDuplicates.emptyHint,
entityDuplicates.errorPrefix, entityDuplicates.heading, entityDuplicates.intro,
entityDuplicates.loading, entityDuplicates.metricAffected,
entityDuplicates.metricClusters, entityDuplicates.metricEntities,
entityDuplicates.namespaceAll, entityDuplicates.namespaceLabel,
entityDuplicates.retry, entityDuplicates.title, entityDuplicates.truncated,
entityDuplicates.variantTitle, and memory.tab.duplicates — update the strings to
fluent Spanish equivalents matching the English intent.
Comment on lines
+4232
to
+4249
| 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.', | ||
| 'entityDuplicates.blankEntity': '(blank)', | ||
| 'entityDuplicates.empty': 'No knowledge graph yet.', | ||
| 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.', | ||
| 'entityDuplicates.errorPrefix': 'Could not load the graph:', | ||
| 'entityDuplicates.heading': 'Likely duplicate entities', | ||
| 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.', | ||
| 'entityDuplicates.loading': 'Scanning for duplicates…', | ||
| 'entityDuplicates.metricAffected': 'Affected', | ||
| 'entityDuplicates.metricClusters': 'Duplicate sets', | ||
| 'entityDuplicates.metricEntities': 'Entities', | ||
| 'entityDuplicates.namespaceAll': 'All namespaces', | ||
| 'entityDuplicates.namespaceLabel': 'Namespace', | ||
| 'entityDuplicates.retry': 'Retry', | ||
| 'entityDuplicates.title': 'Duplicate Entities', | ||
| 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.', | ||
| 'entityDuplicates.variantTitle': '{degree} connections', | ||
| 'memory.tab.duplicates': 'Duplicates', |
Contributor
There was a problem hiding this comment.
French locale entries are still in English
All newly added entityDuplicates.* and memory.tab.duplicates values are English literals, so French users will see untranslated UI for this feature. Please replace these with proper French translations in this file.
As per coding guidelines: “When adding or changing i18n keys in app/src/lib/i18n/en.ts, add the same key with a real, correct translation (not English) to every non-English locale file.”
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@app/src/lib/i18n/fr.ts` around lines 4232 - 4249, Replace the English
literals for the i18n keys introduced: all keys under entityDuplicates
(entityDuplicates.allClean, blankEntity, empty, emptyHint, errorPrefix, heading,
intro, loading, metricAffected, metricClusters, metricEntities, namespaceAll,
namespaceLabel, retry, title, truncated, variantTitle) and memory.tab.duplicates
with proper French translations in fr.ts; keep interpolation placeholders
exactly as-is (e.g., {shown}, {total}, {degree}) and preserve
punctuation/ellipsis/ellipsis characters used in strings (e.g., "…") while
matching the style of surrounding translations.
Comment on lines
+4210
to
+4227
| 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.', | ||
| 'entityDuplicates.blankEntity': '(blank)', | ||
| 'entityDuplicates.empty': 'No knowledge graph yet.', | ||
| 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.', | ||
| 'entityDuplicates.errorPrefix': 'Could not load the graph:', | ||
| 'entityDuplicates.heading': 'Likely duplicate entities', | ||
| 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.', | ||
| 'entityDuplicates.loading': 'Scanning for duplicates…', | ||
| 'entityDuplicates.metricAffected': 'Affected', | ||
| 'entityDuplicates.metricClusters': 'Duplicate sets', | ||
| 'entityDuplicates.metricEntities': 'Entities', | ||
| 'entityDuplicates.namespaceAll': 'All namespaces', | ||
| 'entityDuplicates.namespaceLabel': 'Namespace', | ||
| 'entityDuplicates.retry': 'Retry', | ||
| 'entityDuplicates.title': 'Duplicate Entities', | ||
| 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.', | ||
| 'entityDuplicates.variantTitle': '{degree} connections', | ||
| 'memory.tab.duplicates': 'Duplicates', |
Contributor
There was a problem hiding this comment.
Replace English placeholders with real Italian translations
These newly added Italian-locale values are still English, so Italian users will see untranslated UI copy (Line 4210 through Line 4227). Please replace all entityDuplicates.* and memory.tab.duplicates values with proper Italian translations.
Suggested translation patch
- 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.',
+ 'entityDuplicates.allClean': 'Nessuna variante duplicata rilevata: le tue entità sembrano pulite.',
- 'entityDuplicates.blankEntity': '(blank)',
+ 'entityDuplicates.blankEntity': '(vuoto)',
- 'entityDuplicates.empty': 'No knowledge graph yet.',
+ 'entityDuplicates.empty': 'Nessun grafo della conoscenza disponibile.',
- 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.',
+ 'entityDuplicates.emptyHint': 'Man mano che l’assistente registra entità, eventuali varianti duplicate appariranno qui.',
- 'entityDuplicates.errorPrefix': 'Could not load the graph:',
+ 'entityDuplicates.errorPrefix': 'Impossibile caricare il grafo:',
- 'entityDuplicates.heading': 'Likely duplicate entities',
+ 'entityDuplicates.heading': 'Entità probabilmente duplicate',
- 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.',
+ 'entityDuplicates.intro': 'Il grafo memorizza i nomi alla lettera, quindi la stessa entità può frammentarsi in varianti di scrittura (maiuscole/minuscole, spazi). Queste entità si normalizzano nello stesso nome: probabili duplicati da riconciliare.',
- 'entityDuplicates.loading': 'Scanning for duplicates…',
+ 'entityDuplicates.loading': 'Ricerca dei duplicati…',
- 'entityDuplicates.metricAffected': 'Affected',
+ 'entityDuplicates.metricAffected': 'Coinvolte',
- 'entityDuplicates.metricClusters': 'Duplicate sets',
+ 'entityDuplicates.metricClusters': 'Gruppi duplicati',
- 'entityDuplicates.metricEntities': 'Entities',
+ 'entityDuplicates.metricEntities': 'Entità',
- 'entityDuplicates.namespaceAll': 'All namespaces',
+ 'entityDuplicates.namespaceAll': 'Tutti i namespace',
- 'entityDuplicates.namespaceLabel': 'Namespace',
+ 'entityDuplicates.namespaceLabel': 'Namespace',
- 'entityDuplicates.retry': 'Retry',
+ 'entityDuplicates.retry': 'Riprova',
- 'entityDuplicates.title': 'Duplicate Entities',
+ 'entityDuplicates.title': 'Entità duplicate',
- 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.',
+ 'entityDuplicates.truncated': 'Mostrando {shown} di {total} gruppi duplicati.',
- 'entityDuplicates.variantTitle': '{degree} connections',
+ 'entityDuplicates.variantTitle': '{degree} connessioni',
- 'memory.tab.duplicates': 'Duplicates',
+ 'memory.tab.duplicates': 'Duplicati',As per coding guidelines: “When adding or changing i18n keys in app/src/lib/i18n/en.ts, add the same key with a real, correct translation (not English) to every non-English locale file: ... it.ts”.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.', | |
| 'entityDuplicates.blankEntity': '(blank)', | |
| 'entityDuplicates.empty': 'No knowledge graph yet.', | |
| 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.', | |
| 'entityDuplicates.errorPrefix': 'Could not load the graph:', | |
| 'entityDuplicates.heading': 'Likely duplicate entities', | |
| 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.', | |
| 'entityDuplicates.loading': 'Scanning for duplicates…', | |
| 'entityDuplicates.metricAffected': 'Affected', | |
| 'entityDuplicates.metricClusters': 'Duplicate sets', | |
| 'entityDuplicates.metricEntities': 'Entities', | |
| 'entityDuplicates.namespaceAll': 'All namespaces', | |
| 'entityDuplicates.namespaceLabel': 'Namespace', | |
| 'entityDuplicates.retry': 'Retry', | |
| 'entityDuplicates.title': 'Duplicate Entities', | |
| 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.', | |
| 'entityDuplicates.variantTitle': '{degree} connections', | |
| 'memory.tab.duplicates': 'Duplicates', | |
| 'entityDuplicates.allClean': 'Nessuna variante duplicata rilevata: le tue entità sembrano pulite.', | |
| 'entityDuplicates.blankEntity': '(vuoto)', | |
| 'entityDuplicates.empty': 'Nessun grafo della conoscenza disponibile.', | |
| 'entityDuplicates.emptyHint': 'Man mano che l'assistente registra entità, eventuali varianti duplicate appariranno qui.', | |
| 'entityDuplicates.errorPrefix': 'Impossibile caricare il grafo:', | |
| 'entityDuplicates.heading': 'Entità probabilmente duplicate', | |
| 'entityDuplicates.intro': 'Il grafo memorizza i nomi alla lettera, quindi la stessa entità può frammentarsi in varianti di scrittura (maiuscole/minuscole, spazi). Queste entità si normalizzano nello stesso nome: probabili duplicati da riconciliare.', | |
| 'entityDuplicates.loading': 'Ricerca dei duplicati…', | |
| 'entityDuplicates.metricAffected': 'Coinvolte', | |
| 'entityDuplicates.metricClusters': 'Gruppi duplicati', | |
| 'entityDuplicates.metricEntities': 'Entità', | |
| 'entityDuplicates.namespaceAll': 'Tutti i namespace', | |
| 'entityDuplicates.namespaceLabel': 'Namespace', | |
| 'entityDuplicates.retry': 'Riprova', | |
| 'entityDuplicates.title': 'Entità duplicate', | |
| 'entityDuplicates.truncated': 'Mostrando {shown} di {total} gruppi duplicati.', | |
| 'entityDuplicates.variantTitle': '{degree} connessioni', | |
| 'memory.tab.duplicates': 'Duplicati', |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@app/src/lib/i18n/it.ts` around lines 4210 - 4227, The Italian locale contains
English placeholders for the entityDuplicates.* keys and memory.tab.duplicates;
replace each value for 'entityDuplicates.allClean',
'entityDuplicates.blankEntity', 'entityDuplicates.empty',
'entityDuplicates.emptyHint', 'entityDuplicates.errorPrefix',
'entityDuplicates.heading', 'entityDuplicates.intro',
'entityDuplicates.loading', 'entityDuplicates.metricAffected',
'entityDuplicates.metricClusters', 'entityDuplicates.metricEntities',
'entityDuplicates.namespaceAll', 'entityDuplicates.namespaceLabel',
'entityDuplicates.retry', 'entityDuplicates.title',
'entityDuplicates.truncated', 'entityDuplicates.variantTitle', and
'memory.tab.duplicates' with correct Italian translations (natural phrasing,
preserve interpolation tokens like {shown}, {total}, {degree}), ensuring
punctuation and capitalization match the UI style.
Comment on lines
+4179
to
+4196
| 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.', | ||
| 'entityDuplicates.blankEntity': '(blank)', | ||
| 'entityDuplicates.empty': 'No knowledge graph yet.', | ||
| 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.', | ||
| 'entityDuplicates.errorPrefix': 'Could not load the graph:', | ||
| 'entityDuplicates.heading': 'Likely duplicate entities', | ||
| 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.', | ||
| 'entityDuplicates.loading': 'Scanning for duplicates…', | ||
| 'entityDuplicates.metricAffected': 'Affected', | ||
| 'entityDuplicates.metricClusters': 'Duplicate sets', | ||
| 'entityDuplicates.metricEntities': 'Entities', | ||
| 'entityDuplicates.namespaceAll': 'All namespaces', | ||
| 'entityDuplicates.namespaceLabel': 'Namespace', | ||
| 'entityDuplicates.retry': 'Retry', | ||
| 'entityDuplicates.title': 'Duplicate Entities', | ||
| 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.', | ||
| 'entityDuplicates.variantTitle': '{degree} connections', | ||
| 'memory.tab.duplicates': 'Duplicates', |
Contributor
There was a problem hiding this comment.
Replace English placeholders with real Russian translations.
Lines 4179-4196 add user-visible strings in English inside ru.ts; these should be proper Russian translations before merge.
Proposed translation patch
- 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.',
- 'entityDuplicates.blankEntity': '(blank)',
- 'entityDuplicates.empty': 'No knowledge graph yet.',
- 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.',
- 'entityDuplicates.errorPrefix': 'Could not load the graph:',
- 'entityDuplicates.heading': 'Likely duplicate entities',
- 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.',
- 'entityDuplicates.loading': 'Scanning for duplicates…',
- 'entityDuplicates.metricAffected': 'Affected',
- 'entityDuplicates.metricClusters': 'Duplicate sets',
- 'entityDuplicates.metricEntities': 'Entities',
- 'entityDuplicates.namespaceAll': 'All namespaces',
- 'entityDuplicates.namespaceLabel': 'Namespace',
- 'entityDuplicates.retry': 'Retry',
- 'entityDuplicates.title': 'Duplicate Entities',
- 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.',
- 'entityDuplicates.variantTitle': '{degree} connections',
- 'memory.tab.duplicates': 'Duplicates',
+ 'entityDuplicates.allClean':
+ 'Дублирующихся вариантов написания не обнаружено — сущности выглядят чистыми.',
+ 'entityDuplicates.blankEntity': '(пусто)',
+ 'entityDuplicates.empty': 'Графа знаний пока нет.',
+ 'entityDuplicates.emptyHint':
+ 'По мере того как ассистент записывает сущности, здесь будут появляться дубли написаний.',
+ 'entityDuplicates.errorPrefix': 'Не удалось загрузить граф:',
+ 'entityDuplicates.heading': 'Вероятные дубликаты сущностей',
+ 'entityDuplicates.intro':
+ 'Граф хранит названия как есть, поэтому одна и та же сущность может распасться на варианты написания (регистр, пробелы). Эти варианты нормализуются к одному имени — вероятные дубликаты для объединения.',
+ 'entityDuplicates.loading': 'Поиск дубликатов…',
+ 'entityDuplicates.metricAffected': 'Затронуто',
+ 'entityDuplicates.metricClusters': 'Наборы дубликатов',
+ 'entityDuplicates.metricEntities': 'Сущности',
+ 'entityDuplicates.namespaceAll': 'Все пространства имён',
+ 'entityDuplicates.namespaceLabel': 'Пространство имён',
+ 'entityDuplicates.retry': 'Повторить',
+ 'entityDuplicates.title': 'Дубликаты сущностей',
+ 'entityDuplicates.truncated': 'Показано {shown} из {total} наборов дубликатов.',
+ 'entityDuplicates.variantTitle': '{degree} связей',
+ 'memory.tab.duplicates': 'Дубликаты',As per coding guidelines: “When adding or changing i18n keys in app/src/lib/i18n/en.ts, add the same key with a real, correct translation (not English) to every non-English locale file.”
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.', | |
| 'entityDuplicates.blankEntity': '(blank)', | |
| 'entityDuplicates.empty': 'No knowledge graph yet.', | |
| 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.', | |
| 'entityDuplicates.errorPrefix': 'Could not load the graph:', | |
| 'entityDuplicates.heading': 'Likely duplicate entities', | |
| 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.', | |
| 'entityDuplicates.loading': 'Scanning for duplicates…', | |
| 'entityDuplicates.metricAffected': 'Affected', | |
| 'entityDuplicates.metricClusters': 'Duplicate sets', | |
| 'entityDuplicates.metricEntities': 'Entities', | |
| 'entityDuplicates.namespaceAll': 'All namespaces', | |
| 'entityDuplicates.namespaceLabel': 'Namespace', | |
| 'entityDuplicates.retry': 'Retry', | |
| 'entityDuplicates.title': 'Duplicate Entities', | |
| 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.', | |
| 'entityDuplicates.variantTitle': '{degree} connections', | |
| 'memory.tab.duplicates': 'Duplicates', | |
| 'entityDuplicates.allClean': | |
| 'Дублирующихся вариантов написания не обнаружено — сущности выглядят чистыми.', | |
| 'entityDuplicates.blankEntity': '(пусто)', | |
| 'entityDuplicates.empty': 'Графа знаний пока нет.', | |
| 'entityDuplicates.emptyHint': | |
| 'По мере того как ассистент записывает сущности, здесь будут появляться дубли написаний.', | |
| 'entityDuplicates.errorPrefix': 'Не удалось загрузить граф:', | |
| 'entityDuplicates.heading': 'Вероятные дубликаты сущностей', | |
| 'entityDuplicates.intro': | |
| 'Граф хранит названия как есть, поэтому одна и та же сущность может распасться на варианты написания (регистр, пробелы). Эти варианты нормализуются к одному имени — вероятные дубликаты для объединения.', | |
| 'entityDuplicates.loading': 'Поиск дубликатов…', | |
| 'entityDuplicates.metricAffected': 'Затронуто', | |
| 'entityDuplicates.metricClusters': 'Наборы дубликатов', | |
| 'entityDuplicates.metricEntities': 'Сущности', | |
| 'entityDuplicates.namespaceAll': 'Все пространства имён', | |
| 'entityDuplicates.namespaceLabel': 'Пространство имён', | |
| 'entityDuplicates.retry': 'Повторить', | |
| 'entityDuplicates.title': 'Дубликаты сущностей', | |
| 'entityDuplicates.truncated': 'Показано {shown} из {total} наборов дубликатов.', | |
| 'entityDuplicates.variantTitle': '{degree} связей', | |
| 'memory.tab.duplicates': 'Дубликаты', |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@app/src/lib/i18n/ru.ts` around lines 4179 - 4196, The i18n entries for
Russian are still in English; replace each English string with a correct Russian
translation for the keys 'entityDuplicates.allClean',
'entityDuplicates.blankEntity', 'entityDuplicates.empty',
'entityDuplicates.emptyHint', 'entityDuplicates.errorPrefix',
'entityDuplicates.heading', 'entityDuplicates.intro',
'entityDuplicates.loading', 'entityDuplicates.metricAffected',
'entityDuplicates.metricClusters', 'entityDuplicates.metricEntities',
'entityDuplicates.namespaceAll', 'entityDuplicates.namespaceLabel',
'entityDuplicates.retry', 'entityDuplicates.title',
'entityDuplicates.truncated', 'entityDuplicates.variantTitle', and
'memory.tab.duplicates'—keeping placeholders like {shown}, {total}, and {degree}
intact and grammar-appropriate in Russian. Ensure translations are natural,
match existing locale style, and follow pluralization/placeholders exactly
before committing.
Comment on lines
+3941
to
+3958
| 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.', | ||
| 'entityDuplicates.blankEntity': '(blank)', | ||
| 'entityDuplicates.empty': 'No knowledge graph yet.', | ||
| 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.', | ||
| 'entityDuplicates.errorPrefix': 'Could not load the graph:', | ||
| 'entityDuplicates.heading': 'Likely duplicate entities', | ||
| 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.', | ||
| 'entityDuplicates.loading': 'Scanning for duplicates…', | ||
| 'entityDuplicates.metricAffected': 'Affected', | ||
| 'entityDuplicates.metricClusters': 'Duplicate sets', | ||
| 'entityDuplicates.metricEntities': 'Entities', | ||
| 'entityDuplicates.namespaceAll': 'All namespaces', | ||
| 'entityDuplicates.namespaceLabel': 'Namespace', | ||
| 'entityDuplicates.retry': 'Retry', | ||
| 'entityDuplicates.title': 'Duplicate Entities', | ||
| 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.', | ||
| 'entityDuplicates.variantTitle': '{degree} connections', | ||
| 'memory.tab.duplicates': 'Duplicates', |
Contributor
There was a problem hiding this comment.
Replace English placeholders with real Simplified Chinese translations.
These new entityDuplicates.* and memory.tab.duplicates strings are still in English in zh-CN, so Chinese users will see untranslated UI copy.
🌐 Suggested zh-CN translations
- 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.',
+ 'entityDuplicates.allClean': '未检测到重复拼写——你的实体看起来很干净。',
- 'entityDuplicates.blankEntity': '(blank)',
+ 'entityDuplicates.blankEntity': '（空白）',
- 'entityDuplicates.empty': 'No knowledge graph yet.',
+ 'entityDuplicates.empty': '暂无知识图谱。',
- 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.',
+ 'entityDuplicates.emptyHint': '随着助手记录实体，任何重复拼写都会显示在这里。',
- 'entityDuplicates.errorPrefix': 'Could not load the graph:',
+ 'entityDuplicates.errorPrefix': '无法加载图谱：',
- 'entityDuplicates.heading': 'Likely duplicate entities',
+ 'entityDuplicates.heading': '可能重复的实体',
- 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.',
+ 'entityDuplicates.intro': '图谱按原样存储名称，因此同一对象可能因拼写（大小写、空格）而分散。以下实体归一化后名称相同——很可能是值得合并的重复项。',
- 'entityDuplicates.loading': 'Scanning for duplicates…',
+ 'entityDuplicates.loading': '正在扫描重复项…',
- 'entityDuplicates.metricAffected': 'Affected',
+ 'entityDuplicates.metricAffected': '受影响',
- 'entityDuplicates.metricClusters': 'Duplicate sets',
+ 'entityDuplicates.metricClusters': '重复组',
- 'entityDuplicates.metricEntities': 'Entities',
+ 'entityDuplicates.metricEntities': '实体',
- 'entityDuplicates.namespaceAll': 'All namespaces',
+ 'entityDuplicates.namespaceAll': '所有命名空间',
- 'entityDuplicates.namespaceLabel': 'Namespace',
+ 'entityDuplicates.namespaceLabel': '命名空间',
- 'entityDuplicates.retry': 'Retry',
+ 'entityDuplicates.retry': '重试',
- 'entityDuplicates.title': 'Duplicate Entities',
+ 'entityDuplicates.title': '重复实体',
- 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.',
+ 'entityDuplicates.truncated': '显示 {total} 个重复组中的 {shown} 个。',
- 'entityDuplicates.variantTitle': '{degree} connections',
+ 'entityDuplicates.variantTitle': '{degree} 个连接',
- 'memory.tab.duplicates': 'Duplicates',
+ 'memory.tab.duplicates': '重复项',As per coding guidelines: “When adding or changing i18n keys in app/src/lib/i18n/en.ts, add the same key with a real, correct translation (not English) to every non-English locale file.”
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| 'entityDuplicates.allClean': 'No duplicate spellings detected — your entities look clean.', | |
| 'entityDuplicates.blankEntity': '(blank)', | |
| 'entityDuplicates.empty': 'No knowledge graph yet.', | |
| 'entityDuplicates.emptyHint': 'As the assistant records entities, any duplicate spellings will surface here.', | |
| 'entityDuplicates.errorPrefix': 'Could not load the graph:', | |
| 'entityDuplicates.heading': 'Likely duplicate entities', | |
| 'entityDuplicates.intro': 'The graph stores names verbatim, so the same thing can fragment across spellings (case, spacing). These entities normalize to the same name — likely duplicates worth reconciling.', | |
| 'entityDuplicates.loading': 'Scanning for duplicates…', | |
| 'entityDuplicates.metricAffected': 'Affected', | |
| 'entityDuplicates.metricClusters': 'Duplicate sets', | |
| 'entityDuplicates.metricEntities': 'Entities', | |
| 'entityDuplicates.namespaceAll': 'All namespaces', | |
| 'entityDuplicates.namespaceLabel': 'Namespace', | |
| 'entityDuplicates.retry': 'Retry', | |
| 'entityDuplicates.title': 'Duplicate Entities', | |
| 'entityDuplicates.truncated': 'Showing {shown} of {total} duplicate sets.', | |
| 'entityDuplicates.variantTitle': '{degree} connections', | |
| 'memory.tab.duplicates': 'Duplicates', | |
| 'entityDuplicates.allClean': '未检测到重复拼写——你的实体看起来很干净。', | |
| 'entityDuplicates.blankEntity': '（空白）', | |
| 'entityDuplicates.empty': '暂无知识图谱。', | |
| 'entityDuplicates.emptyHint': '随着助手记录实体，任何重复拼写都会显示在这里。', | |
| 'entityDuplicates.errorPrefix': '无法加载图谱：', | |
| 'entityDuplicates.heading': '可能重复的实体', | |
| 'entityDuplicates.intro': '图谱按原样存储名称，因此同一对象可能因拼写（大小写、空格）而分散。以下实体归一化后名称相同——很可能是值得合并的重复项。', | |
| 'entityDuplicates.loading': '正在扫描重复项…', | |
| 'entityDuplicates.metricAffected': '受影响', | |
| 'entityDuplicates.metricClusters': '重复组', | |
| 'entityDuplicates.metricEntities': '实体', | |
| 'entityDuplicates.namespaceAll': '所有命名空间', | |
| 'entityDuplicates.namespaceLabel': '命名空间', | |
| 'entityDuplicates.retry': '重试', | |
| 'entityDuplicates.title': '重复实体', | |
| 'entityDuplicates.truncated': '显示 {total} 个重复组中的 {shown} 个。', | |
| 'entityDuplicates.variantTitle': '{degree} 个连接', | |
| 'memory.tab.duplicates': '重复项', |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@app/src/lib/i18n/zh-CN.ts` around lines 3941 - 3958, The listed i18n keys
(entityDuplicates.allClean, entityDuplicates.blankEntity,
entityDuplicates.empty, entityDuplicates.emptyHint,
entityDuplicates.errorPrefix, entityDuplicates.heading, entityDuplicates.intro,
entityDuplicates.loading, entityDuplicates.metricAffected,
entityDuplicates.metricClusters, entityDuplicates.metricEntities,
entityDuplicates.namespaceAll, entityDuplicates.namespaceLabel,
entityDuplicates.retry, entityDuplicates.title, entityDuplicates.truncated,
entityDuplicates.variantTitle, and memory.tab.duplicates) are still in English
in zh-CN.ts—replace each value with a correct Simplified Chinese translation
while preserving placeholders like {shown}, {total}, and {degree} and keeping
punctuation/ellipsis semantics exactly; ensure translations are natural,
concise, and match the tone of the other zh-CN entries.
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
2 participants
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.
Summary
Adds a new read-only "Duplicates" tab that surfaces entities recorded under different spellings —
"Alice"/"alice"/" Alice ","New York"/"New York"— as likely duplicates worth reconciling. It's the natural companion to the merged Knowledge Graph Centrality lens, which deliberately keeps spelling variants distinct; this tab makes that fragmentation visible.Design
lib/memory/entityDuplicates.ts): groups every entity by a normalized key (trim→ collapse internal whitespace →lowercase) and keeps only clusters with ≥ 2 distinct raw spellings. Each variant carries its distinct-neighbour degree so the most-connected (likely-canonical) spelling stands out. No clock, no RNG; variant ordering is degree-desc/id-asc and cluster ordering is variant-count-desc / totalDegree-desc / key-asc — all stable and relation-order-invariant. Never mutates the stored names (normalization is display-only).memoryGraphQueryandmemoryListNamespaceswrappers. Read-only — recomputed from the live graph on mount.Adversarially reviewed (correctness / a11y / integration lenses) before opening; the three confirmed findings — an inaccurate
degreedoc comment and two missing test cases (self-loop-only entities; cluster tie-break ordering) — are all fixed.Test plan
vitest— 23 tests (engine: normalization/grouping/degree-ordering/cluster-tie-breaks/self-loops/order-invariance/malformed; api facade; panel states incl. all-clean; container load + selector + error)tsc --noEmit— cleaneslint— 0 errorsprettier --check— cleanentityDuplicates.*keys🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Localization
Tests