pi-lens 2.0.7 → 2.0.9

package/clients/ast-grep-client.test.ts

@@ -1,139 +1,169 @@
-import { describe, it, expect, beforeEach, afterEach } from "vitest";
+import { afterEach, beforeEach, describe, expect, it } from "vitest";
 import { AstGrepClient } from "./ast-grep-client.js";
 import { createTempFile, setupTestEnvironment } from "./test-utils.js";
 
 describe("AstGrepClient", () => {
-  let client: AstGrepClient;
-  let tmpDir: string;
-  let cleanup: () => void;
-
-  beforeEach(() => {
-    client = new AstGrepClient();
-    ({ tmpDir, cleanup } = setupTestEnvironment("pi-lens-astgrep-test-"));
-  });
-
-  afterEach(() => {
-    cleanup();
-  });
-
-  describe("isAvailable", () => {
-    it("should check ast-grep availability", () => {
-      const available = client.isAvailable();
-      expect(typeof available).toBe("boolean");
-    });
-  });
-
-  describe("scanFile", () => {
-    it("should return empty array for non-existent files", () => {
-      if (!client.isAvailable()) return;
-      const result = client.scanFile("/nonexistent/file.ts");
-      expect(result).toEqual([]);
-    });
-
-    it("should detect var usage (no-var rule)", () => {
-      if (!client.isAvailable()) return;
-
-      const content = `
+	let client: AstGrepClient;
+	let tmpDir: string;
+	let cleanup: () => void;
+
+	beforeEach(() => {
+		client = new AstGrepClient();
+		({ tmpDir, cleanup } = setupTestEnvironment("pi-lens-astgrep-test-"));
+	});
+
+	afterEach(() => {
+		cleanup();
+	});
+
+	describe("isAvailable", () => {
+		it("should check ast-grep availability", () => {
+			const available = client.isAvailable();
+			expect(typeof available).toBe("boolean");
+		});
+	});
+
+	describe("scanFile", () => {
+		it("should return empty array for non-existent files", () => {
+			if (!client.isAvailable()) return;
+			const result = client.scanFile("/nonexistent/file.ts");
+			expect(result).toEqual([]);
+		});
+
+		it("should detect var usage (no-var rule)", () => {
+			if (!client.isAvailable()) return;
+
+			const content = `
 var x = 1;
 var y = 2;
 `;
-      const filePath = createTempFile(tmpDir, "test.ts", content);
-      const result = client.scanFile(filePath);
+			const filePath = createTempFile(tmpDir, "test.ts", content);
+			const result = client.scanFile(filePath);
 
-      // Should detect var usage
-      expect(result.some(d => d.rule === "no-var")).toBe(true);
-    });
+			// Should detect var usage
+			expect(result.some((d) => d.rule === "no-var")).toBe(true);
+		});
 
-    it("should detect console.log usage", () => {
-      if (!client.isAvailable()) return;
+		it("should detect console.log usage", () => {
+			if (!client.isAvailable()) return;
 
-      const content = `
+			const content = `
 console.log("test");
 `;
-      const filePath = createTempFile(tmpDir, "test.ts", content);
-      const result = client.scanFile(filePath);
-
-      // May detect console.log depending on rules
-      expect(Array.isArray(result)).toBe(true);
-    });
-  });
-
-  describe("formatDiagnostics", () => {
-    it("should format diagnostics for display", () => {
-      const diags = [
-        {
-          line: 1,
-          column: 0,
-          endLine: 1,
-          endColumn: 10,
-          severity: "warning" as const,
-          message: "Unexpected var, use let or const instead",
-          rule: "no-var",
-          file: "test.ts",
-        },
-      ];
-
-      const formatted = client.formatDiagnostics(diags);
-      expect(formatted).toContain("ast-grep");
-      expect(formatted).toContain("no-var");
-    });
-
-    it("should categorize by severity", () => {
-      const diags = [
-        {
-          line: 1, column: 0, endLine: 1, endColumn: 10,
-          severity: "warning" as const, message: "Warning", rule: "rule1", file: "test.ts",
-        },
-        {
-          line: 2, column: 0, endLine: 2, endColumn: 10,
-          severity: "error" as const, message: "Error", rule: "rule2", file: "test.ts",
-        },
-      ];
-
-      const formatted = client.formatDiagnostics(diags);
-      expect(formatted).toContain("warning(s)");
-      expect(formatted).toContain("error(s)");
-    });
-
-    it("should show fixable indicator", () => {
-      const diags = [
-        {
-          line: 1, column: 0, endLine: 1, endColumn: 10,
-          severity: "warning" as const, message: "Use const", rule: "prefer-const",
-          file: "test.ts", fix: "const",
-        },
-      ];
-
-      const formatted = client.formatDiagnostics(diags);
-      expect(formatted).toContain("fixable");
-    });
-  });
-
-  describe("search", () => {
-    it("should search for patterns", async () => {
-      if (!client.isAvailable()) return;
-
-      createTempFile(tmpDir, "test.ts", `
+			const filePath = createTempFile(tmpDir, "test.ts", content);
+			const result = client.scanFile(filePath);
+
+			// May detect console.log depending on rules
+			expect(Array.isArray(result)).toBe(true);
+		});
+	});
+
+	describe("formatDiagnostics", () => {
+		it("should format diagnostics for display", () => {
+			const diags = [
+				{
+					line: 1,
+					column: 0,
+					endLine: 1,
+					endColumn: 10,
+					severity: "warning" as const,
+					message: "Unexpected var, use let or const instead",
+					rule: "no-var",
+					file: "test.ts",
+				},
+			];
+
+			const formatted = client.formatDiagnostics(diags);
+			expect(formatted).toContain("ast-grep");
+			expect(formatted).toContain("no-var");
+		});
+
+		it("should categorize by severity", () => {
+			const diags = [
+				{
+					line: 1,
+					column: 0,
+					endLine: 1,
+					endColumn: 10,
+					severity: "warning" as const,
+					message: "Warning",
+					rule: "rule1",
+					file: "test.ts",
+				},
+				{
+					line: 2,
+					column: 0,
+					endLine: 2,
+					endColumn: 10,
+					severity: "error" as const,
+					message: "Error",
+					rule: "rule2",
+					file: "test.ts",
+				},
+			];
+
+			const formatted = client.formatDiagnostics(diags);
+			expect(formatted).toContain("warning(s)");
+			expect(formatted).toContain("error(s)");
+		});
+
+		it("should show fixable indicator", () => {
+			const diags = [
+				{
+					line: 1,
+					column: 0,
+					endLine: 1,
+					endColumn: 10,
+					severity: "warning" as const,
+					message: "Use const",
+					rule: "prefer-const",
+					file: "test.ts",
+					fix: "const",
+				},
+			];
+
+			const formatted = client.formatDiagnostics(diags);
+			expect(formatted).toContain("fixable");
+		});
+	});
+
+	describe("search", () => {
+		it("should search for patterns", async () => {
+			if (!client.isAvailable()) return;
+
+			createTempFile(
+				tmpDir,
+				"test.ts",
+				`
 function test() {
   console.log("hello");
 }
-`);
+`,
+			);
 
-      const result = await client.search("console.log($MSG)", "typescript", [tmpDir]);
+			const result = await client.search("console.log($MSG)", "typescript", [
+				tmpDir,
+			]);
 
-      expect(result.matches.length).toBeGreaterThan(0);
-    });
+			expect(result.matches.length).toBeGreaterThan(0);
+		});
 
-    it("should return empty matches for no match", async () => {
-      if (!client.isAvailable()) return;
+		it("should return empty matches for no match", async () => {
+			if (!client.isAvailable()) return;
 
-      createTempFile(tmpDir, "test.ts", `
+			createTempFile(
+				tmpDir,
+				"test.ts",
+				`
 const x = 1;
-`);
+`,
+			);
 
-      const result = await client.search("console.log($MSG)", "typescript", [tmpDir]);
+			const result = await client.search("console.log($MSG)", "typescript", [
+				tmpDir,
+			]);
 
-      expect(result.matches.length).toBe(0);
-    });
-  });
+			expect(result.matches.length).toBe(0);
+		});
+	});
 });