@oh-my-pi/pi-utils 15.9.0 → 15.9.1

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-utils",
-	"version": "15.9.0",
+	"version": "15.9.1",
 	"description": "Shared utilities for pi packages",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -31,7 +31,7 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-natives": "15.9.0",
+		"@oh-my-pi/pi-natives": "15.9.1",
 		"beautiful-mermaid": "^1.1.3",
 		"handlebars": "^4.7.9",
 		"winston": "^3.19.0",

package/src/tab-spacing.ts

@@ -4,17 +4,25 @@
  */
 import * as fs from "node:fs";
 import * as path from "node:path";
-import { isEnoent } from "./fs-error";
+import { isFsError } from "./fs-error";
 
 export const MIN_TAB_WIDTH = 1;
 export const MAX_TAB_WIDTH = 16;
 export const DEFAULT_TAB_WIDTH = 3;
 
+/**
+ * Per-component path length cap on common filesystems (`NAME_MAX = 255` on
+ * Linux ext4 / macOS APFS / Windows NTFS). Paths with components longer than
+ * this cannot be opened at all, so editorconfig discovery short-circuits to
+ * the default instead of running into `ENAMETOOLONG` from `readFileSync`.
+ */
+const NAME_MAX_BYTES = 255;
+
 const EDITORCONFIG_NAME = ".editorconfig";
 
 let defaultTabWidth = DEFAULT_TAB_WIDTH;
 
-const editorConfigCache = new Map<string, ParsedEditorConfig>();
+const editorConfigCache = new Map<string, ParsedEditorConfig | null>();
 const editorConfigChainCache = new Map<string, ChainEntry[]>();
 const indentationCache = new Map<string, number>();
 
@@ -145,14 +153,18 @@ function parseCachedEditorConfig(configPath: string): ParsedEditorConfig | undef
 	const key = path.resolve(configPath);
 	const hit = editorConfigCache.get(key);
 	if (hit !== undefined) {
-		return hit;
+		return hit ?? undefined;
 	}
 
 	let content: string;
 	try {
 		content = fs.readFileSync(key, "utf8");
 	} catch (err) {
-		if (isEnoent(err)) return undefined;
+		// editorconfig discovery is best-effort. Any filesystem error
+		// (`ENOENT`, `ENAMETOOLONG`, `ENOTDIR`, `EACCES`, `ELOOP`, `EINVAL`,
+		// …) means "no usable config at this path" — never a fatal condition
+		// for callers like the edit renderer that hand us arbitrary strings.
+		if (isFsError(err)) return undefined;
 		throw err;
 	}
 	const parsed = parseEditorConfigFile(content);
@@ -279,6 +291,15 @@ function resolveEditorConfigTabWidth(match: EditorConfigMatch | undefined, fallb
 	return undefined;
 }
 
+function hasOverlongPathComponent(filePath: string): boolean {
+	for (const part of filePath.split(/[\\/]/)) {
+		if (part.length > 0 && Buffer.byteLength(part) > NAME_MAX_BYTES) {
+			return true;
+		}
+	}
+	return false;
+}
+
 export function getDefaultTabWidth(): number {
 	return defaultTabWidth;
 }
@@ -298,6 +319,15 @@ export function getIndentation(file?: string | null, projectDir?: string | null)
 
 	const cwd = projectDir ?? process.cwd();
 	const absoluteFile = resolveFilePath(cwd, file);
+
+	// Renderers can hand us arbitrary strings (e.g. a malformed edit tool
+	// call whose `file_path` is gibberish). Reject paths whose normalized
+	// absolute form still has any component longer than `NAME_MAX_BYTES` —
+	// the editorconfig chain would only trip `ENAMETOOLONG` from
+	// `readFileSync` and escape.
+	if (hasOverlongPathComponent(absoluteFile)) {
+		return fallback;
+	}
 	const absKey = absoluteFile;
 	const cached = indentationCache.get(absKey);
 	if (cached !== undefined) {