feat(deps)!: disable resolveDepSubpath by default

Author: sxzz

docs/guide/how-it-works.md

@@ -38,7 +38,7 @@ When you publish a library, your consumers install its `dependencies`, `peerDepe
 | [`deps.onlyBundle`](../options/dependencies.md#deps-onlybundle)               | Whitelist of dependencies allowed to be bundled. Any unlisted dependency that ends up in the bundle causes an error. Useful for catching accidental inlining in large projects. |
 | [`deps.neverBundle`](../options/dependencies.md#deps-neverbundle)             | Explicitly mark additional packages as external (never bundled), or `true` to externalize all dependencies.                                                                     |
 | [`deps.alwaysBundle`](../options/dependencies.md#deps-alwaysbundle)           | Force specific packages to be bundled, even if they're in `dependencies`.                                                                                                       |
-| [`deps.resolveDepSubpath`](../options/dependencies.md#deps-resolvedepsubpath) | Resolve external dependency subpath imports to their actual package-relative paths.                                                                                             |
+| [`deps.resolveDepSubpath`](../options/dependencies.md#deps-resolvedepsubpath) | Optionally resolve external dependency subpath imports to their actual package-relative paths.                                                                                  |
 
 See [Dependencies](../options/dependencies.md) for details.
 

docs/options/dependencies.md

@@ -32,7 +32,7 @@ export default defineConfig({
     alwaysBundle: ['some-package'],
     onlyBundle: ['cac', 'bumpp'],
     onlyImport: ['cac'],
-    resolveDepSubpath: false,
+    resolveDepSubpath: true,
   },
 })
 ```
@@ -45,21 +45,21 @@ export default defineConfig({
 
 ### `deps.resolveDepSubpath`
 
-When an external dependency has no `exports` field, tsdown resolves subpath imports to their actual package-relative paths by default. For example, `my-dep/functions/lt` may become `my-dep/functions/lt.js`, and `my-dep/folder` may become `my-dep/folder/index.js`.
+By default, tsdown preserves external dependency subpath imports as written. Enable `resolveDepSubpath` to resolve subpath imports to their actual package-relative paths when a package has no `exports` field. For example, `my-dep/functions/lt` may become `my-dep/functions/lt.js`, and `my-dep/folder` may become `my-dep/folder/index.js`.
 
-Set `resolveDepSubpath` to `false` to preserve the original import specifier:
+Set `resolveDepSubpath` to `true` to enable this behavior:
 
 ```ts [tsdown.config.ts]
 import { defineConfig } from 'tsdown'
 
 export default defineConfig({
   deps: {
-    resolveDepSubpath: false,
+    resolveDepSubpath: true,
   },
 })
 ```
 
-The default value is `true`.
+The default value is `false`.
 
 ### `deps.onlyBundle`
 
@@ -220,7 +220,7 @@ The following top-level options are deprecated. Please migrate to the `deps` nam
   - Use `deps.onlyImport` to whitelist packages the output is allowed to import at runtime.
   - Use `deps.neverBundle` to mark specific dependencies as external, or set it to `true` to externalize all dependencies.
   - Use `deps.alwaysBundle` to force specific dependencies to be bundled.
-  - Set `deps.resolveDepSubpath` to `false` to preserve external dependency subpath imports as written.
+  - Enable `deps.resolveDepSubpath` to resolve external dependency subpath imports to their package-relative paths.
 - **Declaration Files**:
   - The bundling logic for declaration files is now the same as for JavaScript.
   - Use `resolver: 'tsc'` for better compatibility with complex third-party types.

docs/zh-CN/guide/how-it-works.md

@@ -38,7 +38,7 @@ tsdown 读取你的 `package.json` 和 `tsconfig.json` 来推断合理的默认
 | [`deps.onlyBundle`](../options/dependencies.md#deps-onlybundle)               | 允许打包的依赖白名单。任何不在列表中的依赖如果出现在 bundle 中将触发错误。适用于防止大型项目中的意外内联。 |
 | [`deps.neverBundle`](../options/dependencies.md#deps-neverbundle)             | 显式将额外的包标记为外部依赖（不打包），设置为 `true` 时外部化所有依赖。                                   |
 | [`deps.alwaysBundle`](../options/dependencies.md#deps-alwaysbundle)           | 强制打包特定的包，即使它们在 `dependencies` 中。                                                           |
-| [`deps.resolveDepSubpath`](../options/dependencies.md#deps-resolvedepsubpath) | 将外部依赖的子路径导入解析为包内的实际相对路径。                                                           |
+| [`deps.resolveDepSubpath`](../options/dependencies.md#deps-resolvedepsubpath) | 可选择将外部依赖的子路径导入解析为包内的实际相对路径。                                                     |
 
 详见[依赖](../options/dependencies.md)。
 

docs/zh-CN/options/dependencies.md

@@ -32,7 +32,7 @@ export default defineConfig({
     alwaysBundle: ['some-package'],
     onlyBundle: ['cac', 'bumpp'],
     onlyImport: ['cac'],
-    resolveDepSubpath: false,
+    resolveDepSubpath: true,
   },
 })
 ```
@@ -45,21 +45,21 @@ export default defineConfig({
 
 ### `deps.resolveDepSubpath`
 
-当外部依赖没有 `exports` 字段时，tsdown 默认会将子路径导入解析为包内的实际相对路径。例如，`my-dep/functions/lt` 可能会变为 `my-dep/functions/lt.js`，而 `my-dep/folder` 可能会变为 `my-dep/folder/index.js`。
+默认情况下，tsdown 会保留外部依赖原有的子路径导入。当依赖没有 `exports` 字段时，可以启用 `resolveDepSubpath`，将子路径导入解析为包内的实际相对路径。例如，`my-dep/functions/lt` 可能会变为 `my-dep/functions/lt.js`，而 `my-dep/folder` 可能会变为 `my-dep/folder/index.js`。
 
-将 `resolveDepSubpath` 设置为 `false` 可保留原始导入说明符：
+将 `resolveDepSubpath` 设置为 `true` 可启用此行为：
 
 ```ts [tsdown.config.ts]
 import { defineConfig } from 'tsdown'
 
 export default defineConfig({
   deps: {
-    resolveDepSubpath: false,
+    resolveDepSubpath: true,
   },
 })
 ```
 
-默认值为 `true`。
+默认值为 `false`。
 
 ### `deps.onlyBundle`
 
@@ -220,7 +220,7 @@ export default defineConfig({
   - 使用 `deps.onlyImport` 设置产物在运行时允许导入的包白名单。
   - 使用 `deps.neverBundle` 将特定依赖标记为外部依赖，或将其设置为 `true` 外部化所有依赖。
   - 使用 `deps.alwaysBundle` 强制将特定依赖打包。
-  - 将 `deps.resolveDepSubpath` 设置为 `false` 可保留外部依赖原有的子路径导入。
+  - 启用 `deps.resolveDepSubpath` 可将外部依赖的子路径导入解析为包内的相对路径。
 - **声明文件**：
   - 声明文件的打包逻辑与 JavaScript 保持一致。
   - 使用 `resolver: 'tsc'` 可提升复杂第三方类型的兼容性。

src/features/deps.test.ts

@@ -9,13 +9,13 @@ import {
 import type { UserConfig } from '../config/types.ts'
 
 describe('resolveDepsConfig', () => {
-  it('enables dependency subpath resolution by default', () => {
-    expect(resolveDepsConfig({}).resolveDepSubpath).toBe(true)
+  it('disables dependency subpath resolution by default', () => {
+    expect(!!resolveDepsConfig({}).resolveDepSubpath).toBe(false)
     expect(
       resolveDepsConfig({
-        deps: { resolveDepSubpath: false },
+        deps: { resolveDepSubpath: true },
       }).resolveDepSubpath,
-    ).toBe(false)
+    ).toBe(true)
   })
 
   it('rejects skipNodeModulesBundle together with neverBundle: true', () => {
@@ -122,11 +122,31 @@ describe('DepsPlugin', () => {
       external: true,
       moduleSideEffects: undefined,
     })
+    await expect(
+      handler.call({ resolve }, 'some-dep/utils', '/project/src/index.ts', {}),
+    ).resolves.toEqual({
+      id: 'some-dep/utils',
+      external: true,
+      moduleSideEffects: undefined,
+    })
     expect(resolve).not.toHaveBeenCalled()
 
-    // subpath imports require resolution for `resolveDepSubpath`
+    // subpath imports require resolution when `resolveDepSubpath` is enabled
+    const subpathPlugin = DepsPlugin(
+      {
+        pkg: { dependencies: { 'some-dep': '^1.0.0' } },
+        deps: resolveDepsConfig({ deps: { resolveDepSubpath: true } }),
+      } as any,
+      { inlinedDeps: new Map() } as any,
+    )
+    const subpathHandler = (subpathPlugin.resolveId as any).handler
     await expect(
-      handler.call({ resolve }, 'some-dep/utils', '/project/src/index.ts', {}),
+      subpathHandler.call(
+        { resolve },
+        'some-dep/utils',
+        '/project/src/index.ts',
+        {},
+      ),
     ).resolves.toEqual({
       id: 'some-dep/utils',
       external: true,

src/features/deps.ts

@@ -91,7 +91,7 @@ export interface DepsConfig {
    * Resolve dependency subpath imports to their actual package-relative paths
    * when externalizing packages without an `exports` field.
    *
-   * @default true
+   * @default false
    */
   resolveDepSubpath?: boolean
 
@@ -125,7 +125,7 @@ export function resolveDepsConfig(
     onlyBundle,
     onlyImport,
     skipNodeModulesBundle,
-    resolveDepSubpath = true,
+    resolveDepSubpath,
   } = config.deps || {}
 
   if (config.external != null) {

tests/e2e.test.ts

@@ -1417,6 +1417,9 @@ describe('resolve dep subpath without exports field', () => {
           dependencies: { 'my-dep': '^1.0.0' },
         }),
       },
+      options: {
+        deps: { resolveDepSubpath: true },
+      },
     })
 
     expect(fileMap['index.mjs']).toContain('my-dep/functions/lt.js')
@@ -1475,6 +1478,9 @@ describe('resolve dep subpath without exports field', () => {
           dependencies: { 'my-dep': '^1.0.0' },
         }),
       },
+      options: {
+        deps: { resolveDepSubpath: true },
+      },
     })
 
     expect(fileMap['index.mjs']).toContain('my-dep/folder/index.js')
@@ -1502,7 +1508,10 @@ describe('resolve dep subpath without exports field', () => {
         }),
       },
       options: {
-        deps: { skipNodeModulesBundle: true },
+        deps: {
+          resolveDepSubpath: true,
+          skipNodeModulesBundle: true,
+        },
       },
     })
 
@@ -1564,7 +1573,10 @@ describe('resolve dep subpath without exports field', () => {
         }),
       },
       options: {
-        deps: { skipNodeModulesBundle: true },
+        deps: {
+          resolveDepSubpath: true,
+          skipNodeModulesBundle: true,
+        },
       },
     })