Avoid running remote queries on v2.6.3 cli or earlier

Also:

- Fix the count of copied files
- A few typos

Author: aeisenberg

extensions/ql-vscode/src/cli.ts

@@ -1134,6 +1134,11 @@ export class CliVersionConstraint {
    */
   public static CLI_VERSION_PACK_PACKLIST_LAYOUT_CHANGE = new SemVer('2.7.1');
 
+  /**
+   * CLI version where remote queries are supported.
+   */
+  public static CLI_VERSION_REMOTE_QUERIES = new SemVer('2.6.3');
+
   constructor(private readonly cli: CodeQLCliServer) {
     /**/
   }
@@ -1177,4 +1182,9 @@ export class CliVersionConstraint {
   async usesNewPackPacklistLayout() {
     return this.isVersionAtLeast(CliVersionConstraint.CLI_VERSION_PACK_PACKLIST_LAYOUT_CHANGE);
   }
+
+  async supportsRemoteQueries() {
+    return this.isVersionAtLeast(CliVersionConstraint.CLI_VERSION_REMOTE_QUERIES);
+  }
+
 }

extensions/ql-vscode/src/run-remote-query.ts

@@ -98,18 +98,27 @@ async function generateQueryPack(cliServer: cli.CodeQLCliServer, queryFile: stri
 
     // also copy the lock file (either new name or old name) and the query file itself. These are not included in the packlist.
     [path.join(originalPackRoot, 'qlpack.lock.yml'), path.join(originalPackRoot, 'codeql-pack.lock.yml'), queryFile]
-      .forEach(aboslutePath => {
-        if (aboslutePath) {
-          toCopy.push(aboslutePath);
+      .forEach(absolutePath => {
+        if (absolutePath) {
+          toCopy.push(absolutePath);
         }
       });
-    void logger.log(`Copying ${toCopy.length} files to ${queryPackDir}`);
+
+    let copiedCount = 0;
     await fs.copy(originalPackRoot, queryPackDir, {
       filter: (file: string) =>
         // copy file if it is in the packlist, or it is a parent directory of a file in the packlist
-        !!toCopy.find(f => f === file || f.startsWith(file + path.sep)
-        )
+        !!toCopy.find(f => {
+          const matches = f === file || f.startsWith(file + path.sep);
+          if (matches) {
+            copiedCount++;
+          }
+          return matches;
+        })
     });
+
+    void logger.log(`Copied ${copiedCount} files to ${queryPackDir}`);
+
     language = await findLanguage(cliServer, Uri.file(targetQueryFileName));
 
   } else {
@@ -173,6 +182,11 @@ export async function runRemoteQuery(
   progress: ProgressCallback,
   token: CancellationToken
 ): Promise<void | string> {
+  if (!(await cliServer.cliConstraints.supportsRemoteQueries())) {
+    throw new Error(`Remote queries are not supported by this version of CodeQL. Please upgrade to v${cli.CliVersionConstraint.CLI_VERSION_REMOTE_QUERIES
+      } or later.`);
+  }
+
   const { remoteQueryDir, queryPackDir } = await createRemoteQueriesTempDirectory();
   try {
     if (!uri?.fsPath.endsWith('.ql')) {
@@ -214,7 +228,6 @@ export async function runRemoteQuery(
     }
 
     if (!repositories || repositories.length === 0) {
-      // No error message needed, since `getRepositories` already displays one.
       throw new UserCancellationException('No repositories to query.');
     }
 

extensions/ql-vscode/src/vscode-tests/cli-integration/run-remote-query.test.ts

@@ -8,12 +8,12 @@ import * as yaml from 'js-yaml';
 
 import { runRemoteQuery } from '../../run-remote-query';
 import { Credentials } from '../../authentication';
-import { CodeQLCliServer } from '../../cli';
+import { CliVersionConstraint, CodeQLCliServer } from '../../cli';
 import { CodeQLExtensionInterface } from '../../extension';
 import { setRemoteControllerRepo } from '../../config';
 import { UserCancellationException } from '../../commandRunner';
 
-describe('Remote queries', function() {
+describe.only('Remote queries', function() {
   const baseDir = path.join(__dirname, '../../../src/vscode-tests/cli-integration');
 
   let sandbox: sinon.SinonSandbox;
@@ -27,7 +27,8 @@ describe('Remote queries', function() {
   let progress: sinon.SinonSpy;
   let showQuickPickSpy: sinon.SinonStub;
 
-  beforeEach(async () => {
+  // use `function` so we have access to `this`
+  beforeEach(async function() {
     sandbox = sinon.createSandbox();
 
     const extension = await extensions.getExtension<CodeQLExtensionInterface | Record<string, never>>('GitHub.vscode-codeql')!.activate();
@@ -36,6 +37,12 @@ describe('Remote queries', function() {
     } else {
       throw new Error('Extension not initialized. Make sure cli is downloaded and installed properly.');
     }
+
+    if (!(await cli.cliConstraints.supportsRemoteQueries())) {
+      console.log(`Remote queries are not supported on CodeQL CLI v${CliVersionConstraint.CLI_VERSION_REMOTE_QUERIES
+        }. Skipping this test.`);
+      this.skip();
+    }
     credentials = {} as unknown as Credentials;
     token = {
       isCancellationRequested: false