@howaboua/pi-codex-conversion 1.0.1 → 1.0.3

package/README.md

@@ -79,6 +79,7 @@ pi install git:github.com/IgorWarzocha/pi-codex-conversion
 The adapter does not build a standalone replacement prompt anymore. Instead it:
 
 - keeps Pi's tool descriptions, Pi docs section, AGENTS/project context, skills inventory, and date/cwd when Pi already surfaced them
+- adds the current shell to the transformed prompt so quoting and escaping can match the runtime environment
 - rewrites the top-level role framing to Codex-style wording
 - adds a small Codex delta to the existing `Guidelines` section
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@howaboua/pi-codex-conversion",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "Codex-oriented tool and prompt adapter for pi coding agent",
   "type": "module",
   "repository": {

package/src/adapter/codex-model.ts

@@ -14,7 +14,8 @@ export function isCodexLikeModel(model: Partial<CodexLikeModelDescriptor> | null
 	const provider = (model.provider ?? "").toLowerCase();
 	const api = (model.api ?? "").toLowerCase();
 	const id = (model.id ?? "").toLowerCase();
-	return provider.includes("codex") || api.includes("codex") || id.includes("codex") || (provider.includes("openai") && id.includes("gpt"));
+	const isCopilotGpt = (provider.includes("copilot") || api.includes("copilot")) && id.includes("gpt");
+	return provider.includes("codex") || api.includes("codex") || id.includes("codex") || (provider.includes("openai") && id.includes("gpt")) || isCopilotGpt;
 }
 
 export function isCodexLikeContext(ctx: ExtensionContext): boolean {

package/src/index.ts

@@ -65,7 +65,12 @@ export default function codexConversion(pi: ExtensionAPI) {
 		if (!isCodexLikeContext(ctx)) {
 			return undefined;
 		}
-		return { systemPrompt: buildCodexSystemPrompt(event.systemPrompt, { skills: state.promptSkills }) };
+		return {
+			systemPrompt: buildCodexSystemPrompt(event.systemPrompt, {
+				skills: state.promptSkills,
+				shell: process.env.SHELL || "/bin/bash",
+			}),
+		};
 	});
 }
 
@@ -94,9 +99,12 @@ function enableAdapter(pi: ExtensionAPI, ctx: ExtensionContext, state: AdapterSt
 }
 
 function disableAdapter(pi: ExtensionAPI, ctx: ExtensionContext, state: AdapterState): void {
+	const previousToolNames = state.previousToolNames && state.previousToolNames.length > 0 ? state.previousToolNames : DEFAULT_TOOL_NAMES;
+	const restoredTools = restoreTools(previousToolNames, pi.getActiveTools());
+	if (state.enabled || hasAdapterTools(pi.getActiveTools())) {
+		pi.setActiveTools(restoredTools);
+	}
 	if (state.enabled) {
-		const previousToolNames = state.previousToolNames && state.previousToolNames.length > 0 ? state.previousToolNames : DEFAULT_TOOL_NAMES;
-		pi.setActiveTools(restoreTools(previousToolNames, pi.getActiveTools()));
 		state.enabled = false;
 	}
 	setStatus(ctx, false);
@@ -128,3 +136,7 @@ export function restoreTools(previousTools: string[], activeTools: string[]): st
 	}
 	return restored;
 }
+
+function hasAdapterTools(activeTools: string[]): boolean {
+	return activeTools.some((toolName) => ADAPTER_TOOL_NAMES.includes(toolName));
+}

package/src/prompt/build-system-prompt.ts

@@ -30,6 +30,13 @@ function insertBeforeTrailingContext(prompt: string, section: string): string {
 	return `${prompt}\n\n${section}`;
 }
 
+function injectShell(prompt: string, shell?: string): string {
+	if (!shell || /\nCurrent shell:/.test(prompt)) {
+		return prompt;
+	}
+	return insertBeforeTrailingContext(prompt, `Current shell: ${shell}`);
+}
+
 function decodeXml(text: string): string {
 	return text
 		.replace(/'/g, "'")
@@ -106,6 +113,6 @@ function injectGuidelines(prompt: string): string {
 	return `${prompt.slice(0, match.index)}${replacement}${prompt.slice(match.index + match[0].length)}`;
 }
 
-export function buildCodexSystemPrompt(basePrompt: string, options: { skills?: PromptSkill[] } = {}): string {
-	return injectSkills(injectGuidelines(rewriteIntro(basePrompt)), options.skills ?? []);
+export function buildCodexSystemPrompt(basePrompt: string, options: { skills?: PromptSkill[]; shell?: string } = {}): string {
+	return injectShell(injectSkills(injectGuidelines(rewriteIntro(basePrompt)), options.skills ?? []), options.shell);
 }