fix(pkg): fix duplicate configuration warning logic (#935)

ho991217 · sxzz · web-flow · commit 6a0d98c2b679 · 2026-05-03T16:10:16.000+09:00
Co-authored-by: Kevin Deng &lt;sxzz@sxzz.moe&gt;
diff --git a/src/features/pkg/index.ts b/src/features/pkg/index.ts
@@ -95,8 +95,9 @@ export async function bundleDone(
   const publintConfigs = dedupeConfigs(configs, 'publint')
   const attwConfigs = dedupeConfigs(configs, 'attw')
 
-  if (publintConfigs.length > 1 || attwConfigs.length > 1) {
-    publintConfigs[1].logger.warn(
+  const duplicate = publintConfigs[1] || attwConfigs[1]
+  if (duplicate) {
+    duplicate.logger.warn(
       `Multiple publint or attw configurations found for package at ${pkg.packageJsonPath}. Consider merging them for better consistency and performance.`,
     )
   }

Original file line number	Diff line number	Diff line change
`@@ -95,8 +95,9 @@ export async function bundleDone(`
`95`	`95`	`const publintConfigs = dedupeConfigs(configs, 'publint')`
`96`	`96`	`const attwConfigs = dedupeConfigs(configs, 'attw')`
`97`	`97`
`98`		`- if (publintConfigs.length > 1 \|\| attwConfigs.length > 1) {`
`99`		`- publintConfigs[1].logger.warn(`
	`98`	`+ const duplicate = publintConfigs[1] \|\| attwConfigs[1]`
	`99`	`+ if (duplicate) {`
	`100`	`+ duplicate.logger.warn(`
`100`	`101`	`Multiple publint or attw configurations found for package at ${pkg.packageJsonPath}. Consider merging them for better consistency and performance.`,
`101`	`102`	`)`
`102`	`103`	`}`