Production Dependency Includes Typescript

[c1moore]

Describe the bug
It seems that the last few versions of @oclif/core has re-introduced TypeScript as a production dependency, significantly increase the bundle size of CLIs using OCLIF. This appears to be due to the tsconfck dependency

└─┬ @oclif/[email protected]
  └─┬ [email protected]
    └── [email protected]

This seems to increase bundle size by about 10MB depending on compression technique and OS target.

To Reproduce
Steps to reproduce the behavior:

1. Build an OCLIF CLI using the latest version
2. Inspect the bundles

Expected behavior
TypeScript is not a production dependency.

Screenshots
N/A

Environment (please complete the following information):

• OS & version: N/A
• Shell/terminal & version: N/A

Additional context
N/A

[git2gus]

This issue has been linked to a new work item: W-14526758

[AllanOricil]

It is weird to see tsconfck depends on ts. If you go to npm, and search for tsconfck you will see it does not have dependencies

https://www.npmjs.com/package/tsconfck

[c1moore]

It's marked as a peer dependency, which, if I'm understanding the docs correctly, means it will be installed with npm v7 if not installed by another dependency.

[AllanOricil]

you are right. It is a peer dependency. Is this tsconfck used in the bundled code? if not I believe it can just be set as a devDependency and the issue will be gone

[c1moore]

It looks like it's used in production code here I'm not sure the value it's adding over just using JSON.parse or using require directly.

[c1moore]

@AllanOricil Any updates on this?

[AllanOricil]

@AllanOricil Any updates on this?

I can take a look at this next weekend if isn't fixed by Salesforce by than