Skip to content

fix: prod deploy#91

Merged
limitofzero merged 2 commits intomainfrom
fix/prod-deploy-build
May 6, 2026
Merged

fix: prod deploy#91
limitofzero merged 2 commits intomainfrom
fix/prod-deploy-build

Conversation

@limitofzero
Copy link
Copy Markdown
Contributor

@limitofzero limitofzero commented May 5, 2026

Problem
The prebuild:strapi script ran strapi ts:generate-types on every build. That command boots a full Strapi instance (Strapi(...).register()) to walk all content types, which triggers every plugin's register() lifecycle — including @strapi/plugin-upload's provider initialization.

So the previous flow only worked when the cloud provider wasn't injected AND the build had a chance to regenerate types — neither pipeline satisfied both.

I've just run yarn gen:strapi-types and commit the result whenever and commit the result, it works fine for current image build but to test it on strapi cloud - need to merge and deploy it

@socket-security
Copy link
Copy Markdown

socket-security Bot commented May 5, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedpostinstall-postinstall@​2.1.0881006075100
Addedstrapi-plugin-import-export-entries@​1.23.18510010077100
Addedpatch-package@​8.0.09710010081100
Addedrimraf@​5.0.510010010083100
Addedreact@​18.3.11001008497100
Addedopenapi-typescript@​6.7.5991009786100
Addedpg@​8.11.3981009887100
Addedopenapi-fetch@​0.9.39910010088100
Addedreact-dom@​18.3.1921009298100

View full report

@limitofzero limitofzero requested a review from a team May 5, 2026 18:48
@limitofzero limitofzero self-assigned this May 5, 2026
@limitofzero limitofzero merged commit 9f4dd5b into main May 6, 2026
3 checks passed
@limitofzero limitofzero deleted the fix/prod-deploy-build branch May 6, 2026 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants