astro 4.13.0 → 4.13.2

package/dist/vite-plugin-scripts/page-ssr.js

@@ -16,7 +16,7 @@ function astroScriptsPostPlugin({
       let fileURL;
       try {
         fileURL = new URL(`file://${filename}`);
-      } catch (e) {
+      } catch {
         return;
       }
       const fileIsPage = isPage(fileURL, settings);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "astro",
-  "version": "4.13.0",
+  "version": "4.13.2",
   "description": "Astro is a modern site builder with web best practices, performance, and DX front-of-mind.",
   "type": "module",
   "author": "withastro",
@@ -83,9 +83,6 @@
     },
     "./virtual-modules/*": "./dist/virtual-modules/*"
   },
-  "imports": {
-    "#astro/*": "./dist/*.js"
-  },
   "bin": {
     "astro": "astro.js"
   },
@@ -110,7 +107,7 @@
     "vendor"
   ],
   "dependencies": {
-    "@astrojs/compiler": "^2.10.0",
+    "@astrojs/compiler": "^2.10.1",
     "@babel/core": "^7.25.2",
     "@babel/generator": "^7.25.0",
     "@babel/parser": "^7.25.3",
@@ -123,7 +120,6 @@
     "aria-query": "^5.3.0",
     "axobject-query": "^4.1.0",
     "boxen": "7.1.1",
-    "chokidar": "^3.6.0",
     "ci-info": "^4.0.0",
     "clsx": "^2.1.1",
     "common-ancestor-path": "^1.0.1",
@@ -157,7 +153,7 @@
     "prompts": "^2.4.2",
     "rehype": "^13.0.1",
     "semver": "^7.6.3",
-    "shiki": "^1.12.0",
+    "shiki": "^1.12.1",
     "string-width": "^7.2.0",
     "strip-ansi": "^7.1.0",
     "tsconfck": "^3.1.1",
@@ -170,20 +166,19 @@
     "zod": "^3.23.8",
     "zod-to-json-schema": "^3.23.2",
     "@astrojs/internal-helpers": "0.4.1",
-    "@astrojs/markdown-remark": "5.2.0",
-    "@astrojs/telemetry": "3.1.0"
+    "@astrojs/telemetry": "3.1.0",
+    "@astrojs/markdown-remark": "5.2.0"
   },
   "optionalDependencies": {
     "sharp": "^0.33.3"
   },
   "devDependencies": {
-    "@astrojs/check": "^0.9.0",
+    "@astrojs/check": "^0.9.1",
     "@playwright/test": "^1.45.3",
     "@types/aria-query": "^5.0.4",
     "@types/babel__generator": "^7.6.8",
     "@types/babel__traverse": "^7.20.6",
     "@types/common-ancestor-path": "^1.0.2",
-    "@types/connect": "^3.4.38",
     "@types/cssesc": "^3.0.2",
     "@types/debug": "^4.1.12",
     "@types/diff": "^5.2.1",
@@ -193,27 +188,23 @@
     "@types/html-escaper": "^3.0.2",
     "@types/http-cache-semantics": "^4.0.4",
     "@types/js-yaml": "^4.0.9",
-    "@types/probe-image-size": "^7.2.5",
     "@types/prompts": "^2.4.9",
     "@types/semver": "^7.5.8",
-    "@types/send": "^0.17.4",
-    "@types/unist": "^3.0.2",
     "@types/yargs-parser": "^21.0.3",
     "cheerio": "1.0.0-rc.12",
     "eol": "^0.9.1",
     "expect-type": "^0.19.0",
     "mdast-util-mdx": "^3.0.0",
     "mdast-util-mdx-jsx": "^3.1.2",
-    "memfs": "^4.11.0",
+    "memfs": "^4.11.1",
     "node-mocks-http": "^1.15.1",
     "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.19.1",
+    "rollup": "^4.20.0",
     "sass": "^1.77.8",
-    "srcset-parse": "^1.1.0",
     "undici": "^6.19.5",
     "unified": "^11.0.5",
     "astro-scripts": "0.0.14"

package/templates/actions.mjs

@@ -1,4 +1,4 @@
-import { ActionError, callSafely, getActionQueryString } from 'astro:actions';
+import { ActionError, deserializeActionResult, getActionQueryString } from 'astro:actions';
 
 function toActionProxy(actionCallback = {}, aggregatedPath = '') {
 	return new Proxy(actionCallback, {
@@ -7,7 +7,9 @@ function toActionProxy(actionCallback = {}, aggregatedPath = '') {
 				return target[objKey];
 			}
 			const path = aggregatedPath + objKey.toString();
-			const action = (param) => callSafely(() => handleActionOrThrow(param, path));
+			function action(param) {
+				return handleAction(param, path, this);
+			}
 
 			Object.assign(action, {
 				queryString: getActionQueryString(path),
@@ -26,8 +28,10 @@ function toActionProxy(actionCallback = {}, aggregatedPath = '') {
 				// Note: `orThrow` does not have progressive enhancement info.
 				// If you want to throw exceptions,
 				//  you must handle those exceptions with client JS.
-				orThrow: (param) => {
-					return handleActionOrThrow(param, path);
+				async orThrow(param) {
+					const { data, error } = await handleAction(param, path, this);
+					if (error) throw error;
+					return data;
 				},
 			});
 
@@ -41,16 +45,18 @@ function toActionProxy(actionCallback = {}, aggregatedPath = '') {
 /**
  * @param {*} param argument passed to the action when called server or client-side.
  * @param {string} path Built path to call action by path name.
+ * @param {import('../dist/@types/astro.d.ts').APIContext | undefined} context Injected API context when calling actions from the server.
  * Usage: `actions.[name](param)`.
+ * @returns {Promise<import('../dist/actions/runtime/virtual/shared.js').SafeResult<any, any>>}
  */
-async function handleActionOrThrow(param, path) {
+async function handleAction(param, path, context) {
 	// When running server-side, import the action and call it.
 	if (import.meta.env.SSR) {
-		const { getAction } = await import('astro/actions/runtime/utils.js');
+		const { getAction } = await import('astro/actions/runtime/virtual/get-action.js');
 		const action = await getAction(path);
 		if (!action) throw new Error(`Action not found: ${path}`);
 
-		return action.orThrow(param);
+		return action.bind(context)(param);
 	}
 
 	// When running client-side, make a fetch request to the action path.
@@ -59,29 +65,30 @@ async function handleActionOrThrow(param, path) {
 	let body = param;
 	if (!(body instanceof FormData)) {
 		try {
-			body = param ? JSON.stringify(param) : undefined;
+			body = JSON.stringify(param);
 		} catch (e) {
 			throw new ActionError({
 				code: 'BAD_REQUEST',
 				message: `Failed to serialize request body to JSON. Full error: ${e.message}`,
 			});
 		}
-		headers.set('Content-Type', 'application/json');
-		headers.set('Content-Length', body?.length.toString() ?? '0');
+		if (body) {
+			headers.set('Content-Type', 'application/json');
+		} else {
+			headers.set('Content-Length', '0');
+		}
 	}
-	const res = await fetch(`/_actions/${path}`, {
+	const rawResult = await fetch(`/_actions/${path}`, {
 		method: 'POST',
 		body,
 		headers,
 	});
-	if (!res.ok) {
-		throw await ActionError.fromResponse(res);
-	}
-	// Check if response body is empty before parsing.
-	if (res.status === 204) return;
+	if (rawResult.status === 204) return;
 
-	const json = await res.json();
-	return json;
+	return deserializeActionResult({
+		type: rawResult.ok ? 'data' : 'error',
+		body: await rawResult.text(),
+	});
 }
 
 export const actions = toActionProxy();

package/templates/content/types.d.ts

@@ -22,29 +22,31 @@ declare module 'astro:content' {
 		ContentEntryMap[C]
 	>['slug'];
 
+	/** @deprecated Use `getEntry` instead. */
 	export function getEntryBySlug<
 		C extends keyof ContentEntryMap,
 		E extends ValidContentEntrySlug<C> | (string & {}),
 	>(
 		collection: C,
 		// Note that this has to accept a regular string too, for SSR
-		entrySlug: E
+		entrySlug: E,
 	): E extends ValidContentEntrySlug<C>
 		? Promise<CollectionEntry<C>>
 		: Promise<CollectionEntry<C> | undefined>;
 
+	/** @deprecated Use `getEntry` instead. */
 	export function getDataEntryById<C extends keyof DataEntryMap, E extends keyof DataEntryMap[C]>(
 		collection: C,
-		entryId: E
+		entryId: E,
 	): Promise<CollectionEntry<C>>;
 
 	export function getCollection<C extends keyof AnyEntryMap, E extends CollectionEntry<C>>(
 		collection: C,
-		filter?: (entry: CollectionEntry<C>) => entry is E
+		filter?: (entry: CollectionEntry<C>) => entry is E,
 	): Promise<E[]>;
 	export function getCollection<C extends keyof AnyEntryMap>(
 		collection: C,
-		filter?: (entry: CollectionEntry<C>) => unknown
+		filter?: (entry: CollectionEntry<C>) => unknown,
 	): Promise<CollectionEntry<C>[]>;
 
 	export function getEntry<
@@ -70,7 +72,7 @@ declare module 'astro:content' {
 		E extends ValidContentEntrySlug<C> | (string & {}),
 	>(
 		collection: C,
-		slug: E
+		slug: E,
 	): E extends ValidContentEntrySlug<C>
 		? Promise<CollectionEntry<C>>
 		: Promise<CollectionEntry<C> | undefined>;
@@ -79,7 +81,7 @@ declare module 'astro:content' {
 		E extends keyof DataEntryMap[C] | (string & {}),
 	>(
 		collection: C,
-		id: E
+		id: E,
 	): E extends keyof DataEntryMap[C]
 		? Promise<DataEntryMap[C][E]>
 		: Promise<CollectionEntry<C> | undefined>;
@@ -89,17 +91,17 @@ declare module 'astro:content' {
 		entries: {
 			collection: C;
 			slug: ValidContentEntrySlug<C>;
-		}[]
+		}[],
 	): Promise<CollectionEntry<C>[]>;
 	export function getEntries<C extends keyof DataEntryMap>(
 		entries: {
 			collection: C;
 			id: keyof DataEntryMap[C];
-		}[]
+		}[],
 	): Promise<CollectionEntry<C>[]>;
 
 	export function reference<C extends keyof AnyEntryMap>(
-		collection: C
+		collection: C,
 	): import('astro/zod').ZodEffects<
 		import('astro/zod').ZodString,
 		C extends keyof ContentEntryMap
@@ -116,7 +118,7 @@ declare module 'astro:content' {
 	// if `dev` is not running to update as you edit.
 	// Invalid collection names will be caught at build time.
 	export function reference<C extends string>(
-		collection: C
+		collection: C,
 	): import('astro/zod').ZodEffects<import('astro/zod').ZodString, never>;
 
 	type ReturnTypeOrOriginal<T> = T extends (...args: any[]) => infer R ? R : T;

package/types/content.d.ts

@@ -47,7 +47,7 @@ declare module 'astro:content' {
 		| DataCollectionConfig<S>;
 
 	export function defineCollection<S extends BaseSchema>(
-		input: CollectionConfig<S>
+		input: CollectionConfig<S>,
 	): CollectionConfig<S>;
 
 	/** Run `astro sync` to generate high fidelity types */

package/dist/actions/runtime/store.d.ts

@@ -1,5 +0,0 @@
-import { AsyncLocalStorage } from 'node:async_hooks';
-import type { APIContext } from '../../@types/astro.js';
-export type ActionAPIContext = Omit<APIContext, 'getActionResult' | 'props'>;
-export declare const ApiContextStorage: AsyncLocalStorage<ActionAPIContext>;
-export declare function getApiContext(): ActionAPIContext;

package/dist/actions/runtime/store.js

@@ -1,18 +0,0 @@
-import { AsyncLocalStorage } from "node:async_hooks";
-import { AstroError } from "../../core/errors/errors.js";
-const ApiContextStorage = new AsyncLocalStorage();
-function getApiContext() {
-  const context = ApiContextStorage.getStore();
-  if (!context) {
-    throw new AstroError({
-      name: "AstroActionError",
-      message: "Unable to get API context.",
-      hint: "If you attempted to call this action from server code, trying using `Astro.getActionResult()` instead."
-    });
-  }
-  return context;
-}
-export {
-  ApiContextStorage,
-  getApiContext
-};