@oh-my-pi/pi-utils 18.3.5 → 18.4.0

package/dist/types/dirs.d.ts

@@ -228,6 +228,11 @@ export declare function getGithubCacheDbPath(): string;
  * Honors `OMP_COMMIT_CACHE_DB` so tests and operators can isolate the cache.
  */
 export declare function getCommitCacheDbPath(): string;
+/**
+ * Get the judgment answer cache database path (~/.omp/cache/judgment-cache.db).
+ * Honors `OMP_JUDGMENT_CACHE_DB` so tests and operators can isolate the cache.
+ */
+export declare function getJudgmentCacheDbPath(): string;
 /** Get the legacy Pi extension parse cache database path. */
 export declare function getLegacyPiExtensionCacheDbPath(): string;
 /**

package/dist/types/postmortem.d.ts

@@ -126,13 +126,17 @@ export declare function registerStdioDisconnectHandling(): () => void;
  * `--bytecode` builds. When a one-shot command awaits a promise that never
  * settles and holds no live handle, the loop drains and Bun exits 0 without
  * output: an unfinished command reported as success. `beforeExit` fires in
- * exactly that state; explicit exits (`process.exit`, {@link quit},
+ * that state; explicit exits (`process.exit`, {@link quit},
  * {@link exitProcess}, the signal handlers) never emit it.
  *
- * The report runs once and only sets `process.exitCode`, so later `beforeExit`
- * listeners (LSP shutdown) still run before the process exits 1. It starts no
- * work of its own beyond the log write; `once` keeps the second `beforeExit`
- * that any drained follow-up work triggers from reporting again.
+ * `beforeExit` alone is not the verdict: on Windows Bun emits it while I/O is
+ * still in flight and then keeps running the loop, so a command that goes on
+ * to complete must keep its own exit code (#13470). The verdict is taken at
+ * `exit` instead, and only when the last `beforeExit` was final: an unref'd
+ * timer armed there runs only if the loop turns again, clearing the mark
+ * before a later explicit exit. Any later drain emits `beforeExit` again and
+ * re-arms it, so other `beforeExit` listeners (LSP shutdown) still run before
+ * the process exits 1.
  *
  * `describe` names what was running (e.g. the resolved subcommand) at report
  * time; it must return only a command name, never user arguments.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-utils",
-	"version": "18.3.5",
+	"version": "18.4.0",
 	"description": "Shared utilities for pi packages",
 	"keywords": [
 		"cli",
@@ -57,7 +57,7 @@
 		"fmt": "oxfmt --no-error-on-unmatched-pattern 'src/**/*.{ts,tsx}' '{test,bench,examples,scripts}/**/*.ts' '*.ts'"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-natives": "18.3.5"
+		"@oh-my-pi/pi-natives": "18.4.0"
 	},
 	"devDependencies": {
 		"@types/bun": "^1.3.14"

package/src/dirs.ts

@@ -14,6 +14,7 @@
 import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
+import { expandWindowsLongPath } from "@oh-my-pi/pi-natives/path";
 import { engines, version } from "../package.json" with { type: "json" };
 import { isEnoent, isEnotdir } from "./fs-error";
 
@@ -154,6 +155,11 @@ function standardizeMacOSPath(p: string): string {
 	return p;
 }
 
+/** Keep the current directory's spelling while expanding Windows 8.3 aliases. */
+function standardizeProjectPath(p: string): string {
+	return process.platform === "win32" ? expandWindowsLongPath(p) : standardizeMacOSPath(p);
+}
+
 export function resolveEquivalentPath(inputPath: string): string {
 	const resolvedPath = path.resolve(inputPath);
 	try {
@@ -200,29 +206,33 @@ let projectDir: string | undefined;
 /** Get the project directory. */
 export function getProjectDir(): string {
 	if (projectDir === undefined) {
+		let cwd: string | undefined;
 		try {
-			projectDir = standardizeMacOSPath(process.cwd());
+			cwd = process.cwd();
 		} catch {
 			const candidates = [process.env.PWD, os.homedir(), os.tmpdir()];
 			for (const candidate of candidates) {
 				if (!candidate || !path.isAbsolute(candidate)) continue;
 				try {
 					process.chdir(candidate);
-					projectDir = standardizeMacOSPath(candidate);
+					cwd = candidate;
 					break;
 				} catch {}
 			}
-			if (projectDir === undefined) {
+			if (cwd === undefined) {
 				throw new Error("Unable to determine an accessible working directory");
 			}
 		}
+		// Normalize outside the fallback: a native-addon failure is not an inaccessible cwd,
+		// and must surface as itself instead of relocating the process.
+		projectDir = standardizeProjectPath(cwd);
 	}
 	return projectDir;
 }
 
 /** Set the project directory. */
 export function setProjectDir(dir: string): void {
-	const resolved = standardizeMacOSPath(path.resolve(dir));
+	const resolved = standardizeProjectPath(path.resolve(dir));
 	process.chdir(resolved);
 	projectDir = resolved;
 }
@@ -787,6 +797,16 @@ export function getCommitCacheDbPath(): string {
 	return dirs.rootSubdir(path.join("cache", "commit-inference.db"), "cache");
 }
 
+/**
+ * Get the judgment answer cache database path (~/.omp/cache/judgment-cache.db).
+ * Honors `OMP_JUDGMENT_CACHE_DB` so tests and operators can isolate the cache.
+ */
+export function getJudgmentCacheDbPath(): string {
+	const override = process.env.OMP_JUDGMENT_CACHE_DB;
+	if (override) return override;
+	return dirs.rootSubdir(path.join("cache", "judgment-cache.db"), "cache");
+}
+
 /** Get the legacy Pi extension parse cache database path. */
 export function getLegacyPiExtensionCacheDbPath(): string {
 	return dirs.rootSubdir(path.join("cache", "legacy-pi-extension-cache.db"), "cache");

package/src/postmortem.ts

@@ -376,26 +376,38 @@ export function registerStdioDisconnectHandling(): () => void {
  * `--bytecode` builds. When a one-shot command awaits a promise that never
  * settles and holds no live handle, the loop drains and Bun exits 0 without
  * output: an unfinished command reported as success. `beforeExit` fires in
- * exactly that state; explicit exits (`process.exit`, {@link quit},
+ * that state; explicit exits (`process.exit`, {@link quit},
  * {@link exitProcess}, the signal handlers) never emit it.
  *
- * The report runs once and only sets `process.exitCode`, so later `beforeExit`
- * listeners (LSP shutdown) still run before the process exits 1. It starts no
- * work of its own beyond the log write; `once` keeps the second `beforeExit`
- * that any drained follow-up work triggers from reporting again.
+ * `beforeExit` alone is not the verdict: on Windows Bun emits it while I/O is
+ * still in flight and then keeps running the loop, so a command that goes on
+ * to complete must keep its own exit code (#13470). The verdict is taken at
+ * `exit` instead, and only when the last `beforeExit` was final: an unref'd
+ * timer armed there runs only if the loop turns again, clearing the mark
+ * before a later explicit exit. Any later drain emits `beforeExit` again and
+ * re-arms it, so other `beforeExit` listeners (LSP shutdown) still run before
+ * the process exits 1.
  *
  * `describe` names what was running (e.g. the resolved subcommand) at report
  * time; it must return only a command name, never user arguments.
  */
 export function reportUnsettledEntry(work: Promise<unknown>, describe?: () => string | undefined): void {
 	let pending = true;
+	let drained = false;
 	const settled = (): void => {
 		pending = false;
 	};
 	// Observe both outcomes without claiming the rejection: the caller's `.catch` still owns it.
 	void work.then(settled, settled);
-	process.once("beforeExit", () => {
+	process.on("beforeExit", () => {
 		if (!pending) return;
+		drained = true;
+		setTimeout(() => {
+			drained = false;
+		}, 0).unref();
+	});
+	process.once("exit", () => {
+		if (!pending || !drained) return;
 		const command = describe?.();
 		const subject = command ? `\`${APP_NAME} ${command}\`` : "command";
 		const message = `${subject} ended before completing: the event loop drained while it was still pending (rerun with PI_DEBUG_STARTUP=1 to see the last phase reached)`;