astro 4.0.6 → 4.0.7

package/components/ViewTransitions.astro

@@ -94,7 +94,7 @@ const { fallback = 'animate' } = Astro.props;
 
 		document.addEventListener('submit', (ev) => {
 			let el = ev.target as HTMLElement;
-			if (el.tagName !== 'FORM' || isReloadEl(el)) {
+			if (el.tagName !== 'FORM' || ev.defaultPrevented || isReloadEl(el)) {
 				return;
 			}
 			const form = el as HTMLFormElement;

package/dist/@types/astro.d.ts

@@ -589,7 +589,7 @@ export interface AstroUserConfig {
      * [See our Server-side Rendering guide](https://docs.astro.build/en/guides/server-side-rendering/) for more on SSR, and [our deployment guides](https://docs.astro.build/en/guides/deploy/) for a complete list of hosts.
      *
      * ```js
-     * import netlify from '@astrojs/netlify/functions';
+     * import netlify from '@astrojs/netlify';
      * {
      *   // Example: Build for Netlify serverless deployment
      *   adapter: netlify(),

package/dist/assets/build/generate.js

@@ -3,6 +3,8 @@ import fs, { readFileSync } from "node:fs";
 import { basename, join } from "node:path/posix";
 import { getOutDirWithinCwd } from "../../core/build/common.js";
 import { getTimeStat } from "../../core/build/util.js";
+import { AstroError } from "../../core/errors/errors.js";
+import { AstroErrorData } from "../../core/errors/index.js";
 import { isRemotePath, prependForwardSlash } from "../../core/path.js";
 import { isServerLikeOutput } from "../../prerender/utils.js";
 import { getConfiguredImageService } from "../internal.js";
@@ -52,9 +54,9 @@ function getFullImagePath(originalFilePath, env) {
 async function generateImagesForPath(originalFilePath, transformsAndPath, env, queue) {
   const originalImageData = await loadImage(originalFilePath, env);
   for (const [_, transform] of transformsAndPath.transforms) {
-    queue.add(
-      async () => generateImage(originalImageData, transform.finalPath, transform.transform)
-    );
+    await queue.add(async () => generateImage(originalImageData, transform.finalPath, transform.transform)).catch((e) => {
+      throw e;
+    });
   }
   if (!env.isSSR && !isRemotePath(originalFilePath) && !globalThis.astroAsset.referencedImages?.has(transformsAndPath.originalSrcPath)) {
     try {
@@ -117,11 +119,23 @@ async function generateImagesForPath(originalFilePath, transformsAndPath, env, q
       expires: originalImage.expires
     };
     const imageService = await getConfiguredImageService();
-    resultData.data = (await imageService.transform(
-      originalImage.data,
-      { ...options, src: originalImagePath },
-      env.imageConfig
-    )).data;
+    try {
+      resultData.data = (await imageService.transform(
+        originalImage.data,
+        { ...options, src: originalImagePath },
+        env.imageConfig
+      )).data;
+    } catch (e) {
+      const error = new AstroError(
+        {
+          ...AstroErrorData.CouldNotTransformImage,
+          message: AstroErrorData.CouldNotTransformImage.message(originalFilePath)
+        },
+        void 0,
+        { cause: e }
+      );
+      throw error;
+    }
     try {
       if (env.useCache) {
         if (isLocalImage) {

package/dist/assets/endpoint/generic.js

@@ -50,6 +50,7 @@ const GET = async ({ request }) => {
       }
     });
   } catch (err) {
+    console.error("Could not process image request:", err);
     return new Response(`Server Error: ${err}`, { status: 500 });
   }
 };

package/dist/assets/endpoint/node.js

@@ -65,6 +65,7 @@ const GET = async ({ request }) => {
       }
     });
   } catch (err) {
+    console.error("Could not process image request:", err);
     return new Response(`Server Error: ${err}`, { status: 500 });
   }
 };

package/dist/cli/add/index.js

@@ -56,7 +56,7 @@ const LIT_NPMRC_STUB = `# Lit libraries are required to be hoisted due to depend
 public-hoist-pattern[]=*lit*
 `;
 const OFFICIAL_ADAPTER_TO_IMPORT_MAP = {
-  netlify: "@astrojs/netlify/functions",
+  netlify: "@astrojs/netlify",
   vercel: "@astrojs/vercel/serverless",
   cloudflare: "@astrojs/cloudflare",
   node: "@astrojs/node"

package/dist/core/build/generate.js

@@ -168,7 +168,7 @@ ${bgGreen(black(` ${verb} static routes `))}`);
     const totalCount = Array.from(staticImageList.values()).map((x) => x.transforms.size).reduce((a, b) => a + b, 0);
     const cpuCount = os.cpus().length;
     const assetsCreationEnvironment = await prepareAssetsGenerationEnv(pipeline, totalCount);
-    const queue = new PQueue({ concurrency: cpuCount });
+    const queue = new PQueue({ concurrency: Math.max(cpuCount, 1) });
     const assetsTimer = performance.now();
     for (const [originalPath, transforms] of staticImageList) {
       await generateImagesForPath(originalPath, transforms, assetsCreationEnvironment, queue);
@@ -230,13 +230,14 @@ async function generatePage(pageData, ssrEntry, builtPaths, pipeline) {
     for (let i = 0; i < paths.length; i++) {
       const path = paths[i];
       pipeline.getEnvironment().logger.debug("build", `Generating: ${path}`);
+      const filePath = getOutputFilename(pipeline.getConfig(), path, pageData.route.type);
+      const lineIcon = i === paths.length - 1 ? "\u2514\u2500" : "\u251C\u2500";
+      logger.info(null, `  ${blue(lineIcon)} ${dim(filePath)}`, false);
       await generatePath(path, pipeline, generationOptions, route);
       const timeEnd = performance.now();
       const timeChange = getTimeStat(prevTimeEnd, timeEnd);
       const timeIncrease = `(+${timeChange})`;
-      const filePath = getOutputFilename(pipeline.getConfig(), path, pageData.route.type);
-      const lineIcon = i === paths.length - 1 ? "\u2514\u2500" : "\u251C\u2500";
-      logger.info(null, `  ${blue(lineIcon)} ${dim(filePath)} ${dim(timeIncrease)}`);
+      logger.info("SKIP_FORMAT", ` ${dim(timeIncrease)}`);
       prevTimeEnd = timeEnd;
     }
   }

package/dist/core/build/plugins/plugin-manifest.js

@@ -148,8 +148,9 @@ function buildManifest(opts, internals, staticFiles) {
       continue;
     const scripts = [];
     if (pageData.hoistedScript) {
+      const shouldPrefixAssetPath = pageData.hoistedScript.type === "external";
       const hoistedValue = pageData.hoistedScript.value;
-      const value = hoistedValue.endsWith(".js") ? prefixAssetPath(hoistedValue) : hoistedValue;
+      const value = shouldPrefixAssetPath ? prefixAssetPath(hoistedValue) : hoistedValue;
       scripts.unshift(
         Object.assign({}, pageData.hoistedScript, {
           value

package/dist/core/build/static-build.js

@@ -94,7 +94,7 @@ async function staticBuild(opts, internals, ssrOutputChunkNames) {
     case settings.config.output === "static": {
       settings.timer.start("Static generate");
       await generatePages(opts, internals);
-      await cleanServerOutput(opts, ssrOutputChunkNames);
+      await cleanServerOutput(opts, ssrOutputChunkNames, internals);
       settings.timer.end("Static generate");
       return;
     }
@@ -314,9 +314,12 @@ export const ${e.n} = noop;`;
     removeEmptyDirs(out);
   }
 }
-async function cleanServerOutput(opts, ssrOutputChunkNames) {
+async function cleanServerOutput(opts, ssrOutputChunkNames, internals) {
   const out = getOutDirWithinCwd(opts.settings.config.outDir);
   const files = ssrOutputChunkNames.filter((f) => f.endsWith(".mjs"));
+  if (internals.manifestFileName) {
+    files.push(internals.manifestFileName);
+  }
   if (files.length) {
     await Promise.all(
       files.map(async (filename) => {

package/dist/core/constants.js

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

package/dist/core/dev/dev.js

@@ -21,7 +21,7 @@ async function dev(inlineConfig) {
       base: restart.container.settings.config.base
     })
   );
-  const currentVersion = "4.0.6";
+  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.",
@@ -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.6";
+  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.6"}`
+        `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/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

@@ -71,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;
     }
@@ -317,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-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.6",
+  "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",
@@ -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",