@webjsdev/server 0.8.57 → 0.8.59

package/src/stream.js

@@ -6,7 +6,7 @@
  * The payload is plain HTML, a `<webjs-stream action target>` wrapping one
  * `<template>`. These helpers compose that string for the two delivery paths,
  * a content-negotiated HTTP form response and a `broadcast()` / WS message, so
- * an app never hand-writes the markup. A page `action` or a `route` handler
+ * an app never hand-writes the markup. A form-bound action or a `route` handler
  * branches on `acceptsStream(request)`, returning `streamResponse(...)` when the
  * router asked for a stream and a normal render otherwise (the JS-off degrade).
  *

package/src/testing.d.ts

@@ -42,6 +42,25 @@ export declare function withCookies(init: RequestInit | undefined, cookieValue:
 /** Merge a session cookie string into a `RequestInit`'s `Cookie` header. */
 export declare function withSessionCookie(init: RequestInit | undefined, sessionCookie: string): RequestInit;
 
+/**
+ * Submit a page's `<form action={action}>` the way a browser with JS OFF does:
+ * render the page, reuse the identity the server put in its hidden
+ * `__webjs_action` field, and POST it back with the given fields. A POST that
+ * omits that field is not a form submission and is answered `405`.
+ */
+export declare function submitForm(
+  handle: Handle,
+  path: string,
+  fields?: Record<string, string> | URLSearchParams | FormData,
+  opts?: {
+    index?: number;
+    match?: string | RegExp;
+    cookies?: string;
+    submitPath?: string;
+    headers?: Record<string, string>;
+  },
+): Promise<Response>;
+
 /**
  * Drive the REAL credentials login (`POST /api/auth/signin/credentials` by
  * default) and capture the genuine signed session `Set-Cookie`.

package/src/testing.js

@@ -33,6 +33,7 @@
  */
 
 import { hashFile, RPC_CONTENT_TYPE } from './actions.js';
+import { FORM_ACTION_FIELD } from '@webjsdev/core';
 import { getSerializer } from './serializer.js';
 import { join, sep } from 'node:path';
 
@@ -200,6 +201,118 @@ export async function loginAndGetCookies(handle, credentials, opts = {}) {
   return { cookies, setCookies, response: res };
 }
 
+/**
+ * Submit a page's `<form action=${action}>` the way a browser with JS OFF does
+ * (#1155): render the page, take the identity the server put in its hidden
+ * `__webjs_action` field, and POST it back with the author's fields.
+ *
+ * A bound form carries that field because the identity is what tells the
+ * dispatcher WHICH action to run, so a hand-written POST that omits it is not a
+ * form submission at all and is answered `405`. Reproducing the scrape in every
+ * app test is how that ends up looking like a database problem instead: the
+ * status is wrong, an assertion fails, and a `catch` around it reports a skip.
+ * Hence a helper, so the identity is never the test's business.
+ *
+ * The form is located by INDEX (`opts.index`, default the first on the page),
+ * which is enough for the common one-form page. For a page with several, pass
+ * `opts.match`: a string or RegExp that must appear in the form's markup, so
+ * the test names the form by something an author recognizes (a heading, a field
+ * name, an id) rather than by counting.
+ *
+ * @param {Handle} handle
+ * @param {string} path the page to render AND, by default, to submit back to
+ * @param {Record<string, string> | URLSearchParams | FormData} [fields]
+ * @param {{
+ *   index?: number,
+ *   match?: string | RegExp,
+ *   cookies?: string,
+ *   submitPath?: string,
+ *   headers?: Record<string, string>,
+ * }} [opts]
+ * @returns {Promise<Response>} the submission response (a `303` on success, a
+ *   `422` re-render on a failure result)
+ */
+/**
+ * The `<form>...</form>` blocks of a rendered page.
+ *
+ * A bare `/<form[\s\S]*?<\/form>/g` gets two things wrong that a test helper
+ * must not, because both fail SILENTLY: a form inside an HTML comment is
+ * matched (so `opts.index` counts a form that is not on the page), and a
+ * `</form>` sitting inside an ATTRIBUTE VALUE ends the match early (so the
+ * identity field falls outside it and the helper reports an unbound form).
+ * Comments are stripped first, and the scan skips quoted attribute values while
+ * looking for the end of the start tag.
+ *
+ * @param {string} html
+ * @returns {string[]}
+ */
+function findForms(html) {
+  const src = html.replace(/<!--[\s\S]*?-->/g, '');
+  const out = [];
+  const open = /<form\b/gi;
+  let m;
+  while ((m = open.exec(src))) {
+    // Walk the start tag, skipping quoted values so `data-x="<form>"` is inert.
+    let i = m.index + m[0].length;
+    let quote = '';
+    for (; i < src.length; i++) {
+      const c = src[i];
+      if (quote) { if (c === quote) quote = ''; continue; }
+      if (c === '"' || c === "'") { quote = c; continue; }
+      if (c === '>') break;
+    }
+    const close = src.toLowerCase().indexOf('</form>', i);
+    if (close === -1) continue;
+    out.push(src.slice(m.index, close + '</form>'.length));
+    open.lastIndex = close;
+  }
+  return out;
+}
+
+export async function submitForm(handle, path, fields = {}, opts = {}) {
+  const getInit = opts.cookies ? withCookies({}, opts.cookies) : {};
+  const page = await testRequest(handle, path, getInit);
+  const html = await page.text();
+
+  const forms = findForms(html);
+  const candidates = opts.match
+    ? forms.filter((f) => (opts.match instanceof RegExp ? opts.match.test(f) : f.includes(opts.match)))
+    : forms;
+  const form = candidates[opts.index || 0];
+  if (!form) {
+    throw new Error(
+      `submitForm: no ${opts.match ? 'matching ' : ''}<form> found on ${path} `
+      + `(status ${page.status}, ${forms.length} form(s) on the page).`,
+    );
+  }
+  const id = form.match(/name="__webjs_action"\s+value="([^"]*)"/);
+  if (!id) {
+    throw new Error(
+      `submitForm: the <form> on ${path} carries no __webjs_action field, so it `
+      + 'is not bound to a server action. Write it as `<form action=${myAction}>` '
+      + 'with the action imported from a `use server` module.',
+    );
+  }
+
+  const body = new URLSearchParams();
+  body.set(FORM_ACTION_FIELD, id[1]);
+  const entries = fields instanceof URLSearchParams || fields instanceof FormData
+    ? fields.entries()
+    : Object.entries(fields);
+  for (const [k, v] of entries) body.append(k, String(v));
+
+  const init = {
+    method: 'POST',
+    headers: { 'content-type': 'application/x-www-form-urlencoded', ...(opts.headers || {}) },
+    body: body.toString(),
+  };
+  return await testRequest(
+    handle,
+    opts.submitPath || path,
+    opts.cookies ? withCookies(init, opts.cookies) : init,
+  );
+}
+
 /**
  * Compute the RPC endpoint path for an action, addressing it the same way the
  * generated client stub does: by the SHA-256 hash of the action's ABSOLUTE file

package/src/vendor.js

@@ -338,6 +338,27 @@ let lastLiveResolveFailed = false;
 
 const JSPM_GENERATE_ENDPOINT = 'https://api.jspm.io/generate';
 const JSPM_GENERATE_TIMEOUT_MS = 10_000;
+// Bounds a single bundle GET during the SERVER's warmup live-integrity pass
+// (`fetchLiveIntegrity`). Short on purpose: that pass gates readiness, so a
+// stalled CDN must not hold the first request, and it is additionally capped
+// by INTEGRITY_TOTAL_BUDGET_MS across all URLs.
+const INTEGRITY_FETCH_TIMEOUT_MS = 10_000;
+
+// Bounds a single bundle GET made by the pin command, which either writes the
+// bytes to disk (`downloadBundle`) or fetches them to hash
+// (`fetchIntegrity`). Deliberately six times the warmup budget, because the
+// two are not the same situation: a pin is a one-shot command a person ran and
+// is waiting on, with a whole multi-megabyte package to transfer, while the
+// warmup is a server holding a request. Ten seconds is generous for the
+// latter and tight for the former on a slow link.
+//
+// 60s matches what importmap-rails effectively allows. It sets no timeout at
+// all, but Ruby's Net::HTTP defaults open_timeout and read_timeout to 60s, so
+// a Rails pin is bounded at a minute without asking. JavaScript's fetch() has
+// no default whatsoever, which is why this has to be explicit: without it a
+// CDN that accepts the connection and then stalls hangs the pin forever, with
+// no ambient deadline on a CLI run to cut it short.
+const PIN_BUNDLE_TIMEOUT_MS = 60_000;
 
 /**
  * Provider names accepted by `webjs vendor pin --from <provider>`.
@@ -387,12 +408,15 @@ export function normalizeProvider(name) {
  *
  * @param {Array<string>} installs  e.g. ['dayjs@1.11.13', '@codemirror/lint@6.9.6']
  * @param {string} provider  one of SUPPORTED_PROVIDERS
+ * @param {number} [timeoutMs]  defaults to the SERVER budget; a CLI caller
+ *   passes the longer one, since it is a command someone is waiting on rather
+ *   than a request being held open.
  * @returns {Promise<JspmCallResult>}
  */
-async function jspmCall(installs, provider) {
+async function jspmCall(installs, provider, timeoutMs = JSPM_GENERATE_TIMEOUT_MS) {
   const label = installs.length === 1 ? `'${installs[0]}'` : `${installs.length} packages`;
   const controller = new AbortController();
-  const timer = setTimeout(() => controller.abort(), JSPM_GENERATE_TIMEOUT_MS);
+  const timer = setTimeout(() => controller.abort(), timeoutMs);
   try {
     const response = await fetch(JSPM_GENERATE_ENDPOINT, {
       method: 'POST',
@@ -440,7 +464,7 @@ async function jspmCall(installs, provider) {
     return { ok: true, imports, transient: false };
   } catch (e) {
     const msg = e && e.name === 'AbortError'
-      ? `timed out after ${JSPM_GENERATE_TIMEOUT_MS}ms`
+      ? `timed out after ${timeoutMs}ms`
       : `${e && e.message}`;
     console.error(`[webjs] could not vendor ${label} via ${provider}: ${msg}`);
     return { ok: false, imports: {}, transient: true };
@@ -466,8 +490,8 @@ async function jspmCall(installs, provider) {
  * @param {string} [provider]  one of SUPPORTED_PROVIDERS; defaults to 'jspm'
  * @returns {Promise<Record<string, string>>}
  */
-async function jspmResolveOne(install, provider = 'jspm') {
-  const { ok, imports, transient } = await jspmProbeOne(install, provider);
+async function jspmResolveOne(install, provider = 'jspm', timeoutMs) {
+  const { ok, imports, transient } = await jspmProbeOne(install, provider, timeoutMs);
   // Preserve the public contract: an empty map on any failure, and the
   // module-global retry flag set ONLY on a transient one (a permanent 401
   // for an unresolvable private/server-only dep is tolerated).
@@ -492,13 +516,13 @@ async function jspmResolveOne(install, provider = 'jspm') {
  * @param {string} provider
  * @returns {Promise<JspmCallResult>}
  */
-function jspmProbeOne(install, provider) {
+function jspmProbeOne(install, provider, timeoutMs) {
   const cacheKey = `${provider}::probe::${install}`;
   const existing = jspmCache.get(cacheKey);
   if (existing) return existing;
 
   const promise = (async () => {
-    const result = await jspmCall([install], provider);
+    const result = await jspmCall([install], provider, timeoutMs);
     // Do not cache a failure: a transient one must be re-attempted on the
     // next resolve, and a permanent one is cheap to re-confirm and must not
     // pin a stale "unresolvable" verdict across a dependency change.
@@ -543,14 +567,18 @@ function jspmProbeOne(install, provider) {
  *
  * @param {Array<string>} installs  e.g. ['dayjs@1.11.13', 'clsx@2.1.1']
  * @param {string} [provider]  one of SUPPORTED_PROVIDERS; defaults to 'jspm'
+ * @param {number} [timeoutMs]  per-call budget. Defaults to the SERVER one,
+ *   because this runs on a cold first request as well as from the CLI; the two
+ *   pin commands pass PIN_BUNDLE_TIMEOUT_MS instead. importmap-rails only ever
+ *   resolves from the CLI, so its flat 60s has no request path to slow down.
  * @returns {Promise<Record<string, string>>}
  */
-export async function jspmGenerate(installs, provider = 'jspm') {
+export async function jspmGenerate(installs, provider = 'jspm', timeoutMs) {
   if (installs.length === 0) return {};
 
   // A single install has no cross-package graph to reconcile, so the
   // isolated path IS the coherent path; reuse the per-install cache.
-  if (installs.length === 1) return jspmResolveOne(installs[0], provider);
+  if (installs.length === 1) return jspmResolveOne(installs[0], provider, timeoutMs);
 
   // Stable key regardless of scan order so the same dep set hits cache.
   const unifiedKey = `${provider}::unified::${[...installs].sort().join('\n')}`;
@@ -558,7 +586,7 @@ export async function jspmGenerate(installs, provider = 'jspm') {
   if (cached) return cached;
 
   const promise = (async () => {
-    const unified = await jspmCall(installs, provider);
+    const unified = await jspmCall(installs, provider, timeoutMs);
     if (unified.ok) return unified.imports;
 
     // The unified call failed. Drop the cached failure so a later retry
@@ -570,14 +598,14 @@ export async function jspmGenerate(installs, provider = 'jspm') {
       // per-install fragments (each may still be cached / reachable) so we
       // serve whatever we can, and flag the transient failure for retry.
       lastLiveResolveFailed = true;
-      return mergePerInstall(await Promise.all(installs.map(i => jspmResolveOne(i, provider))));
+      return mergePerInstall(await Promise.all(installs.map(i => jspmResolveOne(i, provider, timeoutMs))));
     }
 
     // Permanent failure: at least one install is unresolvable. Probe each
     // in isolation to learn which ones jspm can resolve, then re-run the
     // unified call over only those so the survivors form one consistent
     // graph (restores #446 coherence for the resolvable subset).
-    const probes = await Promise.all(installs.map(i => jspmProbeOne(i, provider)));
+    const probes = await Promise.all(installs.map(i => jspmProbeOne(i, provider, timeoutMs)));
 
     // A GOOD package whose isolated probe failed TRANSIENTLY (a network blip
     // mid-probe) must NOT be classified as unresolvable and dropped. Only a
@@ -612,11 +640,11 @@ export async function jspmGenerate(installs, provider = 'jspm') {
       return mergePerInstall(probes.map(p => p.imports));
     }
     if (resolvable.length === 0) return {};
-    if (resolvable.length === 1) return jspmResolveOne(resolvable[0], provider);
+    if (resolvable.length === 1) return jspmResolveOne(resolvable[0], provider, timeoutMs);
 
     // Re-run unified over the resolvable subset. If even that fails (a
     // conflict among the survivors), fall back to their merged fragments.
-    const retry = await jspmCall(resolvable, provider);
+    const retry = await jspmCall(resolvable, provider, timeoutMs);
     if (retry.ok) return retry.imports;
     return mergePerInstall(resolvable.map(i => probes[installs.indexOf(i)].imports));
   })();
@@ -679,6 +707,12 @@ export async function vendorImportMapEntries(bareImports, appDir) {
 export function clearVendorCache() {
   jspmCache.clear();
   liveIntegrityCache.clear();
+  // Deliberately does NOT touch `lastLiveResolveFailed`, which looks like an
+  // omission and is not. `resolveVendorImports` is the flag's only reader and
+  // it resets the flag on entry, while the pinned short-circuit above that
+  // returns `ok: true` outright, so a value left behind by an earlier
+  // `pinAll` or `vendorImportMapEntries` can never be observed. Clearing it
+  // here would be a change nothing could write a failing test for (#1150).
 }
 
 /**
@@ -1071,14 +1105,21 @@ async function writePinFile(appDir, imports, integrity, provider) {
  * success or null on failure. The integrity hash is computed from the
  * downloaded bytes so it's always consistent with what's on disk.
  *
+ * Bounded by PIN_BUNDLE_TIMEOUT_MS. `pinAll(dir, { download: true })`
+ * runs this once per resolved URL on a CLI run with no ambient
+ * deadline, so a CDN that accepts the connection and then stalls would
+ * otherwise hang the pin with nothing to interrupt it (#1150).
+ *
  * @param {string} url
  * @param {string} appDir
  * @param {string} filename
  * @returns {Promise<{ bytes: number, integrity: string } | null>}
  */
 async function downloadBundle(url, appDir, filename) {
+  const controller = new AbortController();
+  const timer = setTimeout(() => controller.abort(), PIN_BUNDLE_TIMEOUT_MS);
   try {
-    const response = await fetch(url);
+    const response = await fetch(url, { signal: controller.signal });
     if (!response.ok) {
       console.error(`[webjs] download ${url} returned ${response.status}`);
       return null;
@@ -1094,8 +1135,13 @@ async function downloadBundle(url, appDir, filename) {
     await writeFile(join(pinDir(appDir), filename), buf);
     return { bytes: buf.byteLength, integrity: await sha384Integrity(buf) };
   } catch (e) {
-    console.error(`[webjs] download ${url} failed: ${e && e.message}`);
+    const why = e && e.name === 'AbortError'
+      ? `timed out after ${PIN_BUNDLE_TIMEOUT_MS}ms`
+      : e && e.message;
+    console.error(`[webjs] download ${url} failed: ${why}`);
     return null;
+  } finally {
+    clearTimeout(timer);
   }
 }
 
@@ -1105,12 +1151,21 @@ async function downloadBundle(url, appDir, filename) {
  * so the importmap can carry SRI hashes even when bundles aren't
  * locally vendored.
  *
+ * Bounded by PIN_BUNDLE_TIMEOUT_MS, the same budget `downloadBundle`
+ * gets, since it transfers the same bytes and differs only in whether
+ * they are written to disk. Default-mode `pinAll` runs this once per
+ * resolved URL, so a CDN that accepts the connection and then stalls
+ * would otherwise hang the pin with nothing to interrupt it: there is
+ * no ambient deadline on a CLI run (#1150).
+ *
  * @param {string} url
  * @returns {Promise<string | null>}  the integrity string, or null on failure
  */
 async function fetchIntegrity(url) {
+  const controller = new AbortController();
+  const timer = setTimeout(() => controller.abort(), PIN_BUNDLE_TIMEOUT_MS);
   try {
-    const response = await fetch(url);
+    const response = await fetch(url, { signal: controller.signal });
     if (!response.ok) {
       console.error(`[webjs] hash ${url} returned ${response.status}`);
       return null;
@@ -1121,8 +1176,13 @@ async function fetchIntegrity(url) {
     const buf = new Uint8Array(await response.arrayBuffer());
     return await sha384Integrity(buf);
   } catch (e) {
-    console.error(`[webjs] hash ${url} failed: ${e && e.message}`);
+    const why = e && e.name === 'AbortError'
+      ? `timed out after ${PIN_BUNDLE_TIMEOUT_MS}ms`
+      : e && e.message;
+    console.error(`[webjs] hash ${url} failed: ${why}`);
     return null;
+  } finally {
+    clearTimeout(timer);
   }
 }
 
@@ -1254,7 +1314,7 @@ export async function pinAll(appDir, opts = {}) {
     installs.push(install);
     partsByInstall.set(spec, { pkg, version, subpath });
   }
-  const resolved = await jspmGenerate(installs, from);
+  const resolved = await jspmGenerate(installs, from, PIN_BUNDLE_TIMEOUT_MS);
 
   /** @type {Record<string, string>} */
   const importmap = {};
@@ -1496,7 +1556,11 @@ export async function listPinned(appDir) {
 // ---------------------------------------------------------------------------
 
 const NPM_REGISTRY = 'https://registry.npmjs.org';
-const NPM_TIMEOUT_MS = 10_000;
+// The npm registry is reached only by `audit`, `outdated`, and `update`,
+// all CLI commands, so it takes the same generous budget the pin bundle
+// fetch does rather than the server's. importmap-rails makes these same two
+// calls with Ruby's 60s Net::HTTP default.
+const NPM_TIMEOUT_MS = 60_000;
 
 /**
  * Fetch one URL from registry.npmjs.org with a small timeout. Returns
@@ -1696,7 +1760,7 @@ export async function updatePinned(appDir, opts = {}) {
       if (specPkg !== pkg) continue;
       const subpath = spec.slice(specPkg.length);
       const install = `${pkg}@${latest}${subpath}`;
-      const resolved = await jspmGenerate([install], from);
+      const resolved = await jspmGenerate([install], from, PIN_BUNDLE_TIMEOUT_MS);
       const newUrl = resolved[spec];
       if (!newUrl) continue;
       newImports[spec] = newUrl;
@@ -2169,7 +2233,6 @@ export async function checkImportmapCoherence(imports, opts) {
  */
 const liveIntegrityCache = new Map();
 
-const INTEGRITY_FETCH_TIMEOUT_MS = 10_000;
 // Cap concurrent bundle fetches so a large dep set does not open dozens of
 // sockets at once during warmup. Matches the bounded posture of the rest of
 // vendor.js (the jspm resolve is per-package but the network is the shared

package/webjs-config.schema.json

@@ -2,8 +2,8 @@
   "$schema": "http://json-schema.org/draft-07/schema#",
   "$id": "https://webjs.dev/schemas/webjs-config.schema.json",
   "title": "webjs config block",
-  "description": "Schema for the `webjs` object in a webjs app's package.json. Each key maps to a documented reader (a server reader, or for `dev`/`start` the CLI's readAppTasks). An unknown key is flagged (additionalProperties is false) so a typo no longer silently falls back to the default.",
-  "$comment": "Single source of truth lives in THREE co-located places that must stay in lockstep: this schema, the TS type at packages/core/src/webjs-config.d.ts, and the reader functions (readElideEnabled in dev.js, readSeedEnabled in dev.js, compileHeaderRules in headers.js, compileRedirectRules / readTrailingSlashPolicy in redirects.js, readBasePath in base-path.js, readCspConfig in csp.js, readBodyLimits / computeServerTimeouts in body-limit.js, and the CLI's readAppTasks in packages/cli/lib/app-tasks.js for dev/start, #550). Adding a webjs.* key means updating all three. See packages/server/AGENTS.md for the one documented procedure.",
+  "description": "Schema for the `webjs` object in a WebJs app's package.json. Each key maps to a documented reader, mostly a server reader and for a few keys a CLI one. An unknown key is flagged (additionalProperties is false) so a typo no longer silently falls back to the default.",
+  "$comment": "Single source of truth lives in THREE co-located places that must stay in lockstep: this schema, the TS type at packages/core/src/webjs-config.d.ts, and the reader functions. Adding a `webjs.*` key means updating all three (plus the KNOWN_KEYS drift test). The reader inventory is deliberately NOT repeated here: packages/server/AGENTS.md holds the one canonical list and the procedure, because every duplicate of that list drifted.",
   "type": "object",
   "additionalProperties": false,
   "properties": {
@@ -160,7 +160,7 @@
       "default": 5000
     },
     "dev": {
-      "description": "Dev task orchestration (#550). `before`/`parallel` are read by the CLI (readAppTasks in packages/cli/lib/app-tasks.js); `regenerate` is read by the server (readRegenerateRules in dev-regenerate.js). webjs dev runs `before` (one-shot, e.g. webjs db migrate) to completion, then `parallel` (long-lived watchers) alongside the server. Replaces the old predev hook + concurrently watchers.",
+      "description": "Dev task orchestration (#550). Split per sub-key: `before`/`parallel` are read by the CLI (readAppTasks in packages/cli/lib/app-tasks.js), while `regenerate` (readRegenerateRules in dev-regenerate.js) and `watch` (readDevWatchPathsFromApp in dev.js) are read by the server. webjs dev runs `before` (one-shot, e.g. webjs db migrate) to completion, then `parallel` (long-lived watchers) alongside the server. Replaces the old predev hook + concurrently watchers.",
       "type": "object",
       "additionalProperties": false,
       "properties": {
@@ -216,6 +216,22 @@
           "items": { "type": "string" }
         }
       }
+    },
+    "doctor": {
+      "description": "`webjs doctor` policy (#1257). Read by the CLI (readDoctorPolicy in packages/cli/lib/doctor.js), NOT the server.",
+      "type": "object",
+      "additionalProperties": false,
+      "properties": {
+        "gate": {
+          "description": "Per-check severity, keyed by the stable DOCTOR_CODES code (NODE_VERSION, UNMARKED_ASSET_LINKS, ELISION_CARRIERS, ...). `error` fails the exit, `warn` reports without failing, `off` silences the check: its finding is not printed and it cannot fail the exit, including under --strict (it still appears on the checklist as [off], in the summary's silenced count, and in full under --json). A code with no entry keeps its default: error for a hard toolchain failure, warn otherwise. This is how CI gates on a chosen subset without --strict making every warning fatal. A malformed gate is a hard error, so a typo cannot silently un-gate CI: that covers an unknown code, a bad severity, a wrong SHAPE (a non-object `doctor` or `gate`), and a misspelled sibling of `gate` such as `gates`. A result that could not check (a network or toolchain outage) is capped at warn and can never be escalated to error.",
+          "type": "object",
+          "propertyNames": { "pattern": "^[A-Z][A-Z0-9_]*$" },
+          "additionalProperties": {
+            "type": "string",
+            "enum": ["off", "warn", "error"]
+          }
+        }
+      }
     }
   }
 }