@arjunkhera/atlas 0.3.34 → 0.3.37

package/.claude-plugin/plugin.json

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

package/door/kit-releases.json

@@ -32,6 +32,10 @@
     {
       "kit_sha256": "c4a506ec261d2d2fd7884943c9d317fb7b522704f4f63669b5bd37d32326bde5",
       "first_package": "0.3.32"
+    },
+    {
+      "kit_sha256": "b5c36f685ba476e09a54e3207e2444b098cc6a26dded8dca65b01e751fe2cbe4",
+      "first_package": "0.3.34"
     }
   ]
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arjunkhera/atlas",
-  "version": "0.3.34",
+  "version": "0.3.37",
   "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

@@ -34,7 +34,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 = 3;
+export const CHECK_VERSION = 4;
 
 // ---------------------------------------------------------------------------
 // The spec. One object per version. The check reads a version it has here and
@@ -55,7 +55,9 @@ export const SPEC_1 = {
   "kinds": {
     "service": "A running product with a deploy target, logs and metrics.",
     "library": "Code another repo takes as a dependency. It is not deployed.",
-    "schema": "Migrations and their release. It has no service and no library build."
+    "schema": "Migrations and their release. It has no service and no library build.",
+    "plugin": "A Claude Code plugin. It is installed, not deployed, and the merge proves it.",
+    "cli": "A command-line tool people install. It is not deployed."
   },
   "files": [
     {
@@ -72,7 +74,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "required",
-            "schema": "required"
+            "schema": "required",
+            "plugin": "required",
+            "cli": "required"
           },
           "check": "markdown",
           "max_lines": 200
@@ -84,7 +88,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "required",
-            "schema": "required"
+            "schema": "required",
+            "plugin": "required",
+            "cli": "required"
           },
           "check": "markdown",
           "max_lines": 200
@@ -106,7 +112,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "optional",
             "library": "optional",
-            "schema": "optional"
+            "schema": "optional",
+            "plugin": "optional",
+            "cli": "optional"
           },
           "check": "path-rules"
         }
@@ -126,7 +134,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "not applicable",
-            "schema": "not applicable"
+            "schema": "not applicable",
+            "plugin": "not applicable",
+            "cli": "not applicable"
           },
           "check": "procedure"
         },
@@ -137,7 +147,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "required",
-            "schema": "not applicable"
+            "schema": "not applicable",
+            "plugin": "required",
+            "cli": "required"
           },
           "check": "procedure"
         },
@@ -148,7 +160,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "not applicable",
-            "schema": "not applicable"
+            "schema": "not applicable",
+            "plugin": "not applicable",
+            "cli": "not applicable"
           },
           "check": "procedure"
         },
@@ -159,7 +173,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "required",
-            "schema": "required"
+            "schema": "required",
+            "plugin": "required",
+            "cli": "required"
           },
           "check": "procedure"
         }
@@ -179,7 +195,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "required",
-            "schema": "required"
+            "schema": "required",
+            "plugin": "required",
+            "cli": "required"
           },
           "check": "facts-root",
           "keys": [
@@ -197,7 +215,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "not applicable",
-            "schema": "not applicable"
+            "schema": "not applicable",
+            "plugin": "not applicable",
+            "cli": "not applicable"
           },
           "check": "facts-section",
           "keys": [
@@ -212,7 +232,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "not applicable",
-            "schema": "not applicable"
+            "schema": "not applicable",
+            "plugin": "not applicable",
+            "cli": "not applicable"
           },
           "check": "facts-section",
           "keys": [
@@ -227,7 +249,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "not applicable",
-            "schema": "required"
+            "schema": "required",
+            "plugin": "optional",
+            "cli": "optional"
           },
           "check": "facts-credentials",
           "keys": [
@@ -242,7 +266,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "optional",
-            "schema": "optional"
+            "schema": "optional",
+            "plugin": "optional",
+            "cli": "optional"
           },
           "check": "facts-section",
           "keys_present": [
@@ -257,7 +283,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "optional",
             "library": "optional",
-            "schema": "optional"
+            "schema": "optional",
+            "plugin": "optional",
+            "cli": "optional"
           },
           "check": "facts-section",
           "keys": [
@@ -280,7 +308,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "required",
-            "schema": "required"
+            "schema": "required",
+            "plugin": "required",
+            "cli": "required"
           },
           "check": "docs-index"
         }
@@ -301,7 +331,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "optional",
-            "schema": "optional"
+            "schema": "optional",
+            "plugin": "optional",
+            "cli": "optional"
           },
           "check": "docs-pages"
         },
@@ -312,7 +344,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "optional",
-            "schema": "optional"
+            "schema": "optional",
+            "plugin": "optional",
+            "cli": "optional"
           },
           "check": "features"
         },
@@ -324,7 +358,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "optional",
             "library": "optional",
-            "schema": "optional"
+            "schema": "optional",
+            "plugin": "optional",
+            "cli": "optional"
           },
           "check": "notes"
         },
@@ -335,7 +371,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "not applicable",
-            "schema": "not applicable"
+            "schema": "not applicable",
+            "plugin": "not applicable",
+            "cli": "not applicable"
           },
           "check": "generated",
           "extractor": "routes"
@@ -347,7 +385,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "not applicable",
-            "schema": "required"
+            "schema": "required",
+            "plugin": "not applicable",
+            "cli": "not applicable"
           },
           "check": "generated",
           "extractor": "migrations"
@@ -359,7 +399,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "optional",
             "library": "optional",
-            "schema": "optional"
+            "schema": "optional",
+            "plugin": "optional",
+            "cli": "optional"
           },
           "check": "generated",
           "extractor": "mcp"
@@ -380,7 +422,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "required",
             "library": "required",
-            "schema": "required"
+            "schema": "required",
+            "plugin": "required",
+            "cli": "required"
           },
           "check": "workflow"
         },
@@ -390,7 +434,9 @@ export const SPEC_1 = {
           "kind": {
             "service": "optional",
             "library": "optional",
-            "schema": "optional"
+            "schema": "optional",
+            "plugin": "optional",
+            "cli": "optional"
           },
           "check": "linters",
           "candidates": [
@@ -458,6 +504,12 @@ deepFreeze(SPEC_1);
 //
 // Nothing else moves. Spec 1 keeps its twenty-two parts, and spec 2 has
 // twenty-three.
+//
+// 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
+// schema reads the same table as before. The kind is now a second coupling
+// beside the spec version: a repo that names plugin or cli needs check 4 or
+// later. Stage 2 of the target design retires these fixed tables.
 const clone = (value) => JSON.parse(JSON.stringify(value));
 
 export const SPEC_2 = (() => {
@@ -471,7 +523,7 @@ export const SPEC_2 = (() => {
     // declared below it. A test keeps the two equal.
     path: '.github/atlas/check.mjs',
     holds: 'The check itself, the copy this repo runs. One file, no dependency, carrying the Atlas version and the package commit it came from.',
-    kind: { service: 'required', library: 'required', schema: 'required' },
+    kind: { service: 'required', library: 'required', schema: 'required', plugin: 'required', cli: 'required' },
     check: 'check-file',
   });
   return deepFreeze(spec);

package/shape/releases.json

@@ -15,6 +15,11 @@
       "check_version": 3,
       "sha256": "0184db66fd5d5061c21ee7d2d88395b5bac64ab1308e02e06781e6015f3d7608",
       "first_package": "0.3.0"
+    },
+    {
+      "check_version": 4,
+      "sha256": "3357ea247694634c3535998368683eceffaf863221dee5fec9e43a03042afb95",
+      "first_package": "0.3.36"
     }
   ]
 }

package/tests/model.mjs

@@ -289,11 +289,14 @@ export async function runClaude({ prompt, args = [], key, cwd, meter, what, reda
     // A call with no cost in its answer counts its whole hold: the meter never guesses low. The hold is at
     // most a quarter of the cap, so one bad call does not end the night.
     meter.add(Number.isFinite(cost) ? cost : left, what, id);
-    if (!envelope) throw new Error(`${what}: the model command gave no answer (exit ${out.exit}): ${(redactor ? redactor.text(out.stderr || out.stdout) : out.stderr || out.stdout).slice(0, 400)}`);
     const clean = (t) => (redactor ? redactor.text(t) : t);
+    // No answer at all: the command could not run (a missing binary, say). That is no product result
+    // either, so it reads blocked, and the log shows what the command printed.
+    if (!envelope) throw new Blocked(`${what}: the model command gave no answer (exit ${out.exit}): ${clean(String(out.stderr || out.stdout).trim()).slice(0, 400)}`);
     // An error answer with no cost never reached the model: a refused sign-in, no credit, a bad model
-    // name. The scenario cannot pass or fail on it, so it reads blocked, with the reason.
-    if (envelope.is_error && !Number.isFinite(cost)) throw new Blocked(`${what}: the model call failed before the model answered: ${clean(String(envelope.result ?? envelope.error ?? (out.stderr || out.stdout))).slice(0, 400)}`);
+    // name. Newer claude commands give a cost of 0 there. The scenario cannot pass or fail on it, so it
+    // reads blocked, with the reason.
+    if (envelope.is_error && !(cost > 0)) throw new Blocked(`${what}: the model call failed before the model answered: ${clean(String(envelope.result ?? envelope.error ?? (out.stderr || out.stdout))).slice(0, 400)}`);
     if (out.signal === 'SIGKILL') throw new Error(`${what}: the model call ran past ${Math.round(timeoutMs / 1000)} seconds and was stopped`);
     return { exit: out.exit, stdout: clean(out.stdout), stderr: clean(out.stderr), envelope, events, cost: Number.isFinite(cost) ? cost : null };
   } finally {