From 758be304747974f84418f66efa4353362ac1edcf Mon Sep 17 00:00:00 2001 From: amankumarsingh77 Date: Tue, 28 Jul 2026 20:01:05 +0530 Subject: style: drop the comments added by the bug-audit fixes The audit fixes were required to add no comments; two doc blocks slipped through review. Removed, leaving only comments that predate this task. Co-Authored-By: Claude Opus 5 (1M context) --- src/daemon/spawn.ts | 10 ---------- src/domains/fill-engine.ts | 15 --------------- 2 files changed, 25 deletions(-) (limited to 'src') diff --git a/src/daemon/spawn.ts b/src/daemon/spawn.ts index 165abcd..63c0dab 100644 --- a/src/daemon/spawn.ts +++ b/src/daemon/spawn.ts @@ -32,16 +32,6 @@ const cleanupStaleSocket = (): void => { unlink(DAEMON_SOCKET_PATH).catch(() => {}); }; -/** - * Whether the daemon endpoint could plausibly exist, judged without connecting. - * - * On POSIX a missing socket file means the daemon definitely isn't running, so - * callers can short-circuit. Windows named pipes are NOT filesystem entries — - * access() always fails on `\\.\pipe\…` even while the daemon is serving it — so - * on Windows the answer is always "maybe" and the caller must probe instead. - * - * Platform and path are parameters so the branch is testable off-platform. - */ export const daemonSocketMayExist = async ( platform: NodeJS.Platform = process.platform, socketPath: string = DAEMON_SOCKET_PATH, diff --git a/src/domains/fill-engine.ts b/src/domains/fill-engine.ts index 32102fe..8fdf239 100644 --- a/src/domains/fill-engine.ts +++ b/src/domains/fill-engine.ts @@ -1,18 +1,3 @@ -/** - * The one in-page fill implementation, shared by browser_fill and - * browser_fill_form so the two can no longer disagree about the same element. - * - * `this` is the target element and `value` is the requested value. Two call-site - * differences are parameters rather than forks: - * rejectSelect — browser_fill refuses a