From 4324e79ae962fde0a649d1d7e0929abfa6363860 Mon Sep 17 00:00:00 2001
From: "deepsource-autofix[bot]"
 <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Date: Thu, 14 May 2026 13:07:36 +0000
Subject: [PATCH] refactor: remove unused variables in TypeScript code
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This PR removes unused variables and parameters across tests and service handlers to improve code cleanliness and satisfy linter rules.

- Found unused variables in TypeScript code: various unused imports (`afterEach`, `embeddingService`) and promise assignments from `service.warmup` were removed; the mock implementation callback signature was simplified by dropping unused arguments; the optional `userId` parameter in `handleRefreshProfile` was prefixed with an underscore to denote it’s intentionally unused; and string replacement logic was corrected to properly escape backslashes.

> This Autofix was generated by AI. Please review the change before merging.
---
 src/services/api-handlers.ts                    | 3 +--
 src/services/user-prompt/user-prompt-manager.ts | 2 +-
 tests/embedding-internals.test.ts               | 4 ++--
 tests/memory-conflicts.test.ts                  | 5 ++---
 4 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/src/services/api-handlers.ts b/src/services/api-handlers.ts
index 5055384..6ec89cf 100644
--- a/src/services/api-handlers.ts
+++ b/src/services/api-handlers.ts
@@ -1050,9 +1050,8 @@ export async function handleGetProfileSnapshot(changelogId: string): Promise<Api
   }
 }
 
-export async function handleRefreshProfile(userId?: string): Promise<ApiResponse<any>> {
+export async function handleRefreshProfile(_userId?: string): Promise<ApiResponse<any>> {
   try {
-    const { getTags } = await import("./tags.js");
     const { userPromptManager } = await import("./user-prompt/user-prompt-manager.js");
     const unanalyzedCount = userPromptManager.countUnanalyzedForUserLearning();
     return {
diff --git a/src/services/user-prompt/user-prompt-manager.ts b/src/services/user-prompt/user-prompt-manager.ts
index ec40f60..e4c3fea 100644
--- a/src/services/user-prompt/user-prompt-manager.ts
+++ b/src/services/user-prompt/user-prompt-manager.ts
@@ -9,7 +9,7 @@ type DatabaseType = Database;
 const USER_PROMPTS_DB_NAME = "user-prompts.db";
 
 function escapeLikePattern(value: string): string {
-  return value.replace(/[\\%_]/g, (char) => String.raw`\${char}`);
+  return value.replace(/[\\%_]/g, (char) => String.raw`\\${char}`);
 }
 
 export interface UserPrompt {
diff --git a/tests/embedding-internals.test.ts b/tests/embedding-internals.test.ts
index 75be72e..bb0de09 100644
--- a/tests/embedding-internals.test.ts
+++ b/tests/embedding-internals.test.ts
@@ -70,9 +70,9 @@ describe("EmbeddingService singleton and internals", () => {
 
     it("reuses in-progress initPromise on second warmup call", () => {
       // First call sets initPromise
-      const promise1 = service.warmup();
+      service.warmup();
       // Second call returns same promise
-      const promise2 = service.warmup();
+      service.warmup();
       expect((service as any).initPromise).not.toBeNull();
       // Clean up
       (service as any).initPromise = null;
diff --git a/tests/memory-conflicts.test.ts b/tests/memory-conflicts.test.ts
index b4502bf..e08caa8 100644
--- a/tests/memory-conflicts.test.ts
+++ b/tests/memory-conflicts.test.ts
@@ -1,4 +1,4 @@
-import { afterEach, describe, it, expect, vi, beforeEach } from "vitest";
+import { describe, it, expect, vi, beforeEach } from "vitest";
 
 vi.mock("../src/services/sqlite/shard-manager.js", () => ({
   shardManager: {
@@ -65,7 +65,6 @@ import {
 import { shardManager } from "../src/services/sqlite/shard-manager.js";
 import { connectionManager } from "../src/services/sqlite/connection-manager.js";
 import { vectorSearch } from "../src/services/sqlite/vector-search.js";
-import { embeddingService } from "../src/services/embedding.js";
 
 function makeMockDb(
   options: {
@@ -96,7 +95,7 @@ function makeMockDb(
         };
       }
       return {
-        all: vi.fn().mockImplementation((...args: any[]) => {
+        all: vi.fn().mockImplementation(() => {
           if (sql.includes("memories_fts MATCH")) return ftsMemories;
           if (sql.includes("content LIKE") || sql.includes("AND id !=")) return likeMemories;
           if (sql.includes("memory_conflicts") && sql.includes("LEFT JOIN")) return conflictRows;