refactor: address review comments - improve path normalization

- Keep check for './' after normalization as path.posix.normalize('./') returns './'
- Use actual Node.js path.posix implementation in tests instead of custom mock
- Apply path.posix.normalize to cleanedPrefix for consistency

All 381 code-index tests pass

Author: daniel-lxs

src/services/code-index/vector-store/__tests__/qdrant-client.spec.ts

@@ -21,22 +21,14 @@ vitest.mock("../../../../i18n", () => ({
 		return key // Just return the key for other cases
 	},
 }))
-vitest.mock("path", () => ({
-	...vitest.importActual("path"),
-	sep: "/",
-	posix: {
-		normalize: (p: string) => {
-			// Simple implementation of posix.normalize for testing
-			// Remove redundant slashes and handle . and .. segments
-			return (
-				p
-					.split("/")
-					.filter((segment) => segment !== "" && segment !== ".")
-					.join("/") || "."
-			)
-		},
-	},
-}))
+vitest.mock("path", async () => {
+	const actual = await vitest.importActual("path")
+	return {
+		...actual,
+		sep: "/",
+		posix: actual.posix,
+	}
+})
 
 const mockQdrantClientInstance = {
 	getCollection: vitest.fn(),

src/services/code-index/vector-store/qdrant-client.ts

@@ -377,15 +377,15 @@ export class QdrantVectorStore implements IVectorStore {
 			if (directoryPrefix) {
 				// Check if the path represents current directory
 				const normalizedPrefix = path.posix.normalize(directoryPrefix.replace(/\\/g, "/"))
-				if (normalizedPrefix === "." || normalizedPrefix === "" || normalizedPrefix === "./") {
+				if (normalizedPrefix === "." || normalizedPrefix === "./" || normalizedPrefix === "") {
 					// Don't create a filter - search entire workspace
 					filter = undefined
 				} else {
 					// Remove leading "./" from paths like "./src" to normalize them
 					const cleanedPrefix = normalizedPrefix.startsWith("./")
 						? normalizedPrefix.slice(2)
 						: normalizedPrefix
-					const segments = cleanedPrefix.split("/").filter(Boolean)
+					const segments = path.posix.normalize(cleanedPrefix).split("/").filter(Boolean)
 					if (segments.length > 0) {
 						filter = {
 							must: segments.map((segment, index) => ({