fix: bind configs to variables before exporting to make sure typeerrors are reported (#1232)

* bind configs

* Create odd-spies-appear.md

* Update .changeset/odd-spies-appear.md

---------

Co-authored-by: Christopher Ehrlich <[email protected]>

Author: juliusmarminge

Diff for: .changeset/odd-spies-appear.md

@@ -0,0 +1,5 @@
+---
+"create-t3-app": patch
+---
+
+fix: bind configs to variables before exporting to make sure typeerrors are reported

Diff for: .eslintrc.cjs

@@ -1,5 +1,5 @@
 /** @type {import("eslint").Linter.Config} */
-module.exports = {
+const config = {
   root: true,
   parser: "@typescript-eslint/parser",
   plugins: ["isaacscript", "import"],
@@ -45,3 +45,5 @@ module.exports = {
     "isaacscript/format-jsdoc-comments": "warn",
   },
 };
+
+module.exports = config;

Diff for: .prettierrc.cjs

@@ -1,5 +1,5 @@
 /** @type {import('prettier').Config} */
-module.exports = {
+const config = {
   arrowParens: "always",
   printWidth: 80,
   singleQuote: false,
@@ -8,3 +8,5 @@ module.exports = {
   trailingComma: "all",
   tabWidth: 2,
 };
+
+module.exports = config;

Diff for: cli/.prettierrc.cjs

@@ -1,6 +1,8 @@
 /** @type {import('prettier').Config} */
-module.exports = {
+const config = {
   ...require("../.prettierrc.cjs"),
   plugins: [require.resolve("prettier-plugin-tailwindcss")],
   tailwindConfig: "./template/extras/config/tailwind.config.cjs",
 };
+
+module.exports = config;

Diff for: cli/template/base/_eslintrc.cjs

@@ -1,5 +1,5 @@
 /** @type {import("eslint").Linter.Config} */
-module.exports = {
+const config = {
   overrides: [
     {
       extends: [
@@ -27,3 +27,5 @@ module.exports = {
     ],
   },
 };
+
+module.exports = config;

Diff for: cli/template/base/src/env.mjs

@@ -1,25 +1,25 @@
 import { z } from "zod";
 
 /**
- * Specify your server-side environment variables schema here.
- * This way you can ensure the app isn't built with invalid env vars.
+ * Specify your server-side environment variables schema here. This way you can ensure the app isn't
+ * built with invalid env vars.
  */
 const server = z.object({
   NODE_ENV: z.enum(["development", "test", "production"]),
 });
 
 /**
- * Specify your client-side environment variables schema here.
- * This way you can ensure the app isn't built with invalid env vars.
- * To expose them to the client, prefix them with `NEXT_PUBLIC_`.
+ * Specify your client-side environment variables schema here. This way you can ensure the app isn't
+ * built with invalid env vars. To expose them to the client, prefix them with `NEXT_PUBLIC_`.
  */
 const client = z.object({
   // NEXT_PUBLIC_CLIENTVAR: z.string().min(1),
 });
 
 /**
- * You can't destruct `process.env` as a regular object in the Next.js
- * edge runtimes (e.g. middlewares) or client-side so we need to destruct manually.
+ * You can't destruct `process.env` as a regular object in the Next.js edge runtimes (e.g.
+ * middlewares) or client-side so we need to destruct manually.
+ *
  * @type {Record<keyof z.infer<typeof server> | keyof z.infer<typeof client>, string | undefined>}
  */
 const processEnv = {

Diff for: cli/template/extras/config/postcss.config.cjs

@@ -1,6 +1,8 @@
-module.exports = {
+const config = {
   plugins: {
     tailwindcss: {},
     autoprefixer: {},
   },
 };
+
+module.exports = config;

Diff for: cli/template/extras/config/prettier.config.cjs

@@ -1,4 +1,6 @@
 /** @type {import("prettier").Config} */
-module.exports = {
+const config = {
   plugins: [require.resolve("prettier-plugin-tailwindcss")],
 };
+
+module.exports = config;

Diff for: cli/template/extras/config/tailwind.config.cjs

@@ -1,8 +1,10 @@
 /** @type {import('tailwindcss').Config} */
-module.exports = {
+const config = {
   content: ["./src/**/*.{js,ts,jsx,tsx}"],
   theme: {
     extend: {},
   },
   plugins: [],
 };
+
+module.exports = config;

Diff for: package.json

@@ -50,6 +50,7 @@
     "@manypkg/cli": "^0.20.0",
     "@types/eslint": "^8.21.1",
     "@types/node": "^18.14.0",
+    "@types/prettier": "^2.7.2",
     "@typescript-eslint/eslint-plugin": "^5.53.0",
     "@typescript-eslint/parser": "^5.53.0",
     "eslint": "^8.34.0",

Diff for: pnpm-lock.yaml

@@ -1,6 +1,8 @@
 /** @type {import("eslint").Linter.Config} */
-module.exports = {
+const config = {
   parserOptions: {
     extraFileExtensions: [".astro"],
   },
 };
+
+module.exports = config;

Diff for: www/.eslintrc.cjs

@@ -1,5 +1,5 @@
 /** @type {import('prettier').Config} */
-module.exports = {
+const config = {
   ...require("../.prettierrc.cjs"),
   plugins: [
     require.resolve("prettier-plugin-astro"),
@@ -17,3 +17,5 @@ module.exports = {
   astroAllowShorthand: false,
   tailwindConfig: "./tailwind.config.cjs",
 };
+
+module.exports = config;

Diff for: www/.prettierrc.cjs

@@ -1,5 +1,5 @@
 /** @type {import('tailwindcss').Config} */
-module.exports = {
+const config = {
   content: ["./src/**/*.{astro,html,js,jsx,md,mdx,svelte,ts,tsx,vue}"],
   darkMode: "class",
   theme: {
@@ -68,3 +68,5 @@ module.exports = {
   // eslint-disable-next-line @typescript-eslint/no-var-requires
   plugins: [require("tailwind-scrollbar")({ nocompatible: true })],
 };
+
+module.exports = config;