astro 2.5.4 → 2.5.5

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

@@ -1170,12 +1170,6 @@ export interface ContentEntryType {
         entry: ContentEntryModule;
     }): rollup.LoadResult | Promise<rollup.LoadResult>;
     contentModuleTypes?: string;
-    /**
-     * Handle asset propagation for rendered content to avoid bleed.
-     * Ex. MDX content can import styles and scripts, so `handlePropagation` should be true.
-     * @default true
-     */
-    handlePropagation?: boolean;
 }
 type GetContentEntryInfoReturnType = {
     data: Record<string, unknown>;

package/dist/content/consts.d.ts

@@ -1,10 +1,9 @@
 export declare const PROPAGATED_ASSET_FLAG = "astroPropagatedAssets";
-export declare const CONTENT_RENDER_FLAG = "astroRenderContent";
 export declare const CONTENT_FLAG = "astroContentCollectionEntry";
 export declare const DATA_FLAG = "astroDataCollectionEntry";
+export declare const CONTENT_FLAGS: readonly ["astroContentCollectionEntry", "astroDataCollectionEntry", "astroPropagatedAssets"];
 export declare const VIRTUAL_MODULE_ID = "astro:content";
 export declare const LINKS_PLACEHOLDER = "@@ASTRO-LINKS@@";
 export declare const STYLES_PLACEHOLDER = "@@ASTRO-STYLES@@";
 export declare const SCRIPTS_PLACEHOLDER = "@@ASTRO-SCRIPTS@@";
-export declare const CONTENT_FLAGS: readonly ["astroContentCollectionEntry", "astroRenderContent", "astroDataCollectionEntry", "astroPropagatedAssets"];
 export declare const CONTENT_TYPES_FILE = "types.d.ts";

package/dist/content/consts.js

@@ -1,22 +1,15 @@
 const PROPAGATED_ASSET_FLAG = "astroPropagatedAssets";
-const CONTENT_RENDER_FLAG = "astroRenderContent";
 const CONTENT_FLAG = "astroContentCollectionEntry";
 const DATA_FLAG = "astroDataCollectionEntry";
+const CONTENT_FLAGS = [CONTENT_FLAG, DATA_FLAG, PROPAGATED_ASSET_FLAG];
 const VIRTUAL_MODULE_ID = "astro:content";
 const LINKS_PLACEHOLDER = "@@ASTRO-LINKS@@";
 const STYLES_PLACEHOLDER = "@@ASTRO-STYLES@@";
 const SCRIPTS_PLACEHOLDER = "@@ASTRO-SCRIPTS@@";
-const CONTENT_FLAGS = [
-  CONTENT_FLAG,
-  CONTENT_RENDER_FLAG,
-  DATA_FLAG,
-  PROPAGATED_ASSET_FLAG
-];
 const CONTENT_TYPES_FILE = "types.d.ts";
 export {
   CONTENT_FLAG,
   CONTENT_FLAGS,
-  CONTENT_RENDER_FLAG,
   CONTENT_TYPES_FILE,
   DATA_FLAG,
   LINKS_PLACEHOLDER,

package/dist/content/runtime.js

@@ -184,7 +184,7 @@ async function render({
   id,
   renderEntryImport
 }) {
-  var _a, _b;
+  var _a;
   const UnexpectedRenderError = new AstroError({
     ...AstroErrorData.UnknownContentCollectionError,
     message: `Unexpected error while rendering ${String(collection)} \u2192 ${String(id)}.`
@@ -194,69 +194,53 @@ async function render({
   const baseMod = await renderEntryImport();
   if (baseMod == null || typeof baseMod !== "object")
     throw UnexpectedRenderError;
-  if (baseMod.default != null && typeof baseMod.default === "object" && baseMod.default.__astroPropagation === true) {
-    const { collectedStyles, collectedLinks, collectedScripts, getMod } = baseMod.default;
-    if (typeof getMod !== "function")
-      throw UnexpectedRenderError;
-    const propagationMod = await getMod();
-    if (propagationMod == null || typeof propagationMod !== "object")
-      throw UnexpectedRenderError;
-    const Content = createComponent({
-      factory(result, baseProps, slots) {
-        let styles = "", links = "", scripts = "";
-        if (Array.isArray(collectedStyles)) {
-          styles = collectedStyles.map((style) => {
-            return renderUniqueStylesheet(result, {
-              type: "inline",
-              content: style
-            });
-          }).join("");
-        }
-        if (Array.isArray(collectedLinks)) {
-          links = collectedLinks.map((link) => {
-            return renderUniqueStylesheet(result, {
-              type: "external",
-              src: prependForwardSlash(link)
-            });
-          }).join("");
-        }
-        if (Array.isArray(collectedScripts)) {
-          scripts = collectedScripts.map((script) => renderScriptElement(script)).join("");
-        }
-        let props = baseProps;
-        if (id.endsWith("mdx")) {
-          props = {
-            components: propagationMod.components ?? {},
-            ...baseProps
-          };
-        }
-        return createHeadAndContent(
-          unescapeHTML(styles + links + scripts),
-          renderTemplate`${renderComponent(
-            result,
-            "Content",
-            propagationMod.Content,
-            props,
-            slots
-          )}`
-        );
-      },
-      propagation: "self"
-    });
-    return {
-      Content,
-      headings: ((_a = propagationMod.getHeadings) == null ? void 0 : _a.call(propagationMod)) ?? [],
-      remarkPluginFrontmatter: propagationMod.frontmatter ?? {}
-    };
-  } else if (baseMod.Content && typeof baseMod.Content === "function") {
-    return {
-      Content: baseMod.Content,
-      headings: ((_b = baseMod.getHeadings) == null ? void 0 : _b.call(baseMod)) ?? [],
-      remarkPluginFrontmatter: baseMod.frontmatter ?? {}
-    };
-  } else {
+  const { collectedStyles, collectedLinks, collectedScripts, getMod } = baseMod;
+  if (typeof getMod !== "function")
     throw UnexpectedRenderError;
-  }
+  const mod = await getMod();
+  if (mod == null || typeof mod !== "object")
+    throw UnexpectedRenderError;
+  const Content = createComponent({
+    factory(result, baseProps, slots) {
+      let styles = "", links = "", scripts = "";
+      if (Array.isArray(collectedStyles)) {
+        styles = collectedStyles.map((style) => {
+          return renderUniqueStylesheet(result, {
+            type: "inline",
+            content: style
+          });
+        }).join("");
+      }
+      if (Array.isArray(collectedLinks)) {
+        links = collectedLinks.map((link) => {
+          return renderUniqueStylesheet(result, {
+            type: "external",
+            src: prependForwardSlash(link)
+          });
+        }).join("");
+      }
+      if (Array.isArray(collectedScripts)) {
+        scripts = collectedScripts.map((script) => renderScriptElement(script)).join("");
+      }
+      let props = baseProps;
+      if (id.endsWith("mdx")) {
+        props = {
+          components: mod.components ?? {},
+          ...baseProps
+        };
+      }
+      return createHeadAndContent(
+        unescapeHTML(styles + links + scripts),
+        renderTemplate`${renderComponent(result, "Content", mod.Content, props, slots)}`
+      );
+    },
+    propagation: "self"
+  });
+  return {
+    Content,
+    headings: ((_a = mod.getHeadings) == null ? void 0 : _a.call(mod)) ?? [],
+    remarkPluginFrontmatter: mod.frontmatter ?? {}
+  };
 }
 function createReference({ lookupMap }) {
   return function reference(collection) {

package/dist/content/vite-plugin-content-assets.js

@@ -1,5 +1,4 @@
-import { extname } from "node:path";
-import { pathToFileURL } from "node:url";
+import { pathToFileURL } from "url";
 import { moduleIsTopLevelPage, walkParentInfos } from "../core/build/graph.js";
 import { getPageDataByViteID } from "../core/build/internal.js";
 import { createViteLoader } from "../core/module-loader/vite.js";
@@ -7,13 +6,15 @@ import { joinPaths, prependForwardSlash } from "../core/path.js";
 import { getStylesForURL } from "../core/render/dev/css.js";
 import { getScriptsForURL } from "../core/render/dev/scripts.js";
 import {
-  CONTENT_RENDER_FLAG,
   LINKS_PLACEHOLDER,
   PROPAGATED_ASSET_FLAG,
   SCRIPTS_PLACEHOLDER,
   STYLES_PLACEHOLDER
 } from "./consts.js";
-import { hasContentFlag } from "./utils.js";
+function isPropagatedAsset(viteId) {
+  const flags = new URLSearchParams(viteId.split("?")[1]);
+  return flags.has(PROPAGATED_ASSET_FLAG);
+}
 function astroContentAssetPropagationPlugin({
   mode,
   settings
@@ -21,21 +22,6 @@ function astroContentAssetPropagationPlugin({
   let devModuleLoader;
   return {
     name: "astro:content-asset-propagation",
-    enforce: "pre",
-    async resolveId(id, importer, opts) {
-      if (hasContentFlag(id, CONTENT_RENDER_FLAG)) {
-        const base = id.split("?")[0];
-        for (const { extensions, handlePropagation = true } of settings.contentEntryTypes) {
-          if (handlePropagation && extensions.includes(extname(base))) {
-            return this.resolve(`${base}?${PROPAGATED_ASSET_FLAG}`, importer, {
-              skipSelf: true,
-              ...opts
-            });
-          }
-        }
-        return this.resolve(base, importer, { skipSelf: true, ...opts });
-      }
-    },
     configureServer(server) {
       if (mode === "dev") {
         devModuleLoader = createViteLoader(server);
@@ -43,7 +29,7 @@ function astroContentAssetPropagationPlugin({
     },
     async transform(_, id, options) {
       var _a;
-      if (hasContentFlag(id, PROPAGATED_ASSET_FLAG)) {
+      if (isPropagatedAsset(id)) {
         const basePath = id.split("?")[0];
         let stringifiedLinks, stringifiedStyles, stringifiedScripts;
         if ((options == null ? void 0 : options.ssr) && devModuleLoader) {
@@ -69,14 +55,12 @@ function astroContentAssetPropagationPlugin({
           stringifiedScripts = JSON.stringify(SCRIPTS_PLACEHOLDER);
         }
         const code = `
-					async function getMod() {
+					export async function getMod() {
 						return import(${JSON.stringify(basePath)});
 					}
-					const collectedLinks = ${stringifiedLinks};
-					const collectedStyles = ${stringifiedStyles};
-					const collectedScripts = ${stringifiedScripts};
-					const defaultMod = { __astroPropagation: true, getMod, collectedLinks, collectedStyles, collectedScripts };
-					export default defaultMod;
+					export const collectedLinks = ${stringifiedLinks};
+					export const collectedStyles = ${stringifiedStyles};
+					export const collectedScripts = ${stringifiedScripts};
 				`;
         return { code, map: { mappings: "" } };
       }

package/dist/core/constants.js

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

package/dist/core/dev/dev.js

@@ -53,7 +53,7 @@ async function dev(settings, options) {
       isRestart: options.isRestart
     })
   );
-  const currentVersion = "2.5.4";
+  const currentVersion = "2.5.5";
   if (currentVersion.includes("-")) {
     warn(options.logging, null, msg.prerelease({ currentVersion }));
   }

package/dist/core/messages.js

@@ -47,7 +47,7 @@ function serverStart({
   base,
   isRestart = false
 }) {
-  const version = "2.5.4";
+  const version = "2.5.5";
   const localPrefix = `${dim("\u2503")} Local    `;
   const networkPrefix = `${dim("\u2503")} Network  `;
   const emptyPrefix = " ".repeat(11);
@@ -233,7 +233,7 @@ function printHelp({
     message.push(
       linebreak(),
       `  ${bgGreen(black(` ${commandName} `))} ${green(
-        `v${"2.5.4"}`
+        `v${"2.5.5"}`
       )} ${headline}`
     );
   }

package/dist/core/render/dev/vite.js

@@ -1,13 +1,15 @@
 import npath from "path";
+import { PROPAGATED_ASSET_FLAG } from "../../../content/consts.js";
 import { SUPPORTED_MARKDOWN_FILE_EXTENSIONS } from "../../constants.js";
 import { unwrapId } from "../../util.js";
 import { isCSSRequest } from "./util.js";
-const fileExtensionsToSSR = /* @__PURE__ */ new Set([".astro", ".mdoc", ...SUPPORTED_MARKDOWN_FILE_EXTENSIONS]);
+const fileExtensionsToSSR = /* @__PURE__ */ new Set([".astro", ...SUPPORTED_MARKDOWN_FILE_EXTENSIONS]);
 const STRIP_QUERY_PARAMS_REGEX = /\?.*$/;
-const ASTRO_PROPAGATED_ASSET_REGEX = /\?astroPropagatedAssets/;
 async function* crawlGraph(loader, _id, isRootFile, scanned = /* @__PURE__ */ new Set()) {
   const id = unwrapId(_id);
   const importedModules = /* @__PURE__ */ new Set();
+  if (new URL(id, "file://").searchParams.has(PROPAGATED_ASSET_FLAG))
+    return;
   const moduleEntriesForId = isRootFile ? (
     // "getModulesByFile" pulls from a delayed module cache (fun implementation detail),
     // So we can get up-to-date info on initial server load.
@@ -31,11 +33,14 @@ async function* crawlGraph(loader, _id, isRootFile, scanned = /* @__PURE__ */ ne
           if (entryIsStyle && !isCSSRequest(importedModulePathname)) {
             continue;
           }
-          const isFileTypeNeedingSSR = fileExtensionsToSSR.has(
-            npath.extname(importedModulePathname)
-          );
-          if (isFileTypeNeedingSSR && // Should not SSR a module with ?astroPropagatedAssets
-          !ASTRO_PROPAGATED_ASSET_REGEX.test(importedModule.id)) {
+          if (fileExtensionsToSSR.has(
+            npath.extname(
+              // Use `id` instead of `pathname` to preserve query params.
+              // Should not SSR a module with an unexpected query param,
+              // like "?astroPropagatedAssets"
+              importedModule.id
+            )
+          )) {
             const mod = loader.getModuleById(importedModule.id);
             if (!(mod == null ? void 0 : mod.ssrModule)) {
               try {

package/dist/core/render/result.js

@@ -1,3 +1,4 @@
+import { isHTMLString } from "../../runtime/server/escape.js";
 import {
   renderSlotToString,
   stringifyChunk
@@ -69,10 +70,10 @@ class Slots {
       const component = typeof slotValue === "function" ? await slotValue(result) : await slotValue;
       const expression = getFunctionExpression(component);
       if (expression) {
-        const slot = () => expression(...args);
-        return await renderSlotToString(result, slot).then(
-          (res) => res != null ? String(res) : res
-        );
+        const slot = async () => isHTMLString(await expression) ? expression : expression(...args);
+        return await renderSlotToString(result, slot).then((res) => {
+          return res != null ? String(res) : res;
+        });
       }
       if (typeof component === "function") {
         return await renderJSX(result, component(...args)).then(

package/dist/runtime/server/render/slot.d.ts

@@ -4,7 +4,7 @@ import type { RenderInstruction } from './types.js';
 import { HTMLString } from '../escape.js';
 type RenderTemplateResult = ReturnType<typeof renderTemplate>;
 export type ComponentSlots = Record<string, ComponentSlotValue>;
-export type ComponentSlotValue = (result: SSRResult) => RenderTemplateResult;
+export type ComponentSlotValue = (result: SSRResult) => RenderTemplateResult | Promise<RenderTemplateResult>;
 declare const slotString: unique symbol;
 export declare class SlotString extends HTMLString {
     instructions: null | RenderInstruction[];

package/dist/vite-plugin-head/index.js

@@ -1,6 +1,6 @@
 import { getTopLevelPages, walkParentInfos } from "../core/build/graph.js";
 import { getAstroMetadata } from "../vite-plugin-astro/index.js";
-const injectExp = /(^\/\/|\/\/!)\s*astro-head-inject/;
+const injectExp = /^\/\/\s*astro-head-inject/;
 function configHeadVitePlugin({
   settings
 }) {

package/dist/vite-plugin-markdown/content-entry-type.js

@@ -10,9 +10,7 @@ const markdownContentEntryType = {
       slug: parsed.data.slug,
       rawData: parsed.matter
     };
-  },
-  // We need to handle propagation for Markdown because they support layouts which will bring in styles.
-  handlePropagation: true
+  }
 };
 const mdxContentEntryType = {
   extensions: [".mdx"],
@@ -25,9 +23,6 @@ const mdxContentEntryType = {
       rawData: parsed.matter
     };
   },
-  // MDX can import scripts and styles,
-  // so wrap all MDX files with script / style propagation checks
-  handlePropagation: true,
   contentModuleTypes: `declare module 'astro:content' {
 	interface Render {
 		'.mdx': Promise<{

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "astro",
-  "version": "2.5.4",
+  "version": "2.5.5",
   "description": "Astro is a modern site builder with web best practices, performance, and DX front-of-mind.",
   "type": "module",
   "author": "withastro",

package/src/content/template/virtual-mod.mjs

@@ -46,7 +46,7 @@ function createGlobLookup(glob) {
 }
 
 const renderEntryGlob = import.meta.glob('@@RENDER_ENTRY_GLOB_PATH@@', {
-	query: { astroRenderContent: true },
+	query: { astroPropagatedAssets: true },
 });
 const collectionToRenderEntryMap = createCollectionToGlobResultMap({
 	globResult: renderEntryGlob,