scavold 0.2.0-rc.6 → 0.2.0-rc.7

package/CHANGELOG.md

@@ -9,6 +9,14 @@ changes may occur in any release.
 
 ## [Unreleased]
 
+## [0.2.0-rc.7] — 2026-09-29
+
+### Fixed
+
+- A link to a media file whose extension VitePress does not know, such as `.diff`,
+  pointed at the file name with `.html` appended and therefore at nothing. It now names
+  the file as published.
+
 ## [0.2.0-rc.6] — 2026-09-13
 
 ### Added
@@ -250,6 +258,9 @@ First published release. Version `0.1.0` existed in-tree only.
 - Only images are processed out of `media_folder`; other file types (video, documents)
   are never copied into the build output and have to live in the static folder.
 
+[Unreleased]: https://gitlab.com/cratly/scavold/-/compare/v0.2.0-rc.7...main
+[0.2.0-rc.7]: https://gitlab.com/cratly/scavold/-/compare/v0.2.0-rc.6...v0.2.0-rc.7
+[0.2.0-rc.6]: https://gitlab.com/cratly/scavold/-/compare/v0.2.0-rc.5...v0.2.0-rc.6
 [0.2.0-rc.5]: https://gitlab.com/cratly/scavold/-/compare/v0.2.0-rc.4...v0.2.0-rc.5
 [0.2.0-rc.4]: https://gitlab.com/cratly/scavold/-/compare/v0.2.0-rc.3...v0.2.0-rc.4
 [0.2.0-rc.3]: https://gitlab.com/cratly/scavold/-/compare/v0.2.0-rc.2...v0.2.0-rc.3

package/lib/config.js

@@ -653,7 +653,7 @@ export async function augmentConfig( rawConfig, options = {} ) {
 			//   during in-app use.
 			//
 			// The redirect value can be either:
-			//   - a site-relative URL  ("/de/jobs")  — written by the Cratly editor
+			//   - a site-relative URL  ("/de/jobs")  — written by the cratly editor
 			//   - a relative .md path  ("jobs.md")   — written by hand in frontmatter
 			//   - an absolute external URL            — e.g. "https://example.com"
 			const redirect = pageData.frontmatter?.redirect;
@@ -714,14 +714,31 @@ export async function augmentConfig( rawConfig, options = {} ) {
 				// offers, which is relative to that folder. Rewrite it to the URL the
 				// built site serves. Targets that are not media files (ordinary page
 				// links, anchors, remote URLs) resolve to null and stay untouched.
-				patchRenderer( md, "link_open", ( defaultHandler, tokens, idx, ...args ) => {
-					const published = media.collectAsset( tokens[idx].attrGet( "href" ) );
+				//
+				// VitePress then takes any extension missing from its own list of file
+				// types (`.diff`, say) for part of a page name and appends `.html`. The
+				// published URL is final, so that suffix is taken back — after VitePress
+				// ran, which leaves prefixing the site's base to it.
+				patchRenderer( md, "link_open", ( defaultHandler, tokens, idx, mdOptions, env, slf ) => {
+					const token = tokens[idx];
+					const published = media.collectAsset( token.attrGet( "href" ) );
+
+					if ( !published ) {
+						return defaultHandler( tokens, idx, mdOptions, env, slf );
+					}
+
+					token.attrSet( "href", published );
+
+					const rendered = defaultHandler( tokens, idx, mdOptions, env, slf );
+					const href = token.attrGet( "href" );
+
+					if ( !published.endsWith( ".html" ) && href?.endsWith( published + ".html" ) ) {
+						token.attrSet( "href", href.slice( 0, -".html".length ) );
 
-					if ( published ) {
-						tokens[idx].attrSet( "href", published );
+						return slf.renderToken( tokens, idx, mdOptions );
 					}
 
-					return defaultHandler( tokens, idx, ...args );
+					return rendered;
 				} );
 
 				registerContainers( md, containerMap, {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "scavold",
-	"version": "0.2.0-rc.6",
+	"version": "0.2.0-rc.7",
 	"type": "module",
 	"description": "VitePress theme framework — a scaffold for building custom VitePress themes with Vue at the core",
 	"keywords": [

package/scripts/check-fixture.js

@@ -53,7 +53,7 @@ function linkSelf() {
 
 /**
  * Creates the fixture's media files. Images are real (the pipeline reads their
- * dimensions); the other two only ever get copied, so a few bytes carrying the right
+ * dimensions); the others only ever get copied, so a few bytes carrying the right
  * extension are enough.
  */
 async function writeMedia() {
@@ -76,6 +76,8 @@ async function writeMedia() {
 
 	writeFileSync( join( media, "doc.pdf" ), "%PDF-1.4\n% fixture placeholder\n" );
 	writeFileSync( join( media, "clip.mp4" ), "fixture placeholder\n" );
+	// An extension VitePress does not know, so its link rules take the file for a page.
+	writeFileSync( join( media, "patch.diff" ), "--- a\n+++ b\n" );
 }
 
 /**
@@ -194,6 +196,8 @@ console.log( "\nmedia that is not an image" );
 expect( published.includes( "doc.pdf" ), "copies a document into the site output" );
 expect( published.includes( "clip.mp4" ), "copies a video into the site output" );
 expect( media.includes( 'href="/media/doc.pdf"' ), "rewrites a link to a media file" );
+expect( media.includes( 'href="/media/patch.diff"' ),
+	"links a media file of an extension VitePress does not know as that file, not as a page" );
 // On the entry page, where the fixture's page links live. VitePress resolves them to
 // .html; what matters is that the media pipeline did not touch them.
 expect( built( "index.html" ).includes( 'href="/containers.html"' ), "leaves a link to a page alone" );