@stigmer/react 3.1.0 → 3.1.2

package/src/session/SessionViewer.tsx

@@ -1,33 +1,126 @@
 "use client";
 
-import { useCallback, useMemo, useRef, type ReactNode } from "react";
+import { memo, useCallback, useEffect, useMemo, useRef, useState, type ReactNode } from "react";
+import { create } from "@bufbuild/protobuf";
 import { cn } from "@stigmer/theme";
-import { getUserMessage, type ResourceRef } from "@stigmer/sdk";
-import type { UseGitHubConnectionReturn } from "../github/useGitHubConnection";
-import type { WorkspaceFileLister } from "../workspace/WorkspaceFileLister";
-import type { InteractionModeOption, SessionComposerHandle } from "../composer";
-import type { ApplyResourceResult } from "../library/useApplyResource";
-import { ResizableSplit } from "../internal/ResizableSplit";
-import { SelectionStore } from "../internal/store/selection-store";
-import { ThreadSelectionContext } from "../execution/ThreadSelectionContext";
-import { useSelectedThreadItem } from "../execution/useThreadSelection";
-import { MessageThread } from "../execution/MessageThread";
-import { FileReviewDock } from "../execution/FileReviewDock";
-import { ThreadSkeleton } from "../execution/ThreadSkeleton";
-import { SessionComposer } from "../composer";
-import { SecretFlowErrorGuide, isSecretFlowError } from "../error";
-import { useSessionPageFlow } from "./useSessionPageFlow";
-import { SessionInspector } from "./inspector/SessionInspector";
-import type { RuntimeEnvProvider } from "./runtime-env";
-import type { SessionAudience } from "./audience";
+import { getUserMessage, type AttachmentInput, type ResourceRef } from "@stigmer/sdk";
+import {
+  GetArtifactContentRequestSchema,
+  UploadAttachmentRequestSchema,
+} from "@stigmer/protos/ai/stigmer/agentic/agentexecution/v1/io_pb";
+import type { UseGitHubConnectionReturn } from "../github/useGitHubConnection.js";
+import type { WorkspaceFileLister } from "../workspace/WorkspaceFileLister.js";
+import type { WorkspaceFileReader } from "../workspace/WorkspaceFileReader.js";
+import type { WorkspaceContentSearcher } from "../workspace/WorkspaceContentSearcher.js";
+import { resolveWorkspaceFileSelection } from "../workspace/resolveWorkspaceFileSelection.js";
+import {
+  WorkspaceSurface,
+  type SurfaceVirtualDocument,
+} from "../workspace/WorkspaceSurface.js";
+import type { InteractionModeOption, SessionComposerHandle } from "../composer/index.js";
+import type { ApplyResourceResult } from "../library/useApplyResource.js";
+import { ResizableSplit } from "../internal/ResizableSplit.js";
+import { SelectionStore } from "../internal/store/selection-store.js";
+import { useWorkspaceEditors, isVirtualEntryId } from "../internal/store/index.js";
+import { ThreadSelectionContext } from "../execution/ThreadSelectionContext.js";
+import { useSelectedThreadItem } from "../execution/useThreadSelection.js";
+import { MessageThread } from "../execution/MessageThread.js";
+import { FileReviewDock } from "../execution/FileReviewDock.js";
+import { ThreadSkeleton } from "../execution/ThreadSkeleton.js";
+import { SessionComposer } from "../composer/index.js";
+import { SecretFlowErrorGuide, isSecretFlowError } from "../error/index.js";
+import { useStigmer } from "../hooks.js";
+import {
+  PLAN_ARTIFACT_NAME,
+  findLatestSessionPlan,
+  findPlanArtifact,
+  type SessionPlan,
+} from "../library/detect-plan-artifact.js";
+import { findStreamingPlan } from "../library/detect-streaming-plan.js";
+import { useSessionPageFlow } from "./useSessionPageFlow.js";
+import { useOpenFileChange } from "./useOpenFileChange.js";
+import { usePlanDraft, planDraftKey, type PlanDraftController } from "./usePlanDraft.js";
+import { PlanEditor } from "./PlanEditor.js";
+import { PlanStreamingDocument } from "./PlanStreamingDocument.js";
+import { PLAN_DOCUMENT_ENTRY_ID, PLAN_DOCUMENT_PATH } from "./plan-document.js";
+import { useSessionRailViews } from "./useSessionRailViews.js";
+import { useSessionPanel, type SessionPanelController } from "./useSessionPanel.js";
+import { SessionPanelChip } from "./SessionPanelChip.js";
+import { useSessionWriteBacks } from "./useSessionWriteBacks.js";
+import { useSessionArtifacts } from "./useSessionArtifacts.js";
+import type { SetupTabProps } from "./facets/SetupTab.js";
+import { ExecutionPhase } from "@stigmer/protos/ai/stigmer/agentic/agentexecution/v1/enum_pb";
+import type { RuntimeEnvProvider } from "./runtime-env.js";
+import type { SessionAudience } from "./audience.js";
 
 /**
- * Message submitted when the user implements a plan. References the published
- * `plan.md` explicitly so the agent acts on the durable artifact rather than
- * relying on re-reading its own prior chat message.
+ * Where the approved plan mounts in the implement execution's workspace —
+ * the harnesses' standard attachment inputs directory, where the runner's
+ * implement-plan directive points the agent (shared/implement-plan-prompt.ts).
+ * Attaching at this exact path is what makes the build implement the document
+ * the user approved (including in-place edits), not a paraphrase from memory.
  */
-const IMPLEMENT_PLAN_MESSAGE =
-  "Implement the plan above (saved as plan.md). Follow it step by step and make the changes it describes.";
+const APPROVED_PLAN_MOUNT_PATH = `.stigmer/inputs/${PLAN_ARTIFACT_NAME}`;
+
+/**
+ * The build turn's user message — a short human-readable label, NOT the
+ * implement instruction. The agent-facing instruction is runner-injected
+ * (keyed off `executionConfig.buildFromPlan`), so this text exists for
+ * humans: the thread renders the turn as a compact chip from the same flag,
+ * and surfaces without the chip treatment (the CLI, older clients) show
+ * this label as-is.
+ */
+const BUILD_FROM_PLAN_MESSAGE = "Build from plan";
+
+/**
+ * A plan the active execution is writing right now, resolved at the viewer
+ * level (from `findStreamingPlan`) so the panel's plan tab can render it
+ * live. The streaming sibling of {@link SessionPlan}: `executionId` keys the
+ * document (a new turn's stream resets the tab's view state) and forms the
+ * `<executionId>:streaming` identity for the plan-tab auto-open trigger.
+ */
+interface SessionStreamingPlan {
+  /** ID of the execution writing the plan. */
+  readonly executionId: string;
+  /** The live plan text (fence-stripped display projection). */
+  readonly displayText: string;
+}
+
+/**
+ * Reads the published plan's full text for the build handoff. Refuses a
+ * truncated read (content RPC caps at 512 KB): attaching a partial plan would
+ * silently implement half a document — the caller falls back to
+ * conversation-history implement instead.
+ */
+async function fetchPlanText(
+  stigmer: ReturnType<typeof useStigmer>,
+  plan: SessionPlan,
+): Promise<string> {
+  const result = await stigmer.agentExecution.getArtifactContent(
+    create(GetArtifactContentRequestSchema, {
+      executionId: plan.executionId,
+      storageKey: plan.artifact.storageKey,
+    }),
+  );
+  if (result.truncated) {
+    throw new Error("plan content truncated — not attaching a partial plan");
+  }
+  return new TextDecoder().decode(result.content);
+}
+
+/**
+ * Width and anchoring of the conversation reading column, shared by the
+ * thread and the composer/banners block so they can never drift apart.
+ *
+ * Centered (the classic chat reading view, matching Cursor's agent panel):
+ * with the session panel collapsed the column sits in the middle of the
+ * row; opening the panel narrows the chat pane below the column max, so
+ * the column simply fills it. The thread applies the same geometry via
+ * `contentColumn="center"`, which adds the item gutter; the composer/banner
+ * block's children carry their own edge padding, so only width and
+ * anchoring live here.
+ */
+const CONVERSATION_COLUMN_CLASS = "mx-auto w-full max-w-3xl";
 
 /** Props for {@link SessionViewer}. */
 export interface SessionViewerProps {
@@ -70,6 +163,21 @@ export interface SessionViewerProps {
    * expandable file tree. (DD-004 capability injection, DD-011 opt-in.)
    */
   readonly workspaceFileLister?: WorkspaceFileLister;
+  /**
+   * Platform-injected content reader for the read-only file viewer. When
+   * provided, clicking a file in the Workspace tree opens it in a contextual
+   * "Viewer" tab. GitHub Contents/blob on web, Tauri fs on desktop; `undefined`
+   * degrades the viewer to an "unavailable here" state (DD-004, DD-011 opt-in).
+   */
+  readonly workspaceFileReader?: WorkspaceFileReader;
+  /**
+   * Platform-injected content (text) searcher. When provided, the session
+   * panel's Search pane gains a `Name | Text` toggle for full-text search
+   * across the workspace. Desktop injects a native ripgrep-backed searcher;
+   * web leaves it undefined (git content search needs a branch-accurate
+   * backend — DD-09), keeping Search filename-only there (DD-004, DD-011 opt-in).
+   */
+  readonly workspaceContentSearcher?: WorkspaceContentSearcher;
   /**
    * Supplies host-app environment variables for every follow-up
    * execution (e.g. short-lived credentials for MCP tools, minted as
@@ -92,11 +200,18 @@ export interface SessionViewerProps {
    */
   readonly audience?: SessionAudience;
   /**
-   * Slot for host-injected header actions (e.g., Share button with
-   * PermissionGate). Rendered in the top-right corner of the viewer.
-   * Keeps the SDK organism unopinionated about Console auth (DD-004).
+   * Slot for host-injected header actions. Rendered in the top-right corner
+   * of the viewer, beside the panel chip. Keeps the SDK organism
+   * unopinionated about Console auth (DD-004).
    */
   readonly headerActions?: ReactNode;
+  /**
+   * Host-injected access management control (e.g. the Console's
+   * `ManageAccessButton` with its own permission gating). Rendered inside
+   * the panel's Config facet rather than the header — access is session
+   * configuration, not a moment-to-moment action (DD-004 slot injection).
+   */
+  readonly accessSlot?: ReactNode;
   /** Called after a resource is applied from the Artifacts tab. */
   readonly onApplied?: (result: ApplyResourceResult) => void;
   /** Additional CSS classes for the root container. */
@@ -110,10 +225,13 @@ export interface SessionViewerProps {
  * Owns `useSessionPageFlow` internally and composes:
  * - **Conversation column** (primary): `MessageThread` + error
  *   banners + `SessionComposer`
- * - **SessionInspector** (secondary): tabbed panel with Workspace,
- *   Config, Changes, Artifacts, Usage, and Inspect facets
+ * - **Session panel** (secondary): the unified `WorkspaceSurface` — explorer,
+ *   search, read-only editor, and the session facets (Config, Changes,
+ *   Artifacts, Usage, Inspect) as injected rail views. Collapsed by default
+ *   to a top-right chip; opening a file (tree, search, or a transcript path)
+ *   expands it.
  *
- * Connected via `ResizableSplit` with persisted width.
+ * Connected via `ResizableSplit` with persisted chat width.
  *
  * Thread selection is provided via `ThreadSelectionContext` (Phase 2
  * opt-in) for render-isolated per-item selection.
@@ -150,9 +268,12 @@ export function SessionViewer({
   enableLocal = false,
   onBrowseLocalFolder,
   workspaceFileLister,
+  workspaceFileReader,
+  workspaceContentSearcher,
   getRuntimeEnv,
   audience = "integrator",
   headerActions,
+  accessSlot,
   onApplied,
   className,
 }: SessionViewerProps) {
@@ -170,16 +291,179 @@ export function SessionViewer({
   }
   const selectionStore = selectionStoreRef.current;
 
+  const phase =
+    flow.displayExecution?.status?.phase ??
+    ExecutionPhase.EXECUTION_PHASE_UNSPECIFIED;
+  const hasPhase = phase !== ExecutionPhase.EXECUTION_PHASE_UNSPECIFIED;
+
+  // Facet arrival counts: rail badges inside the panel, the chip's dot-count
+  // while collapsed. Arrivals never auto-open the panel (badge-only signal).
+  const { hasWriteBacks, writeBackCount } = useSessionWriteBacks(flow.allExecutions);
+  const { artifactCount } = useSessionArtifacts(flow.allExecutions);
+
+  // The session's current plan (latest published plan.md across executions)
+  // and its viewer-owned draft. Ownership at this level is deliberate: the
+  // panel unmounts facet content on every view switch, so a facet-local
+  // draft would silently lose the user's edits (see usePlanDraft).
+  const stigmer = useStigmer();
+  const sessionPlan = useMemo(
+    () => findLatestSessionPlan(flow.allExecutions),
+    [flow.allExecutions],
+  );
+  const planDraft = usePlanDraft(sessionPlan);
+
+  // The plan the ACTIVE turn is writing right now (the H1 convention — see
+  // findStreamingPlan), resolved at this level for the same reason
+  // sessionPlan is: the panel's plan tab renders it. Recomputed per stream
+  // commit by design — this component already re-renders on every commit
+  // (it reads `flow`), and the scan touches only the last AI message.
+  const streamingPlan = useMemo<SessionStreamingPlan | undefined>(() => {
+    const activeExec = conv.activeStreamExecution;
+    const live = findStreamingPlan(activeExec);
+    const executionId = activeExec?.metadata?.id;
+    return live && executionId
+      ? { executionId, displayText: live.displayText }
+      : undefined;
+  }, [conv.activeStreamExecution]);
+  const [isBuildingFromPlan, setIsBuildingFromPlan] = useState(false);
+  const [planAttachFailed, setPlanAttachFailed] = useState(false);
+
+  // WHICH plan the panel's plan document tab shows. `null` means the session's
+  // current (latest) plan — the editable, buildable one; an execution id
+  // selects that turn's historical plan, rendered read-only. One tab, host-
+  // controlled content (Decision 3 in DD-16).
+  const [openPlanExecutionId, setOpenPlanExecutionId] = useState<string | null>(null);
+
+  // The plan tab's identity across both plan lifecycles: while a plan
+  // streams, the in-flight turn owns the tab (`<executionId>:streaming`);
+  // once published, the artifact identity (planDraftKey) takes over. Each
+  // transition is a NEW identity, so useSessionPanel's planKey trigger
+  // auto-opens the tab the moment a plan starts streaming AND (idempotently)
+  // when it settles into the published plan.
+  const currentPlanKey = sessionPlan ? planDraftKey(sessionPlan) : null;
+  const panelPlanKey = streamingPlan
+    ? `${streamingPlan.executionId}:streaming`
+    : currentPlanKey;
+
+  // A new plan identity (a plan starting to stream, the first published
+  // plan, or a refinement) supersedes any historical plan the user had
+  // opened: the tab snaps back to "latest" so the auto-open always surfaces
+  // the incoming plan. Adjust-state-during-render — own state only, the
+  // established idiom.
+  const [prevPlanKey, setPrevPlanKey] = useState(panelPlanKey);
+  if (panelPlanKey !== prevPlanKey) {
+    setPrevPlanKey(panelPlanKey);
+    setOpenPlanExecutionId(null);
+  }
+
+  // The unified-panel controller: owns the open-editor group store, the
+  // open/collapsed state, and the rail-view FSM. Shared with the launcher
+  // (DD-016). The editor store is owned here (never subscribed at this level)
+  // so opening/switching files re-renders only the panel subtree — the
+  // SessionPanelRegion subscribes, not the conversation column — preserving
+  // streaming render isolation (DD-009/DD-010, invariant 2).
+  const panel = useSessionPanel({
+    phase: hasPhase ? phase : null,
+    hasChanges: hasWriteBacks,
+    planKey: panelPlanKey,
+  });
+
   const handleBuildFromPlan = useCallback(() => {
     // Switch the picker to Agent (so subsequent turns stay in Agent) and submit
-    // the implement message immediately through the composer's full pipeline.
+    // the implement message through the composer's full pipeline.
     // `interactionMode: "agent"` is passed explicitly to win the same-tick race
     // where the composer prop has not yet re-rendered from "plan".
     setInteractionMode("agent");
-    composerRef.current?.submit(IMPLEMENT_PLAN_MESSAGE, {
-      interactionMode: "agent",
-    });
-  }, [setInteractionMode]);
+    setPlanAttachFailed(false);
+
+    // `buildFromPlan` (not message text) is what tells the runner to inject
+    // the implement directive and the thread to render the compact chip —
+    // with or without the attachment (the runner picks the directive variant).
+    const submitImplementTurn = (attachments?: AttachmentInput[]) => {
+      composerRef.current?.submit(BUILD_FROM_PLAN_MESSAGE, {
+        interactionMode: "agent",
+        buildFromPlan: true,
+        attachments,
+      });
+    };
+
+    // No published artifact (the bare-CTA fallback card) — nothing to attach.
+    if (!sessionPlan) {
+      submitImplementTurn();
+      return;
+    }
+
+    // Deterministic handoff: upload the APPROVED plan (draft if edited, else
+    // the artifact text — one uniform path) and attach it to the implement
+    // execution. The published artifact itself stays immutable; the approved
+    // copy is a new input on the build turn (edit-as-input provenance).
+    setIsBuildingFromPlan(true);
+    void (async () => {
+      try {
+        const approvedText =
+          planDraft.readDraft() ?? (await fetchPlanText(stigmer, sessionPlan));
+        const response = await stigmer.agentExecution.uploadAttachment(
+          create(UploadAttachmentRequestSchema, {
+            filename: PLAN_ARTIFACT_NAME,
+            content: new TextEncoder().encode(approvedText),
+            contentType: "text/markdown",
+          }),
+        );
+        submitImplementTurn([
+          {
+            filename: PLAN_ARTIFACT_NAME,
+            storageKey: response.storageKey,
+            mountPath: APPROVED_PLAN_MOUNT_PATH,
+            contentType: "text/markdown",
+          },
+        ]);
+      } catch {
+        // Attachment plumbing must never block the build: implement from
+        // conversation history alone, and say so (non-blocking notice).
+        setPlanAttachFailed(true);
+        submitImplementTurn();
+      } finally {
+        setIsBuildingFromPlan(false);
+      }
+    })();
+  }, [setInteractionMode, sessionPlan, planDraft.readDraft, stigmer]);
+
+  // "Open plan" (a thread plan card, or the Artifacts facet's plan.md) → the
+  // panel's plan document tab. Opening the LATEST plan clears the historical
+  // selection (the tab shows the editable current plan); any other execution
+  // id opens that turn's plan read-only in the same tab.
+  const handleOpenPlan = useCallback(
+    (executionId: string) => {
+      setOpenPlanExecutionId(
+        sessionPlan && executionId === sessionPlan.executionId
+          ? null
+          : executionId,
+      );
+      panel.openPlanDocument();
+    },
+    [sessionPlan, panel.openPlanDocument],
+  );
+
+  // Open a transcript tool-call file path in the panel's read-only editor.
+  // Resolves the (possibly absolute / subdir-prefixed) path to a
+  // repo/root-relative selection the editor can fetch; on a definite hit it
+  // opens the file (expanding the panel) and returns `true` (handled),
+  // otherwise returns `false` so the path keeps its default copy / GitHub-link
+  // behavior. Writing the editor store re-renders only the panel subtree
+  // (which subscribes), never this streaming column.
+  const handleTranscriptFilePathClick = useCallback(
+    (path: string): boolean => {
+      const selection = resolveWorkspaceFileSelection(
+        path,
+        flow.workspace.entries,
+        flow.sandboxWorkspaceRoot,
+      );
+      if (!selection) return false;
+      panel.openFile(selection.entryId, selection.path);
+      return true;
+    },
+    [flow.workspace.entries, flow.sandboxWorkspaceRoot, panel.openFile],
+  );
 
   if (conv.isLoading) {
     return (
@@ -199,22 +483,36 @@ export function SessionViewer({
 
   return (
     <div className={cn("relative flex h-full w-full flex-col", className)}>
-      {headerActions && (
-        <div className="absolute top-2 right-6 z-10">
-          {headerActions}
-        </div>
-      )}
+      {/* Top-right controls: host actions + the panel chip. The chip is the
+          panel's always-mounted toggle; while collapsed it carries only the
+          pending-item count. Execution status is never surfaced as header
+          chrome — the thread itself communicates run state. */}
+      <div className="absolute top-2 right-6 z-10 flex items-center gap-2">
+        {headerActions}
+        <SessionPanelChip
+          isOpen={panel.isOpen}
+          onToggle={panel.isOpen ? panel.closePanel : panel.openPanel}
+          badgeCount={writeBackCount + artifactCount}
+        />
+      </div>
 
       <ThreadSelectionContext.Provider value={selectionStore}>
+        {/* One layout, collapsed by default: chat fills the row until the panel
+            opens, then becomes the fixed narrow pane on the left while the
+            panel takes the flexible region. Collapsing goes through the
+            split's `collapsedPane` (CSS, not conditional structure), so the
+            conversation is always the same first child and an open/close never
+            remounts it (invariant 1). */}
         <ResizableSplit
-          defaultSize={384}
-          minSize={280}
-          maxSize={600}
-          storageKey="stgm-session-inspector-width"
-          className={cn(
-            "min-h-0 flex-1",
-            "[&>*:nth-child(2)]:max-lg:hidden [&>*:nth-child(3)]:max-lg:hidden",
-          )}
+          resizablePane="primary"
+          collapsedPane={panel.isOpen ? "none" : "secondary"}
+          defaultSize={420}
+          minSize={320}
+          maxSize={640}
+          storageKey="stgm-session-chat-width"
+          responsiveCollapse={panel.isOpen ? "primary" : "none"}
+          ariaLabel="Resize chat panel"
+          className="min-h-0 flex-1"
           primary={
             <ConversationColumn
               flow={flow}
@@ -229,23 +527,37 @@ export function SessionViewer({
               enableLocal={enableLocal}
               onBrowseLocalFolder={onBrowseLocalFolder}
               onBuildFromPlan={handleBuildFromPlan}
+              onOpenPlan={handleOpenPlan}
+              isBuildingFromPlan={isBuildingFromPlan}
+              planAttachFailed={planAttachFailed}
+              onDismissPlanAttachFailed={() => setPlanAttachFailed(false)}
+              onFilePathClick={handleTranscriptFilePathClick}
               isEndUser={isEndUser}
             />
           }
           secondary={
-            <InspectorPanel
-              flow={flow}
-              org={org}
-              selectionStore={selectionStore}
-              onApplied={onApplied}
-              onImplementPlan={handleBuildFromPlan}
-              enableGitHub={enableGitHub}
-              enableLocal={enableLocal}
-              gitHubConnection={gitHubConnection}
-              onBrowseLocalFolder={onBrowseLocalFolder}
-              workspaceFileLister={workspaceFileLister}
-              isEndUser={isEndUser}
-            />
+            panel.isOpen ? (
+              <SessionPanelRegion
+                flow={flow}
+                org={org}
+                panel={panel}
+                accessSlot={accessSlot}
+                onApplied={onApplied}
+                onImplementPlan={handleBuildFromPlan}
+                implementPlanDisabled={!conv.canSendFollowUp || isBuildingFromPlan}
+                sessionPlan={sessionPlan}
+                streamingPlan={streamingPlan}
+                planDraft={planDraft}
+                openPlanExecutionId={openPlanExecutionId}
+                onOpenPlan={handleOpenPlan}
+                enableLocal={enableLocal}
+                onBrowseLocalFolder={onBrowseLocalFolder}
+                workspaceFileLister={workspaceFileLister}
+                workspaceFileReader={workspaceFileReader}
+                workspaceContentSearcher={workspaceContentSearcher}
+                isEndUser={isEndUser}
+              />
+            ) : null
           }
         />
       </ThreadSelectionContext.Provider>
@@ -270,10 +582,23 @@ interface ConversationColumnProps {
   readonly enableLocal: boolean;
   readonly onBrowseLocalFolder?: () => Promise<string | null>;
   readonly onBuildFromPlan: () => void;
+  /** Opens a plan in the panel's plan document tab (thread card "Open plan"). */
+  readonly onOpenPlan: (executionId: string) => void;
+  /** True while the approved plan is being uploaded ahead of the build turn. */
+  readonly isBuildingFromPlan: boolean;
+  /** True when the last build fell back to a message-only implement. */
+  readonly planAttachFailed: boolean;
+  readonly onDismissPlanAttachFailed: () => void;
+  /**
+   * Opens a transcript tool-call file path in the Viewer. Returns `true` when it
+   * resolved and opened the file (suppressing the link's default), `false` to
+   * let the path keep its copy / GitHub-link behavior.
+   */
+  readonly onFilePathClick: (path: string) => boolean;
   readonly isEndUser: boolean;
 }
 
-function ConversationColumn({
+const ConversationColumn = memo(function ConversationColumn({
   flow,
   modelId,
   setModelId,
@@ -286,6 +611,11 @@ function ConversationColumn({
   enableLocal,
   onBrowseLocalFolder,
   onBuildFromPlan,
+  onOpenPlan,
+  isBuildingFromPlan,
+  planAttachFailed,
+  onDismissPlanAttachFailed,
+  onFilePathClick,
   isEndUser,
 }: ConversationColumnProps) {
   const { conv } = flow;
@@ -340,14 +670,20 @@ function ConversationColumn({
         showFileReviewRecords
         onEditMessage={conv.isStoppable ? handleEditMessage : undefined}
         workspaceEntries={conv.workspaceEntries}
+        onFilePathClick={onFilePathClick}
         sandboxWorkspaceRoot={flow.sandboxWorkspaceRoot}
         onBuildFromPlan={onBuildFromPlan}
+        onOpenPlan={onOpenPlan}
         org={org}
-        planActionsDisabled={!conv.canSendFollowUp}
-        centerContent
+        planActionsDisabled={!conv.canSendFollowUp || isBuildingFromPlan}
+        planBuildPending={isBuildingFromPlan}
+        contentColumn="center"
         className="flex-1"
       />
-      <div className="mx-auto w-full max-w-3xl">
+      <div className={CONVERSATION_COLUMN_CLASS}>
+        {planAttachFailed && (
+          <PlanAttachFailedNotice onDismiss={onDismissPlanAttachFailed} />
+        )}
         {conv.isReconnecting && <ReconnectingIndicator />}
         {conv.connectTimedOut && (
           <ConnectTimedOutBanner onRetry={conv.reconnectStream} />
@@ -407,41 +743,165 @@ function ConversationColumn({
       </div>
     </div>
   );
-}
+});
 
 // ---------------------------------------------------------------------------
-// Inspector panel (right/secondary) — reads selection from store
+// Session panel region (right/secondary) — the unified panel's subtree
 // ---------------------------------------------------------------------------
 
-interface InspectorPanelProps {
+interface SessionPanelRegionProps {
   readonly flow: ReturnType<typeof useSessionPageFlow>;
   readonly org: string;
-  readonly selectionStore: SelectionStore;
+  /**
+   * The panel controller owned by `SessionViewer`. Its editor store is
+   * subscribed here (not at `SessionViewer`) so opening/switching files
+   * re-renders only this region, never the conversation column.
+   */
+  readonly panel: SessionPanelController;
+  /** Host access management control, surfaced in the Config facet. */
+  readonly accessSlot?: ReactNode;
   readonly onApplied?: (result: ApplyResourceResult) => void;
-  /** Implement a plan from the Artifacts tab (same action as the thread card). */
+  /** Implement a plan (plan tab primary + Artifacts preview action). */
   readonly onImplementPlan?: () => void;
-  readonly enableGitHub: boolean;
+  readonly implementPlanDisabled?: boolean;
+  /** The session's latest plan — the plan tab's editable content. */
+  readonly sessionPlan?: SessionPlan;
+  /**
+   * The plan the active turn is writing right now. While present it OWNS the
+   * plan tab: the tab renders the live document (`PlanStreamingDocument`)
+   * instead of a published plan — the in-flight plan is the newest
+   * deliverable, and the snap-back in `SessionViewer` has already cleared any
+   * historical selection when this identity appeared.
+   */
+  readonly streamingPlan?: SessionStreamingPlan;
+  /** Viewer-owned plan draft (survives panel collapse and view switches). */
+  readonly planDraft: PlanDraftController;
+  /**
+   * Which plan the plan document tab shows: `null` for the latest (editable),
+   * or an execution id whose historical plan renders read-only.
+   */
+  readonly openPlanExecutionId: string | null;
+  /** Opens a plan in the plan tab (routed to the Artifacts facet's plan.md). */
+  readonly onOpenPlan: (executionId: string) => void;
   readonly enableLocal: boolean;
-  readonly gitHubConnection?: UseGitHubConnectionReturn;
   readonly onBrowseLocalFolder?: () => Promise<string | null>;
   readonly workspaceFileLister?: WorkspaceFileLister;
+  readonly workspaceFileReader?: WorkspaceFileReader;
+  readonly workspaceContentSearcher?: WorkspaceContentSearcher;
   readonly isEndUser: boolean;
 }
 
-function InspectorPanel({
+function SessionPanelRegion({
   flow,
   org,
-  selectionStore,
+  panel,
+  accessSlot,
   onApplied,
   onImplementPlan,
-  enableGitHub,
+  implementPlanDisabled,
+  sessionPlan,
+  streamingPlan,
+  planDraft,
+  openPlanExecutionId,
+  onOpenPlan,
   enableLocal,
-  gitHubConnection,
   onBrowseLocalFolder,
   workspaceFileLister,
+  workspaceFileReader,
+  workspaceContentSearcher,
   isEndUser,
-}: InspectorPanelProps) {
+}: SessionPanelRegionProps) {
   const selectedItem = useSelectedThreadItem();
+  const { editors, activeKey, activeFile, reveal } = useWorkspaceEditors(
+    panel.editorsStore,
+  );
+
+  // Honor a pending jump-to-line reveal only while it targets the active editor
+  // (an EditorReveal is structurally a RevealTarget). Switching tabs changes
+  // activeKey, so a stale reveal for another tab is naturally ignored.
+  const activeReveal =
+    reveal && reveal.key === activeKey ? reveal : undefined;
+
+  // This region is the level that subscribes to thread selection; report it to
+  // the controller so an open panel can auto-surface the Inspect view. (A
+  // collapsed panel never sees — and deliberately never reacts to — selection.)
+  useEffect(() => {
+    panel.notifySelection(selectedItem);
+  }, [selectedItem, panel.notifySelection]);
+
+  // Correlate the active file with its session change for diff-as-default
+  // (DD-06 parity with the transcript's rendering of the same change). A
+  // virtual document (the plan tab) is not a workspace file — pass `null` so
+  // the hook's no-file guard skips the net-change fold entirely, rather than
+  // relying on the correlation harmlessly missing the sentinel id.
+  const openFileChange = useOpenFileChange(
+    activeFile && !isVirtualEntryId(activeFile.entryId) ? activeFile : null,
+    flow.allExecutions,
+    flow.workspace.entries,
+    flow.sandboxWorkspaceRoot,
+  );
+
+  // Resolve WHICH plan the plan document tab shows: the latest (editable,
+  // buildable, draft-backed) unless a historical execution id was opened —
+  // that plan renders read-only. A stale id (execution gone, or its plan
+  // artifact missing) degrades to the latest rather than an empty tab.
+  const openPlan = useMemo<SessionPlan | undefined>(() => {
+    if (
+      openPlanExecutionId === null ||
+      openPlanExecutionId === sessionPlan?.executionId
+    ) {
+      return sessionPlan;
+    }
+    const execution = flow.allExecutions.find(
+      (e) => e.metadata?.id === openPlanExecutionId,
+    );
+    const artifact = findPlanArtifact(execution);
+    return artifact
+      ? { executionId: openPlanExecutionId, artifact }
+      : sessionPlan;
+  }, [openPlanExecutionId, sessionPlan, flow.allExecutions]);
+
+  // The plan document tab (SurfaceVirtualDocument): the panel's editor-area
+  // rendering of the session's plan. Three states, in precedence order:
+  // a STREAMING plan owns the tab (live document, no actions); otherwise the
+  // resolved `openPlan` renders in the editor (keyed by plan identity so
+  // switching plans resets view state cleanly); otherwise an honest empty
+  // notice (DD-006) — reachable only when a streaming plan auto-opened the
+  // tab and its turn then ended without publishing, with no earlier plan to
+  // fall back to.
+  const openPlanIsLatest = openPlan?.executionId === sessionPlan?.executionId;
+  const virtualDocuments = useMemo<readonly SurfaceVirtualDocument[]>(() => {
+    return [
+      {
+        entryId: PLAN_DOCUMENT_ENTRY_ID,
+        path: PLAN_DOCUMENT_PATH,
+        content: streamingPlan ? (
+          <PlanStreamingDocument
+            key={`streaming-${streamingPlan.executionId}`}
+            displayText={streamingPlan.displayText}
+          />
+        ) : openPlan ? (
+          <PlanEditor
+            key={planDraftKey(openPlan)}
+            plan={openPlan}
+            draft={openPlanIsLatest ? planDraft : undefined}
+            onBuildFromPlan={openPlanIsLatest ? onImplementPlan : undefined}
+            buildDisabled={implementPlanDisabled}
+            readOnly={!openPlanIsLatest}
+          />
+        ) : (
+          <PlanUnavailableNotice />
+        ),
+      },
+    ];
+  }, [
+    streamingPlan,
+    openPlan,
+    openPlanIsLatest,
+    planDraft,
+    onImplementPlan,
+    implementPlanDisabled,
+  ]);
 
   const handleRemoveAgent = useCallback(() => {
     flow.setAgentRef(null);
@@ -470,7 +930,7 @@ function InspectorPanel({
     [flow.skillRefs, flow.setSkillRefs],
   );
 
-  const sessionConfig = useMemo(
+  const sessionConfig = useMemo<SetupTabProps>(
     () => ({
       agentRef: flow.agentRef,
       isDefaultAgent: flow.isDefaultAgent,
@@ -480,8 +940,8 @@ function InspectorPanel({
       harness: flow.harness,
       executionTarget: flow.executionTarget,
       modelId: flow.model[0],
-      // End users see the configuration but cannot strip it — the Setup
-      // tab renders read-only without mutation callbacks (DD-011).
+      // End users see the configuration but cannot strip it — the Config
+      // facet renders read-only without mutation callbacks (DD-011).
       mutations: isEndUser
         ? undefined
         : {
@@ -489,42 +949,59 @@ function InspectorPanel({
             onRemoveMcp: handleRemoveMcp,
             onRemoveSkill: handleRemoveSkill,
           },
+      accessSlot,
     }),
     [
       flow.agentRef, flow.isDefaultAgent, flow.mcpServerUsages, flow.skillRefs,
       flow.sessionVariables, flow.harness, flow.executionTarget, flow.model,
       isEndUser, handleRemoveAgent, handleRemoveMcp, handleRemoveSkill,
+      accessSlot,
     ],
   );
 
-  const workspaceConfig = useMemo(
-    () => ({
-      actions: {
-        workspace: flow.workspace,
-        enableGitHub,
-        enableLocal,
-        gitHubConnection,
-        onBrowseLocalFolder,
-        workspaceFileLister,
-      },
-    }),
-    [flow.workspace, enableGitHub, enableLocal, gitHubConnection, onBrowseLocalFolder, workspaceFileLister],
-  );
+  // The session facets (Config / Changes / Artifacts / Usage / Inspect) as
+  // injected rail views — the full inspector feature set inside one panel.
+  const railViews = useSessionRailViews({
+    allExecutions: flow.allExecutions,
+    org,
+    sessionConfig,
+    selectedItem,
+    onApplied,
+    onImplementPlan,
+    onOpenPlan,
+  });
+
+  // Explorer-footer folder attach (desktop only — needs the native picker).
+  // Same wiring the retired Workspace tab used.
+  const canAddLocalFolder = enableLocal && !!onBrowseLocalFolder;
+  const handleAddLocalFolder = useCallback(async () => {
+    const path = await onBrowseLocalFolder?.();
+    if (path) flow.workspace.addLocalPath(path);
+  }, [onBrowseLocalFolder, flow.workspace.addLocalPath]);
 
   return (
-    <aside className="flex h-full flex-col overflow-hidden">
-      <SessionInspector
-        displayExecution={flow.displayExecution}
-        allExecutions={flow.allExecutions}
-        org={org}
-        selectedItem={selectedItem}
-        onApplied={onApplied}
-        onImplementPlan={onImplementPlan}
-        sessionConfig={sessionConfig}
-        workspaceConfig={workspaceConfig}
-        className="min-h-0 flex-1"
-      />
-    </aside>
+    <WorkspaceSurface
+      entries={flow.workspace.entries}
+      lister={workspaceFileLister}
+      reader={workspaceFileReader}
+      searcher={workspaceContentSearcher}
+      view={panel.view}
+      onViewChange={panel.setView}
+      extraViews={railViews}
+      virtualDocuments={virtualDocuments}
+      onRemoveEntry={flow.workspace.remove}
+      onAddLocalFolder={canAddLocalFolder ? handleAddLocalFolder : undefined}
+      editors={editors}
+      selectedFile={activeFile}
+      onOpenFile={panel.openFile}
+      onActivateEditor={panel.activateEditor}
+      onPinEditor={panel.pinEditor}
+      onCloseEditor={panel.closeEditor}
+      onCollapse={panel.closePanel}
+      change={openFileChange ?? undefined}
+      reveal={activeReveal}
+      className="h-full"
+    />
   );
 }
 
@@ -593,6 +1070,56 @@ function AutoApproveIndicator({ onTurnOff }: { onTurnOff: () => void }) {
   );
 }
 
+/**
+ * The plan document tab's empty state — reachable only when a streaming plan
+ * auto-opened the tab and its turn then ended without publishing (stopped or
+ * failed) while the session has no earlier published plan to fall back to.
+ * The partial plan reverts to the conversation (the thread un-collapses it),
+ * so the notice points there. Never a blank pane (DD-006).
+ */
+function PlanUnavailableNotice() {
+  return (
+    <div
+      role="status"
+      className="mx-auto flex w-full max-w-3xl flex-col items-center gap-1 px-4 py-8 text-center"
+    >
+      <p className="text-xs font-medium text-foreground">
+        This turn ended before a plan was completed.
+      </p>
+      <p className="text-xs text-muted-foreground">
+        Any partial plan text remains in the conversation.
+      </p>
+    </div>
+  );
+}
+
+/**
+ * Non-blocking notice for a build that fell back to message-only implement
+ * (the approved plan.md could not be uploaded/attached). The build itself
+ * proceeded — the agent follows the plan from conversation history — so this
+ * is a status, not an alert, and it is dismissible.
+ */
+function PlanAttachFailedNotice({ onDismiss }: { onDismiss: () => void }) {
+  return (
+    <div
+      role="status"
+      className="flex items-center gap-2 border-t border-border-muted px-4 py-1.5 text-xs text-muted-foreground"
+    >
+      <span className="min-w-0 flex-1 truncate">
+        Couldn&rsquo;t attach plan.md — the agent will follow the plan from the
+        conversation instead.
+      </span>
+      <button
+        type="button"
+        onClick={onDismiss}
+        className="shrink-0 rounded font-medium text-foreground underline-offset-2 hover:underline focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring"
+      >
+        Dismiss
+      </button>
+    </div>
+  );
+}
+
 function SendErrorBanner({ error }: { error: Error }) {
   if (isSecretFlowError(error)) {
     return <SecretFlowErrorGuide error={error} className="mx-4 my-2" />;