Skip to content

Update supported file formats#845

Merged
Ruchika4 merged 3 commits into
stagefrom
file-format
Jul 16, 2026
Merged

Update supported file formats#845
Ruchika4 merged 3 commits into
stagefrom
file-format

Conversation

@Ruchika4

@Ruchika4 Ruchika4 commented Jul 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

Restricts accepted file formats per verb to match the defined support matrix, rather than allowing all file types for most conversion verbs.

Screenshot 2026-07-11 at 8 57 22 PM

Changes

unitylibs/core/workflow/workflow-acrobat/limits.json

  • Added allowed-filetypes-no-adobe-design — all formats from allowed-filetypes-all except PSD, AI, and INDD
  • Added allowed-filetypes-psd-onlyimage/vnd.adobe.photoshop only
  • Added allowed-filetypes-ai-onlyapplication/illustrator only
  • Added allowed-filetypes-indd-onlyapplication/x-indesign only

unitylibs/core/workflow/workflow-acrobat/action-binder.js

  • psd-to-pdf, ai-to-pdf, indd-to-pdf → each restricted to their own format only (previously accepted all file types)
  • jpg-to-pdf, png-to-pdf, heic-to-pdf, image-to-pdf, bmp-to-pdf, gif-to-pdf, tiff-to-pdf, word-to-pdf, excel-to-pdf, ppt-to-pdf → switched to allowed-filetypes-no-adobe-design + allowed-filetypes-heic (removes PSD/AI/INDD acceptance, adds HEIC where missing)
  • createpdf (Convert PDF) — unchanged, continues to accept all formats

test/core/workflow/workflow-acrobat/action-binder.test.js

  • Updated image-to-pdf verbs onboarding test block to match new configuration

Testing

  • Unit tests: npm test — 846 passed, 0 failed
  • Upload a PSD to psd-to-pdf — only PSD should be accepted
  • Upload an AI file to ai-to-pdf — only AI should be accepted
  • Upload an INDD file to indd-to-pdf — only INDD should be accepted
  • Upload a PSD/AI/INDD to any image-to-pdf verb (e.g. jpg-to-pdf) — should be rejected
  • Upload a HEIC file to word-to-pdf, jpg-to-pdf, etc. — should be accepted
  • Upload any file type to createpdf (Convert PDF) — all formats should still be accepted
  • psd-to-pdf, ai-to-pdf, and indd-to-pdf should redirect to the createpdf endpoint instead of jpgtopdf, aligning the redirect destination with the Convert PDF flow.

Resolves: MWPW-200291

Test URLs:

Note: DC PRs for reference adobecom/dc-frictionless#62 and adobecom/da-dc#142

@aem-code-sync

aem-code-sync Bot commented Jul 12, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

@Ruchika4
Ruchika4 marked this pull request as ready for review July 15, 2026 18:38
@Ruchika4
Ruchika4 requested review from arugupta1992, maagrawal16, sanjayms01 and vipu0303 and removed request for maagrawal16 July 15, 2026 18:39
@asonnalagi asonnalagi assigned asonnalagi and unassigned asonnalagi Jul 16, 2026
"application/msexcel",
"application/vnd.openxmlformats-officedocument.spreadsheetml.sheet",
"application/x-tika-msworks-spreadsheet",
"application/vnd.adobe.form.fillsign",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ruchika4
Do we want "application/vnd.adobe.form.fillsign" in no-adobe-design?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @vipu0303 , requirements called out .ai, .indd and .psd adobe files only hence they were only removed for no-adobe-design from already existing list of allowed file types. I will talk to Andy to get confirmation and we can handle that in a separate PR if needed. Please let me know if any concern.

@praghu-9

Copy link
Copy Markdown

Verified in the branch. All looks fine. Test results are in the task
https://jira.corp.adobe.com/browse/MWPW-200945

@Ruchika4
Ruchika4 merged commit 9af87b3 into stage Jul 16, 2026
4 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants