@oh-my-pi/pi-utils 17.3.5 → 17.3.8

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [17.3.8] - 2026-08-19
+
+### Added
+
+- Exported `BINARY_SNIFF_BYTES`, the header window `isProbablyBinary` sniffs, so a caller holding the whole file in memory can classify the identical prefix through `isProbablyBinaryHeader` instead of reopening the file.
+
 ## [17.3.5] - 2026-08-16
 
 ### Fixed

package/dist/types/binary.d.ts

@@ -1,3 +1,9 @@
+/**
+ * Header window sniffed for the binary heuristic; mirrors git's 8000-byte scan.
+ * Exported so callers that already hold the whole file in memory can sniff the
+ * identical prefix through {@link isProbablyBinaryHeader} instead of reopening.
+ */
+export declare const BINARY_SNIFF_BYTES = 8192;
 /**
  * Classify an in-memory byte header as binary (non-UTF-8-text).
  *

package/dist/types/index.d.ts

@@ -28,6 +28,7 @@ export { AbortError, ChildProcess, Exception, NonZeroExitError } from "./ptree.j
 export * from "./runtime-install.js";
 export * from "./sanitize-text.js";
 export * from "./snowflake.js";
+export * from "./sqlite.js";
 export * from "./stderr-guard.js";
 export * from "./stream.js";
 export * from "./tab-spacing.js";

package/dist/types/sqlite.d.ts

@@ -0,0 +1,23 @@
+/**
+ * Shared classifiers for `bun:sqlite` error result codes.
+ *
+ * Every omp SQLite store (`agent.db` credential/usage store, `models.db` model
+ * cache, `history.db`) needs the same two distinctions: a transient BUSY that
+ * clears by retrying, and an unrecoverable corruption that never does. Keeping
+ * one implementation here prevents the classifiers from drifting between the
+ * credential store and the model cache.
+ */
+/**
+ * SQLite's busy result-code family — base `SQLITE_BUSY` plus the extended
+ * variants `SQLITE_BUSY_RECOVERY` (concurrent WAL recovery), `SQLITE_BUSY_SNAPSHOT`,
+ * and `SQLITE_BUSY_TIMEOUT`. All warrant the same backoff-and-retry treatment.
+ */
+export declare function isSqliteBusyError(err: unknown): boolean;
+/**
+ * SQLite's unrecoverable-corruption result codes — the `SQLITE_CORRUPT` family
+ * (base plus extended variants like `SQLITE_CORRUPT_VTAB` / `SQLITE_CORRUPT_INDEX`)
+ * and `SQLITE_NOTADB` (the file header is not a database). Unlike
+ * {@link isSqliteBusyError}, these never clear by retrying: the store must be
+ * repaired or replaced, so callers latch, quarantine, or recreate the file.
+ */
+export declare function isSqliteCorruptionError(err: unknown): boolean;

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-utils",
-	"version": "17.3.5",
+	"version": "17.3.8",
 	"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": "17.3.5"
+		"@oh-my-pi/pi-natives": "17.3.8"
 	},
 	"devDependencies": {
 		"@types/bun": "^1.3.14"

package/src/binary.ts

@@ -14,8 +14,12 @@
  */
 import { peekFile, peekFileSync } from "./peek-file";
 
-/** Header window sniffed for the binary heuristic; mirrors git's 8000-byte scan. */
-const BINARY_SNIFF_BYTES = 8192;
+/**
+ * Header window sniffed for the binary heuristic; mirrors git's 8000-byte scan.
+ * Exported so callers that already hold the whole file in memory can sniff the
+ * identical prefix through {@link isProbablyBinaryHeader} instead of reopening.
+ */
+export const BINARY_SNIFF_BYTES = 8192;
 
 /**
  * Classify an in-memory byte header as binary (non-UTF-8-text).

package/src/index.ts

@@ -28,6 +28,7 @@ export { AbortError, ChildProcess, Exception, NonZeroExitError } from "./ptree";
 export * from "./runtime-install";
 export * from "./sanitize-text";
 export * from "./snowflake";
+export * from "./sqlite";
 export * from "./stderr-guard";
 export * from "./stream";
 export * from "./tab-spacing";

package/src/sqlite.ts

@@ -0,0 +1,33 @@
+/**
+ * Shared classifiers for `bun:sqlite` error result codes.
+ *
+ * Every omp SQLite store (`agent.db` credential/usage store, `models.db` model
+ * cache, `history.db`) needs the same two distinctions: a transient BUSY that
+ * clears by retrying, and an unrecoverable corruption that never does. Keeping
+ * one implementation here prevents the classifiers from drifting between the
+ * credential store and the model cache.
+ */
+
+/**
+ * SQLite's busy result-code family — base `SQLITE_BUSY` plus the extended
+ * variants `SQLITE_BUSY_RECOVERY` (concurrent WAL recovery), `SQLITE_BUSY_SNAPSHOT`,
+ * and `SQLITE_BUSY_TIMEOUT`. All warrant the same backoff-and-retry treatment.
+ */
+export function isSqliteBusyError(err: unknown): boolean {
+	if (err === null || typeof err !== "object" || !("code" in err)) return false;
+	const code = err.code;
+	return typeof code === "string" && code.startsWith("SQLITE_BUSY");
+}
+
+/**
+ * SQLite's unrecoverable-corruption result codes — the `SQLITE_CORRUPT` family
+ * (base plus extended variants like `SQLITE_CORRUPT_VTAB` / `SQLITE_CORRUPT_INDEX`)
+ * and `SQLITE_NOTADB` (the file header is not a database). Unlike
+ * {@link isSqliteBusyError}, these never clear by retrying: the store must be
+ * repaired or replaced, so callers latch, quarantine, or recreate the file.
+ */
+export function isSqliteCorruptionError(err: unknown): boolean {
+	if (err === null || typeof err !== "object" || !("code" in err)) return false;
+	const code = err.code;
+	return typeof code === "string" && (code.startsWith("SQLITE_CORRUPT") || code === "SQLITE_NOTADB");
+}