chore(controller): fix lint errors

Author: claytercek

packages/controller/src/ipc/__tests__/ipc-client.test.ts

@@ -85,7 +85,10 @@ describe("IPCClient", () => {
 			const result = await client.connect("/test/socket");
 
 			expect(result.isOk()).toBe(true);
-			expect(net.createConnection).toHaveBeenCalledWith(getOSSocketPath("/test/socket"), expect.any(Function));
+			expect(net.createConnection).toHaveBeenCalledWith(
+				getOSSocketPath("/test/socket"),
+				expect.any(Function),
+			);
 		});
 
 		it("should return error if connection fails", async () => {

packages/controller/src/ipc/__tests__/ipc-utils.test.ts

@@ -2,26 +2,26 @@ import { describe, expect, it } from "vitest";
 import { getOSSocketPath } from "../ipc-utils.js";
 
 describe("getOSSocketPath", () => {
-  describe.runIf(process.platform !== "win32")("on posix", () => {
-    it("returns the same path", () => {
-      const inputPath = "/tmp/socket.sock";
-      const result = getOSSocketPath(inputPath);
-      expect(result).toBe(inputPath);
-    });
-  });
+	describe.runIf(process.platform !== "win32")("on posix", () => {
+		it("returns the same path", () => {
+			const inputPath = "/tmp/socket.sock";
+			const result = getOSSocketPath(inputPath);
+			expect(result).toBe(inputPath);
+		});
+	});
 
-  describe.runIf(process.platform === "win32")("on Windows", () => {
-    it('updates the path to use named pipe format if not already in that format', () => {
-      const inputPath = "C:\\temp\\socket.sock";
-      const expectedPath = "\\\\?\\pipe\\C:\\temp\\socket.sock";
-      const result = getOSSocketPath(inputPath);
-      expect(result).toBe(expectedPath);
-    });
+	describe.runIf(process.platform === "win32")("on Windows", () => {
+		it("updates the path to use named pipe format if not already in that format", () => {
+			const inputPath = "C:\\temp\\socket.sock";
+			const expectedPath = "\\\\?\\pipe\\C:\\temp\\socket.sock";
+			const result = getOSSocketPath(inputPath);
+			expect(result).toBe(expectedPath);
+		});
 
-    it('returns the same path if already in named pipe format with \\\\?\\pipe prefix', () => {
-      const inputPath = "\\\\?\\pipe\\my_named_pipe";
-      const result = getOSSocketPath(inputPath);
-      expect(result).toBe(inputPath);
-    });
-  });
-})
+		it("returns the same path if already in named pipe format with \\\\?\\pipe prefix", () => {
+			const inputPath = "\\\\?\\pipe\\my_named_pipe";
+			const result = getOSSocketPath(inputPath);
+			expect(result).toBe(inputPath);
+		});
+	});
+});

packages/controller/src/ipc/ipc-client.ts

@@ -232,7 +232,7 @@ export class IPCClient {
 		try {
 			this._lastState = applyPatches(this._lastState, patches);
 			this._lastStateVersion = version;
-		} catch (e) {
+		} catch (_e) {
 			return errAsync(new IPCMessageError("Failed to apply patches"));
 		}
 

packages/controller/src/ipc/ipc-utils.ts

@@ -5,17 +5,14 @@ import path from "node:path";
  * https://nodejs.org/api/net.html#identifying-paths-for-ipc-connections
  */
 export function getOSSocketPath(socketPath: string) {
-  if (process.platform === "win32") {
-    // On Windows, we need to use a named pipe
+	if (process.platform === "win32") {
+		// On Windows, we need to use a named pipe
 
-    if (socketPath.includes("\\\\?\\pipe") || socketPath.includes("\\\\.\\pipe")) {
-      // Already in correct format
-      return socketPath;
-    }
-    return path.join(
-      '\\\\?\\pipe',
-      socketPath
-    )
-  }
-  return socketPath;
-}
+		if (socketPath.includes("\\\\?\\pipe") || socketPath.includes("\\\\.\\pipe")) {
+			// Already in correct format
+			return socketPath;
+		}
+		return path.join("\\\\?\\pipe", socketPath);
+	}
+	return socketPath;
+}

packages/controller/src/transports/ipc-transport.ts

@@ -7,11 +7,11 @@ import fs from "node:fs";
 import net from "node:net";
 import path from "node:path";
 import type { LaunchpadEvents } from "@bluecadet/launchpad-utils";
+import chalk from "chalk";
 import type { Patch } from "immer";
 import { ResultAsync } from "neverthrow";
 import type { VersionedLaunchpadState } from "../core/state-store.js";
 import type { Transport, TransportContext } from "../core/transport.js";
-import chalk from 'chalk'
 import {
 	CommandExecutionError,
 	IPCMessageError,
@@ -64,16 +64,16 @@ export function createIPCTransport(options: IPCTransportOptions): Transport {
 		start(ctx: TransportContext): ResultAsync<void, TransportError> {
 			const { logger, abortSignal } = ctx;
 
-			if (process.platform == "win32" && (options.socketPath !== socketPath)) {
+			if (process.platform === "win32" && options.socketPath !== socketPath) {
 				// notify user that the socket path has been updated to conform with windows named pipe reqs,
 				// as it might not be where they expect it
 
 				logger.warn(
-					`Windows named pipes must be located in ${chalk.grey("\\\\?\\pipe\\")} or ${chalk.grey("\\\\.\\pipe\\")}. `
-				)
+					`Windows named pipes must be located in ${chalk.grey("\\\\?\\pipe\\")} or ${chalk.grey("\\\\.\\pipe\\")}. `,
+				);
 				logger.warn(
-					`The configured socketPath has been moved to the ${chalk.grey("\\\\?\\pipe\\")} directory to conform with this requirement.`
-				)
+					`The configured socketPath has been moved to the ${chalk.grey("\\\\?\\pipe\\")} directory to conform with this requirement.`,
+				);
 			}
 
 			const promise = new Promise<void>((resolve, reject) => {
@@ -385,4 +385,4 @@ function sendError(socket: net.Socket, id: string, error: Error): void {
 		type: "error",
 		error: error,
 	});
-}
+}