astro 5.1.6 → 5.1.7

package/dist/assets/layout.js

@@ -84,10 +84,14 @@ const getSizesAttribute = ({
     return void 0;
   }
   switch (layout) {
+    // If screen is wider than the max size then image width is the max size,
+    // otherwise it's the width of the screen
     case `responsive`:
       return `(min-width: ${width}px) ${width}px, 100vw`;
+    // Image is always the same width, whatever the size of the screen
     case `fixed`:
       return `${width}px`;
+    // Image is always the width of the screen
     case `full-width`:
       return `100vw`;
     case "none":

package/dist/cli/add/index.js

@@ -307,6 +307,8 @@ async function add(names, { flags }) {
       logger.info("SKIP_FORMAT", msg.success(`Configuration up-to-date.`));
       break;
     }
+    // NOTE: failure shouldn't happen in practice because `updateAstroConfig` doesn't return that.
+    // Pipe this to the same handling as `UpdateResult.updated` for now.
     case 3 /* failure */:
     case 1 /* updated */:
     case void 0: {

package/dist/content/content-layer.js

@@ -148,7 +148,7 @@ ${contentConfig.error.message}`);
       logger.info("Content config changed");
       shouldClear = true;
     }
-    if (previousAstroVersion && previousAstroVersion !== "5.1.6") {
+    if (previousAstroVersion && previousAstroVersion !== "5.1.7") {
       logger.info("Astro version changed");
       shouldClear = true;
     }
@@ -156,8 +156,8 @@ ${contentConfig.error.message}`);
       logger.info("Clearing content store");
       this.#store.clearAll();
     }
-    if ("5.1.6") {
-      await this.#store.metaStore().set("astro-version", "5.1.6");
+    if ("5.1.7") {
+      await this.#store.metaStore().set("astro-version", "5.1.7");
     }
     if (currentConfigDigest) {
       await this.#store.metaStore().set("content-config-digest", currentConfigDigest);

package/dist/core/constants.js

@@ -1,4 +1,4 @@
-const ASTRO_VERSION = "5.1.6";
+const ASTRO_VERSION = "5.1.7";
 const REROUTE_DIRECTIVE_HEADER = "X-Astro-Reroute";
 const REWRITE_DIRECTIVE_HEADER_KEY = "X-Astro-Rewrite";
 const REWRITE_DIRECTIVE_HEADER_VALUE = "yes";

package/dist/core/dev/dev.js

@@ -22,7 +22,7 @@ async function dev(inlineConfig) {
   await telemetry.record([]);
   const restart = await createContainerWithAutomaticRestart({ inlineConfig, fs });
   const logger = restart.container.logger;
-  const currentVersion = "5.1.6";
+  const currentVersion = "5.1.7";
   const isPrerelease = currentVersion.includes("-");
   if (!isPrerelease) {
     try {

package/dist/core/messages.js

@@ -38,7 +38,7 @@ function serverStart({
   host,
   base
 }) {
-  const version = "5.1.6";
+  const version = "5.1.7";
   const localPrefix = `${dim("\u2503")} Local    `;
   const networkPrefix = `${dim("\u2503")} Network  `;
   const emptyPrefix = " ".repeat(11);
@@ -276,7 +276,7 @@ function printHelp({
     message.push(
       linebreak(),
       `  ${bgGreen(black(` ${commandName} `))} ${green(
-        `v${"5.1.6"}`
+        `v${"5.1.7"}`
       )} ${headline}`
     );
   }

package/dist/core/routing/match.d.ts

@@ -4,6 +4,8 @@ import type { RouteData } from '../../types/public/internal.js';
 export declare function matchRoute(pathname: string, manifest: ManifestData): RouteData | undefined;
 /** Finds all matching routes from pathname */
 export declare function matchAllRoutes(pathname: string, manifest: ManifestData): RouteData[];
+export declare function isRoute404(route: string): boolean;
+export declare function isRoute500(route: string): boolean;
 /**
  * Determines if the given route matches a 404 or 500 error page.
  *

package/dist/core/routing/match.js

@@ -7,11 +7,21 @@ function matchRoute(pathname, manifest) {
 function matchAllRoutes(pathname, manifest) {
   return manifest.routes.filter((route) => route.pattern.test(decodeURI(pathname)));
 }
+const ROUTE404_RE = /^\/404\/?$/;
+const ROUTE500_RE = /^\/500\/?$/;
+function isRoute404(route) {
+  return ROUTE404_RE.test(route);
+}
+function isRoute500(route) {
+  return ROUTE500_RE.test(route);
+}
 function isRoute404or500(route) {
-  return route.pattern.test("/404") || route.pattern.test("/500");
+  return isRoute404(route.route) || isRoute500(route.route);
 }
 export {
+  isRoute404,
   isRoute404or500,
+  isRoute500,
   matchAllRoutes,
   matchRoute
 };

package/dist/i18n/index.js

@@ -3,6 +3,7 @@ import { shouldAppendForwardSlash } from "../core/build/util.js";
 import { REROUTE_DIRECTIVE_HEADER } from "../core/constants.js";
 import { MissingLocale, i18nNoLocaleFoundInPath } from "../core/errors/errors-data.js";
 import { AstroError } from "../core/errors/index.js";
+import { isRoute404, isRoute500 } from "../core/routing/match.js";
 import { createI18nMiddleware } from "./middleware.js";
 function requestHasLocale(locales) {
   return function(context) {
@@ -11,7 +12,8 @@ function requestHasLocale(locales) {
 }
 function requestIs404Or500(request, base = "") {
   const url = new URL(request.url);
-  return url.pathname.startsWith(`${base}/404`) || url.pathname.startsWith(`${base}/500`);
+  const pathname = url.pathname.slice(base.length);
+  return isRoute404(pathname) || isRoute500(pathname);
 }
 function pathHasLocale(path, locales) {
   const segments = path.split("/");

package/dist/i18n/middleware.js

@@ -60,6 +60,7 @@ function createI18nMiddleware(i18n, base, trailingSlash, format) {
     }
     const { currentLocale } = context;
     switch (i18n.strategy) {
+      // NOTE: theoretically, we should never hit this code path
       case "manual": {
         return response;
       }

package/dist/jsx/rehype.js

@@ -111,14 +111,17 @@ function findMatchingImport(tagName, imports) {
       if (local === tagSpecifier) {
         if (tagSpecifier !== tagName) {
           switch (imported) {
+            // Namespace import: "<buttons.Foo.Bar />" => name: "Foo.Bar"
             case "*": {
               const accessPath = tagName.slice(tagSpecifier.length + 1);
               return { name: accessPath, path: source };
             }
+            // Default import: "<buttons.Foo.Bar />" => name: "default.Foo.Bar"
             case "default": {
               const accessPath = tagName.slice(tagSpecifier.length + 1);
               return { name: `default.${accessPath}`, path: source };
             }
+            // Named import: "<buttons.Foo.Bar />" => name: "buttons.Foo.Bar"
             default: {
               return { name: tagName, path: source };
             }

package/dist/runtime/server/hydration.js

@@ -37,6 +37,8 @@ function extractDirectives(inputProps, clientDirectives) {
           extracted.hydration.componentExport.value = value;
           break;
         }
+        // This is a special prop added to prove that the client hydration method
+        // was added statically.
         case "client:component-hydration": {
           break;
         }

package/dist/vite-plugin-astro-server/route.js

@@ -14,6 +14,7 @@ import { getProps } from "../core/render/index.js";
 import { createRequest } from "../core/request.js";
 import { redirectTemplate } from "../core/routing/3xx.js";
 import { matchAllRoutes } from "../core/routing/index.js";
+import { isRoute404, isRoute500 } from "../core/routing/match.js";
 import { PERSIST_SYMBOL } from "../core/session.js";
 import { getSortedPreloadedMatches } from "../prerender/routing.js";
 import { writeSSRResult, writeWebResponse } from "./response.js";
@@ -21,12 +22,10 @@ function isLoggedRequest(url) {
   return url !== "/favicon.ico";
 }
 function getCustom404Route(manifestData) {
-  const route404 = /^\/404\/?$/;
-  return manifestData.routes.find((r) => route404.test(r.route));
+  return manifestData.routes.find((r) => isRoute404(r.route));
 }
 function getCustom500Route(manifestData) {
-  const route500 = /^\/500\/?$/;
-  return manifestData.routes.find((r) => route500.test(r.route));
+  return manifestData.routes.find((r) => isRoute500(r.route));
 }
 async function matchRoute(pathname, manifestData, pipeline) {
   const { config, logger, routeCache, serverLike, settings } = pipeline;
@@ -175,7 +174,9 @@ async function handleRoute({
       })
     );
   }
-  if (statusCode === 404 && response.headers.get(REROUTE_DIRECTIVE_HEADER) !== "no") {
+  if (statusCode === 404 && // If the body isn't null, that means the user sets the 404 status
+  // but uses the current route to handle the 404
+  response.body === null && response.headers.get(REROUTE_DIRECTIVE_HEADER) !== "no") {
     const fourOhFourRoute = await matchRoute("/404", manifestData, pipeline);
     if (fourOhFourRoute) {
       renderContext = await RenderContext.create({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "astro",
-  "version": "5.1.6",
+  "version": "5.1.7",
   "description": "Astro is a modern site builder with web best practices, performance, and DX front-of-mind.",
   "type": "module",
   "author": "withastro",
@@ -105,7 +105,7 @@
   "dependencies": {
     "@astrojs/compiler": "^2.10.3",
     "@oslojs/encoding": "^1.1.0",
-    "@rollup/pluginutils": "^5.1.3",
+    "@rollup/pluginutils": "^5.1.4",
     "@types/cookie": "^0.6.0",
     "acorn": "^8.14.0",
     "aria-query": "^5.3.2",
@@ -116,51 +116,51 @@
     "common-ancestor-path": "^1.0.1",
     "cookie": "^0.7.2",
     "cssesc": "^3.0.0",
-    "debug": "^4.3.7",
+    "debug": "^4.4.0",
     "deterministic-object-hash": "^2.0.2",
     "devalue": "^5.1.1",
     "diff": "^5.2.0",
     "dlv": "^1.1.3",
     "dset": "^3.1.4",
-    "es-module-lexer": "^1.5.4",
-    "esbuild": "^0.21.5",
+    "es-module-lexer": "^1.6.0",
+    "esbuild": "^0.24.2",
     "estree-walker": "^3.0.3",
-    "fast-glob": "^3.3.2",
+    "fast-glob": "^3.3.3",
     "flattie": "^1.1.1",
     "github-slugger": "^2.0.0",
     "html-escaper": "^3.0.3",
     "http-cache-semantics": "^4.1.1",
     "js-yaml": "^4.1.0",
     "kleur": "^4.1.5",
-    "magic-string": "^0.30.14",
+    "magic-string": "^0.30.17",
     "magicast": "^0.3.5",
     "micromatch": "^4.0.8",
     "mrmime": "^2.0.0",
     "neotraverse": "^0.6.18",
-    "p-limit": "^6.1.0",
+    "p-limit": "^6.2.0",
     "p-queue": "^8.0.1",
     "preferred-pm": "^4.0.0",
     "prompts": "^2.4.2",
     "rehype": "^13.0.2",
     "semver": "^7.6.3",
-    "shiki": "^1.23.1",
-    "tinyexec": "^0.3.1",
+    "shiki": "^1.26.2",
+    "tinyexec": "^0.3.2",
     "tsconfck": "^3.1.4",
     "ultrahtml": "^1.5.3",
     "unist-util-visit": "^5.0.0",
-    "unstorage": "^1.14.0",
+    "unstorage": "^1.14.4",
     "vfile": "^6.0.3",
-    "vite": "^6.0.5",
-    "vitefu": "^1.0.4",
+    "vite": "^6.0.7",
+    "vitefu": "^1.0.5",
     "which-pm": "^3.0.0",
     "xxhash-wasm": "^1.1.0",
     "yargs-parser": "^21.1.1",
-    "yocto-spinner": "^0.1.0",
-    "zod": "^3.23.8",
-    "zod-to-json-schema": "^3.23.5",
+    "yocto-spinner": "^0.1.2",
+    "zod": "^3.24.1",
+    "zod-to-json-schema": "^3.24.1",
     "zod-to-ts": "^1.2.0",
     "@astrojs/internal-helpers": "0.4.2",
-    "@astrojs/markdown-remark": "6.0.1",
+    "@astrojs/markdown-remark": "6.0.2",
     "@astrojs/telemetry": "3.2.0"
   },
   "optionalDependencies": {
@@ -168,7 +168,7 @@
   },
   "devDependencies": {
     "@astrojs/check": "^0.9.4",
-    "@playwright/test": "^1.49.0",
+    "@playwright/test": "^1.49.1",
     "@types/aria-query": "^5.0.4",
     "@types/common-ancestor-path": "^1.0.2",
     "@types/cssesc": "^3.0.2",
@@ -189,18 +189,18 @@
     "expect-type": "^1.1.0",
     "fs-fixture": "^2.6.0",
     "mdast-util-mdx": "^3.0.0",
-    "mdast-util-mdx-jsx": "^3.1.3",
-    "node-mocks-http": "^1.16.1",
+    "mdast-util-mdx-jsx": "^3.2.0",
+    "node-mocks-http": "^1.16.2",
     "parse-srcset": "^1.0.2",
     "rehype-autolink-headings": "^7.1.0",
     "rehype-slug": "^6.0.0",
     "rehype-toc": "^3.0.2",
     "remark-code-titles": "^0.1.2",
-    "rollup": "^4.27.4",
-    "sass": "^1.81.0",
-    "undici": "^7.2.0",
+    "rollup": "^4.30.1",
+    "sass": "^1.83.1",
+    "undici": "^7.2.1",
     "unified": "^11.0.5",
-    "vitest": "^3.0.0-beta.3",
+    "vitest": "^3.0.0-beta.4",
     "astro-scripts": "0.0.14"
   },
   "engines": {