Platforms Impacted

• all

Description of changes

The repo had a large number of missing or incorrect dependencies, some of which were being hidden by the ignore list in depcheck, others being hidden by the use of the old-style node_modules layout in yarn. Because of this we often have to inject dependencies in OMR to make things work correctly.

This change includes a few key pieces:

More comprehensive depcheck command

The previous use of depcheck had a large list of ignored packages that were hiding a number of issues. For various issues I had run into the issue of needing to manually make changes to large number of packages to fix things, to address this I got rid of the special casing in depcheck and adding fix capabilities.

• centralized the tool versions into a new package in scripts/src/preinstall this is where various tools pull common versions.
• depcheck and the dynamic dependencies now rely on this.
• depcheck now takes the dependencies injected via dynamic dependencies into account and ignores them.
• depcheck does the ignore handling as post-processing rather than built-into the tool to not hide them
• --verbose option that will show ignored errors
• --fix-errors option that will automatically fix errors if it can. For packages that it can't determine the version it will fail.
• --fix-warnings option that will fix warnings. Use this with caution as it will flag some devDependencies that are actually needed.
• --dry-run option that will show what the fix mechanisms will do without making changes.
• added better coloring so you can clearly see what is going on.

The fix can be run from root by executing yarn workspaces foreach --all --no-private run depcheck --fix-errors. The no-private option is needed for now as it will fail on the root.

Dedicated packages for jest and babel configurations

This helps isolate the dependencies required for those packages so they aren't all jammed together in scripts. These were created under packages/configs and I will move the eslint config package there as a separate change and likely eventually create a separate tsconfig package as well. These are all private packages just used within the repo.

• Note that the checkbox component hasn't been running tests due to a missing jest configuration, I restored this and updated the snapshots to be current now.

Got rid of codegen as part of the build-step

This was unnecessary as it will be done either way as part of metro bundling as it is handled by the metro babel transforms.

Switched to pnpm mode

Once fixing depcheck and doing the other work this was mainly tweaking dependency declarations to be correct.

Verification

• Automated tests.