[PR #6461] [MERGED] chore: enforce imports to use node: protocol #14929

Closed
opened 2026-04-13 09:42:36 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/6461
Author: @himself65
Created: 12/1/2025
Status: Merged
Merged: 12/2/2025
Merged by: @himself65

Base: canaryHead: himself65/2025/12/01/node-protocol


📝 Commits (2)

  • e9ec7a3 chore: enforce imports to use node: protocol
  • 8ba1ce3 fix: ignore

📊 Changes

30 files changed (+59 additions, -58 deletions)

View changed files

📝 biome.json (+2 -1)
📝 demo/expo/metro.config.js (+1 -1)
📝 demo/oidc-client/vite.config.ts (+1 -1)
📝 docs/lib/blog.ts (+2 -2)
📝 docs/scripts/endpoint-to-doc/index.ts (+2 -2)
📝 packages/better-auth/src/adapters/drizzle-adapter/test/adapter.drizzle.mysql.test.ts (+1 -1)
📝 packages/better-auth/src/adapters/drizzle-adapter/test/adapter.drizzle.pg.test.ts (+1 -1)
📝 packages/better-auth/src/adapters/drizzle-adapter/test/adapter.drizzle.sqlite.test.ts (+3 -3)
📝 packages/better-auth/src/adapters/drizzle-adapter/test/generate-schema.ts (+2 -2)
📝 packages/better-auth/src/adapters/kysely-adapter/test/adapter.kysely.sqlite.test.ts (+2 -2)
📝 packages/better-auth/src/adapters/prisma-adapter/test/generate-auth-config.ts (+2 -2)
📝 packages/better-auth/src/adapters/prisma-adapter/test/generate-prisma-schema.ts (+2 -2)
📝 packages/better-auth/src/adapters/prisma-adapter/test/prisma.sqlite.test.ts (+1 -1)
📝 packages/better-auth/src/integrations/node.ts (+1 -1)
📝 packages/cli/src/commands/generate.ts (+3 -3)
📝 packages/cli/src/commands/info.ts (+4 -4)
📝 packages/cli/src/commands/init.ts (+3 -3)
📝 packages/cli/src/commands/login.ts (+3 -3)
📝 packages/cli/src/commands/mcp.ts (+4 -4)
📝 packages/cli/src/commands/migrate.ts (+2 -2)

...and 10 more files

📄 Description

Summary by cubic

Enforces the node: protocol for all Node.js built-in module imports and adds a Biome rule to prevent non-prefixed imports. Standardizes imports across the repo with no runtime changes.

  • Refactors
    • Added "useNodejsImportProtocol": "error" to biome.json.
    • Replaced built-in imports with node: across CLI, adapters, tests, docs, demos, and integrations.

Written for commit 8ba1ce385c. Summary will update automatically on new commits.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/better-auth/better-auth/pull/6461 **Author:** [@himself65](https://github.com/himself65) **Created:** 12/1/2025 **Status:** ✅ Merged **Merged:** 12/2/2025 **Merged by:** [@himself65](https://github.com/himself65) **Base:** `canary` ← **Head:** `himself65/2025/12/01/node-protocol` --- ### 📝 Commits (2) - [`e9ec7a3`](https://github.com/better-auth/better-auth/commit/e9ec7a3fc69a302989e3165105f8835e219e282e) chore: enforce imports to use `node:` protocol - [`8ba1ce3`](https://github.com/better-auth/better-auth/commit/8ba1ce385c06bc5311438735bd578d6888dfe1ca) fix: ignore ### 📊 Changes **30 files changed** (+59 additions, -58 deletions) <details> <summary>View changed files</summary> 📝 `biome.json` (+2 -1) 📝 `demo/expo/metro.config.js` (+1 -1) 📝 `demo/oidc-client/vite.config.ts` (+1 -1) 📝 `docs/lib/blog.ts` (+2 -2) 📝 `docs/scripts/endpoint-to-doc/index.ts` (+2 -2) 📝 `packages/better-auth/src/adapters/drizzle-adapter/test/adapter.drizzle.mysql.test.ts` (+1 -1) 📝 `packages/better-auth/src/adapters/drizzle-adapter/test/adapter.drizzle.pg.test.ts` (+1 -1) 📝 `packages/better-auth/src/adapters/drizzle-adapter/test/adapter.drizzle.sqlite.test.ts` (+3 -3) 📝 `packages/better-auth/src/adapters/drizzle-adapter/test/generate-schema.ts` (+2 -2) 📝 `packages/better-auth/src/adapters/kysely-adapter/test/adapter.kysely.sqlite.test.ts` (+2 -2) 📝 `packages/better-auth/src/adapters/prisma-adapter/test/generate-auth-config.ts` (+2 -2) 📝 `packages/better-auth/src/adapters/prisma-adapter/test/generate-prisma-schema.ts` (+2 -2) 📝 `packages/better-auth/src/adapters/prisma-adapter/test/prisma.sqlite.test.ts` (+1 -1) 📝 `packages/better-auth/src/integrations/node.ts` (+1 -1) 📝 `packages/cli/src/commands/generate.ts` (+3 -3) 📝 `packages/cli/src/commands/info.ts` (+4 -4) 📝 `packages/cli/src/commands/init.ts` (+3 -3) 📝 `packages/cli/src/commands/login.ts` (+3 -3) 📝 `packages/cli/src/commands/mcp.ts` (+4 -4) 📝 `packages/cli/src/commands/migrate.ts` (+2 -2) _...and 10 more files_ </details> ### 📄 Description <!-- This is an auto-generated description by cubic. --> ## Summary by cubic Enforces the node: protocol for all Node.js built-in module imports and adds a Biome rule to prevent non-prefixed imports. Standardizes imports across the repo with no runtime changes. - **Refactors** - Added "useNodejsImportProtocol": "error" to biome.json. - Replaced built-in imports with node:<module> across CLI, adapters, tests, docs, demos, and integrations. <sup>Written for commit 8ba1ce385c06bc5311438735bd578d6888dfe1ca. Summary will update automatically on new commits.</sup> <!-- End of auto-generated description by cubic. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-13 09:42:36 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#14929