Add bun support.

[acopier]

This PR Adds support for Bun, a blazingly fast JS runtime, package manager and much more.

closes #13

[Dionysusnu]

New iteration of roblox-ts/roblox-ts#2485

[Dionysusnu]

a blazingly fast JS runtime

Note that this won't allow you to use Bun's runtime itself, just the package manager. You'd need to change the compiler itself to support the runtime. We've tried in the past, and it turned out to be quite a lot of work.

[acopier]

@Dionysusnu status?

[acopier]

@Dionysusnu status?

?

[Dionysusnu]

Apologies for missing your first message, but I'm not the maintainer of roblox-ts, just a contributor.

[acopier]

@osyrisrblx do you mind looking at this?

[Dionysusnu]

I've looked into it some more, and I think it's a better option to stick with npm init -y for the init command. As explained in the earlier comment, bun can only work as a package manager for roblox-ts, the runtime isn't compatible. Therefore, I think it's better to init with npm, which just creates the package.json. The things bun init does on top of that are either incorrect for roblox-ts, or already covered by other parts of the init process.

I've left suggestions on how to add this in the code.

[acopier]

I've looked into it some more, and I think it's a better option to stick with npm init -y for the init command. As explained in the earlier comment, bun can only work as a package manager for roblox-ts, the runtime isn't compatible. Therefore, I think it's better to init with npm, which just creates the package.json. The things bun init does on top of that are either incorrect for roblox-ts, or already covered by other parts of the init process.

I've left suggestions on how to add this in the code.

We cannot depend on NPM, some users don't even have NPM installed.

[Dionysusnu]

My previous comment (in the suggestion thread) applies:

If only bun is installed, I don't think we can reasonably provide the bun init option, because we wouldn't get the correct defaults for package.json.

[acopier]

My previous comment (in the suggestion thread) applies:

If only bun is installed, I don't think we can reasonably provide the bun init option, because we wouldn't get the correct defaults for package.json.

I think the condition check is enough.

[Dionysusnu]

What do you mean by condition check?