fix(keybindings): honor null overrides and terminal shortcut forwarding

Author: bajrangCoder

src/cm/commandRegistry.js

@@ -108,6 +108,7 @@ const commandKeymapCompartment = new Compartment();
  *  run: (view?: EditorView | null) => boolean | void;
  *  requiresView?: boolean;
  *  defaultDescription?: string;
+ *  defaultKey?: string | null;
  *  key?: string | null;
  * }} CommandEntry
  */
@@ -1177,6 +1178,7 @@ function addCommand(entry) {
 	const command = {
 		...entry,
 		defaultDescription: entry.description || entry.name,
+		defaultKey: entry.key ?? null,
 		key: entry.key ?? null,
 	};
 	commandMap.set(entry.name, command);
@@ -1314,6 +1316,26 @@ function parseKeyString(keyString) {
 		.filter(Boolean);
 }
 
+function hasOwnBindingOverride(name) {
+	return Object.prototype.hasOwnProperty.call(resolvedKeyBindings ?? {}, name);
+}
+
+function resolveBindingInfo(name) {
+	const baseBinding = keyBindings[name] ?? null;
+	if (!hasOwnBindingOverride(name)) return baseBinding;
+
+	const override = resolvedKeyBindings?.[name];
+	if (override === null) {
+		return baseBinding ? { ...baseBinding, key: null } : { key: null };
+	}
+
+	if (!override || typeof override !== "object") {
+		return baseBinding;
+	}
+
+	return baseBinding ? { ...baseBinding, ...override } : override;
+}
+
 function toCodeMirrorKey(combo) {
 	if (!combo) return null;
 	const parts = combo
@@ -1356,10 +1378,13 @@ function toCodeMirrorKey(combo) {
 function rebuildKeymap() {
 	const bindings = [];
 	commandMap.forEach((command, name) => {
-		const bindingInfo = resolvedKeyBindings?.[name];
+		const bindingInfo = resolveBindingInfo(name);
 		command.description =
 			bindingInfo?.description || command.defaultDescription;
-		const keySource = bindingInfo?.key ?? command.key ?? null;
+		const keySource =
+			bindingInfo && Object.prototype.hasOwnProperty.call(bindingInfo, "key")
+				? bindingInfo.key
+				: (command.defaultKey ?? null);
 		command.key = keySource;
 		const combos = parseKeyString(keySource);
 		combos.forEach((combo) => {
@@ -1410,6 +1435,19 @@ export function getRegisteredCommands() {
 	}));
 }
 
+export function getResolvedKeyBindings() {
+	const bindingNames = new Set([
+		...Object.keys(keyBindings),
+		...Object.keys(resolvedKeyBindings ?? {}),
+	]);
+
+	return Object.fromEntries(
+		Array.from(bindingNames, (name) => [name, resolveBindingInfo(name)]).filter(
+			([, binding]) => binding,
+		),
+	);
+}
+
 export function getCommandKeymapExtension() {
 	return commandKeymapCompartment.of(keymap.of(cachedKeymap));
 }

src/components/terminal/terminal.js

@@ -11,10 +11,10 @@ import { Unicode11Addon } from "@xterm/addon-unicode11";
 import { WebLinksAddon } from "@xterm/addon-web-links";
 import { WebglAddon } from "@xterm/addon-webgl";
 import { Terminal as Xterm } from "@xterm/xterm";
+import { getResolvedKeyBindings } from "cm/commandRegistry";
 import toast from "components/toast";
 import confirm from "dialogs/confirm";
 import fonts from "lib/fonts";
-import keyBindings from "lib/keyBindings";
 import appSettings from "lib/settings";
 import LigaturesAddon from "./ligatures";
 import { getTerminalSettings } from "./terminalDefaults";
@@ -337,7 +337,7 @@ export default class TerminalComponent {
 	parseAppKeybindings() {
 		const parsedBindings = [];
 
-		Object.values(keyBindings).forEach((binding) => {
+		Object.values(getResolvedKeyBindings()).forEach((binding) => {
 			if (!binding.key) return;
 
 			// Skip editor-only keybindings in terminal
@@ -368,7 +368,7 @@ export default class TerminalComponent {
 						parsed.meta = true;
 					} else {
 						// This is the actual key
-						parsed.key = part;
+						parsed.key = part.toLowerCase();
 					}
 				});
 
@@ -432,7 +432,7 @@ export default class TerminalComponent {
 						binding.shift === event.shiftKey &&
 						binding.alt === event.altKey &&
 						binding.meta === event.metaKey &&
-						binding.key === event.key,
+						binding.key === event.key.toLowerCase(),
 				);
 
 				if (isAppKeybinding) {