fix: update app details handling and sync logic in app_sync_task (#11830)

- Modified the logic to retain app details computed from remote sources, only falling back to existing details when necessary.
- Enhanced the synchronization process to ensure only remote resources are included in the synced app IDs for both addition and update scenarios.

Author: HynoR

agent/app/service/app_sync_task.go

@@ -363,7 +363,9 @@ func (c *appSyncContext) classifyAndPersistAppsWithStats(addCount, updateCount,
 			for _, app := range addAppArray {
 				if existing, ok := existingMap[app.Key]; ok {
 					app.ID = existing.ID
-					app.Details = existing.Details
+					if len(app.Details) == 0 {
+						app.Details = existing.Details
+					}
 					updateAppArray = append(updateAppArray, app)
 				} else {
 					filteredAdd = append(filteredAdd, app)
@@ -477,27 +479,31 @@ func (c *appSyncContext) classifyAndPersistAppsWithStats(addCount, updateCount,
 		}
 	}
 
-	syncedAppIds := make([]uint, 0, len(addAppArray)+len(updateAppArray)+len(deleteIds))
-	for _, app := range addAppArray {
-		if app.ID > 0 {
-			syncedAppIds = append(syncedAppIds, app.ID)
+	if len(c.appTags) > 0 {
+		syncedAppIds := make([]uint, 0, len(addAppArray)+len(updateAppArray)+len(deleteIds))
+		for _, app := range addAppArray {
+			if app.ID > 0 && app.Resource == constant.AppResourceRemote {
+				syncedAppIds = append(syncedAppIds, app.ID)
+			}
 		}
-	}
-	for _, app := range updateAppArray {
-		syncedAppIds = append(syncedAppIds, app.ID)
-	}
-	syncedAppIds = append(syncedAppIds, deleteIds...)
-
-	if len(syncedAppIds) > 0 {
-		if err = appTagRepo.DeleteByAppIds(ctx, syncedAppIds); err != nil {
-			return
+		for _, app := range updateAppArray {
+			if app.Resource == constant.AppResourceRemote {
+				syncedAppIds = append(syncedAppIds, app.ID)
+			}
+		}
+		syncedAppIds = append(syncedAppIds, deleteIds...)
+		if len(syncedAppIds) > 0 {
+			if err = appTagRepo.DeleteByAppIds(ctx, syncedAppIds); err != nil {
+				return
+			}
 		}
-	}
-
-	if len(c.appTags) > 0 {
 		if err = appTagRepo.BatchCreate(ctx, c.appTags); err != nil {
 			return
 		}
+	} else if len(deleteIds) > 0 {
+		if err = appTagRepo.DeleteByAppIds(ctx, deleteIds); err != nil {
+			return
+		}
 	}
 
 	if err = tx.Commit().Error; err != nil {