fix: fix shortcut file check to handle deleted shortcuts with existing bat files (#134)

hsluoyz · hsluoyz · commit ea2bc2ae7b79 · 2025-11-20T01:04:25.000+08:00
diff --git a/run/run.go b/run/run.go
@@ -89,12 +89,13 @@ func CreateRepo(siteName string, needStart bool, diff string, providerName strin
 			}
 		}
 
-		batExisted, err := updateBatFile(siteName)
+		_, err = updateBatFile(siteName)
 		if err != nil {
 			return wrapRepoError("updateBatFile", path, err)
 		}
 
-		if !batExisted {
+		shortcutPath := getShortcutPath(siteName)
+		if !util.FileExist(shortcutPath) {
 			err = updateShortcutFile(siteName)
 			if err != nil {
 				return wrapRepoError("updateShortcutFile", path, err)
@@ -147,12 +148,13 @@ func CreateRepo(siteName string, needStart bool, diff string, providerName strin
 			}
 		}
 
-		batExisted, err := updateBatFile(siteName)
+		_, err = updateBatFile(siteName)
 		if err != nil {
 			return wrapRepoError("updateBatFile", path, err)
 		}
 
-		if !batExisted {
+		shortcutPath := getShortcutPath(siteName)
+		if !util.FileExist(shortcutPath) {
 			err = updateShortcutFile(siteName)
 			if err != nil {
 				return wrapRepoError("updateShortcutFile", path, err)
diff --git a/run/self.go b/run/self.go
@@ -37,12 +37,13 @@ func InitSelfStart() {
 		panic("InitSelfStart() error, siteName should not be empty")
 	}
 
-	batExisted, err := updateBatFile(siteName)
+	_, err := updateBatFile(siteName)
 	if err != nil {
 		panic(err)
 	}
 
-	if !batExisted {
+	shortcutPath := getShortcutPath(siteName)
+	if !util.FileExist(shortcutPath) {
 		err = updateShortcutFile(siteName)
 		if err != nil {
 			panic(err)

Original file line number	Diff line number	Diff line change
`@@ -89,12 +89,13 @@ func CreateRepo(siteName string, needStart bool, diff string, providerName strin`
`89`	`89`	`}`
`90`	`90`	`}`
`91`	`91`
`92`		`- batExisted, err := updateBatFile(siteName)`
	`92`	`+ _, err = updateBatFile(siteName)`
`93`	`93`	`if err != nil {`
`94`	`94`	`return wrapRepoError("updateBatFile", path, err)`
`95`	`95`	`}`
`96`	`96`
`97`		`- if !batExisted {`
	`97`	`+ shortcutPath := getShortcutPath(siteName)`
	`98`	`+ if !util.FileExist(shortcutPath) {`
`98`	`99`	`err = updateShortcutFile(siteName)`
`99`	`100`	`if err != nil {`
`100`	`101`	`return wrapRepoError("updateShortcutFile", path, err)`
`@@ -147,12 +148,13 @@ func CreateRepo(siteName string, needStart bool, diff string, providerName strin`
`147`	`148`	`}`
`148`	`149`	`}`
`149`	`150`
`150`		`- batExisted, err := updateBatFile(siteName)`
	`151`	`+ _, err = updateBatFile(siteName)`
`151`	`152`	`if err != nil {`
`152`	`153`	`return wrapRepoError("updateBatFile", path, err)`
`153`	`154`	`}`
`154`	`155`
`155`		`- if !batExisted {`
	`156`	`+ shortcutPath := getShortcutPath(siteName)`
	`157`	`+ if !util.FileExist(shortcutPath) {`
`156`	`158`	`err = updateShortcutFile(siteName)`
`157`	`159`	`if err != nil {`
`158`	`160`	`return wrapRepoError("updateShortcutFile", path, err)`
Original file line number	Diff line number	Diff line change
`@@ -37,12 +37,13 @@ func InitSelfStart() {`
`37`	`37`	`panic("InitSelfStart() error, siteName should not be empty")`
`38`	`38`	`}`
`39`	`39`
`40`		`- batExisted, err := updateBatFile(siteName)`
	`40`	`+ _, err := updateBatFile(siteName)`
`41`	`41`	`if err != nil {`
`42`	`42`	`panic(err)`
`43`	`43`	`}`
`44`	`44`
`45`		`- if !batExisted {`
	`45`	`+ shortcutPath := getShortcutPath(siteName)`
	`46`	`+ if !util.FileExist(shortcutPath) {`
`46`	`47`	`err = updateShortcutFile(siteName)`
`47`	`48`	`if err != nil {`
`48`	`49`	`panic(err)`