feat: improve oxc lint/fmt, add husky hooks to pre-commit and CI/CD (#386)

azizbecha · juliusmarminge · web-flow · commit 5f68dca51843 · 2025-12-15T04:33:18.000Z
Co-authored-by: Julius &lt;julius0216@outlook.com&gt;
diff --git a/.config/.oxlintrc.json b/.config/.oxlintrc.json
@@ -1,5 +1,5 @@
 {
-  "$schema": "./node_modules/oxlint/configuration_schema.json",
+  "$schema": "../node_modules/oxlint/configuration_schema.json",
   "plugins": ["typescript", "import"],
   "categories": {
     "correctness": "warn",
diff --git a/.config/turbo.jsonc b/.config/turbo.jsonc
@@ -4,31 +4,31 @@
   "tasks": {
     "dev": {
       "dependsOn": ["^dev"],
-      "cache": false
+      "cache": false,
     },
     "clean": {
-      "cache": false
+      "cache": false,
     },
     "build": {
       "dependsOn": ["^build"],
-      "outputs": ["dist/**", ".next/**", ".nuxt"]
+      "outputs": ["dist/**", ".next/**", ".nuxt"],
     },
     "typecheck": {
       "dependsOn": ["transit", "^build"],
-      "outputs": [".cache/tsbuildinfo.json"]
+      "outputs": [".cache/tsbuildinfo.json"],
     },
     "//#test": {
       "persistent": true,
-      "outputs": []
+      "outputs": [],
     },
     "//#test:run": {
-      "outputs": []
+      "outputs": [],
     },
     // Transit node in the turbo graph
     // no package.json scrips actually implement transit but it creates a dependency link for other tasks:
     // https://turbo.build/docs/crafting-your-repository/configuring-tasks#dependent-tasks-that-can-be-run-in-parallel
     "transit": {
-      "dependsOn": ["^transit"]
-    }
-  }
+      "dependsOn": ["^transit"],
+    },
+  },
 }
diff --git a/.github/setup/action.yml b/.github/setup/action.yml
@@ -1,7 +1,7 @@
 name: Setup Workflow
 description: Composite action that sets up bun and installs dependencies
 runs:
-  using: 'composite'
+  using: "composite"
   steps:
     - uses: actions/setup-node@v6
       with:
diff --git a/.github/workflows/release-canary.yaml b/.github/workflows/release-canary.yaml
@@ -25,12 +25,14 @@ jobs:
       - name: Checkout repo
         uses: actions/checkout@v6
       - uses: ./.github/setup
+      - run: bun lint
+      - run: bun fmt --check
 
       - name: Build
         run: bun run build
 
       - name: Authenticate to npm
-        run:  echo "//registry.npmjs.org/:_authToken=${{ secrets.NPM_TOKEN }}" > .npmrc
+        run: echo "//registry.npmjs.org/:_authToken=${{ secrets.NPM_TOKEN }}" > .npmrc
 
       - name: Publish
         run: bun run scripts/canary.ts
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -25,5 +25,6 @@
   "oxc.fmt.configPath": ".config/.oxfmtrc.json",
   "typescript.tsdk": "node_modules/typescript/lib",
   "typescript.experimental.useTsgo": true,
-  "typescript.enablePromptUseWorkspaceTsdk": true
+  "typescript.enablePromptUseWorkspaceTsdk": true,
+  "oxc.enable": true
 }
diff --git a/README.md b/README.md
@@ -27,15 +27,14 @@ Deploying your app with invalid environment variables is a hassle. This package
 
  # deno
  deno add jsr:@t3-oss/env-nuxt
- ```
+```
 
 ## Usage
 
 > [!NOTE]
 >
 > You may use any Standard Schema compliant validator of your choice. This example uses Zod
 
-
 This package supports the full power of Zod/Valibot etc, meaning you can use `transforms` and `default` values.
 
 ### Define your schema
@@ -137,7 +136,7 @@ Implement your changes, then submit a [Pull Request](https://docs.github.com/en/
 - Run the linters using `bun lint` (`bun lint:fix` to fix auto-fixable issues)
 - Run the tests using `bun run test`. Alternatively, you can use the test explorer in VSCode to target individual tests.
 - Commit with whatever commit message you want. We squash PRs so the individual commit messages does not matter.
-  - For bug fixes, it's a good practise to first add a failing regression test in one commit, and then implement the fix in another. This helps the review process as it's easy to see what was failing before, and how it succeeded after your change. 
+  - For bug fixes, it's a good practise to first add a failing regression test in one commit, and then implement the fix in another. This helps the review process as it's easy to see what was failing before, and how it succeeded after your change.
 - Add a pull request description explaining your changes. Link any relevant issues (e.g. by including `Closes #123` in the description).
 - If some change might require extra consideration, a quick self-review can help maintainers more easily review your code and get it merged faster.
 
diff --git a/bun.lock b/bun.lock
diff --git a/docs/src/app/docs/customization/page.mdx b/docs/src/app/docs/customization/page.mdx
@@ -104,8 +104,8 @@ T3 Env ships the following presets out of the box, all importable from the `/pre
 - `railway` - Railway provided system environment variables. See full list [here](https://docs.railway.app/reference/variables#railway-provided-variables).
 - `fly.io` - Fly.io provided machine runtime environment variables. See full list [here](https://fly.io/docs/machines/runtime-environment/#environment-variables).
 - `netlify` - Netlify provided system environment variables. See full list [here](https://docs.netlify.com/configure-builds/environment-variables).
-- `upstashRedis` -  Upstash Redis environment variables. More info [here](https://upstash.com/docs/redis/howto/connectwithupstashredis).
-- `coolify` -  Coolify environment variables. More info [here](https://coolify.io/docs/knowledge-base/environment-variables#predefined-variables).
+- `upstashRedis` - Upstash Redis environment variables. More info [here](https://upstash.com/docs/redis/howto/connectwithupstashredis).
+- `coolify` - Coolify environment variables. More info [here](https://coolify.io/docs/knowledge-base/environment-variables#predefined-variables).
 - `vite` - Vite environment variables. More info [here](https://vite.dev/guide/env-and-mode).
 - `wxt` - WXT environment variables. More info [here](https://wxt.dev/guide/essentials/config/environment-variables.html#built-in-environment-variables).
 
diff --git a/docs/src/app/docs/nextjs/page.mdx b/docs/src/app/docs/nextjs/page.mdx
@@ -116,7 +116,7 @@ export const env = createEnv({
 
 <Callout type="info">
 
-If you're using the `standalone` output in your `next.config.ts`, make sure to include the following:  
+If you're using the `standalone` output in your `next.config.ts`, make sure to include the following:
 
 ```ts title="next.config.ts"
 import type { NextConfig } from "next"
@@ -150,7 +150,7 @@ export default {
 };
 ```
 
-We do not recommend using `next.config.ts` as it does not support loading ESM. 
+We do not recommend using `next.config.ts` as it does not support loading ESM.
 
 ### Use your schema
 
diff --git a/docs/src/app/docs/recipes/page.mdx b/docs/src/app/docs/recipes/page.mdx
@@ -61,10 +61,10 @@ export const env = createEnv({
 
 ## Storybook
 
-[Storybook](https://storybook.js.org/) uses its own bundler, which is otherwise 
-unaware of `t3-env` and won't call into `runtimeEnv` to ensure that the environment 
-variables are present. You can use Storybook's support for defining environment 
-variables separately to ensure that all environment variables are actually 
+[Storybook](https://storybook.js.org/) uses its own bundler, which is otherwise
+unaware of `t3-env` and won't call into `runtimeEnv` to ensure that the environment
+variables are present. You can use Storybook's support for defining environment
+variables separately to ensure that all environment variables are actually
 available for Storybook:
 
 ```ts
diff --git a/docs/tsconfig.json b/docs/tsconfig.json
@@ -25,16 +25,16 @@
 
     /* Path Aliases */
     "paths": {
-      "@/*": ["./src/*"]
-    }
+      "@/*": ["./src/*"],
+    },
   },
   "include": [
     "next-env.d.ts",
     "**/*.ts",
     "**/*.tsx",
     "*.js",
     ".next/types/**/*.ts",
-    ".next/dev/types/**/*.ts"
+    ".next/dev/types/**/*.ts",
   ],
-  "exclude": ["node_modules"]
+  "exclude": ["node_modules"],
 }
diff --git a/examples/astro-arktype/tsconfig.json b/examples/astro-arktype/tsconfig.json
@@ -9,7 +9,7 @@
     "declaration": false,
     "declarationMap": false,
     "noEmit": true,
-    "types": ["astro/client"]
+    "types": ["astro/client"],
   },
-  "include": ["src", "astro.config.mjs"]
+  "include": ["src", "astro.config.mjs"],
 }
diff --git a/examples/astro-valibot/tsconfig.json b/examples/astro-valibot/tsconfig.json
@@ -9,7 +9,7 @@
     "declaration": false,
     "declarationMap": false,
     "noEmit": true,
-    "types": ["astro/client"]
+    "types": ["astro/client"],
   },
-  "include": ["src", "astro.config.mjs"]
+  "include": ["src", "astro.config.mjs"],
 }
diff --git a/examples/astro-zod/tsconfig.json b/examples/astro-zod/tsconfig.json
@@ -9,7 +9,7 @@
     "declaration": false,
     "declarationMap": false,
     "noEmit": true,
-    "types": ["astro/client"]
+    "types": ["astro/client"],
   },
-  "include": ["src", "astro.config.mjs"]
+  "include": ["src", "astro.config.mjs"],
 }
diff --git a/examples/nextjs/app/globals.css b/examples/nextjs/app/globals.css
@@ -2,9 +2,8 @@
   --max-width: 1100px;
   --border-radius: 12px;
   --font-mono:
-    ui-monospace, Menlo, Monaco, "Cascadia Mono", "Segoe UI Mono",
-    "Roboto Mono", "Oxygen Mono", "Ubuntu Monospace", "Source Code Pro",
-    "Fira Mono", "Droid Sans Mono", "Courier New", monospace;
+    ui-monospace, Menlo, Monaco, "Cascadia Mono", "Segoe UI Mono", "Roboto Mono", "Oxygen Mono",
+    "Ubuntu Monospace", "Source Code Pro", "Fira Mono", "Droid Sans Mono", "Courier New", monospace;
 
   --foreground-rgb: 0, 0, 0;
   --background-start-rgb: 214, 219, 220;
@@ -18,10 +17,7 @@
     #0071ff33 160deg,
     transparent 360deg
   );
-  --secondary-glow: radial-gradient(
-    rgba(255, 255, 255, 1),
-    rgba(255, 255, 255, 0)
-  );
+  --secondary-glow: radial-gradient(rgba(255, 255, 255, 1), rgba(255, 255, 255, 0));
 
   --tile-start-rgb: 239, 245, 249;
   --tile-end-rgb: 228, 232, 233;
@@ -94,11 +90,7 @@ body {
 
 body {
   color: rgb(var(--foreground-rgb));
-  background: linear-gradient(
-      to bottom,
-      transparent,
-      rgb(var(--background-end-rgb))
-    )
+  background: linear-gradient(to bottom, transparent, rgb(var(--background-end-rgb)))
     rgb(var(--background-start-rgb));
 }
 
diff --git a/examples/nextjs/tsconfig.json b/examples/nextjs/tsconfig.json
@@ -10,14 +10,8 @@
     "noEmit": true,
     "plugins": [{ "name": "next" }],
     "paths": {
-      "~/*": ["./app/*"]
-    }
+      "~/*": ["./app/*"],
+    },
   },
-  "include": [
-    "next-env.d.ts",
-    "app",
-    ".next/types/**/*.ts",
-    ".next/dev/types/**/*.ts",
-    "*.js"
-  ]
+  "include": ["next-env.d.ts", "app", ".next/types/**/*.ts", ".next/dev/types/**/*.ts", "*.js"],
 }
diff --git a/package.json b/package.json
@@ -19,6 +19,7 @@
     "dev": "bun run turbo watch build test --filter @t3-oss/env* --filter //",
     "fmt": "oxfmt -c .config/.oxfmtrc.json",
     "lint": "oxlint -c .config/.oxlintrc.json --type-aware --type-check --deny-warnings --report-unused-disable-directives",
+    "lint:fix": "oxlint -c .config/.oxlintrc.json --fix && oxfmt -c .config/.oxfmtrc.json",
     "test": "vitest --silent=passed-only",
     "test:run": "vitest run --silent=passed-only"
   },
@@ -27,10 +28,10 @@
     "@changesets/cli": "^2.29.7",
     "@types/bun": "^1.3.2",
     "@typescript/native-preview": "7.0.0-dev.20251125.1",
-    "oxfmt": "0.15.0",
-    "oxlint": "1.30.0",
-    "oxlint-tsgolint": "^0.8.2",
+    "oxfmt": "0.17.0",
+    "oxlint": "1.32.0",
+    "oxlint-tsgolint": "0.9.0",
     "turbo": "2.6.1",
-    "vitest": "4.0.14"
+    "vitest": "^4.0.15"
   }
 }
diff --git a/packages/core/CHANGELOG.md b/packages/core/CHANGELOG.md
@@ -61,7 +61,6 @@
   Combination of schemas can now be customised using the `createFinalSchema` option. This allows further refinement or transformation of the environment variables.
 
   For 99% of users, this is a non-breaking change. If you were relying on internal types, there's a type-only breaking change:
-
   - `CreateEnv` now has the signature `CreateEnv<TFinalSchema, TExtends>`, instead of the previous `CreateEnv<TServer, TClient, TShared, TExtends>`.
     - Previous behaviour can be achieved by using `DefaultCombinedSchema<TServer, TClient, TShared>` as the type for `TFinalSchema`.
 
@@ -79,7 +78,6 @@
   for example Zod & Valibot.
 
   This feature comes with some breaking changes:
-
   - If using Zod, the minimum required version is now 3.24.
   - `onValidationError` now gets `StandardSchemaV1.Issue[]` instead of `ZodError`
 
@@ -88,7 +86,6 @@
   Presets using Valibot are now available using the `/presets-valibot` entrypoint.
 
   This feature comes with some breaking changes:
-
   - Zod presets have now been moved to `/presets-zod`.
 
 - [`bb80923`](https://github.com/t3-oss/t3-env/commit/bb809231d07a089391ed2949e5f1161caf1b3a30) Thanks [@juliusmarminge](https://github.com/juliusmarminge)! - update uploadthing preset to v7. add `uploadthingV6` for legacy config
diff --git a/packages/core/README.md b/packages/core/README.md
@@ -9,7 +9,6 @@ This is the framework agnostic core package of t3-env.
 
 > For full documentation, see https://env.t3.gg
 
-
 ## Installation
 
 ```bash
@@ -60,4 +59,4 @@ export const env = createEnv({
    */
   runtimeEnv: process.env,
 });
-```
+```
diff --git a/packages/core/src/index.ts b/packages/core/src/index.ts
@@ -236,9 +236,9 @@ export interface ServerOptions<
       : TPrefix extends ""
         ? TServer[TKey]
         : TKey extends `${TPrefix}${string}`
-          ? ErrorMessage<`${
-              TKey extends `${TPrefix}${string}` ? TKey : never
-            } should not prefixed with ${TPrefix}.`>
+          ? ErrorMessage<`${TKey extends `${TPrefix}${string}`
+              ? TKey
+              : never} should not prefixed with ${TPrefix}.`>
           : TServer[TKey];
   }>;
 }
diff --git a/packages/core/tsconfig.json b/packages/core/tsconfig.json
@@ -1,5 +1,5 @@
 {
   "extends": "../../tsconfig.json",
   "include": ["src", "test"],
-  "exclude": ["dist"]
+  "exclude": ["dist"],
 }
diff --git a/packages/nextjs/CHANGELOG.md b/packages/nextjs/CHANGELOG.md
@@ -79,7 +79,6 @@
   Combination of schemas can now be customised using the `createFinalSchema` option. This allows further refinement or transformation of the environment variables.
 
   For 99% of users, this is a non-breaking change. If you were relying on internal types, there's a type-only breaking change:
-
   - `CreateEnv` now has the signature `CreateEnv<TFinalSchema, TExtends>`, instead of the previous `CreateEnv<TServer, TClient, TShared, TExtends>`.
     - Previous behaviour can be achieved by using `DefaultCombinedSchema<TServer, TClient, TShared>` as the type for `TFinalSchema`.
 
@@ -98,7 +97,6 @@
   for example Zod & Valibot.
 
   This feature comes with some breaking changes:
-
   - If using Zod, the minimum required version is now 3.24.
   - `onValidationError` now gets `StandardSchemaV1.Issue[]` instead of `ZodError`
 
@@ -107,7 +105,6 @@
   Presets using Valibot are now available using the `/presets-valibot` entrypoint.
 
   This feature comes with some breaking changes:
-
   - Zod presets have now been moved to `/presets-zod`.
 
 ### Patch Changes
diff --git a/packages/nextjs/README.md b/packages/nextjs/README.md
@@ -8,7 +8,7 @@
 The Next.js package comes preconfigured for Next.js and also enforces some extra rules by default to make sure you have out-of-the-box compatibility in all different Next.js runtimes.
 
 > For full documentation, see https://env.t3.gg
-> 
+
 ## Installation
 
 ```bash
@@ -55,7 +55,7 @@ export const env = createEnv({
   },
   /*
    * Specify what values should be validated by your schemas above.
-   * 
+   *
    * If you're using Next.js < 13.4.4, you'll need to specify the runtimeEnv manually
    * For Next.js >= 13.4.4, you can use the experimental__runtimeEnv option and
    * only specify client-side variables.
@@ -69,4 +69,4 @@ export const env = createEnv({
   //   NEXT_PUBLIC_PUBLISHABLE_KEY: process.env.NEXT_PUBLIC_PUBLISHABLE_KEY,
   // }
 });
-```
+```
diff --git a/packages/nextjs/tsconfig.json b/packages/nextjs/tsconfig.json
@@ -1,5 +1,5 @@
 {
   "extends": "../../tsconfig.json",
   "include": ["src", "test"],
-  "exclude": ["dist"]
+  "exclude": ["dist"],
 }
diff --git a/packages/nuxt/CHANGELOG.md b/packages/nuxt/CHANGELOG.md
diff --git a/packages/nuxt/README.md b/packages/nuxt/README.md
diff --git a/packages/nuxt/tsconfig.json b/packages/nuxt/tsconfig.json
diff --git a/tsconfig.json b/tsconfig.json

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`{`
`2`		`- "$schema": "./node_modules/oxlint/configuration_schema.json",`
	`2`	`+ "$schema": "../node_modules/oxlint/configuration_schema.json",`
`3`	`3`	`"plugins": ["typescript", "import"],`
`4`	`4`	`"categories": {`
`5`	`5`	`"correctness": "warn",`
Original file line number	Diff line number	Diff line change
`@@ -25,5 +25,6 @@`
`25`	`25`	`"oxc.fmt.configPath": ".config/.oxfmtrc.json",`
`26`	`26`	`"typescript.tsdk": "node_modules/typescript/lib",`
`27`	`27`	`"typescript.experimental.useTsgo": true,`
`28`		`- "typescript.enablePromptUseWorkspaceTsdk": true`
	`28`	`+ "typescript.enablePromptUseWorkspaceTsdk": true,`
	`29`	`+ "oxc.enable": true`
`29`	`30`	`}`