Test+Lint+Refactor

.vscode/launch.json

@@ -66,18 +66,17 @@
         // Pass a directory to run tests in
         "${workspaceFolder}/extensions/vscode/manual-testing-sandbox",
         "--extensionDevelopmentPath=${workspaceFolder}/extensions/vscode",
-        "--extensionTestsPath=${workspaceFolder}/extensions/vscode/out/test-runner/mochaRunner"
+        "--extensionTestsPath=${workspaceFolder}/extensions/vscode/out/test/runner/mochaRunner"
       ],
       "outFiles": [
         // Allows setting breakpoints in test suites across the /src folder
-        "${workspaceFolder}/extensions/vscode/out/test-suites/**/*.js",
+        "${workspaceFolder}/extensions/vscode/out/test/test-suites/**/*.js",
         // Allows setting breakpoints in mocha test runner file
-        "${workspaceFolder}/extensions/vscode/out/test-runner/**/*.js"
+        "${workspaceFolder}/extensions/vscode/out/test/runner/**/*.js"
       ],
       "internalConsoleOptions": "openOnSessionStart",
       "preLaunchTask": "vscode-extension:tests:build",
       "env": {
-        "CONTINUE_SERVER_URL": "http://localhost:65432",
         // Avoid timing out when stopping on breakpoints during debugging in VSCode
         "MOCHA_TIMEOUT": "0"
       }

binary/package.json

@@ -29,6 +29,7 @@
   "author": "",
   "license": "Apache-2.0",
   "devDependencies": {
+    "@biomejs/biome": "1.6.4",
     "@types/follow-redirects": "^1.14.4",
     "@types/uuid": "^9.0.8",
     "@vercel/ncc": "^0.38.1",

binary/src/IpcIde.ts

@@ -1,5 +1,5 @@
 import { MessageIde } from "core/util/messageIde";
-import { IpcMessenger } from "./messenger";
+import type { IpcMessenger } from "./messenger";
 
 export class IpcIde extends MessageIde {
   constructor(messenger: IpcMessenger) {

binary/src/core.ts

@@ -1,4 +1,4 @@
-import { ContextItemId, IDE } from "core";
+import type { ContextItemId, IDE } from "core";
 import { CompletionProvider } from "core/autocomplete/completionProvider";
 import { ConfigHandler } from "core/config/handler";
 import { addModel, addOpenAIKey, deleteModel } from "core/config/util";
@@ -7,12 +7,12 @@ import TransformersJsEmbeddingsProvider from "core/indexing/embeddings/Transform
 import { CodebaseIndexer, PauseToken } from "core/indexing/indexCodebase";
 import { logDevData } from "core/util/devdata";
 import historyManager from "core/util/history";
-import { Message } from "core/util/messenger";
+import type { Message } from "core/util/messenger";
 import { Telemetry } from "core/util/posthog";
 import { streamDiffLines } from "core/util/verticalEdit";
 import { v4 as uuidv4 } from "uuid";
-import { IpcMessenger } from "./messenger";
-import { Protocol } from "./protocol";
+import type { IpcMessenger } from "./messenger";
+import type { Protocol } from "./protocol";
 
 export class Core {
   private messenger: IpcMessenger;

binary/src/index.ts

@@ -1,7 +1,7 @@
 process.env.IS_BINARY = "true";
+import fs from "node:fs";
 import { Command } from "commander";
 import { getCoreLogsPath } from "core/util/paths";
-import fs from "fs";
 import { IpcIde } from "./IpcIde";
 import { Core } from "./core";
 import { IpcMessenger } from "./messenger";

binary/src/messenger.ts

@@ -1,9 +1,9 @@
 import { v4 as uuidv4 } from "uuid";
 
+import * as fs from "node:fs";
 import { getCoreLogsPath } from "core/util/paths";
-import * as fs from "fs";
-import { Message } from "../../core/util/messenger";
-import { Protocol, ReverseProtocol } from "./protocol";
+import type { Message } from "../../core/util/messenger";
+import type { Protocol, ReverseProtocol } from "./protocol";
 
 export class IpcMessenger {
   typeListeners = new Map<keyof Protocol, ((message: Message) => any)[]>();
@@ -50,7 +50,7 @@ export class IpcMessenger {
     try {
       const msg: Message = JSON.parse(line);
       if (msg.messageType === undefined || msg.messageId === undefined) {
-        throw new Error("Invalid message sent: " + JSON.stringify(msg));
+        throw new Error(`Invalid message sent: ${JSON.stringify(msg)}`);
       }
 
       // Call handler and respond with return value
@@ -99,7 +99,7 @@ export class IpcMessenger {
       messageId,
     };
     // process.send?.(data);
-    process.stdout?.write(JSON.stringify(data) + "\r\n");
+    process.stdout?.write(`${JSON.stringify(data)}\r\n`);
     return messageId;
   }
 

biome.json

@@ -0,0 +1,22 @@
+{
+  "$schema": "https://biomejs.dev/schemas/1.6.4/schema.json",
+  "organizeImports": {
+    "enabled": true
+  },
+  "linter": {
+    "enabled": true,
+    "rules": {
+      "recommended": true,
+      "suspicious": {
+        "noExplicitAny": "off"
+      },
+      "style": {
+        "noUselessElse": "off",
+        "noNonNullAssertion": "off"
+      }
+    }
+  },
+  "formatter": {
+    "indentStyle": "space"
+  }
+}

core/autocomplete/ast.ts

@@ -1,5 +1,5 @@
-import Parser from "web-tree-sitter";
-import { RangeInFileWithContents } from "../commands/util";
+import type Parser from "web-tree-sitter";
+import type { RangeInFileWithContents } from "../commands/util";
 import { getParserForFile } from "../util/treeSitter";
 
 export async function getAst(
@@ -27,7 +27,7 @@ export async function getTreePathAtCursor(
   const path = [ast.rootNode];
   while (path[path.length - 1].childCount > 0) {
     let foundChild = false;
-    for (let child of path[path.length - 1].children) {
+    for (const child of path[path.length - 1].children) {
       if (child.startIndex <= cursorIndex && child.endIndex >= cursorIndex) {
         path.push(child);
         foundChild = true;
@@ -63,7 +63,7 @@ export async function getScopeAroundRange(
   let node = ast.rootNode;
   while (node.childCount > 0) {
     let foundChild = false;
-    for (let child of node.children) {
+    for (const child of node.children) {
       if (child.startIndex < startIndex && child.endIndex > endIndex) {
         node = child;
         foundChild = true;

core/autocomplete/cache.ts

@@ -1,10 +1,10 @@
 import { open } from "sqlite";
 import sqlite3 from "sqlite3";
-import { DatabaseConnection } from "../indexing/refreshIndex";
+import type { DatabaseConnection } from "../indexing/refreshIndex";
 import { getTabAutocompleteCacheSqlitePath } from "../util/paths";
 
 export class AutocompleteLruCache {
-  private static capacity: number = 1000;
+  private static capacity = 1000;
 
   db: DatabaseConnection;
 

core/autocomplete/charStream.ts

@@ -27,7 +27,7 @@ export async function* onlyWhitespaceAfterEndOfLine(
 
 export async function* noFirstCharNewline(stream: AsyncGenerator<string>) {
   let first = true;
-  for await (let char of stream) {
+  for await (const char of stream) {
     if (first) {
       first = false;
       if (char === "\n") return;