Remove package manager dependency

[takkuumi]

Crux only supports pnpm

In fact, there are many package managers, such as yarn, bun, and npm. It is not reasonable to only support pnpm.

For me, I just use bun because it is so easy to use.

So, could support more package manager?

[charypar]

Yes, we definitely should not require the use of pnpm. This side of Crux - the type generation and "foreign" code packaging - is still under active development and changing a fair bit, so we've taken some shortcuts.

Ideally we would find a way to not depend on a package manager at all, failing that, we should allow it to be configured.

[mhedgpeth]

I took a look at this, and it's not clear to me why tsc --build is needed here at all. I would think the package.json that contains the solution with the type-generated types would do this work for the user.

I propose we remove it, and leave it up to the app creator to manage it. @charypar or @StuartHarris would love your thoughts.

[mhedgpeth]

If we remove the build here, it would be a breaking change, so targeting this for 0.18.0 milestone

[charypar]

I think I agree - the other languages don't run a build step, and the examples have all moved to a just driven task managmeent which manages dependency installation and running the native build, so maybe we can just defer it there.

I'm not entirely sure why the Typescript generation runs the tsc step, rather than just generating the code - probably because just a bare a bundle of TS code is not a valid package usable as a dependency, to match how we generate a swift package for example.

It is a strangre assymetry that in the code generation step for typescript we run an implicit compilation step, unlike the others, but the typescript ecosystem is complicated enough that it may well be required.

If that is a thing, maybe we'll need to make facet_generate smart enough to generate a ready-made npm package with the *.d.ts files, rather than typescript source that needs compiling.

[StuartHarris]

Yep totally agree. We should move that to facet_generate. I'll add it there so we can remove it here.

[mhedgpeth]

I think this is ready to do and I'm happy to tackle it.

@StuartHarris there are two things to do here from my understanding:

1. remove tsc calls
2. remove crux_cli - however the examples still use cli::{BindgenArgsBuilder, bindgen} but nothing uses anything else and I think we have gone away from that. Could I move it to crux_core::generate::{BindgenArgsBuilder, bindgen} as a breaking change for 0.19?