style: simplify skipable method logic

subframe7536 · subframe7536 · commit 84e3748c022c · 2025-06-13T17:01:37.000+08:00
diff --git a/src/manager/base.ts b/src/manager/base.ts
@@ -51,14 +51,12 @@ export abstract class BaseFileManager implements FileManager {
   }
 
   async skipable(fn: () => Promisable<void>) {
-    let skipMessage
-    // eslint-disable-next-line no-cond-assign
-    if (skipMessage = await this.skipAll?.()) {
+    let skipMessage = await this.skipAll?.()
+    if (skipMessage) {
       promptWarn(skipMessage)
-      return
+    } else {
+      await fn()
     }
-
-    await fn()
   }
 
   abstract patch(content: string): Promisable<string>

Original file line number	Diff line number	Diff line change
`@@ -51,14 +51,12 @@ export abstract class BaseFileManager implements FileManager {`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`async skipable(fn: () => Promisable<void>) {`
`54`		`- let skipMessage`
`55`		`- // eslint-disable-next-line no-cond-assign`
`56`		`- if (skipMessage = await this.skipAll?.()) {`
	`54`	`+ let skipMessage = await this.skipAll?.()`
	`55`	`+ if (skipMessage) {`
`57`	`56`	`promptWarn(skipMessage)`
`58`		`- return`
	`57`	`+ } else {`
	`58`	`+ await fn()`
`59`	`59`	`}`
`60`		`-`
`61`		`- await fn()`
`62`	`60`	`}`
`63`	`61`
`64`	`62`	`abstract patch(content: string): Promisable<string>`