Clarify Workspace V2 tool launcher to distinguish standalone vs session-based launches - PR_11_317

Author: DavidQ

docs/dev/codex_commands.md

@@ -159,3 +159,10 @@ PR_11_316
 ```bash
 npx @openai/codex run --model gpt-5.3-codex --reasoning medium "Implement PR_11_316: Harden Asset Manager V2 add/remove actions by rejecting duplicate ids, blank/whitespace fields, and missing remove ids with actionable status messages while preserving valid persistence behavior."
 ```
+
+---
+PR_11_317
+
+```bash
+npx @openai/codex run --model gpt-5.3-codex --reasoning medium "Implement PR_11_317: Clarify Workspace V2 Asset Manager launcher with explicit no-session vs active-session state while preserving session-routed launch behavior."
+```

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Harden Asset Manager V2 add/remove rejection paths and preserve valid persistence behavior - PR 11.316
+Clarify Workspace V2 Asset Manager launcher session/no-session state and prevent no-session confusion - PR 11.317

docs/dev/reports/PR_11_317_report.md

@@ -0,0 +1,30 @@
+# PR_11_317 Report
+
+## Purpose
+Clarify Workspace V2 Asset Manager launcher behavior so standalone/no-session state is explicitly labeled and session-based launch is unambiguous.
+
+## Files Changed
+- `tools/workspace-v2/index.html`
+- `tools/workspace-v2/index.js`
+- `docs/pr/PR_11_317_WORKSPACE_TOOL_LAUNCHER_SESSION_CLARITY/PLAN_PR.md`
+- `docs/pr/PR_11_317_WORKSPACE_TOOL_LAUNCHER_SESSION_CLARITY/BUILD_PR.md`
+- `docs/dev/reports/PR_11_317_report.md`
+- `docs/dev/codex_commands.md`
+- `docs/dev/commit_comment.txt`
+
+## Implementation Summary
+- Set tools launcher button default to `Open Asset Manager V2 (no session)`.
+- Added computed launch readiness and label in Workspace V2 UI state model:
+  - launch ready only when there is a valid active session payload with `payloadJson`.
+- Wired button state in renderer:
+  - disabled + no-session label when active session is missing
+  - enabled + active-session label when launch can route through active session.
+- Kept existing `openAssetManagerFromWorkspace()` launch flow unchanged for valid active session routing.
+
+## Validation Commands
+- `node --check tools/workspace-v2/index.js` -> **PASS**
+- `node tests/runtime/V2WorkspaceAssetManagerLaunch.test.mjs` -> **PASS**
+
+## Full Samples Smoke Decision
+- **Skipped** full samples smoke.
+- Reason: this PR is Workspace V2 UI-only and validated with targeted runtime test plus syntax check.

docs/pr/PR_11_317_WORKSPACE_TOOL_LAUNCHER_SESSION_CLARITY/BUILD_PR.md

@@ -0,0 +1,18 @@
+# BUILD_PR_11_317
+
+## Implementation
+- Updated Workspace V2 tools launcher button text to start in explicit no-session mode:
+  - `Open Asset Manager V2 (no session)`
+- Extended `computeWorkspaceSessionUiStateModel()` in `tools/workspace-v2/index.js` with:
+  - `assetManagerLaunchReady`
+  - `assetManagerLaunchLabel`
+- Updated `renderWorkspaceSessionUiStateModel(model)` to:
+  - disable launch when no valid active session payload is available
+  - relabel button dynamically:
+    - `Open Asset Manager V2 (active session)` when launch is session-routed
+    - `Open Asset Manager V2 (no session)` when launch is blocked
+- Preserved existing session-based launch path in `openAssetManagerFromWorkspace()` and did not alter producer/recent-session flows.
+
+## Validation
+- `node --check tools/workspace-v2/index.js`
+- `node tests/runtime/V2WorkspaceAssetManagerLaunch.test.mjs`

docs/pr/PR_11_317_WORKSPACE_TOOL_LAUNCHER_SESSION_CLARITY/PLAN_PR.md

@@ -0,0 +1,17 @@
+# PLAN_PR_11_317
+
+## Purpose
+Clarify Workspace V2 Asset Manager launch behavior so users can distinguish session-based launch from no-session state.
+
+## Scope
+- `tools/workspace-v2/index.html`
+- `tools/workspace-v2/index.js`
+- `docs/dev/reports/PR_11_317_report.md`
+- `docs/dev/codex_commands.md`
+- `docs/dev/commit_comment.txt`
+
+## Steps
+1. Update Asset Manager launcher button default text to indicate no-session state.
+2. Drive launcher button enable/disable + label from the existing Workspace V2 computed UI state model.
+3. Keep launch path routed through active session payload when present.
+4. Run targeted validation for changed JS and targeted Workspace Asset Manager launch runtime test.

tools/workspace-v2/index.html

@@ -36,7 +36,7 @@ <h2>Producer</h2>
       <section class="hub-panel">
         <h2>Tools</h2>
         <p>Open Workspace V2 tools from this explicit launcher menu.</p>
-        <button id="workspaceV2OpenAssetManagerButton" type="button">Open Asset Manager V2</button>
+        <button id="workspaceV2OpenAssetManagerButton" type="button">Open Asset Manager V2 (no session)</button>
       </section>
 
       <section class="hub-panel">

tools/workspace-v2/index.js

@@ -879,6 +879,12 @@ class WorkspaceV2SessionProducer {
           : (diffCanCompute ? "Selections are valid." : "Select two different sessions to compute diff.")
       );
     const authoritativeLastMergedHostContextId = this.resolveAuthoritativeLastMergedHostContextId();
+    const assetManagerLaunchReady = Boolean(
+      this.isValidSessionPayload(this.currentSessionPayload) &&
+      this.currentSessionPayload.payloadJson &&
+      typeof this.currentSessionPayload.payloadJson === "object" &&
+      !Array.isArray(this.currentSessionPayload.payloadJson)
+    );
     const mergePreviewVisible = Boolean(
       this.pendingMergePreview ||
       this.mergeOutputSelectionKey ||
@@ -914,7 +920,11 @@ class WorkspaceV2SessionProducer {
       mergeSelectionText,
       mergePreviewStale: Boolean(this.pendingMergePreview && !mergePreviewFresh),
       mergePreviewVisible,
-      undoEnabled: Boolean(authoritativeLastMergedHostContextId)
+      undoEnabled: Boolean(authoritativeLastMergedHostContextId),
+      assetManagerLaunchReady,
+      assetManagerLaunchLabel: assetManagerLaunchReady
+        ? "Open Asset Manager V2 (active session)"
+        : "Open Asset Manager V2 (no session)"
     };
   }
 
@@ -946,6 +956,8 @@ class WorkspaceV2SessionProducer {
     this.undoLastMergeButton.disabled = !model.undoEnabled;
     this.mergeOutputNode.hidden = !model.mergePreviewVisible;
     this.renderMergeConflictSummary();
+    this.openAssetManagerButton.disabled = !model.assetManagerLaunchReady;
+    this.openAssetManagerButton.textContent = model.assetManagerLaunchLabel;
   }
 
   refreshWorkspaceSessionUiStateModel(actionName = "refresh_load") {