@biffo/cli 0.185.4 → 0.186.0

package/dist/index.js

@@ -824,6 +824,29 @@ function classifyUpgradeBranches(refs, currentBranch) {
 // src/adapters/git/index.ts
 var GitAdapter = class {
   /** True if `cwd` is inside a git working tree. */
+  /**
+   * The committer identity git would resolve here, or `null` per unresolved value
+   * (#737).
+   *
+   * `git config --get` exits 1 when a key is unset, which `execa` throws on — so a
+   * missing value is caught and reported as `null` rather than as an error. That
+   * distinction is the whole point: an unset identity is a condition the caller
+   * must act on, not a failure of this lookup.
+   *
+   * No `cwd` resolves against the process's own configuration, which is what the
+   * scaffold's throwaway temp dir inherits.
+   */
+  async configuredIdentity(cwd) {
+    const read2 = async (key) => {
+      try {
+        const { stdout } = await execa2("git", ["config", "--get", key], cwd ? { cwd } : {});
+        return stdout.trim() || null;
+      } catch {
+        return null;
+      }
+    };
+    return { name: await read2("user.name"), email: await read2("user.email") };
+  }
   async isGitRepo(cwd) {
     try {
       await execa2("git", ["rev-parse", "--is-inside-work-tree"], { cwd });
@@ -5652,6 +5675,7 @@ async function runSiblingCreate(github, aws, coreAws, git, config, session, opti
   const totalSteps = 8;
   const { org, repo } = githubRepo(config);
   const pathPrefix = resolvePathPrefix(config);
+  assertGitIdentity(await git.configuredIdentity());
   if (!session.completedSteps.includes("verify_credentials")) {
     log.step(1, totalSteps, "Verifying AWS credentials...");
     await aws.verifyCredentials();
@@ -5944,6 +5968,18 @@ function readExistingSiblingOrigins(filePath) {
     throw err;
   }
 }
+function assertGitIdentity(identity) {
+  const missing = [identity.name ? null : "user.name", identity.email ? null : "user.email"].filter(
+    (v) => v !== null
+  );
+  if (missing.length === 0) return;
+  throw new Error(
+    `git has no ${missing.join(" or ")} configured, so the scaffold commit would fail after the GitHub repository had already been created. Set it first:
+` + missing.map(
+      (k) => `  git config --global ${k} "${k === "user.name" ? "Your Name" : "you@example.com"}"`
+    ).join("\n")
+  );
+}
 function assertCoreSupportsSiblingRouting(cloneDir, coreRepo, pathPrefix = "x") {
   const cdnVarsPath = join20(cloneDir, "modules", "cloud", "aws", "cdn", "variables.tf");
   let declaresSiblingOrigins = false;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@biffo/cli",
-  "version": "0.185.4",
+  "version": "0.186.0",
   "description": "Biffo project scaffolding CLI",
   "license": "MIT",
   "type": "module",