From e3d28a6cbe9267732058a2b5d7e1fb4f093ddaa1 Mon Sep 17 00:00:00 2001
From: Mark IJbema <mark@kilocode.ai>
Date: Tue, 24 Feb 2026 15:37:50 +0100
Subject: [PATCH] fix: resolve eslint warnings in kilo-vscode autocomplete code

- Remove unused eslint-disable directives for require-yield in
  GeneratorReuseManager.test.ts and llm/index.ts
- Fix == to === in llamaTokenizer.js PriorityQueue.isEmpty()
- Wrap thrown string literals in Error objects in llamaTokenizer.js
---
 .../autocomplete/generation/GeneratorReuseManager.test.ts   | 1 -
 .../src/services/autocomplete/continuedev/core/llm/index.ts | 2 --
 .../autocomplete/continuedev/core/llm/llamaTokenizer.js     | 6 +++---
 3 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/generation/GeneratorReuseManager.test.ts b/packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/generation/GeneratorReuseManager.test.ts
index 3b31572b263..a30399957dd 100644
--- a/packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/generation/GeneratorReuseManager.test.ts
+++ b/packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/generation/GeneratorReuseManager.test.ts
@@ -161,7 +161,6 @@ describe("GeneratorReuseManager", () => {
 
   test("calls onError when generator throws an error", async () => {
     const error = new Error("Generator error")
-    // eslint-disable-next-line require-yield
     const mockGenerator = async function* () {
       throw error
     }
diff --git a/packages/kilo-vscode/src/services/autocomplete/continuedev/core/llm/index.ts b/packages/kilo-vscode/src/services/autocomplete/continuedev/core/llm/index.ts
index 8888ab01604..6d6fef85ba6 100644
--- a/packages/kilo-vscode/src/services/autocomplete/continuedev/core/llm/index.ts
+++ b/packages/kilo-vscode/src/services/autocomplete/continuedev/core/llm/index.ts
@@ -232,7 +232,6 @@ export abstract class BaseLLM implements ILLM {
     return `<${msg.role}>\n${contentToShow}\n\n`
   }
 
-  // eslint-disable-next-line require-yield
   protected async *_streamFim(
     _prefix: string,
     _suffix: string,
@@ -629,7 +628,6 @@ export abstract class BaseLLM implements ILLM {
     throw new Error(`Reranking is not supported for provider type ${this.providerName}`)
   }
 
-  // eslint-disable-next-line require-yield
   protected async *_streamComplete(
     _prompt: string,
     _signal: AbortSignal,
diff --git a/packages/kilo-vscode/src/services/autocomplete/continuedev/core/llm/llamaTokenizer.js b/packages/kilo-vscode/src/services/autocomplete/continuedev/core/llm/llamaTokenizer.js
index 11c720272c0..14a2e2f264c 100644
--- a/packages/kilo-vscode/src/services/autocomplete/continuedev/core/llm/llamaTokenizer.js
+++ b/packages/kilo-vscode/src/services/autocomplete/continuedev/core/llm/llamaTokenizer.js
@@ -35,7 +35,7 @@ class PriorityQueue {
     return this._heap.length
   }
   isEmpty() {
-    return this.size() == 0
+    return this.size() === 0
   }
   peek() {
     return this._heap[0]
@@ -395,10 +395,10 @@ class LlamaTokenizer {
     function testCase(inputString, expectedTokenIds) {
       const actualTokens = tokenizer.encode(inputString, true, true, true)
       if (!isEqual(actualTokens, expectedTokenIds)) {
-        throw `Test failed. LLaMA Tokenizer Encoder returned unexpected result: expected tokenize(${inputString}) === ${expectedTokenIds}, actual was: ${actualTokens}`
+        throw new Error(`Test failed. LLaMA Tokenizer Encoder returned unexpected result: expected tokenize(${inputString}) === ${expectedTokenIds}, actual was: ${actualTokens}`)
       }
       if (inputString !== tokenizer.decode(actualTokens)) {
-        throw `Test failed. LLaMA Tokenizer Decoder returned unexpected result: expected decode(${actualTokens}) === ${inputString}, actual was: ${decode(actualTokens)}`
+        throw new Error(`Test failed. LLaMA Tokenizer Decoder returned unexpected result: expected decode(${actualTokens}) === ${inputString}, actual was: ${decode(actualTokens)}`)
       }
     }