pi-pwsh-native 0.1.2 → 0.1.3

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 All notable changes to this project will be documented here.
 
+## 0.1.3 — 2026-08-01
+
+### Fixed
+
+- Respect the final PowerShell command's success state instead of allowing a stale native `$LASTEXITCODE` to override it.
+
+### Changed
+
+- Reduced recurring model prompt overhead by shortening the shell description and consolidating PowerShell guidance.
+- Deferred PowerShell runtime probing until a session starts, avoiding unnecessary subprocesses in Pi invocations that only load extensions.
+- Declared the supported Pi and Node.js version ranges explicitly.
+- Added minimum/latest Pi compatibility coverage and prompt-budget regression tests.
+
 ## 0.1.2 — 2026-07-30
 
 ### Added

package/README.md

@@ -47,7 +47,8 @@ Choose `pi-pwsh-native` when you want a small, auditable PowerShell 7 replacemen
 
 - Windows
 - PowerShell 7+
-- Pi with extension support
+- Node.js 22.19 or newer
+- Pi 0.83.0 or newer (before 1.0)
 
 The initial implementation is developed and tested against:
 
@@ -59,7 +60,7 @@ The initial implementation is developed and tested against:
 Install the pinned npm release:
 
 ```powershell
-pi install npm:pi-pwsh-native@0.1.2
+pi install npm:pi-pwsh-native@0.1.3
 ```
 
 Alternatively, install the public GitHub package directly:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-pwsh-native",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Native PowerShell 7 tooling for the Pi coding agent on Windows",
   "license": "MIT",
   "author": "Tako (https://github.com/takomine)",
@@ -20,7 +20,7 @@
     "win32"
   ],
   "engines": {
-    "node": ">=20"
+    "node": ">=22.19.0"
   },
   "keywords": [
     "pi-package",
@@ -55,7 +55,7 @@
     "smoke": "npm run test:runtime && npm run test:exitcode && npm run test:timeout"
   },
   "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "*"
+    "@earendil-works/pi-coding-agent": ">=0.83.0 <1.0.0"
   },
   "devDependencies": {
     "@earendil-works/pi-coding-agent": "^0.83.0",

package/src/index.ts

@@ -27,11 +27,17 @@ export function selectActiveTools(
 	return [...new Set(selected)];
 }
 
-function toolDescription(runtime: PowerShellRuntime, config: PwshNativeConfig): string {
-	const elevation = config.elevationGuidance
-		? " Elevation guidance is enabled, but elevate only after an explicit user request."
-		: " Do not elevate unless the user explicitly requests it.";
-	return `Execute a native PowerShell 7 command on Windows with PowerShell ${runtime.version} (${runtime.executable}). Returns stdout and stderr. Output is truncated to the last 2000 lines or 50KB, whichever is reached first; full truncated output is saved to a temporary file. Commands have no default timeout; set timeout in seconds when needed.${elevation}`;
+export const PWSH_TOOL_DESCRIPTION =
+	"Execute native PowerShell 7 on Windows. Output is tail-truncated to 2,000 lines or 50 KB; full output is saved to a temporary file. Optional timeout is in seconds.";
+
+export function createPromptGuidelines(config: PwshNativeConfig): string[] {
+	return [
+		"Use pwsh only when a shell is needed; use native PowerShell syntax and $env:NAME for environment variables.",
+		"Prefer dedicated file and search tools over pwsh; otherwise bound recursive output and prefer rg or fd when available.",
+		config.elevationGuidance
+			? "In pwsh, elevation guidance is enabled, but elevate only after an explicit user request."
+			: "Do not elevate from pwsh unless the user explicitly requests it.",
+	];
 }
 
 function unavailableUserCommand(reason: string) {
@@ -55,51 +61,51 @@ export default function piPwshNative(pi: ExtensionAPI): void {
 
 	let config: PwshNativeConfig | undefined;
 	let runtime: PowerShellRuntime | undefined;
+	let operations: ReturnType<typeof createPowerShellOperations> | undefined;
 	let setupError: string | undefined;
 	try {
 		config = loadConfig({ agentDir: getAgentDir() }).config;
-		runtime = resolvePowerShellRuntime(config);
 	} catch (error) {
 		setupError = errorMessage(error);
 	}
 
-	if (setupError || !config || !runtime) {
-		const reason = setupError ?? "unknown setup failure";
-		pi.on("session_start", (_event, ctx) => {
+	pi.on("session_start", (_event, ctx) => {
+		if (!setupError && config && !runtime) {
+			try {
+				runtime = resolvePowerShellRuntime(config);
+				operations = createPowerShellOperations(runtime, config);
+			} catch (error) {
+				setupError = errorMessage(error);
+			}
+		}
+
+		if (setupError || !config || !runtime || !operations) {
+			const reason = setupError ?? "unknown setup failure";
 			pi.setActiveTools(selectActiveTools(pi.getActiveTools(), true, false));
 			ctx.ui.notify(`pi-pwsh-native: ${reason} Bash was disabled; no shell fallback was activated.`, "error");
-		});
-		pi.on("user_bash", () => unavailableUserCommand(reason));
-		return;
-	}
-
-	const resolvedConfig = config;
-	const resolvedRuntime = runtime;
-	const operations = createPowerShellOperations(resolvedRuntime, resolvedConfig);
+			return;
+		}
 
-	pi.on("session_start", (_event, ctx) => {
 		const definition = createBashToolDefinition(ctx.cwd, { operations });
 		pi.registerTool({
 			...definition,
 			name: "pwsh",
 			label: "pwsh",
-			description: toolDescription(resolvedRuntime, resolvedConfig),
-			promptSnippet: "Execute native PowerShell 7 commands on Windows",
-			promptGuidelines: [
-				"Use pwsh only when a shell is needed; write native PowerShell syntax and avoid Bash-only syntax or implicit Bash translation.",
-				"Prefer read, write, edit, ls, find, and grep over pwsh when a dedicated tool can complete the task more directly.",
-				"In pwsh, use $env:NAME for environment variables; single quotes are literal, double quotes interpolate, and the backtick is the escape character.",
-				"Prefer bounded rg or fd commands for repository search; tightly bound recursive Get-ChildItem and Select-String operations.",
-				"Do not elevate pwsh commands unless the user explicitly requests elevation.",
-				"Inspect PI_* environment variables in pwsh when current Pi model or session metadata is needed.",
-			],
+			description: PWSH_TOOL_DESCRIPTION,
+			promptSnippet: "Execute native PowerShell 7 commands",
+			promptGuidelines: createPromptGuidelines(config),
 		});
 		pi.setActiveTools(
-			selectActiveTools(pi.getActiveTools(), resolvedConfig.preserveDiscoveryTools, true),
+			selectActiveTools(pi.getActiveTools(), config.preserveDiscoveryTools, true),
 		);
 	});
 
-	if (resolvedConfig.replaceUserBash) {
-		pi.on("user_bash", () => ({ operations }));
+	const replaceUserBash = config?.replaceUserBash ?? true;
+	if (replaceUserBash) {
+		pi.on("user_bash", () =>
+			operations
+				? { operations }
+				: unavailableUserCommand(setupError ?? "PowerShell has not been initialized"),
+		);
 	}
 }

package/src/powershell-source.ts

@@ -10,15 +10,16 @@ export const UTF8_PREFIX = [
 ].join("; ") + "; ";
 
 /**
- * Preserve a native program's real exit code. If no native program set
- * LASTEXITCODE, map PowerShell's success state to 0/1.
+ * Report the final command's status. A successful final command wins over a
+ * stale LASTEXITCODE; a failed final native command retains its real code;
+ * other PowerShell failures map to 1.
  *
  * The leading newline detaches the epilogue from a trailing line comment. The
  * leading semicolon also prevents a trailing backtick from joining the
  * epilogue to the user's final command.
  */
 export const EXIT_EPILOGUE =
-	"\n; $__pi_ok = $?; if ($null -ne $global:LASTEXITCODE) { exit $global:LASTEXITCODE } else { exit ($__pi_ok ? 0 : 1) }";
+	"\n; $__pi_ok = $?; if ($__pi_ok) { exit 0 } elseif ($null -ne $global:LASTEXITCODE -and $global:LASTEXITCODE -ne 0) { exit $global:LASTEXITCODE } else { exit 1 }";
 
 export function buildPowerShellSource(command: string, config: PwshNativeConfig): string {
 	const strict = config.strictMode ? "$ErrorActionPreference = 'Stop'; " : "";