fix(markdown): file.url fixes (#3198)

* fix(markdown): file.url fixes

* Added tests

* Changed the default of `base` from `./` to `/`

* Make the url work with subpath

* Changeset

* Forgot to change this comparison

.changeset/large-birds-repair.md

@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+Markdown file.url now respects `trailingSlash` and `base`

packages/astro/src/core/build/generate.ts

@@ -179,7 +179,7 @@ async function generatePath(
 	// If a base path was provided, append it to the site URL. This ensures that
 	// all injected scripts and links are referenced relative to the site and subpath.
 	const site =
-		astroConfig.base && astroConfig.base !== './'
+		astroConfig.base !== '/'
 			? joinPaths(astroConfig.site?.toString() || 'http://localhost/', astroConfig.base)
 			: astroConfig.site;
 	const links = createLinkStylesheetElementSet(linkIds.reverse(), site);

packages/astro/src/core/config.ts

@@ -94,8 +94,8 @@ export const AstroConfigSchema = z.object({
 	base: z
 		.string()
 		.optional()
-		.default('./')
+		.default('/')
-		.transform((val) => (val ? appendForwardSlash(trimSlashes(val)) : val)),
+		.transform((val) => appendForwardSlash(trimSlashes(val))),
 	trailingSlash: z
 		.union([z.literal('always'), z.literal('never'), z.literal('ignore')])
 		.optional()

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

@@ -114,8 +114,7 @@ async function handle404Response(
 		// HACK: redirect without the base path for assets in publicDir
 		const redirectTo =
 			req.method === 'GET' &&
-			config.base &&
+			config.base !== '/' &&
-			config.base !== './' &&
 			pathname.startsWith(config.base) &&
 			pathname.replace(config.base, '/');
 

packages/astro/src/vite-plugin-markdown/index.ts

@@ -78,13 +78,18 @@ export default function markdown({ config }: AstroPluginOptions): Plugin {
 			// Return the file's JS representation, including all Markdown
 			// frontmatter and a deferred `import() of the compiled markdown content.
 			if (id.endsWith(`.md${MARKDOWN_IMPORT_FLAG}`)) {
-				const sitePathname = config.site
+				const sitePathname = appendForwardSlash(
-					? appendForwardSlash(new URL(config.base, config.site).pathname)
+					config.site ? new URL(config.base, config.site).pathname : config.base
-					: '/';
+				);
+
 				const fileId = id.replace(MARKDOWN_IMPORT_FLAG, '');
-				const fileUrl = fileId.includes('/pages/')
+				let fileUrl = fileId.includes('/pages/')
-					? fileId.replace(/^.*\/pages\//, sitePathname).replace(/(\/index)?\.md$/, '')
+					? fileId.replace(/^.*?\/pages\//, sitePathname).replace(/(\/index)?\.md$/, '')
 					: undefined;
+				if (fileUrl && config.trailingSlash === 'always') {
+					fileUrl = appendForwardSlash(fileUrl);
+				}
+
 				const source = await fs.promises.readFile(fileId, 'utf8');
 				const { data: frontmatter } = matter(source);
 				return {

packages/astro/test/astro-markdown-url.test.js

@@ -0,0 +1,100 @@
+import { expect } from 'chai';
+import cheerio from 'cheerio';
+import { loadFixture } from './test-utils.js';
+
+describe('Astro Markdown URL', () => {
+	describe('With subpath', () => {
+		const baseUrl = `/my-cool-base/docs/pages/how-to-make-a-page`;
+
+		it('trailingSlash: always', async () => {
+			let fixture = await loadFixture({
+				root: './fixtures/astro-markdown-url/',
+				outDir: new URL('./fixtures/astro-markdown-url/with-subpath-always/', import.meta.url),
+				base: '/my-cool-base',
+				trailingSlash: 'always',
+			});
+			await fixture.build();
+
+			const html = await fixture.readFile('/index.html');
+			const $ = cheerio.load(html);
+
+			expect($('#url').attr('href')).to.equal(baseUrl + '/');
+		});
+
+		it('trailingSlash: never', async () => {
+			let fixture = await loadFixture({
+				root: './fixtures/astro-markdown-url/',
+				outDir: new URL('./fixtures/astro-markdown-url/with-subpath-never/', import.meta.url),
+				base: '/my-cool-base',
+				trailingSlash: 'never',
+			});
+			await fixture.build();
+
+			const html = await fixture.readFile('/index.html');
+			const $ = cheerio.load(html);
+
+			expect($('#url').attr('href')).to.equal(baseUrl);
+		});
+
+		it('trailingSlash: ignore', async () => {
+			let fixture = await loadFixture({
+				root: './fixtures/astro-markdown-url/',
+				outDir: new URL('./fixtures/astro-markdown-url/with-subpath-ignore/', import.meta.url),
+				base: '/my-cool-base',
+				trailingSlash: 'ignore',
+			});
+			await fixture.build();
+
+			const html = await fixture.readFile('/index.html');
+			const $ = cheerio.load(html);
+
+			expect($('#url').attr('href')).to.equal(baseUrl);
+		});
+	});
+
+	describe('Without subpath', () => {
+		const baseUrl = `/docs/pages/how-to-make-a-page`;
+
+		it('trailingSlash: always', async () => {
+			let fixture = await loadFixture({
+				root: './fixtures/astro-markdown-url/',
+				outDir: new URL('./fixtures/astro-markdown-url/without-subpath-always/', import.meta.url),
+				trailingSlash: 'always',
+			});
+			await fixture.build();
+
+			const html = await fixture.readFile('/index.html');
+			const $ = cheerio.load(html);
+
+			expect($('#url').attr('href')).to.equal(baseUrl + '/');
+		});
+
+		it('trailingSlash: never', async () => {
+			let fixture = await loadFixture({
+				root: './fixtures/astro-markdown-url/',
+				outDir: new URL('./fixtures/astro-markdown-url/without-subpath-never/', import.meta.url),
+				trailingSlash: 'never',
+			});
+			await fixture.build();
+
+			const html = await fixture.readFile('/index.html');
+			const $ = cheerio.load(html);
+
+			expect($('#url').attr('href')).to.equal(baseUrl);
+		});
+
+		it('trailingSlash: ignore', async () => {
+			let fixture = await loadFixture({
+				root: './fixtures/astro-markdown-url/',
+				outDir: new URL('./fixtures/astro-markdown-url/without-subpath-ignore/', import.meta.url),
+				trailingSlash: 'ignore',
+			});
+			await fixture.build();
+
+			const html = await fixture.readFile('/index.html');
+			const $ = cheerio.load(html);
+
+			expect($('#url').attr('href')).to.equal(baseUrl);
+		});
+	});
+});

packages/astro/test/fixtures/astro-markdown-url/.gitignore

@@ -0,0 +1,6 @@
+with-subpath-always/
+with-subpath-never/
+with-subpath-ignore/
+without-subpath-always/
+without-subpath-never/
+without-subpath-ignore/

packages/astro/test/fixtures/astro-markdown-url/package.json

@@ -0,0 +1,8 @@
+{
+  "name": "@test/astro-markdown-url",
+  "version": "0.0.0",
+  "private": true,
+  "dependencies": {
+    "astro": "workspace:*"
+  }
+}

packages/astro/test/fixtures/astro-markdown-url/src/pages/docs/pages/how-to-make-a-page.md

@@ -0,0 +1,5 @@
+---
+title: How to make a page
+---
+
+Test

packages/astro/test/fixtures/astro-markdown-url/src/pages/index.astro

@@ -0,0 +1,5 @@
+---
+import * as page from './docs/pages/how-to-make-a-page.md'
+---
+
+<a id="url" href={page.url}>{page.frontmatter.title}</a>

pnpm-lock.yaml

@@ -860,6 +860,12 @@ importers:
     dependencies:
       astro: link:../../../..
 
+  packages/astro/test/fixtures/astro-markdown-url:
+    specifiers:
+      astro: workspace:*
+    dependencies:
+      astro: link:../../..
+
   packages/astro/test/fixtures/astro-page-directory-url:
     specifiers:
       astro: workspace:*