workproof 0.1.0 → 0.1.1

package/dist/src/analyse.js

@@ -1,7 +1,7 @@
 import { createHash } from "node:crypto";
 import { basename } from "node:path";
 import { createRequire } from "node:module";
-import { listCommits, listTags, rootCommit, headSha, remoteUrl } from "./git.js";
+import { listCommits, listTags, rootCommit, headSha, remoteUrl, assertRepository } from "./git.js";
 import { resolveIdentity } from "./figures/identity.js";
 import { tenure, commitShare } from "./figures/commits.js";
 import { cadence } from "./figures/cadence.js";
@@ -23,6 +23,7 @@ const survivingVersion = () => {
     }
 };
 export async function analyseRepo(cwd, params) {
+    await assertRepository(cwd);
     const all = await listCommits(cwd, {});
     const id = await resolveIdentity(all, params.author, cwd);
     const t = tenure(all, id, { ...(params.since ? { since: params.since } : {}), ...(params.until ? { until: params.until } : {}) });

package/dist/src/figures/identity.js

@@ -1,13 +1,15 @@
-import { configuredEmail } from "../git.js";
+import { configuredEmail, configuredName } from "../git.js";
 /**
  * Resolve the author to report on. `author` entries match mailmapped emails or
  * names, case-insensitively; with none given, the repository's configured
  * user.email is used.
  */
 export async function resolveIdentity(commits, author, cwd) {
-    const wanted = (author && author.length ? author : [await configuredEmail(cwd)]).map((a) => a.toLowerCase()).filter(Boolean);
+    const explicit = author && author.length ? author : [];
+    // Without --author, try the configured email, then the configured name.
+    const wanted = (explicit.length ? explicit : [await configuredEmail(cwd), await configuredName(cwd)]).map((a) => a.toLowerCase()).filter(Boolean);
     if (!wanted.length)
-        throw new Error("no author given and git config user.email is empty; pass --author");
+        throw new Error(`no author given and git config has no user.email or user.name; pass --author. ${authorsHint(commits)}`);
     const emails = new Set();
     const names = new Set();
     for (const c of commits) {
@@ -17,7 +19,15 @@ export async function resolveIdentity(commits, author, cwd) {
         }
     }
     if (!emails.size)
-        throw new Error(`no commits by ${wanted.join(", ")} in this repository`);
+        throw new Error(`no commits by ${wanted.join(" or ")} in this repository. ${authorsHint(commits)}`);
     return { emails: [...emails].sort(), names: [...names].sort() };
 }
+/** The most frequent author names, so the error tells the user what to pass. */
+function authorsHint(commits) {
+    const counts = new Map();
+    for (const c of commits)
+        counts.set(c.name, (counts.get(c.name) ?? 0) + 1);
+    const top = [...counts.entries()].sort((a, b) => b[1] - a[1]).slice(0, 5).map(([name, n]) => `"${name}" (${n})`);
+    return top.length ? `Authors here: ${top.join(", ")}. Pass one with --author.` : "The repository has no commits.";
+}
 export const isMine = (c, id) => id.emails.includes(c.email);

package/dist/src/git.d.ts

@@ -26,4 +26,7 @@ export declare function listTags(cwd: string): Promise<Tag[]>;
 export declare const rootCommit: (cwd: string) => Promise<string>;
 export declare const headSha: (cwd: string) => Promise<string>;
 export declare function remoteUrl(cwd: string): Promise<string>;
+export declare function configuredName(cwd: string): Promise<string>;
+/** Throws a plain sentence when the directory is not inside a git repository. */
+export declare function assertRepository(cwd: string): Promise<void>;
 export declare function configuredEmail(cwd: string): Promise<string>;

package/dist/src/git.js

@@ -66,6 +66,23 @@ export async function remoteUrl(cwd) {
         return "";
     }
 }
+export async function configuredName(cwd) {
+    try {
+        return (await git(["config", "--get", "user.name"], cwd)).trim();
+    }
+    catch {
+        return "";
+    }
+}
+/** Throws a plain sentence when the directory is not inside a git repository. */
+export async function assertRepository(cwd) {
+    try {
+        await git(["rev-parse", "--is-inside-work-tree"], cwd);
+    }
+    catch {
+        throw new Error(`${cwd} is not inside a git repository (use --repo to point at one)`);
+    }
+}
 export async function configuredEmail(cwd) {
     try {
         return (await git(["config", "--get", "user.email"], cwd)).trim().toLowerCase();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "workproof",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Turn a private git repository into a verifiable engineering report for one author, without showing any code.",
   "type": "module",
   "main": "./dist/src/index.js",