@forwardimpact/libeval 0.1.19 → 0.1.20

package/README.md

@@ -0,0 +1,9 @@
+# libeval
+
+Process Claude Code stream-json output into structured traces.
+
+## Getting Started
+
+```js
+import { createTraceCollector, createTraceQuery, createAgentRunner } from '@forwardimpact/libeval';
+```

package/bin/fit-trace.js

@@ -42,7 +42,8 @@ const definition = {
         },
         repo: {
           type: "string",
-          description: "GitHub repo override (default: git remote)",
+          description:
+            "GitHub repo override (default: $GITHUB_REPOSITORY or 'origin' git remote)",
         },
       },
     },
@@ -55,7 +56,8 @@ const definition = {
         artifact: { type: "string", description: "Artifact name override" },
         repo: {
           type: "string",
-          description: "GitHub repo override (default: git remote)",
+          description:
+            "GitHub repo override (default: $GITHUB_REPOSITORY or 'origin' git remote)",
         },
       },
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forwardimpact/libeval",
-  "version": "0.1.19",
+  "version": "0.1.20",
   "description": "Process Claude Code stream-json output into structured traces",
   "license": "Apache-2.0",
   "author": "D. Olsson <hi@senzilla.io>",

package/src/agent-runner.js

@@ -38,7 +38,7 @@ export class AgentRunner {
    * @param {function} deps.query - SDK query function (injected for testing)
    * @param {import("stream").Writable} deps.output - Stream to emit NDJSON to
    * @param {string} [deps.model] - Claude model identifier
-   * @param {number} [deps.maxTurns] - Maximum agentic turns
+   * @param {number} [deps.maxTurns] - Maximum agentic turns; 0 means unlimited
    * @param {string[]} [deps.allowedTools] - Tools the agent may use
    * @param {function} [deps.onLine] - Callback invoked with each NDJSON line as it's produced
    * @param {function} [deps.onBatch] - Async callback invoked with a batch of NDJSON lines at flush boundaries: every `batchSize` assistant text blocks, the terminal `result` message, and — on iterator crash/abort — once more in a final flush carrying any lines that never reached a boundary. Receives `(lines, { abort })` where calling `abort()` stops the in-flight SDK session via the AbortController. Optional; assignable at runtime so the Supervisor can swap it per turn.
@@ -73,7 +73,8 @@ export class AgentRunner {
         options: {
           cwd: this.cwd,
           allowedTools: this.allowedTools,
-          ...(this.maxTurns > 0 && { maxTurns: this.maxTurns }),
+          maxTurns:
+            this.maxTurns === 0 ? Number.MAX_SAFE_INTEGER : this.maxTurns,
           model: this.model,
           permissionMode: PERMISSION_MODE,
           allowDangerouslySkipPermissions: true,

package/src/index.js

@@ -3,6 +3,7 @@ export { TraceQuery, createTraceQuery } from "./trace-query.js";
 export {
   TraceGitHub,
   createTraceGitHub,
+  detectRepoSlug,
   parseGitRemote,
 } from "./trace-github.js";
 export { AgentRunner, createAgentRunner } from "./agent-runner.js";

package/src/trace-github.js

@@ -1,3 +1,4 @@
+import { execSync } from "node:child_process";
 import { createWriteStream } from "node:fs";
 import { mkdir } from "node:fs/promises";
 import path from "node:path";
@@ -116,7 +117,6 @@ export class TraceGitHub {
     // Stream to disk then extract.
     await pipeline(Readable.fromWeb(response.body), createWriteStream(zipPath));
 
-    const { execSync } = await import("node:child_process");
     execSync(
       `unzip -o -q ${JSON.stringify(zipPath)} -d ${JSON.stringify(dir)}`,
     );
@@ -182,9 +182,51 @@ export function parseGitRemote(remote) {
   const simple = remote.match(/^([^/:@]+)\/([^/]+)$/);
   if (simple) return { owner: simple[1], repo: simple[2] };
 
+  // Generic URL fallback: any remote whose path ends in /owner/repo(.git)?
+  // Covers GitHub Enterprise, proxied git URLs, and mirrors.
+  const generic = remote.match(/[/:]([^/:@?#]+)\/([^/:@?#]+?)(?:\.git)?\/?$/);
+  if (generic) return { owner: generic[1], repo: generic[2] };
+
   throw new Error(`Cannot parse GitHub remote: ${remote}`);
 }
 
+/**
+ * Detect the current GitHub repository slug as `{owner, repo}`.
+ *
+ * Resolution order:
+ *   1. `GITHUB_REPOSITORY` env var (set automatically by GitHub Actions).
+ *   2. `git remote get-url origin` in the current working directory.
+ *
+ * @returns {{owner: string, repo: string}}
+ * @throws {Error} with a clear message if neither source yields a parseable slug.
+ */
+export function detectRepoSlug() {
+  const env = process.env.GITHUB_REPOSITORY;
+  if (env && env.trim()) {
+    return parseGitRemote(env.trim());
+  }
+
+  let remote;
+  try {
+    remote = execSync("git remote get-url origin", {
+      encoding: "utf8",
+      stdio: ["ignore", "pipe", "ignore"],
+    }).trim();
+  } catch {
+    throw new Error(
+      "Cannot detect repository: set --repo <owner/repo>, export GITHUB_REPOSITORY, or run inside a git checkout with an 'origin' remote.",
+    );
+  }
+
+  if (!remote) {
+    throw new Error(
+      "Cannot detect repository: 'git remote get-url origin' returned an empty value. Pass --repo <owner/repo> or set GITHUB_REPOSITORY.",
+    );
+  }
+
+  return parseGitRemote(remote);
+}
+
 /**
  * Create a TraceGitHub instance. The caller is responsible for resolving
  * the GitHub token — typically via `Config.ghToken()` — so credential
@@ -207,16 +249,9 @@ export async function createTraceGitHub(opts = {}) {
     );
   }
 
-  let owner, repo;
-  if (repoOverride) {
-    ({ owner, repo } = parseGitRemote(repoOverride));
-  } else {
-    const { execSync } = await import("node:child_process");
-    const remote = execSync("git remote get-url origin", {
-      encoding: "utf8",
-    }).trim();
-    ({ owner, repo } = parseGitRemote(remote));
-  }
+  const { owner, repo } = repoOverride
+    ? parseGitRemote(repoOverride)
+    : detectRepoSlug();
 
   return new TraceGitHub({ token, owner, repo });
 }