@taprootio/site-authoring 0.12.5 → 0.12.6

package/README.md

@@ -537,8 +537,8 @@ completed deployment for the target, HTTP delivery of the authored routes and
 the favicon, images, module preloads and site bundle they declare (status and
 content type, no redirects followed), internal link targets, accidental
 local-only references, and the pinned runtime — the version the page names,
-its entry module and the capability modules the page declares, with a page
-that still names the pre-pinning runtime pointer reported as unverified until it is republished. The
+its entry module and the capability modules the page declares (a page that
+pins no runtime version and entry fails the runtime check). The
 browser dimension (a fresh and a returning load
 compared against the page's pinned entry, every document request in the browser,
 popups included, held to the site origin before it is sent, and cache reuse

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@taprootio/site-authoring",
-  "version": "0.12.5",
+  "version": "0.12.6",
   "description": "Taproot site authoring CLI: sign in once in the browser, then pull, validate, push, preview, approve, and deploy a site from the terminal",
   "type": "module",
   "license": "ISC",

package/src/delivery-check.js

@@ -205,8 +205,6 @@ export function parseBootstrap(html) {
   return {
     version: text(parsed.version).slice(0, 64),
     entry: text(parsed.entry),
-    // A release published before runtime pinning named a mutable pointer instead.
-    legacyPointer: typeof parsed.runtimeManifestUrl === "string" && parsed.runtimeManifestUrl !== "",
     siteBundleUrl: text(parsed.siteBundleUrl),
     runtimeCapabilities,
     capabilities,
@@ -506,9 +504,6 @@ async function observe(context, origin, sampledRoutes) {
     }
   }
   runtime.pinned = runtime.version !== "" && Boolean(bootstrap?.entry);
-  // Such a page still loads in a browser, but this check no longer reads the
-  // pointer, so its runtime is reported unverified rather than as passed or failed.
-  runtime.legacy = !runtime.pinned && Boolean(bootstrap?.legacyPointer);
   const entryUrl = runtime.pinned ? admit(bootstrap.entry) : undefined;
   if (entryUrl) {
     const { row } = await probe(context, entryUrl.href, "runtime-entry");
@@ -570,10 +565,7 @@ async function observe(context, origin, sampledRoutes) {
     ...assetResults.filter((row) => !row.ok).map((row) => `${row.kind} ${row.url}: ${row.failure}`),
     ...linkResults.filter((row) => !row.ok).map((row) => `link ${row.path}: ${row.failure}`),
     ...(runtime.declared ? [] : ["runtime: the home page declares no runtime bootstrap"]),
-    ...(runtime.declared && runtime.legacy
-      ? ["runtime: the page names a pre-pinning runtime pointer, which is not verified here; republish the site so it pins a runtime version"]
-      : []),
-    ...(runtime.declared && !runtime.pinned && !runtime.legacy
+    ...(runtime.declared && !runtime.pinned
       ? ["runtime: the page does not pin a runtime version and entry"]
       : []),
     ...(runtime.declared && runtime.pinned && !runtime.entry.ok
@@ -677,7 +669,7 @@ export async function checkDelivery({
 
   const verdict = observed.failuresTotal === 0
     ? "delivered"
-    : observed.routeRows.every((row) => row.ok) && (observed.runtime.compatible || observed.runtime.legacy)
+    : observed.routeRows.every((row) => row.ok) && observed.runtime.compatible
     ? "degraded"
     : "failed";
   const report = {
@@ -757,7 +749,6 @@ function withinOutputBudget(report, failuresTotal) {
         declared: runtime.declared,
         version: runtime.version,
         pinned: runtime.pinned,
-        legacy: runtime.legacy,
         capabilities: runtime.capabilities.length,
         capabilitiesResolvable: runtime.capabilitiesResolvable,
         compatible: runtime.compatible,

package/src/reference-help.js

@@ -480,8 +480,7 @@ const WORKFLOW_REFERENCES = Object.freeze({
       + "image fails.",
       "Runtime: the page's bootstrap pins one runtime version, its entry module and the module each declared "
       + "capability resolves to. The entry must load as JavaScript and every declared capability must resolve and "
-      + "load. A page that still names the pre-pinning runtime pointer is reported as unverified, so the verdict is "
-      + "degraded, until the site is republished.",
+      + "load. A page whose bootstrap does not pin a runtime version and entry fails the runtime check.",
       "Browser: a fresh and a returning (same-context, cached) load are compared against the page's pinned entry when "
       + "Playwright resolves from the CLI's own install: npm install --global @taprootio/site-authoring@latest "
       + "playwright && npx playwright install chromium, then rerun through the installed taproot-site command (a "
@@ -505,8 +504,8 @@ const WORKFLOW_REFERENCES = Object.freeze({
       + "not delivery), local-only references (localhost, *.test, *.local) are reported rather than fetched, a "
       + "redirect Location that could reflect a credential is withheld, bodies are read up to a fixed bound, "
       + "member-only content is not exercised, and nothing is purged, republished or rolled back.",
-      "Verdicts: delivered (no findings), degraded (every route served and the runtime is compatible or a pre-pinning "
-      + "reference, but an asset, link or browser load did not match or the runtime could not be verified), failed (a route or the runtime did not deliver).",
+      "Verdicts: delivered (no findings), degraded (every route served and the runtime is compatible, "
+      + "but an asset, link or browser load did not match), failed (a route or the runtime did not deliver).",
     ]),
     example: Object.freeze({
       command: `${CLI_BINARY_NAME} delivery check --production --url https://www.example.com/ --wait 30`,

package/src/verbs/delivery-check.js

@@ -156,13 +156,6 @@ export async function deliveryCheck(invocation) {
     let browser;
     if (invocation.browser === false) {
       browser = { status: "unchecked", reason: "disabled", note: "Browser verification was not requested." };
-    } else if (report.runtime.legacy) {
-      // There is no pinned entry to compare a browser load against.
-      browser = {
-        status: "unchecked",
-        reason: "legacy_runtime_reference",
-        note: "The page names a pre-pinning runtime pointer, so no entry is pinned to compare a browser load against; republish the site and rerun.",
-      };
     } else {
       onProgress("Browser dimension: probing with Playwright when it is installed.");
       browser = await probeDeliveryWithBrowser({