fix(show-all-commands): apply scope and button set to quick pick

Show All Commands (Ctrl+Shift+;) ignored active scope and button set, always displaying default buttons regardless of configuration

Applied same pattern as StatusBarManager and CommandTreeProvider using buttonSetManager.getButtonsForActiveSet() with configManager.getButtonsWithFallback() fallback logic

Author: KubrickCode

src/extension/main.spec.ts

@@ -193,7 +193,9 @@ describe("main", () => {
       expect(createShowAllCommandsCommand).toHaveBeenCalledWith(
         mockConfigReader,
         mockTerminalManager.executeCommand,
-        mockQuickPickCreator
+        mockQuickPickCreator,
+        mockConfigManager,
+        mockButtonSetManager
       );
       expect(commands.showAllCommandsCommand).toBe("mockDisposable");
     });

src/extension/main.ts

@@ -77,7 +77,13 @@ export const registerCommands = (
 
   const showAllCommandsCommand = vscode.commands.registerCommand(
     "quickCommandButtons.showAllCommands",
-    createShowAllCommandsCommand(configReader, terminalManager.executeCommand, quickPickCreator)
+    createShowAllCommandsCommand(
+      configReader,
+      terminalManager.executeCommand,
+      quickPickCreator,
+      configManager,
+      buttonSetManager
+    )
   );
 
   const openConfigCommand = vscode.commands.registerCommand(

src/internal/show-all-commands.ts

@@ -2,15 +2,20 @@ import * as vscode from "vscode";
 import { MESSAGES } from "../shared/constants";
 import { ConfigReader, QuickPickCreator, TerminalExecutor } from "./adapters";
 import { createQuickPickWithShortcuts } from "./command-executor";
+import { ButtonSetManager } from "./managers/button-set-manager";
+import { ConfigManager } from "./managers/config-manager";
 import { createQuickPickItems } from "./utils/ui-items";
 
 export const createShowAllCommandsCommand = (
   configReader: ConfigReader,
   terminalExecutor: TerminalExecutor,
-  quickPickCreator: QuickPickCreator
+  quickPickCreator: QuickPickCreator,
+  configManager: ConfigManager,
+  buttonSetManager: ButtonSetManager
 ) => {
   return () => {
-    const buttons = configReader.getButtons();
+    const activeSetButtons = buttonSetManager.getButtonsForActiveSet();
+    const buttons = activeSetButtons ?? configManager.getButtonsWithFallback(configReader).buttons;
 
     if (buttons.length === 0) {
       vscode.window.showInformationMessage(MESSAGES.ERROR.noCommands);