astro 4.0.5 → 4.0.7

package/dist/core/constants.js

@@ -1,4 +1,4 @@
-const ASTRO_VERSION = "4.0.5";
+const ASTRO_VERSION = "4.0.7";
 const SUPPORTED_MARKDOWN_FILE_EXTENSIONS = [
   ".markdown",
   ".mdown",

package/dist/core/dev/container.js

@@ -1,6 +1,6 @@
 import nodeFs from "node:fs";
 import * as vite from "vite";
-import { injectImageEndpoint } from "../../assets/internal.js";
+import { injectImageEndpoint } from "../../assets/endpoint/config.js";
 import {
   runHookConfigDone,
   runHookConfigSetup,

package/dist/core/dev/dev.js

@@ -21,7 +21,7 @@ async function dev(inlineConfig) {
       base: restart.container.settings.config.base
     })
   );
-  const currentVersion = "4.0.5";
+  const currentVersion = "4.0.7";
   if (currentVersion.includes("-")) {
     logger.warn("SKIP_FORMAT", msg.prerelease({ currentVersion }));
   }

package/dist/core/errors/errors-data.d.ts

@@ -622,6 +622,21 @@ export declare const MarkdownImageNotFound: {
     message: (imagePath: string, fullImagePath: string | undefined) => string;
     hint: string;
 };
+/**
+ * @docs
+ * @see
+ * - [Images](https://docs.astro.build/en/guides/images/)
+ * @description
+ * Astro could not transform one of your images. Often, this is caused by a corrupted or malformed image. Re-exporting the image from your image editor may fix this issue.
+ *
+ * Depending on the image service you are using, the stack trace may contain more information on the specific error encountered.
+ */
+export declare const CouldNotTransformImage: {
+    name: string;
+    title: string;
+    message: (imagePath: string) => string;
+    hint: string;
+};
 /**
  * @docs
  * @description
@@ -690,6 +705,25 @@ export declare const LocalsNotAnObject: {
     message: string;
     hint: string;
 };
+/**
+ * @docs
+ * @description
+ * Thrown in development mode when middleware throws an error while attempting to loading it.
+ *
+ * For example:
+ * ```ts
+ * import {defineMiddleware} from "astro:middleware";
+ * throw new Error("Error thrown while loading the middleware.")
+ * export const onRequest = defineMiddleware(() => {
+ *   return "string"
+ * });
+ * ```
+ */
+export declare const MiddlewareCantBeLoaded: {
+    name: string;
+    title: string;
+    message: string;
+};
 /**
  * @docs
  * @see

package/dist/core/errors/errors-data.js

@@ -223,6 +223,12 @@ const MarkdownImageNotFound = {
   message: (imagePath, fullImagePath) => `Could not find requested image \`${imagePath}\`${fullImagePath ? ` at \`${fullImagePath}\`.` : "."}`,
   hint: "This is often caused by a typo in the image path. Please make sure the file exists, and is spelled correctly."
 };
+const CouldNotTransformImage = {
+  name: "CouldNotTransformImage",
+  title: "Could not transform image.",
+  message: (imagePath) => `Could not transform image \`${imagePath}\`. See the stack trace for more information.`,
+  hint: "This is often caused by a corrupted or malformed image. Re-exporting the image from your image editor may fix this issue."
+};
 const ResponseSentError = {
   name: "ResponseSentError",
   title: "Unable to set response.",
@@ -230,8 +236,8 @@ const ResponseSentError = {
 };
 const MiddlewareNoDataOrNextCalled = {
   name: "MiddlewareNoDataOrNextCalled",
-  title: "The middleware didn't return a response or call `next`.",
-  message: "The middleware needs to either return a `Response` object or call the `next` function."
+  title: "The middleware didn't return a `Response`.",
+  message: "Make sure your middleware returns a `Response` object, either directly or by returning the `Response` from calling the `next` function."
 };
 const MiddlewareNotAResponse = {
   name: "MiddlewareNotAResponse",
@@ -244,6 +250,11 @@ const LocalsNotAnObject = {
   message: "`locals` can only be assigned to an object. Other values like numbers, strings, etc. are not accepted.",
   hint: "If you tried to remove some information from the `locals` object, try to use `delete` or set the property to `undefined`."
 };
+const MiddlewareCantBeLoaded = {
+  name: "MiddlewareCantBeLoaded",
+  title: "Can't load the middleware.",
+  message: "The middleware threw an error while Astro was trying to loading it."
+};
 const LocalImageUsedWrongly = {
   name: "LocalImageUsedWrongly",
   title: "Local images must be imported.",
@@ -460,6 +471,7 @@ export {
   ConfigNotFound,
   ContentCollectionTypeMismatchError,
   ContentSchemaContainsSlugError,
+  CouldNotTransformImage,
   DataCollectionEntryParseError,
   DuplicateContentEntrySlugError,
   ExpectedImage,
@@ -490,6 +502,7 @@ export {
   MarkdownFrontmatterParseError,
   MarkdownImageNotFound,
   MdxIntegrationMissingError,
+  MiddlewareCantBeLoaded,
   MiddlewareNoDataOrNextCalled,
   MiddlewareNotAResponse,
   MissingImageDimension,

package/dist/core/logger/core.d.ts

@@ -17,17 +17,18 @@ export interface LogMessage {
     label: string | null;
     level: LoggerLevel;
     message: string;
+    newLine: boolean;
 }
 export declare const levels: Record<LoggerLevel, number>;
 /** Full logging API */
-export declare function log(opts: LogOptions, level: LoggerLevel, label: string | null, message: string): void;
+export declare function log(opts: LogOptions, level: LoggerLevel, label: string | null, message: string, newLine?: boolean): void;
 export declare function isLogLevelEnabled(configuredLogLevel: LoggerLevel, level: LoggerLevel): boolean;
 /** Emit a user-facing message. Useful for UI and other console messages. */
-export declare function info(opts: LogOptions, label: string | null, message: string): void;
+export declare function info(opts: LogOptions, label: string | null, message: string, newLine?: boolean): void;
 /** Emit a warning message. Useful for high-priority messages that aren't necessarily errors. */
-export declare function warn(opts: LogOptions, label: string | null, message: string): void;
+export declare function warn(opts: LogOptions, label: string | null, message: string, newLine?: boolean): void;
 /** Emit a error message, Useful when Astro can't recover from some error. */
-export declare function error(opts: LogOptions, label: string | null, message: string): void;
+export declare function error(opts: LogOptions, label: string | null, message: string, newLine?: boolean): void;
 type LogFn = typeof info | typeof warn | typeof error;
 export declare function table(opts: LogOptions, columns: number[]): (logFn: LogFn, ...input: Array<any>) => void;
 export declare function debug(...args: any[]): void;
@@ -43,9 +44,9 @@ export declare function timerMessage(message: string, startTime?: number): strin
 export declare class Logger {
     options: LogOptions;
     constructor(options: LogOptions);
-    info(label: LoggerLabel | null, message: string): void;
-    warn(label: LoggerLabel | null, message: string): void;
-    error(label: LoggerLabel | null, message: string): void;
+    info(label: LoggerLabel | null, message: string, newLine?: boolean): void;
+    warn(label: LoggerLabel | null, message: string, newLine?: boolean): void;
+    error(label: LoggerLabel | null, message: string, newLine?: boolean): void;
     debug(label: LoggerLabel, ...messages: any[]): void;
     level(): LoggerLevel;
     forkIntegrationLogger(label: string): AstroIntegrationLogger;

package/dist/core/logger/core.js

@@ -13,13 +13,14 @@ const levels = {
   error: 50,
   silent: 90
 };
-function log(opts, level, label, message) {
+function log(opts, level, label, message, newLine = true) {
   const logLevel = opts.level;
   const dest = opts.dest;
   const event = {
     label,
     level,
-    message
+    message,
+    newLine
   };
   if (!isLogLevelEnabled(logLevel, level)) {
     return;
@@ -29,14 +30,14 @@ function log(opts, level, label, message) {
 function isLogLevelEnabled(configuredLogLevel, level) {
   return levels[configuredLogLevel] <= levels[level];
 }
-function info(opts, label, message) {
-  return log(opts, "info", label, message);
+function info(opts, label, message, newLine = true) {
+  return log(opts, "info", label, message, newLine);
 }
-function warn(opts, label, message) {
-  return log(opts, "warn", label, message);
+function warn(opts, label, message, newLine = true) {
+  return log(opts, "warn", label, message, newLine);
 }
-function error(opts, label, message) {
-  return log(opts, "error", label, message);
+function error(opts, label, message, newLine = true) {
+  return log(opts, "error", label, message, newLine);
 }
 function table(opts, columns) {
   return function logTable(logFn, ...input) {
@@ -107,14 +108,14 @@ class Logger {
   constructor(options) {
     this.options = options;
   }
-  info(label, message) {
-    info(this.options, label, message);
+  info(label, message, newLine = true) {
+    info(this.options, label, message, newLine);
   }
-  warn(label, message) {
-    warn(this.options, label, message);
+  warn(label, message, newLine = true) {
+    warn(this.options, label, message, newLine);
   }
-  error(label, message) {
-    error(this.options, label, message);
+  error(label, message, newLine = true) {
+    error(this.options, label, message, newLine);
   }
   debug(label, ...messages) {
     debug(label, ...messages);

package/dist/core/logger/node.js

@@ -1,15 +1,16 @@
 import debugPackage from "debug";
 import { getEventPrefix, levels } from "./core.js";
 const nodeLogDestination = {
-  write(event) {
+  write(event, newLine = true) {
     let dest = process.stderr;
     if (levels[event.level] < levels["error"]) {
       dest = process.stdout;
     }
+    let trailingLine = event.newLine ? "\n" : "";
     if (event.label === "SKIP_FORMAT") {
-      dest.write(event.message + "\n");
+      dest.write(event.message + trailingLine);
     } else {
-      dest.write(getEventPrefix(event) + " " + event.message + "\n");
+      dest.write(getEventPrefix(event) + " " + event.message + trailingLine);
     }
     return true;
   }

package/dist/core/messages.js

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

package/dist/core/middleware/loadMiddleware.d.ts

@@ -4,4 +4,4 @@ import type { ModuleLoader } from '../module-loader/index.js';
  *
  * If not middlewares were not set, the function returns an empty array.
  */
-export declare function loadMiddleware(moduleLoader: ModuleLoader): Promise<Record<string, any> | undefined>;
+export declare function loadMiddleware(moduleLoader: ModuleLoader): Promise<Record<string, any>>;

package/dist/core/middleware/loadMiddleware.js

@@ -1,10 +1,12 @@
 import { MIDDLEWARE_MODULE_ID } from "./vite-plugin.js";
+import { MiddlewareCantBeLoaded } from "../errors/errors-data.js";
+import { AstroError } from "../errors/index.js";
 async function loadMiddleware(moduleLoader) {
   try {
-    const module = await moduleLoader.import(MIDDLEWARE_MODULE_ID);
-    return module;
-  } catch {
-    return void 0;
+    return await moduleLoader.import(MIDDLEWARE_MODULE_ID);
+  } catch (error) {
+    const astroError = new AstroError(MiddlewareCantBeLoaded, void 0, { cause: error });
+    throw astroError;
   }
 }
 export {

package/dist/core/redirects/helpers.js

@@ -10,7 +10,13 @@ function redirectRouteGenerate(redirectRoute, data) {
   if (typeof routeData !== "undefined") {
     return routeData?.generate(data) || routeData?.pathname || "/";
   } else if (typeof route === "string") {
-    return route;
+    let target = route;
+    for (const param of Object.keys(data)) {
+      const paramValue = data[param];
+      target = target.replace(`[${param}]`, paramValue);
+      target = target.replace(`[...${param}]`, paramValue);
+    }
+    return target;
   } else if (typeof route === "undefined") {
     return "/";
   }

package/dist/core/routing/manifest/create.js

@@ -318,7 +318,7 @@ This route collides with: "${collision.component}".`
       if (/^https?:\/\//.test(destination)) {
         logger.warn(
           "redirects",
-          `Redirecting to an external URL is not officially supported: ${from} -> ${to}`
+          `Redirecting to an external URL is not officially supported: ${from} -> ${destination}`
         );
       }
     }

package/dist/runtime/client/dev-overlay/plugins/audit/a11y.js

@@ -33,6 +33,7 @@ const a11y_required_attributes = {
   object: ["title", "aria-label", "aria-labelledby"]
 };
 const interactiveElements = ["button", "details", "embed", "iframe", "label", "select", "textarea"];
+const labellableElements = ["input", "meter", "output", "progress", "select", "textarea"];
 const aria_non_interactive_roles = [
   "alert",
   "alertdialog",
@@ -204,7 +205,7 @@ const a11y = [
   {
     code: "a11y-aria-activedescendant-has-tabindex",
     title: "Elements with attribute `aria-activedescendant` must be tabbable",
-    message: "This element must either have an inherent `tabindex` or declare `tabindex` as an attribute.",
+    message: "Element with the `aria-activedescendant` attribute must either have an inherent `tabindex` or declare `tabindex` as an attribute.",
     selector: "[aria-activedescendant]",
     match(element) {
       if (!element.tabIndex && !element.hasAttribute("tabindex"))
@@ -268,13 +269,17 @@ const a11y = [
     selector: 'a[href]:is([href=""], [href="#"], [href^="javascript:" i])'
   },
   {
-    code: "a11y-label-has-associated-control",
-    title: "`label` tag should have an associated control and a text content.",
-    message: "The `label` tag must be associated with a control using either `for` or having a nested input. Additionally, the `label` tag must have text content.",
-    selector: "label:not([for])",
+    code: "a11y-invalid-label",
+    title: "`label` element should have an associated control and a text content.",
+    message: "The `label` element must be associated with a control either by using the `for` attribute or by containing a nested form element. Additionally, the `label` element must have text content.",
+    selector: "label",
     match(element) {
-      const inputChild = element.querySelector("input");
-      if (!inputChild?.textContent)
+      const hasFor = element.hasAttribute("for");
+      const nestedLabellableElement = element.querySelector(`${labellableElements.join(", ")}`);
+      if (!hasFor && !nestedLabellableElement)
+        return true;
+      const innerText = element.innerText.trim();
+      if (innerText === "")
         return true;
     }
   },
@@ -327,7 +332,8 @@ const a11y = [
     message: "Headings and anchors must have content to be accessible.",
     selector: a11y_required_content.join(","),
     match(element) {
-      if (!element.textContent)
+      const innerText = element.innerText.trim();
+      if (innerText === "")
         return true;
     }
   },

package/dist/transitions/router.js

@@ -1,11 +1,13 @@
 import { TRANSITION_AFTER_SWAP, doPreparation, doSwap } from "./events.js";
+const inBrowser = import.meta.env.SSR === false;
+const pushState = inBrowser && history.pushState.bind(history);
+const replaceState = inBrowser && history.replaceState.bind(history);
 const updateScrollPosition = (positions) => {
   if (history.state) {
     history.scrollRestoration = "manual";
-    history.replaceState({ ...history.state, ...positions }, "");
+    replaceState({ ...history.state, ...positions }, "");
   }
 };
-const inBrowser = import.meta.env.SSR === false;
 const supportsViewTransitions = inBrowser && !!document.startViewTransition;
 const transitionEnabledOnThisPage = () => inBrowser && !!document.querySelector('[name="astro-view-transitions-enabled"]');
 const samePage = (thisLocation, otherLocation) => thisLocation.pathname === otherLocation.pathname && thisLocation.search === otherLocation.search;
@@ -43,7 +45,7 @@ if (inBrowser) {
     currentHistoryIndex = history.state.index;
     scrollTo({ left: history.state.scrollX, top: history.state.scrollY });
   } else if (transitionEnabledOnThisPage()) {
-    history.replaceState({ index: currentHistoryIndex, scrollX, scrollY }, "");
+    replaceState({ index: currentHistoryIndex, scrollX, scrollY }, "");
     history.scrollRestoration = "manual";
   }
 }
@@ -69,7 +71,8 @@ const throttle = (cb, delay) => {
 async function fetchHTML(href, init) {
   try {
     const res = await fetch(href, init);
-    const mediaType = res.headers.get("content-type")?.replace(/;.*$/, "");
+    const contentType = res.headers.get("content-type") ?? "";
+    const mediaType = contentType.split(";", 1)[0].trim();
     if (mediaType !== "text/html" && mediaType !== "application/xhtml+xml") {
       return null;
     }
@@ -117,7 +120,7 @@ const moveToLocation = (to, from, options, historyState) => {
   if (to.href !== location.href && !historyState) {
     if (options.history === "replace") {
       const current = history.state;
-      history.replaceState(
+      replaceState(
         {
           ...options.state,
           index: current.index,
@@ -128,7 +131,7 @@ const moveToLocation = (to, from, options, historyState) => {
         to.href
       );
     } else {
-      history.pushState(
+      pushState(
         { ...options.state, index: ++currentHistoryIndex, scrollX: 0, scrollY: 0 },
         "",
         to.href
@@ -315,7 +318,8 @@ async function transition(direction, from, to, options, historyState) {
     const init = {};
     if (preparationEvent.formData) {
       init.method = "POST";
-      init.body = preparationEvent.formData;
+      const form = preparationEvent.sourceElement instanceof HTMLFormElement ? preparationEvent.sourceElement : preparationEvent.sourceElement instanceof HTMLElement && "form" in preparationEvent.sourceElement ? preparationEvent.sourceElement.form : preparationEvent.sourceElement?.closest("form");
+      init.body = form?.attributes.getNamedItem("enctype")?.value === "application/x-www-form-urlencoded" ? new URLSearchParams(preparationEvent.formData) : preparationEvent.formData;
     }
     const response = await fetchHTML(href, init);
     if (response === null) {

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

@@ -1,8 +1,4 @@
-import { collectErrorMetadata } from "../core/errors/dev/index.js";
-import { createSafeError } from "../core/errors/index.js";
-import { formatErrorMessage } from "../core/messages.js";
 import { collapseDuplicateSlashes, removeTrailingForwardSlash } from "../core/path.js";
-import { eventError, telemetry } from "../events/index.js";
 import { isServerLikeOutput } from "../prerender/utils.js";
 import { runWithErrorHandling } from "./controller.js";
 import { handle500Response } from "./response.js";

package/dist/vite-plugin-dev-overlay/vite-plugin-dev-overlay.js

@@ -12,7 +12,7 @@ function astroDevOverlay({ settings }) {
       if (id === resolvedVirtualModuleId) {
         return `
 					export const loadDevOverlayPlugins = async () => {
-						return [${settings.devToolbarApps.map((plugin) => `(await import('${plugin}')).default`).join(",")}];
+						return [${settings.devToolbarApps.map((plugin) => `(await import(${JSON.stringify(plugin)})).default`).join(",")}];
 					};
 				`;
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "astro",
-  "version": "4.0.5",
+  "version": "4.0.7",
   "description": "Astro is a modern site builder with web best practices, performance, and DX front-of-mind.",
   "type": "module",
   "author": "withastro",
@@ -99,7 +99,7 @@
     "vendor"
   ],
   "dependencies": {
-    "@astrojs/compiler": "^2.3.2",
+    "@astrojs/compiler": "^2.3.4",
     "@babel/core": "^7.23.3",
     "@babel/generator": "^7.23.3",
     "@babel/parser": "^7.23.3",
@@ -137,7 +137,7 @@
     "mime": "^3.0.0",
     "ora": "^7.0.1",
     "p-limit": "^5.0.0",
-    "p-queue": "^7.4.1",
+    "p-queue": "^8.0.1",
     "path-to-regexp": "^6.2.1",
     "preferred-pm": "^3.1.2",
     "probe-image-size": "^7.2.3",
@@ -152,7 +152,7 @@
     "tsconfck": "^3.0.0",
     "unist-util-visit": "^5.0.0",
     "vfile": "^6.0.1",
-    "vite": "^5.0.0",
+    "vite": "^5.0.10",
     "vitefu": "^0.2.5",
     "which-pm": "^2.1.1",
     "yargs-parser": "^21.1.1",