@oh-my-pi/pi-natives 18.10.4 → 18.11.1

package/native/index.d.ts

@@ -2058,7 +2058,10 @@ export interface GrepOptions {
   pattern: string
   /** Directory or file to search: a host path or an absolute `scheme://` URL. */
   path: string
-  /** Glob filter for filenames (e.g., "*.ts"). */
+  /**
+   * Glob filter for file paths (e.g., "*.ts"); a matched directory selects
+   * every file under it.
+   */
   glob?: string
   /**
    * Match simple glob patterns at any depth (default: true; `*.ts` ->
@@ -2459,6 +2462,13 @@ export interface ListWorkspaceOptions {
    * to find them. Default: false.
    */
   collectAgentsMd?: boolean
+  /**
+   * Report unreadable directories instead of silently skipping them: an
+   * unreadable root fails the scan, and unreadable subdirectories are
+   * returned in `unreadableDirectories` while the walk continues. Default:
+   * false (startup discovery stays best-effort).
+   */
+  reportDirectoryErrors?: boolean
   /** Timeout in milliseconds for the operation. */
   timeoutMs?: number
   /** Abort signal for cancelling the operation. */
@@ -2476,6 +2486,11 @@ export interface ListWorkspaceResult {
   agentsMdFiles: Array<string>
   /** True when any output cap was hit. */
   truncated: boolean
+  /**
+   * Subdirectories whose contents could not be listed. Always empty unless
+   * `reportDirectoryErrors` is set.
+   */
+  unreadableDirectories: Array<WorkspaceDirectoryError>
 }
 
 /**
@@ -3908,6 +3923,14 @@ export interface WorkProfile {
   sampleCount: number
 }
 
+/** A subdirectory the scan could not list. */
+export interface WorkspaceDirectoryError {
+  /** Forward-slash path relative to the scan root. */
+  path: string
+  /** Platform error, e.g. `Permission denied (os error 13)`. */
+  message: string
+}
+
 /**
  * Wrap text to a visible width, preserving ANSI escape codes across line
  * breaks.

package/native/loader-state.js

@@ -499,13 +499,11 @@ export function extractEmbeddedAddons({ files, targetDir }) {
 	return writtenPaths;
 }
 
-function maybeExtractEmbeddedAddon(ctx, errors) {
+function maybeExtractEmbeddedAddon(ctx, errors, file = selectEmbeddedAddonFile(ctx.selectedVariant)) {
 	if (!ctx.isCompiledBinary || !embeddedAddon) return null;
 	if (embeddedAddon.platformTag !== ctx.platformTag || embeddedAddon.version !== ctx.packageVersion) return null;
-
-	const selectedEmbeddedFile = selectEmbeddedAddonFile(ctx.selectedVariant);
-	if (!selectedEmbeddedFile) return null;
-	const targetPath = path.join(ctx.versionedDir, selectedEmbeddedFile.filename);
+	if (!file) return null;
+	const targetPath = path.join(ctx.versionedDir, file.filename);
 
 	startupMarker("native:extractEmbeddedAddon:start");
 	try {
@@ -517,11 +515,11 @@ function maybeExtractEmbeddedAddon(ctx, errors) {
 	}
 
 	try {
-		extractEmbeddedAddons({ files: embeddedAddon.files, targetDir: ctx.versionedDir });
+		extractEmbeddedAddons({ files: [file], targetDir: ctx.versionedDir });
 		return targetPath;
 	} catch (err) {
 		const message = err instanceof Error ? err.message : String(err);
-		errors.push(`embedded addon extraction (${selectedEmbeddedFile.filename}): ${message}`);
+		errors.push(`embedded addon extraction (${file.filename}): ${message}`);
 		return null;
 	}
 }
@@ -866,8 +864,16 @@ export function loadNative() {
 	const stagedCandidate = embeddedCandidate ? null : maybeStageNodeModulesAddon(ctx, errors);
 	const prepended = [embeddedCandidate, stagedCandidate].filter(c => typeof c === "string");
 	const runtimeCandidates = prepended.length > 0 ? [...prepended, ...ctx.candidates] : ctx.candidates;
+	const fallbackEmbeddedFile =
+		ctx.selectedVariant === "modern" && embeddedAddon
+			? embeddedAddon.files.find(file => file.variant === "baseline")
+			: null;
+	const fallbackEmbeddedPath = fallbackEmbeddedFile && path.join(ctx.versionedDir, fallbackEmbeddedFile.filename);
 
 	for (const candidate of runtimeCandidates) {
+		if (fallbackEmbeddedFile && candidate === fallbackEmbeddedPath) {
+			maybeExtractEmbeddedAddon(ctx, errors, fallbackEmbeddedFile);
+		}
 		try {
 			startupMarker(`native:require:${path.basename(candidate)}`);
 			const bindings = require_(candidate);

package/native/vcs.d.ts

@@ -81,6 +81,9 @@ export declare function detachGitDir(
 	signal?: AbortSignal,
 ): Promise<"no-git" | "independent" | "detached">;
 
+/** Preserve isolation-local Git objects in the source without changing its refs or index. */
+export declare function copyGitObjects(isolationRoot: string, sourceRoot: string): Promise<void>;
+
 /** Run Git's checkout hook in a new linked worktree and return its output and exit status. */
 export declare function runPostCheckoutHook(
 	worktreeRoot: string,

package/native/vcs.js

@@ -1,5 +1,6 @@
 import { $ } from "bun";
 import * as fs from "node:fs";
+import * as path from "node:path";
 import { loadNative } from "./loader-state.js";
 
 let native;
@@ -96,6 +97,58 @@ export function detachGitDir(worktreeRoot, sourceCommonDir, signal) {
 	return api().vcsDetachGitDir(worktreeRoot, sourceCommonDir, signal);
 }
 
+/**
+ * Publish `from` at `to` only once fully written: copy to a sibling temp name,
+ * then hard-link it into place (Git's own `finalize_object_file` strategy) so
+ * readers never see a partial file and an interrupted copy leaves only a
+ * `tmp_` file that `git gc` prunes. An existing `to` was itself published
+ * whole and holds the same content-addressed bytes, so it is kept.
+ */
+async function publishGitObjectFile(from, to) {
+	const tmp = path.join(path.dirname(to), `tmp_obj_omp_${crypto.randomUUID()}`);
+	try {
+		await fs.promises.copyFile(from, tmp, fs.constants.COPYFILE_EXCL);
+		try {
+			await fs.promises.link(tmp, to);
+		} catch (error) {
+			if (error?.code === "EEXIST") return;
+			// Filesystems without hard links: rename is still atomic, and any
+			// file it replaces holds identical content-addressed bytes.
+			if (error?.code !== "EPERM" && error?.code !== "ENOTSUP" && error?.code !== "EXDEV") throw error;
+			await fs.promises.rename(tmp, to);
+		}
+	} finally {
+		await fs.promises.rm(tmp, { force: true });
+	}
+}
+
+/**
+ * Keep objects created in an isolated Git repository reachable after its
+ * worktree is removed, without changing the destination's refs or index.
+ * Every file is published atomically; a pack is published before its index.
+ */
+export async function copyGitObjects(isolationRoot, sourceRoot) {
+	const from = path.join(requireGit(isolationRoot).info().commonDir, "objects");
+	const to = path.join(requireGit(sourceRoot).info().commonDir, "objects");
+	for (const entry of await fs.promises.readdir(from, { withFileTypes: true })) {
+		if (!entry.isDirectory() || !/^[0-9a-f]{2}$/.test(entry.name)) continue;
+		await fs.promises.mkdir(path.join(to, entry.name), { recursive: true });
+		for (const name of await fs.promises.readdir(path.join(from, entry.name))) {
+			if (!/^[0-9a-f]{38,62}$/.test(name)) continue;
+			await publishGitObjectFile(path.join(from, entry.name, name), path.join(to, entry.name, name));
+		}
+	}
+	const packs = (await fs.promises.readdir(path.join(from, "pack"))).filter(name => name.startsWith("pack-"));
+	await fs.promises.mkdir(path.join(to, "pack"), { recursive: true });
+	// The index must only become visible after the pack it describes.
+	for (const name of packs.filter(name => name.endsWith(".pack"))) {
+		await publishGitObjectFile(path.join(from, "pack", name), path.join(to, "pack", name));
+	}
+	for (const name of packs.filter(name => /\.(?:idx|rev|keep|promisor|mtimes)$/.test(name))) {
+		await publishGitObjectFile(path.join(from, "pack", name), path.join(to, "pack", name));
+	}
+}
+
 /**
  * Run Git's checkout hook in a newly created linked worktree, using the same
  * null old HEAD and branch-switch flag as `git worktree add`.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-natives",
-	"version": "18.10.4",
+	"version": "18.11.1",
 	"description": "Native Rust bindings for PDF conversion, audio, WebRTC, grep, clipboard, image processing, syntax highlighting, PTY, and shell operations via N-API",
 	"type": "module",
 	"homepage": "https://omp.sh",
@@ -113,11 +113,11 @@
 		}
 	},
 	"optionalDependencies": {
-		"@oh-my-pi/pi-natives-linux-x64": "18.10.4",
-		"@oh-my-pi/pi-natives-linux-arm64": "18.10.4",
-		"@oh-my-pi/pi-natives-darwin-x64": "18.10.4",
-		"@oh-my-pi/pi-natives-darwin-arm64": "18.10.4",
-		"@oh-my-pi/pi-natives-win32-x64": "18.10.4",
-		"@oh-my-pi/pi-natives-win32-arm64": "18.10.4"
+		"@oh-my-pi/pi-natives-linux-x64": "18.11.1",
+		"@oh-my-pi/pi-natives-linux-arm64": "18.11.1",
+		"@oh-my-pi/pi-natives-darwin-x64": "18.11.1",
+		"@oh-my-pi/pi-natives-darwin-arm64": "18.11.1",
+		"@oh-my-pi/pi-natives-win32-x64": "18.11.1",
+		"@oh-my-pi/pi-natives-win32-arm64": "18.11.1"
 	}
 }