fix!: change failOnWarn default from 'ci-only' to false

sxzz · sxzz · commit ad8db9b05630 · 2026-02-26T17:08:55.000+09:00
diff --git a/docs/advanced/ci.md b/docs/advanced/ci.md
@@ -28,7 +28,7 @@ The following options accept CI-aware values:
 - [`exports`](/options/package-exports) — Auto-generate `package.json` exports
 - `unused` — Unused dependency check
 - `devtools` — DevTools integration
-- `failOnWarn` — Fail on warnings (defaults to `'ci-only'`)
+- `failOnWarn` — Fail on warnings (defaults to `false`)
 
 ### Basic Usage
 
@@ -42,7 +42,7 @@ export default defineConfig({
   dts: 'local-only',
   // Only run publint in CI
   publint: 'ci-only',
-  // Fail on warnings in CI only (this is the default)
+  // Fail on warnings in CI only
   failOnWarn: 'ci-only',
 })
 ```
@@ -90,7 +90,7 @@ export default defineConfig({
   entry: 'src/index.ts',
   format: ['esm', 'cjs'],
   dts: true,
-  // Fail on warnings in CI
+  // Fail on warnings in CI (opt-in)
   failOnWarn: 'ci-only',
   // Run package validators in CI
   publint: 'ci-only',
diff --git a/docs/options/log-level.md b/docs/options/log-level.md
@@ -29,20 +29,16 @@ Choose the log level that best fits your workflow to control the amount of infor
 
 ## Fail on Warnings
 
-The `failOnWarn` option controls whether warnings cause the build to exit with a non-zero code. By default, this is set to `'ci-only'`, which means **warnings will cause the build to fail in CI environments** but not locally.
-
-This default behavior ensures that CI pipelines catch potential issues while keeping local development uninterrupted.
+The `failOnWarn` option controls whether warnings cause the build to exit with a non-zero code.
 
 ```ts [tsdown.config.ts]
 import { defineConfig } from 'tsdown'
 
 export default defineConfig({
-  // Default: fail on warnings only in CI
-  failOnWarn: 'ci-only',
   // Always fail on warnings
-  // failOnWarn: true,
-  // Never fail on warnings
-  // failOnWarn: false,
+  failOnWarn: true,
+  // Fail on warnings only in CI
+  failOnWarn: 'ci-only',
 })
 ```
 
diff --git a/docs/zh-CN/advanced/ci.md b/docs/zh-CN/advanced/ci.md
@@ -28,7 +28,7 @@ tsdown 使用 [`is-in-ci`](https://www.npmjs.com/package/is-in-ci) 包来检测
 - [`exports`](/zh-CN/options/package-exports) — 自动生成 `package.json` exports 字段
 - `unused` — 未使用依赖检查
 - `devtools` — DevTools 集成
-- `failOnWarn` — 遇警告时失败（默认值为 `'ci-only'`）
+- `failOnWarn` — 遇警告时失败（默认值为 `false`）
 
 ### 基本用法
 
@@ -42,7 +42,7 @@ export default defineConfig({
   dts: 'local-only',
   // 仅在 CI 中运行 publint
   publint: 'ci-only',
-  // 仅在 CI 中遇警告时失败（这是默认行为）
+  // 仅在 CI 中遇警告时失败
   failOnWarn: 'ci-only',
 })
 ```
@@ -90,7 +90,7 @@ export default defineConfig({
   entry: 'src/index.ts',
   format: ['esm', 'cjs'],
   dts: true,
-  // 在 CI 中遇警告时失败
+  // 在 CI 中遇警告时失败（需手动启用）
   failOnWarn: 'ci-only',
   // 在 CI 中运行包校验工具
   publint: 'ci-only',
diff --git a/docs/zh-CN/options/log-level.md b/docs/zh-CN/options/log-level.md
@@ -29,20 +29,16 @@ tsdown --log-level error
 
 ## 警告时失败
 
-`failOnWarn` 选项控制警告是否会导致构建以非零退出码退出。默认值为 `'ci-only'`，这意味着**在 CI 环境中，警告会导致构建失败**，但在本地开发时不会。
-
-这种默认行为确保 CI 流水线能够发现潜在问题，同时不会影响本地开发体验。
+`failOnWarn` 选项控制警告是否会导致构建以非零退出码退出。
 
 ```ts [tsdown.config.ts]
 import { defineConfig } from 'tsdown'
 
 export default defineConfig({
-  // 默认：仅在 CI 中遇警告时失败
-  failOnWarn: 'ci-only',
   // 始终在遇到警告时失败
-  // failOnWarn: true,
-  // 从不在遇到警告时失败
-  // failOnWarn: false,
+  failOnWarn: true,
+  // 仅在 CI 中遇警告时失败
+  failOnWarn: 'ci-only',
 })
 ```
 
diff --git a/skills/tsdown/SKILL.md b/skills/tsdown/SKILL.md
@@ -77,7 +77,7 @@ export default defineConfig({
 | Source maps | `sourcemap: true`, `'inline'`, `'hidden'` | [option-sourcemap](references/option-sourcemap.md) |
 | Watch mode | `watch: true`, watch options | [option-watch-mode](references/option-watch-mode.md) |
 | Cleaning | `clean: true`, clean patterns | [option-cleaning](references/option-cleaning.md) |
-| Log level | `logLevel: 'silent'`, `failOnWarn: 'ci-only'` | [option-log-level](references/option-log-level.md) |
+| Log level | `logLevel: 'silent'`, `failOnWarn: false` | [option-log-level](references/option-log-level.md) |
 
 ## Dependency Handling
 
@@ -182,7 +182,7 @@ export default defineConfig({
   entry: ['src/index.ts'],
   format: ['esm', 'cjs'],
   dts: true,
-  failOnWarn: 'ci-only',
+  failOnWarn: 'ci-only',  // opt-in: fail on warnings in CI
   publint: 'ci-only',
   attw: 'ci-only',
 })
diff --git a/skills/tsdown/references/advanced-ci.md b/skills/tsdown/references/advanced-ci.md
@@ -28,7 +28,7 @@ These options accept CI-aware values:
 - `exports` - Auto-generate `package.json` exports
 - `unused` - Unused dependency check
 - `devtools` - DevTools integration
-- `failOnWarn` - Fail on warnings (defaults to `'ci-only'`)
+- `failOnWarn` - Fail on warnings (defaults to `false`)
 
 ## Usage
 
@@ -38,7 +38,7 @@ These options accept CI-aware values:
 export default defineConfig({
   dts: 'local-only',        // Skip DTS in CI for faster builds
   publint: 'ci-only',       // Only run publint in CI
-  failOnWarn: 'ci-only',    // Fail on warnings in CI only (default)
+  failOnWarn: 'ci-only',    // Fail on warnings in CI only (opt-in)
 })
 ```
 
diff --git a/skills/tsdown/references/option-log-level.md b/skills/tsdown/references/option-log-level.md
@@ -72,13 +72,13 @@ export default defineConfig({
 
 ## Fail on Warnings
 
-The `failOnWarn` option controls whether warnings cause the build to exit with a non-zero code. Defaults to `'ci-only'` — warnings fail the build in CI but not locally.
+The `failOnWarn` option controls whether warnings cause the build to exit with a non-zero code. Defaults to `false` — warnings never fail the build.
 
 ```ts
 export default defineConfig({
-  failOnWarn: 'ci-only', // Default: fail on warnings only in CI
+  failOnWarn: false,     // Default: never fail on warnings
   // failOnWarn: true,   // Always fail on warnings
-  // failOnWarn: false,  // Never fail on warnings
+  // failOnWarn: 'ci-only', // Fail on warnings only in CI
 })
 ```
 
diff --git a/src/config/options.ts b/src/config/options.ts
@@ -52,7 +52,7 @@ export async function resolveUserConfig(
     plugins = [],
     clean = true,
     logLevel = 'info',
-    failOnWarn = 'ci-only',
+    failOnWarn = false,
     customLogger,
     treeshake = true,
     platform = 'node',
diff --git a/src/config/types.ts b/src/config/types.ts
@@ -449,7 +449,7 @@ export interface UserConfig {
   logLevel?: LogLevel
   /**
    * If true, fails the build on warnings.
-   * @default 'ci-only'
+   * @default false
    */
   failOnWarn?: boolean | CIOption
   /**
