Split codeQL.copyVariantAnalysisRepoList into two commands

Author: robertbrignull

extensions/ql-vscode/src/common/commands.ts

@@ -244,7 +244,11 @@ export type VariantAnalysisCommands = {
     scannedRepo: VariantAnalysisScannedRepository,
     variantAnalysisSummary: VariantAnalysis,
   ) => Promise<void>;
-  "codeQL.copyVariantAnalysisRepoList": (
+  "codeQL.copyVariantAnalysisRepoListQueryHistory": (
+    variantAnalysisId: number,
+    filterSort?: RepositoriesFilterSortStateWithIds,
+  ) => Promise<void>;
+  "codeQL.copyVariantAnalysisRepoListView": (
     variantAnalysisId: number,
     filterSort?: RepositoriesFilterSortStateWithIds,
   ) => Promise<void>;

extensions/ql-vscode/src/query-history/query-history-manager.ts

@@ -953,7 +953,7 @@ export class QueryHistoryManager extends DisposableObject {
     }
 
     await this.app.commands.execute(
-      "codeQL.copyVariantAnalysisRepoList",
+      "codeQL.copyVariantAnalysisRepoListQueryHistory",
       item.variantAnalysis.id,
     );
   }

extensions/ql-vscode/src/variant-analysis/variant-analysis-manager.ts

@@ -148,7 +148,9 @@ export class VariantAnalysisManager
     return {
       "codeQL.autoDownloadVariantAnalysisResult":
         this.enqueueDownload.bind(this),
-      "codeQL.copyVariantAnalysisRepoList":
+      "codeQL.copyVariantAnalysisRepoListQueryHistory":
+        this.copyRepoListToClipboard.bind(this),
+      "codeQL.copyVariantAnalysisRepoListView":
         this.copyRepoListToClipboard.bind(this),
       "codeQL.loadVariantAnalysisRepoResults": this.loadResults.bind(this),
       "codeQL.monitorNewVariantAnalysis":

extensions/ql-vscode/src/variant-analysis/variant-analysis-view.ts

@@ -140,7 +140,7 @@ export class VariantAnalysisView
         break;
       case "copyRepositoryList":
         void this.app.commands.execute(
-          "codeQL.copyVariantAnalysisRepoList",
+          "codeQL.copyVariantAnalysisRepoListView",
           this.variantAnalysisId,
           msg.filterSort,
         );

extensions/ql-vscode/test/vscode-tests/no-workspace/query-history/query-history-manager.test.ts

@@ -714,7 +714,7 @@ describe("QueryHistoryManager", () => {
       const item = variantAnalysisHistory[1];
       await queryHistoryManager.handleCopyRepoList(item);
       expect(executeCommand).toBeCalledWith(
-        "codeQL.copyVariantAnalysisRepoList",
+        "codeQL.copyVariantAnalysisRepoListQueryHistory",
         item.variantAnalysis.id,
       );
     });