@arjunkhera/atlas 0.2.0 → 0.2.2

package/.claude-plugin/plugin.json

@@ -1,8 +1,10 @@
 {
   "name": "atlas",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "Atlas: the delivery lifecycle, its crews and the Atlas tools, for any repository.",
-  "author": { "name": "Arjun Khera" },
+  "author": {
+    "name": "Arjun Khera"
+  },
   "repository": "https://github.com/Arjunkhera/atlas",
   "license": "UNLICENSED",
   "userConfig": {

package/door/lib/pull-request.mjs

@@ -15,7 +15,9 @@ import { checkChanges, HELPER_LABEL, PROVENANCE_SOURCES } from './helpers.mjs';
 
 export { HELPER_LABEL, PROVENANCE_SOURCES };
 
-const escapeCell = (text) => String(text ?? '').replaceAll('|', '\\|').replaceAll('\n', ' ').trim();
+// A backtick breaks a table cell the same way a pipe does, and a provenance
+// line often carries one. Both are escaped (acceptance review of slice B2).
+const escapeCell = (text) => String(text ?? '').replaceAll('|', '\\|').replaceAll('`', '\\`').replaceAll('\n', ' ').trim();
 
 // One request may reach two helpers. "We now deploy prod on Fly" reaches Facts
 // and Procedures. It is still ONE pull request: design page D2's rule, carried
@@ -62,7 +64,7 @@ export function provenanceBody({ helper, helpers, why, changes, groups, repo = n
     '2. An agent made every change. No person edited these files by hand.',
     `3. "Source" is one of: ${PROVENANCE_SOURCES.join(', ')}.`,
     '',
-    `Label: \`${HELPER_LABEL}\`. Authority: Arjunkhera/atlas, \Atlas amendment A1.6\`, section 5, slice L2 stage one.`,
+    `Label: \`${HELPER_LABEL}\`. Authority: Arjunkhera/atlas, amendment A1.6, section 5, slice L2 stage one.`,
     '',
   );
   return lines.join('\n');
@@ -88,7 +90,7 @@ const run = (command, args, cwd) => execFileSync(command, args, { cwd, encoding:
 //
 // It never merges. A person merges every file a helper writes, because every
 // one of them steers agents.
-export function open({ root, helperId, changes, groups, why, branch, title, base = null, label = HELPER_LABEL, dryRun = false }) {
+export function open({ root, helperId, changes, groups, why, branch, title, base = null, label = HELPER_LABEL, dryRun = false, exec = run }) {
   const wanted = groups ?? [{ helperId, changes }];
   const checked = checkAll(wanted);
   const all = checked.flatMap((group) => group.changes);
@@ -96,20 +98,24 @@ export function open({ root, helperId, changes, groups, why, branch, title, base
   // printed "Nothing was pushed", which was true and misleading (acceptance
   // review of batch 16, finding S3-5).
   if (!dryRun) for (const group of checked) applyChanges(root, group.helper.id, group.changes);
-  const repo = (() => { try { return run('gh', ['repo', 'view', '--json', 'nameWithOwner', '--jq', '.nameWithOwner'], root); } catch { return null; } })();
+  const repo = (() => { try { return exec('gh', ['repo', 'view', '--json', 'nameWithOwner', '--jq', '.nameWithOwner'], root); } catch { return null; } })();
   const body = provenanceBody({ groups: checked, why, repo });
-  const onto = base ?? run('git', ['symbolic-ref', '--short', 'HEAD'], root);
+  const onto = base ?? exec('git', ['symbolic-ref', '--short', 'HEAD'], root);
   const helper = checked[0].helper;
   if (dryRun) return { helper, helpers: checked.map((group) => group.helper), repo, branch, title, body, base: onto, written: all.map((change) => change.path), opened: false };
 
-  run('git', ['checkout', '-b', branch], root);
-  run('git', ['add', '--', ...all.map((change) => change.path)], root);
-  run('git', ['commit', '-m', title, '-m', body], root);
-  run('git', ['push', '-u', 'origin', branch], root);
+  exec('git', ['checkout', '-b', branch], root);
+  exec('git', ['add', '--', ...all.map((change) => change.path)], root);
+  exec('git', ['commit', '-m', title, '-m', body], root);
+  exec('git', ['push', '-u', 'origin', branch], root);
   // A label that does not exist yet makes `gh pr create` fail, so it is made
   // first. A label that already exists makes that call fail instead, and that
   // failure is the expected one.
-  try { run('gh', ['label', 'create', label, '--description', 'Written by an Atlas helper. A person merges it.', '--color', 'C5A882'], root); } catch { /* it already exists */ }
-  const url = run('gh', ['pr', 'create', '--title', title, '--body', body, '--label', label, '--base', onto], root);
+  try { exec('gh', ['label', 'create', label, '--description', 'Written by an Atlas helper. A person merges it.', '--color', 'C5A882'], root); } catch { /* it already exists */ }
+  // --head names the branch. `git push -u` does not always record the upstream
+  // in a shallow clone, and then `gh pr create` refuses with "you must first
+  // push the current branch" although the branch is already on GitHub
+  // (measured on an Engram clone, 2026-09-20).
+  const url = exec('gh', ['pr', 'create', '--title', title, '--body', body, '--label', label, '--base', onto, '--head', branch], root);
   return { helper, helpers: checked.map((group) => group.helper), repo, branch, title, body, base: onto, written: all.map((change) => change.path), opened: true, url };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arjunkhera/atlas",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "Atlas: the delivery lifecycle, its crews and the Atlas tools, as a Claude Code plugin for any repository.",
   "type": "module",
   "license": "UNLICENSED",