From 48f0353b7fc1b9a51067fa3b0b62f4a8d703351b Mon Sep 17 00:00:00 2001
From: Fusion <noreply@runfusion.ai>
Date: Sun, 10 May 2026 12:05:59 -0700
Subject: [PATCH] feat(FN-3943): align tracking resolution to honor task-level
 override seman

Aligned GitHub tracking resolution to resolve task-level overrides, with regression tests covering the create-flow path and documentation of the override semantics.

Fusion-Task-Id: FN-3943
---
 docs/missions.md                              |   1 +
 docs/task-management.md                       |   1 +
 .../app/components/MissionManager.css         |  21 +++
 .../app/components/MissionManager.tsx         | 101 ++++++++++----
 .../__tests__/MissionManager.test.tsx         | 126 ++++++++++++++++++
 .../src/__tests__/github-tracking.test.ts     |  13 ++
 .../src/__tests__/routes-tasks.test.ts        |  37 +++++
 packages/dashboard/src/github-tracking.ts     |  26 ++--
 8 files changed, 282 insertions(+), 44 deletions(-)

diff --git a/docs/missions.md b/docs/missions.md
index e850e6fc7..1877f84b9 100644
--- a/docs/missions.md
+++ b/docs/missions.md
@@ -49,6 +49,7 @@ The dashboard supports mission planning workflows where you can:
 - Break work into milestones/slices/features
 - Associate features to executable tasks
 - Track progress at each layer
+- Resume in-progress mission interview sessions directly from the main missions list (generating, awaiting input, or error) before a mission record is created
 
 ### Auto-Generated Assertions
 
diff --git a/docs/task-management.md b/docs/task-management.md
index 92ea2402b..0e4bd98ae 100644
--- a/docs/task-management.md
+++ b/docs/task-management.md
@@ -389,6 +389,7 @@ Tracking behavior is controlled per task:
 - `task.githubTracking.repoOverride` optionally forces a specific target repo (`owner/repo`).
 - In the dashboard **Task Detail** modal, eligible existing tasks (`triage`, `todo`, `in-progress`, `in-review`) always show GitHub tracking controls so tracking can be enabled, disabled, or retargeted without reopening the task in a creation flow.
 - Clearing the Task Detail repo override stores `null`, which reverts repo resolution to project/global defaults.
+- Explicit task-level enablement is honored even when project/global GitHub tracking defaults are unset. If `enabled: true` and the repo resolves at task scope (for example via `repoOverride`), Fusion still attempts create-time tracking-issue creation.
 
 Repository resolution order:
 
diff --git a/packages/dashboard/app/components/MissionManager.css b/packages/dashboard/app/components/MissionManager.css
index 3a74d5a28..526cd826e 100644
--- a/packages/dashboard/app/components/MissionManager.css
+++ b/packages/dashboard/app/components/MissionManager.css
@@ -722,6 +722,27 @@
   box-shadow: inset 0 0 0 1px var(--todo);
 }
 
+.mission-list__item--interview {
+  border-style: dashed;
+}
+
+.mission-interview-status {
+  background: color-mix(in srgb, var(--text-muted) 16%, transparent);
+  color: var(--text);
+}
+
+.mission-interview-status--generating {
+  background: color-mix(in srgb, var(--color-warning) 14%, transparent);
+}
+
+.mission-interview-status--awaiting_input {
+  background: color-mix(in srgb, var(--todo) 16%, transparent);
+}
+
+.mission-interview-status--error {
+  background: color-mix(in srgb, var(--color-error) 12%, transparent);
+}
+
 /* Desktop split-only selected-state background keeps focus contrast in sidebar cards. */
 .mission-manager__sidebar .mission-list__item--selected {
   background: var(--card);
diff --git a/packages/dashboard/app/components/MissionManager.tsx b/packages/dashboard/app/components/MissionManager.tsx
index 68cae54ce..051d01742 100644
--- a/packages/dashboard/app/components/MissionManager.tsx
+++ b/packages/dashboard/app/components/MissionManager.tsx
@@ -24,7 +24,6 @@ import {
   Activity,
   FileText,
   RefreshCw,
-  AlertCircle,
 } from "lucide-react";
 import type { ToastType } from "../hooks/useToast";
 import { useViewportMode } from "../hooks/useViewportMode";
@@ -174,6 +173,28 @@ const validationStateColors: Record<string, { bg: string; text: string }> = {
 };
 
 const featureRetryBudgetMax = 3;
+const missionInterviewListStatuses: ReadonlySet<AiSessionSummary["status"]> = new Set([
+  "generating",
+  "awaiting_input",
+  "error",
+]);
+
+function getInterviewStatusLabel(status: AiSessionSummary["status"]): string {
+  switch (status) {
+    case "generating":
+      return "Generating plan";
+    case "awaiting_input":
+      return "Awaiting input";
+    case "error":
+      return "Needs retry";
+    default:
+      return status;
+  }
+}
+
+function getInterviewActionLabel(status: AiSessionSummary["status"]): string {
+  return status === "error" ? "Retry interview" : "Resume interview";
+}
 
 /** Get the plan state for a milestone (derived from interviewState) */
 function getMilestonePlanState(interviewState?: string): "not_started" | "planned" | "needs_update" {
@@ -576,7 +597,7 @@ export function MissionManager({ isOpen, isInline = false, onClose, addToast, pr
     fetchAiSessions(projectId).then((sessions) => {
       if (cancelled) return;
       const pending = sessions.filter(
-        (s) => s.type === "mission_interview" && (s.status === "awaiting_input" || s.status === "error"),
+        (s) => s.type === "mission_interview" && missionInterviewListStatuses.has(s.status),
       );
       setPendingInterviewSessions(pending);
     }).catch((err) => {
@@ -3472,6 +3493,53 @@ export function MissionManager({ isOpen, isInline = false, onClose, addToast, pr
     );
   };
 
+  const handleResumeInterviewSession = (sessionId: string) => {
+    setLocalResumeSessionId(sessionId);
+    setShowInterviewModal(true);
+  };
+
+  const renderInterviewSessionItems = () => pendingInterviewSessions.map((session) => {
+    const isErrored = session.status === "error";
+    const isGenerating = session.status === "generating";
+
+    return (
+      <div
+        key={session.id}
+        className="mission-list__item mission-list__item--interview"
+        onClick={() => handleResumeInterviewSession(session.id)}
+      >
+        <div className="mission-list__item-content">
+          <div className="mission-list__item-header">
+            <Sparkles size={16} className="mission-list__item-icon" />
+            <span className="mission-list__item-title">{session.title || "Mission interview"}</span>
+          </div>
+          <div className="mission-list__item-tags">
+            <span className={`mission-status-badge mission-status-badge--sm mission-interview-status mission-interview-status--${session.status}`}>
+              {getInterviewStatusLabel(session.status)}
+            </span>
+          </div>
+          <p className="mission-list__item-description">
+            {isGenerating
+              ? "Generating mission hierarchy from interview context."
+              : isErrored
+                ? "Interview hit an error. Retry from this list item."
+                : "Interview is waiting for your next response."}
+          </p>
+        </div>
+        <div className="mission-list__item-actions" onClick={(event) => event.stopPropagation()}>
+          <button
+            className={`mission-icon-btn ${isErrored ? "mission-icon-btn--danger" : "mission-icon-btn--success"}`}
+            onClick={() => handleResumeInterviewSession(session.id)}
+            title={getInterviewActionLabel(session.status)}
+            aria-label={getInterviewActionLabel(session.status)}
+          >
+            {isGenerating ? <Loader2 size={14} className="spinner" /> : isErrored ? <RefreshCw size={14} /> : <Sparkles size={14} />}
+          </button>
+        </div>
+      </div>
+    );
+  });
+
   const renderMissionListItems = () => missions.map((mission) => {
     const m = mission;
     const isSelected = selectedMission?.id === m.id;
@@ -3657,8 +3725,9 @@ export function MissionManager({ isOpen, isInline = false, onClose, addToast, pr
                 </div>
               )}
 
-              {/* Mission items */}
+              {/* Mission and interview items */}
               {renderMissionListItems()}
+              {renderInterviewSessionItems()}
 
               {/* Edit mission form */}
               {editingMissionId && (
@@ -3730,32 +3799,6 @@ export function MissionManager({ isOpen, isInline = false, onClose, addToast, pr
 
               {!isCreatingMission && (
                 <div className="mission-list__footer">
-                  {pendingInterviewSessions.length > 0 && (
-                    <div className="mission-resume-prompt">
-                      <AlertCircle size={16} />
-                      <span>
-                        {pendingInterviewSessions.length === 1
-                          ? `Resume "${pendingInterviewSessions[0].title}"?`
-                          : `${pendingInterviewSessions.length} interview sessions pending`}
-                      </span>
-                      <div className="mission-list__resume-actions">
-                        {pendingInterviewSessions.map((s) => (
-                          <button
-                            key={s.id}
-                            className="mission-add-btn"
-                            onClick={() => {
-                              setLocalResumeSessionId(s.id);
-                              setShowInterviewModal(true);
-                              setPendingInterviewSessions([]);
-                            }}
-                          >
-                            {s.status === "error" ? <RefreshCw size={14} /> : <Sparkles size={14} />}
-                            {s.status === "error" ? "Retry" : "Resume"}
-                          </button>
-                        ))}
-                      </div>
-                    </div>
-                  )}
                   {showBottomPlanButton && (
                     <div className="mission-list__footer-actions">
                       <button className="mission-add-btn" onClick={() => setShowInterviewModal(true)}>
diff --git a/packages/dashboard/app/components/__tests__/MissionManager.test.tsx b/packages/dashboard/app/components/__tests__/MissionManager.test.tsx
index 08bbbf315..68daedd4f 100644
--- a/packages/dashboard/app/components/__tests__/MissionManager.test.tsx
+++ b/packages/dashboard/app/components/__tests__/MissionManager.test.tsx
@@ -55,6 +55,8 @@ vi.mock("lucide-react", () => ({
   Loader2: ({ className }: any) => <span data-testid="loader-icon" className={className}>Loader</span>,
   Link: () => <span data-testid="link-icon">Link</span>,
   Unlink: () => <span data-testid="unlink-icon">Unlink</span>,
+  ArrowLeft: () => <span data-testid="arrow-left-icon">ArrowLeft</span>,
+  ArrowRight: () => <span data-testid="arrow-right-icon">ArrowRight</span>,
   Play: () => <span data-testid="play-icon">Play</span>,
   Square: () => <span data-testid="square-icon">Square</span>,
   Sparkles: () => <span data-testid="sparkles-icon">Sparkles</span>,
@@ -1649,6 +1651,130 @@ describe("MissionManager", () => {
     });
   });
 
+  it("shows in-progress interview sessions in the main mission list without footer resume duplication", async () => {
+    mockFetchAiSessions.mockResolvedValueOnce([
+      {
+        id: "session-awaiting",
+        type: "mission_interview",
+        status: "awaiting_input",
+        title: "Payment workflow planning",
+        projectId: null,
+        lockedByTab: null,
+        updatedAt: "2026-01-03T00:00:00.000Z",
+      },
+      {
+        id: "session-generating",
+        type: "mission_interview",
+        status: "generating",
+        title: "Analytics mission drafting",
+        projectId: null,
+        lockedByTab: null,
+        updatedAt: "2026-01-04T00:00:00.000Z",
+      },
+      {
+        id: "session-error",
+        type: "mission_interview",
+        status: "error",
+        title: "SRE guardrails",
+        projectId: null,
+        lockedByTab: null,
+        updatedAt: "2026-01-05T00:00:00.000Z",
+      },
+      {
+        id: "session-complete",
+        type: "mission_interview",
+        status: "complete",
+        title: "Should not render",
+        projectId: null,
+        lockedByTab: null,
+        updatedAt: "2026-01-06T00:00:00.000Z",
+      },
+    ]);
+    mockFetchAiSession.mockResolvedValue({
+      id: "session-awaiting",
+      type: "mission_interview",
+      status: "awaiting_input",
+      title: "Payment workflow planning",
+      inputPayload: JSON.stringify({ missionGoal: "Payment workflow planning" }),
+      conversationHistory: "[]",
+      currentQuestion: JSON.stringify({
+        question: "Which payment providers should be supported first?",
+        kind: "text",
+        key: "provider_scope",
+      }),
+      result: null,
+      thinkingOutput: "",
+      error: null,
+      projectId: null,
+      createdAt: "2026-01-03T00:00:00.000Z",
+      updatedAt: "2026-01-03T00:00:00.000Z",
+    });
+    globalThis.fetch = createFetchMock();
+
+    render(<MissionManager isOpen={true} onClose={vi.fn()} addToast={vi.fn()} />);
+
+    await waitFor(() => {
+      expect(screen.getByText("Build Auth System")).toBeInTheDocument();
+      expect(screen.getByText("Payment workflow planning")).toBeInTheDocument();
+      expect(screen.getByText("Analytics mission drafting")).toBeInTheDocument();
+      expect(screen.getByText("SRE guardrails")).toBeInTheDocument();
+    });
+
+    expect(screen.queryByText("Should not render")).not.toBeInTheDocument();
+    expect(screen.queryByText(/interview sessions pending/i)).not.toBeInTheDocument();
+
+    const awaitingInterviewRow = screen.getByText("Payment workflow planning").closest(".mission-list__item");
+    expect(awaitingInterviewRow).toBeTruthy();
+    fireEvent.click(within(awaitingInterviewRow as HTMLElement).getByLabelText("Resume interview"));
+
+    await waitFor(() => {
+      expect(screen.getByText("Plan Mission with AI")).toBeInTheDocument();
+    });
+  });
+
+  it("exposes retry action for errored interview sessions from the mission list", async () => {
+    mockFetchAiSessions.mockResolvedValueOnce([
+      {
+        id: "session-error",
+        type: "mission_interview",
+        status: "error",
+        title: "Mission in error",
+        projectId: null,
+        lockedByTab: null,
+        updatedAt: "2026-01-05T00:00:00.000Z",
+      },
+    ]);
+    mockFetchAiSession.mockResolvedValue({
+      id: "session-error",
+      type: "mission_interview",
+      status: "error",
+      title: "Mission in error",
+      inputPayload: "{}",
+      conversationHistory: "[]",
+      currentQuestion: null,
+      result: null,
+      thinkingOutput: "",
+      error: "Planning failed",
+      projectId: null,
+      createdAt: "2026-01-05T00:00:00.000Z",
+      updatedAt: "2026-01-05T00:00:00.000Z",
+    });
+    globalThis.fetch = createFetchMock();
+
+    render(<MissionManager isOpen={true} onClose={vi.fn()} addToast={vi.fn()} />);
+
+    await waitFor(() => {
+      expect(screen.getByLabelText("Retry interview")).toBeInTheDocument();
+      expect(screen.getByText("Needs retry")).toBeInTheDocument();
+    });
+
+    fireEvent.click(screen.getByLabelText("Retry interview"));
+
+    await waitFor(() => {
+      expect(screen.getByText("Plan Mission with AI")).toBeInTheDocument();
+    });
+  });
+
   it("logs a warning when pending interview session fetch fails", async () => {
     const warnSpy = vi.spyOn(console, "warn").mockImplementation(() => {});
     const pendingFetchError = new Error("Pending sessions failed");
diff --git a/packages/dashboard/src/__tests__/github-tracking.test.ts b/packages/dashboard/src/__tests__/github-tracking.test.ts
index 8a1cee18b..9839d5321 100644
--- a/packages/dashboard/src/__tests__/github-tracking.test.ts
+++ b/packages/dashboard/src/__tests__/github-tracking.test.ts
@@ -148,6 +148,19 @@ describe("maybeCreateTrackingIssue", () => {
     expect(createIssueMock).toHaveBeenCalledWith(expect.objectContaining({ owner, repo }));
   });
 
+  it("creates a tracking issue from explicit task override when defaults are unset", async () => {
+     const linkGithubIssue = vi.fn();
+
+     await maybeCreateTrackingIssue(buildTask({ githubTracking: { enabled: true, repoOverride: "task/repo" } }), {
+       taskStore: { linkGithubIssue, recordActivity: vi.fn() } as any,
+       projectSettings: {},
+       globalSettings: {},
+       logger: console,
+     });
+
+     expect(createIssueMock).toHaveBeenCalledWith(expect.objectContaining({ owner: "task", repo: "repo" }));
+   });
+
   it("skips creation when tracking is on but no repo is configured", async () => {
     const result = await maybeCreateTrackingIssue(buildTask({ githubTracking: { enabled: true } }), {
       taskStore: { recordActivity: vi.fn() } as any,
diff --git a/packages/dashboard/src/__tests__/routes-tasks.test.ts b/packages/dashboard/src/__tests__/routes-tasks.test.ts
index 88daacc5b..501804f9e 100644
--- a/packages/dashboard/src/__tests__/routes-tasks.test.ts
+++ b/packages/dashboard/src/__tests__/routes-tasks.test.ts
@@ -209,6 +209,8 @@ function createMockStore(overrides: Partial<TaskStore> = {}): TaskStore {
     deleteTaskDocument: vi.fn().mockResolvedValue(undefined),
     updatePrInfo: vi.fn().mockResolvedValue(undefined),
     updateIssueInfo: vi.fn().mockResolvedValue(undefined),
+    linkGithubIssue: vi.fn().mockResolvedValue(undefined),
+    recordActivity: vi.fn().mockResolvedValue(undefined),
     getRootDir: vi.fn().mockReturnValue("/fake/root"),
     getDistributedTaskIdAllocator: vi.fn().mockReturnValue({
       reserveDistributedTaskId: vi.fn().mockResolvedValue({ reservationId: "res-1", taskId: "FN-7001" }),
@@ -680,6 +682,41 @@ describe("POST /tasks", () => {
     );
   });
 
+  it("attempts tracking issue creation for explicit task-level override when defaults are unset", async () => {
+    const createIssueSpy = vi.spyOn(GitHubClient.prototype, "createIssue").mockResolvedValue({
+      owner: "task",
+      repo: "repo",
+      number: 42,
+      htmlUrl: "https://github.com/task/repo/issues/42",
+      createdAt: "2026-01-01T00:00:00.000Z",
+    });
+
+    (store.getSettings as ReturnType<typeof vi.fn>).mockResolvedValue({ githubAuthMode: "token", githubAuthToken: "tok" });
+    const linkGithubIssue = store.linkGithubIssue as ReturnType<typeof vi.fn>;
+    const recordActivity = store.recordActivity as ReturnType<typeof vi.fn>;
+    (store.createTask as ReturnType<typeof vi.fn>).mockResolvedValue({
+      ...FAKE_TASK_DETAIL,
+      githubTracking: { enabled: true, repoOverride: "task/repo" },
+    });
+
+    const res = await REQUEST(
+      buildApp(),
+      "POST",
+      "/api/tasks",
+      JSON.stringify({
+        description: "Track this task",
+        githubTracking: { enabled: true, repoOverride: "task/repo" },
+      }),
+      { "Content-Type": "application/json" },
+    );
+
+    expect(res.status).toBe(201);
+    expect(createIssueSpy).toHaveBeenCalledWith(expect.objectContaining({ owner: "task", repo: "repo" }));
+    expect(linkGithubIssue).toHaveBeenCalledWith("FN-001", expect.objectContaining({ owner: "task", repo: "repo", number: 42 }));
+    expect(recordActivity).toHaveBeenCalledWith(expect.objectContaining({ metadata: expect.objectContaining({ type: "github-issue-created" }) }));
+    createIssueSpy.mockRestore();
+  });
+
   it("uses distributed allocator flow when reserved-id create is available", async () => {
     const createTaskWithReservedId = vi.fn().mockResolvedValue({
       ...FAKE_TASK_DETAIL,
diff --git a/packages/dashboard/src/github-tracking.ts b/packages/dashboard/src/github-tracking.ts
index 88baab3bf..f94bfa6b0 100644
--- a/packages/dashboard/src/github-tracking.ts
+++ b/packages/dashboard/src/github-tracking.ts
@@ -1,4 +1,10 @@
-import type { GlobalSettings, ProjectSettings, Task, TaskStore } from "@fusion/core";
+import {
+  resolveTaskGithubTracking,
+  type GlobalSettings,
+  type ProjectSettings,
+  type Task,
+  type TaskStore,
+} from "@fusion/core";
 import type { CreatedIssue } from "./github.js";
 import { GitHubClient } from "./github.js";
 import { resolveGithubTrackingAuth } from "./github-auth.js";
@@ -84,24 +90,17 @@ export type MaybeCreateTrackingIssueReason =
   | "auth_gh_not_authenticated"
   | "auth_invalid_mode";
 
-function parseRepo(value: string | undefined): { owner: string; repo: string } | null {
-  if (!value) return null;
-  const trimmed = value.trim();
-  const [owner, repo, ...rest] = trimmed.split("/");
-  if (!owner || !repo || rest.length > 0) return null;
-  return { owner, repo };
-}
-
 export async function maybeCreateTrackingIssue(
   task: Task,
   deps: MaybeCreateTrackingIssueDeps,
 ): Promise<{ created: false; reason: MaybeCreateTrackingIssueReason } | { created: true; issue: CreatedIssue }> {
   const tracking = task.githubTracking;
-  if (tracking?.enabled !== true) {
+  const resolvedTracking = resolveTaskGithubTracking(task, deps.projectSettings, deps.globalSettings);
+  if (!resolvedTracking.enabled) {
     return { created: false, reason: "tracking_disabled" };
   }
 
-  if (tracking.issue) {
+  if (tracking?.issue) {
     return { created: false, reason: "issue_already_linked" };
   }
 
@@ -109,10 +108,7 @@ export async function maybeCreateTrackingIssue(
     return { created: false, reason: "github_import_source" };
   }
 
-  const repo =
-    parseRepo(tracking.repoOverride) ??
-    parseRepo(deps.projectSettings.githubTrackingDefaultRepo) ??
-    parseRepo(deps.globalSettings.githubTrackingDefaultRepo);
+  const repo = resolvedTracking.repo;
 
   if (!repo) {
     deps.logger?.warn?.(`[github-tracking] No repo configured for ${task.id}`);