hyper-pm 0.1.3 → 0.1.4

package/README.md

@@ -93,11 +93,16 @@ GitHub Issues sync (outbound + inbound). **Outbound** sets issue `labels` to `hy
 
 Skips network work if `sync` is `off` in config, or if you pass `--no-github`.
 
-| Option        | Description          | Default                          |
-| ------------- | -------------------- | -------------------------------- |
-| `--no-github` | Skip GitHub API sync | sync runs when enabled in config |
+When sync runs, after GitHub API work, **`hyper-pm` commits on the data branch** using `git -c user.name=… -c user.email=… commit` so a missing global `user.name` / `user.email` does not block you. Identity is taken from existing git config in the repo, then optional **`HYPER_PM_GIT_USER_NAME`** / **`HYPER_PM_GIT_USER_EMAIL`** (or standard **`GIT_AUTHOR_NAME`** / **`GIT_AUTHOR_EMAIL`**), then built-in defaults.
 
-When sync is not skipped, requires `GITHUB_TOKEN` **or** a local `gh` session (`gh auth token` must succeed).
+It then **attempts** `git push -u` to the configured `remote` (default `origin`). A missing remote or a push error **does not fail the command**: JSON output still has `"ok":true` with **`dataBranchPush`** set to `pushed`, `skipped_no_remote`, `failed`, or (with **`--skip-push`**) `skipped_cli`, and optional **`dataBranchPushDetail`**. Push failures are also written to stderr as a single-line hint. Pass **`--skip-push`** to skip the push attempt entirely.
+
+| Option        | Description                                   | Default                          |
+| ------------- | --------------------------------------------- | -------------------------------- |
+| `--no-github` | Skip GitHub API sync                          | sync runs when enabled in config |
+| `--skip-push` | Skip `git push` of the data branch after sync | `false` (push is attempted)      |
+
+When sync is not skipped, requires `GITHUB_TOKEN` **or** a local `gh` session (`gh auth token` must succeed). For a successful push you still need a configured remote and credentials that allow `git push` to that remote.
 
 ### `audit`
 

package/dist/index.cjs

@@ -7622,6 +7622,20 @@ var envSchema = external_exports.object({
   HYPER_PM_AI_API_KEY: external_exports.string().optional(),
   /** Optional override for hyper-pm JSONL event `actor` on CLI mutations. */
   HYPER_PM_ACTOR: external_exports.string().optional(),
+  /**
+   * Optional `git user.name` for hyper-pm data-branch commits when repo git
+   * identity is unset (falls back after `GIT_AUTHOR_*`, then a built-in default).
+   */
+  HYPER_PM_GIT_USER_NAME: external_exports.string().optional(),
+  /**
+   * Optional `git user.email` for hyper-pm data-branch commits when repo git
+   * identity is unset (falls back after `GIT_AUTHOR_*`, then a built-in default).
+   */
+  HYPER_PM_GIT_USER_EMAIL: external_exports.string().optional(),
+  /** Standard Git override for commit author name (optional). */
+  GIT_AUTHOR_NAME: external_exports.string().optional(),
+  /** Standard Git override for commit author email (optional). */
+  GIT_AUTHOR_EMAIL: external_exports.string().optional(),
   /**
    * Absolute path to the hyper-pm CLI bundle (`main.cjs`) for hyper-pm-mcp and
    * other integrations when auto-resolution from the `hyper-pm` package is insufficient.
@@ -13363,6 +13377,33 @@ var tryReadGithubOwnerRepoSlugFromGit = async (params) => {
   }
 };
 
+// src/git/resolve-effective-git-author-for-data-commit.ts
+var defaultDataCommitName = "hyper-pm";
+var defaultDataCommitEmail = "hyper-pm@users.noreply.github.com";
+var tryReadGitConfigUserName = async (cwd, runGit2) => {
+  try {
+    const { stdout } = await runGit2(cwd, ["config", "--get", "user.name"]);
+    return stdout.trim();
+  } catch {
+    return "";
+  }
+};
+var tryReadGitConfigUserEmail = async (cwd, runGit2) => {
+  try {
+    const { stdout } = await runGit2(cwd, ["config", "--get", "user.email"]);
+    return stdout.trim();
+  } catch {
+    return "";
+  }
+};
+var resolveEffectiveGitAuthorForDataCommit = async (cwd, runGit2, authorEnv) => {
+  const fromGitName = await tryReadGitConfigUserName(cwd, runGit2);
+  const fromGitEmail = await tryReadGitConfigUserEmail(cwd, runGit2);
+  const name = fromGitName || authorEnv.HYPER_PM_GIT_USER_NAME?.trim() || authorEnv.GIT_AUTHOR_NAME?.trim() || defaultDataCommitName;
+  const email = fromGitEmail || authorEnv.HYPER_PM_GIT_USER_EMAIL?.trim() || authorEnv.GIT_AUTHOR_EMAIL?.trim() || defaultDataCommitEmail;
+  return { name, email };
+};
+
 // src/run/commit-data.ts
 var maxActorSuffixLen = 60;
 var formatDataBranchCommitMessage = (base, actorSuffix) => {
@@ -13374,11 +13415,51 @@ var formatDataBranchCommitMessage = (base, actorSuffix) => {
   const suffix = collapsed.length > maxActorSuffixLen ? `${collapsed.slice(0, maxActorSuffixLen - 1)}\u2026` : collapsed;
   return `${base} (${suffix})`;
 };
-var commitDataWorktreeIfNeeded = async (worktreePath, message, runGit2) => {
+var commitDataWorktreeIfNeeded = async (worktreePath, message, runGit2, opts) => {
   const { stdout } = await runGit2(worktreePath, ["status", "--porcelain"]);
   if (!stdout.trim()) return;
   await runGit2(worktreePath, ["add", "."]);
-  await runGit2(worktreePath, ["commit", "-m", message]);
+  const authorEnv = opts?.authorEnv ?? env;
+  const { name, email } = await resolveEffectiveGitAuthorForDataCommit(
+    worktreePath,
+    runGit2,
+    authorEnv
+  );
+  await runGit2(worktreePath, [
+    "-c",
+    `user.name=${name}`,
+    "-c",
+    `user.email=${email}`,
+    "commit",
+    "-m",
+    message
+  ]);
+};
+
+// src/run/push-data-branch.ts
+var firstLineFromUnknown = (err) => {
+  const raw = err instanceof Error ? err.message : String(err);
+  const line = raw.trim().split("\n")[0]?.trim() ?? raw.trim();
+  return line.length > 0 ? line : "git error";
+};
+var tryPushDataBranchToRemote = async (worktreePath, remote, branch, runGit2) => {
+  try {
+    await runGit2(worktreePath, ["remote", "get-url", remote]);
+  } catch (e) {
+    return {
+      status: "skipped_no_remote",
+      detail: firstLineFromUnknown(e)
+    };
+  }
+  try {
+    await runGit2(worktreePath, ["push", "-u", remote, branch]);
+    return { status: "pushed" };
+  } catch (e) {
+    return {
+      status: "failed",
+      detail: firstLineFromUnknown(e)
+    };
+  }
 };
 
 // src/storage/append-event.ts
@@ -15539,7 +15620,11 @@ ${body}`
     }
     deps.exit(ExitCode.Success);
   });
-  program2.command("sync").description("GitHub Issues sync").option("--no-github", "skip network sync", false).action(async function() {
+  program2.command("sync").description("GitHub Issues sync").option("--no-github", "skip network sync", false).option(
+    "--skip-push",
+    "after a successful sync, do not push the data branch to the remote",
+    false
+  ).action(async function() {
     const g = readGlobals(this);
     const o = this.opts();
     const repoRoot = await resolveRepoRoot(g.repo);
@@ -15621,7 +15706,33 @@ ${body}`
         formatDataBranchCommitMessage("hyper-pm: sync", outboundActor),
         runGit
       );
-      deps.log(formatOutput(g.format, { ok: true }));
+      let dataBranchPush;
+      let dataBranchPushDetail;
+      if (o.skipPush) {
+        dataBranchPush = "skipped_cli";
+        dataBranchPushDetail = "skip-push";
+      } else {
+        const pushResult = await tryPushDataBranchToRemote(
+          session.worktreePath,
+          cfg.remote,
+          cfg.dataBranch,
+          runGit
+        );
+        dataBranchPush = pushResult.status;
+        dataBranchPushDetail = pushResult.detail;
+        if (pushResult.status === "failed" && pushResult.detail) {
+          deps.error(
+            `hyper-pm: data branch not pushed (${cfg.remote}/${cfg.dataBranch}): ${pushResult.detail}`
+          );
+        }
+      }
+      deps.log(
+        formatOutput(g.format, {
+          ok: true,
+          dataBranchPush,
+          ...dataBranchPushDetail !== void 0 ? { dataBranchPushDetail } : {}
+        })
+      );
     } catch (e) {
       deps.error(e instanceof Error ? e.message : String(e));
       deps.exit(ExitCode.ExternalApi);

package/dist/main.cjs

@@ -7612,6 +7612,20 @@ var envSchema = external_exports.object({
   HYPER_PM_AI_API_KEY: external_exports.string().optional(),
   /** Optional override for hyper-pm JSONL event `actor` on CLI mutations. */
   HYPER_PM_ACTOR: external_exports.string().optional(),
+  /**
+   * Optional `git user.name` for hyper-pm data-branch commits when repo git
+   * identity is unset (falls back after `GIT_AUTHOR_*`, then a built-in default).
+   */
+  HYPER_PM_GIT_USER_NAME: external_exports.string().optional(),
+  /**
+   * Optional `git user.email` for hyper-pm data-branch commits when repo git
+   * identity is unset (falls back after `GIT_AUTHOR_*`, then a built-in default).
+   */
+  HYPER_PM_GIT_USER_EMAIL: external_exports.string().optional(),
+  /** Standard Git override for commit author name (optional). */
+  GIT_AUTHOR_NAME: external_exports.string().optional(),
+  /** Standard Git override for commit author email (optional). */
+  GIT_AUTHOR_EMAIL: external_exports.string().optional(),
   /**
    * Absolute path to the hyper-pm CLI bundle (`main.cjs`) for hyper-pm-mcp and
    * other integrations when auto-resolution from the `hyper-pm` package is insufficient.
@@ -13353,6 +13367,33 @@ var tryReadGithubOwnerRepoSlugFromGit = async (params) => {
   }
 };
 
+// src/git/resolve-effective-git-author-for-data-commit.ts
+var defaultDataCommitName = "hyper-pm";
+var defaultDataCommitEmail = "hyper-pm@users.noreply.github.com";
+var tryReadGitConfigUserName = async (cwd, runGit2) => {
+  try {
+    const { stdout } = await runGit2(cwd, ["config", "--get", "user.name"]);
+    return stdout.trim();
+  } catch {
+    return "";
+  }
+};
+var tryReadGitConfigUserEmail = async (cwd, runGit2) => {
+  try {
+    const { stdout } = await runGit2(cwd, ["config", "--get", "user.email"]);
+    return stdout.trim();
+  } catch {
+    return "";
+  }
+};
+var resolveEffectiveGitAuthorForDataCommit = async (cwd, runGit2, authorEnv) => {
+  const fromGitName = await tryReadGitConfigUserName(cwd, runGit2);
+  const fromGitEmail = await tryReadGitConfigUserEmail(cwd, runGit2);
+  const name = fromGitName || authorEnv.HYPER_PM_GIT_USER_NAME?.trim() || authorEnv.GIT_AUTHOR_NAME?.trim() || defaultDataCommitName;
+  const email = fromGitEmail || authorEnv.HYPER_PM_GIT_USER_EMAIL?.trim() || authorEnv.GIT_AUTHOR_EMAIL?.trim() || defaultDataCommitEmail;
+  return { name, email };
+};
+
 // src/run/commit-data.ts
 var maxActorSuffixLen = 60;
 var formatDataBranchCommitMessage = (base, actorSuffix) => {
@@ -13364,11 +13405,51 @@ var formatDataBranchCommitMessage = (base, actorSuffix) => {
   const suffix = collapsed.length > maxActorSuffixLen ? `${collapsed.slice(0, maxActorSuffixLen - 1)}\u2026` : collapsed;
   return `${base} (${suffix})`;
 };
-var commitDataWorktreeIfNeeded = async (worktreePath, message, runGit2) => {
+var commitDataWorktreeIfNeeded = async (worktreePath, message, runGit2, opts) => {
   const { stdout } = await runGit2(worktreePath, ["status", "--porcelain"]);
   if (!stdout.trim()) return;
   await runGit2(worktreePath, ["add", "."]);
-  await runGit2(worktreePath, ["commit", "-m", message]);
+  const authorEnv = opts?.authorEnv ?? env;
+  const { name, email } = await resolveEffectiveGitAuthorForDataCommit(
+    worktreePath,
+    runGit2,
+    authorEnv
+  );
+  await runGit2(worktreePath, [
+    "-c",
+    `user.name=${name}`,
+    "-c",
+    `user.email=${email}`,
+    "commit",
+    "-m",
+    message
+  ]);
+};
+
+// src/run/push-data-branch.ts
+var firstLineFromUnknown = (err) => {
+  const raw = err instanceof Error ? err.message : String(err);
+  const line = raw.trim().split("\n")[0]?.trim() ?? raw.trim();
+  return line.length > 0 ? line : "git error";
+};
+var tryPushDataBranchToRemote = async (worktreePath, remote, branch, runGit2) => {
+  try {
+    await runGit2(worktreePath, ["remote", "get-url", remote]);
+  } catch (e) {
+    return {
+      status: "skipped_no_remote",
+      detail: firstLineFromUnknown(e)
+    };
+  }
+  try {
+    await runGit2(worktreePath, ["push", "-u", remote, branch]);
+    return { status: "pushed" };
+  } catch (e) {
+    return {
+      status: "failed",
+      detail: firstLineFromUnknown(e)
+    };
+  }
 };
 
 // src/storage/append-event.ts
@@ -15529,7 +15610,11 @@ ${body}`
     }
     deps.exit(ExitCode.Success);
   });
-  program2.command("sync").description("GitHub Issues sync").option("--no-github", "skip network sync", false).action(async function() {
+  program2.command("sync").description("GitHub Issues sync").option("--no-github", "skip network sync", false).option(
+    "--skip-push",
+    "after a successful sync, do not push the data branch to the remote",
+    false
+  ).action(async function() {
     const g = readGlobals(this);
     const o = this.opts();
     const repoRoot = await resolveRepoRoot(g.repo);
@@ -15611,7 +15696,33 @@ ${body}`
         formatDataBranchCommitMessage("hyper-pm: sync", outboundActor),
         runGit
       );
-      deps.log(formatOutput(g.format, { ok: true }));
+      let dataBranchPush;
+      let dataBranchPushDetail;
+      if (o.skipPush) {
+        dataBranchPush = "skipped_cli";
+        dataBranchPushDetail = "skip-push";
+      } else {
+        const pushResult = await tryPushDataBranchToRemote(
+          session.worktreePath,
+          cfg.remote,
+          cfg.dataBranch,
+          runGit
+        );
+        dataBranchPush = pushResult.status;
+        dataBranchPushDetail = pushResult.detail;
+        if (pushResult.status === "failed" && pushResult.detail) {
+          deps.error(
+            `hyper-pm: data branch not pushed (${cfg.remote}/${cfg.dataBranch}): ${pushResult.detail}`
+          );
+        }
+      }
+      deps.log(
+        formatOutput(g.format, {
+          ok: true,
+          dataBranchPush,
+          ...dataBranchPushDetail !== void 0 ? { dataBranchPushDetail } : {}
+        })
+      );
     } catch (e) {
       deps.error(e instanceof Error ? e.message : String(e));
       deps.exit(ExitCode.ExternalApi);

package/env.example

@@ -6,3 +6,8 @@ GITHUB_TOKEN=""
 HYPER_PM_AI_API_KEY=""
 # Optional: default audit actor for CLI mutations (overridden by --actor)
 HYPER_PM_ACTOR=""
+# Optional: git author for data-branch commits when user.name / user.email are unset (see README sync)
+HYPER_PM_GIT_USER_NAME=""
+HYPER_PM_GIT_USER_EMAIL=""
+GIT_AUTHOR_NAME=""
+GIT_AUTHOR_EMAIL=""

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hyper-pm",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Git-backed work-item CLI with optional GitHub sync",
   "repository": {
     "type": "git",