diff --git a/packages/dashboard/app/components/TaskSummaryTab.tsx b/packages/dashboard/app/components/TaskSummaryTab.tsx
index a30ebeab0f..24edbb852c 100644
--- a/packages/dashboard/app/components/TaskSummaryTab.tsx
+++ b/packages/dashboard/app/components/TaskSummaryTab.tsx
@@ -7,6 +7,7 @@ import type { TaskDetail, TaskStep, WorkflowStepResult } from "@fusion/core";
import type { ModelPricingOverrides } from "../../../core/src/model-pricing";
import { createMermaidCodeComponent, sharedRehypePlugins } from "./markdownPipeline";
import { ProviderIcon } from "./ProviderIcon";
+import { MergeDetails } from "./MergeDetails";
import { linkifyFilePaths, linkifyReactChildren } from "../utils/filePathLinkify";
import { inferProviderIconKey } from "../utils/providerIconKey";
import { buildTokenCostRows, formatCost, formatCount, totalCostForRows } from "../utils/taskTokenCost";
@@ -170,6 +171,14 @@ export function TaskSummaryTab({ task, pricingOverrides }: TaskSummaryTabProps)
) : null}
+ {/*
+ FNXC:TaskDetailSummaryTab 2026-07-29-00:00:
+ FN-8197 / issue #2248 moves Merge Details from Definition into the done-only Summary tab. Both
+ surfaces apply only to completed tasks, and merge status, PR, and commit-message metadata belong
+ with completion data rather than the task plan.
+ */}
+
+
{hasAgentWork ? (
{t("taskDetail.summaryTab.agentWorkHeading", "Work done by agents")}
diff --git a/packages/dashboard/app/components/__tests__/TaskDetailModal.responsive-and-dependencies.test.tsx b/packages/dashboard/app/components/__tests__/TaskDetailModal.responsive-and-dependencies.test.tsx
index 0fd566bedc..68e00a030e 100644
--- a/packages/dashboard/app/components/__tests__/TaskDetailModal.responsive-and-dependencies.test.tsx
+++ b/packages/dashboard/app/components/__tests__/TaskDetailModal.responsive-and-dependencies.test.tsx
@@ -1557,23 +1557,24 @@ describe("TaskDetailModal", () => {
expect(screen.getByRole("link", { name: "#42" })).toHaveAttribute("href", "https://github.com/owner/repo/pull/42");
});
- it("shows linked PR number in merge details for done tasks", () => {
- render(
+ it("shows linked PR number in Summary merge details, not Definition, for done tasks", () => {
+ const task = makeTask({
+ column: "done" as Column,
+ prInfo: {
+ url: "https://github.com/owner/repo/pull/42",
+ number: 42,
+ status: "merged",
+ title: "Task",
+ headBranch: "fusion/fn-099",
+ baseBranch: "main",
+ commentCount: 0,
+ },
+ mergeDetails: { prNumber: 42 },
+ });
+ const summary = render(
{
/>,
);
- const links = screen.getAllByRole("link", { name: "#42" });
- expect(links.length).toBeGreaterThan(0);
- expect(links[0]).toHaveAttribute("href", "https://github.com/owner/repo/pull/42");
+ expect(summary.container.querySelector(".merge-details-card a")).toHaveAttribute("href", "https://github.com/owner/repo/pull/42");
+ summary.unmount();
+
+ render(
+ ,
+ );
+
+ expect(screen.queryByText("Merge Details")).toBeNull();
});
it("shows PR automation waiting label instead of Merge & Close when awaiting PR checks", () => {
diff --git a/packages/dashboard/app/components/__tests__/TaskDetailModal.summary-tab.test.tsx b/packages/dashboard/app/components/__tests__/TaskDetailModal.summary-tab.test.tsx
index 8b502b4a6b..5c2ad73111 100644
--- a/packages/dashboard/app/components/__tests__/TaskDetailModal.summary-tab.test.tsx
+++ b/packages/dashboard/app/components/__tests__/TaskDetailModal.summary-tab.test.tsx
@@ -455,6 +455,71 @@ describe("TaskDetailModal Summary tab", () => {
expect(css).not.toMatch(/task-summary-token[^{}]*#[0-9a-fA-F]{3,8}/);
});
+ it("renders every Merge Details data state inside the done-only Summary tab", () => {
+ const linkedTask = doneTask({
+ mergeDetails: {
+ commitSha: "abcdef1234567890",
+ mergeConfirmed: true,
+ prNumber: 42,
+ mergedAt: "2026-07-29T12:00:00Z",
+ mergeCommitMessage: "feat: land summary merge details",
+ },
+ prInfo: {
+ url: "https://github.com/owner/repo/pull/42",
+ number: 42,
+ status: "merged",
+ title: "Task",
+ headBranch: "fusion/fn-8197",
+ baseBranch: "main",
+ commentCount: 0,
+ },
+ });
+ const view = render();
+
+ const summary = screen.getByTestId("task-summary-tab");
+ const mergeCard = summary.querySelector(".merge-details-card");
+ expect(screen.getByText("Merge Details")).toBeTruthy();
+ expect(screen.getByText("Merged successfully")).toBeTruthy();
+ expect(screen.getByText("Merged at")).toBeTruthy();
+ expect(screen.getByText("feat: land summary merge details")).toBeTruthy();
+ expect(screen.getByRole("link", { name: "#42" })).toHaveAttribute("href", "https://github.com/owner/repo/pull/42");
+ expect(mergeCard?.classList.contains("pr-card")).toBe(true);
+ expect(summary.contains(mergeCard)).toBe(true);
+
+ view.rerender(
+ ,
+ );
+ expect(screen.getByText("Recorded without local merge confirmation")).toBeTruthy();
+ expect(screen.getByText("#7")).toBeTruthy();
+ expect(screen.queryByRole("link", { name: "#7" })).toBeNull();
+ expect(screen.queryByText("Merged at")).toBeNull();
+ expect(screen.queryByText("Message")).toBeNull();
+
+ view.rerender();
+ expect(screen.queryByText("PR")).toBeNull();
+ expect(screen.queryByText("Merged at")).toBeNull();
+ expect(screen.queryByText("Message")).toBeNull();
+
+ view.rerender();
+ expect(screen.getByTestId("task-summary-tab")).toBeTruthy();
+ expect(screen.queryByText("Merge Details")).toBeNull();
+ });
+
+ it("keeps relocated Merge Details inside the existing mobile pr-card containment", () => {
+ const { container } = render();
+ const summary = screen.getByTestId("task-summary-tab");
+ const mergeCard = container.querySelector(".task-summary-tab .merge-details-card.pr-card");
+
+ expect(mergeCard).toBeTruthy();
+ expect(summary.contains(mergeCard)).toBe(true);
+ expect(readDashboardStylesSource()).toMatch(/@media \(max-width: 768px\)[\s\S]*\.pr-card/);
+ });
+
it("renders graceful empty states without orphaned changed-file headings", () => {
render(