@arjunkhera/atlas 0.3.50 → 0.3.52

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "atlas",
-  "version": "0.3.50",
+  "version": "0.3.52",
   "description": "Atlas: the delivery lifecycle, its crews and the Atlas tools, for any repository.",
   "author": {
     "name": "Arjun Khera"

package/door/cli.mjs

@@ -137,6 +137,10 @@ const HELP = `atlas — the door into a repo's Atlas files
       --root <repo>                    the checkout to read atlas.yaml from (default: this folder)
       --dry-run                        print each call; change nothing
       --off | --on                     break glass: turn both rulesets off, or back on
+      On a GitHub plan with no rulesets, apply changes nothing and reports advisory mode.
+  atlas gate ready --repo <owner/name> --pr <n>   may this pull request merge: every required check green on its head
+  atlas gate audit --repo <owner/name> [--since <date>]   each merge since the date (default: one day) that came before its checks were green
+  atlas gate audit --repo <owner/name> --commit <sha>   one commit on the main branch: red when no pull request merged it, or one merged it before its checks were green
 
   atlas judge seal <tasks.md>          the fresh-agent test: check the judge's six candidates, print the seal and the three that run
   atlas judge brief <tasks.md> <id>    print the exact words for one run: the task, then the report form
@@ -177,7 +181,7 @@ export const FLAGS = Object.freeze({
   view: { product: 'value', name: 'value', out: 'value' },
   tests: { root: 'optional', halves: 'value', evidence: 'value', 'guards-from': 'value', tests: 'value', 'dry-run': 'switch', run: 'value', own: 'value', base: 'value', head: 'value', list: 'value', side: 'value', old: 'value', new: 'value', covers: 'switch', findings: 'value', repo: 'value', max: 'value', bot: 'value', since: 'value', out: 'value', text: 'switch', json: 'switch', scenario: 'value', language: 'value' },
   upkeep: { root: 'optional', event: 'value', json: 'switch' },
-  gate: { repo: 'value', root: 'optional', 'dry-run': 'switch', off: 'switch', on: 'switch' },
+  gate: { repo: 'value', root: 'optional', 'dry-run': 'switch', off: 'switch', on: 'switch', pr: 'value', since: 'value', commit: 'value' },
   judge: { tasks: 'value', verdict: 'value', seal: 'value', runs: 'value', out: 'value', repo: 'value', budget: 'value', model: 'value' },
   install: { local: 'switch', from: 'value', 'skip-global': 'switch', yes: 'switch', cloud: 'switch' },
   upgrade: { version: 'optional', yes: 'switch' },
@@ -414,7 +418,7 @@ function upkeepVerb(chosen) {
 }
 
 function gateVerb(chosen) {
-  return gateCommand({ what: chosen._[1], repo: chosen.repo, root: rootOf(chosen), dryRun: Boolean(chosen['dry-run']), off: Boolean(chosen.off), on: Boolean(chosen.on) }, line);
+  return gateCommand({ what: chosen._[1], repo: chosen.repo, root: rootOf(chosen), dryRun: Boolean(chosen['dry-run']), off: Boolean(chosen.off), on: Boolean(chosen.on), pr: chosen.pr, since: chosen.since, commit: chosen.commit }, line);
 }
 
 function judgeVerb(chosen) {

package/door/lib/gate.mjs

@@ -6,6 +6,12 @@
 //   2. "atlas release tags" on v* tags: no update, no delete.
 // The bypass list stays empty. --off and --on are the break glass.
 //
+// A private repo on a free GitHub plan gets no rulesets. The owner chose on 24
+// October 2026 to stay on that plan (decision dd90d8e8). There the
+// gate is advisory: apply changes nothing and says so, `atlas gate ready`
+// says if one pull request may merge, and `atlas gate audit` names each merge
+// that came before its checks were green (design docs/design-docs/gate-advisory/).
+//
 // The plan is built by pure functions, so a test reads it without GitHub. The
 // gh calls sit in one place, behind `gh`, which a test replaces.
 import { execFileSync } from 'node:child_process';
@@ -26,7 +32,11 @@ export class GateError extends Error {}
 export function gateFacts(root) {
   const path = join(root, 'atlas.yaml');
   if (!existsSync(path)) throw new GateError(`there is no atlas.yaml in ${root}. Nothing was changed.`);
-  const facts = parseYaml(readFileSync(path, 'utf8')) ?? {};
+  return factsOf(readFileSync(path, 'utf8'));
+}
+
+function factsOf(text) {
+  const facts = parseYaml(text) ?? {};
   const checks = facts.required_checks;
   if (!Array.isArray(checks) || checks.length === 0 || checks.some((name) => typeof name !== 'string' || !name.trim())) {
     throw new GateError('atlas.yaml lists no required_checks. Nothing was changed.');
@@ -141,20 +151,135 @@ export function compare(wanted, got) {
 export function ghApi(method, path, body) {
   const args = ['api', '--method', method, path, '-H', 'Accept: application/vnd.github+json'];
   if (body !== undefined) args.push('--input', '-');
-  const out = execFileSync('gh', args, { encoding: 'utf8', input: body === undefined ? undefined : JSON.stringify(body), env: childEnvironment(), stdio: ['pipe', 'pipe', 'pipe'] });
+  const out = execFileSync('gh', args, { encoding: 'utf8', input: body === undefined ? undefined : JSON.stringify(body), env: childEnvironment(), stdio: ['pipe', 'pipe', 'pipe'], maxBuffer: 64 * 1024 * 1024 });
   return out.trim() ? JSON.parse(out) : null;
 }
 
+// GitHub's answer when the repo's plan has no rulesets.
+const PLAN_REFUSAL = /Upgrade to GitHub Pro|make this repository public/i;
+export const ADVISORY = 'advisory mode: the checks run on each pull request, but GitHub does not block a merge on this plan. Merge only when "atlas gate ready" says READY.';
+
 const explain = (error) => {
   const text = `${error.stderr ?? ''}${error.message ?? ''}`;
   if (/HTTP 403|Upgrade to GitHub Pro|403/.test(text)) return 'GitHub refused the ruleset (403). A private repo needs GitHub Pro for rulesets, and only a repo admin may set one.';
   return text.trim().split('\n').slice(-3).join(' ');
 };
 
-// `atlas gate apply`. Returns the exit code.
-export function gateCommand({ what, repo, root, dryRun = false, off = false, on = false }, line, gh = ghApi) {
-  if (what !== 'apply') throw new GateError(`there is no gate verb "${what ?? ''}". Use apply.`);
-  if (!repo || repo === true || !/^[A-Za-z0-9_.-]+\/[A-Za-z0-9_.-]+$/.test(repo)) throw new GateError('atlas gate apply needs --repo <owner/name>.');
+// The state of each required check on one commit, from its GitHub Actions
+// check runs. `at` is a time: a run that ended after it does not count.
+// A check is green only when its newest run from GitHub Actions ended in success.
+export function checkStates(checks, runs, at = null) {
+  return checks.map((name) => {
+    const mine = (runs ?? [])
+      .filter((run) => run.name === name && run.app?.id === ACTIONS_APP_ID)
+      .filter((run) => at === null || (run.started_at ?? run.completed_at ?? '') <= at)
+      .sort((a, b) => String(b.started_at ?? '').localeCompare(String(a.started_at ?? '')));
+    const newest = mine[0];
+    if (!newest) return { name, state: 'missing' };
+    if (newest.status !== 'completed' || (at !== null && (newest.completed_at ?? '') > at)) return { name, state: 'pending' };
+    return { name, state: newest.conclusion === 'success' ? 'green' : 'red', conclusion: newest.conclusion };
+  });
+}
+
+// The required checks as atlas.yaml listed them at one commit on GitHub. Ready
+// reads the base branch, so a pull request cannot drop its own red check, and
+// audit reads the base of each merge, so a later rename gives no false alarm.
+function checksAt(gh, repo, ref) {
+  const file = gh('GET', `repos/${repo}/contents/atlas.yaml?ref=${encodeURIComponent(ref)}`);
+  if (typeof file?.content !== 'string') throw new GateError(`there is no atlas.yaml in ${repo} at ${ref}.`);
+  return factsOf(Buffer.from(file.content, 'base64').toString('utf8')).checks;
+}
+
+const checkRuns = (gh, repo, sha) => gh('GET', `repos/${repo}/commits/${sha}/check-runs?per_page=100`)?.check_runs ?? [];
+const checkLine = (row) => `  ${row.state === 'green' ? '✓' : '✗'} ${row.state.padEnd(8)} ${row.name}${row.conclusion && row.state === 'red' ? ` (${row.conclusion})` : ''}`;
+
+// `atlas gate ready --pr <n>`: may this pull request merge now? Exit 0 when
+// every required check is green on its head commit, 1 when not.
+export function readyCommand({ repo, pr }, line, gh = ghApi) {
+  if (!pr || pr === true || !/^\d+$/.test(String(pr))) throw new GateError('atlas gate ready needs --pr <number>.');
+  const pull = gh('GET', `repos/${repo}/pulls/${pr}`);
+  if (pull?.state !== 'open') { line(`atlas gate ready: pull request #${pr} is ${pull?.merged ? 'merged' : pull?.state ?? 'not found'}. NOT READY.`); return 1; }
+  const sha = pull.head.sha;
+  const checks = checksAt(gh, repo, pull.base.ref);
+  const rows = checkStates(checks, checkRuns(gh, repo, sha));
+  line(`atlas gate ready: #${pr} at ${sha.slice(0, 7)}, ${checks.length} required check(s) from atlas.yaml on ${pull.base.ref}.`);
+  for (const row of rows) line(checkLine(row));
+  const ready = rows.every((row) => row.state === 'green');
+  line(ready ? 'READY: every required check is green. Merge with a merge commit, at this head.' : 'NOT READY: do not merge. Wait for a pending check; fix a red or missing one.');
+  return ready ? 0 : 1;
+}
+
+// `atlas gate audit --since <date>`: each pull request merged since the date
+// whose required checks were not all green at the moment of the merge.
+export function auditCommand({ repo, since, commit }, line, gh = ghApi) {
+  if (commit !== undefined) return commitAudit({ repo, commit }, line, gh);
+  const from = since && since !== true ? since : new Date(Date.now() - 864e5).toISOString().slice(0, 10);
+  if (Number.isNaN(Date.parse(from))) throw new GateError(`--since must be a date, not "${from}".`);
+  const closed = gh('GET', `repos/${repo}/pulls?state=closed&sort=updated&direction=desc&per_page=100`) ?? [];
+  const merged = closed.filter((pull) => pull.merged_at && pull.merged_at >= new Date(from).toISOString());
+  line(`atlas gate audit: ${merged.length} pull request(s) merged into ${repo} since ${from}.`);
+  if (closed.length >= 100 && (closed.at(-1)?.updated_at ?? '') >= new Date(from).toISOString()) line('  (only the 100 closed pull requests updated last were read; give a later --since to read all of them)');
+  const lists = new Map();
+  const checksOf = (sha) => {
+    if (!lists.has(sha)) { try { lists.set(sha, checksAt(gh, repo, sha)); } catch (error) { if (!(error instanceof GateError)) throw error; lists.set(sha, null); } }
+    return lists.get(sha);
+  };
+  let red = 0;
+  let ungated = 0;
+  for (const pull of merged) {
+    const checks = checksOf(pull.base.sha);
+    if (!checks) { ungated += 1; continue; }
+    const rows = checkStates(checks, checkRuns(gh, repo, pull.head.sha), pull.merged_at);
+    const bad = rows.filter((row) => row.state !== 'green');
+    if (!bad.length) continue;
+    red += 1;
+    line(`  ✗ #${pull.number} merged ${pull.merged_at} before its checks were green: ${bad.map((row) => `${row.name} ${row.state}`).join(', ')}`);
+  }
+  if (ungated) line(`  (${ungated} merge(s) had a base whose atlas.yaml listed no required_checks; nothing to audit there)`);
+  line(red ? `RED MERGES: ${red}. Look at each one now; the main branch may hold an untested change.` : 'every merge came after its required checks were green.');
+  return red ? 1 : 0;
+}
+
+// `atlas gate audit --commit <sha>`: one commit that reached the main branch.
+// The push job runs it on each push. A commit is good when a pull request
+// merged it after that pull request's required checks were green. A commit
+// that no pull request merged was pushed past the gate, so it is red.
+function commitAudit({ repo, commit }, line, gh) {
+  if (!commit || commit === true || !/^[0-9a-f]{7,40}$/i.test(String(commit))) throw new GateError('--commit needs a commit SHA.');
+  // GitHub expands a short SHA, and names the first parent: the main branch
+  // just before this commit, whose atlas.yaml held the gate at that moment.
+  const found = gh('GET', `repos/${repo}/commits/${commit}`);
+  const sha = found?.sha;
+  if (!sha) throw new GateError(`there is no commit ${commit} in ${repo}.`);
+  const pull = (gh('GET', `repos/${repo}/commits/${sha}/pulls`) ?? []).find((each) => each.merged_at && each.merge_commit_sha === sha);
+  if (!pull) {
+    line(`- ✗ ${sha.slice(0, 7)} reached the main branch with no pull request: a push past the gate.`);
+    line('RED PUSH: look at it now; the main branch may hold an untested change.');
+    return 1;
+  }
+  const parent = found.parents?.[0]?.sha ?? pull.base.sha;
+  let checks;
+  try { checks = checksAt(gh, repo, parent); } catch (error) {
+    if (!(error instanceof GateError)) throw error;
+    line(`- atlas gate audit: ${sha.slice(0, 7)} is the merge of #${pull.number}; atlas.yaml before it listed no required_checks, so nothing to audit.`);
+    return 0;
+  }
+  const bad = checkStates(checks, checkRuns(gh, repo, pull.head.sha), pull.merged_at).filter((row) => row.state !== 'green');
+  if (bad.length) {
+    line(`- ✗ #${pull.number} merged ${pull.merged_at} before its checks were green: ${bad.map((row) => `${row.name} ${row.state}`).join(', ')}`);
+    line('RED MERGES: 1. Look at it now; the main branch may hold an untested change.');
+    return 1;
+  }
+  line(`- atlas gate audit: ${sha.slice(0, 7)} is the merge of #${pull.number}, after its ${checks.length} required check(s) were green.`);
+  return 0;
+}
+
+// `atlas gate apply|ready|audit`. Returns the exit code.
+export function gateCommand({ what, repo, root, dryRun = false, off = false, on = false, pr, since, commit }, line, gh = ghApi) {
+  if (!['apply', 'ready', 'audit'].includes(what)) throw new GateError(`there is no gate verb "${what ?? ''}". Use apply, ready or audit.`);
+  if (!repo || repo === true || !/^[A-Za-z0-9_.-]+\/[A-Za-z0-9_.-]+$/.test(repo)) throw new GateError(`atlas gate ${what} needs --repo <owner/name>.`);
+  if (what === 'ready') return readyCommand({ repo, pr }, line, gh);
+  if (what === 'audit') return auditCommand({ repo, since, commit }, line, gh);
   if (off && on) throw new GateError('pick --off or --on, not both.');
   const mode = off ? 'off' : on ? 'on' : 'apply';
   const facts = mode === 'apply' ? gateFacts(root) : null;
@@ -163,6 +288,11 @@ export function gateCommand({ what, repo, root, dryRun = false, off = false, on
   line(`GET    repos/${repo}/rulesets`);
   let existing;
   try { existing = gh('GET', `repos/${repo}/rulesets`) ?? []; } catch (error) {
+    if (mode === 'apply' && PLAN_REFUSAL.test(`${error.stderr ?? ''}${error.message ?? ''}`)) {
+      line(`atlas gate apply: ${ADVISORY}`);
+      line('Nothing was changed. Auto-merge stays off: with no ruleset, GitHub would merge at once, before the checks end.');
+      return 0;
+    }
     if (!dryRun) { line(`atlas gate apply: ${explain(error)} Nothing was changed.`); return 1; }
     line(`       (the read failed: ${explain(error)} The plan below assumes no ruleset exists.)`);
     existing = [];

package/package.json

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

package/skills/lead/SKILL.md

@@ -105,7 +105,7 @@ set. A test keeps the two lists equal.
 | `atlas design` | `folder` to learn where designs live; `check` before you show a design; `build` to make its page |
 | `atlas view` | `build --product <id>` makes the project view page of one product, when the owner asks for it or after an Atlas release changes it. Publish it with only the grant the command prints |
 | `atlas upkeep` | `check` before you ask for a merge: the docs index, dead doc links, and the tracker item a pull request names. The daily upkeep run calls it on the main branch |
-| `atlas gate` | `apply` once per repo, by the owner on his Mac, after the repo's `required_checks` exist. `--dry-run` first. `--off` and `--on` are the break glass |
+| `atlas gate` | `apply` once per repo, by the owner on his Mac, after the repo's `required_checks` exist. `--dry-run` first. `--off` and `--on` are the break glass. On a plan with no rulesets it reports advisory mode. `ready` before each merge. `audit` in the daily upkeep run, and with `--commit` in the push job of a repo that has one |
 | `atlas judge` | In the fresh-agent test only, as the `atlas:fresh-agent-test` skill says. `seal` after the pick, `brief` to read a sealed ask, `run` for each sealed task, `check` after the grade |
 
 A person merges every file that `atlas tooling` writes.

package/skills/sdlc-task/SKILL.md

@@ -144,6 +144,15 @@ A lock is the owner's approval of a short design.
    posts its own proof comment. When it returns a comment file instead,
    post that file unchanged. Never edit that comment. Ask the owner for the
    merge only after the proof is there, and show the proof first.
+8. **Merge only on green.** On a plan with no rulesets, GitHub blocks
+   nothing, so this step is the gate (owner, 24 October 2026).
+   1. Right before the merge, run `atlas gate` with the verb `ready`,
+      `--repo` and `--pr`.
+   2. A pending check: wait for it. A red or missing check: fix it.
+   3. Merge only when it prints READY. Name the head SHA that it read in
+      the merge call, so a later push cannot slip in.
+   4. Where the verb `apply` set the rulesets, the branch rule holds the
+      merge. Turn on auto-merge there, and never press merge.
 
 ## pause
 

package/skills/upkeep/SKILL.md

@@ -65,6 +65,10 @@ Pass the product to every read. Show no other product.
 4. **Read only, report only.** Call `sweep_check` and `needs_me` for the
    product. Put any mismatch, and any question older than seven days, in the
    report. Do not act on them.
+5. **Red merges.** When `atlas.yaml` lists `required_checks`, run
+   `atlas gate` with the verb `audit`, `--repo` and `--since` set to
+   yesterday's date. Put each merge it names in the report, under "Left for
+   the owner", with its pull request. Do not revert it.
 
 If the tracker cannot be reached, skip this half and say so.
 
@@ -106,7 +110,7 @@ Upkeep <date> · Atlas <version>
 Marked: <n> (<item ids and pull requests>)
 Next steps fixed: <n> · Comments: <n> · Issues closed: <n>
 Docs: <n> fixes, <merged #n | open #n, why | none>
-Left for the owner: <sweep mismatch, old questions, unclear links, or none>
+Left for the owner: <red merges, sweep mismatch, old questions, unclear links, or none>
 ```
 
 ## Example