@oh-my-pi/pi-utils 18.11.1 → 18.11.2

package/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 ## [Unreleased]
 
+## [18.11.2] - 2026-10-16
+
+### Added
+
+- Exported `sqliteFileIdentity()`, so a caller holding a long-lived SQLite connection can tell when its path now names a different file (e.g. after a corrupt store was quarantined and replaced) ([#14361](https://github.com/can1357/oh-my-pi/pull/14361) by [@F0Rextasy](https://github.com/F0Rextasy))
+- Added `slugify()`, the shared lowercase ASCII kebab-case helper with an optional `maxLength` ([#14283](https://github.com/can1357/oh-my-pi/pull/14283) by [@jaredlyon](https://github.com/jaredlyon))
+
+### Fixed
+
+- Fixed an unhandled rejection when a subprocess is spawned with an already-aborted signal ([#15750](https://github.com/can1357/oh-my-pi/pull/15750) by [@harshaygadekar](https://github.com/harshaygadekar))
+
 ## [18.10.0] - 2026-10-12
 
 ### Added

package/dist/types/index.d.ts

@@ -31,6 +31,7 @@ export * as ptree from "./ptree.js";
 export { AbortError, ChildProcess, Exception, NonZeroExitError } from "./ptree.js";
 export * from "./runtime-install.js";
 export * from "./sanitize-text.js";
+export * from "./slugify.js";
 export * from "./snowflake.js";
 export * from "./sqlite.js";
 export * from "./stderr-guard.js";

package/dist/types/slugify.d.ts

@@ -0,0 +1,4 @@
+/** Lowercase ASCII kebab slug: [^a-z0-9]+ → "-", edge dashes trimmed; optional maxLength truncates then re-trims trailing dashes; throws RangeError unless maxLength is a non-negative integer. Returns "" when nothing survives. */
+export declare function slugify(input: string, options?: {
+    maxLength?: number;
+}): string;

package/dist/types/sqlite.d.ts

@@ -1,5 +1,17 @@
 /** Shared SQLite opening, error attribution, and result-code classification for persistent stores. */
 import { Database } from "bun:sqlite";
+/**
+ * A store's identity as `dev:ino:birthtimeMs`, `null` when the path holds no
+ * file, or `undefined` when the stat itself failed and no conclusion is safe.
+ */
+export type SqliteFileIdentity = string | null | undefined;
+/**
+ * Identity of the file currently at `dbPath`. Quarantine moves a corrupt store
+ * aside and recreates one at the same path, so a connection cached on the path
+ * alone keeps writing to the unlinked inode: callers that hold a long-lived
+ * handle compare this before reusing it.
+ */
+export declare function sqliteFileIdentity(dbPath: string): SqliteFileIdentity;
 /** Controls opt-in replacement of an unrecoverably corrupt SQLite store. */
 export interface SqliteOpenOptions {
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-utils",
-	"version": "18.11.1",
+	"version": "18.11.2",
 	"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.11.1"
+		"@oh-my-pi/pi-natives": "18.11.2"
 	},
 	"devDependencies": {
 		"@types/bun": "^1.3.14"

package/src/index.ts

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

package/src/ptree.ts

@@ -443,7 +443,9 @@ export class ChildProcess<In extends InMask = InMask> {
 	// ── Output helpers ───────────────────────────────────────────────────
 
 	async #throwIfAborted(): Promise<void> {
-		const exitReason = this.exitReason;
+		// A kill() issued before Bun reports the root's exit only records the
+		// reason as pending; output readers that do not wait for exit must still see it.
+		const exitReason = this.#exitReason ?? this.#exitReasonPending;
 		if (!exitReason?.aborted) return;
 		if (this.#terminating) await this.#terminating;
 		throw exitReason;
@@ -589,10 +591,11 @@ export class ChildProcess<In extends InMask = InMask> {
 	// ── Signal / timeout ─────────────────────────────────────────────────
 
 	attachSignal(signal: AbortSignal): void {
+		const cleanup = this.#exited.catch(() => {});
 		const onAbort = () => this.kill(new AbortError(signal.reason, "<cancelled>"));
 		if (signal.aborted) return void onAbort();
 		signal.addEventListener("abort", onAbort, { once: true });
-		this.#exited.catch(() => {}).finally(() => signal.removeEventListener("abort", onAbort));
+		cleanup.finally(() => signal.removeEventListener("abort", onAbort));
 	}
 
 	#clearTimeout(): void {

package/src/slugify.ts

@@ -0,0 +1,11 @@
+/** Lowercase ASCII kebab slug: [^a-z0-9]+ → "-", edge dashes trimmed; optional maxLength truncates then re-trims trailing dashes; throws RangeError unless maxLength is a non-negative integer. Returns "" when nothing survives. */
+export function slugify(input: string, options?: { maxLength?: number }): string {
+	if (options?.maxLength !== undefined && !(Number.isInteger(options.maxLength) && options.maxLength >= 0)) {
+		throw new RangeError(`slugify maxLength must be a non-negative integer, got ${options.maxLength}`);
+	}
+	const slug = input
+		.toLowerCase()
+		.replace(/[^a-z0-9]+/g, "-")
+		.replace(/^-+|-+$/g, "");
+	return options?.maxLength === undefined ? slug : slug.slice(0, options.maxLength).replace(/-+$/g, "");
+}

package/src/sqlite.ts

@@ -11,7 +11,11 @@ const BUSY_MAX_ATTEMPTS = 4;
 const BUSY_BASE_DELAY_MS = 100;
 const SQLITE_STORE_SUFFIXES = ["-wal", "-shm", "-journal", ""];
 
-type SqliteFileIdentity = string | null | undefined;
+/**
+ * A store's identity as `dev:ino:birthtimeMs`, `null` when the path holds no
+ * file, or `undefined` when the stat itself failed and no conclusion is safe.
+ */
+export type SqliteFileIdentity = string | null | undefined;
 
 /**
  * Corrupt handles held open by in-process openers awaiting recovery, keyed by
@@ -77,7 +81,13 @@ class SqliteAttemptFailure extends Error {
 	}
 }
 
-function sqliteFileIdentity(dbPath: string): SqliteFileIdentity {
+/**
+ * Identity of the file currently at `dbPath`. Quarantine moves a corrupt store
+ * aside and recreates one at the same path, so a connection cached on the path
+ * alone keeps writing to the unlinked inode: callers that hold a long-lived
+ * handle compare this before reusing it.
+ */
+export function sqliteFileIdentity(dbPath: string): SqliteFileIdentity {
 	try {
 		const stat = fs.statSync(dbPath);
 		return `${stat.dev}:${stat.ino}:${stat.birthtimeMs}`;