fix: address code review feedback for keybindings feature

- Fix hasJsonView to omit (array format, not key-value) and add docsUrl
- Add runtime validation in buildKeyMaps for malformed override entries
- Fix mock default config to use "[]" instead of empty string

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: mswiszcz

frontend/app/store/keymodel.ts

@@ -840,6 +840,14 @@ function buildKeyMaps(userOverrides: KeybindingEntry[]): void {
 
     // 2. Apply user overrides in order (last wins)
     for (const override of userOverrides) {
+        if (!override.command || typeof override.command !== "string") {
+            console.warn("Skipping keybinding entry with missing/invalid command");
+            continue;
+        }
+        if (override.key != null && typeof override.key !== "string") {
+            console.warn(`Skipping keybinding entry with invalid key type for command: ${override.command}`);
+            continue;
+        }
         const commandId = override.command.startsWith("-") ? override.command.substring(1) : override.command;
         if (override.command.startsWith("-") || override.key == null) {
             // Unbind: remove the action

frontend/app/view/waveconfig/waveconfig-model.ts

@@ -69,7 +69,7 @@ function makeConfigFiles(isWindows: boolean): ConfigFile[] {
             path: "keybindings.json",
             language: "json",
             description: "Custom keyboard shortcuts",
-            hasJsonView: true,
+            docsUrl: "https://docs.waveterm.dev/keybindings",
         },
         {
             name: "Connections",

frontend/preview/mock/defaultconfig.ts

@@ -20,6 +20,6 @@ export const DefaultFullConfig: FullConfigType = {
     bookmarks: {},
     waveai: waveaiJson as unknown as { [key: string]: AIModeConfigType },
     backgrounds: backgroundsJson as { [key: string]: BackgroundConfigType },
-    keybindings: "",
+    keybindings: "[]",
     configerrors: [],
 };