gnhf 0.1.19 → 0.1.20

package/dist/cli.mjs

@@ -6,7 +6,7 @@ import process$1 from "node:process";
 import { createInterface } from "node:readline";
 import { Command, InvalidArgumentError } from "commander";
 import yaml from "js-yaml";
-import { execFileSync, execSync, spawn } from "node:child_process";
+import { execFileSync, spawn } from "node:child_process";
 import { createServer } from "node:net";
 import { EventEmitter } from "node:events";
 import { createHash } from "node:crypto";
@@ -302,32 +302,24 @@ const NOT_GIT_REPOSITORY_MESSAGE = "This command must be run inside a Git reposi
 function translateGitError(error) {
 	return error instanceof Error ? error : new Error(String(error));
 }
-function git(args, cwd) {
+function git(args, cwd, options = {}) {
 	try {
-		return execSync(`git ${args}`, {
+		return execFileSync("git", args, {
 			cwd,
 			encoding: "utf-8",
-			stdio: "pipe"
+			stdio: "pipe",
+			...options.env ? { env: options.env } : {}
 		}).trim();
 	} catch (error) {
 		throw translateGitError(error);
 	}
 }
-/** Wrap a value in single quotes, escaping embedded single quotes for POSIX shells. */
-function shellEscape(value) {
-	return `'${value.replace(/'/g, "'\\''")}'`;
-}
 function isGitRepository(cwd) {
 	try {
-		execSync("git rev-parse --git-dir", {
-			cwd,
-			encoding: "utf-8",
-			stdio: "pipe",
-			env: {
-				...process.env,
-				LC_ALL: "C"
-			}
-		});
+		git(["rev-parse", "--git-dir"], cwd, { env: {
+			...process.env,
+			LC_ALL: "C"
+		} });
 		return true;
 	} catch {
 		return false;
@@ -339,23 +331,41 @@ function ensureGitRepository(cwd) {
 function getCurrentBranch(cwd) {
 	ensureGitRepository(cwd);
 	try {
-		return git("symbolic-ref --short HEAD", cwd);
+		return git([
+			"symbolic-ref",
+			"--short",
+			"HEAD"
+		], cwd);
 	} catch {
-		return git("rev-parse --abbrev-ref HEAD", cwd);
+		return git([
+			"rev-parse",
+			"--abbrev-ref",
+			"HEAD"
+		], cwd);
 	}
 }
 function ensureCleanWorkingTree(cwd) {
-	if (git("status --porcelain", cwd)) throw new Error("Working tree is not clean. Commit or stash changes first.");
+	if (git(["status", "--porcelain"], cwd)) throw new Error("Working tree is not clean. Commit or stash changes first.");
 }
 function createBranch(branchName, cwd) {
-	git(`checkout -b ${branchName}`, cwd);
+	git([
+		"checkout",
+		"-b",
+		branchName
+	], cwd);
 }
 function getHeadCommit(cwd) {
-	return git("rev-parse HEAD", cwd);
+	return git(["rev-parse", "HEAD"], cwd);
 }
 function findLegacyRunBaseCommit(runId, cwd) {
 	try {
-		const marker = git("log --first-parent --reverse --format=%H%x09%s HEAD", cwd).split("\n").map((line) => {
+		const marker = git([
+			"log",
+			"--first-parent",
+			"--reverse",
+			"--format=%H%x09%s",
+			"HEAD"
+		], cwd).split("\n").map((line) => {
 			const [sha, ...subjectParts] = line.split("	");
 			return {
 				sha,
@@ -363,33 +373,57 @@ function findLegacyRunBaseCommit(runId, cwd) {
 			};
 		}).find(({ subject }) => subject === `gnhf: initialize run ${runId}` || subject === `gnhf: overwrite run ${runId}`);
 		if (!marker?.sha) return null;
-		return git(`rev-parse ${marker.sha}^`, cwd);
+		return git(["rev-parse", `${marker.sha}^`], cwd);
 	} catch {
 		return null;
 	}
 }
 function getBranchCommitCount(baseCommit, cwd) {
 	if (!baseCommit) return 0;
-	return Number.parseInt(git(`rev-list --count --first-parent ${baseCommit}..HEAD`, cwd), 10);
+	return Number.parseInt(git([
+		"rev-list",
+		"--count",
+		"--first-parent",
+		`${baseCommit}..HEAD`
+	], cwd), 10);
 }
 function commitAll(message, cwd) {
-	git("add -A", cwd);
+	git(["add", "-A"], cwd);
 	try {
-		git(`commit -m "${message.replace(/"/g, "\\\"")}"`, cwd);
+		git([
+			"commit",
+			"-m",
+			message
+		], cwd);
 	} catch {}
 }
 function resetHard(cwd) {
-	git("reset --hard HEAD", cwd);
-	git("clean -fd", cwd);
+	git([
+		"reset",
+		"--hard",
+		"HEAD"
+	], cwd);
+	git(["clean", "-fd"], cwd);
 }
 function getRepoRootDir(cwd) {
-	return git("rev-parse --show-toplevel", cwd);
+	return git(["rev-parse", "--show-toplevel"], cwd);
 }
 function createWorktree(baseCwd, worktreePath, branchName) {
-	git(`worktree add -b ${shellEscape(branchName)} ${shellEscape(worktreePath)}`, baseCwd);
+	git([
+		"worktree",
+		"add",
+		"-b",
+		branchName,
+		worktreePath
+	], baseCwd);
 }
 function removeWorktree(baseCwd, worktreePath) {
-	git(`worktree remove --force ${shellEscape(worktreePath)}`, baseCwd);
+	git([
+		"worktree",
+		"remove",
+		"--force",
+		worktreePath
+	], baseCwd);
 }
 //#endregion
 //#region src/core/agents/types.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gnhf",
-  "version": "0.1.19",
+  "version": "0.1.20",
   "description": "Before I go to bed, I tell my agents: good night, have fun",
   "type": "module",
   "bin": {