@maycombhq/atticus 2.0.0-beta.2 → 2.0.0-beta.21

package/dist/{chunk-4RS5KZQD.js → chunk-YAH5IQAL.js}

@@ -9,12 +9,12 @@ import {
   todoList,
   todoRun,
   todoShow
-} from "./chunk-HLJAXMM4.js";
-import "./chunk-QSXK2RGH.js";
+} from "./chunk-ZFQMMU5Y.js";
+import "./chunk-DDJPM5YX.js";
 import "./chunk-VOGCSYOI.js";
-import "./chunk-7JUKDRSZ.js";
-import "./chunk-S2A7YC3Q.js";
-import "./chunk-D3PO4DOZ.js";
+import "./chunk-4KNOJC6J.js";
+import "./chunk-PAINMSID.js";
+import "./chunk-ODMBPDDV.js";
 import "./chunk-TV45EUGM.js";
 export {
   executorsCheck,

package/dist/{chunk-64EQYWCA.js → chunk-YBOVNMHC.js}

@@ -6,12 +6,12 @@ var __atticusPath = (p) => __atticusFileURLToPath(__atticusUrl(p));
 import {
   diffCatalogs,
   readCatalog
-} from "./chunk-27NKUS3X.js";
-import "./chunk-L24C55VN.js";
-import "./chunk-PAMPPUR6.js";
-import "./chunk-23W7VWKR.js";
-import "./chunk-7EKXJZ2Y.js";
-import "./chunk-MC3H7736.js";
+} from "./chunk-4SLBCEPD.js";
+import "./chunk-64AADMTW.js";
+import "./chunk-JYMIB5XY.js";
+import "./chunk-CPK7QSBG.js";
+import "./chunk-YO2PKTXT.js";
+import "./chunk-TWQRPM76.js";
 import {
   baseClonePath,
   branchSlug,
@@ -19,22 +19,24 @@ import {
   gitUser,
   loadRepos,
   remoteDefaultBranch
-} from "./chunk-EFHQ3TWP.js";
-import "./chunk-GQ4N4WKF.js";
-import "./chunk-CNUKCS2T.js";
+} from "./chunk-XOL3GFW2.js";
+import "./chunk-QHJJMZLX.js";
+import "./chunk-7AB6OWTV.js";
 import {
   ManifestEditError,
   REPO_DESCRIPTION_MAX,
   RepoDescription,
   RepoManifest,
   catalogDirOf,
+  eolOf,
+  inEol,
   orgPackDirOf,
   require_dist,
   setManifestDescription
-} from "./chunk-S2A7YC3Q.js";
+} from "./chunk-PAINMSID.js";
 import {
-  writeFileAtomic
-} from "./chunk-D3PO4DOZ.js";
+  writeFileIfUnchanged
+} from "./chunk-ODMBPDDV.js";
 import {
   __toESM
 } from "./chunk-TV45EUGM.js";
@@ -78,6 +80,8 @@ async function repoBranches(root) {
       ...remoteDefault ? { remoteDefault } : {},
       branches: r.branches,
       ...commit ? { commit } : {},
+      // Seventh UX walk: the build command a task's PR waits on, shown and set on Repos.
+      ...r.build ? { build: r.build } : {},
       scans: scans.flatMap((s) => s.repos.filter((x) => x.id === r.id).map((x) => ({ branch: s.branch, scannedAt: s.scannedAt, commit: x.commit })))
     });
   }
@@ -116,9 +120,12 @@ async function editRepoBranches(root, repoId, set) {
   const repo = (await loadRepos(root)).find((r) => r.id === repoId);
   if (!repo)
     throw new RepoEditError("not-found", `There is no repo ${repoId} in any manifest.`);
+  if (set.was && (set.was.defaultBranch !== repo.defaultBranch || set.was.branches.join("\n") !== repo.branches.join("\n")))
+    throw new RepoEditError("conflict", `${repoId}'s branches changed in the manifest since you opened the form (now ${repo.defaultBranch}, ${repo.branches.join(", ") || "no others"}): open Edit again to see them, then save.`);
   const rel = `org-pack/profiles/${repo.platform}/repos.manifest.yaml`;
   const file = join(orgPackDirOf(root), "profiles", repo.platform, "repos.manifest.yaml");
-  const doc = (0, import_yaml.parseDocument)(await readFile(file, "utf8"));
+  const was = await readFile(file, "utf8");
+  const doc = (0, import_yaml.parseDocument)(was);
   const list = doc.get("repos");
   const item = (0, import_yaml.isSeq)(list) ? list.items.find((x) => (0, import_yaml.isMap)(x) && x.get("id") === repoId) : void 0;
   if (!(0, import_yaml.isMap)(item))
@@ -138,13 +145,42 @@ async function editRepoBranches(root, repoId, set) {
   const check = RepoManifest.safeParse(doc.toJS());
   if (!check.success)
     throw new RepoEditError("invalid", check.error.issues.map((i) => `${i.path.join(".")}: ${i.message}`).join("; "));
-  await writeFileAtomic(file, doc.toString({ flowCollectionPadding: true }));
+  await writeFileIfUnchanged(file, was, inEol(doc.toString({ flowCollectionPadding: true }), eolOf(was)));
   return rel;
 }
+async function editRepoBuild(root, repoId, set) {
+  const repo = (await loadRepos(root)).find((r) => r.id === repoId);
+  if (!repo)
+    throw new RepoEditError("not-found", `There is no repo ${repoId} in any manifest.`);
+  if (set.was !== void 0 && set.was !== (repo.build ?? ""))
+    throw new RepoEditError("conflict", `${repoId}'s build command changed in the manifest since you opened the form (now ${repo.build ? `\`${repo.build}\`` : "none"}): open it again to see it, then save.`);
+  const build = set.build.trim();
+  if (/[\r\n]/.test(build))
+    throw new RepoEditError("invalid", "A build command is one line: join the steps with && (for example npm ci && npm test).");
+  const rel = `org-pack/profiles/${repo.platform}/repos.manifest.yaml`;
+  const file = join(orgPackDirOf(root), "profiles", repo.platform, "repos.manifest.yaml");
+  const was = await readFile(file, "utf8");
+  const doc = (0, import_yaml.parseDocument)(was);
+  const list = doc.get("repos");
+  const item = (0, import_yaml.isSeq)(list) ? list.items.find((x) => (0, import_yaml.isMap)(x) && x.get("id") === repoId) : void 0;
+  if (!(0, import_yaml.isMap)(item))
+    throw new RepoEditError("not-found", `${rel} doesn't list ${repoId}.`);
+  if (build)
+    item.set("build", build);
+  else
+    item.delete("build");
+  const check = RepoManifest.safeParse(doc.toJS());
+  if (!check.success)
+    throw new RepoEditError("invalid", check.error.issues.map((i) => `${i.path.join(".")}: ${i.message}`).join("; "));
+  await writeFileIfUnchanged(file, was, inEol(doc.toString({ flowCollectionPadding: true }), eolOf(was)));
+  return { file: rel, ...build ? { build } : {} };
+}
 async function editRepoDescription(root, repoId, set, o = {}) {
   const repo = (await loadRepos(root)).find((r) => r.id === repoId);
   if (!repo)
     throw new RepoEditError("not-found", `There is no repo ${repoId} in any manifest.`);
+  if (set.was !== void 0 && set.was !== (repo.description?.text ?? ""))
+    throw new RepoEditError("conflict", `What ${repoId} is changed in the manifest since you opened the form: open Edit again to see it, then save.`);
   const now = o.now ?? /* @__PURE__ */ new Date();
   let next;
   if (!set.reject) {
@@ -175,6 +211,7 @@ export {
   RepoEditError,
   compareBranch,
   editRepoBranches,
+  editRepoBuild,
   editRepoDescription,
   repoBranches
 };