Remove palette manager from Workspace V2 toolState producer flow while preserving global palette state - PR_26124_013-remove-palette-manager-producer

Author: DavidQ

tests/playwright/tool-validation/workspace-v2.tool-validation.spec.js

@@ -127,6 +127,7 @@ const toolSelectors = {
 
 const auditTools = parseAuditToolList();
 const toolIds = auditTools.filter((toolId) => toolId !== "workspace-v2");
+const workspaceToolStateCapableToolIds = toolIds.filter((toolId) => toolId !== "palette-manager-v2");
 
 test("@workspace-v2 tool validation list includes all audited tools", async () => {
   expect(auditTools).toEqual([
@@ -166,7 +167,7 @@ test("@workspace-v2 invalid workspace manifest payloadJson is rejected", async (
   }
 });
 
-for (const toolId of toolIds) {
+for (const toolId of workspaceToolStateCapableToolIds) {
   const selectors = toolSelectors[toolId];
   if (!selectors) {
     continue;

tests/playwright/workspace-v2.validation.spec.js

@@ -27,6 +27,7 @@ test.describe("Workspace V2 validation coverage", () => {
     try {
       await page.goto(`${server.baseUrl}/tools/workspace-v2/index.html`);
       await ctrlTapClick(page, page.getByRole("button", { name: "Full Reset" }));
+      await expect(page.locator("#workspaceV2ToolSelect option[value='palette-manager-v2']")).toHaveCount(0);
       await expect(page.locator("#workspaceV2WorkspaceToolsSummary")).toContainText("palette-browser");
       await expect(page.locator("#workspaceV2WorkspaceToolsSummary")).toContainText("workspace-v2");
 

tools/workspace-v2/index.js

@@ -681,17 +681,6 @@ class WorkspaceV2ToolStateProducer {
   }
 
   createProducerPayloadForTool(toolId) {
-    if (toolId === "palette-manager-v2") {
-      return this.withToolStateVersion({
-        toolId: "palette-manager-v2",
-        payloadJson: {
-          paletteDocument: {
-            name: "Workspace Palette",
-            swatches: []
-          }
-        }
-      });
-    }
     return this.withToolStateVersion({
       toolId,
       payloadJson: {}