feat(cli): support removing keybindings via '-' prefix (#22042)

packages/cli/src/ui/key/keyBindings.test.ts

@@ -206,7 +206,7 @@ describe('loadCustomKeybindings', () => {
 
     expect(errors.length).toBeGreaterThan(0);
 
-    expect(errors[0]).toMatch(/error at 0.command: Invalid enum value/);
+    expect(errors[0]).toMatch(/error at 0.command: Invalid command: "unknown"/);
     // Should still have defaults
     expect(config.get(Command.RETURN)).toEqual([new KeyBinding('enter')]);
   });
@@ -227,4 +227,34 @@ describe('loadCustomKeybindings', () => {
       new KeyBinding('ctrl+c'),
     ]);
   });
+
+  it('removes specific bindings when using the minus prefix', async () => {
+    const customJson = JSON.stringify([
+      { command: `-${Command.RETURN}`, key: 'enter' },
+      { command: Command.RETURN, key: 'ctrl+a' },
+    ]);
+    await fs.writeFile(tempFilePath, customJson, 'utf8');
+
+    const { config, errors } = await loadCustomKeybindings();
+
+    expect(errors).toHaveLength(0);
+    // 'enter' should be gone, only 'ctrl+a' should remain
+    expect(config.get(Command.RETURN)).toEqual([new KeyBinding('ctrl+a')]);
+  });
+
+  it('returns an error when attempting to negate a non-existent binding', async () => {
+    const customJson = JSON.stringify([
+      { command: `-${Command.RETURN}`, key: 'ctrl+z' },
+    ]);
+    await fs.writeFile(tempFilePath, customJson, 'utf8');
+
+    const { config, errors } = await loadCustomKeybindings();
+
+    expect(errors.length).toBe(1);
+    expect(errors[0]).toMatch(
+      /Invalid keybinding for command "-basic.confirm": Error: cannot remove "ctrl\+z" since it is not bound/,
+    );
+    // Defaults should still be present
+    expect(config.get(Command.RETURN)).toEqual([new KeyBinding('enter')]);
+  });
 });

packages/cli/src/ui/key/keyBindings.ts

@@ -216,6 +216,16 @@ export class KeyBinding {
       !!key.cmd === !!this.cmd
     );
   }
+
+  equals(other: KeyBinding): boolean {
+    return (
+      this.key === other.key &&
+      this.shift === other.shift &&
+      this.alt === other.alt &&
+      this.ctrl === other.ctrl &&
+      this.cmd === other.cmd
+    );
+  }
 }
 
 /**
@@ -622,10 +632,32 @@ export const commandDescriptions: Readonly<Record<Command, string>> = {
 };
 
 const keybindingsSchema = z.array(
-  z.object({
-    command: z.nativeEnum(Command),
-    key: z.string(),
-  }),
+  z
+    .object({
+      command: z.string().transform((val, ctx) => {
+        const negate = val.startsWith('-');
+        const commandId = negate ? val.slice(1) : val;
+
+        const result = z.nativeEnum(Command).safeParse(commandId);
+        if (!result.success) {
+          ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message: `Invalid command: "${val}".`,
+          });
+          return z.NEVER;
+        }
+
+        return {
+          command: result.data,
+          negate,
+        };
+      }),
+      key: z.string(),
+    })
+    .transform((val) => ({
+      commandEntry: val.command,
+      key: val.key,
+    })),
 );
 
 /**
@@ -648,15 +680,29 @@ export async function loadCustomKeybindings(): Promise<{
 
     if (result.success) {
       config = new Map(defaultKeyBindingConfig);
-      for (const { command, key } of result.data) {
+      for (const { commandEntry, key } of result.data) {
+        const { command, negate } = commandEntry;
         const currentBindings = config.get(command) ?? [];
 
         try {
           const keyBinding = new KeyBinding(key);
-          // Add new binding (prepend so it's the primary one shown in UI)
-          config.set(command, [keyBinding, ...currentBindings]);
+
+          if (negate) {
+            const updatedBindings = currentBindings.filter(
+              (b) => !b.equals(keyBinding),
+            );
+            if (updatedBindings.length === currentBindings.length) {
+              throw new Error(`cannot remove "${key}" since it is not bound`);
+            }
+            config.set(command, updatedBindings);
+          } else {
+            // Add new binding (prepend so it's the primary one shown in UI)
+            config.set(command, [keyBinding, ...currentBindings]);
+          }
         } catch (e) {
-          errors.push(`Invalid keybinding for command "${command}": ${e}`);
+          errors.push(
+            `Invalid keybinding for command "${negate ? '-' : ''}${command}": ${e}`,
+          );
         }
       }
     } else {