pi-libtui 0.3.7 → 0.3.8

package/README.md

@@ -79,13 +79,13 @@ guarded Pi 0.84–0.85 adapter; unsupported hosts keep their native transcript.
 Feature packages shell out to Rust binaries such as `terminal_bridge` (the
 exported `TERMINAL_BRIDGE` descriptor). Requires a Rust toolchain
 (https://rustup.rs). The `terminal_bridge` binary builds itself on first use
-under Pi's agent directory (`native/terminal-bridge/v<version>/`), where
-`<version>` is this package's version. Set `PI_TERMINAL_BRIDGE_BINARY` to use
+under Pi's agent directory (`native/terminal-bridge/<revision>/`), where
+`<revision>` is the source commit recorded in the packaged `native-revision` file. Set `PI_TERMINAL_BRIDGE_BINARY` to use
 a prebuilt binary; it must point at an executable file.
 
 `ensureNativeBinary(binary, hooks?)` resolves in this order: the descriptor's
-env override, then `<agentDir>/native/<crate>/v<version>/bin/<name>`, building
-it on first use when absent. Builds are keyed by crate and version, so every
+env override, then `<agentDir>/native/<crate>/<revision>/bin/<name>`, building
+it on first use when absent. Builds are keyed by crate and source commit, so every
 installed copy shares them, and concurrent requests within one process share
 one build. Pass `onBuild` to show the delay in the UI. The extension host keeps
 the shared PTY host alive across an extension reload; a session switch or quit
@@ -203,5 +203,6 @@ Source: https://github.com/luan/agents, directory
 harnesses/pi/agent/packages/pi-libtui. Run `bun run typecheck` and
 `bun test test` in that directory. When running from that checkout,
 `ensureNativeBinary` also looks for `target/{release,debug}/<name>` in the Cargo
-workspace and reports an unbuilt checkout instead of building; builds are pinned
-to the `v<version>` git tag, so publishing requires a matching tag.
+workspace and reports an unbuilt checkout instead of building. `just pi-pack`
+records the checkout commit in every bundled `pi-libtui/native-revision` file,
+so native builds do not depend on npm versions or repository-wide release tags.

package/native-revision

@@ -0,0 +1 @@
+93ecc14e6cb9dc5228165a4b605b933ea595d69e

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-libtui",
-  "version": "0.3.7",
+  "version": "0.3.8",
   "license": "MIT",
   "repository": {
     "type": "git",

package/src/native-binary.ts

@@ -81,7 +81,8 @@ export interface EnsureNativeBinaryHooks {
  * revision under Pi's agent directory and shared by every installed pi-libtui copy.
  */
 export async function ensureNativeBinary(binary: NativeBinary, hooks: EnsureNativeBinaryHooks = {}): Promise<string> {
-	const location = locateNativeBinary(binary, processOptions());
+	const options = processOptions();
+	const location = locateNativeBinary(binary, options);
 	if (location.kind === "found") return location.path;
 	if (location.kind === "unbuilt-checkout") {
 		throw new Error(
@@ -92,7 +93,7 @@ export async function ensureNativeBinary(binary: NativeBinary, hooks: EnsureNati
 	const existing = builds.get(location.path);
 	if (existing) return existing;
 	hooks.onBuild?.(`Building ${binary.binaryName} (first use); this takes a minute.`);
-	const build = runCargo(nativeBuildArguments(binary, nativeRevision(), location.installRoot), location.installRoot)
+	const build = runCargo(nativeBuildArguments(binary, options.rev, location.installRoot), location.installRoot)
 		.then(() => {
 			if (!isExecutable(location.path)) throw new Error(`cargo install did not produce ${location.path}`);
 			return location.path;
@@ -106,12 +107,11 @@ export function terminalBridgeBinaryPath(): Promise<string> {
 	return ensureNativeBinary(TERMINAL_BRIDGE);
 }
 
-/** Builds are pinned to the git tag matching this package's version. Tag the repository when publishing. */
+/** The pack step records the exact source commit independently of npm versions. */
 export function nativeRevision(): string {
-	const manifest = JSON.parse(readFileSync(new URL("../package.json", import.meta.url), "utf8")) as {
-		version: string;
-	};
-	return `v${manifest.version}`;
+	const revision = readFileSync(new URL("../native-revision", import.meta.url), "utf8").trim();
+	if (!/^[a-f0-9]{40}$/.test(revision)) throw new Error("Invalid native source revision; reinstall the package.");
+	return revision;
 }
 
 function processOptions(): NativeBinaryOptions {