astro 1.1.4 → 1.1.7

package/dist/core/build/vite-plugin-analyzer.js

@@ -1,5 +1,4 @@
 import { prependForwardSlash } from "../../core/path.js";
-import { resolveClientDevPath } from "../../core/render/dev/resolve.js";
 import { getTopLevelPages } from "./graph.js";
 import { getPageDataByViteID, trackClientOnlyPageDatas } from "./internal.js";
 function vitePluginAnalyzer(internals) {
@@ -66,14 +65,14 @@ function vitePluginAnalyzer(internals) {
           continue;
         const astro = info.meta.astro;
         for (const c of astro.hydratedComponents) {
-          const rid = c.resolvedPath ? resolveClientDevPath(c.resolvedPath) : c.specifier;
+          const rid = c.resolvedPath ? decodeURI(c.resolvedPath) : c.specifier;
           internals.discoveredHydratedComponents.add(rid);
         }
         hoistScanner.scan.call(this, astro.scripts, id);
         if (astro.clientOnlyComponents.length) {
           const clientOnlys = [];
           for (const c of astro.clientOnlyComponents) {
-            const cid = c.resolvedPath ? resolveClientDevPath(c.resolvedPath) : c.specifier;
+            const cid = c.resolvedPath ? decodeURI(c.resolvedPath) : c.specifier;
             internals.discoveredClientOnlyComponents.add(cid);
             clientOnlys.push(cid);
           }

package/dist/core/build/vite-plugin-css.js

@@ -92,6 +92,7 @@ function rollupPluginAstroBuildCSS(options) {
                   if (Object.keys(c.modules).every((id) => internals.cssChunkModuleIds.has(id))) {
                     for (const importedCssImport of meta.importedCss) {
                       delete bundle[importedCssImport];
+                      meta.importedCss.delete(importedCssImport);
                     }
                     return;
                   }

package/dist/core/create-vite.js

@@ -1,4 +1,6 @@
 import fs from "fs";
+import { createRequire } from "module";
+import path from "path";
 import { fileURLToPath } from "url";
 import * as vite from "vite";
 import astroPostprocessVitePlugin from "../vite-plugin-astro-postprocess/index.js";
@@ -116,32 +118,82 @@ function sortPlugins(pluginOptions) {
   pluginOptions.splice(jsxPluginIndex, 0, mdxPlugin);
 }
 async function getAstroPackages({ root }) {
-  const pkgUrl = new URL("./package.json", root);
-  const pkgPath = fileURLToPath(pkgUrl);
-  if (!fs.existsSync(pkgPath))
-    return [];
-  const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
-  const deps = [...Object.keys(pkg.dependencies || {}), ...Object.keys(pkg.devDependencies || {})];
-  return deps.filter((dep) => {
-    if (isCommonNotAstro(dep))
-      return false;
-    if (/^astro\-/.test(dep))
-      return true;
-    const depPkgUrl = new URL(`./node_modules/${dep}/package.json`, root);
-    const depPkgPath = fileURLToPath(depPkgUrl);
-    if (!fs.existsSync(depPkgPath))
-      return false;
-    const {
-      dependencies = {},
-      peerDependencies = {},
-      keywords = []
-    } = JSON.parse(fs.readFileSync(depPkgPath, "utf-8"));
-    if (peerDependencies.astro || dependencies.astro)
-      return true;
-    if (keywords.includes("astro") || keywords.includes("astro-component"))
-      return true;
-    return false;
-  });
+  const { astroPackages } = new DependencyWalker(root);
+  return astroPackages;
+}
+class DependencyWalker {
+  constructor(root) {
+    this.astroDeps = /* @__PURE__ */ new Set();
+    this.nonAstroDeps = /* @__PURE__ */ new Set();
+    const pkgUrl = new URL("./package.json", root);
+    this.require = createRequire(pkgUrl);
+    const pkgPath = fileURLToPath(pkgUrl);
+    if (!fs.existsSync(pkgPath))
+      return;
+    const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
+    const deps = [
+      ...Object.keys(pkg.dependencies || {}),
+      ...Object.keys(pkg.devDependencies || {})
+    ];
+    this.scanDependencies(deps);
+  }
+  get astroPackages() {
+    return Array.from(this.astroDeps);
+  }
+  seen(dep) {
+    return this.astroDeps.has(dep) || this.nonAstroDeps.has(dep);
+  }
+  readPkgJSON(dir) {
+    try {
+      const filePath = path.join(dir, "package.json");
+      return JSON.parse(fs.readFileSync(filePath, "utf-8"));
+    } catch (e) {
+    }
+  }
+  resolvePkgJSON(dep) {
+    try {
+      const pkgJson = this.require(dep + "/package.json");
+      return pkgJson;
+    } catch (e) {
+      try {
+        let dir = path.dirname(this.require.resolve(dep));
+        while (dir) {
+          const pkgJSON = this.readPkgJSON(dir);
+          if (pkgJSON && pkgJSON.name === dep)
+            return pkgJSON;
+          const parentDir = path.dirname(dir);
+          if (parentDir === dir)
+            break;
+          dir = parentDir;
+        }
+      } catch {
+      }
+    }
+  }
+  scanDependencies(deps) {
+    const newDeps = [];
+    for (const dep of deps) {
+      if (isCommonNotAstro(dep)) {
+        this.nonAstroDeps.add(dep);
+        continue;
+      }
+      const pkgJson = this.resolvePkgJSON(dep);
+      if (!pkgJson) {
+        this.nonAstroDeps.add(dep);
+        continue;
+      }
+      const { dependencies = {}, peerDependencies = {}, keywords = [] } = pkgJson;
+      if (peerDependencies.astro || dependencies.astro || keywords.includes("astro") || keywords.includes("astro-component") || /^(@[^\/]+\/)?astro\-/.test(dep)) {
+        this.astroDeps.add(dep);
+        const unknownDependencies = Object.keys(dependencies).filter((d) => !this.seen(d));
+        newDeps.push(...unknownDependencies);
+      } else {
+        this.nonAstroDeps.add(dep);
+      }
+    }
+    if (newDeps.length)
+      this.scanDependencies(newDeps);
+  }
 }
 const COMMON_DEPENDENCIES_NOT_ASTRO = [
   "autoprefixer",

package/dist/core/dev/index.js

@@ -46,7 +46,7 @@ async function dev(config, options) {
       https: !!((_a = viteConfig.server) == null ? void 0 : _a.https)
     })
   );
-  const currentVersion = "1.1.4";
+  const currentVersion = "1.1.7";
   if (currentVersion.includes("-")) {
     warn(options.logging, null, msg.prerelease({ currentVersion }));
   }

package/dist/core/messages.js

@@ -46,7 +46,7 @@ function devStart({
   https,
   site
 }) {
-  const version = "1.1.4";
+  const version = "1.1.7";
   const rootPath = site ? site.pathname : "/";
   const localPrefix = `${dim("\u2503")} Local    `;
   const networkPrefix = `${dim("\u2503")} Network  `;
@@ -225,7 +225,7 @@ function printHelp({
     message.push(
       linebreak(),
       `  ${bgGreen(black(` ${commandName} `))} ${green(
-        `v${"1.1.4"}`
+        `v${"1.1.7"}`
       )} ${headline}`
     );
   }

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

@@ -3,14 +3,14 @@ import { viteID } from "../../util.js";
 import { STYLE_EXTENSIONS } from "../util.js";
 import { crawlGraph } from "./vite.js";
 async function getStylesForURL(filePath, viteServer, mode) {
-  var _a;
   const importedCssUrls = /* @__PURE__ */ new Set();
   const importedStylesMap = /* @__PURE__ */ new Map();
   for await (const importedModule of crawlGraph(viteServer, viteID(filePath), true)) {
     const ext = path.extname(importedModule.url).toLowerCase();
     if (STYLE_EXTENSIONS.has(ext)) {
-      if (mode === "development" && typeof ((_a = importedModule.ssrModule) == null ? void 0 : _a.default) === "string") {
-        importedStylesMap.set(importedModule.url, importedModule.ssrModule.default);
+      const ssrModule = importedModule.ssrModule ?? await viteServer.ssrLoadModule(importedModule.url);
+      if (mode === "development" && typeof (ssrModule == null ? void 0 : ssrModule.default) === "string") {
+        importedStylesMap.set(importedModule.url, ssrModule.default);
       } else {
         importedCssUrls.add(importedModule.url);
       }

package/dist/core/util.js

@@ -5,7 +5,7 @@ import resolve from "resolve";
 import slash from "slash";
 import { fileURLToPath, pathToFileURL } from "url";
 import { prependForwardSlash, removeTrailingForwardSlash } from "./path.js";
-const ASTRO_VERSION = "1.1.4";
+const ASTRO_VERSION = "1.1.7";
 function isObject(value) {
   return typeof value === "object" && value != null;
 }

package/dist/jsx/babel.js

@@ -1,5 +1,6 @@
 import * as t from "@babel/types";
 import { pathToFileURL } from "node:url";
+import { resolveClientDevPath } from "../core/render/dev/resolve.js";
 import { HydrationDirectiveProps } from "../runtime/server/hydration.js";
 const ClientOnlyPlaceholder = "astro-client-only";
 function isComponent(tagName) {
@@ -184,7 +185,8 @@ function astroJSX() {
               break;
             }
             if (namespace.at(0) === local) {
-              path.setData("import", { name: imported, path: source });
+              const name = imported === "*" ? imported : tagName;
+              path.setData("import", { name, path: source });
               break;
             }
           }
@@ -194,10 +196,7 @@ function astroJSX() {
           let resolvedPath;
           if (meta.path.startsWith(".")) {
             const fileURL = pathToFileURL(state.filename);
-            resolvedPath = `/@fs${new URL(meta.path, fileURL).pathname}`;
-            if (resolvedPath.endsWith(".jsx")) {
-              resolvedPath = resolvedPath.slice(0, -4);
-            }
+            resolvedPath = resolveClientDevPath(`/@fs${new URL(meta.path, fileURL).pathname}`);
           } else {
             resolvedPath = meta.path;
           }
@@ -273,10 +272,7 @@ function astroJSX() {
           let resolvedPath;
           if (meta.path.startsWith(".")) {
             const fileURL = pathToFileURL(state.filename);
-            resolvedPath = `/@fs${new URL(meta.path, fileURL).pathname}`;
-            if (resolvedPath.endsWith(".jsx")) {
-              resolvedPath = resolvedPath.slice(0, -4);
-            }
+            resolvedPath = resolveClientDevPath(`/@fs${new URL(meta.path, fileURL).pathname}`);
           } else {
             resolvedPath = meta.path;
           }

package/dist/runtime/server/astro-global.js

@@ -1,4 +1,4 @@
-const ASTRO_VERSION = "1.1.4";
+const ASTRO_VERSION = "1.1.7";
 function createDeprecatedFetchContentFn() {
   return () => {
     throw new Error("Deprecated: Astro.fetchContent() has been replaced with Astro.glob().");

package/dist/runtime/server/hydration.js

@@ -85,11 +85,11 @@ async function generateHydrateScript(scriptOptions, metadata) {
       island.props[key] = value;
     }
   }
-  island.props["component-url"] = await result.resolve(componentUrl);
+  island.props["component-url"] = await result.resolve(decodeURI(componentUrl));
   if (renderer.clientEntrypoint) {
     island.props["component-export"] = componentExport.value;
-    island.props["renderer-url"] = await result.resolve(renderer.clientEntrypoint);
-    island.props["props"] = escapeHTML(serializeProps(props));
+    island.props["renderer-url"] = await result.resolve(decodeURI(renderer.clientEntrypoint));
+    island.props["props"] = escapeHTML(serializeProps(props, metadata));
   }
   island.props["ssr"] = "";
   island.props["client"] = hydrate;

package/dist/runtime/server/render/component.js

@@ -231,7 +231,8 @@ If you're still stuck, please open an issue on GitHub or join us at https://astr
     `<!--${metadata.componentExport.value}:${metadata.componentUrl}-->
 ${html}
 ${serializeProps(
-      props
+      props,
+      metadata
     )}`
   );
   const island = await generateHydrateScript(

package/dist/runtime/server/serialize.d.ts

@@ -1 +1,2 @@
-export declare function serializeProps(props: any): string;
+import type { AstroComponentMetadata } from '../../@types/astro';
+export declare function serializeProps(props: any, metadata: AstroComponentMetadata): string;

package/dist/runtime/server/serialize.js

@@ -8,17 +8,23 @@ const PROP_TYPE = {
   BigInt: 6,
   URL: 7
 };
-function serializeArray(value) {
-  return value.map((v) => convertToSerializedForm(v));
+function serializeArray(value, metadata) {
+  return value.map((v) => convertToSerializedForm(v, metadata));
 }
-function serializeObject(value) {
+function serializeObject(value, metadata) {
+  if (cyclicRefs.has(value)) {
+    throw new Error(`Cyclic reference detected while serializing props for <${metadata.displayName} client:${metadata.hydrate}>!
+
+Cyclic references cannot be safely serialized for client-side usage. Please remove the cyclic reference.`);
+  }
+  cyclicRefs.add(value);
   return Object.fromEntries(
     Object.entries(value).map(([k, v]) => {
-      return [k, convertToSerializedForm(v)];
+      return [k, convertToSerializedForm(v, metadata)];
     })
   );
 }
-function convertToSerializedForm(value) {
+function convertToSerializedForm(value, metadata) {
   const tag = Object.prototype.toString.call(value);
   switch (tag) {
     case "[object Date]": {
@@ -28,10 +34,16 @@ function convertToSerializedForm(value) {
       return [PROP_TYPE.RegExp, value.source];
     }
     case "[object Map]": {
-      return [PROP_TYPE.Map, JSON.stringify(serializeArray(Array.from(value)))];
+      return [
+        PROP_TYPE.Map,
+        JSON.stringify(serializeArray(Array.from(value), metadata))
+      ];
     }
     case "[object Set]": {
-      return [PROP_TYPE.Set, JSON.stringify(serializeArray(Array.from(value)))];
+      return [
+        PROP_TYPE.Set,
+        JSON.stringify(serializeArray(Array.from(value), metadata))
+      ];
     }
     case "[object BigInt]": {
       return [PROP_TYPE.BigInt, value.toString()];
@@ -40,19 +52,22 @@ function convertToSerializedForm(value) {
       return [PROP_TYPE.URL, value.toString()];
     }
     case "[object Array]": {
-      return [PROP_TYPE.JSON, JSON.stringify(serializeArray(value))];
+      return [PROP_TYPE.JSON, JSON.stringify(serializeArray(value, metadata))];
     }
     default: {
       if (value !== null && typeof value === "object") {
-        return [PROP_TYPE.Value, serializeObject(value)];
+        return [PROP_TYPE.Value, serializeObject(value, metadata)];
       } else {
         return [PROP_TYPE.Value, value];
       }
     }
   }
 }
-function serializeProps(props) {
-  return JSON.stringify(serializeObject(props));
+let cyclicRefs = /* @__PURE__ */ new WeakSet();
+function serializeProps(props, metadata) {
+  const serialized = JSON.stringify(serializeObject(props, metadata));
+  cyclicRefs = /* @__PURE__ */ new WeakSet();
+  return serialized;
 }
 export {
   serializeProps

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

@@ -83,6 +83,7 @@ async function handle404Response(origin, config, req, res) {
 async function handle500Response(viteServer, origin, req, res, err) {
   res.on("close", () => setTimeout(() => viteServer.ws.send(getViteErrorPayload(err)), 200));
   if (res.headersSent) {
+    res.write(`<script type="module" src="/@vite/client"><\/script>`);
     res.end();
   } else {
     writeHtmlResponse(

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

@@ -112,7 +112,7 @@ function jsx({ config, logging }) {
     enforce: "pre",
     async configResolved(resolvedConfig) {
       viteConfig = resolvedConfig;
-      const possibleRenderers = await collectJSXRenderers(config._ctx.renderers);
+      const possibleRenderers = collectJSXRenderers(config._ctx.renderers);
       for (const [importSource, renderer] of possibleRenderers) {
         jsxRenderers.set(importSource, renderer);
         if (importSource === "astro") {
@@ -163,8 +163,8 @@ function jsx({ config, logging }) {
       if (!importSource && IMPORT_KEYWORD_REGEX.test(code)) {
         importSource = await detectImportSourceFromImports(code, id, jsxRenderers);
       }
-      if (!importSource) {
-        const [defaultRendererName] = defaultJSXRendererEntry[0];
+      if (!importSource && defaultJSXRendererEntry) {
+        const [defaultRendererName] = defaultJSXRendererEntry;
         error(
           logging,
           "renderer",
@@ -173,6 +173,16 @@ Unable to resolve a renderer that handles this file! With more than one renderer
 Add ${colors.cyan(
             IMPORT_STATEMENTS[defaultRendererName] || `import '${defaultRendererName}';`
           )} or ${colors.cyan(`/* jsxImportSource: ${defaultRendererName} */`)} to this file.
+`
+        );
+        return null;
+      } else if (!importSource) {
+        error(
+          logging,
+          "renderer",
+          `${colors.yellow(id)}
+Unable to find a renderer for JSX. Do you have one configured in your Astro config? See this page to learn how:
+https://docs.astro.build/en/core-concepts/framework-components/#installing-integrations
 `
         );
         return null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "astro",
-  "version": "1.1.4",
+  "version": "1.1.7",
   "description": "Astro is a modern site builder with web best practices, performance, and DX front-of-mind.",
   "type": "module",
   "author": "withastro",
@@ -82,7 +82,7 @@
     "vendor"
   ],
   "dependencies": {
-    "@astrojs/compiler": "^0.23.4",
+    "@astrojs/compiler": "^0.23.5",
     "@astrojs/language-server": "^0.23.0",
     "@astrojs/markdown-remark": "^1.1.1",
     "@astrojs/telemetry": "^1.0.0",