Skip to content

Conversation

@renan628
Copy link
Member

@renan628 renan628 commented Aug 14, 2025

Summary

This PR implements a workaround to handle a Strapi issue.

Problem

Currently, we internally use the strapi/LaunchPad CLI to provide Strapi integration. When running the command seed, the following error occurs:

src/middlewares/deepPopulate.ts:23:33 - error TS2345: Argument of type src/middlewares/deepPopulate.ts:23:33 - error TS2345: Argument of type
│  'import(".../project/strapi/nodemodules/@strapi/types/dist/uid/index").Schema' is not assignable to parameter of type
│  'import(".../project/strapi/nodemodules/@strapi/core/node_modules/@strapi/types/dist/uid/index").Schema'.

Work around

The error is caused by a mismatch between the @strapi/type package version in the dependencies root and inside the @strapi/core module. To work around that, a step installing the same specific version from @strapi/core on the root was added. An additional care is to set the LaunchPad version specifying the current commit hash, so any update will not break the implementation again.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

@renan628 renan628 self-assigned this Aug 14, 2025
@renan628 renan628 added the bug Something isn't working label Aug 14, 2025
@renan628 renan628 changed the title Add Strapi work around Add Strapi integration work around Aug 14, 2025
@marcospassos marcospassos merged commit b044e81 into master Aug 14, 2025
3 of 4 checks passed
@marcospassos marcospassos deleted the add-strapi-workaround branch August 14, 2025 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants