gloomberb 0.8.1 → 0.8.2

package/src/plugins/builtin/account-management/model.test.ts

@@ -0,0 +1,54 @@
+import { describe, expect, test } from "bun:test";
+import type { Portfolio, TickerRecord } from "../../../types/ticker";
+import { buildPortfolioChoices, countPortfolioHoldings, NO_PORTFOLIO_VALUE } from "./model";
+
+function makeTicker(symbol: string, portfolios: string[]): TickerRecord {
+  return {
+    metadata: {
+      ticker: symbol,
+      exchange: "XNAS",
+      currency: "USD",
+      name: symbol,
+      portfolios,
+      watchlists: [],
+      positions: [],
+      custom: {},
+      tags: [],
+    },
+  };
+}
+
+describe("account management model", () => {
+  test("counts portfolio holdings from app ticker state", () => {
+    const tickers = new Map([
+      ["AAPL", makeTicker("AAPL", ["main"])],
+      ["MSFT", makeTicker("MSFT", ["main", "income"])],
+      ["SPY", makeTicker("SPY", [])],
+    ]);
+
+    expect(countPortfolioHoldings(tickers)).toEqual({ main: 2, income: 1 });
+  });
+
+  test("describes public portfolio analytics in the shared portfolio picker", () => {
+    const portfolios: Portfolio[] = [
+      { id: "main", name: "Main Portfolio", currency: "USD" },
+    ];
+
+    const choices = buildPortfolioChoices(portfolios, { main: 2 });
+
+    expect(choices).toEqual([
+      {
+        id: NO_PORTFOLIO_VALUE,
+        label: "None",
+        detail: "Off",
+        description: "Do not show portfolio analytics on your public profile.",
+      },
+      {
+        id: "main",
+        label: "Main Portfolio",
+        detail: "2 tickers",
+        description: "Shares this portfolio's YTD % and SPY Beta on your public profile.",
+      },
+    ]);
+  });
+});

package/src/plugins/builtin/account-management/model.ts

@@ -1,6 +1,6 @@
 import type { ChoiceDialogChoice } from "../../../components";
 import type { AccountProfile } from "../../../api-client";
-import type { Portfolio } from "../../../types/ticker";
+import type { Portfolio, TickerRecord } from "../../../types/ticker";
 
 export type AccountFieldKey =
   | "username"
@@ -75,14 +75,24 @@ export function formatPlan(plan: AccountProfile["plan"] | null | undefined): str
   return plan === "pro" ? "Pro" : "Free";
 }
 
+export function countPortfolioHoldings(tickers: ReadonlyMap<string, TickerRecord>): Record<string, number> {
+  const counts: Record<string, number> = {};
+  for (const record of tickers.values()) {
+    for (const portfolioId of record.metadata.portfolios) {
+      counts[portfolioId] = (counts[portfolioId] ?? 0) + 1;
+    }
+  }
+  return counts;
+}
+
 export function buildPortfolioChoices(portfolios: Portfolio[], holdingCounts: Record<string, number>): ChoiceDialogChoice[] {
   return [
-    { id: NO_PORTFOLIO_VALUE, label: "None", detail: "Off", description: "No public analytics portfolio." },
+    { id: NO_PORTFOLIO_VALUE, label: "None", detail: "Off", description: "Do not show portfolio analytics on your public profile." },
     ...portfolios.map((portfolio) => ({
       id: portfolio.id,
       label: portfolio.name,
       detail: `${holdingCounts[portfolio.id] ?? 0} tickers`,
-      description: portfolio.description || `${portfolio.currency} portfolio`,
+      description: "Shares this portfolio's YTD % and SPY Beta on your public profile.",
     })),
   ];
 }

package/src/plugins/builtin/account-management/pane.tsx

@@ -13,6 +13,7 @@ import {
   BASE_FIELD_ORDER,
   NO_PORTFOLIO_VALUE,
   buildPortfolioChoices,
+  countPortfolioHoldings,
   emptyToNull,
   portfolioOptionIds,
   profileToDraft,
@@ -49,15 +50,7 @@ export function AccountManagementPane({ focused, width, height }: PaneProps) {
   const bodyHeight = Math.max(5, height);
   const fieldOrder = BASE_FIELD_ORDER;
 
-  const portfolioHoldingCounts = useMemo(() => {
-    const counts: Record<string, number> = {};
-    for (const record of Object.values(tickers)) {
-      for (const portfolioId of record.metadata.portfolios) {
-        counts[portfolioId] = (counts[portfolioId] ?? 0) + 1;
-      }
-    }
-    return counts;
-  }, [tickers]);
+  const portfolioHoldingCounts = useMemo(() => countPortfolioHoldings(tickers), [tickers]);
   const portfolioChoices = useMemo(
     () => buildPortfolioChoices(portfolios, portfolioHoldingCounts),
     [portfolioHoldingCounts, portfolios],
@@ -138,6 +131,7 @@ export function AccountManagementPane({ focused, width, height }: PaneProps) {
 
   const openPortfolioDialog = useCallback(async () => {
     setActiveField("sharedPortfolioId");
+    const currentPortfolioChoiceId = draftRef.current.sharedPortfolioId || NO_PORTFOLIO_VALUE;
     const selected = await dialog.prompt<string>({
       closeOnClickOutside: false,
       content: (context: PromptContext<string>) => (
@@ -145,7 +139,7 @@ export function AccountManagementPane({ focused, width, height }: PaneProps) {
           {...context}
           title="Shared Portfolio"
           choices={portfolioChoices}
-          footer="↑↓ choose · Enter/click select · Esc cancel"
+          selectedChoiceId={currentPortfolioChoiceId}
         />
       ),
     }).catch(() => "");
@@ -365,7 +359,7 @@ export function AccountManagementPane({ focused, width, height }: PaneProps) {
           <PickerRow
             label="Profile Analytics"
             value={selectedPortfolioLabel(portfolios, draft.sharedPortfolioId)}
-            detail={draft.sharedPortfolioId ? "YTD% + beta source" : "No shared portfolio"}
+            detail={draft.sharedPortfolioId ? "Shares portfolio YTD % + SPY Beta" : "No public portfolio analytics"}
             active={activeField === "sharedPortfolioId"}
             width={formWidth}
             onFocus={() => setActiveField("sharedPortfolioId")}

package/src/plugins/builtin/account-management/password-dialog.tsx

@@ -78,7 +78,7 @@ export function PasswordChangeDialog({
 
   const fieldWidth = 42;
   return (
-    <DialogFrame title="Change Password" footer="Esc cancel">
+    <DialogFrame title="Change Password">
       <Box flexDirection="column" gap={1}>
         <TextField
           label={activeField === "current" ? "> Current Password" : "  Current Password"}

package/src/plugins/builtin/alerts/pane.tsx

@@ -279,7 +279,11 @@ export function AlertsPane({ focused, width, height, close }: PaneProps) {
   return (
     <DataTableView<AlertRule, AlertColumn>
       focused={focused}
-      selectedIndex={selectedIdx}
+      selection={{
+        kind: "index",
+        selectedIndex: selectedIdx,
+        onChange: (index) => setSelectedIdx(index),
+      }}
       onRootKeyDown={handleTableKeyDown}
       rootWidth={width}
       rootHeight={height}
@@ -290,8 +294,6 @@ export function AlertsPane({ focused, width, height, close }: PaneProps) {
       sortDirection="asc"
       onHeaderClick={() => {}}
       getItemKey={(alert) => alert.id}
-      isSelected={(_alert, index) => index === selectedIdx}
-      onSelect={(_alert, index) => setSelectedIdx(index)}
       onActivate={(alert) => {
         if (alert.status === "triggered") rearmAlert(alert.id);
       }}

package/src/plugins/builtin/analytics/index.tsx

@@ -198,8 +198,6 @@ function PortfolioAnalyticsPane({ focused, width, height }: PaneProps) {
   const effectiveSelectedSectorId = selectedSectorId && sortedSectorRows.some((row) => row.id === selectedSectorId)
     ? selectedSectorId
     : sortedSectorRows[0]?.id ?? null;
-  const selectedSectorIdx = sortedSectorRows.findIndex((row) => row.id === effectiveSelectedSectorId);
-  const safeSelectedSectorIdx = selectedSectorIdx >= 0 ? selectedSectorIdx : 0;
   const sectorColumns = useMemo(() => buildSectorColumns(width), [width]);
   const hasPositions = portfolioTickers.length > 0;
 
@@ -302,7 +300,6 @@ function PortfolioAnalyticsPane({ focused, width, height }: PaneProps) {
 
               <SectorAllocationTable
                 focused={focused}
-                selectedIndex={safeSelectedSectorIdx}
                 resetScrollKey={activePortfolioId}
                 columns={sectorColumns}
                 rows={sortedSectorRows}

package/src/plugins/builtin/analytics/view.tsx

@@ -143,7 +143,6 @@ export function PortfolioHistorySection({
 
 export function SectorAllocationTable({
   focused,
-  selectedIndex,
   resetScrollKey,
   columns,
   rows,
@@ -153,7 +152,6 @@ export function SectorAllocationTable({
   onSelectSector,
 }: {
   focused: boolean;
-  selectedIndex: number;
   resetScrollKey: string;
   columns: SectorTableColumn[];
   rows: SectorTableRow[];
@@ -165,7 +163,12 @@ export function SectorAllocationTable({
   return (
     <DataTableView<SectorTableRow, SectorTableColumn>
       focused={focused}
-      selectedIndex={selectedIndex}
+      selection={{
+        kind: "id",
+        selectedId: selectedSectorId,
+        getId: (row) => row.id,
+        onChange: (id) => onSelectSector(id),
+      }}
       resetScrollKey={resetScrollKey}
       columns={columns}
       items={rows}
@@ -173,8 +176,6 @@ export function SectorAllocationTable({
       sortDirection={sort.direction}
       onHeaderClick={onHeaderClick}
       getItemKey={(row) => row.id}
-      isSelected={(row) => selectedSectorId === row.id}
-      onSelect={(row) => onSelectSector(row.id)}
       emptyStateTitle="No sector data available"
       emptyStateHint="Load profile data or add sectors to the portfolio positions."
       renderCell={(row, column) => {

package/src/plugins/builtin/broker-manager/index.tsx

@@ -276,18 +276,18 @@ export function BrokersPane({ focused, width, height }: PaneProps) {
           detailTitle={selectedRow?.label}
           rootWidth={tableWidth}
           rootHeight={bodyHeight}
-          selectedIndex={Math.min(selectedIndex, Math.max(0, rows.length - 1))}
-          onSelectIndex={selectBrokerRow}
-          onActivateIndex={openSelectedDetail}
+          selection={{
+            kind: "index",
+            selectedIndex: Math.min(selectedIndex, Math.max(0, rows.length - 1)),
+            onChange: (index, row) => selectBrokerRow(index, row),
+          }}
+          onActivate={(row, index) => openSelectedDetail(index, row)}
           columns={columns}
           items={rows}
           sortColumnId={null}
           sortDirection="asc"
           onHeaderClick={() => {}}
           getItemKey={(row) => row.id}
-          isSelected={(row) => selectedRow?.id === row.id}
-          onSelect={(row, index) => selectBrokerRow(index, row)}
-          onActivate={(row, index) => openSelectedDetail(index, row)}
           renderCell={renderBrokerCell}
           emptyStateTitle="No broker profiles."
           emptyStateHint="Add a broker profile to test connections and sync positions."

package/src/plugins/builtin/buildout/pane/index.tsx

@@ -373,16 +373,16 @@ export function BuildoutPane({ focused, width, height }: PaneProps) {
       )}
       columns={columns}
       items={rows}
-      selectedIndex={selectedIndex}
-      onSelectIndex={(index) => setSelectedIndex(index)}
-      onActivateIndex={(_index, row) => activateRow(row)}
+      selection={{
+        kind: "index",
+        selectedIndex,
+        onChange: (index) => setSelectedIndex(index),
+      }}
+      onActivate={activateRow}
       sortColumnId={sortColumnId}
       sortDirection={sortDirection}
       onHeaderClick={handleHeaderClick}
       getItemKey={rowKey}
-      isSelected={(row) => selectedRow ? rowKey(row) === rowKey(selectedRow) : false}
-      onSelect={(_row, index) => setSelectedIndex(index)}
-      onActivate={activateRow}
       renderCell={renderCell}
       bodyAfter={hiddenCompanyCount > 0 ? (
         <CompaniesUpgradeCta

package/src/plugins/builtin/changelog/index.tsx

@@ -16,11 +16,16 @@ import { fetchChangelogReleases, type ChangelogRelease } from "../../../updater/
 import { colors } from "../../../theme/colors";
 import type { GloomPlugin, PaneProps } from "../../../types/plugin";
 import { isPlainKey } from "../../../utils/keyboard";
-import { usePluginPaneState } from "../../runtime";
+import {
+  DEFAULT_CHANGELOG_SORT,
+  nextChangelogSortPreference,
+  resolveSelectedReleaseIndex,
+  sortChangelogReleases,
+  type ChangelogColumnId,
+} from "./model";
 
 const CHANGELOG_LIMIT = 40;
 
-type ChangelogColumnId = "date" | "version" | "title";
 type ChangelogColumn = DataTableColumn & { id: ChangelogColumnId };
 type LoadStatus = "idle" | "loading" | "loaded" | "error";
 
@@ -103,7 +108,8 @@ function ChangelogPane({ focused, width, height }: PaneProps) {
   const [releases, setReleases] = useState<ChangelogRelease[]>([]);
   const [status, setStatus] = useState<LoadStatus>("idle");
   const [error, setError] = useState<string | null>(null);
-  const [selectedIdx, setSelectedIdx] = usePluginPaneState<number>("selectedIdx", 0);
+  const [selectedReleaseId, setSelectedReleaseId] = useState<string | null>(null);
+  const [sortPreference, setSortPreference] = useState(DEFAULT_CHANGELOG_SORT);
   const [openReleaseId, setOpenReleaseId] = useState<string | null>(null);
   const detailScrollRef = useRef<ScrollBoxRenderable>(null);
   const abortRef = useRef<AbortController | null>(null);
@@ -143,11 +149,18 @@ function ChangelogPane({ focused, width, height }: PaneProps) {
     };
   }, [loadReleases]);
 
+  const sortedReleases = useMemo(
+    () => sortChangelogReleases(releases, sortPreference),
+    [releases, sortPreference],
+  );
+  const activeSelectedIdx = resolveSelectedReleaseIndex(sortedReleases, selectedReleaseId);
+  const activeSelectedReleaseId = sortedReleases[activeSelectedIdx]?.id ?? null;
+
   useEffect(() => {
-    if (releases.length > 0 && selectedIdx >= releases.length) {
-      setSelectedIdx(Math.max(0, releases.length - 1));
+    if (activeSelectedReleaseId !== selectedReleaseId) {
+      setSelectedReleaseId(activeSelectedReleaseId);
     }
-  }, [releases.length, selectedIdx, setSelectedIdx]);
+  }, [activeSelectedReleaseId, selectedReleaseId]);
 
   const openRelease = useMemo(
     () =>
@@ -177,9 +190,6 @@ function ChangelogPane({ focused, width, height }: PaneProps) {
   });
 
   const columns = useMemo(() => buildColumns(width, releases), [releases, width]);
-  const activeSelectedIdx = releases.length > 0
-    ? Math.max(0, Math.min(selectedIdx, releases.length - 1))
-    : -1;
 
   const scrollDetailBy = useCallback((delta: number) => {
     const scrollBox = detailScrollRef.current;
@@ -237,6 +247,14 @@ function ChangelogPane({ focused, width, height }: PaneProps) {
     }
   }, []);
 
+  const handleHeaderClick = useCallback((columnId: string) => {
+    setSortPreference((current) => nextChangelogSortPreference(current, columnId));
+  }, []);
+
+  const selectRelease = useCallback((release: ChangelogRelease) => {
+    setSelectedReleaseId(release.id);
+  }, []);
+
   const footerInfo = useMemo<PaneFooterSegment[]>(() => {
     const segments: PaneFooterSegment[] = [];
     if (status === "loading") {
@@ -251,17 +269,8 @@ function ChangelogPane({ focused, width, height }: PaneProps) {
         parts: [{ text: "error", tone: "warning" }],
       });
     }
-    if (releases.length > 0) {
-      segments.push({
-        id: "versions",
-        parts: [
-          { text: "versions", tone: "label" },
-          { text: String(releases.length), tone: "value" },
-        ],
-      });
-    }
     return segments;
-  }, [releases.length, status]);
+  }, [status]);
 
   const footerHints = useMemo<PaneHint[]>(() => [{
     id: "refresh",
@@ -296,7 +305,7 @@ function ChangelogPane({ focused, width, height }: PaneProps) {
     );
   }
 
-  if (releases.length === 0) {
+  if (sortedReleases.length === 0) {
     return (
       <Box paddingX={1} paddingY={1}>
         <Text fg={colors.textDim}>No changelog entries found.</Text>
@@ -319,18 +328,22 @@ function ChangelogPane({ focused, width, height }: PaneProps) {
         <Box flexGrow={1} />
       )}
       detailTitle={openRelease ? releaseDetailTitle(openRelease) : undefined}
-      selectedIndex={activeSelectedIdx}
-      onSelectIndex={(index) => setSelectedIdx(index)}
-      onActivateIndex={(_index, release) => setOpenReleaseId(release.id)}
+      selection={{
+        kind: "id",
+        selectedId: activeSelectedReleaseId,
+        getId: (release) => release.id,
+        onChange: (_id, release) => selectRelease(release),
+      }}
+      onActivate={(release) => setOpenReleaseId(release.id)}
       onDetailKeyDown={handleDetailKeyDown}
       rootWidth={width}
       rootHeight={height}
       columns={columns}
-      items={releases}
+      items={sortedReleases}
+      sortColumnId={sortPreference.columnId}
+      sortDirection={sortPreference.direction}
+      onHeaderClick={handleHeaderClick}
       getItemKey={(release) => release.id}
-      isSelected={(_release, index) => index === activeSelectedIdx}
-      onSelect={(_release, index) => setSelectedIdx(index)}
-      onActivate={(release) => setOpenReleaseId(release.id)}
       renderCell={renderCell}
       emptyStateTitle="No changelog entries."
       showHorizontalScrollbar={false}

package/src/plugins/builtin/changelog/model.test.ts

@@ -0,0 +1,77 @@
+import { describe, expect, test } from "bun:test";
+import type { ChangelogRelease } from "../../../updater/github-releases";
+import {
+  DEFAULT_CHANGELOG_SORT,
+  nextChangelogSortPreference,
+  resolveSelectedReleaseIndex,
+  sortChangelogReleases,
+} from "./model";
+
+function release(
+  id: string,
+  version: string,
+  title: string,
+  publishedAt: string,
+): ChangelogRelease {
+  return {
+    id,
+    tagName: version,
+    version,
+    title,
+    publishedAt,
+    body: "",
+    url: "",
+  };
+}
+
+describe("changelog table model", () => {
+  const releases = [
+    release("8", "v0.8.0", "Desktop fixes", "2026-05-20T10:00:00Z"),
+    release("10", "v0.10.0", "Accounts", "2026-05-22T10:00:00Z"),
+    release("9", "v0.9.0", "Buildout", "2026-05-21T10:00:00Z"),
+  ];
+
+  test("sorts releases by newest date by default", () => {
+    expect(
+      sortChangelogReleases(releases, DEFAULT_CHANGELOG_SORT).map((entry) => entry.id),
+    )
+      .toEqual(["10", "9", "8"]);
+  });
+
+  test("sorts version tags numerically", () => {
+    expect(
+      sortChangelogReleases(
+        releases,
+        { columnId: "version", direction: "asc" },
+      ).map((entry) => entry.version),
+    )
+      .toEqual(["v0.8.0", "v0.9.0", "v0.10.0"]);
+  });
+
+  test("cycles header sort direction without returning an undefined handler state", () => {
+    const versionSort = nextChangelogSortPreference(DEFAULT_CHANGELOG_SORT, "version");
+    expect(versionSort).toEqual({ columnId: "version", direction: "desc" });
+    expect(nextChangelogSortPreference(versionSort, "version"))
+      .toEqual({ columnId: "version", direction: "asc" });
+  });
+
+  test("ignores unknown header ids instead of creating broken sort state", () => {
+    expect(nextChangelogSortPreference(DEFAULT_CHANGELOG_SORT, "missing"))
+      .toBe(DEFAULT_CHANGELOG_SORT);
+  });
+
+  test("resolves selected rows by stable release id after sorting", () => {
+    const sorted = sortChangelogReleases(
+      releases,
+      { columnId: "title", direction: "asc" },
+    );
+
+    expect(sorted.map((entry) => entry.id)).toEqual(["10", "9", "8"]);
+    expect(resolveSelectedReleaseIndex(sorted, "8")).toBe(2);
+  });
+
+  test("falls back to the first release when the selected id disappears", () => {
+    expect(resolveSelectedReleaseIndex(releases, "removed")).toBe(0);
+    expect(resolveSelectedReleaseIndex([], "removed")).toBe(-1);
+  });
+});

package/src/plugins/builtin/changelog/model.ts

@@ -0,0 +1,98 @@
+import type { ChangelogRelease } from "../../../updater/github-releases";
+import type { SortDirection } from "../../../utils/sort-values";
+
+export type ChangelogColumnId = "date" | "version" | "title";
+
+export interface ChangelogSortPreference {
+  columnId: ChangelogColumnId;
+  direction: SortDirection;
+}
+
+export const DEFAULT_CHANGELOG_SORT: ChangelogSortPreference = {
+  columnId: "date",
+  direction: "desc",
+};
+
+function isChangelogColumnId(columnId: string): columnId is ChangelogColumnId {
+  return columnId === "date" || columnId === "version" || columnId === "title";
+}
+
+function defaultDirection(columnId: ChangelogColumnId): SortDirection {
+  return columnId === "title" ? "asc" : "desc";
+}
+
+function releaseDateValue(release: ChangelogRelease): number {
+  const timestamp = Date.parse(release.publishedAt);
+  return Number.isFinite(timestamp) ? timestamp : 0;
+}
+
+function compareText(left: string, right: string): number {
+  return left.localeCompare(right, undefined, {
+    numeric: true,
+    sensitivity: "base",
+  });
+}
+
+function compareReleaseValue(
+  left: ChangelogRelease,
+  right: ChangelogRelease,
+  columnId: ChangelogColumnId,
+): number {
+  switch (columnId) {
+    case "date":
+      return releaseDateValue(left) - releaseDateValue(right);
+    case "version":
+      return compareText(left.version, right.version);
+    case "title":
+      return compareText(left.title, right.title);
+  }
+}
+
+export function sortChangelogReleases(
+  releases: ChangelogRelease[],
+  sortPreference: ChangelogSortPreference,
+): ChangelogRelease[] {
+  return releases
+    .map((release, index) => ({ release, index }))
+    .sort((left, right) => {
+      const comparison = compareReleaseValue(
+        left.release,
+        right.release,
+        sortPreference.columnId,
+      );
+      const directed = sortPreference.direction === "asc"
+        ? comparison
+        : -comparison;
+      return directed || left.index - right.index;
+    })
+    .map((entry) => entry.release);
+}
+
+export function nextChangelogSortPreference(
+  current: ChangelogSortPreference,
+  columnId: string,
+): ChangelogSortPreference {
+  if (!isChangelogColumnId(columnId)) return current;
+  const nextColumnId = columnId;
+  if (current.columnId !== nextColumnId) {
+    return {
+      columnId: nextColumnId,
+      direction: defaultDirection(nextColumnId),
+    };
+  }
+  return {
+    columnId: nextColumnId,
+    direction: current.direction === "asc" ? "desc" : "asc",
+  };
+}
+
+export function resolveSelectedReleaseIndex(
+  releases: ChangelogRelease[],
+  selectedReleaseId: string | null,
+): number {
+  const selectedIndex = selectedReleaseId
+    ? releases.findIndex((release) => release.id === selectedReleaseId)
+    : -1;
+  if (selectedIndex >= 0) return selectedIndex;
+  return releases.length > 0 ? 0 : -1;
+}

package/src/plugins/builtin/chat/channel-labels.ts

@@ -0,0 +1,53 @@
+import type { ChatChannel } from "../../../api-client";
+import { normalizeChannelId } from "./controller/state";
+
+function normalizeMentionUsername(username: string | null | undefined): string | null {
+  const normalized = username?.trim().replace(/^@+/, "");
+  return normalized || null;
+}
+
+function formatMentionUsername(username: string | null | undefined): string | null {
+  const normalized = normalizeMentionUsername(username);
+  return normalized ? `@${normalized}` : null;
+}
+
+function formatDirectChannelLabel(channel: ChatChannel, fallbackId: string) {
+  const usernameLabel = formatMentionUsername(channel.dmUser?.username);
+  if (usernameLabel) return usernameLabel;
+
+  const displayName = channel.dmUser?.displayName?.trim();
+  if (displayName) return displayName;
+
+  const channelName = channel.name?.trim();
+  if (channelName && channelName !== channel.id) {
+    return channelName.startsWith("@")
+      ? formatMentionUsername(channelName) ?? channelName
+      : channelName;
+  }
+
+  return fallbackId.startsWith("dm:") ? "DM" : fallbackId;
+}
+
+export function formatChannelLabel(channel: ChatChannel | undefined, fallbackId: string) {
+  if (!channel) return fallbackId;
+  if (channel.kind === "direct") {
+    return formatDirectChannelLabel(channel, fallbackId);
+  }
+  if (channel.kind === "group") {
+    return channel.name?.trim() || "Group";
+  }
+  return channel.name?.trim() || fallbackId;
+}
+
+export function formatChatPaneTitle(channel: ChatChannel | undefined, fallbackId: string) {
+  const normalizedFallbackId = normalizeChannelId(fallbackId);
+  if (channel?.kind === "direct") {
+    return formatChannelLabel(channel, normalizedFallbackId);
+  }
+  if (channel?.kind === "group") {
+    return formatChannelLabel(channel, normalizedFallbackId);
+  }
+  if (!channel && normalizedFallbackId.startsWith("dm:")) return "DM";
+  if (!channel && (normalizedFallbackId.startsWith("grp:") || normalizedFallbackId.startsWith("group:"))) return "Group";
+  return `#${formatChannelLabel(channel, normalizedFallbackId)}`;
+}

package/src/plugins/builtin/chat/channels.test.ts

@@ -0,0 +1,28 @@
+import { describe, expect, test } from "bun:test";
+import type { ChatChannel } from "../../../api-client";
+import {
+  channelPrefix,
+  formatChannelLabel,
+  formatChatPaneTitle,
+} from "./channels";
+
+describe("chat channel labels", () => {
+  test("formats direct-message labels and titles with one mention prefix", () => {
+    const channel: ChatChannel = {
+      id: "dm:c7f89ce26f9da83d6be08a4838738074",
+      name: "@risto",
+      kind: "direct",
+      created_at: "2026-05-27T10:30:03.712Z",
+      dmUser: { id: "u2", username: "@risto", displayName: "Risto" },
+    };
+
+    expect(formatChannelLabel(channel, channel.id)).toBe("@risto");
+    expect(channelPrefix(channel, true)).toBe(" ");
+    expect(formatChatPaneTitle(channel, channel.id)).toBe("@risto");
+    expect(formatChatPaneTitle(undefined, channel.id)).toBe("DM");
+  });
+
+  test("uses a neutral group title before private channel metadata loads", () => {
+    expect(formatChatPaneTitle(undefined, "grp:93f14c6b9827c30f")).toBe("Group");
+  });
+});