fix(pi): single-quote the extension ENTIRE_CMD so local-dev is valid JS

The launcher value (LocalDevHookScript) carries its own shell quotes, so the
double-quoted template literal produced the malformed
const ENTIRE_CMD = ""\$(git rev-parse --show-toplevel)"/scripts/entire-dev".
Switch the template to single quotes, matching opencode, so the value nests
cleanly and the shell still sees the space-protecting quotes via sh -c.

Regenerate the dogfood .pi extension and pin the exact generated line in the
local-dev test (the old substring check passed on the broken output).

Addresses PR review feedback.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Entire-Checkpoint: 01KY9X6EJ1EK9MAGYJSVC8XVQN

Author: gtrrz-victor

.pi/extensions/entire/index.ts

@@ -11,7 +11,7 @@ import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { execFile } from "node:child_process";
 
 export default function (pi: ExtensionAPI) {
-  const ENTIRE_CMD = ""$(git rev-parse --show-toplevel)"/scripts/entire-dev";
+  const ENTIRE_CMD = '"$(git rev-parse --show-toplevel)"/scripts/entire-dev';
   let pendingSkillEvents: Array<{ skill_name: string; invocation: string; timestamp: string }> = [];
 
   // fireHook pipes data to `entire hooks pi <hookName>` and resolves with the

cmd/entire/cli/agent/pi/entire_extension.ts

@@ -11,7 +11,7 @@ import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { execFile } from "node:child_process";
 
 export default function (pi: ExtensionAPI) {
-  const ENTIRE_CMD = "__ENTIRE_CMD__";
+  const ENTIRE_CMD = '__ENTIRE_CMD__';
   let pendingSkillEvents: Array<{ skill_name: string; invocation: string; timestamp: string }> = [];
 
   // fireHook pipes data to `entire hooks pi <hookName>` and resolves with the

cmd/entire/cli/agent/pi/hooks_test.go

@@ -29,7 +29,7 @@ func TestInstallHooks_FreshInstall(t *testing.T) {
 	}
 	body := string(data)
 
-	if !strings.Contains(body, `const ENTIRE_CMD = "entire"`) {
+	if !strings.Contains(body, `const ENTIRE_CMD = 'entire'`) {
 		t.Error("production ENTIRE_CMD missing")
 	}
 	if !strings.Contains(body, "hooks pi ") {
@@ -53,8 +53,13 @@ func TestInstallHooks_LocalDev(t *testing.T) {
 	if err != nil {
 		t.Fatal(err)
 	}
-	if !strings.Contains(string(data), `"$(git rev-parse --show-toplevel)"/scripts/entire-dev`) {
-		t.Error("local-dev extension should delegate to the entire-dev launcher via git rev-parse")
+	// Assert the exact, well-formed line. The launcher value carries its own
+	// shell quotes, so the template must wrap the placeholder in single quotes;
+	// wrapping in double quotes yields the malformed `""$(...)"/..."` (a broken
+	// JS string literal). A substring check alone would pass on that broken
+	// output, so pin the whole line.
+	if !strings.Contains(string(data), `const ENTIRE_CMD = '"$(git rev-parse --show-toplevel)"/scripts/entire-dev'`) {
+		t.Errorf("local-dev ENTIRE_CMD malformed; got:\n%s", data)
 	}
 }