@growthagent/ci 0.5.0 → 0.7.0

package/dist/cli.js

@@ -1,9 +1,9 @@
 #!/usr/bin/env node
 
 // src/cli.ts
-import { execFileSync, spawnSync } from "node:child_process";
-import { cpSync, existsSync as existsSync2, mkdirSync, readFileSync, writeFileSync as writeFileSync2 } from "node:fs";
-import path5 from "node:path";
+import { execFileSync, spawnSync as spawnSync2 } from "node:child_process";
+import { cpSync, existsSync as existsSync3, mkdirSync, readFileSync, writeFileSync as writeFileSync2 } from "node:fs";
+import path6 from "node:path";
 import { fileURLToPath } from "node:url";
 
 // ../github/src/app.ts
@@ -53,6 +53,34 @@ import path3 from "node:path";
 import { promisify } from "node:util";
 var run = promisify(execFile);
 
+// src/lockfiles.ts
+import { spawnSync } from "node:child_process";
+import { existsSync as existsSync2, rmSync as rmSync2 } from "node:fs";
+import path4 from "node:path";
+var LOCKFILES = [
+  "pnpm-lock.yaml",
+  "package-lock.json",
+  "yarn.lock",
+  "bun.lockb",
+  "bun.lock",
+  "npm-shrinkwrap.json"
+];
+function discardLockfileChanges(cwd = process.cwd()) {
+  const git = (args) => spawnSync("git", args, { cwd, encoding: "utf8", stdio: ["ignore", "pipe", "pipe"] });
+  const touched = [];
+  for (const lockfile of LOCKFILES) {
+    if (!existsSync2(path4.join(cwd, lockfile))) continue;
+    const tracked = git(["ls-files", "--error-unmatch", "--", lockfile]).status === 0;
+    if (tracked) {
+      git(["restore", "--staged", "--worktree", "--", lockfile]);
+    } else {
+      rmSync2(path4.join(cwd, lockfile), { force: true });
+    }
+    touched.push(lockfile);
+  }
+  return touched;
+}
+
 // src/patch.ts
 var MAX_CAPTURED_LINES = 400;
 var ALLOWED_DEPENDENCIES = ["posthog-js", "posthog-node"];
@@ -292,11 +320,11 @@ function validatePatch(patch, limits = DEFAULT_PATCH_LIMITS) {
 // src/provider-config.ts
 import { mkdtempSync as mkdtempSync2, writeFileSync } from "node:fs";
 import { tmpdir as tmpdir3 } from "node:os";
-import path4 from "node:path";
+import path5 from "node:path";
 var RUN_TOKEN_ENV = "GX_RUN_TOKEN";
 function writeGoogleProviderConfig(model) {
   const gateway = (process.env.GX_GATEWAY_URL ?? "").replace(/\/$/, "");
-  const dir = mkdtempSync2(path4.join(tmpdir3(), "growth-agent-pi-"));
+  const dir = mkdtempSync2(path5.join(tmpdir3(), "growth-agent-pi-"));
   const config = {
     providers: {
       "growthagent-google": {
@@ -307,7 +335,7 @@ function writeGoogleProviderConfig(model) {
       }
     }
   };
-  writeFileSync(path4.join(dir, "models.json"), JSON.stringify(config, null, 2));
+  writeFileSync(path5.join(dir, "models.json"), JSON.stringify(config, null, 2));
   return dir;
 }
 
@@ -416,14 +444,14 @@ function buildPrompt(fields, briefing) {
 }
 
 // src/cli.ts
-var HERE = path5.dirname(fileURLToPath(import.meta.url));
+var HERE = path6.dirname(fileURLToPath(import.meta.url));
 var PATCH_FILE = "growth-agent.patch";
 function assetsDir() {
-  const packaged = path5.join(HERE, "assets", "skills");
-  return existsSync2(packaged) ? packaged : path5.join(HERE, "..", "assets", "skills");
+  const packaged = path6.join(HERE, "assets", "skills");
+  return existsSync3(packaged) ? packaged : path6.join(HERE, "..", "assets", "skills");
 }
 function sh(cmd, args, opts = {}) {
-  const r = spawnSync(cmd, args, { encoding: "utf8", stdio: ["inherit", "pipe", "inherit"] });
+  const r = spawnSync2(cmd, args, { encoding: "utf8", stdio: ["inherit", "pipe", "inherit"] });
   if (r.status !== 0 && !opts.allowFail) {
     throw new Error(`${cmd} ${args.join(" ")} exited ${r.status}`);
   }
@@ -547,7 +575,7 @@ async function implement() {
   console.log(
     meta.kind === "instrumentation" ? `\u25B8 instrumentation request #${issue.number}` : `\u25B8 change request #${issue.number} \u2014 flag ${meta.flagKey} on ${meta.surface}`
   );
-  const skillsDir = path5.resolve(process.cwd(), ".pi", "skills");
+  const skillsDir = path6.resolve(process.cwd(), ".pi", "skills");
   mkdirSync(skillsDir, { recursive: true });
   cpSync(assetsDir(), skillsDir, { recursive: true });
   const prompt = buildPrompt(meta, extractBriefing(issue.body));
@@ -563,7 +591,7 @@ async function implement() {
   console.log(`\u25B8 running pi (${piProvider}/${model})`);
   let piStatus;
   try {
-    const pi = spawnSync(
+    const pi = spawnSync2(
       "pi",
       ["-p", prompt, "--provider", piProvider, "--model", model, "--thinking", "high"],
       {
@@ -594,6 +622,10 @@ async function implement() {
   const piTracked = sh("git", ["ls-files", "--", ".pi"], { allowFail: true });
   if (piTracked) sh("git", ["checkout", "--", ".pi"], { allowFail: true });
   sh("git", ["clean", "-fdq", ".pi"], { allowFail: true });
+  const discarded = discardLockfileChanges();
+  if (discarded.length) {
+    console.log(`\u25B8 discarded the model's ${discarded.join(", ")}; the trusted job derives it`);
+  }
   sh("git", ["add", "-A"]);
   const patch = sh("git", ["diff", "--staged"]);
   if (!patch.trim()) {
@@ -619,9 +651,9 @@ function regenerateLockfile() {
     { lockfile: "yarn.lock", bin: "yarn", args: ["install", "--mode=update-lockfile"] }
   ];
   for (const m of managers) {
-    if (!existsSync2(m.lockfile)) continue;
+    if (!existsSync3(m.lockfile)) continue;
     console.log(`\u25B8 regenerating ${m.lockfile} from the manifest`);
-    const result = spawnSync(m.bin, m.args, { stdio: ["ignore", "inherit", "inherit"] });
+    const result = spawnSync2(m.bin, m.args, { stdio: ["ignore", "inherit", "inherit"] });
     if (result.status !== 0) {
       console.error(
         `Could not regenerate ${m.lockfile} (${m.bin} exited ${result.status}). The dependency change is in package.json but the lockfile is stale \u2014 run the install locally on this branch before merging.`
@@ -641,7 +673,7 @@ function apply() {
   const meta = validatedMeta(issue);
   const title = safeTitle(issue);
   const branch = branchFor(issue, meta.slug);
-  if (!existsSync2(PATCH_FILE)) {
+  if (!existsSync3(PATCH_FILE)) {
     console.error(`No ${PATCH_FILE} was produced by the implementer job.`);
     return 1;
   }
@@ -674,7 +706,7 @@ function apply() {
 Implements #${issue.number}.
 
 Flag: ${meta.flagKey}`]);
-  sh("git", ["push", "-u", "origin", branch]);
+  sh("git", ["push", "--force", "-u", "origin", branch]);
   const stat2 = sh("git", ["diff", "--stat", "HEAD~1", "HEAD"]);
   const prBody = meta.kind === "instrumentation" ? [
     `Implements #${issue.number}.`,
@@ -705,18 +737,18 @@ Flag: ${meta.flagKey}`]);
     "",
     `Primary metric: \`${meta.primaryMetric}\`.`
   ].join("\n");
+  const prTitle = title.startsWith("[growth]") ? title : `[growth] ${title}`;
   const prUrl = sh("gh", [
     "pr",
     "create",
     "--title",
-    `[growth] ${title}`,
+    prTitle,
     "--body",
     prBody,
     "--head",
-    branch,
-    "--label",
-    "growth-agent"
+    branch
   ]);
+  sh("gh", ["pr", "edit", branch, "--add-label", "growth-agent"], { allowFail: true });
   console.log(`\u25B8 opened ${prUrl}`);
   return 0;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@growthagent/ci",
-  "version": "0.5.0",
+  "version": "0.7.0",
   "private": false,
   "type": "module",
   "bin": {