feat(create-astro): add astro checks to build script when user choose TypeScript

[rayriffy]

Changes

• If user choose TypeScript strictness to strictest, the create-astro command will also add astro checks to build script
• Additionally, @astrojs/check and typescript will be installed if user previously opt-in to install dependencies
• Internally, some parts of typescript.ts uses both async/sync of node:fs. I also did some refactoring to be all async

fixes #7031

Testing

• Writing test to make sure that build script package.json is actually changed

Docs

Maybe need to add extra docs to TypeScript to explain strictest behavior when use pnpm create astro

[changeset-bot]

🦋 Changeset detected

Latest commit: ff7d695

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[natemoo-re]

This is great, thanks so much for contributing!

Just wondering about always applying this if the user sets up TypeScript. I usually go with strict but I'd still like this to be setup for me.

[natemoo-re]

Looks great! Thanks so much for the PR!

[sarah11918]

Just a quick suggestion from docs re: the wording here! See whether you think the extra context is helpful!

[sarah11918]

Suggested change

	Automatically setup `astro check` command when configuring TypeScript
	Automatically installs the required dependencies to run the `astro check` command when `strict` or `strictest`
	TypeScript mode is selected.

Since running astro check requires both @astrojs/check and typescript, maybe makes sense to say "required dependencies" (plural) here, because these will be added to your project?

[rayriffy]

I do agree. But I have to adjust some wording at the end because astro check will be applied to all TypeScript options (relaxed, strict, strictest).

Let me know if this change looks good to you.

[sarah11918]

I commented above to clarify that this happens on relaxed (which seems now far removed from the PR title itself, and maybe installs a dependency for people who won't really use it?) But, wording should be fine.

[Princesseuh]

Just a comment, but feature looks fine to me. I do feel like we'll get support threads of people complaining that their project doesn't build with various errors, but meh, it's okay, people should and will learn.

[sarah11918]

Left a comment to clarify with an optional suggestion, but also fine as is!