refactor(components): extract handleAcceptNextAction to new hook

The inline handleAcceptNextAction callback in DraftTab was a ~90-line
switch on NextActionRecommendation.kind (answer, clarify, approval,
runbook, escalate, and a default KB-draft-copy fallback) — each branch
routed to different workspace side effects through a dense set of
state writers and sibling handlers.

Extracts to a sibling hook useNextActionHandler with an explicit
options interface: state reads, state writers, delegated cross-hook
handlers, logEvent, and onShowSuccess. DraftTab's call site collapses
from a ~90-line useCallback to a ~17-line hook invocation with named
bindings, and the useCallback's full dep array now lives next to the
body it memoizes.

Removes the last two DraftTab-only imports made dead by the extraction
(compactLines, NextActionRecommendation) and keeps parseCaseIntake on
its own import line.

DraftTab.tsx: 1278 -> 1207 lines (-71).
Branch total vs master: 1412 -> 1207 (-205, -14.5%).

Also drops one react-hooks/exhaustive-deps warning in DraftTab — the
dep array that lint was flagging now lives inside the new hook where
it is complete.

No behavior change. Typecheck + vitest (241/241) + eslint clean.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: saagar210

src/components/Draft/DraftTab.tsx

@@ -28,6 +28,7 @@ import { useDraftIntake } from "./useDraftIntake";
 import { useDraftLifecycle } from "./useDraftLifecycle";
 import { useDraftPersistence } from "./useDraftPersistence";
 import { useGuidedRunbook } from "./useGuidedRunbook";
+import { useNextActionHandler } from "./useNextActionHandler";
 import { useResponseActions } from "./useResponseActions";
 import { useWorkspaceArtifacts } from "./useWorkspaceArtifacts";
 import { useWorkspaceClipboardPacks } from "./useWorkspaceClipboardPacks";
@@ -53,10 +54,7 @@ import { useWorkspaceCatalog } from "../../features/workspace/useWorkspaceCatalo
 import { useWorkspaceDerivedArtifacts } from "../../features/workspace/useWorkspaceDerivedArtifacts";
 import { useWorkspaceCommandBridge } from "../../features/workspace/useWorkspaceCommandBridge";
 import { useWorkspaceDraftState } from "../../features/workspace/useWorkspaceDraftState";
-import {
-  compactLines,
-  parseCaseIntake,
-} from "../../features/workspace/workspaceAssistant";
+import { parseCaseIntake } from "../../features/workspace/workspaceAssistant";
 import { countWords } from "../../features/analytics/qualityMetrics";
 import type { JiraTicket } from "../../hooks/useJira";
 import type {
@@ -66,7 +64,6 @@ import type {
 } from "../../types/llm";
 import type { ContextSource } from "../../types/knowledge";
 import type {
-  NextActionRecommendation,
   ResponseLength,
   SavedDraft,
   SimilarCase,
@@ -813,92 +810,24 @@ export const DraftTab = forwardRef<DraftTabHandle, DraftTabProps>(
       ],
     );
 
-    const handleAcceptNextAction = useCallback(
-      (action: NextActionRecommendation) => {
-        void logEvent("workspace_next_action_accepted", {
-          ticket_id: currentTicketId,
-          action_kind: action.kind,
-          action_id: action.id,
-        });
-
-        if (action.kind === "answer") {
-          void handleGenerate();
-          return;
-        }
-
-        if (action.kind === "clarify") {
-          const clarifyPrompt = compactLines([
-            diagnosticNotes,
-            "Clarifying questions to ask:",
-            ...missingQuestions.map((question) => `- ${question.question}`),
-          ]);
-          setDiagnosticNotes(clarifyPrompt);
-          setPanelDensityMode("focus-intake");
-          showSuccess("Added clarifying questions to the diagnostic notes");
-          return;
-        }
-
-        if (action.kind === "approval") {
-          const querySeed = compactLines([
-            caseIntake.issue,
-            currentTicket?.summary,
-            input,
-          ]);
-          setApprovalQuery(`${querySeed || "support request"} policy approval`);
-          setPanelDensityMode("focus-intake");
-          showSuccess("Primed the approval search query");
-          return;
-        }
-
-        if (action.kind === "runbook") {
-          setPanelDensityMode("focus-intake");
-          setDiagnosticNotes((prev) =>
-            compactLines([
-              prev,
-              "Runbook kickoff:",
-              `- ${action.rationale}`,
-              ...action.prerequisites.map((item) => `- ${item}`),
-            ]),
-          );
-          const incidentTemplate = runbookTemplates.find((template) =>
-            /incident|security/i.test(`${template.name} ${template.scenario}`),
-          );
-          if (incidentTemplate) {
-            void handleStartGuidedRunbook(incidentTemplate.id);
-          }
-          showSuccess("Prepared the workspace for guided runbook steps");
-          return;
-        }
-
-        if (action.kind === "escalate") {
-          setCaseIntake((prev) => ({
-            ...prev,
-            note_audience: "escalation-note",
-          }));
-          setDiagnosticNotes((prev) =>
-            compactLines([prev, "Escalation focus:", `- ${action.rationale}`]),
-          );
-          showSuccess("Switched the workspace into escalation-note mode");
-          return;
-        }
-
-        void handleCopyKbDraft();
-      },
-      [
-        logEvent,
-        currentTicketId,
-        handleGenerate,
-        diagnosticNotes,
-        missingQuestions,
-        showSuccess,
-        caseIntake.issue,
-        currentTicket?.summary,
-        input,
-        runbookTemplates,
-        handleStartGuidedRunbook,
-        handleCopyKbDraft,
-      ],
-    );
+    const handleAcceptNextAction = useNextActionHandler({
+      currentTicketId,
+      currentTicketSummary: currentTicket?.summary,
+      diagnosticNotes,
+      input,
+      caseIntakeIssue: caseIntake.issue,
+      missingQuestions,
+      runbookTemplates,
+      setDiagnosticNotes,
+      setPanelDensityMode,
+      setApprovalQuery,
+      setCaseIntake,
+      handleGenerate,
+      handleStartGuidedRunbook,
+      handleCopyKbDraft,
+      logEvent,
+      onShowSuccess: showSuccess,
+    });
 
     useWorkspaceCommandBridge({
       enabled: workspaceRailEnabled,

src/components/Draft/useNextActionHandler.ts

@@ -0,0 +1,155 @@
+import { useCallback } from "react";
+import type { Dispatch, SetStateAction } from "react";
+import { compactLines } from "../../features/workspace/workspaceAssistant";
+import type {
+  CaseIntake,
+  GuidedRunbookTemplate,
+  MissingQuestion,
+  NextActionRecommendation,
+} from "../../types/workspace";
+import type { DraftPanelDensityMode } from "./draftTabDefaults";
+
+export interface UseNextActionHandlerOptions {
+  // State reads
+  currentTicketId: string | null;
+  currentTicketSummary: string | undefined;
+  diagnosticNotes: string;
+  input: string;
+  caseIntakeIssue: string | null | undefined;
+  missingQuestions: MissingQuestion[];
+  runbookTemplates: GuidedRunbookTemplate[];
+
+  // State writers
+  setDiagnosticNotes: Dispatch<SetStateAction<string>>;
+  setPanelDensityMode: (mode: DraftPanelDensityMode) => void;
+  setApprovalQuery: (value: string) => void;
+  setCaseIntake: Dispatch<SetStateAction<CaseIntake>>;
+
+  // Cross-hook handlers we delegate to
+  handleGenerate: () => unknown;
+  handleStartGuidedRunbook: (templateId: string) => unknown;
+  handleCopyKbDraft: () => unknown;
+
+  // Observability + UX
+  logEvent: (event: string, payload?: Record<string, unknown>) => unknown;
+  onShowSuccess: (message: string) => void;
+}
+
+/**
+ * Handles the user's acceptance of a next-best-action recommendation. Each
+ * action kind routes to a different workspace side effect — triggering
+ * generation, primer notes, approval search seed, guided runbook kickoff,
+ * escalation-note mode, or a default KB draft copy.
+ *
+ * Extracted from DraftTab so the ~90-line switch lives next to the hook
+ * family instead of inflating the orchestrator component.
+ */
+export function useNextActionHandler({
+  currentTicketId,
+  currentTicketSummary,
+  diagnosticNotes,
+  input,
+  caseIntakeIssue,
+  missingQuestions,
+  runbookTemplates,
+  setDiagnosticNotes,
+  setPanelDensityMode,
+  setApprovalQuery,
+  setCaseIntake,
+  handleGenerate,
+  handleStartGuidedRunbook,
+  handleCopyKbDraft,
+  logEvent,
+  onShowSuccess,
+}: UseNextActionHandlerOptions) {
+  return useCallback(
+    (action: NextActionRecommendation) => {
+      void logEvent("workspace_next_action_accepted", {
+        ticket_id: currentTicketId,
+        action_kind: action.kind,
+        action_id: action.id,
+      });
+
+      if (action.kind === "answer") {
+        void handleGenerate();
+        return;
+      }
+
+      if (action.kind === "clarify") {
+        const clarifyPrompt = compactLines([
+          diagnosticNotes,
+          "Clarifying questions to ask:",
+          ...missingQuestions.map((question) => `- ${question.question}`),
+        ]);
+        setDiagnosticNotes(clarifyPrompt);
+        setPanelDensityMode("focus-intake");
+        onShowSuccess("Added clarifying questions to the diagnostic notes");
+        return;
+      }
+
+      if (action.kind === "approval") {
+        const querySeed = compactLines([
+          caseIntakeIssue,
+          currentTicketSummary,
+          input,
+        ]);
+        setApprovalQuery(`${querySeed || "support request"} policy approval`);
+        setPanelDensityMode("focus-intake");
+        onShowSuccess("Primed the approval search query");
+        return;
+      }
+
+      if (action.kind === "runbook") {
+        setPanelDensityMode("focus-intake");
+        setDiagnosticNotes((prev) =>
+          compactLines([
+            prev,
+            "Runbook kickoff:",
+            `- ${action.rationale}`,
+            ...action.prerequisites.map((item) => `- ${item}`),
+          ]),
+        );
+        const incidentTemplate = runbookTemplates.find((template) =>
+          /incident|security/i.test(`${template.name} ${template.scenario}`),
+        );
+        if (incidentTemplate) {
+          void handleStartGuidedRunbook(incidentTemplate.id);
+        }
+        onShowSuccess("Prepared the workspace for guided runbook steps");
+        return;
+      }
+
+      if (action.kind === "escalate") {
+        setCaseIntake((prev) => ({
+          ...prev,
+          note_audience: "escalation-note",
+        }));
+        setDiagnosticNotes((prev) =>
+          compactLines([prev, "Escalation focus:", `- ${action.rationale}`]),
+        );
+        onShowSuccess("Switched the workspace into escalation-note mode");
+        return;
+      }
+
+      void handleCopyKbDraft();
+    },
+    [
+      logEvent,
+      currentTicketId,
+      handleGenerate,
+      diagnosticNotes,
+      missingQuestions,
+      onShowSuccess,
+      caseIntakeIssue,
+      currentTicketSummary,
+      input,
+      runbookTemplates,
+      handleStartGuidedRunbook,
+      handleCopyKbDraft,
+      setDiagnosticNotes,
+      setPanelDensityMode,
+      setApprovalQuery,
+      setCaseIntake,
+    ],
+  );
+}