chore: clean up PR scope, revert unrelated changes

Revert changes unrelated to OG image migration:
- CI NODE_OPTIONS memory bump
- site.description casing change
- nitro prerender failOnError workaround
- favicon.svg and logo-icon.svg branding changes
- a11y test cosmetic string template refactoring

Author: harlan-zw

.github/workflows/ci.yml

@@ -145,7 +145,6 @@ jobs:
         run: pnpm vp run build:test
         env:
           VALIDATE_HTML: true
-          NODE_OPTIONS: --max-old-space-size=6144
 
       - name: 🖥️ Test project (browser)
         run: pnpm vp run test:browser:prebuilt
@@ -172,8 +171,6 @@ jobs:
 
       - name: 🏗️ Build project
         run: pnpm vp run build:test
-        env:
-          NODE_OPTIONS: --max-old-space-size=6144
 
       - name: ♿ Accessibility audit (Lighthouse - ${{ matrix.mode }} mode)
         run: pnpm vp run test:a11y:prebuilt

nuxt.config.ts

@@ -87,7 +87,7 @@ export default defineNuxtConfig({
   site: {
     url: 'https://npmx.dev',
     name: 'npmx',
-    description: 'a fast, modern browser for the npm registry',
+    description: 'A fast, modern browser for the npm registry',
   },
 
   router: {
@@ -206,10 +206,6 @@ export default defineNuxtConfig({
   compatibilityDate: '2026-01-31',
 
   nitro: {
-    prerender: {
-      // OAuth plugin crashes in CI/test (no valid clientUri), matching main branch behavior
-      failOnError: !isTest,
-    },
     externals: {
       inline: [
         'shiki',

public/favicon.svg

@@ -207,23 +207,27 @@ describe('a11y component test coverage', () => {
 function buildMissingTestsMessage(missingTests: string[]): string {
   if (missingTests.length === 0) return ''
   return (
-    `Missing a11y tests for ${missingTests.length} component(s):\n${missingTests
-      .map(c => `  - ${c}`)
-      .join('\n')}\n\nTo fix: Add tests in test/nuxt/a11y.spec.ts or add to SKIPPED_COMPONENTS ` +
-    `in test/unit/a11y-component-coverage.spec.ts with justification.`
+    `Missing a11y tests for ${missingTests.length} component(s):\n` +
+    missingTests.map(c => `  - ${c}`).join('\n') +
+    '\n\nTo fix: Add tests in test/nuxt/a11y.spec.ts or add to SKIPPED_COMPONENTS ' +
+    'in test/unit/a11y-component-coverage.spec.ts with justification.'
   )
 }
 
 function buildObsoleteSkipsMessage(obsoleteSkips: string[]): string {
   if (obsoleteSkips.length === 0) return ''
-  return `Obsolete SKIPPED_COMPONENTS entries:\n${obsoleteSkips
-    .map(c => `  - ${c}`)
-    .join('\n')}\n\nThese components no longer exist. Remove them from SKIPPED_COMPONENTS.`
+  return (
+    `Obsolete SKIPPED_COMPONENTS entries:\n` +
+    obsoleteSkips.map(c => `  - ${c}`).join('\n') +
+    '\n\nThese components no longer exist. Remove them from SKIPPED_COMPONENTS.'
+  )
 }
 
 function buildUnnecessarySkipsMessage(unnecessarySkips: string[]): string {
   if (unnecessarySkips.length === 0) return ''
-  return `Unnecessary SKIPPED_COMPONENTS entries:\n${unnecessarySkips
-    .map(c => `  - ${c}`)
-    .join('\n')}\n\nThese components have tests now. Remove them from SKIPPED_COMPONENTS.`
+  return (
+    `Unnecessary SKIPPED_COMPONENTS entries:\n` +
+    unnecessarySkips.map(c => `  - ${c}`).join('\n') +
+    '\n\nThese components have tests now. Remove them from SKIPPED_COMPONENTS.'
+  )
 }