fix: use raw addresses as Plotly labels key to prevent slice merging

Plotly merges pie slices with duplicate labels and sums their values,
which would silently misrepresent LP concentration when two addresses
share the same custom label (e.g. both labelled 'Team Wallet').

Fix: use raw addresses as the  key (guaranteed unique) and put
the resolved human names in  (shown in legend) and
(shown in hover). Also:
- Remove duplicated truncateAddress, import from shared @/lib/format
- Narrow getLabel prop type to (string) => string; call site wraps adapter
- Update unit tests: rename _ params to _addr to pass ESLint no-unused-vars,
  add duplicate-label collision test documenting the labels/text split

Author: gisk0

ui-dashboard/src/components/__tests__/lp-concentration-chart.test.ts

@@ -11,7 +11,8 @@ describe("resolvePieLabel", () => {
   });
 
   it("returns named label when getLabel resolves a real name", () => {
-    expect(resolvePieLabel(ADDR, () => "Team Wallet")).toBe("Team Wallet");
+    const getLabel = (_address: string) => "Team Wallet";
+    expect(resolvePieLabel(ADDR, getLabel)).toBe("Team Wallet");
   });
 
   it("returns truncated address when getLabel returns the truncated form", () => {
@@ -20,7 +21,8 @@ describe("resolvePieLabel", () => {
   });
 
   it("does not include raw address when a named label exists", () => {
-    const result = resolvePieLabel(ADDR, () => "Team Wallet");
+    const getLabel = (_address: string) => "Team Wallet";
+    const result = resolvePieLabel(ADDR, getLabel);
     expect(result).not.toContain("0x1015");
     expect(result).toBe("Team Wallet");
   });
@@ -33,7 +35,8 @@ describe("resolvePieLabel", () => {
   });
 
   it("allows multiple addresses to resolve to the same human label", () => {
-    expect(resolvePieLabel(ADDR, () => "Shared Label")).toBe("Shared Label");
-    expect(resolvePieLabel(ADDR2, () => "Shared Label")).toBe("Shared Label");
+    const getLabel = (_address: string) => "Shared Label";
+    expect(resolvePieLabel(ADDR, getLabel)).toBe("Shared Label");
+    expect(resolvePieLabel(ADDR2, getLabel)).toBe("Shared Label");
   });
 });

ui-dashboard/src/components/lp-concentration-chart.tsx

@@ -1,8 +1,8 @@
 "use client";
 
 import dynamic from "next/dynamic";
-import { PLOTLY_BASE_LAYOUT, PLOTLY_CONFIG } from "@/lib/plot";
 import { truncateAddress } from "@/lib/format";
+import { PLOTLY_BASE_LAYOUT, PLOTLY_CONFIG } from "@/lib/plot";
 import { USDM_SYMBOLS } from "@/lib/tokens";
 import type { Pool } from "@/lib/types";
 
@@ -57,7 +57,13 @@ export function LpConcentrationChart({
 
   const resolveLabel = (addr: string) => resolvePieLabel(addr, getLabel);
 
+  // Use raw addresses as labels — Plotly merges slices with duplicate labels,
+  // so human names must not be the label key. Display names go in `text`.
   const labels = [
+    ...top.map((p) => p.address),
+    ...(otherTotal > BigInt(0) ? ["other"] : []),
+  ];
+  const text = [
     ...top.map((p) => resolveLabel(p.address)),
     ...(otherTotal > BigInt(0) ? ["Other"] : []),
   ];
@@ -81,6 +87,7 @@ export function LpConcentrationChart({
     type: "pie" as const,
     hole: 0.4,
     labels,
+    text,
     values,
     customdata,
     hovertemplate,