@xynogen/pix-core 0.4.50 → 0.4.52

package/README.md

@@ -39,6 +39,12 @@ Pi activates extensions per installed package via each package's `pi.extensions`
 | `pix-todo` | `todo` — durable execution checklist |
 | `pix-ask` | `ask_user` — structured TUI questionnaire |
 
+**Shell integration**
+
+| Feature | Description |
+|---|---|
+| `$SHELL` respect | User `!` commands run through `$SHELL` instead of Pi's default `/bin/bash`. For zsh, `.zshrc` is sourced and aliases expand correctly via `eval`. Other non-bash shells get the shell redirect without rc wrapping. |
+
 **Shared data + behaviour**
 
 | Package | Description |

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@xynogen/pix-core",
-	"version": "0.4.50",
-	"description": "Pi extension bundle — installs and activates all core pix-* extensions",
+	"version": "0.4.52",
+	"description": "Pi extension bundle \u2014 installs and activates all core pix-* extensions",
 	"type": "module",
 	"main": "src/extension.ts",
 	"pi": {
@@ -38,31 +38,31 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@xynogen/pix-runtime": "^0.3.1",
-		"@xynogen/pix-data": "^0.4.0",
-		"@xynogen/pix-pretty": "^1.8.0",
-		"@xynogen/pix-welcome": "^0.1.17",
-		"@xynogen/pix-footer": "^0.1.19",
-		"@xynogen/pix-diagnostics": "^0.1.9",
+		"@xynogen/pix-runtime": "^0.4.0",
+		"@xynogen/pix-data": "^0.4.1",
+		"@xynogen/pix-pretty": "^1.8.1",
+		"@xynogen/pix-welcome": "^0.1.18",
+		"@xynogen/pix-footer": "^0.1.20",
+		"@xynogen/pix-diagnostics": "^0.1.10",
 		"@xynogen/pix-display": "^0.1.13",
-		"@xynogen/pix-prompts": "^0.3.4",
-		"@xynogen/pix-skills": "^0.7.4",
-		"@xynogen/pix-models": "^0.1.23",
-		"@xynogen/pix-update": "^0.1.16",
-		"@xynogen/pix-commands": "^0.2.4",
-		"@xynogen/pix-nudge": "^0.1.12",
-		"@xynogen/pix-read": "^0.1.19",
-		"@xynogen/pix-write": "^0.1.18",
-		"@xynogen/pix-edit": "^0.1.20",
-		"@xynogen/pix-find": "^0.1.19",
-		"@xynogen/pix-grep": "^0.1.19",
-		"@xynogen/pix-ls": "^0.1.19",
-		"@xynogen/pix-bash": "^0.1.17",
-		"@xynogen/pix-todo": "^0.1.19",
-		"@xynogen/pix-ask": "^0.2.12",
+		"@xynogen/pix-prompts": "^0.3.5",
+		"@xynogen/pix-skills": "^0.7.5",
+		"@xynogen/pix-models": "^0.1.24",
+		"@xynogen/pix-update": "^0.1.17",
+		"@xynogen/pix-commands": "^0.2.5",
+		"@xynogen/pix-nudge": "^0.1.13",
+		"@xynogen/pix-read": "^0.1.20",
+		"@xynogen/pix-write": "^0.1.19",
+		"@xynogen/pix-edit": "^0.1.21",
+		"@xynogen/pix-find": "^0.1.20",
+		"@xynogen/pix-grep": "^0.1.20",
+		"@xynogen/pix-ls": "^0.1.20",
+		"@xynogen/pix-bash": "^0.1.18",
+		"@xynogen/pix-todo": "^0.1.20",
+		"@xynogen/pix-ask": "^0.2.13",
 		"@xynogen/pix-optimizer": "^1.1.20",
-		"@xynogen/pix-gate": "^0.2.0",
-		"@xynogen/pix-subagent": "^0.6.2"
+		"@xynogen/pix-gate": "^0.2.1",
+		"@xynogen/pix-subagent": "^0.6.3"
 	},
 	"peerDependencies": {
 		"@earendil-works/pi-coding-agent": "*",

package/src/extension.ts

@@ -12,6 +12,7 @@
  */
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import { createLocalBashOperations } from "@earendil-works/pi-coding-agent";
 import registerAsk from "@xynogen/pix-ask/src/index.ts";
 import registerBash from "@xynogen/pix-bash/src/extension.ts";
 import registerCommands from "@xynogen/pix-commands/src/extension.ts";
@@ -81,4 +82,57 @@ export default function (pi: ExtensionAPI): void {
 	for (const register of MEMBERS) {
 		(register as (pi: ExtensionAPI) => void)(pi);
 	}
+
+	// ── Respect $SHELL for user ! commands ──────────────────────────────────
+	// Pi defaults to /bin/bash for ! commands. When the user's login shell
+	// differs (zsh, fish, nushell, …), redirect ! execution through $SHELL
+	// in interactive mode so aliases, functions, and rc config all work.
+	const userShell = process.env.SHELL;
+	if (userShell && !/\bbash$/.test(userShell)) {
+		const ops = createInteractiveShellOps(userShell);
+		pi.on("user_bash", () => ({ operations: ops }));
+	}
+}
+
+/**
+ * Build BashOperations that run commands through the user's real shell with
+ * rc files loaded and alias expansion enabled.
+ *
+ * Problem: `zsh -c "gss"` is non-interactive — it skips `.zshrc` and disables
+ * alias expansion.  Even `zsh -c ". ~/.zshrc; gss"` fails because aliases
+ * are resolved at parse-time, before `.zshrc` is sourced.
+ *
+ * Solution: source the rc file, then `eval` the command.  `eval` creates a
+ * new parse boundary, so aliases defined by the source are available.
+ */
+function createInteractiveShellOps(
+	shellPath: string,
+): import("@earendil-works/pi-coding-agent").BashOperations {
+	const inner = createLocalBashOperations({ shellPath });
+	const name = shellPath.split("/").pop() ?? "";
+	const home = process.env.HOME ?? "~";
+
+	return {
+		exec(command, cwd, options) {
+			return inner.exec(wrapForShell(name, home, command), cwd, options);
+		},
+	};
+}
+
+/**
+ * Wrap a command so it runs inside the user's shell with rc and alias support.
+ */
+function wrapForShell(shell: string, home: string, command: string): string {
+	if (shell === "zsh") {
+		// 1. source .zshrc for aliases/functions/PATH
+		// 2. eval to create a fresh parse boundary so aliases expand
+		return `. ${home}/.zshrc 2>/dev/null; eval ${escapeForShell(command)}`;
+	}
+	// Other shells: run as-is, rely on shellPath alone
+	return command;
+}
+
+/** Single-quote a string for safe shell embedding. */
+function escapeForShell(s: string): string {
+	return `'${s.replace(/'/g, "'\\''")}'`;
 }