feat: add EEXIST and EISDIR error handling to getErrorMessage

Signed-off-by: leocavalcante <leo@cavalcante.dev>

Author: leocavalcante

src/paths.mjs

@@ -73,6 +73,10 @@ export function getErrorMessage(error, file, targetPath) {
 		case "EMFILE":
 		case "ENFILE":
 			return "Too many open files. Close some applications and try again"
+		case "EEXIST":
+			return `Target already exists: ${targetPath}`
+		case "EISDIR":
+			return `Expected a file but found a directory: ${targetPath}`
 		default:
 			return error.message || "Unknown error"
 	}

tests/paths.test.ts

@@ -141,6 +141,18 @@ describe("paths.mjs exports", () => {
 			expect(result).toBe("Too many open files. Close some applications and try again")
 		})
 
+		it("should return target exists message for EEXIST error", () => {
+			const error = Object.assign(new Error("EEXIST"), { code: "EEXIST" })
+			const result = getErrorMessage(error, testFile, testTargetPath)
+			expect(result).toBe(`Target already exists: ${testTargetPath}`)
+		})
+
+		it("should return is directory message for EISDIR error", () => {
+			const error = Object.assign(new Error("EISDIR"), { code: "EISDIR" })
+			const result = getErrorMessage(error, testFile, testTargetPath)
+			expect(result).toBe(`Expected a file but found a directory: ${testTargetPath}`)
+		})
+
 		it("should return error message for unknown error codes", () => {
 			const error = Object.assign(new Error("Something went wrong"), { code: "UNKNOWN" })
 			const result = getErrorMessage(error, testFile, testTargetPath)

tests/plugin.test.ts

@@ -1,4 +1,5 @@
 import { describe, expect, it } from "bun:test"
+import type { PluginInput } from "@opencode-ai/plugin"
 import { OpenCoderPlugin } from "../src/plugin"
 
 describe("OpenCoderPlugin", () => {
@@ -7,17 +8,17 @@ describe("OpenCoderPlugin", () => {
 	})
 
 	it("should return a hooks object when called", async () => {
-		// Create a mock context (minimal implementation)
+		// Create a mock context (minimal implementation for testing)
 		const mockContext = {
 			project: {},
 			client: {},
 			$: () => {},
 			directory: "/tmp",
 			worktree: "/tmp",
 			serverUrl: new URL("http://localhost:3000"),
-		}
+		} as unknown as PluginInput
 
-		const result = await OpenCoderPlugin(mockContext as Parameters<typeof OpenCoderPlugin>[0])
+		const result = await OpenCoderPlugin(mockContext)
 
 		expect(result).toBeDefined()
 		expect(typeof result).toBe("object")
@@ -31,9 +32,9 @@ describe("OpenCoderPlugin", () => {
 			directory: "/tmp",
 			worktree: "/tmp",
 			serverUrl: new URL("http://localhost:3000"),
-		}
+		} as unknown as PluginInput
 
-		const result = await OpenCoderPlugin(mockContext as Parameters<typeof OpenCoderPlugin>[0])
+		const result = await OpenCoderPlugin(mockContext)
 
 		expect(Object.keys(result)).toHaveLength(0)
 	})