@taskforcehq/taskforce 0.3.305 → 0.3.307

package/dist/components/features/TaskSettings.test.js

@@ -1,5 +1,5 @@
 import { jsx as _jsx } from "react/jsx-runtime";
-import { fireEvent, render, screen } from '@testing-library/react';
+import { fireEvent, render, screen, waitFor } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { describe, it, expect, vi } from 'vitest';
 import { TaskSettings } from './TaskSettings';
@@ -109,8 +109,22 @@ describe('TaskSettings Component', () => {
         };
         return render(_jsx(TaskSettings, { ...props }));
     };
+    const renderMcpComponent = (overrides = {}) => {
+        const { settingsModel: settingsModelOverrides = {}, ...rest } = overrides;
+        return renderComponent({
+            ...rest,
+            renderMode: 'mcp-only',
+            settingsModel: {
+                ...settingsModelOverrides,
+                initialSection: 'mcp'
+            }
+        });
+    };
     const openMcpSettings = async (user) => {
-        await user.click(screen.getByRole('button', { name: /MCP/i }));
+        const mcpButton = screen.queryByRole('button', { name: /^MCP$/i });
+        if (mcpButton) {
+            await user.click(mcpButton);
+        }
     };
     const openCloudMcpSettings = async (user) => {
         await openMcpSettings(user);
@@ -139,7 +153,7 @@ describe('TaskSettings Component', () => {
         expect(screen.queryByRole('button', { name: /workflows/i })).not.toBeInTheDocument();
         expect(screen.queryByRole('button', { name: /agents/i })).not.toBeInTheDocument();
         expect(screen.getByRole('button', { name: /taxonomy/i })).toBeInTheDocument();
-        expect(screen.getByRole('button', { name: /^mcp$/i })).toBeInTheDocument();
+        expect(screen.queryByRole('button', { name: /^mcp$/i })).not.toBeInTheDocument();
         expect(screen.getByRole('button', { name: /integrations/i })).toBeInTheDocument();
     });
     it('4.1b General tab removes old profile/account chrome', () => {
@@ -852,7 +866,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'cloud',
                 isAuthenticated: true,
@@ -860,8 +874,14 @@ describe('TaskSettings Component', () => {
                 fetchCloudAuthApi
             }
         });
-        await user.click(screen.getByRole('button', { name: /MCP/i }));
+        await openMcpSettings(user);
         expect(await screen.findByText('Cloud MCP Configuration')).toBeInTheDocument();
+        expect(screen.getByText(/Cloud MCP connects agents to Taskforce through the hosted MCP endpoint/i)).toBeInTheDocument();
+        expect(screen.getByText(/Once the MCP is configured, prompt your agent:/i)).toBeInTheDocument();
+        expect(screen.getByRole('button', { name: 'Resolve your profile with Taskforce.' })).toBeInTheDocument();
+        const cloudInstructionLabel = screen.getByText('General MCP Configuration Instructions');
+        const cloudProfilePrompt = screen.getByText(/Once the MCP is configured, prompt your agent:/i);
+        expect(cloudInstructionLabel.compareDocumentPosition(cloudProfilePrompt) & Node.DOCUMENT_POSITION_FOLLOWING).toBeTruthy();
         expect(screen.getByRole('button', { name: 'Generate Token' })).toBeInTheDocument();
         expect(screen.queryByRole('button', { name: 'Local MCP' })).not.toBeInTheDocument();
         expect(screen.queryByText('MCP Path Mapping (Host Root)')).not.toBeInTheDocument();
@@ -870,13 +890,28 @@ describe('TaskSettings Component', () => {
             credentials: 'include'
         }));
     }, 10000);
+    it('4.20a MCP setup tells users how to register the agent profile', async () => {
+        renderMcpComponent({
+            settingsModel: {
+                runtimeMode: 'local',
+                isAuthenticated: false
+            }
+        });
+        expect(screen.getByText(/Local MCP connects through your local Taskforce install/i)).toBeInTheDocument();
+        expect(screen.getByText(/Once the MCP is configured, prompt your agent:/i)).toBeInTheDocument();
+        expect(screen.getByRole('button', { name: 'Resolve your profile with Taskforce.' })).toBeInTheDocument();
+        const localInstructionLabel = screen.getByText('General MCP Configuration Instructions');
+        const localProfilePrompt = screen.getByText(/Once the MCP is configured, prompt your agent:/i);
+        expect(localInstructionLabel.compareDocumentPosition(localProfilePrompt) & Node.DOCUMENT_POSITION_FOLLOWING).toBeTruthy();
+        expect(screen.queryByText('Ask Your Agent to Register')).not.toBeInTheDocument();
+    });
     it('4.21 Local authenticated runtime defaults MCP tab to local with local-first toggle', async () => {
         const fetchCloudAuthApi = vi.fn(async () => ({
             ok: true,
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -885,7 +920,7 @@ describe('TaskSettings Component', () => {
                 fetchCloudAuthApi
             }
         });
-        await user.click(screen.getByRole('button', { name: /MCP/i }));
+        await openMcpSettings(user);
         const localButton = await screen.findByRole('button', { name: 'Local MCP' });
         const cloudButton = screen.getByRole('button', { name: 'Cloud MCP' });
         expect(localButton.compareDocumentPosition(cloudButton) & Node.DOCUMENT_POSITION_FOLLOWING).toBeTruthy();
@@ -926,7 +961,7 @@ describe('TaskSettings Component', () => {
             };
         });
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -947,11 +982,11 @@ describe('TaskSettings Component', () => {
                 expiresAt: null
             })
         }));
-        expect((await screen.findAllByText('tfmcp_example_secret')).length).toBeGreaterThan(0);
+        expect(await screen.findByDisplayValue('tfmcp_example_secret')).toBeInTheDocument();
     }, 10000);
     it('4.23c MCP token name defaults to the workspace name', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -982,7 +1017,7 @@ describe('TaskSettings Component', () => {
             };
         });
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -991,9 +1026,9 @@ describe('TaskSettings Component', () => {
             }
         });
         await openCloudMcpSettings(user);
-        await user.click(screen.getByRole('button', { name: 'Test Connection' }));
-        await user.type(await screen.findByPlaceholderText('tfmcp_xxxxx_replace-me'), 'tfmcp_example_secret');
-        await user.click(screen.getAllByRole('button', { name: 'Test Connection' })[1]);
+        expect(screen.queryByText('Test Connection')).not.toBeInTheDocument();
+        await user.type(await screen.findByPlaceholderText('Generate a token to populate the full secret'), 'tfmcp_example_secret');
+        await user.click(screen.getByRole('button', { name: 'Test Token' }));
         expect(fetchCloudAuthApi).toHaveBeenCalledWith('/api/taskforce/settings/mcp/test-connection', expect.objectContaining({
             method: 'POST',
             credentials: 'include'
@@ -1023,7 +1058,7 @@ describe('TaskSettings Component', () => {
             };
         });
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1033,9 +1068,8 @@ describe('TaskSettings Component', () => {
             }
         });
         await openCloudMcpSettings(user);
-        await user.click(screen.getByRole('button', { name: 'Test Connection' }));
-        await user.type(await screen.findByPlaceholderText('tfmcp_xxxxx_replace-me'), 'tfmcp_example_secret');
-        await user.click(screen.getAllByRole('button', { name: 'Test Connection' })[1]);
+        await user.type(await screen.findByPlaceholderText('Generate a token to populate the full secret'), 'tfmcp_example_secret');
+        await user.click(screen.getByRole('button', { name: 'Test Token' }));
         expect(await screen.findByText('The dedicated MCP host is responding with a gateway error.')).toBeInTheDocument();
         expect(screen.getByText(/Endpoint: https:\/\/performance-mcp\.taskforcehq\.ai\/mcp/i)).toBeInTheDocument();
         expect(screen.getByText(/Error code: MCP_ENDPOINT_GATEWAY_ERROR/i)).toBeInTheDocument();
@@ -1047,7 +1081,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1069,7 +1103,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1089,7 +1123,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1109,7 +1143,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1131,7 +1165,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1153,7 +1187,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1175,7 +1209,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1197,7 +1231,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1219,7 +1253,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1242,7 +1276,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1269,7 +1303,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1346,7 +1380,7 @@ describe('TaskSettings Component', () => {
             json: async () => ({ items: [] })
         }));
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1361,7 +1395,7 @@ describe('TaskSettings Component', () => {
     }, 10000);
     it('4.23g Cloud MCP configuration offers snippet and full-config copy actions', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1374,7 +1408,7 @@ describe('TaskSettings Component', () => {
     }, 10000);
     it('4.23g1 Cloud MCP copy warns before a token is generated', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1388,7 +1422,7 @@ describe('TaskSettings Component', () => {
     }, 10000);
     it('4.23g2 Cloud MCP settings do not surface automation identity controls', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1402,7 +1436,7 @@ describe('TaskSettings Component', () => {
     }, 10000);
     it('4.23g3 Cloud MCP config does not include AI identity headers by default', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1415,34 +1449,52 @@ describe('TaskSettings Component', () => {
         expect(screen.queryByText(/X-Taskforce-AI-Name/i)).not.toBeInTheDocument();
         expect(screen.queryByText(/X-Taskforce-AI-Profile-Token/i)).not.toBeInTheDocument();
     }, 10000);
-    it('4.23h Active tokens sort first and revoked tokens do not show revoke or regenerate actions', async () => {
+    it('4.23h Active tokens sort first and inactive tokens only show delete actions', async () => {
         const user = userEvent.setup();
-        const fetchCloudAuthApi = vi.fn(async () => ({
-            ok: true,
-            json: async () => ({
-                items: [
-                    {
-                        id: 'token-revoked',
-                        name: 'Old Token',
-                        scopes: ['tasks:read'],
-                        tokenPrefix: 'tfmcp_old',
-                        status: 'revoked',
-                        createdAt: '2026-03-01T00:00:00.000Z',
-                        lastUsedAt: null,
-                    },
-                    {
-                        id: 'token-active',
-                        name: 'Current Token',
-                        scopes: ['tasks:read', 'tasks:write'],
-                        tokenPrefix: 'tfmcp_new',
-                        status: 'active',
-                        createdAt: '2026-03-02T00:00:00.000Z',
-                        lastUsedAt: null,
-                    }
-                ]
-            })
-        }));
-        renderComponent({
+        const confirmSpy = vi.spyOn(window, 'confirm').mockImplementation(() => true);
+        const fetchCloudAuthApi = vi.fn(async (path, init) => {
+            if (path === '/api/taskforce/settings/mcp/tokens/token-expired' && init?.method === 'DELETE') {
+                return {
+                    ok: true,
+                    json: async () => ({ success: true, deleted: true })
+                };
+            }
+            return {
+                ok: true,
+                json: async () => ({
+                    items: [
+                        {
+                            id: 'token-expired',
+                            name: 'Expired Token',
+                            scopes: ['tasks:read'],
+                            tokenPrefix: 'tfmcp_expired',
+                            status: 'expired',
+                            createdAt: '2026-03-03T00:00:00.000Z',
+                            lastUsedAt: null,
+                        },
+                        {
+                            id: 'token-revoked',
+                            name: 'Old Token',
+                            scopes: ['tasks:read'],
+                            tokenPrefix: 'tfmcp_old',
+                            status: 'revoked',
+                            createdAt: '2026-03-01T00:00:00.000Z',
+                            lastUsedAt: null,
+                        },
+                        {
+                            id: 'token-active',
+                            name: 'Current Token',
+                            scopes: ['tasks:read', 'tasks:write'],
+                            tokenPrefix: 'tfmcp_new',
+                            status: 'active',
+                            createdAt: '2026-03-02T00:00:00.000Z',
+                            lastUsedAt: null,
+                        }
+                    ]
+                })
+            };
+        });
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1452,16 +1504,27 @@ describe('TaskSettings Component', () => {
         });
         await openCloudMcpSettings(user);
         expect(await screen.findByText('Current Token')).toBeInTheDocument();
+        expect(screen.getByText('Expired Token')).toBeInTheDocument();
         expect(screen.getByText('Old Token')).toBeInTheDocument();
-        const tokenLabels = screen.getAllByText(/^(Current Token|Old Token)$/);
+        const tokenLabels = screen.getAllByText(/^(Current Token|Expired Token|Old Token)$/);
         expect(tokenLabels[0]).toHaveTextContent('Current Token');
-        expect(tokenLabels[1]).toHaveTextContent('Old Token');
+        expect(tokenLabels[1]).toHaveTextContent('Expired Token');
+        expect(tokenLabels[2]).toHaveTextContent('Old Token');
         expect(screen.getAllByRole('button', { name: 'Regenerate' })).toHaveLength(1);
         expect(screen.getAllByRole('button', { name: 'Revoke' })).toHaveLength(1);
+        expect(screen.getAllByRole('button', { name: 'Delete' })).toHaveLength(2);
+        await user.click(screen.getAllByRole('button', { name: 'Delete' })[0]);
+        expect(confirmSpy).toHaveBeenCalledWith('Delete this token record? This does not affect active tokens.');
+        expect(fetchCloudAuthApi).toHaveBeenCalledWith('/api/taskforce/settings/mcp/tokens/token-expired', expect.objectContaining({
+            method: 'DELETE',
+            credentials: 'include'
+        }));
+        await waitFor(() => expect(screen.queryByText('Expired Token')).not.toBeInTheDocument());
+        confirmSpy.mockRestore();
     }, 10000);
     it('4.23f Cloud MCP snippet keeps the server id short enough for client tool-name limits', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: true,
@@ -1480,33 +1543,59 @@ describe('TaskSettings Component', () => {
         expect(serverId.length).toBeLessThanOrEqual(33);
         expect(`mcp_${serverId}_set_schedule_preferences`.length).toBeLessThanOrEqual(64);
     }, 10000);
-    it('4.24 Local signed-out runtime shows local MCP only', async () => {
+    it('4.24 Local signed-out runtime keeps Cloud MCP visible with an auth prompt', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        const fetchCloudAuthApi = vi.fn(async () => ({
+            ok: true,
+            json: async () => ({ items: [] })
+        }));
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
-                isAuthenticated: false
+                isAuthenticated: false,
+                fetchCloudAuthApi
             }
         });
-        await user.click(screen.getByRole('button', { name: /MCP/i }));
-        expect(screen.getByText('Local MCP')).toBeInTheDocument();
+        await openMcpSettings(user);
+        expect(screen.getByRole('button', { name: 'Local MCP' })).toBeInTheDocument();
+        expect(screen.getByRole('button', { name: 'Cloud MCP' })).toBeInTheDocument();
         expect(screen.getByRole('combobox', { name: 'MCP client' })).toHaveValue('general');
         expect(screen.getByText('General MCP Configuration Instructions')).toBeInTheDocument();
         expect(screen.queryByText('MCP Path Mapping (Host Root)')).not.toBeInTheDocument();
         await user.click(screen.getByRole('button', { name: 'Advanced' }));
         expect(screen.getByText('MCP Path Mapping (Host Root)')).toBeInTheDocument();
+        await user.click(screen.getByRole('button', { name: 'Cloud MCP' }));
+        expect(screen.getByText('Sign in to use Cloud MCP')).toBeInTheDocument();
+        expect(screen.getByText(/Register or sign in to generate cloud MCP tokens/i)).toBeInTheDocument();
         expect(screen.queryByText('Cloud MCP Configuration')).not.toBeInTheDocument();
-        expect(screen.queryByRole('button', { name: 'Cloud MCP' })).not.toBeInTheDocument();
+        expect(fetchCloudAuthApi).not.toHaveBeenCalled();
     });
-    it('4.24a Local MCP selector offers General first and keeps the rest alphabetical', async () => {
+    it('4.24a Local signed-out Cloud MCP prompt can open login and registration', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        const onOpenCloudAuth = vi.fn();
+        renderMcpComponent({
+            settingsModel: {
+                runtimeMode: 'local',
+                isAuthenticated: false,
+                onOpenCloudAuth
+            }
+        });
+        await openMcpSettings(user);
+        await user.click(screen.getByRole('button', { name: 'Cloud MCP' }));
+        await user.click(screen.getByRole('button', { name: 'Sign In' }));
+        expect(onOpenCloudAuth).toHaveBeenCalledWith('login');
+        await user.click(screen.getByRole('button', { name: 'Register' }));
+        expect(onOpenCloudAuth).toHaveBeenCalledWith('register');
+    });
+    it('4.24b Local MCP selector offers General first and keeps the rest alphabetical', async () => {
+        const user = userEvent.setup();
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: false
             }
         });
-        await user.click(screen.getByRole('button', { name: /MCP/i }));
+        await openMcpSettings(user);
         const options = screen.getAllByRole('option').map((option) => option.textContent);
         expect(options).toEqual([
             'General',
@@ -1521,9 +1610,9 @@ describe('TaskSettings Component', () => {
             'Windsurf'
         ]);
     });
-    it('4.24b Local Codex shows config.toml MCP output without token headers', async () => {
+    it('4.24c Local Codex shows config.toml MCP output without token headers', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: false,
@@ -1534,7 +1623,7 @@ describe('TaskSettings Component', () => {
                 mcpScriptPath: '/usr/local/bin/taskforce'
             }
         });
-        await user.click(screen.getByRole('button', { name: /MCP/i }));
+        await openMcpSettings(user);
         await user.selectOptions(screen.getByRole('combobox', { name: 'MCP client' }), 'codex');
         expect(await screen.findByText(/Codex Configuration Instructions/i)).toBeInTheDocument();
         expect((await screen.findAllByText(/\.codex\/config\.toml/i)).length).toBeGreaterThan(0);
@@ -1542,9 +1631,9 @@ describe('TaskSettings Component', () => {
         expect(screen.getByText(/command = "taskforce"/i)).toBeInTheDocument();
         expect(screen.queryByText(/Authorization/i)).not.toBeInTheDocument();
     });
-    it('4.24c Local VS Code uses stdio config under servers', async () => {
+    it('4.24d Local VS Code uses stdio config under servers', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: false,
@@ -1555,16 +1644,16 @@ describe('TaskSettings Component', () => {
                 mcpScriptPath: '/usr/local/bin/taskforce'
             }
         });
-        await user.click(screen.getByRole('button', { name: /MCP/i }));
+        await openMcpSettings(user);
         await user.selectOptions(screen.getByRole('combobox', { name: 'MCP client' }), 'vscode');
         expect(await screen.findByText(/VS Code Configuration Instructions/i)).toBeInTheDocument();
         expect(screen.getByText(/"servers"/)).toBeInTheDocument();
         expect(screen.getByText(/"type"/)).toBeInTheDocument();
         expect(screen.getByText(/"stdio"/)).toBeInTheDocument();
     });
-    it('4.24d Local Claude Code shows a stdio add command', async () => {
+    it('4.24e Local Claude Code shows a stdio add command', async () => {
         const user = userEvent.setup();
-        renderComponent({
+        renderMcpComponent({
             settingsModel: {
                 runtimeMode: 'local',
                 isAuthenticated: false,
@@ -1575,7 +1664,7 @@ describe('TaskSettings Component', () => {
                 mcpScriptPath: '/usr/local/bin/taskforce'
             }
         });
-        await user.click(screen.getByRole('button', { name: /MCP/i }));
+        await openMcpSettings(user);
         await user.selectOptions(screen.getByRole('combobox', { name: 'MCP client' }), 'claude-code');
         expect(await screen.findByText(/Claude Code Configuration Instructions/i)).toBeInTheDocument();
         expect(screen.getByText(/claude mcp add taskforce-/i)).toBeInTheDocument();

package/dist/components/features/documentWorkspaceModel.d.ts

@@ -15,7 +15,7 @@ export interface DocMeta {
     updatedAt: string | null;
     editable?: boolean;
     attachmentCount?: number;
-    attachmentState?: 'attached' | 'unattached' | 'standalone';
+    attachmentState?: 'attached' | 'unattached';
 }
 export declare function normalizeDocPath(value: string): string;
 export declare function docPathMatches(candidate: string, requested: string): boolean;

package/dist/components/task/TaskActionHeader.js

@@ -1,11 +1,12 @@
 import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-runtime";
 import React from 'react';
-import { Check, Loader2, RotateCcw, Plus, Trash2, X } from 'lucide-react';
+import { Check, Link2, Loader2, Plus, RotateCcw, Trash2, Unlink, X } from 'lucide-react';
 import styles from '../../Taskforce.module.css';
 import { t } from '../../localization';
 import { TaskStatusActions } from './TaskStatusActions';
 import { getTaskReferenceDisplayState } from '../../utils/taskReferences';
 import { TaskReferenceBadge } from './TaskReferenceBadge';
+import { HierarchyActionBadge } from '../ui/HierarchyActionBadge';
 import { formatInitiativeReference, getWorkstreamReferenceLabel } from '../../utils/planningReferences';
 export function TaskActionHeader({ editingTaskId, loading, autoSaveState = 'idle', title, handleSubmit, resetForm, handleCopyId, copiedId, tasks, currentTask: currentTaskProp, currentTaskWorkstream, currentTaskInitiative, workstreamInput = '', onWorkstreamInputChange, onSetWorkstreamForCurrentTask, handleToggleInProgress, handleToggleReview, handleToggleComplete, handleToggleCancel, handleSetStatus, handleArchiveTask, handleUnarchiveTask, handleRestoreDeletedTask, handlePermanentlyDeleteDeletedTask }) {
     const [showWorkstreamEditor, setShowWorkstreamEditor] = React.useState(false);
@@ -32,10 +33,15 @@ export function TaskActionHeader({ editingTaskId, loading, autoSaveState = 'idle
             return;
         onSetWorkstreamForCurrentTask?.();
     }, [onSetWorkstreamForCurrentTask, workstreamInput]);
-    return (_jsx("div", { className: styles.stickyActionHeader, children: editingTaskId ? (_jsxs(_Fragment, { children: [_jsxs("div", { className: styles.taskHierarchyHeader, children: [_jsxs("div", { className: styles.taskHierarchyBadgeRow, children: [currentTaskInitiative ? (_jsxs(_Fragment, { children: [_jsx(TaskReferenceBadge, { copied: copiedId === currentInitiativeReferenceLabel, onClick: (e) => handleCopyId(e, currentInitiativeReferenceLabel), disabled: loading, title: "Copy initiative reference", label: currentInitiativeReferenceLabel }), _jsx("span", { className: styles.taskHierarchyDivider, children: "/" })] })) : null, currentTaskWorkstream ? (_jsxs(_Fragment, { children: [_jsx(TaskReferenceBadge, { copied: copiedId === currentWorkstreamReferenceLabel, onClick: (e) => handleCopyId(e, currentWorkstreamReferenceLabel), disabled: loading, title: "Copy workstream reference", label: currentWorkstreamReferenceLabel }), _jsx("span", { className: styles.taskHierarchyDivider, children: "/" })] })) : (_jsxs(_Fragment, { children: [_jsx("button", { type: "button", className: `${styles.taskIdBadge} ${styles.taskHierarchyAddBtn}`, onClick: () => {
-                                                onWorkstreamInputChange?.('');
-                                                setShowWorkstreamEditor((value) => !value);
-                                            }, disabled: loading || isArchivedReadOnly || isDeletedReadOnly, title: "Attach to workstream", "aria-label": "Attach to workstream", children: _jsx(Plus, { size: 13 }) }), _jsx("span", { className: styles.taskHierarchyDivider, children: "/" })] })), currentTaskReferenceLabel ? (_jsx(TaskReferenceBadge, { copied: canCopyCurrentTaskReference && copiedId === currentTaskReference.label, onClick: (e) => handleCopyId(e, currentTaskReference.label), disabled: loading || !canCopyCurrentTaskReference, title: canCopyCurrentTaskReference ? t('actionHeader.copyTaskIdTitle') : 'Task reference pending sync', label: currentTaskReferenceLabel })) : null, currentTaskReferenceIsProvisional ? (_jsx("span", { className: styles.taskHierarchyMeta, title: "Temporary local reference until cloud sync assigns the final task number.", children: "Pending sync" })) : null] }), showWorkstreamEditor && !currentTaskWorkstream ? (_jsxs("div", { className: styles.taskHierarchyEditor, children: [_jsx("input", { type: "text", className: `${styles.input} ${styles.taskHierarchyInput}`, value: workstreamInput, onChange: (event) => onWorkstreamInputChange?.(event.target.value), placeholder: "WS-123", "aria-label": "Attach workstream reference", onKeyDown: (event) => {
+    const handleWorkstreamRemove = React.useCallback(() => {
+        setShowWorkstreamEditor(false);
+        onWorkstreamInputChange?.('');
+        onSetWorkstreamForCurrentTask?.(null);
+    }, [onSetWorkstreamForCurrentTask, onWorkstreamInputChange]);
+    return (_jsx("div", { className: styles.stickyActionHeader, children: editingTaskId ? (_jsxs(_Fragment, { children: [_jsxs("div", { className: styles.taskHierarchyHeader, children: [_jsxs("div", { className: styles.taskHierarchyBadgeRow, children: [currentTaskWorkstream ? (_jsx(_Fragment, { children: _jsx(HierarchyActionBadge, { onClick: handleWorkstreamRemove, disabled: loading || isArchivedReadOnly || isDeletedReadOnly, title: "Remove from workstream", ariaLabel: "Remove from workstream", children: _jsx(Unlink, { size: 13 }) }) })) : null, currentTaskInitiative ? (_jsxs("span", { className: styles.taskHierarchySegment, children: [_jsx("span", { className: styles.taskHierarchyDivider, children: "/" }), _jsx(TaskReferenceBadge, { copied: copiedId === currentInitiativeReferenceLabel, onClick: (e) => handleCopyId(e, currentInitiativeReferenceLabel), disabled: loading, title: "Copy initiative reference", label: currentInitiativeReferenceLabel })] })) : null, currentTaskWorkstream ? (_jsxs("span", { className: styles.taskHierarchySegment, children: [_jsx("span", { className: styles.taskHierarchyDivider, children: "/" }), _jsx(TaskReferenceBadge, { copied: copiedId === currentWorkstreamReferenceLabel, onClick: (e) => handleCopyId(e, currentWorkstreamReferenceLabel), disabled: loading, title: "Copy workstream reference", label: currentWorkstreamReferenceLabel })] })) : (_jsx(_Fragment, { children: _jsx(HierarchyActionBadge, { onClick: () => {
+                                            onWorkstreamInputChange?.('');
+                                            setShowWorkstreamEditor((value) => !value);
+                                        }, disabled: loading || isArchivedReadOnly || isDeletedReadOnly, title: "Attach to workstream", ariaLabel: "Attach to workstream", children: _jsx(Link2, { size: 13 }) }) })), currentTaskReferenceLabel ? (_jsxs("span", { className: styles.taskHierarchySegment, children: [_jsx("span", { className: styles.taskHierarchyDivider, children: "/" }), _jsx(TaskReferenceBadge, { copied: canCopyCurrentTaskReference && copiedId === currentTaskReference.label, onClick: (e) => handleCopyId(e, currentTaskReference.label), disabled: loading || !canCopyCurrentTaskReference, title: canCopyCurrentTaskReference ? t('actionHeader.copyTaskIdTitle') : 'Task reference pending sync', label: currentTaskReferenceLabel })] })) : null, currentTaskReferenceIsProvisional ? (_jsx("span", { className: styles.taskHierarchyMeta, title: "Temporary local reference until cloud sync assigns the final task number.", children: "Pending sync" })) : null] }), showWorkstreamEditor && !currentTaskWorkstream ? (_jsxs("div", { className: styles.taskHierarchyEditor, children: [_jsx("input", { type: "text", className: `${styles.input} ${styles.taskHierarchyInput}`, value: workstreamInput, onChange: (event) => onWorkstreamInputChange?.(event.target.value), placeholder: "WS-123", "aria-label": "Attach workstream reference", onKeyDown: (event) => {
                                         if (event.key === 'Enter') {
                                             event.preventDefault();
                                             handleWorkstreamConfirm();

package/dist/components/task/TaskActionHeader.test.js

@@ -60,6 +60,16 @@ describe('TaskActionHeader', () => {
         expect(screen.getByText('IN-5')).toBeInTheDocument();
         expect(screen.getByText('WS-12')).toBeInTheDocument();
         expect(screen.getByText('T-1')).toBeInTheDocument();
+        expect(screen.getByRole('button', { name: /remove from workstream/i })).toBeInTheDocument();
+    });
+    it('removes the current workstream from the hierarchy header', async () => {
+        const user = userEvent.setup();
+        const onSetWorkstreamForCurrentTask = vi.fn();
+        const onWorkstreamInputChange = vi.fn();
+        render(_jsx(TaskActionHeader, { ...baseProps, editingTaskId: "task-1", title: "Polish task modal", currentTask: baseTask, currentTaskWorkstream: workstream, onSetWorkstreamForCurrentTask: onSetWorkstreamForCurrentTask, onWorkstreamInputChange: onWorkstreamInputChange }));
+        await user.click(screen.getByRole('button', { name: /remove from workstream/i }));
+        expect(onWorkstreamInputChange).toHaveBeenCalledWith('');
+        expect(onSetWorkstreamForCurrentTask).toHaveBeenCalledWith(null);
     });
     it('opens and cancels the inline workstream attach editor for unlinked tasks', async () => {
         const user = userEvent.setup();