Skip to content

Commit 6a8d39c

Browse files
authored
fix(appstore): verify pdf worker in js/ instead of dist/ (#7426) (#7427)
After updating @libresign/pdf-elements, the build no longer produces dist/pdf.worker.min-*.mjs. The pdf worker is now bundled into js/ by Vite (via the dynamic import in pdf-elements' ensureWorkerReady()). Update verify-appstore-package to: - Assert js/ exists (unconditional, needed for frontend assets) - Find pdf.worker.min-*.mjs in js/ (where it now lives) - Keep the dist/ conditional only to check the directory was copied when it was produced (e.g. local env with an old build artifact) [skip ci] (cherry picked from commit b45ece9) Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
1 parent 233953d commit 6a8d39c

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

Makefile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,8 @@ appstore:
177177
.PHONY: verify-appstore-package
178178
verify-appstore-package:
179179
test -d $(appstore_sign_dir)/$(app_name)/css
180+
test -d $(appstore_sign_dir)/$(app_name)/js
181+
find $(appstore_sign_dir)/$(app_name)/js -maxdepth 1 -name 'pdf.worker.min-*.mjs' | grep -q .
180182
if [ -d dist ]; then \
181183
test -d $(appstore_sign_dir)/$(app_name)/dist; \
182-
find $(appstore_sign_dir)/$(app_name)/dist -maxdepth 1 -name 'pdf.worker.min-*.mjs' | grep -q .; \
183184
fi

0 commit comments

Comments
 (0)