完善同步

src/main/event/ListEvent.ts

@@ -141,12 +141,12 @@ export class Event extends EventEmitter {
 
   /**
    * 清空列表内的歌曲
-   * @param listId 列表Id
+   * @param ids 列表Id
    * @param isRemote 是否属于远程操作
    */
-  async list_music_clear(listId: string, isRemote: boolean = false) {
-    await global.lx.worker.dbService.musicsClear(listId)
-    this.emit('list_music_clear', listId, isRemote)
+  async list_music_clear(ids: string[], isRemote: boolean = false) {
+    await global.lx.worker.dbService.musicsClear(ids)
+    this.emit('list_music_clear', ids, isRemote)
     this.list_changed()
   }
 

src/main/modules/commonRenderers/list/winRendererEvent.ts

@@ -36,8 +36,8 @@ export const registerRendererEvents = (sendEvent: <T = any>(name: string, params
   const list_music_overwrite = async(listId: string, musicInfos: LX.Music.MusicInfo[]) => {
     sendEvent<LX.List.ListActionMusicOverwrite>(PLAYER_EVENT_NAME.list_music_overwrite, { listId, musicInfos })
   }
-  const list_music_clear = async(listId: string) => {
-    sendEvent<LX.List.ListActionMusicClear>(PLAYER_EVENT_NAME.list_data_overwire, listId)
+  const list_music_clear = async(ids: string[]) => {
+    sendEvent<LX.List.ListActionMusicClear>(PLAYER_EVENT_NAME.list_data_overwire, ids)
   }
   global.lx.event_list.on('list_data_overwrite', list_data_overwrite)
   global.lx.event_list.on('list_create', list_create)

src/main/modules/sync/modules/list.ts

@@ -56,51 +56,51 @@ const handleListAction = ({ action, data }: LX.Sync.ActionList) => {
 const registerListActionEvent = () => {
   const list_data_overwrite = async(listData: MakeOptional<LX.List.ListDataFull, 'tempList'>, isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_data_overwrite', listData)
+    await sendListAction({ action: 'list_data_overwrite', data: listData })
   }
-  const list_create = async(position: number, lists: LX.List.UserListInfo[], isRemote: boolean = false) => {
+  const list_create = async(position: number, listInfos: LX.List.UserListInfo[], isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_create', { position, lists })
+    await sendListAction({ action: 'list_create', data: { position, listInfos } })
   }
   const list_remove = async(ids: string[], isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_remove', ids)
+    await sendListAction({ action: 'list_remove', data: ids })
   }
   const list_update = async(lists: LX.List.UserListInfo[], isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_update', lists)
+    await sendListAction({ action: 'list_update', data: lists })
   }
   const list_update_position = async(position: number, ids: string[], isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_update_position', { position, ids })
+    await sendListAction({ action: 'list_update_position', data: { position, ids } })
   }
   const list_music_overwrite = async(listId: string, musicInfos: LX.Music.MusicInfo[], isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_music_overwrite', { listId, musicInfos })
+    await sendListAction({ action: 'list_music_overwrite', data: { listId, musicInfos } })
   }
-  const list_music_add = async(listId: string, musicInfos: LX.Music.MusicInfo[], addMusicLocationType: LX.AddMusicLocationType, isRemote: boolean = false) => {
+  const list_music_add = async(id: string, musicInfos: LX.Music.MusicInfo[], addMusicLocationType: LX.AddMusicLocationType, isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_music_add', { listId, musicInfos, addMusicLocationType })
+    await sendListAction({ action: 'list_music_add', data: { id, musicInfos, addMusicLocationType } })
   }
   const list_music_move = async(fromId: string, toId: string, musicInfos: LX.Music.MusicInfo[], addMusicLocationType: LX.AddMusicLocationType, isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_music_move', { fromId, toId, musicInfos, addMusicLocationType })
+    await sendListAction({ action: 'list_music_move', data: { fromId, toId, musicInfos, addMusicLocationType } })
   }
   const list_music_remove = async(listId: string, ids: string[], isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_music_remove', { listId, ids })
+    await sendListAction({ action: 'list_music_remove', data: { listId, ids } })
   }
   const list_music_update = async(musicInfos: LX.List.ListActionMusicUpdate, isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_music_update', musicInfos)
+    await sendListAction({ action: 'list_music_update', data: musicInfos })
   }
-  const list_music_clear = async(listId: string, isRemote: boolean = false) => {
+  const list_music_clear = async(ids: string[], isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_music_clear', listId)
+    await sendListAction({ action: 'list_music_clear', data: ids })
   }
   const list_music_update_position = async(listId: string, position: number, ids: string[], isRemote: boolean = false) => {
     if (isRemote) return
-    await sendListAction('list_music_update_position', { listId, position, ids })
+    await sendListAction({ action: 'list_music_update_position', data: { listId, position, ids } })
   }
   global.lx.event_list.on('list_data_overwrite', list_data_overwrite)
   global.lx.event_list.on('list_create', list_create)
@@ -143,9 +143,9 @@ const broadcast = async(action: listAction, data: any, excludeIds: string[] = []
   }
 }
 
-export const sendListAction = async(action: string, data: any) => {
+export const sendListAction = async(action: LX.Sync.ActionList) => {
   // io.sockets
-  return await broadcast('list:action', JSON.stringify({ action, data }))
+  return await broadcast('list:action', JSON.stringify(action))
 }
 
 export const registerListHandler = (_io: Server, socket: LX.Sync.Socket) => {

src/main/modules/sync/server/auth.ts

@@ -8,7 +8,7 @@ import {
 } from './utils'
 
 const authMsg = 'lx-music auth::'
-const helloMsg = 'Hello~::^-^::~v2'
+const helloMsg = 'Hello~::^-^::~v2~'
 
 const requestIps = new Map<string, number>()
 

src/main/modules/sync/server/syncList.ts

@@ -108,9 +108,8 @@ const updateSnapshot = async(path: string, data: string) => {
   })
 }
 
-interface UserDataObj {
-  [listId: string]: LX.List.UserListInfoFull
-}
+type UserDataObj = Record<string, LX.List.UserListInfoFull>
+
 const createUserListDataObj = (listData: LX.Sync.ListData): UserDataObj => {
   const userListDataObj: UserDataObj = {}
   for (const list of listData.userList) userListDataObj[list.id] = list

src/main/types/sync.d.ts

@@ -1,8 +1,6 @@
 import { Socket as _Socket, RemoteSocket as _RemoteSocket } from 'socket.io'
 
-interface DefaultEventsMap {
-  [event: string]: (...args: any[]) => void
-}
+type DefaultEventsMap = Record<string, (...args: any[]) => void>
 
 
 declare global {

src/main/worker/dbService/modules/list/dbHelper.ts

@@ -250,14 +250,16 @@ export const removeMusicInfos = (listId: string, ids: string[]) => {
  * 清空列表内歌曲
  * @param listId 列表id
  */
-export const removeMusicInfoByListId = (listId: string) => {
+export const removeMusicInfoByListId = (ids: string[]) => {
   const db = getDB()
   const musicInfoDeleteByListIdStatement = createMusicInfoDeleteByListIdStatement()
   const musicInfoOrderDeleteByListIdStatement = createMusicInfoOrderDeleteByListIdStatement()
-  db.transaction((listId: string) => {
-    musicInfoDeleteByListIdStatement.run(listId)
-    musicInfoOrderDeleteByListIdStatement.run(listId)
-  })(listId)
+  db.transaction((ids: string[]) => {
+    for (const id of ids) {
+      musicInfoDeleteByListIdStatement.run(id)
+      musicInfoOrderDeleteByListIdStatement.run(id)
+    }
+  })(ids)
 }
 
 /**

src/main/worker/dbService/modules/list/index.ts

@@ -305,11 +305,13 @@ export const musicsUpdate = (musicInfos: LX.List.ListActionMusicUpdate) => {
  * 清空列表内的歌曲
  * @param listId 列表Id
  */
-export const musicsClear = (listId: string) => {
-  removeMusicInfoByListId(listId)
-  const targetList = getListMusics(listId)
-  if (targetList == null) return
-  targetList.slice(0, targetList.length)
+export const musicsClear = (ids: string[]) => {
+  removeMusicInfoByListId(ids)
+  for (const id of ids) {
+    const targetList = musicLists.get(id)
+    if (!targetList) continue
+    targetList.splice(0, targetList.length)
+  }
 }
 
 /**