fix auto selection

spx-gui/src/models/project/index.test.ts

@@ -73,4 +73,55 @@ describe('Project', () => {
     expect(project.sprites.map((s) => s.name)).toEqual(['Sprite1', 'Sprite3', 'Sprite2'])
     expect(project.sounds.map((s) => s.name)).toEqual(['sound1', 'sound3', 'sound2'])
   })
+
+  it('should select correctly after sound removed', async () => {
+    const project = makeProject()
+    const sprite2 = new Sprite('Sprite2')
+    project.addSprite(sprite2)
+    const sound2 = new Sound('sound2', mockFile())
+    project.addSound(sound2)
+    const sound3 = new Sound('sound3', mockFile())
+    project.addSound(sound3)
+
+    project.select({ type: 'stage' })
+    project.removeSound('sound3')
+    expect(project.selected).toEqual({ type: 'stage' })
+
+    project.select({ type: 'sound', name: 'sound' })
+
+    project.removeSound('sound')
+    expect(project.selected).toEqual({
+      type: 'sound',
+      name: 'sound2'
+    })
+
+    project.removeSound('sound2')
+    expect(project.selected).toEqual({
+      type: 'sprite',
+      name: 'Sprite'
+    })
+  })
+
+  it('should select correctly after sprite removed', async () => {
+    const project = makeProject()
+    const sprite2 = new Sprite('Sprite2')
+    project.addSprite(sprite2)
+    const sprite3 = new Sprite('Sprite3')
+    project.addSprite(sprite3)
+
+    project.select({ type: 'stage' })
+    project.removeSprite('Sprite3')
+    expect(project.selected).toEqual({ type: 'stage' })
+
+    project.select({ type: 'sprite', name: 'Sprite' })
+
+    project.removeSprite('Sprite')
+    expect(project.selected).toEqual({
+      type: 'sprite',
+      name: 'Sprite2'
+    })
+
+    project.removeSprite('Sprite2')
+    expect(project.selected).toBeNull()
+  })
 })

spx-gui/src/models/project/index.ts

@@ -240,11 +240,12 @@ export class Project extends Disposable {
    */
   private autoSelect() {
     const selected = this.selected
-    if (selected?.type === 'sprite' && this.selectedSprite == null) {
-      this.select(this.sprites[0] != null ? { type: 'sprite', name: this.sprites[0].name } : null)
-    } else if (selected?.type === 'sound' && this.selectedSound == null) {
-      this.select(this.sounds[0] != null ? { type: 'sound', name: this.sounds[0].name } : null)
-    } else if (selected == null) {
+    if (selected?.type === 'stage') return
+    if (selected?.type === 'sound' && this.selectedSound == null && this.sounds[0] != null) {
+      this.select({ type: 'sound', name: this.sounds[0].name })
+      return
+    }
+    if (this.selectedSprite == null) {
       this.select(this.sprites[0] != null ? { type: 'sprite', name: this.sprites[0].name } : null)
     }
   }