emdash 1.2.0 → 1.3.1

package/src/cli/commands/import/wordpress.ts

@@ -16,6 +16,7 @@ import pc from "picocolors";
 import { slugify } from "#utils/slugify.js";
 
 import { validateExternalUrl, ssrfSafeFetch } from "../../../import/ssrf.js";
+import { INTERNAL_POST_TYPES, isInternalPostType } from "../../../import/utils.js";
 import { parseWxr, type WxrData, type WxrPost, type WxrAttachment } from "../../wxr/parser.js";
 
 // Regex patterns for WordPress import
@@ -385,20 +386,7 @@ function generateMigrationConfig(wxr: WxrData, analysis: ContentAnalysis): Migra
 
 	// Map post types to collections
 	for (const [type, count] of analysis.postTypes) {
-		// Skip internal types (see INTERNAL_POST_TYPES in utils.ts)
-		const skip = [
-			"revision",
-			"nav_menu_item",
-			"custom_css",
-			"customize_changeset",
-			"oembed_cache",
-			"wp_global_styles",
-			"wp_navigation",
-			"wp_template",
-			"wp_template_part",
-			"attachment", // Handled separately as media
-			"wp_block", // Handled separately as sections (reusable blocks)
-		].includes(type);
+		const skip = isInternalPostType(type);
 
 		collections[type] = {
 			collection: mapPostTypeToCollection(type),
@@ -425,19 +413,7 @@ function generateMigrationConfig(wxr: WxrData, analysis: ContentAnalysis): Migra
 		},
 		collections,
 		fields,
-		skipPostTypes: [
-			"revision",
-			"nav_menu_item",
-			"custom_css",
-			"customize_changeset",
-			"oembed_cache",
-			"wp_global_styles",
-			"wp_navigation",
-			"wp_template",
-			"wp_template_part",
-			"attachment",
-			"wp_block",
-		],
+		skipPostTypes: [...INTERNAL_POST_TYPES],
 		skipMetaKeys: ["_edit_last", "_edit_lock", "_pingme", "_encloseme"],
 	};
 }

package/src/cli/commands/login.ts

@@ -23,6 +23,7 @@ import {
 	resolveCustomHeaders,
 	runCloudflaredLogin,
 } from "../../client/cf-access.js";
+import { connectionArgs, resolveBaseUrl } from "../client-factory.js";
 import {
 	getCredentials,
 	removeCredentials,
@@ -188,7 +189,6 @@ export const loginCommand = defineCommand({
 			type: "string",
 			alias: "u",
 			description: "EmDash instance URL",
-			default: "http://localhost:4321",
 		},
 		header: {
 			type: "string",
@@ -197,7 +197,7 @@ export const loginCommand = defineCommand({
 		},
 	},
 	async run({ args }) {
-		const baseUrl = args.url || "http://localhost:4321";
+		const baseUrl = resolveBaseUrl(args);
 		consola.start(`Connecting to ${baseUrl}...`);
 
 		// Resolve custom headers from --header flags and EMDASH_HEADERS env
@@ -370,11 +370,10 @@ export const logoutCommand = defineCommand({
 			type: "string",
 			alias: "u",
 			description: "EmDash instance URL",
-			default: "http://localhost:4321",
 		},
 	},
 	async run({ args }) {
-		const baseUrl = args.url || "http://localhost:4321";
+		const baseUrl = resolveBaseUrl(args);
 
 		// Get stored credentials
 		const cred = getCredentials(baseUrl);
@@ -409,42 +408,27 @@ export const whoamiCommand = defineCommand({
 		name: "whoami",
 		description: "Show current user and auth method",
 	},
-	args: {
-		url: {
-			type: "string",
-			alias: "u",
-			description: "EmDash instance URL",
-			default: "http://localhost:4321",
-		},
-		token: {
-			type: "string",
-			alias: "t",
-			description: "Auth token",
-		},
-		json: {
-			type: "boolean",
-			description: "Output as JSON",
-		},
-	},
+	args: { ...connectionArgs },
 	async run({ args }) {
 		configureOutputMode(args);
-		const baseUrl = args.url || "http://localhost:4321";
+		const baseUrl = resolveBaseUrl(args);
 
 		// Resolve token: --token flag > EMDASH_TOKEN env > stored credentials
 		let token = args.token || process.env["EMDASH_TOKEN"];
 		let authMethod = token ? "token" : "none";
-		let storedHeaders: Record<string, string> = {};
+		const cred = token ? null : getCredentials(baseUrl);
+		const headerFetch = createHeaderAwareFetch({
+			...cred?.customHeaders,
+			...resolveCustomHeaders(),
+		});
 
 		if (!token) {
-			const cred = getCredentials(baseUrl);
 			if (cred) {
 				token = cred.accessToken;
 				authMethod = "stored";
-				storedHeaders = cred.customHeaders ?? {};
 
 				// Check if expired
 				if (new Date(cred.expiresAt) < new Date()) {
-					const headerFetch = createHeaderAwareFetch(storedHeaders);
 					// Try to refresh
 					try {
 						const refreshRes = await headerFetch(
@@ -490,40 +474,46 @@ export const whoamiCommand = defineCommand({
 			const isLocal = baseUrl.includes("localhost") || baseUrl.includes("127.0.0.1");
 			if (isLocal) {
 				authMethod = "dev-bypass";
-				consola.info(`Auth method: ${pc.cyan("dev-bypass")}`);
-				consola.info("No stored credentials. Client will use dev bypass for localhost.");
-				return;
+			} else {
+				consola.error("Not logged in. Run: emdash login");
+				process.exit(2);
 			}
-
-			consola.error("Not logged in. Run: emdash login");
-			process.exit(2);
 		}
 
-		const headerFetch = createHeaderAwareFetch(storedHeaders);
-
 		try {
-			const meRes = await headerFetch(new URL("/_emdash/api/auth/me", baseUrl), {
-				headers: { Authorization: `Bearer ${token}` },
-			});
+			const meRes = token
+				? await headerFetch(new URL("/_emdash/api/auth/me", baseUrl), {
+						headers: { Authorization: `Bearer ${token}` },
+					})
+				: await headerFetch(new URL("/_emdash/api/auth/dev-bypass", baseUrl), {
+						redirect: "manual",
+					});
 
 			if (!meRes.ok) {
-				if (meRes.status === 401) {
+				if (token && meRes.status === 401) {
 					consola.error("Token is invalid or expired. Run: emdash login");
 					process.exit(1);
 				}
+				if (!token && meRes.status === 403) {
+					consola.error("The dev bypass only works under `astro dev`. Run: emdash login");
+					process.exit(1);
+				}
+				if (!token && meRes.status === 404) {
+					consola.error("The dev bypass is not available on this site. Run: emdash login");
+					process.exit(1);
+				}
 				consola.error(`Failed to fetch user info: ${meRes.status}`);
 				process.exit(1);
 			}
 
-			const raw = (await meRes.json()) as {
-				data: {
-					id: string;
-					email: string;
-					name: string | null;
-					role: number;
-				};
-			};
-			const me = raw.data;
+			interface Me {
+				id: string;
+				email: string;
+				name: string | null;
+				role: number;
+			}
+			const raw = (await meRes.json()) as { data: Me | { user: Me } };
+			const me = "user" in raw.data ? raw.data.user : raw.data;
 
 			const roleNames: Record<number, string> = {
 				10: "subscriber",

package/src/cli/commands/migrate.ts

@@ -52,6 +52,7 @@ export interface MigrateCommandOptions extends MigrationTargetOverrides {
 	json?: boolean;
 	expectedTargetFingerprint?: string;
 	releaseLock?: string;
+	takeOverLock?: string;
 }
 
 export interface MigrateCommandDependencies {
@@ -242,6 +243,16 @@ function validateOptions(options: MigrateCommandOptions): void {
 			throw new MigrateCommandError("--release-lock requires the lock id shown by --status.");
 		}
 	}
+	if (options.takeOverLock !== undefined) {
+		if (options.check || options.status || options.releaseLock !== undefined) {
+			throw new MigrateCommandError(
+				"--take-over-lock cannot be used with --check, --status, or --release-lock.",
+			);
+		}
+		if (!LOCK_ID_PATTERN.test(options.takeOverLock)) {
+			throw new MigrateCommandError("--take-over-lock requires the lock id shown by --status.");
+		}
+	}
 	if (
 		options.expectedTargetFingerprint &&
 		!FINGERPRINT_PATTERN.test(options.expectedTargetFingerprint)
@@ -515,12 +526,15 @@ export async function runMigrateCommand(
 			);
 
 			const releasing = options.releaseLock !== undefined;
+			const takingOver = options.takeOverLock !== undefined;
 			const applying = !options.check && !options.status && !releasing;
 			if (applying || releasing) {
-				const operation = releasing ? "lock release" : "apply";
+				const operation = releasing ? "lock release" : takingOver ? "lock take-over" : "apply";
 				const question = releasing
 					? `Release migration lock ${options.releaseLock} on ${target.label}? Release it only when no migration is running.`
-					: `Apply EmDash migrations to ${target.label}?`;
+					: takingOver
+						? `Take over migration lock ${options.takeOverLock} on ${target.label} and apply EmDash migrations? Take it over only when no migration is running.`
+						: `Apply EmDash migrations to ${target.label}?`;
 				if (options.expectedTargetFingerprint) {
 					if (options.expectedTargetFingerprint !== target.fingerprint) {
 						dependencies.writeStderr("Expected target fingerprint does not match the target.");
@@ -534,7 +548,13 @@ export async function runMigrateCommand(
 					exitCode = MIGRATE_EXIT_CODES.confirmation;
 					return exitCode;
 				} else if (!(await dependencies.confirm(question))) {
-					dependencies.writeStderr(releasing ? "Lock release cancelled." : "Migration cancelled.");
+					dependencies.writeStderr(
+						releasing
+							? "Lock release cancelled."
+							: takingOver
+								? "Lock take-over cancelled."
+								: "Migration cancelled.",
+					);
 					exitCode = MIGRATE_EXIT_CODES.confirmation;
 					return exitCode;
 				}
@@ -549,6 +569,10 @@ export async function runMigrateCommand(
 				},
 			};
 			if (applying) request.action = "apply";
+			if (takingOver) {
+				request.action = "take-over-lock";
+				request.lockId = options.takeOverLock;
+			}
 			if (releasing) {
 				request.action = "release-lock";
 				request.lockId = options.releaseLock;
@@ -563,6 +587,8 @@ export async function runMigrateCommand(
 				dependencies.writeStdout(JSON.stringify(report));
 			} else {
 				if (releasing) dependencies.writeStdout(`Released migration lock ${options.releaseLock}.`);
+				if (takingOver)
+					dependencies.writeStdout(`Took over migration lock ${options.takeOverLock}.`);
 				printHumanReport(report, dependencies.writeStdout);
 			}
 			exitCode = reportExitCode(options, report);
@@ -604,12 +630,17 @@ export const migrateCommand = defineCommand({
 		json: { type: "boolean", description: "Print the stable JSON report" },
 		"expected-target-fingerprint": {
 			type: "string",
-			description: "Required target fingerprint for noninteractive apply or lock release",
+			description:
+				"Required target fingerprint for noninteractive apply, lock release, or take-over",
 		},
 		"release-lock": {
 			type: "string",
 			description: "Release the migration lock with the id shown by --status",
 		},
+		"take-over-lock": {
+			type: "string",
+			description: "Apply migrations, taking over the migration lock with the id shown by --status",
+		},
 		database: { type: "string", description: "Override the SQLite database path" },
 		"database-url-env": {
 			type: "string",
@@ -630,6 +661,7 @@ export const migrateCommand = defineCommand({
 			json: args.json,
 			expectedTargetFingerprint: args["expected-target-fingerprint"],
 			releaseLock: args["release-lock"],
+			takeOverLock: args["take-over-lock"],
 			database: args.database,
 			databaseUrlEnv: args["database-url-env"],
 			d1: args.d1,

package/src/cli/commands/seed.ts

@@ -17,6 +17,7 @@ import { claimExplicitSeedOwnership } from "../../seed/ownership.js";
 import type { SeedFile, SeedApplyOptions } from "../../seed/types.js";
 import { validateSeed } from "../../seed/validate.js";
 import { LocalStorage } from "../../storage/local.js";
+import type { Storage } from "../../storage/types.js";
 
 interface PackageJson {
 	name?: string;
@@ -128,6 +129,12 @@ export const seedCommand = defineCommand({
 			description: "Base URL for media files",
 			default: "/_emdash/api/media/file",
 		},
+		"skip-media": {
+			type: "boolean",
+			description:
+				"Resolve $media references to external URLs instead of downloading and storing files",
+			default: false,
+		},
 	},
 	async run({ args }) {
 		const cwd = resolve(args.cwd);
@@ -200,14 +207,16 @@ export const seedCommand = defineCommand({
 			process.exit(1);
 		}
 
-		// Set up storage for $media resolution
-		const uploadsDir = resolve(cwd, args["uploads-dir"]);
-		await mkdir(uploadsDir, { recursive: true });
-
-		const storage = new LocalStorage({
-			directory: uploadsDir,
-			baseUrl: args["media-base-url"],
-		});
+		// Set up storage for $media resolution unless media handling is skipped
+		let storage: Storage | undefined;
+		if (!args["skip-media"]) {
+			const uploadsDir = resolve(cwd, args["uploads-dir"]);
+			await mkdir(uploadsDir, { recursive: true });
+			storage = new LocalStorage({
+				directory: uploadsDir,
+				baseUrl: args["media-base-url"],
+			});
+		}
 
 		// Prepare apply options
 		const onConflictRaw = args["on-conflict"];
@@ -222,6 +231,7 @@ export const seedCommand = defineCommand({
 			includeContent: args.content,
 			onConflict: onConflictRaw,
 			storage,
+			skipMediaDownload: args["skip-media"],
 		};
 
 		// Apply seed

package/src/cli/wxr/parser.ts

@@ -863,8 +863,8 @@ export function parseWxrString(xml: string): Promise<WxrData> {
 			const tag = tagName.toLowerCase();
 			const text = currentText.trim();
 
-			// Site metadata
-			if (currentPath.length === 2 && currentPath[0] === "rss") {
+			// Site metadata: the closing tag is still on the path (rss > channel > tag)
+			if (currentPath.length === 3 && currentPath[1] === "channel") {
 				switch (tag) {
 					case "title":
 						data.site.title = text;

package/src/client/index.ts

@@ -620,6 +620,7 @@ export class EmDashClient {
 								options.onTokenRefresh!(accessToken, expiresIn);
 							}
 						: undefined,
+					fetch: createTransport({ interceptors: options.interceptors }).fetch,
 				}),
 			);
 		}
@@ -652,7 +653,12 @@ export class EmDashClient {
 		if (col.fields) {
 			this.fieldSchemaCache.set(
 				slug,
-				col.fields.map((f) => ({ slug: f.slug, type: f.type, blockTypes: f.blockTypes })),
+				col.fields.map((field) => ({
+					slug: field.slug,
+					type: field.type,
+					blockTypes: field.blockTypes,
+					validation: field.validation,
+				})),
 			);
 		}
 		return col;
@@ -1648,7 +1654,12 @@ export class EmDashClient {
 
 		try {
 			const col = await this.collection(collection);
-			cached = col.fields.map((f) => ({ slug: f.slug, type: f.type, blockTypes: f.blockTypes }));
+			cached = col.fields.map((field) => ({
+				slug: field.slug,
+				type: field.type,
+				blockTypes: field.blockTypes,
+				validation: field.validation,
+			}));
 			this.fieldSchemaCache.set(collection, cached);
 			return cached;
 		} catch {

package/src/client/portable-text.ts

@@ -373,6 +373,7 @@ export function resetKeyCounter(): void {
 export interface FieldSchema {
 	slug: string;
 	type: string;
+	validation?: unknown;
 	blockTypes?: Array<{
 		slug: string;
 		currentVersion: number;
@@ -389,10 +390,21 @@ function convertNestedBlockPortableText(
 	direction: "read" | "write",
 ): unknown {
 	if (!Array.isArray(value)) return value;
+	const validation = field.validation;
+	const allowedTypes =
+		validation && typeof validation === "object" && "allowedTypes" in validation
+			? validation.allowedTypes
+			: undefined;
 	return value.map((block) => {
 		if (!block || typeof block !== "object" || Array.isArray(block)) return block;
 		const record = { ...block };
-		const type = typeof record._type === "string" ? record._type : undefined;
+		const type =
+			record._type === undefined &&
+			direction === "write" &&
+			Array.isArray(allowedTypes) &&
+			allowedTypes.length === 1
+				? allowedTypes[0]
+				: record._type;
 		const version = typeof record._version === "number" ? record._version : undefined;
 		const blockType = field.blockTypes?.find((candidate) => candidate.slug === type);
 		const definition = blockType?.versions.find(
@@ -413,8 +425,9 @@ function convertNestedBlockPortableText(
 }
 
 /**
- * Convert content data for reading: PT fields -> markdown strings.
- * Only converts fields with type "portableText" that contain arrays.
+ * Convert content data for reading: Portable Text arrays -> Markdown strings.
+ * Converts top-level `portableText` arrays and nested `portableText` arrays
+ * inside `blocks` arrays.
  */
 export function convertDataForRead(
 	data: Record<string, unknown>,
@@ -427,7 +440,7 @@ export function convertDataForRead(
 	for (const field of fields) {
 		if (field.type === "portableText" && Array.isArray(result[field.slug])) {
 			result[field.slug] = portableTextToMarkdown(result[field.slug] as PortableTextBlock[]);
-		} else if (field.type === "blocks") {
+		} else if (field.type === "blocks" && Array.isArray(result[field.slug])) {
 			result[field.slug] = convertNestedBlockPortableText(result[field.slug], field, "read");
 		}
 	}
@@ -435,8 +448,9 @@ export function convertDataForRead(
 }
 
 /**
- * Convert content data for writing: markdown strings -> PT arrays.
- * Only converts fields with type "portableText" that contain strings.
+ * Convert content data for writing: Markdown strings -> Portable Text arrays.
+ * Converts top-level `portableText` strings and nested Markdown strings
+ * in `portableText` fields inside `blocks` arrays.
  */
 export function convertDataForWrite(
 	data: Record<string, unknown>,
@@ -446,7 +460,7 @@ export function convertDataForWrite(
 	for (const field of fields) {
 		if (field.type === "portableText" && typeof result[field.slug] === "string") {
 			result[field.slug] = markdownToPortableText(result[field.slug] as string);
-		} else if (field.type === "blocks") {
+		} else if (field.type === "blocks" && Array.isArray(result[field.slug])) {
 			result[field.slug] = convertNestedBlockPortableText(result[field.slug], field, "write");
 		}
 	}

package/src/client/transport.ts

@@ -140,18 +140,22 @@ export function refreshInterceptor(options: {
 	refreshToken: string;
 	tokenEndpoint: string;
 	onTokenRefreshed?: (accessToken: string, refreshToken: string, expiresAt: string) => void;
+	fetch?: (request: Request) => Promise<Response>;
 }): Interceptor {
+	const send = options.fetch ?? baseFetch;
 	let refreshing: Promise<string | null> | null = null;
 
 	async function refresh(): Promise<string | null> {
-		const res = await globalThis.fetch(options.tokenEndpoint, {
-			method: "POST",
-			headers: { "Content-Type": "application/json" },
-			body: JSON.stringify({
-				grant_type: "refresh_token",
-				refresh_token: options.refreshToken,
+		const res = await send(
+			new Request(options.tokenEndpoint, {
+				method: "POST",
+				headers: { "Content-Type": "application/json" },
+				body: JSON.stringify({
+					grant_type: "refresh_token",
+					refresh_token: options.refreshToken,
+				}),
 			}),
-		});
+		);
 
 		if (!res.ok) return null;
 

package/src/comments/query.ts

@@ -78,8 +78,6 @@ export async function getCommentsWithDb(
 ): Promise<GetCommentsResult> {
 	const repo = new CommentRepository(db);
 
-	const total = await repo.countByContent(options.collection, options.contentId, "approved");
-
 	// Server-rendered: fetch all comments (capped for safety).
 	// The API route handles paginated access; this is for full-page renders.
 	const MAX_COMMENTS = 500;
@@ -89,6 +87,14 @@ export async function getCommentsWithDb(
 		limit: MAX_COMMENTS,
 	});
 
+	// When the list is not truncated, the total is available from the page
+	// itself and we can skip the separate COUNT query. Fall back only when
+	// cursor pagination signals there are more rows.
+	const total =
+		result.nextCursor === undefined
+			? result.items.length
+			: await repo.countByContent(options.collection, options.contentId, "approved");
+
 	const items: PublicComment[] = options.threaded
 		? CommentRepository.assembleThreads(result.items).map((c) =>
 				CommentRepository.toPublicComment(c),

package/src/components/CommentForm.astro

@@ -108,7 +108,7 @@ const endpoint = `/_emdash/api/comments/${encodeURIComponent(collection)}/${enco
 			{/* Honeypot — hidden from humans, filled by bots */}
 			<div
 				aria-hidden="true"
-				style="position:absolute;left:-9999px;top:-9999px;"
+				style="position:absolute;width:1px;height:1px;overflow:hidden;clip-path:inset(50%);white-space:nowrap;"
 			>
 				<label>
 					Don't fill this out
@@ -323,7 +323,7 @@ const endpoint = `/_emdash/api/comments/${encodeURIComponent(collection)}/${enco
 
 	.ec-comment-user-email::before {
 		content: "·";
-		margin-right: 0.5rem;
+		margin-inline-end: 0.5rem;
 	}
 
 	.ec-comment-form-submit {

package/src/components/Comments.astro

@@ -313,7 +313,7 @@ const REACTION_SCRIPT = `
 
 	.ec-comment-replies {
 		margin-top: var(--ec-comment-gap);
-		padding-left: var(--ec-comment-indent);
+		padding-inline-start: var(--ec-comment-indent);
 	}
 
 	.ec-comment-replies > li + li {

package/src/components/EmDashImage.astro

@@ -22,6 +22,7 @@
  */
 import type { ImageValue, MediaValue } from "../fields/types.js";
 import type { HTMLAttributes } from "astro/types";
+import type { LocalImageProps } from "astro:assets";
 import { Image as AstroImage, getImage } from "astro:assets";
 import type { ImageEmbed } from "../media/types.js";
 import { getMediaProvider } from "../media/provider-loader.js";
@@ -52,6 +53,8 @@ interface Props extends Omit<
 	width?: number;
 	/** Override height (uses image.height by default) */
 	height?: number;
+	/** Output quality for Astro's image service or providers that support it. */
+	quality?: LocalImageProps["quality"];
 	/** Priority loading (disables lazy loading) */
 	priority?: boolean;
 	/** Render the LQIP placeholder as an inline background. Disable for strict CSP. */
@@ -64,6 +67,7 @@ const {
 	alt,
 	width,
 	height,
+	quality,
 	priority,
 	placeholder = true,
 	class: className,
@@ -105,7 +109,7 @@ function generateSrcset(
 	return responsiveWidths(maxWidth, originalWidth)
 		.map((w) => {
 			const h = aspectRatio ? Math.round(w / aspectRatio) : undefined;
-			return `${getSrc({ width: w, height: h })} ${w}w`;
+			return `${getSrc({ width: w, height: h, quality })} ${w}w`;
 		})
 		.join(", ");
 }
@@ -149,6 +153,7 @@ async function resolveImage(img: MediaValue): Promise<ResolvedImage> {
 				width: finalWidth,
 				height: finalHeight,
 				originalWidth: img.width,
+				quality,
 			});
 			if (optimized) {
 				src = optimized.src;
@@ -164,6 +169,7 @@ async function resolveImage(img: MediaValue): Promise<ResolvedImage> {
 				const result = provider.getEmbed(img, {
 					width: finalWidth,
 					height: finalHeight,
+					quality,
 				});
 				const embed = result instanceof Promise ? await result : result;
 				if (embed.type === "image") {
@@ -212,8 +218,12 @@ async function resolveImage(img: MediaValue): Promise<ResolvedImage> {
 	// The focal point travels with the media value, so the dark variant gets
 	// its own object-position rather than inheriting the light image's.
 	const objectPosition = focalPointToObjectPosition(img.focalX, img.focalY);
+	const normalizedPlaceholder =
+		placeholderStyle && !placeholderStyle.endsWith(";")
+			? `${placeholderStyle};`
+			: placeholderStyle;
 	const generatedStyle = [
-		placeholderStyle,
+		normalizedPlaceholder,
 		objectPosition ? `object-position: ${objectPosition};` : "",
 	]
 		.filter(Boolean)
@@ -277,6 +287,7 @@ const astroClass = (variantClass: string | undefined) =>
 			alt={finalAlt}
 			width={resolved.width}
 			height={resolved.height}
+			quality={quality}
 			priority={priority}
 			layout="constrained"
 			loading={priority ? "eager" : "lazy"}
@@ -297,6 +308,7 @@ const astroClass = (variantClass: string | undefined) =>
 			alt={finalAlt}
 			width={resolvedDark.width}
 			height={resolvedDark.height}
+			quality={quality}
 			priority={priority}
 			layout="constrained"
 			loading={priority ? "eager" : "lazy"}