@isentinel/hooks 1.2.0 → 1.2.3

package/dist/hooks/lint-stop.mjs

@@ -1,13 +1,18 @@
-import { findSourceRoot, getTransitiveDependents, isLintableFile, lint, readEditedFiles, readLintAttempts, readSettings, readStopAttempts, stopDecision, writeStopAttempts } from "../scripts/lint.mjs";
+import { findSourceRoot, getTransitiveDependents, isLintableFile, lint, readEditedFiles, readLintAttempts, readSettings, readStopAttempts, restartDaemon, stopDecision, writeStopAttempts } from "../scripts/lint.mjs";
 import { n as writeStdoutJson, t as readStdinJson } from "../io.mjs";
 import { resolve } from "node:path";
 import process from "node:process";
 
 //#region hooks/lint-stop.ts
+const debugLog = [];
 const settings = readSettings();
+function debug(message) {
+	if (settings.debug) debugLog.push(message);
+}
 if (!settings.lint) process.exit(0);
 const SESSION_ID = (await readStdinJson()).session_id;
 const editedFiles = readEditedFiles(SESSION_ID);
+debug(`editedFiles=${JSON.stringify(editedFiles)}`);
 if (editedFiles.length === 0) process.exit(0);
 const dependents = /* @__PURE__ */ new Set();
 const seen = /* @__PURE__ */ new Set();
@@ -18,21 +23,36 @@ for (const file of editedFiles) {
 	for (const dependent of getTransitiveDependents(editedFiles, sourceRoot, settings.runner)) dependents.add(dependent);
 }
 const files = [...new Set([...editedFiles, ...dependents])].filter((file) => isLintableFile(file));
+debug(`lintable files: ${JSON.stringify(files)}`);
 if (files.length === 0) process.exit(0);
 const errorFiles = [];
-for (const file of files) if (lint(file, ["--fix"], settings) !== void 0) errorFiles.push(file);
+for (const file of files) {
+	debug(`linting: ${file}`);
+	const lintResult = lint(file, ["--fix"], settings, { restart: false });
+	debug(`result: ${lintResult === void 0 ? "ok" : "errors"}`);
+	if (lintResult !== void 0) errorFiles.push(file);
+}
+debug(`errorFiles: ${JSON.stringify(errorFiles)}`);
+debug("restartDaemon: start");
+restartDaemon(settings.runner);
+debug("restartDaemon: end");
 const result = stopDecision({
 	errorFiles,
 	lintAttempts: readLintAttempts(),
 	maxLintAttempts: settings.maxLintAttempts,
 	stopAttempts: readStopAttempts()
 });
-if (result === void 0) process.exit(0);
+debug(`stopDecision: ${JSON.stringify(result)}`);
+if (result === void 0) {
+	if (debugLog.length > 0) writeStdoutJson({ reason: `[lint-stop debug]\n${debugLog.join("\n")}` });
+	process.exit(0);
+}
 if (result.resetStopAttempts) {
 	writeStopAttempts(0);
 	process.exit(0);
 }
 writeStopAttempts(readStopAttempts() + 1);
+if (debugLog.length > 0) result.reason = `${result.reason ?? ""}\n\n[lint-stop debug]\n${debugLog.join("\n")}`;
 writeStdoutJson(result);
 
 //#endregion

package/dist/hooks/type-check-stop.mjs

@@ -1,15 +1,20 @@
 import { findSourceRoot, getTransitiveDependents, readEditedFiles, readLintAttempts, readSettings } from "../scripts/lint.mjs";
 import { isTypeCheckable, readTypecheckStopAttempts, resolveTsconfig, runTypeCheck, typecheckStopDecision, writeTypecheckStopAttempts } from "../scripts/type-check.mjs";
 import { n as writeStdoutJson, t as readStdinJson } from "../io.mjs";
-import { join, resolve } from "node:path";
+import { isAbsolute, join, resolve } from "node:path";
 import process from "node:process";
 
 //#region hooks/type-check-stop.ts
+const debugLog = [];
 const settings = readSettings();
+function debug(message) {
+	if (settings.debug) debugLog.push(message);
+}
 if (!settings.typecheck) process.exit(0);
 const SESSION_ID = (await readStdinJson()).session_id;
 const PROJECT_ROOT = process.env["CLAUDE_PROJECT_DIR"] ?? process.cwd();
 const editedFiles = readEditedFiles(SESSION_ID);
+debug(`editedFiles=${JSON.stringify(editedFiles)}`);
 if (editedFiles.length === 0) process.exit(0);
 const allFiles = new Set(editedFiles);
 const seenRoots = /* @__PURE__ */ new Set();
@@ -20,28 +25,40 @@ for (const file of editedFiles) {
 	for (const dependent of getTransitiveDependents(editedFiles, sourceRoot, settings.runner)) allFiles.add(dependent);
 }
 const files = [...allFiles].filter((file) => isTypeCheckable(file));
+debug(`type-checkable files: ${JSON.stringify(files)}`);
 if (files.length === 0) process.exit(0);
 const errorFiles = [];
 for (const file of files) {
-	const tsconfig = resolveTsconfig(join(PROJECT_ROOT, file), PROJECT_ROOT);
+	const absolutePath = isAbsolute(file) ? file : join(PROJECT_ROOT, file);
+	debug(`file=${file} isAbsolute=${String(isAbsolute(file))} absolutePath=${absolutePath}`);
+	const tsconfig = resolveTsconfig(absolutePath, PROJECT_ROOT);
+	debug(`tsconfig=${String(tsconfig)}`);
 	if (tsconfig === void 0) continue;
 	const output = runTypeCheck(tsconfig, settings.runner, settings.typecheckArgs);
 	if (output !== void 0) {
-		if (/error TS/i.test(output)) errorFiles.push(file);
+		const hasErrors = /error TS/i.test(output);
+		debug(`typecheck ${file}: ${hasErrors ? "errors" : "ok"}`);
+		if (hasErrors) errorFiles.push(file);
 	}
 }
+debug(`errorFiles: ${JSON.stringify(errorFiles)}`);
 const result = typecheckStopDecision({
 	errorFiles,
 	lintAttempts: readLintAttempts(),
 	maxLintAttempts: settings.maxLintAttempts,
 	stopAttempts: readTypecheckStopAttempts()
 });
-if (result === void 0) process.exit(0);
+debug(`stopDecision: ${JSON.stringify(result)}`);
+if (result === void 0) {
+	if (debugLog.length > 0) writeStdoutJson({ reason: `[type-check-stop debug]\n${debugLog.join("\n")}` });
+	process.exit(0);
+}
 if (result.resetStopAttempts) {
 	writeTypecheckStopAttempts(0);
 	process.exit(0);
 }
 writeTypecheckStopAttempts(readTypecheckStopAttempts() + 1);
+if (debugLog.length > 0) result.reason = `${result.reason ?? ""}\n\n[type-check-stop debug]\n${debugLog.join("\n")}`;
 writeStdoutJson(result);
 
 //#endregion

package/dist/scripts/lint.d.mts

@@ -3,6 +3,7 @@ import { t as PostToolUseHookOutput } from "../events.mjs";
 //#region scripts/lint.d.ts
 interface LintSettings {
   cacheBust: Array<string>;
+  debug: boolean;
   eslint: boolean;
   lint: boolean;
   maxLintAttempts: number;
@@ -49,10 +50,14 @@ declare function clearCache(): void;
 declare function invalidateCacheEntries(filePaths: Array<string>): void;
 declare function runOxlint(filePath: string, extraFlags?: Array<string>, runner?: string): string | undefined;
 declare function runEslint(filePath: string, extraFlags?: Array<string>, runner?: string): string | undefined;
-declare function restartDaemon(runner?: string): void;
+declare function restartDaemon(runner?: string, warmupFile?: string): void;
 declare function formatErrors(output: string): Array<string>;
-declare function buildHookOutput(filePath: string, errors: Array<string>): PostToolUseHookOutput;
-declare function lint(filePath: string, extraFlags?: Array<string>, settings?: LintSettings): PostToolUseHookOutput | undefined;
+declare function buildHookOutput(filePath: string, errors: Array<string>, debugInfo?: string): PostToolUseHookOutput;
+declare function lint(filePath: string, extraFlags?: Array<string>, settings?: LintSettings, {
+  restart
+}?: {
+  restart?: boolean | undefined;
+}): PostToolUseHookOutput | undefined;
 declare function main(targets: Array<string>, settings?: LintSettings): void;
 //#endregion
 export { DEFAULT_CACHE_BUST, DependencyGraph, LintSettings, StopDecisionResult, buildHookOutput, clearCache, clearEditedFiles, clearLintAttempts, clearStopAttempts, findEntryPoints, findSourceRoot, formatErrors, getChangedFiles, getDependencyGraph, getTransitiveDependents, invalidateCacheEntries, invertGraph, isLintableFile, isProtectedFile, lint, main, readEditedFiles, readLintAttempts, readSettings, readStopAttempts, resolveBustFiles, restartDaemon, runEslint, runOxlint, shouldBustCache, stopDecision, writeEditedFile, writeLintAttempts, writeStopAttempts };

package/dist/scripts/lint.mjs

@@ -1,10 +1,33 @@
 import { createFromFile } from "file-entry-cache";
-import { execSync, spawn } from "node:child_process";
+import { execFileSync, execSync, spawn, spawnSync } from "node:child_process";
 import { existsSync, globSync, mkdirSync, readFileSync, statSync, unlinkSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
 import { dirname, join, relative, resolve } from "node:path";
 import process from "node:process";
 
 //#region scripts/lint.ts
+function spawnBackground(script, extraEnvironment = {}) {
+	const scriptFile = join(tmpdir(), `.eslint_bg_${Date.now()}_${Math.random().toString(36).slice(2, 8)}.cjs`);
+	writeFileSync(scriptFile, script);
+	const environment = {
+		...process.env,
+		...extraEnvironment
+	};
+	if (process.platform === "win32") spawnSync("powershell.exe", [
+		"-NoProfile",
+		"-Command",
+		`Start-Process -FilePath 'node' -ArgumentList '${scriptFile}' -WindowStyle Hidden`
+	], {
+		env: environment,
+		stdio: "ignore",
+		windowsHide: true
+	});
+	else spawn("node", [scriptFile], {
+		detached: true,
+		env: environment,
+		stdio: "ignore"
+	}).unref();
+}
 const PROTECTED_PATTERNS = [
 	"eslint.config.",
 	"oxlint.config.",
@@ -17,6 +40,9 @@ function isProtectedFile(filename) {
 const LINT_STATE_PATH = ".claude/state/lint-attempts.json";
 const STOP_STATE_PATH = ".claude/state/stop-attempts.json";
 const EDITED_FILES_PATH = ".claude/state/edited-files.json";
+const RESTART_DAEMON_LOG = ".claude/state/restartDaemon.log";
+const IS_RESTART_DAEMON_DEBUG = false;
+const CLAUDE_PID_PATH = ".claude/state/claude-pid";
 function readEditedFiles(sessionId) {
 	if (!existsSync(EDITED_FILES_PATH)) return [];
 	try {
@@ -74,6 +100,43 @@ function getTransitiveDependents(files, sourceRoot, runner = DEFAULT_SETTINGS.ru
 	const originals = new Set(files.map((file) => relative(sourceRoot, resolve(file)).replaceAll("\\", "/")));
 	return [...visited].filter((file) => !originals.has(file)).map((file) => join(sourceRoot, file));
 }
+function getClaudePid() {
+	const ssePort = process.env["CLAUDE_CODE_SSE_PORT"] ?? "";
+	const cacheFile = ssePort.length > 0 ? `${CLAUDE_PID_PATH}-${ssePort}` : CLAUDE_PID_PATH;
+	if (existsSync(cacheFile)) try {
+		const cached = readFileSync(cacheFile, "utf-8").trim();
+		const pid = Number(cached);
+		process.kill(pid, 0);
+		return cached;
+	} catch {}
+	if (process.platform === "win32") try {
+		const result = execFileSync("powershell.exe", [
+			"-NoProfile",
+			"-Command",
+			`
+$currentPid = ${process.ppid}
+while ($currentPid -and $currentPid -ne 0) {
+  $p = Get-CimInstance Win32_Process -Filter "ProcessId=$currentPid" -Property Name,ParentProcessId -ErrorAction SilentlyContinue
+  if (-not $p) { break }
+  if ($p.Name -eq 'claude.exe') { Write-Output $currentPid; exit }
+  $currentPid = $p.ParentProcessId
+}`
+		], {
+			encoding: "utf-8",
+			stdio: [
+				"pipe",
+				"pipe",
+				"pipe"
+			],
+			timeout: 5e3
+		}).trim();
+		if (result.length > 0) {
+			mkdirSync(dirname(cacheFile), { recursive: true });
+			writeFileSync(cacheFile, result);
+			return result;
+		}
+	} catch {}
+}
 const ESLINT_CACHE_PATH = ".eslintcache";
 const DEFAULT_EXTENSIONS = [
 	".ts",
@@ -96,6 +159,7 @@ const DEFAULT_MAX_LINT_ATTEMPTS = 1;
 const DEFAULT_MAX_STOP_ATTEMPTS = 1;
 const DEFAULT_SETTINGS = {
 	cacheBust: [...DEFAULT_CACHE_BUST],
+	debug: false,
 	eslint: true,
 	lint: true,
 	maxLintAttempts: DEFAULT_MAX_LINT_ATTEMPTS,
@@ -113,6 +177,7 @@ function readSettings() {
 	const maxLintAttempts = maxAttemptsRaw !== void 0 ? Number(maxAttemptsRaw) : DEFAULT_MAX_LINT_ATTEMPTS;
 	return {
 		cacheBust: [...DEFAULT_CACHE_BUST, ...userPatterns],
+		debug: fields.get("debug") === "true",
 		eslint: fields.get("eslint") !== "false",
 		lint: fields.get("lint") !== "false",
 		maxLintAttempts,
@@ -258,9 +323,11 @@ function runOxlint(filePath, extraFlags = [], runner = DEFAULT_SETTINGS.runner)
 function runEslint(filePath, extraFlags = [], runner = DEFAULT_SETTINGS.runner) {
 	const flags = ["--cache", ...extraFlags].join(" ");
 	try {
+		const claudePid = getClaudePid();
 		execSync(`${runner} eslint_d ${flags} "${filePath}"`, {
 			env: {
 				...process.env,
+				...claudePid !== void 0 && { ESLINT_D_PPID: claudePid },
 				ESLINT_IN_EDITOR: "true"
 			},
 			stdio: "pipe"
@@ -274,45 +341,91 @@ function runEslint(filePath, extraFlags = [], runner = DEFAULT_SETTINGS.runner)
 		return stdout || stderr || message;
 	}
 }
-function restartDaemon(runner = DEFAULT_SETTINGS.runner) {
-	const [command = "pnpm", ...prefixArgs] = runner.split(/\s+/);
-	const child = spawn(command, [
-		...prefixArgs,
-		"eslint_d",
-		"restart"
-	], {
-		detached: true,
-		env: {
-			...process.env,
+function restartDaemon(runner = DEFAULT_SETTINGS.runner, warmupFile) {
+	const markerFile = void 0;
+	try {
+		const restartAndWarmupScript = `
+const { execSync } = require("child_process");
+const fs = require("fs");
+
+const runner = ${JSON.stringify(runner)};
+const warmupFile = ${JSON.stringify(warmupFile)};
+const debug = ${IS_RESTART_DAEMON_DEBUG};
+const logFile = ${JSON.stringify(RESTART_DAEMON_LOG)};
+
+try {
+  // Run restart
+  if (debug) {
+    fs.appendFileSync(logFile, \`restart: start \${new Date().toISOString()}\\n\`);
+  }
+  execSync(\`\${runner} eslint_d restart\`, {
+    env: { ...process.env, ESLINT_D_PPID: process.env.ESLINT_D_PPID, ESLINT_IN_EDITOR: "true" },
+    stdio: "pipe",
+  });
+  if (debug) {
+    fs.appendFileSync(logFile, \`restart: end \${new Date().toISOString()}\\n\`);
+  }
+
+  // Run warmup lint if file provided
+  if (warmupFile) {
+    if (debug) {
+      fs.appendFileSync(logFile, \`warmup: start \${warmupFile} \${new Date().toISOString()}\\n\`);
+    }
+    execSync(\`\${runner} eslint_d "\${warmupFile}"\`, {
+      env: { ...process.env, ESLINT_D_PPID: process.env.ESLINT_D_PPID, ESLINT_IN_EDITOR: "true" },
+      stdio: "pipe",
+    });
+    if (debug) {
+      fs.appendFileSync(logFile, \`warmup: end \${new Date().toISOString()}\\n\`);
+    }
+  }
+} catch (err) {
+  if (debug) {
+    fs.appendFileSync(logFile, \`error: \${err.message} \${new Date().toISOString()}\\n\`);
+  }
+} finally {
+  try { fs.unlinkSync(process.argv[1]); } catch {}
+}
+`;
+		const claudePid = getClaudePid();
+		spawnBackground(restartAndWarmupScript, {
+			...claudePid !== void 0 && { ESLINT_D_PPID: claudePid },
 			ESLINT_IN_EDITOR: "true"
-		},
-		stdio: "pipe"
-	});
-	child.stderr.on("data", (data) => {
-		process.stderr.write(`[eslint_d restart] ${data.toString()}`);
-	});
-	child.on("error", (error) => {
-		process.stderr.write(`[eslint_d restart] failed: ${error.message}\n`);
-	});
-	child.unref();
+		});
+		if (markerFile !== void 0) spawnBackground(`
+const fs = require("fs");
+const logFile = ${JSON.stringify(RESTART_DAEMON_LOG)};
+
+setTimeout(() => {
+  try {
+    fs.appendFileSync(logFile, \`daemon exit: \${new Date().toISOString()}\\n\`);
+  } catch {}
+  try { fs.unlinkSync(process.argv[1]); } catch {}
+  process.exit(0);
+}, 10000);
+`);
+	} catch (err) {
+		const message = err instanceof Error ? err.message : String(err);
+		process.stderr.write(`[eslint_d restart] ${message}\n`);
+	}
 }
 function formatErrors(output) {
 	return output.split("\n").filter((line) => /error/i.test(line)).slice(0, MAX_ERRORS);
 }
-function buildHookOutput(filePath, errors) {
+function buildHookOutput(filePath, errors, debugInfo = "") {
 	const errorText = errors.join("\n");
 	const isTruncated = errors.length >= MAX_ERRORS;
-	const userMessage = `⚠️ Lint errors in ${filePath}:\n${errorText}${isTruncated ? "\n..." : ""}`;
+	const userMessage = `⚠️ Lint errors in ${filePath}:\n${errorText}${isTruncated ? "\n..." : ""}${debugInfo}`;
 	return {
 		decision: void 0,
 		hookSpecificOutput: {
-			additionalContext: `⚠️ Lint errors in ${filePath}:\n${errorText}${isTruncated ? "\n(run lint to view more)" : ""}`,
+			additionalContext: `⚠️ Lint errors in ${filePath}:\n${errorText}${isTruncated ? "\n(run lint to view more)" : ""}${debugInfo}`,
 			hookEventName: "PostToolUse"
 		},
 		systemMessage: userMessage
 	};
 }
-function lint(filePath, extraFlags = [], settings = DEFAULT_SETTINGS) {
+function lint(filePath, extraFlags = [], settings = DEFAULT_SETTINGS, { restart = true } = {}) {
 	if (shouldBustCache(settings.cacheBust)) clearCache();
 	else invalidateCacheEntries(findImporters(filePath, settings.runner));
 	const outputs = [];
@@ -324,10 +437,15 @@ function lint(filePath, extraFlags = [], settings = DEFAULT_SETTINGS) {
 		const output = runEslint(filePath, extraFlags, settings.runner);
 		if (output !== void 0) outputs.push(output);
 	}
-	if (settings.eslint) restartDaemon(settings.runner);
+	let debugInfo = "";
+	if (settings.eslint && restart) {
+		const startTime = Date.now();
+		restartDaemon(settings.runner, filePath);
+		Date.now() - startTime;
+	}
 	if (outputs.length > 0) {
 		const errors = formatErrors(outputs.join("\n"));
-		if (errors.length > 0) return buildHookOutput(filePath, errors);
+		if (errors.length > 0) return buildHookOutput(filePath, errors, debugInfo);
 	}
 }
 function main(targets, settings = DEFAULT_SETTINGS) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@isentinel/hooks",
-  "version": "1.2.0",
+  "version": "1.2.3",
   "description": "Claude Code hooks for linting and type-checking TypeScript projects",
   "keywords": [
     "claude",
@@ -42,12 +42,13 @@
     "*.{ts,mts}": "eslint --fix"
   },
   "dependencies": {
-    "eslint_d": "^14.3.0",
+    "eslint_d": "^15.0.0",
     "file-entry-cache": "^11.1.2",
     "get-tsconfig": "^4.13.6",
     "madge": "^8.0.0"
   },
   "devDependencies": {
+    "@antfu/ni": "^29.0.0",
     "@clack/prompts": "^0.11.0",
     "@constellos/claude-code-kit": "^0.4.0",
     "@isentinel/eslint-config": "5.0.0-beta.8",
@@ -81,6 +82,9 @@
   "engines": {
     "node": ">=24.11.0"
   },
+  "publishConfig": {
+    "provenance": true
+  },
   "scripts": {
     "build": "tsdown",
     "lint": "eslint --cache",