@oh-my-pi/hashline 16.2.7 → 16.2.8

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [16.2.8] - 2026-06-30
+
+### Fixed
+
+- Fixed hashline writes preserving UTF-8 BOM bytes when the host text decoder hides the leading `U+FEFF`. ([#3867](https://github.com/can1357/oh-my-pi/issues/3867))
+
 ## [16.2.6] - 2026-06-29
 
 ### Fixed

package/dist/types/fs.d.ts

@@ -37,6 +37,8 @@ export declare function isNotFound(error: unknown): boolean;
 export declare abstract class Filesystem {
     /** Read the file's full text content. Throw on missing file. */
     abstract readText(path: string): Promise<string>;
+    /** Read raw bytes for backends whose text is a direct decode of persisted bytes. */
+    readBinary?(path: string): Promise<Uint8Array | undefined>;
     /** Validate that `path` is writable before a prepared batch starts committing. */
     preflightWrite(_path: string, _options?: PreflightWriteOptions): Promise<void>;
     /** Persist `content` at `path`. Returns the actual final text that was written. */
@@ -98,6 +100,7 @@ export declare class InMemoryFilesystem extends Filesystem {
  */
 export declare class NodeFilesystem extends Filesystem {
     readText(path: string): Promise<string>;
+    readBinary(path: string): Promise<Uint8Array>;
     writeText(path: string, content: string): Promise<WriteResult>;
     delete(path: string): Promise<void>;
     move(from: string, to: string, content?: string): Promise<void>;

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/hashline",
-	"version": "16.2.7",
+	"version": "16.2.8",
 	"description": "Hashline: a compact, line-anchored patch language and applier. Pluggable FS/IO so it works over disk, in-memory, or any custom backend.",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",

package/src/fs.ts

@@ -65,6 +65,9 @@ export abstract class Filesystem {
 	/** Read the file's full text content. Throw on missing file. */
 	abstract readText(path: string): Promise<string>;
 
+	/** Read raw bytes for backends whose text is a direct decode of persisted bytes. */
+	readBinary?(path: string): Promise<Uint8Array | undefined>;
+
 	/** Validate that `path` is writable before a prepared batch starts committing. */
 	async preflightWrite(_path: string, _options?: PreflightWriteOptions): Promise<void> {}
 
@@ -196,6 +199,15 @@ export class NodeFilesystem extends Filesystem {
 		return file.text();
 	}
 
+	async readBinary(path: string): Promise<Uint8Array> {
+		try {
+			return await fs.readFile(path);
+		} catch (error) {
+			if (isNotFound(error)) throw new NotFoundError(path, error);
+			throw error;
+		}
+	}
+
 	async writeText(path: string, content: string): Promise<WriteResult> {
 		await Bun.write(path, content);
 		return { text: content };

package/src/patcher.ts

@@ -148,6 +148,10 @@ function mergeWarnings(...sources: ReadonlyArray<readonly string[] | undefined>)
 	return out;
 }
 
+function hasUtf8Bom(bytes: Uint8Array | undefined): boolean {
+	return bytes !== undefined && bytes.length >= 3 && bytes[0] === 0xef && bytes[1] === 0xbb && bytes[2] === 0xbf;
+}
+
 function assertUniqueCanonicalPaths(prepared: readonly PreparedSection[]): void {
 	const seen = new Map<string, string>();
 	for (const entry of prepared) {
@@ -295,7 +299,8 @@ export class Patcher {
 			throw new Error(`MV destination is the same as ${target.path}.`);
 		}
 
-		const { bom, text } = stripBom(read.rawContent);
+		const { bom: bomFromText, text } = stripBom(read.rawContent);
+		const bom = bomFromText || (await this.#readBinaryBom(target.path));
 		const lineEnding = detectLineEnding(text);
 		const normalized = normalizeToLF(text);
 
@@ -453,6 +458,12 @@ export class Patcher {
 		};
 	}
 
+	async #readBinaryBom(path: string): Promise<string> {
+		if (!this.fs.readBinary) return "";
+		const bytes = await this.fs.readBinary(path);
+		return hasUtf8Bom(bytes) ? "\uFEFF" : "";
+	}
+
 	async #tryRead(path: string): Promise<{ exists: boolean; rawContent: string }> {
 		try {
 			const content = await this.fs.readText(path);