lisa/fix-screenshots-only-as-image-types

[gusfcarvalho]

automated implementation by lisa.

[Copilot]

Pull request overview

This PR updates the workflow evidence submission UI to restrict “screenshot” evidence uploads to image-only file extensions (instead of allowing documents like PDFs), and adds unit tests to verify the new behavior.

Changes:

• Make the <input type="file"> accept attribute dynamic based on selected evidence type (document vs screenshot).
• Add client-side validation to reject files whose extensions aren’t allowed for the selected evidence type (both on file selection and on submit).
• Add Vitest coverage for screenshot-vs-document accepted extensions and validation messaging.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/views/workflow-executions/partials/EvidenceSubmissionForm.vue	Adds per-evidence-type file extension allowlists, dynamic accept, and validation error handling.
src/views/workflow-executions/partials/tests/EvidenceSubmissionForm.spec.ts	Introduces tests validating screenshot-only image extensions and preserved document support.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

src/composables/workflows/useStepExecutions.ts:165

• The extension-based MIME type fallback map is missing entries for .doc/.docx, even though the UI allows those file types for document evidence. If a browser provides an empty/unknown File.type, document uploads will be sent as application/octet-stream instead of the expected Word MIME types. Add doc and docx mappings here (and any other supported extensions) to keep the inferred mediaType consistent with the upload constraints.

        } else if (ev.fileName) {
          // Guess from extension
          const ext = ev.fileName.split('.').pop()?.toLowerCase();
          const mimeTypes: Record<string, string> = {
            pdf: 'application/pdf',
            png: 'image/png',
            jpg: 'image/jpeg',
            jpeg: 'image/jpeg',
            gif: 'image/gif',
            webp: 'image/webp',
            txt: 'text/plain',
            json: 'application/json',
            xml: 'application/xml',
          };

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

[gusfcarvalho]

lisa: implementation done. no actionable copilot review threads found.