Replace unset unknown keybinding error with warn (#2186)

src/stores/keybindingStore.ts

@@ -227,7 +227,7 @@ export const useKeybindingStore = defineStore('keybinding', () => {
       return
     }
 
-    throw new Error(`Unknown keybinding: ${JSON.stringify(keybinding)}`)
+    console.warn(`Unset unknown keybinding: ${JSON.stringify(keybinding)}`)
   }
 
   /**

tests-ui/tests/store/keybindingStore.test.ts

@@ -149,6 +149,24 @@ describe('useKeybindingStore', () => {
     expect(() => store.unsetKeybinding(keybinding)).not.toThrow()
   })
 
+  it('should not throw an error when unsetting unknown keybinding', () => {
+    const store = useKeybindingStore()
+    const keybinding = new KeybindingImpl({
+      commandId: 'test.command',
+      combo: { key: 'I', ctrl: true }
+    })
+    store.addUserKeybinding(keybinding)
+
+    expect(() =>
+      store.unsetKeybinding(
+        new KeybindingImpl({
+          commandId: 'test.foo',
+          combo: { key: 'I', ctrl: true }
+        })
+      )
+    ).not.toThrow()
+  })
+
   it('should remove unset keybinding when adding back a default keybinding', () => {
     const store = useKeybindingStore()
     const defaultKeybinding = new KeybindingImpl({