Do we need prettier on @remix-run/dev

[fernandocanizo]

Hi all, I noticed prettier is required for @remix-run/dev, not under devDependencies but as a normal dependency. Possibly there's something I'm missing, but off the top of my head I cannot reason why Prettier would be needed on production for an application that compiles your code and runs it.

So I was thinking it maybe slipped-in somehow and nobody has noticed.

It's not a big deal, but since that way is usable from your project if you are using npm, it could lead to problems for the unaware.

Example:

npx create-remix@latest deleteme-npm
# ...
cd deleteme-npm
npx prettier --version
2.8.8

npm reports prettier version from the package required by@remix-run/dev

pnpx create-remix deleteme-pnpm
cd deleteme-pnpm
pnpx prettier --version
# ...
3.1.1

pnpm reports the (currently) latest prettier version from npm registry

[MichaelDeBoey]

prettier is used in https://github.com/remix-run/remix/blob/main/packages/remix-dev/cli/useJavascript.ts, which is used by the reveal command

remix/packages/remix-dev/cli/useJavascript.ts

Lines 25 to 29 in 2ba86f1

	/**
	* Babel's `compact` and `retainLines` options are both bad at formatting code.
	* Use Prettier for nicer formatting.
	*/
	return prettier.format(mjs.code, { parser: "babel" });

remix/packages/remix-dev/cli/commands.ts

Lines 285 to 291 in 2ba86f1

	if (!useTypeScript) {
	let javascript = convertFileToJS(contents, {
	cwd: config.rootDirectory,
	filename: isServerEntry ? defaultEntryServer : defaultEntryClient,
	});
	await fse.writeFile(outputFile, javascript, "utf-8");
	} else {