@arjunkhera/atlas 0.4.16 → 0.4.17

package/.claude-plugin/plugin.json

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

package/door/lib/doc-sources.mjs

@@ -96,7 +96,6 @@ export function docUnits(root) {
   const units = [];
   const folders = new Set();
   for (const file of markdownFiles(root, 'docs')) {
-    if (file.startsWith('docs/site/')) continue;
     const folder = designFolderOf(root, file);
     if (folder) {
       if (folders.has(folder)) continue;

package/door/lib/kind-drift.mjs

@@ -3,7 +3,7 @@
 // Milestone M2, slice L2 stage one: "The facts helper compares the `kind` in
 // `atlas.yaml` with the `kind` on the repo record and reports drift."
 //
-// The kind decides which of the seven files a repository needs, so the two
+// The kind decides which of the six files a repository needs, so the two
 // copies disagreeing is not a detail. A repo record that says `service` while
 // atlas.yaml says `library` means four required parts read "not applicable" in
 // CI and "required" everywhere else.

package/door/lib/upkeep.mjs

@@ -119,7 +119,7 @@ export function docsIndex(root) {
     if (path.startsWith('docs/')) listed.add(path);
   }
   const unlisted = markdownFiles(root, 'docs').filter((path) => {
-    if (path === INDEX || path.startsWith('docs/site/') || listed.has(path)) return false;
+    if (path === INDEX || listed.has(path)) return false;
     const folder = designFolderOf(root, path);
     return !(folder && listed.has(`${folder}/design.yaml`));
   });

package/package.json

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

package/shape/check.mjs

@@ -11,9 +11,9 @@
 // mode only, it reads git objects with the git binary, and writes two trees
 // into a temp folder outside the checkout.
 //
-// What it reports: one state per part of the seven files. Five states: filled,
-// empty, stale, not applicable, no extractor. It leads with the count of
-// required parts that are not filled.
+// What it reports: one state per part of the six files. Four states: filled,
+// empty, stale, not applicable. It leads with the count of required parts
+// that are not filled.
 //
 // It carries the version of the Atlas package it came from, so a copy in a
 // repo says where it came from. It refuses a spec version it cannot read.
@@ -38,7 +38,7 @@ export const sha256 = (text) => createHash('sha256').update(String(text ?? ''),
 // Whether a copy is current or behind is not decided here. CI has no package
 // and no network. The session verb decides it, from the release list that
 // ships beside this file in the package, shape/releases.json.
-export const CHECK_VERSION = 6;
+export const CHECK_VERSION = 7;
 
 // ---------------------------------------------------------------------------
 // The spec. One object per version. The check reads a version it has here and
@@ -53,8 +53,7 @@ export const SPEC_1 = {
     "filled": "The part exists and matches the repo.",
     "empty": "The part is missing, or it holds nothing.",
     "stale": "The part exists but disagrees with the repo.",
-    "not applicable": "This repo kind does not need the part.",
-    "no extractor": "The part is generated, and no extractor exists for it yet."
+    "not applicable": "This repo kind does not need the part."
   },
   "kinds": {
     "service": "A running product with a deploy target, logs and metrics.",
@@ -320,101 +319,9 @@ export const SPEC_1 = {
         }
       ]
     },
-    {
-      "id": "docs-content",
-      "number": 6,
-      "title": "Docs content",
-      "class": "docs",
-      "written_by": "the upkeep task and the talk-through; merges on its own",
-      "parts": [
-        {
-          "id": "pages",
-          "path": "docs/site",
-          "glob": "*.md",
-          "holds": "The pages: home, features, flows with their step lists, concepts. Each page carries the commit it was built from.",
-          "kind": {
-            "service": "required",
-            "library": "optional",
-            "schema": "optional",
-            "plugin": "optional",
-            "cli": "optional"
-          },
-          "check": "docs-pages"
-        },
-        {
-          "id": "features",
-          "path": "docs/site/features.yaml",
-          "holds": "The feature list the renderer reads.",
-          "kind": {
-            "service": "required",
-            "library": "optional",
-            "schema": "optional",
-            "plugin": "optional",
-            "cli": "optional"
-          },
-          "check": "features"
-        },
-        {
-          "id": "notes",
-          "path": "docs/site/notes.md",
-          "class": "instruction",
-          "holds": "The standing corrections. Each note names who said it and the session. A person merges it.",
-          "kind": {
-            "service": "optional",
-            "library": "optional",
-            "schema": "optional",
-            "plugin": "optional",
-            "cli": "optional"
-          },
-          "check": "notes"
-        },
-        {
-          "id": "api",
-          "path": "docs/site/reference/api.md",
-          "holds": "Every route with its method, path, principal, handler file and line.",
-          "kind": {
-            "service": "required",
-            "library": "not applicable",
-            "schema": "not applicable",
-            "plugin": "not applicable",
-            "cli": "not applicable"
-          },
-          "check": "generated",
-          "extractor": "routes"
-        },
-        {
-          "id": "database",
-          "path": "docs/site/reference/database.md",
-          "holds": "The tables, their fields, their types and the purpose of each field.",
-          "kind": {
-            "service": "required",
-            "library": "not applicable",
-            "schema": "required",
-            "plugin": "not applicable",
-            "cli": "not applicable"
-          },
-          "check": "generated",
-          "extractor": "migrations"
-        },
-        {
-          "id": "mcp-tools",
-          "path": "docs/site/reference/mcp-tools.md",
-          "holds": "Every MCP tool, its inputs, and when to use it.",
-          "kind": {
-            "service": "optional",
-            "library": "optional",
-            "schema": "optional",
-            "plugin": "optional",
-            "cli": "optional"
-          },
-          "check": "generated",
-          "extractor": "mcp"
-        }
-      ]
-    },
     {
       "id": "tooling",
-      "number": 7,
+      "number": 6,
       "title": "Tooling",
       "class": "instruction",
       "written_by": "the helper; a person merges",
@@ -467,11 +374,6 @@ export const SPEC_1 = {
       ]
     }
   ],
-  "extractors": {
-    "routes": "Slice L3 builds it.",
-    "migrations": "Slice L3 builds it.",
-    "mcp": "Slice L3 builds it."
-  },
   "by_reference": {
     "rule": "A file under docs/ that an instruction file points at is instruction class by reference (D7 finding R1). The check computes that set and reports each member.",
     "scanned": [
@@ -480,8 +382,7 @@ export const SPEC_1 = {
       ".claude/rules/*.md",
       ".claude/skills/*/SKILL.md",
       "atlas.yaml",
-      "atlas/*.yaml",
-      "docs/site/notes.md"
+      "atlas/*.yaml"
     ]
   }
 };
@@ -498,7 +399,7 @@ function deepFreeze(value) {
 }
 deepFreeze(SPEC_1);
 
-// Spec 2 adds one part: the check file itself, scored inside file 7.
+// Spec 2 adds one part: the check file itself, scored inside file 6.
 //
 // Why a new version and not a line in spec 1. Adding a scored part changes what
 // `spec: 1` means, and the spec version is the whole coupling between a
@@ -506,8 +407,9 @@ deepFreeze(SPEC_1);
 // P11). The check reads both versions, so a repository stays green while it is
 // still on spec 1, and it opts in by moving `atlas.yaml` to `spec: 2`.
 //
-// Nothing else moves. Spec 1 keeps its twenty-two parts, and spec 2 has
-// twenty-three.
+// Nothing else moves. Spec 1 keeps its sixteen parts, and spec 2 has
+// seventeen. Check version 7 took the six parts of the retired docs site out
+// of both (roadmap R37).
 //
 // Check version 4 adds two kinds, plugin and cli, to both specs (issue #141).
 // No cell of an older kind moves, so a repo that names service, library or
@@ -965,8 +867,8 @@ export function frontMatter(source) {
 // The scanners. One per part of the spec.
 // ---------------------------------------------------------------------------
 
-export const STATES = Object.freeze(['filled', 'empty', 'stale', 'not applicable', 'no extractor']);
-export const NOT_FILLED = Object.freeze(['empty', 'stale', 'no extractor']);
+export const STATES = Object.freeze(['filled', 'empty', 'stale', 'not applicable']);
+export const NOT_FILLED = Object.freeze(['empty', 'stale']);
 export const CREDENTIAL_NAME = /^[A-Z][A-Z0-9_]*$/;
 const PINNED = /^[0-9a-f]{40}$/;
 const MAX_LISTED = 12;
@@ -1007,10 +909,10 @@ function symlinks(root, path, out = []) {
 
 // A symbolic link is a finding, never a state on its own: the part it sits
 // under says what state it makes.
-function symlinkFindings(root, path, keep = () => true) {
-  const found = symlinks(root, path).filter(keep);
+function symlinkFindings(root, path) {
+  const found = symlinks(root, path);
   return found.length
-    ? [`${found.length} symbolic link(s) under ${path}/: ${found.slice(0, MAX_LISTED).join(', ')}; the check does not follow one, and a renderer that does would publish what it points at`]
+    ? [`${found.length} symbolic link(s) under ${path}/: ${found.slice(0, MAX_LISTED).join(', ')}; the check does not follow one`]
     : [];
 }
 
@@ -1213,59 +1115,16 @@ function checkDocsIndex(context, part) {
   const text = readText(context.root, part.path);
   if (text === null || !text.trim()) return result(part, 'empty', [`${part.path} is missing or holds nothing`]);
   const listed = new Set(pointers(text).map((path) => (path.startsWith('docs/') ? path : posix.join('docs', path))));
-  const onDisk = walk(context.root, 'docs').filter((path) => path.endsWith('.md') && path !== part.path && !path.startsWith('docs/site/'));
+  const onDisk = walk(context.root, 'docs').filter((path) => path.endsWith('.md') && path !== part.path);
   const unlisted = onDisk.filter((path) => !listed.has(path));
   const gone = [...listed].filter((path) => !existsSync(join(context.root, path)));
   const findings = [];
   if (unlisted.length) findings.push(`${unlisted.length} file(s) under docs/ are not listed: ${unlisted.slice(0, MAX_LISTED).join(', ')}`);
   if (gone.length) findings.push(`${gone.length} listed file(s) are gone: ${gone.slice(0, MAX_LISTED).join(', ')}`);
-  // The docs tree outside docs/site/. The pages part below owns docs/site/.
-  findings.push(...symlinkFindings(context.root, 'docs', (path) => !path.startsWith('docs/site/')));
+  findings.push(...symlinkFindings(context.root, 'docs'));
   return findings.length ? result(part, 'stale', findings) : result(part, 'filled', [`${onDisk.length} file(s) listed`]);
 }
 
-function checkDocsPages(context, part) {
-  const links = symlinkFindings(context.root, part.path);
-  if (!isDirectory(context.root, part.path)) return result(part, 'empty', [`${part.path}/ does not exist`]);
-  const pages = walk(context.root, part.path).filter((path) => path.endsWith('.md') && !path.endsWith('/notes.md'));
-  if (!pages.length) return result(part, links.length ? 'stale' : 'empty', [`${part.path}/ holds no page`, ...links]);
-  const without = pages.filter((path) => !frontMatter(readText(context.root, path) ?? '')?.built_from);
-  const findings = [...(without.length ? [`${without.length} page(s) do not name the commit they were built from: ${without.slice(0, MAX_LISTED).join(', ')}`] : []), ...links];
-  return findings.length ? result(part, 'stale', findings) : result(part, 'filled', [`${pages.length} page(s)`]);
-}
-
-function checkFeatures(context, part) {
-  const text = readText(context.root, part.path);
-  if (text === null || !text.trim()) return result(part, 'empty', [`${part.path} is missing or holds nothing`]);
-  let data;
-  try { data = parseYaml(text); } catch (error) { return result(part, 'stale', [`${part.path} cannot be read: ${error.message}`]); }
-  const rows = data?.features;
-  if (emptyValue(rows)) return result(part, 'stale', [`${part.path} lists no feature`]);
-  const list = Array.isArray(rows) ? rows : Object.values(rows);
-  const dead = list.filter((row) => row?.page && !existsSync(join(context.root, String(row.page)))).map((row) => row.page);
-  return dead.length ? result(part, 'stale', [`${dead.length} feature page(s) are gone: ${dead.slice(0, MAX_LISTED).join(', ')}`]) : result(part, 'filled', [`${list.length} feature(s)`]);
-}
-
-function checkNotes(context, part) {
-  const text = readText(context.root, part.path);
-  if (text === null || !text.trim()) return result(part, 'empty', [`${part.path} is missing or holds nothing`]);
-  // Finding R12 and the spec: each note names who said it and the session.
-  // A note that does not is a stale note, not a remark on a filled file.
-  const entries = text.split('\n').filter((line) => /^\s*[-*]\s+/.test(line));
-  if (!entries.length) return result(part, 'empty', [`${part.path} holds no note`]);
-  const without = entries.filter((line) => !/session/i.test(line));
-  if (without.length) return result(part, 'stale', [`${without.length} of ${entries.length} note(s) do not name the session they came from`]);
-  return result(part, 'filled', [`${entries.length} note(s)`]);
-}
-
-function checkGenerated(context, part) {
-  const available = context.extractors.includes(part.extractor);
-  if (!available) return result(part, 'no extractor', [`no ${part.extractor} extractor exists yet; slice L3 builds it`]);
-  const text = readText(context.root, part.path);
-  if (text === null || !text.trim()) return result(part, 'empty', [`${part.path} is missing or holds nothing`]);
-  return result(part, 'filled');
-}
-
 // Every permission the workflow grants at anything other than read. It reads a
 // block map under `permissions:` and a flow map on the same line.
 function writeGrants(text) {
@@ -1459,10 +1318,6 @@ const CHECKS = {
   'facts-section': checkFactsSection,
   'facts-credentials': checkCredentials,
   'docs-index': checkDocsIndex,
-  'docs-pages': checkDocsPages,
-  features: checkFeatures,
-  notes: checkNotes,
-  generated: checkGenerated,
   workflow: checkWorkflow,
   'check-file': checkCheckFile,
   linters: checkLinters,
@@ -1470,7 +1325,7 @@ const CHECKS = {
 
 // The one exemption in the by-reference set, and it is exactly one.
 //
-// File 1, the entry map, must hold a pointer to each of the seven files, and
+// File 1, the entry map, must hold a pointer to each of the six files, and
 // `docs/index.md` is one of them. Rule R1 makes any `docs/` file an instruction
 // file points at instruction class by reference. Those two together make the
 // index instruction class, and the frozen L4 path limit lets the upkeep routine
@@ -1484,8 +1339,7 @@ const CHECKS = {
 // only this one path, only these two pointing files, and the row stays in the
 // report with `exempt: true` so nothing is hidden. Any other instruction file
 // that names the index still makes it instruction class, and the index's own
-// row then carries that file. The docs lint of slice L3 bounds the rest, by
-// treating the index as an agent-facing surface.
+// row then carries that file.
 export const EXEMPT_PATH = 'docs/index.md';
 export const EXEMPT_FROM = Object.freeze(['CLAUDE.md', 'AGENTS.md']);
 
@@ -1571,7 +1425,7 @@ export function readFacts(root) {
 // `kind: library` would be two different spec tables, and the comparison would
 // mean nothing (architect finding P2). Nothing else about the scan changes: the
 // tree still speaks for itself about every part.
-export function scan({ root, repoId = null, extractors = [], secretPatterns: patterns = secretPatterns(), scoreAs = null }) {
+export function scan({ root, repoId = null, secretPatterns: patterns = secretPatterns(), scoreAs = null }) {
   const { facts, error: factsError } = readFacts(root);
   const errors = [];
   // The credential-value check is the only thing that turns the job red at the
@@ -1603,7 +1457,7 @@ export function scan({ root, repoId = null, extractors = [], secretPatterns: pat
   // that a pull request moves is also scored where the base had it (roadmap R7
   // line 3).
   const pointed = scoreAs?.pointers ? { ...facts, files: scoreAs.pointers.files, procedures: scoreAs.pointers.procedures } : facts;
-  const context = { root, facts: pointed, factsError, repoId, extractors, secretPatterns: usable, errors };
+  const context = { root, facts: pointed, factsError, repoId, secretPatterns: usable, errors };
   // atlas.yaml's own values reach the report: the kind, the repo id, a
   // procedure's skill name, every pointer path. Scan it before any of them is
   // quoted (acceptance review of batch 16, note S3-2).
@@ -1626,7 +1480,7 @@ export function scan({ root, repoId = null, extractors = [], secretPatterns: pat
 }
 
 export function counts(files) {
-  const out = { filled: 0, empty: 0, stale: 0, 'not applicable': 0, 'no extractor': 0, required: 0, optional: 0, not_filled: 0, required_not_filled: 0, optional_not_filled: 0 };
+  const out = { filled: 0, empty: 0, stale: 0, 'not applicable': 0, required: 0, optional: 0, not_filled: 0, required_not_filled: 0, optional_not_filled: 0 };
   for (const file of files) for (const part of file.parts) {
     out[part.state] += 1;
     if (part.required === 'required') out.required += 1;
@@ -1845,7 +1699,7 @@ export function gateLines(answer, state) {
 // the job summary. Both list every part, its state and its findings.
 // ---------------------------------------------------------------------------
 
-const MARK = { filled: '+', empty: '-', stale: '~', 'not applicable': '.', 'no extractor': '?' };
+const MARK = { filled: '+', empty: '-', stale: '~', 'not applicable': '.' };
 const label = (part) => part.procedure ? `procedure ${part.procedure}` : part.path ?? part.id;
 
 // The one number the owner sees is the count of REQUIRED files not filled.
@@ -1959,7 +1813,6 @@ export function verdict(report, failOn) {
 // ---------------------------------------------------------------------------
 
 export const FAIL_ON = Object.freeze(['error', 'required', 'any']);
-export const EXTRACTORS = Object.freeze([]); // Slice L3 adds routes, migrations and mcp.
 
 export function parseArguments(argv) {
   const out = { root: process.cwd(), failOn: 'error', json: null, summary: null, quiet: false, base: null, baseSha: null, baseCommit: null, headCommit: null, git: process.cwd(), gate: false };
@@ -1999,7 +1852,7 @@ export function run(argv, env = process.env) {
   if (options.gate) return runGate({ options, env });
   const repoId = options.repo ?? env.GITHUB_REPOSITORY ?? null;
   const patterns = secretPatterns();
-  const report = scan({ root: options.root, repoId, extractors: EXTRACTORS, secretPatterns: patterns });
+  const report = scan({ root: options.root, repoId, secretPatterns: patterns });
   // No `--base` means no pull request to measure: a push build, or a run by
   // hand. The no-loss rule then does not run at all, and the report says nothing
   // about it rather than claiming a pass.

package/shape/releases.json

@@ -34,6 +34,12 @@
     {
       "check_version": 6,
       "sha256": "bad1d05853e173c034971282c5762a1628dfd9f198c8276178bd17280afdf6e7",
+      "first_package": "0.4.16",
+      "workflow_sha256": "fcddebef030e6564f97e1a5fb81c8ad1b3084ebef9622167c6565698312dcdd7"
+    },
+    {
+      "check_version": 7,
+      "sha256": "b29fdb036170419b9f0642ef0ca7da45a80ff53492421eb57beaf157df13d057",
       "workflow_sha256": "fcddebef030e6564f97e1a5fb81c8ad1b3084ebef9622167c6565698312dcdd7"
     }
   ]