Skip to content

Commit ed7b4d8

Browse files
kirill3333cipolleschi
authored andcommitted
Allow the .pnpm folder to be discovered during code generation (#49983)
Summary: This PR (#48182) introduced skipping hidden folders during Codegen generation. However, when using pnpm, all files are stored in the `.pnpm` folder (see explanation here: https://pnpm.io/symlinked-node-modules-structure). As a result, some libraries that support the new architecture but lack the `ios.codegenConfig.componentProvider` field - like [FlashList](https://github.com/Shopify/flash-list/blob/main/package.json) - will be skipped during Codegen generation. This PR explicitly includes `.pnpm` to prevent this issue. ## Changelog: [iOS][Fixed] - Check .pnpm folder when looking for third-party components. Pull Request resolved: #49983 Test Plan: Tested on: RN 0.78.0 PNPM: 10 Flashlist: 1.7.3 Reviewed By: cipolleschi Differential Revision: D71047936 Pulled By: cortinico fbshipit-source-id: fa9caab23dea8c92ef5f23c997812d348eb19e08
1 parent a47b9aa commit ed7b4d8

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

packages/react-native/scripts/codegen/generate-artifacts-executor.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -847,8 +847,11 @@ function findFilesWithExtension(filePath, extension) {
847847
return null;
848848
}
849849

850-
// Skip hidden folders, that starts with `.`
851-
if (absolutePath.includes(`${path.sep}.`)) {
850+
// Skip hidden folders, that starts with `.` but allow `.pnpm`
851+
if (
852+
absolutePath.includes(`${path.sep}.`) &&
853+
!absolutePath.includes(`${path.sep}.pnpm`)
854+
) {
852855
return null;
853856
}
854857

0 commit comments

Comments
 (0)