taskchef 7.24.0 → 7.24.1

package/src/dashboard/github-links.js

@@ -60,12 +60,20 @@ function compareReferenceNumbers(left, right) {
 
 export function groupRelatedGitHubLinks(links) {
   const groups = new Map();
-  const seen = new Set();
+  const seenLabels = new Set();
+  const seenNumbers = new Set();
+  const seenUrls = new Set();
   for (const link of links ?? []) {
-    if (link.type === "repository" || seen.has(link.label)) continue;
-    seen.add(link.label);
     const repository = repositoryIdentity(link);
-    if (!repository) continue;
+    const reference = link.number ? `${repository}#${link.number}` : null;
+    if (link.type === "repository"
+      || !repository
+      || seenLabels.has(link.label)
+      || seenUrls.has(link.url)
+      || (reference && seenNumbers.has(reference))) continue;
+    seenLabels.add(link.label);
+    seenUrls.add(link.url);
+    if (reference) seenNumbers.add(reference);
     const group = groups.get(repository) ?? [];
     group.push(link);
     groups.set(repository, group);

package/src/dashboard/react/components/ActivityTimeline.tsx

@@ -3,6 +3,7 @@ import { mergeProjectedTurns, turnPresentation } from "../../state.js";
 import { turnUsageView } from "../presentation";
 import type { Task } from "../types";
 import { RelativeTime } from "./RelativeTime";
+import { LinkedText } from "./LinkedText";
 import { ShimmerText } from "./ShimmerText";
 import { StatusBadge } from "./StatusBadge";
 
@@ -34,14 +35,16 @@ export function ActivityTimeline({ highlightTurnRef, task }: { highlightTurnRef:
               {presentation.sourceLabel && <Text c="dimmed" size="xs">{presentation.sourceLabel}</Text>}
               <Box>
                 <Text c="dimmed" className="taskchef-field-label" size="xs">Request</Text>
-                <Text className="taskchef-preserve-lines" size="sm">{turn.requestSummary ?? "Request not recorded."}</Text>
+                <Text className="taskchef-preserve-lines" size="sm">
+                  <LinkedText task={task} text={turn.requestSummary ?? "Request not recorded."} />
+                </Text>
               </Box>
               <Box>
                 <Text c="dimmed" className="taskchef-field-label" size="xs">Result</Text>
                 <Text className="taskchef-preserve-lines" size="sm">
                   {presentation.status === "working"
                     ? <ShimmerText>{presentation.summary}</ShimmerText>
-                    : presentation.summary}
+                    : <LinkedText task={task} text={presentation.summary} />}
                 </Text>
               </Box>
               <Text

package/src/dashboard/react/components/GitHubLinks.tsx

@@ -1,25 +1,46 @@
-import { Anchor, Group, Text } from "@mantine/core";
-import type { Task } from "../types";
+import { Anchor, Box, Group, Text } from "@mantine/core";
+import {
+  githubReferenceAccessibleLabel,
+  githubReferenceDisplayLabels,
+  groupRelatedGitHubLinks,
+} from "../../github-links.js";
+import type { GitHubLink, Task } from "../types";
 
 export function GitHubLinks({ task }: { task: Task }) {
   const links = task.relatedGitHubLinks ?? [];
   if (links.length === 0 && !task.relatedGitHubLinksTruncated) return null;
+  const groups = groupRelatedGitHubLinks(links) as GitHubLink[][];
+  const displayLabels = githubReferenceDisplayLabels(groups.flat(), { related: true }) as string[];
+  let linkIndex = 0;
   return (
-    <Group aria-label={`Related GitHub links for ${task.title}`} component="nav" gap={5} wrap="wrap">
-      {links.map((link) => (
-        <Anchor
-          aria-label={`${link.label}, GitHub ${link.type === "pull" ? "pull request" : link.type ?? "link"} (opens in a new tab)`}
-          className="taskchef-github-link"
-          href={link.url}
-          key={link.url}
-          rel="noopener noreferrer"
-          size="xs"
-          target="_blank"
-        >
-          {link.label}
-        </Anchor>
+    <Box aria-label={`Related GitHub links for ${task.title}`} className="taskchef-github-groups" component="nav">
+      {groups.map((group) => (
+        <Group className="taskchef-github-group" gap={5} key={`${group[0].owner}/${group[0].repository}`} wrap="wrap">
+          {group.map((link) => {
+            const displayLabel = displayLabels[linkIndex];
+            linkIndex += 1;
+            const kind = link.type === "pull"
+              ? ", GitHub pull request"
+              : link.type === "issue"
+                ? ", GitHub issue"
+                : " on GitHub";
+            return (
+              <Anchor
+                aria-label={`${githubReferenceAccessibleLabel(link)}${kind} (opens in a new tab)`}
+                className="taskchef-github-link"
+                href={link.url}
+                key={link.url}
+                rel="noopener noreferrer"
+                size="xs"
+                target="_blank"
+              >
+                {displayLabel}
+              </Anchor>
+            );
+          })}
+        </Group>
       ))}
       {task.relatedGitHubLinksTruncated && <Text c="dimmed" size="xs">More links in task history</Text>}
-    </Group>
+    </Box>
   );
 }

package/src/dashboard/react/components/LinkedText.tsx

@@ -0,0 +1,63 @@
+import { Anchor } from "@mantine/core";
+import {
+  githubReferenceAccessibleLabel,
+  githubReferenceDisplayLabels,
+  referenceSegments,
+} from "../../github-links.js";
+import type { Task } from "../types";
+
+interface ReferenceSegment {
+  kind: "ambiguous" | "link" | "text";
+  label?: string;
+  owner?: string;
+  reason?: string;
+  repository?: string;
+  text: string;
+  type?: "generic" | "issue" | "pull" | "repository";
+  url?: string;
+}
+
+export function LinkedText({ task, text }: { task: Task; text: string }) {
+  const segments = referenceSegments(text, {
+    projectRepositories: task.project.githubRepos,
+    taskRepository: task.relatedGitHubRepository,
+  }) as ReferenceSegment[];
+  const links = segments.filter((segment) => segment.kind === "link");
+  const displayLabels = githubReferenceDisplayLabels(links) as string[];
+  let linkIndex = 0;
+  return segments.map((segment, index) => {
+    if (segment.kind === "text") return segment.text;
+    if (segment.kind === "ambiguous") {
+      return (
+        <span
+          aria-label={segment.reason}
+          className="taskchef-github-reference-ambiguous"
+          key={`${index}:${segment.text}`}
+          tabIndex={0}
+          title={segment.reason}
+        >
+          {segment.text}
+        </span>
+      );
+    }
+    const displayLabel = displayLabels[linkIndex];
+    linkIndex += 1;
+    const kind = segment.type === "pull"
+      ? ", GitHub pull request"
+      : segment.type === "issue"
+        ? ", GitHub issue"
+        : segment.owner ? " on GitHub" : "";
+    return (
+      <Anchor
+        aria-label={`${githubReferenceAccessibleLabel(segment)}${kind} (opens in a new tab)`}
+        className="taskchef-github-link"
+        href={segment.url}
+        key={`${index}:${segment.url}`}
+        rel="noopener noreferrer"
+        target="_blank"
+      >
+        {displayLabel}
+      </Anchor>
+    );
+  });
+}

package/src/dashboard/react/components/TaskCard.tsx

@@ -3,6 +3,7 @@ import { IconExternalLink } from "@tabler/icons-react";
 import { latestTurnPresentation } from "../../state.js";
 import type { Task } from "../types";
 import { GitHubLinks } from "./GitHubLinks";
+import { LinkedText } from "./LinkedText";
 import { RelativeTime } from "./RelativeTime";
 import { ShimmerText } from "./ShimmerText";
 import { StatusBadge } from "./StatusBadge";
@@ -34,12 +35,14 @@ export function TaskCard({
 
           <Box className="taskchef-summary-grid">
             <Text c="dimmed" className="taskchef-field-label" size="xs">Request</Text>
-            <Text className="taskchef-preserve-lines" lineClamp={3} size="sm">{latest.requestSummary}</Text>
+            <Text className="taskchef-preserve-lines" lineClamp={3} size="sm">
+              <LinkedText task={task} text={latest.requestSummary} />
+            </Text>
             <Text c="dimmed" className="taskchef-field-label" size="xs">Result</Text>
             <Text className="taskchef-preserve-lines" lineClamp={3} size="sm">
               {latest.resultStatus === "working"
                 ? <ShimmerText>{latest.resultSummary}</ShimmerText>
-                : latest.resultSummary}
+                : <LinkedText task={task} text={latest.resultSummary} />}
             </Text>
           </Box>
 

package/src/dashboard/react/components/UsagePanel.tsx

@@ -1,7 +1,12 @@
 import { Box, Group, Paper, Text } from "@mantine/core";
 import { IconCoins } from "@tabler/icons-react";
 import type { Task } from "../types";
-import { usageView } from "../presentation";
+import {
+  formatCompactTokens,
+  formatEstimatedCost,
+  formatFullTokens,
+  usageView,
+} from "../presentation";
 import { ShimmerText } from "./ShimmerText";
 
 export function UsagePanel({ task }: { task: Task }) {
@@ -22,18 +27,45 @@ export function UsagePanel({ task }: { task: Task }) {
   const cached = usage.usage.cachedInputTokens ?? 0;
   const input = usage.usage.inputTokens ?? 0;
   const cacheRatio = input + cached > 0 ? `${Math.round((cached / (input + cached)) * 100)}%` : "—";
-  const values = [
-    [usage.knownSoFar ? "Tokens · known so far" : "Tokens", new Intl.NumberFormat().format(usage.usage.totalTokens)],
-    ["Estimated cost", usage.usage.estimatedCostUsd == null ? "—" : `$${usage.usage.estimatedCostUsd.toFixed(4)}`],
-    ["Model", Object.keys(usage.usage.models ?? {}).join(", ") || "—"],
-    ["Cache ratio", cacheRatio],
-  ] as const;
+  const values: Array<{
+    accessibleValue?: string;
+    label: string;
+    title?: string;
+    value: string;
+  }> = [
+    {
+      accessibleValue: `${formatFullTokens(usage.usage.totalTokens)} tokens`,
+      label: usage.knownSoFar ? "Tokens · known so far" : "Tokens",
+      title: `${formatFullTokens(usage.usage.totalTokens)} tokens`,
+      value: formatCompactTokens(usage.usage.totalTokens),
+    },
+    {
+      accessibleValue: usage.usage.estimatedCostUsd == null
+        ? "Estimated cost unavailable"
+        : `Estimated cost ${formatEstimatedCost(usage.usage.estimatedCostUsd)}`,
+      label: "Estimated cost",
+      title: usage.usage.estimatedCostUsd == null
+        ? undefined
+        : `Unrounded estimate: $${usage.usage.estimatedCostUsd}`,
+      value: formatEstimatedCost(usage.usage.estimatedCostUsd),
+    },
+    { label: "Model", value: Object.keys(usage.usage.models ?? {}).join(", ") || "—" },
+    { label: "Cache ratio", value: cacheRatio },
+  ];
   return (
     <Box aria-live="polite" className="taskchef-metrics">
-      {values.map(([label, value]) => (
+      {values.map(({ accessibleValue, label, title, value }) => (
         <Paper className="taskchef-metric" key={label} p="sm" radius="md" withBorder>
           <Text c="dimmed" size="xs">{label}</Text>
-          <Text className="taskchef-metric-value" fw={650} size="sm">{value}</Text>
+          <Text
+            aria-label={accessibleValue}
+            className="taskchef-metric-value"
+            fw={650}
+            size="sm"
+            title={title}
+          >
+            {value}
+          </Text>
         </Paper>
       ))}
     </Box>

package/src/dashboard/react/presentation.test.tsx

@@ -1,10 +1,17 @@
 import { MantineProvider } from "@mantine/core";
-import { render, screen } from "@testing-library/react";
+import { render, screen, within } from "@testing-library/react";
 import { describe, expect, test } from "vitest";
 import { fixtureTask } from "./fixtures";
-import { turnUsageView, usageStillCalculating, usageView } from "./presentation";
+import {
+  formatCompactTokens,
+  formatEstimatedCost,
+  turnUsageView,
+  usageStillCalculating,
+  usageView,
+} from "./presentation";
 import { ShimmerText } from "./components/ShimmerText";
 import { GitHubLinks } from "./components/GitHubLinks";
+import { LinkedText } from "./components/LinkedText";
 import { UsagePanel } from "./components/UsagePanel";
 
 describe("token and working presentation", () => {
@@ -21,7 +28,7 @@ describe("token and working presentation", () => {
         status: "available",
         task: { totalTokens: 12345, estimatedCostUsd: 0.1234 },
       },
-    })).label).toBe("12,345 tokens · Estimated cost $0.1234");
+    })).label).toBe("12.35K tokens · Estimated cost $0.12");
     expect(usageView(fixtureTask({ status: "completed", usage: { status: "unavailable" } })).label)
       .toBe("Token usage unavailable");
     expect(usageView(fixtureTask({
@@ -43,7 +50,7 @@ describe("token and working presentation", () => {
       },
     }));
     expect(view.kind).toBe("ready");
-    expect(view.label).toBe("12,345 tokens · Estimated cost $0.1234 · known so far");
+    expect(view.label).toBe("12.35K tokens · Estimated cost $0.12 · known so far");
     if (view.kind === "ready") expect(view.knownSoFar).toBe(true);
   });
 
@@ -64,7 +71,7 @@ describe("token and working presentation", () => {
         updatedAt: task.updatedAt,
       },
     };
-    expect(turnUsageView(task, completedTurn).label).toBe("330 tokens · Estimated cost $0.1200");
+    expect(turnUsageView(task, completedTurn).label).toBe("330 tokens · Estimated cost $0.12");
     expect(turnUsageView(task, { ...completedTurn, result: null }).label).toBe("Turn usage pending");
     expect(turnUsageView({
       ...task,
@@ -115,7 +122,7 @@ describe("token and working presentation", () => {
     );
 
     expect(screen.getByText("Tokens").nextSibling).toHaveTextContent("330");
-    expect(screen.getByText("Estimated cost").nextSibling).toHaveTextContent("$0.1200");
+    expect(screen.getByText("Estimated cost").nextSibling).toHaveTextContent("$0.12");
     expect(screen.getByText("Model").nextSibling).toHaveTextContent("gpt-5.6-sol, gpt-5.6-luna");
     expect(screen.getByText("Cache ratio").nextSibling).toHaveTextContent("67%");
     expect(screen.queryByText(/API-equivalent/i)).not.toBeInTheDocument();
@@ -125,4 +132,110 @@ describe("token and working presentation", () => {
     render(<MantineProvider><GitHubLinks task={fixtureTask()} /></MantineProvider>);
     expect(screen.getByRole("link")).toHaveClass("taskchef-github-link");
   });
+
+  test("formats compact token boundaries without scientific notation", () => {
+    expect([
+      0,
+      999,
+      1_000,
+      1_100,
+      12_345,
+      999_999,
+      1_320_000,
+      1_000_000_000,
+    ].map((value) => formatCompactTokens(value, "en-US"))).toEqual([
+      "0",
+      "999",
+      "1K",
+      "1.1K",
+      "12.35K",
+      "1M",
+      "1.32M",
+      "1B",
+    ]);
+  });
+
+  test("formats estimates to two decimals across zero, sub-cent, missing, and large values", () => {
+    expect(formatEstimatedCost(0, "en-US")).toBe("$0.00");
+    expect(formatEstimatedCost(0.0049, "en-US")).toBe("$0.00");
+    expect(formatEstimatedCost(12.344, "en-US")).toBe("$12.34");
+    expect(formatEstimatedCost(12.345, "en-US")).toBe("$12.35");
+    expect(formatEstimatedCost(null, "en-US")).toBe("—");
+    expect(formatEstimatedCost(Number.NaN, "en-US")).toBe("—");
+    expect(formatEstimatedCost(1_234_567_890.125, "en-US")).toBe("$1,234,567,890.13");
+  });
+
+  test("exposes full token precision and the unrounded estimate on ready metric cards", () => {
+    render(
+      <MantineProvider>
+        <UsagePanel task={fixtureTask({
+          status: "completed",
+          usage: {
+            status: "available",
+            task: { totalTokens: 1_324_567, estimatedCostUsd: 12.3449 },
+          },
+        })} />
+      </MantineProvider>,
+    );
+    expect(screen.getByLabelText("1,324,567 tokens")).toHaveTextContent("1.32M");
+    expect(screen.getByLabelText("1,324,567 tokens")).toHaveAttribute("title", "1,324,567 tokens");
+    expect(screen.getByLabelText("Estimated cost $12.34")).toHaveTextContent("$12.34");
+    expect(screen.getByLabelText("Estimated cost $12.34")).toHaveAttribute(
+      "title",
+      "Unrounded estimate: $12.3449",
+    );
+  });
+
+  test("groups, deduplicates, and naturally sorts compact related links by repository", () => {
+    const task = fixtureTask({
+      relatedGitHubLinks: [
+        ["taskchef", "83", "pull"],
+        ["guzuoshou-workspace", "124", "issue"],
+        ["taskchef", "79", "issue"],
+        ["guzuoshou-workspace", "108", "issue"],
+        ["taskchef", "80", "issue"],
+        ["guzuoshou-workspace", "109", "pull"],
+        ["taskchef", "79", "pull"],
+      ].map(([repository, number, type]) => ({
+        label: `favoyang/${repository}#${number}`,
+        number,
+        owner: "favoyang",
+        repository,
+        type: type as "issue" | "pull",
+        url: `https://github.com/favoyang/${repository}/${type === "pull" ? "pull" : "issues"}/${number}`,
+      })),
+    });
+    const { container } = render(<MantineProvider><GitHubLinks task={task} /></MantineProvider>);
+    const groups = container.querySelectorAll(".taskchef-github-group");
+    expect(groups).toHaveLength(2);
+    expect(within(groups[0] as HTMLElement).getAllByRole("link").map((link) => link.textContent))
+      .toEqual(["taskchef #79", "#80", "#83"]);
+    expect(within(groups[1] as HTMLElement).getAllByRole("link").map((link) => link.textContent))
+      .toEqual(["guzuoshou-workspace #108", "#109", "#124"]);
+    const first = screen.getByRole("link", { name: /favoyang\/taskchef Issue #79/ });
+    expect(first).toHaveAttribute("href", "https://github.com/favoyang/taskchef/issues/79");
+    expect(first).toHaveAttribute("target", "_blank");
+    expect(first).toHaveAttribute("rel", "noopener noreferrer");
+    expect(screen.queryByText(/PR #|Issue #/)).not.toBeInTheDocument();
+  });
+
+  test("keeps established inline request and result link labels", () => {
+    const task = fixtureTask({
+      project: {
+        name: "TaskChef Preview",
+        path: "/tmp/taskchef-preview-project",
+        githubRepos: ["https://github.com/favoyang/taskchef"],
+      },
+      relatedGitHubRepository: "favoyang/taskchef",
+    });
+    const { container } = render(
+      <MantineProvider>
+        <div>
+          <LinkedText task={task} text="Review https://github.com/favoyang/taskchef/pull/83." />
+        </div>
+      </MantineProvider>,
+    );
+    expect(within(container).getByRole("link", { name: /favoyang\/taskchef PR #83/ }))
+      .toHaveTextContent("taskchef PR #83");
+  });
 });

package/src/dashboard/react/presentation.ts

@@ -1,5 +1,27 @@
 import type { Task, UsageNumbers, UsageProjection } from "./types";
 
+export function formatCompactTokens(value: number, locales?: Intl.LocalesArgument) {
+  return new Intl.NumberFormat(locales, {
+    maximumFractionDigits: 2,
+    notation: "compact",
+  }).format(value);
+}
+
+export function formatFullTokens(value: number, locales?: Intl.LocalesArgument) {
+  return new Intl.NumberFormat(locales, { maximumFractionDigits: 2 }).format(value);
+}
+
+export function formatEstimatedCost(value: number | null | undefined, locales?: Intl.LocalesArgument) {
+  if (typeof value !== "number" || !Number.isFinite(value)) return "—";
+  return new Intl.NumberFormat(locales, {
+    currency: "USD",
+    currencyDisplay: "narrowSymbol",
+    maximumFractionDigits: 2,
+    minimumFractionDigits: 2,
+    style: "currency",
+  }).format(value);
+}
+
 export type UsageView =
   | { kind: "pending"; label: "Token usage pending" }
   | { kind: "calculating"; label: "Calculating token usage" }
@@ -29,10 +51,10 @@ export function usageView(task: Task): UsageView {
 }
 
 function readyUsageView(usage: UsageNumbers, knownSoFar: boolean): UsageView {
-  const tokens = new Intl.NumberFormat().format(usage.totalTokens);
+  const tokens = formatCompactTokens(usage.totalTokens);
   const cost = usage.estimatedCostUsd == null
     ? "Estimated cost unavailable"
-    : `Estimated cost $${usage.estimatedCostUsd.toFixed(4)}`;
+    : `Estimated cost ${formatEstimatedCost(usage.estimatedCostUsd)}`;
   return {
     kind: "ready",
     knownSoFar,
@@ -49,10 +71,10 @@ export function turnUsageView(task: Task, turn: NonNullable<Task["turns"]>[numbe
     return { kind: "calculating" as const, label: "Calculating turn usage" };
   }
   if (usage?.status === "available" && typeof usage.totalTokens === "number") {
-    const tokens = new Intl.NumberFormat().format(usage.totalTokens);
+    const tokens = formatFullTokens(usage.totalTokens);
     const cost = usage.estimatedCostUsd == null
       ? "Estimated cost unavailable"
-      : `Estimated cost $${usage.estimatedCostUsd.toFixed(4)}`;
+      : `Estimated cost ${formatEstimatedCost(usage.estimatedCostUsd)}`;
     return { kind: "ready" as const, label: `${tokens} tokens · ${cost}` };
   }
   return {

package/src/dashboard/react/styles.css

@@ -48,6 +48,11 @@ body {
 .taskchef-preserve-lines { white-space: pre-wrap; overflow-wrap: anywhere; }
 .taskchef-github-link { color: var(--taskchef-accent); text-decoration: none; }
 .taskchef-github-link:hover { text-decoration: underline; text-underline-offset: 3px; }
+.taskchef-github-link:focus-visible { outline: 2px solid var(--mantine-color-teal-6); outline-offset: 2px; border-radius: 2px; }
+.taskchef-github-groups { display: grid; min-width: 0; gap: 3px; }
+.taskchef-github-group { min-width: 0; }
+.taskchef-github-reference-ambiguous { border-bottom: 2px dotted var(--mantine-color-yellow-7); cursor: help; }
+.taskchef-github-reference-ambiguous:focus-visible { outline: 2px solid var(--mantine-color-yellow-7); outline-offset: 2px; }
 .taskchef-time { display: inline-flex; align-items: center; gap: 4px; width: fit-content; padding: 2px 3px; border-radius: 4px; color: var(--taskchef-muted); font-size: 0.7rem; line-height: 1.2; }
 .taskchef-time:hover { color: var(--taskchef-text); background: var(--taskchef-surface-muted); }
 .taskchef-time:focus-visible { outline: 2px solid var(--mantine-color-teal-6); outline-offset: 2px; }