web: improve getCommandById && improve remove from recents logic

Signed-off-by: 01zulfi <85733202+01zulfi@users.noreply.github.com>
streetwriters · Mar 3, 2025 · 8bdb8d6 · 8bdb8d6
1 parent 230d6d7
commit 8bdb8d6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/apps/web/src/dialogs/command-palette/command-palette-dialog.tsx b/apps/web/src/dialogs/command-palette/command-palette-dialog.tsx
@@ -69,6 +69,7 @@ export const CommandPaletteDialog = DialogManager.register(
 
     const commands = usePromise(async () => {
       select(0);
+      if (!defaultCommands.current)
       defaultCommands.current = await getDefaultCommands();
       const commands = props.isCommandMode
         ? sortCommands(commandSearch(query, defaultCommands.current))
@@ -280,9 +281,10 @@ export const CommandPaletteDialog = DialogManager.register(
                   {command.group === "recent" && (
                     <Button
                       title={strings.removeFromRecents()}
-                      onClick={(e) => {
+                      onClick={async (e) => {
                         e.stopPropagation();
                         removeRecentCommand(command.id);
+                        defaultCommands.current = await getDefaultCommands();
                         commands.refresh();
                       }}
                       variant="secondary"

diff --git a/apps/web/src/dialogs/command-palette/commands.ts b/apps/web/src/dialogs/command-palette/commands.ts
@@ -234,9 +234,10 @@ export async function getDefaultCommands(): Promise<Command[]> {
 }
 
 export function getCommandById(id: string): Command | undefined {
-  return staticCommands
-    .concat(getEditorCommands())
-    .find((command) => command.id === id);
+  return (
+    staticCommands.find((command) => command.id === id) ||
+    getEditorCommands().find((command) => command.id === id)
+  );
 }
 
 export async function resolveRecentCommand(