@taskforcehq/taskforce 0.3.311 → 0.3.313

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

@@ -1,506 +0,0 @@
-import { jsx as _jsx } from "react/jsx-runtime";
-import { render, screen, fireEvent, waitFor } from '@testing-library/react';
-import userEvent from '@testing-library/user-event';
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { TaskContextUpload } from './TaskContextUpload';
-class MockFileReader {
-    result = null;
-    error = null;
-    onload = null;
-    onerror = null;
-    readAsDataURL(_file) {
-        this.result = 'data:application/octet-stream;base64,ZmFrZQ==';
-        this.onload?.call(this, new ProgressEvent('load'));
-    }
-}
-const createFileList = (files) => {
-    const list = {
-        length: files.length,
-        item: (index) => files[index] || null
-    };
-    files.forEach((file, index) => {
-        list[index] = file;
-    });
-    list[Symbol.iterator] = () => files.values();
-    return list;
-};
-describe('TaskContextUpload drag and drop', () => {
-    const CONTEXT_UPLOAD_OPEN_STORAGE_KEY = 'taskforce-show-context-upload';
-    const onAddContextFile = vi.fn();
-    const onRemoveContextFile = vi.fn();
-    const onUpdateContextCaption = vi.fn();
-    const renderUpload = (contextFiles = [], taskId, extraProps) => render(_jsx(TaskContextUpload, { taskId: taskId, contextFiles: contextFiles, onAddContextFile: onAddContextFile, onRemoveContextFile: onRemoveContextFile, onUpdateContextCaption: onUpdateContextCaption, ...extraProps }));
-    const openSection = async (user) => {
-        const localUser = user ?? userEvent.setup();
-        if (!screen.queryByText('Drop files or URLs here')) {
-            const showToggle = await screen.findByTitle('Show context documents');
-            await localUser.click(showToggle);
-        }
-        return screen.findByText('Drop files or URLs here');
-    };
-    beforeEach(() => {
-        vi.clearAllMocks();
-        vi.stubGlobal('FileReader', MockFileReader);
-        vi.spyOn(window, 'open').mockImplementation(() => null);
-        window.localStorage.removeItem(CONTEXT_UPLOAD_OPEN_STORAGE_KEY);
-    });
-    afterEach(() => {
-        vi.unstubAllGlobals();
-    });
-    it('adds dropped URL links', async () => {
-        renderUpload();
-        const dropzone = await openSection();
-        fireEvent.drop(dropzone, {
-            dataTransfer: {
-                files: createFileList([]),
-                getData: (type) => (type === 'text/plain' ? 'https://example.com/spec' : '')
-            }
-        });
-        expect(onAddContextFile).toHaveBeenCalledTimes(1);
-        expect(onAddContextFile).toHaveBeenCalledWith(expect.objectContaining({
-            path: 'https://example.com/spec',
-            caption: 'spec'
-        }));
-    });
-    it('rejects dropped file:// URLs and instructs the user to upload files instead', async () => {
-        renderUpload();
-        const dropzone = await openSection();
-        fireEvent.drop(dropzone, {
-            dataTransfer: {
-                files: createFileList([]),
-                getData: (type) => type === 'text/uri-list' ? 'file:///Users/rossburke/Documents/spec.pdf' : ''
-            }
-        });
-        expect(onAddContextFile).not.toHaveBeenCalled();
-        expect(screen.getByText(/local path/i)).toBeInTheDocument();
-        expect(screen.getByText(/upload files to attach them/i)).toBeInTheDocument();
-    });
-    it('shows an error for empty drops', async () => {
-        renderUpload();
-        const dropzone = await openSection();
-        fireEvent.drop(dropzone, {
-            dataTransfer: {
-                files: createFileList([]),
-                getData: () => ''
-            }
-        });
-        expect(screen.getByText('Drop a file or URL.')).toBeInTheDocument();
-    });
-    it('skips duplicate dropped links and shows notice', async () => {
-        renderUpload();
-        const dropzone = await openSection();
-        const dataTransfer = {
-            files: createFileList([]),
-            getData: (type) => (type === 'text/plain' ? 'https://example.com/spec' : '')
-        };
-        fireEvent.drop(dropzone, { dataTransfer });
-        fireEvent.drop(dropzone, { dataTransfer });
-        expect(onAddContextFile).toHaveBeenCalledTimes(1);
-        expect(screen.getByText('1 duplicate link skipped.')).toBeInTheDocument();
-    });
-    it('skips duplicate uploaded files and shows notice', async () => {
-        vi.stubGlobal('fetch', vi.fn().mockResolvedValue({
-            ok: true,
-            json: async () => ({
-                success: true,
-                path: '/api/taskforce/context/shared.pdf',
-                fsPath: 'context-files/shared.pdf'
-            })
-        }));
-        renderUpload();
-        const dropzone = await openSection();
-        const fileA = new File(['a'], 'one.pdf', { type: 'application/pdf' });
-        const fileB = new File(['b'], 'two.pdf', { type: 'application/pdf' });
-        fireEvent.drop(dropzone, {
-            dataTransfer: {
-                files: createFileList([fileA, fileB]),
-                getData: () => ''
-            }
-        });
-        await waitFor(() => {
-            expect(onAddContextFile).toHaveBeenCalledTimes(1);
-        });
-        expect(screen.getByText('1 duplicate file skipped.')).toBeInTheDocument();
-    });
-    it('uploads task attachments same-origin even when a cloud api base url is configured', async () => {
-        const dispatchSpy = vi.spyOn(window, 'dispatchEvent');
-        const fetchMock = vi.fn()
-            .mockResolvedValueOnce({
-            ok: true,
-            json: async () => ({
-                success: true,
-                uploadUrl: 'https://uploads.example.com/put',
-                method: 'PUT',
-                headers: { 'Content-Type': 'text/markdown' },
-                path: '/api/taskforce/documents/asset-doc-1/content',
-                relativePath: 'workspaces/workspace-230/assets/documents/asset-doc-1-notes.md'
-            })
-        })
-            .mockResolvedValueOnce({
-            ok: true
-        })
-            .mockResolvedValueOnce({
-            ok: true,
-            json: async () => ({
-                success: true,
-                path: '/api/taskforce/documents/asset-doc-1/content',
-                fsPath: 'workspaces/workspace-230/assets/documents/asset-doc-1-notes.md',
-                assetId: 'asset-doc-1',
-                taskId: 'task-230',
-                caption: 'notes.md'
-            })
-        });
-        vi.stubGlobal('fetch', fetchMock);
-        renderUpload([], 'task-230', {
-            workspaceId: 'workspace-230',
-            apiBaseUrl: 'https://api.taskforcehq.test'
-        });
-        const dropzone = await openSection();
-        const file = new File(['# Notes'], 'notes.md', { type: 'text/markdown' });
-        fireEvent.drop(dropzone, {
-            dataTransfer: {
-                files: createFileList([file]),
-                getData: () => ''
-            }
-        });
-        await waitFor(() => {
-            expect(onAddContextFile).toHaveBeenCalledTimes(1);
-        });
-        expect(fetchMock).toHaveBeenNthCalledWith(1, '/api/taskforce/context-upload/init', expect.objectContaining({
-            method: 'POST',
-            headers: expect.objectContaining({
-                'Content-Type': 'application/json',
-                'x-taskforce-workspace-id': 'workspace-230'
-            })
-        }));
-        expect(JSON.parse(String(fetchMock.mock.calls[0]?.[1]?.body || '{}'))).toEqual(expect.objectContaining({
-            workspaceId: 'workspace-230',
-            taskId: 'task-230',
-            originalName: 'notes.md'
-        }));
-        expect(fetchMock).toHaveBeenNthCalledWith(3, '/api/taskforce/context-upload/finalize', expect.objectContaining({
-            method: 'POST',
-            headers: expect.objectContaining({
-                'Content-Type': 'application/json',
-                'x-taskforce-workspace-id': 'workspace-230'
-            })
-        }));
-        expect(JSON.parse(String(fetchMock.mock.calls[2]?.[1]?.body || '{}'))).toEqual(expect.objectContaining({
-            workspaceId: 'workspace-230',
-            taskId: 'task-230',
-            relativePath: 'workspaces/workspace-230/assets/documents/asset-doc-1-notes.md'
-        }));
-        expect(dispatchSpy).toHaveBeenCalledWith(expect.objectContaining({
-            type: 'taskforce:context-assets-mutated'
-        }));
-    });
-    it('validates unsupported file types before upload', async () => {
-        const fetchMock = vi.fn();
-        vi.stubGlobal('fetch', fetchMock);
-        renderUpload();
-        const dropzone = await openSection();
-        const file = new File(['bad'], 'script.exe', { type: 'application/octet-stream' });
-        fireEvent.drop(dropzone, {
-            dataTransfer: {
-                files: createFileList([file]),
-                getData: () => ''
-            }
-        });
-        expect(onAddContextFile).not.toHaveBeenCalled();
-        expect(fetchMock).not.toHaveBeenCalled();
-        expect(screen.getByText(/unsupported file/i)).toBeInTheDocument();
-        expect(screen.getByText(/script\.exe/i)).toBeInTheDocument();
-    });
-    it('validates oversized files before upload', async () => {
-        const fetchMock = vi.fn();
-        vi.stubGlobal('fetch', fetchMock);
-        renderUpload();
-        const dropzone = await openSection();
-        const largePayload = new Uint8Array(10 * 1024 * 1024 + 1);
-        const file = new File([largePayload], 'huge.pdf', { type: 'application/pdf' });
-        fireEvent.drop(dropzone, {
-            dataTransfer: {
-                files: createFileList([file]),
-                getData: () => ''
-            }
-        });
-        expect(onAddContextFile).not.toHaveBeenCalled();
-        expect(fetchMock).not.toHaveBeenCalled();
-        expect(screen.getByText(/over 10 mb/i)).toBeInTheDocument();
-        expect(screen.getByText(/huge\.pdf/i)).toBeInTheDocument();
-    });
-    it('skips local duplicate files before upload network calls', async () => {
-        const fetchMock = vi.fn().mockResolvedValue({
-            ok: true,
-            json: async () => ({
-                success: true,
-                path: '/api/taskforce/context/dup.pdf',
-                fsPath: 'context-files/dup.pdf'
-            })
-        });
-        vi.stubGlobal('fetch', fetchMock);
-        renderUpload();
-        const dropzone = await openSection();
-        const duplicateA = new File(['same-data'], 'dup.pdf', { type: 'application/pdf', lastModified: 1739366400000 });
-        const duplicateB = new File(['same-data'], 'dup.pdf', { type: 'application/pdf', lastModified: 1739366400000 });
-        fireEvent.drop(dropzone, {
-            dataTransfer: {
-                files: createFileList([duplicateA, duplicateB]),
-                getData: () => ''
-            }
-        });
-        await waitFor(() => {
-            expect(onAddContextFile).toHaveBeenCalledTimes(1);
-        });
-        expect(fetchMock).toHaveBeenCalledTimes(1);
-        expect(screen.getByText(/local duplicate file/i)).toBeInTheDocument();
-    });
-    it('sends taskId to finalize and uses canonical finalize payload', async () => {
-        const uploadFetch = vi.fn()
-            .mockResolvedValueOnce({
-            ok: true,
-            json: async () => ({
-                success: true,
-                uploadUrl: 'https://example.r2/upload',
-                method: 'PUT',
-                headers: { 'Content-Type': 'text/markdown' },
-                relativePath: 'workspaces/workspace-1/assets/documents/asset-1-notes.md',
-                path: '/api/taskforce/documents/asset-1/content',
-                fsPath: '.taskforce/workspaces/workspace-1/assets/documents/asset-1-notes.md'
-            })
-        })
-            .mockResolvedValueOnce({
-            ok: true
-        })
-            .mockResolvedValueOnce({
-            ok: true,
-            json: async () => ({
-                success: true,
-                path: '/api/taskforce/documents/asset-1/content',
-                fsPath: 'workspaces/workspace-1/assets/documents/asset-1-notes.md',
-                caption: 'notes.md',
-                assetId: 'asset-1',
-                taskId: 'task-1'
-            })
-        });
-        vi.stubGlobal('fetch', uploadFetch);
-        renderUpload([], 'task-1');
-        const dropzone = await openSection();
-        const file = new File(['# Hello'], 'notes.md', { type: 'text/markdown' });
-        fireEvent.drop(dropzone, {
-            dataTransfer: {
-                files: createFileList([file]),
-                getData: () => ''
-            }
-        });
-        await waitFor(() => {
-            expect(onAddContextFile).toHaveBeenCalledTimes(1);
-        });
-        const finalizeCall = uploadFetch.mock.calls[2];
-        expect(finalizeCall?.[0]).toBe('/api/taskforce/context-upload/finalize');
-        const finalizeBody = JSON.parse(String(finalizeCall?.[1]?.body || '{}'));
-        expect(finalizeBody.taskId).toBe('task-1');
-        expect(onAddContextFile).toHaveBeenCalledWith(expect.objectContaining({
-            path: '/api/taskforce/documents/asset-1/content',
-            fsPath: 'workspaces/workspace-1/assets/documents/asset-1-notes.md'
-        }));
-    });
-    it('renders image attachments as preview tiles and document attachments as rows', async () => {
-        renderUpload([
-            '/uploads/mockup.png',
-            {
-                path: '/api/taskforce/context/spec.pdf',
-                fsPath: 'context/spec.pdf',
-                caption: 'Product spec',
-                originalFilename: 'product-spec.pdf'
-            }
-        ]);
-        await openSection();
-        expect(screen.getByRole('img', { name: /context file 0/i })).toBeInTheDocument();
-        expect(screen.getByRole('button', { name: /product-spec\.pdf pdf/i })).toBeInTheDocument();
-        expect(screen.getByDisplayValue('Product spec')).toBeInTheDocument();
-    });
-    it('dispatches annotated attachment open events from image preview tiles with canonical metadata', async () => {
-        const dispatchSpy = vi.spyOn(window, 'dispatchEvent').mockReturnValue(false);
-        const user = userEvent.setup();
-        renderUpload([
-            {
-                path: '/api/taskforce/context/workspaces%2Fworkspace-1%2Fassets%2Fimages%2Fasset-image-1-ui.png',
-                fsPath: 'workspaces/workspace-1/assets/images/asset-image-1-ui.png',
-                assetId: 'asset-image-1',
-                taskId: 'task-1',
-                displayName: 'UI Screenshot',
-                originalFilename: 'ui.png',
-                caption: 'UI Screenshot',
-            }
-        ], 'task-1', { taskReferenceLabel: 'T-230' });
-        await openSection(user);
-        await user.click(screen.getByRole('img', { name: /ui screenshot/i }));
-        expect(dispatchSpy).toHaveBeenCalledWith(expect.objectContaining({
-            type: 'taskforce:open-annotated-attachment',
-            detail: expect.objectContaining({
-                taskId: 'task-1',
-                taskReferenceLabel: 'T-230',
-                assetId: 'asset-image-1',
-                path: '/api/taskforce/context/workspaces%2Fworkspace-1%2Fassets%2Fimages%2Fasset-image-1-ui.png',
-                displayName: 'UI Screenshot'
-            })
-        }));
-        dispatchSpy.mockRestore();
-    });
-    it('shows an annotate action for canonical image attachments and dispatches the open event', async () => {
-        const dispatchSpy = vi.spyOn(window, 'dispatchEvent').mockReturnValue(false);
-        const user = userEvent.setup();
-        renderUpload([
-            {
-                path: '/api/taskforce/context/workspaces%2Fworkspace-1%2Fassets%2Fimages%2Fasset-image-2-ui.png',
-                fsPath: 'workspaces/workspace-1/assets/images/asset-image-2-ui.png',
-                assetId: 'asset-image-2',
-                taskId: 'task-1',
-                displayName: 'Settings screen',
-                originalFilename: 'settings.png',
-                caption: 'Settings screen',
-            }
-        ], 'task-1', { taskReferenceLabel: 'T-230' });
-        await openSection(user);
-        await user.click(screen.getByTitle('Annotate'));
-        expect(dispatchSpy).toHaveBeenCalledWith(expect.objectContaining({
-            type: 'taskforce:open-annotated-attachment',
-            detail: expect.objectContaining({
-                taskId: 'task-1',
-                taskReferenceLabel: 'T-230',
-                assetId: 'asset-image-2',
-                displayName: 'Settings screen'
-            })
-        }));
-        dispatchSpy.mockRestore();
-    });
-    it('prefers the attachment caption over the raw image filename for image notes launches', async () => {
-        const dispatchSpy = vi.spyOn(window, 'dispatchEvent').mockReturnValue(false);
-        const user = userEvent.setup();
-        renderUpload([
-            {
-                path: '/api/taskforce/context/workspaces%2Fworkspace-1%2Fassets%2Fimages%2Fasset-image-3-ui.png',
-                fsPath: 'workspaces/workspace-1/assets/images/asset-image-3-ui.png',
-                assetId: 'asset-image-3',
-                taskId: 'task-1',
-                displayName: 'pasted-image-1774497398365.png',
-                originalFilename: 'pasted-image-1774497398365.png',
-                caption: 'Image Notes',
-            }
-        ], 'task-1', { taskReferenceLabel: 'T-230' });
-        await openSection(user);
-        await user.click(screen.getByTitle('Annotate'));
-        expect(dispatchSpy).toHaveBeenCalledWith(expect.objectContaining({
-            type: 'taskforce:open-annotated-attachment',
-            detail: expect.objectContaining({
-                taskId: 'task-1',
-                taskReferenceLabel: 'T-230',
-                assetId: 'asset-image-3',
-                displayName: 'Image Notes'
-            })
-        }));
-        dispatchSpy.mockRestore();
-    });
-    it('does not show annotate for non-editable gif attachments', async () => {
-        const user = userEvent.setup();
-        renderUpload([
-            {
-                path: '/api/taskforce/context/workspaces%2Fworkspace-1%2Fassets%2Fimages%2Fasset-image-anim.gif',
-                fsPath: 'workspaces/workspace-1/assets/images/asset-image-anim.gif',
-                assetId: 'asset-image-anim',
-                taskId: 'task-1',
-                displayName: 'Animated preview',
-                originalFilename: 'animated.gif',
-                caption: 'Animated preview',
-            }
-        ], 'task-1');
-        await openSection(user);
-        expect(screen.queryByTitle('Annotate')).not.toBeInTheDocument();
-    });
-    it('shows inline document actions and keeps caption editing working', async () => {
-        const user = userEvent.setup();
-        renderUpload([
-            {
-                path: 'https://example.com/files/briefing.md',
-                caption: 'Briefing',
-                originalFilename: 'briefing.md'
-            }
-        ]);
-        await openSection();
-        await user.click(screen.getByTitle('briefing.md'));
-        expect(window.open).toHaveBeenCalledWith('https://example.com/files/briefing.md', '_blank');
-        const captionInput = screen.getByDisplayValue('Briefing');
-        fireEvent.change(captionInput, { target: { value: 'Briefing v2' } });
-        expect(onUpdateContextCaption).toHaveBeenLastCalledWith(0, 'Briefing v2');
-        await user.click(screen.getByTitle('Delete'));
-        expect(onRemoveContextFile).toHaveBeenCalledWith(0);
-    });
-    it('renders D- references for canonical document attachments in the task form surface', async () => {
-        renderUpload([
-            {
-                path: '/api/taskforce/documents/asset-doc-1/content',
-                fsPath: 'workspaces/workspace-1/assets/documents/asset-doc-1-briefing.md',
-                caption: 'Briefing',
-                displayName: 'Briefing',
-                originalFilename: 'briefing.md',
-                assetId: 'asset-doc-1',
-                referenceNumber: 7,
-                referenceLabel: 'D-7'
-            }
-        ]);
-        await openSection();
-        expect(screen.getByText('D-7')).toBeInTheDocument();
-        expect(screen.getByText('Briefing')).toBeInTheDocument();
-    });
-    it('renders local document actions without hover-only dependence', async () => {
-        const user = userEvent.setup();
-        renderUpload([
-            {
-                path: '/api/taskforce/context/notes.json',
-                fsPath: 'context/notes.json',
-                caption: '',
-                originalFilename: 'notes.json'
-            }
-        ]);
-        await openSection();
-        expect(screen.getByTitle('Copy path: context/notes.json')).toBeInTheDocument();
-        await user.click(screen.getByTitle('Download'));
-        expect(window.open).toHaveBeenCalledWith(expect.stringContaining('/api/taskforce/context/notes.json?download=1&filename=notes.json'), '_blank');
-    });
-    it('pastes an image from the clipboard into task context', async () => {
-        const user = userEvent.setup();
-        const fetchMock = vi.fn().mockResolvedValue({
-            ok: true,
-            json: async () => ({
-                success: true,
-                path: '/api/taskforce/context/workspaces%2Fworkspace-230%2Fassets%2Fimages%2Fasset-image-1-ui.png',
-                fsPath: 'workspaces/workspace-230/assets/images/asset-image-1-ui.png',
-                assetId: 'asset-image-1',
-                taskId: 'task-230',
-                caption: 'pasted-image.png'
-            })
-        });
-        vi.stubGlobal('fetch', fetchMock);
-        Object.defineProperty(navigator, 'clipboard', {
-            configurable: true,
-            value: {
-                read: vi.fn().mockResolvedValue([
-                    {
-                        types: ['image/png'],
-                        getType: vi.fn().mockResolvedValue(new Blob(['fake-image'], { type: 'image/png' }))
-                    }
-                ])
-            }
-        });
-        renderUpload([], 'task-230', { workspaceId: 'workspace-230' });
-        await openSection(user);
-        await user.click(screen.getByRole('button', { name: /paste image from clipboard/i }));
-        await waitFor(() => {
-            expect(onAddContextFile).toHaveBeenCalledWith(expect.objectContaining({
-                assetId: 'asset-image-1',
-                taskId: 'task-230'
-            }));
-        });
-    });
-});

package/dist/components/task/TaskForm.test.d.ts

@@ -1 +0,0 @@
-export {};