Remove remaining deprecated APIs (#9263)

Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>

Author: bluwy

.changeset/green-parrots-brake.md

@@ -0,0 +1,10 @@
+---
+'astro': major
+---
+
+Removes additional deprecated APIs:
+
+- The Astro preview server now returns a 404 status instead of a 301 redirect when requesting assets from the public directory without a base.
+- Removes special handling when referencing the `astro/client-image` type. You should use the `astro/client` type instead.
+- Removes deprecated built-in `rss` support in `getStaticPaths`. You should use `@astrojs/rss` instead.
+- Removes deprecated `Astro.request.params` support. You should use `Astro.params` instead.

.changeset/little-beers-sit.md

@@ -0,0 +1,5 @@
+---
+'@astrojs/markdoc': minor
+---
+
+Removes internal `propagators` handling for Astro 3

.changeset/slimy-jeans-peel.md

@@ -0,0 +1,5 @@
+---
+'@astrojs/vercel': major
+---
+
+Removes the deprecated `@astrojs/vercel/edge` export. You should use `@astrojs/vercel/serverless` instead with the `edgeMiddleware` option.

packages/astro/src/@types/astro.ts

@@ -1793,13 +1793,9 @@ export type GetHydrateCallback = () => Promise<() => void | Promise<void>>;
  * getStaticPaths() options
  *
  * [Astro Reference](https://docs.astro.build/en/reference/api-reference/#getstaticpaths)
- */ export interface GetStaticPathsOptions {
+ */
+export interface GetStaticPathsOptions {
 	paginate: PaginateFunction;
-	/**
-	 * The RSS helper has been removed from getStaticPaths! Try the new @astrojs/rss package instead.
-	 * @see https://docs.astro.build/en/guides/rss/
-	 */
-	rss(): never;
 }
 
 export type GetStaticPathsItem = {

packages/astro/src/core/build/plugins/index.ts

@@ -1,7 +1,6 @@
 import { astroConfigBuildPlugin } from '../../../content/vite-plugin-content-assets.js';
 import { astroHeadBuildPlugin } from '../../../vite-plugin-head/index.js';
 import type { AstroBuildPluginContainer } from '../plugin.js';
-import { pluginAliasResolve } from './plugin-alias-resolve.js';
 import { pluginAnalyzer } from './plugin-analyzer.js';
 import { pluginChunks } from './plugin-chunks.js';
 import { pluginComponentEntry } from './plugin-component-entry.js';
@@ -18,7 +17,6 @@ import { pluginSSR, pluginSSRSplit } from './plugin-ssr.js';
 
 export function registerAllPlugins({ internals, options, register }: AstroBuildPluginContainer) {
 	register(pluginComponentEntry(internals));
-	register(pluginAliasResolve(internals));
 	register(pluginAnalyzer(options, internals));
 	register(pluginInternals(internals));
 	register(pluginManifest(options, internals));

packages/astro/src/core/build/plugins/plugin-alias-resolve.ts

@@ -292,6 +292,7 @@ export const InvalidGetStaticPathsReturn = {
 
 /**
  * @docs
+ * @deprecated Deprecated since Astro 4.0. The RSS helper no longer exists with an error fallback.
  * @see
  * - [RSS Guide](https://docs.astro.build/en/guides/rss/)
  * @description

packages/astro/src/core/errors/errors-data.ts

@@ -59,9 +59,6 @@ export async function callGetStaticPaths({
 		// Q: Why the cast?
 		// A: So users downstream can have nicer typings, we have to make some sacrifice in our internal typings, which necessitate a cast here
 		paginate: generatePaginateFunction(route) as PaginateFunction,
-		rss() {
-			throw new AstroError(AstroErrorData.GetStaticPathsRemovedRSSHelper);
-		},
 	});
 
 	validateGetStaticPathsResult(staticPaths, logger, route);

packages/astro/src/core/render/route-cache.ts

@@ -39,15 +39,6 @@ export function createRequest({
 		body,
 	});
 
-	Object.defineProperties(request, {
-		params: {
-			get() {
-				logger.warn('deprecated', `Astro.request.params has been moved to Astro.params`);
-				return undefined;
-			},
-		},
-	});
-
 	if (!ssr) {
 		// Warn when accessing headers in SSG mode
 		const _headers = request.headers;

packages/astro/src/core/request.ts

@@ -43,21 +43,18 @@ export function baseMiddleware(
 		}
 
 		// Check to see if it's in public and if so 404
-		// TODO: Remove redirect, turn this warning into an error in Astro 4.0
 		const publicPath = new URL('.' + req.url, config.publicDir);
 		fs.stat(publicPath, (_err, stats) => {
 			if (stats) {
 				const expectedLocation = new URL('.' + url, devRootURL).pathname;
-				logger.warn(
+				logger.error(
 					'router',
 					`Request URLs for ${bold(
 						'public/'
 					)} assets must also include your base. "${expectedLocation}" expected, but received "${url}".`
 				);
-				res.writeHead(301, {
-					Location: expectedLocation,
-				});
-				res.end();
+				const html = subpathNotUsedTemplate(devRoot, pathname);
+				return writeHtmlResponse(res, 404, html);
 			} else {
 				next();
 			}

packages/astro/src/vite-plugin-astro-server/base.ts

@@ -50,16 +50,6 @@ export async function setUpEnvTs({
 	if (fs.existsSync(envTsPath)) {
 		let typesEnvContents = await fs.promises.readFile(envTsPath, 'utf-8');
 
-		// TODO: Remove this in 4.0, this code is only to help users migrate away from assets being experimental for a long time
-		if (typesEnvContents.includes('types="astro/client-image"')) {
-			typesEnvContents = typesEnvContents.replace(
-				'types="astro/client-image"',
-				'types="astro/client"'
-			);
-			await fs.promises.writeFile(envTsPath, typesEnvContents, 'utf-8');
-			logger.info('types', `Removed ${bold(envTsPathRelativetoRoot)} type declarations`);
-		}
-
 		if (!fs.existsSync(dotAstroDir))
 			// Add `.astro` types reference if none exists
 			return;

packages/astro/src/vite-plugin-inject-env-ts/index.ts

@@ -253,8 +253,7 @@ describe('dev container', () => {
 				container.handle(r.req, r.res);
 				await r.done;
 
-				expect(r.res.statusCode).to.equal(301);
-				expect(r.res.getHeader('location')).to.equal('/sub/test.txt');
+				expect(r.res.statusCode).to.equal(404);
 			}
 		);
 	});

packages/astro/test/units/dev/dev.test.js

@@ -89,10 +89,7 @@ export const ComponentNode = createComponent({
 			);
 
 			// Let the runtime know that this component is being used.
-			// `result.propagators` has been moved to `result._metadata.propagators`
-			// TODO: remove this fallback in the next markdoc integration major
-			const propagators = result._metadata.propagators || result.propagators;
-			propagators.add({
+			result._metadata.propagators.add({
 				init() {
 					return headAndContent;
 				},

packages/integrations/markdoc/components/TreeNode.ts

@@ -17,8 +17,6 @@
   "bugs": "https://github.com/withastro/astro/issues",
   "homepage": "https://docs.astro.build/en/guides/integrations-guide/vercel/",
   "exports": {
-    "./edge": "./dist/edge/throw.js",
-    "./edge/entrypoint": "./dist/edge/throw.js",
     "./serverless": "./dist/serverless/adapter.js",
     "./serverless/entrypoint": "./dist/serverless/entrypoint.js",
     "./static": "./dist/static/adapter.js",