fix: resolve CI failures in lint, build, typecheck, and tests

harlan-zw · harlan-zw · commit 47e85d35fcc1 · 2026-03-24T21:29:47.000+11:00
- Fix pnpm-workspace.yaml key sort order (eslint yaml/sort-keys)
- Remove tsconfig.json paths override that broke module-builder build
  and wiped .nuxt/tsconfig.json aliases (#app, #site-config, etc.)
- Inline createNitroRouteRuleMatcher (not in nuxtseo-shared/server),
  import withoutQuery from nuxtseo-shared/utils
- Exclude devtools/ from root typecheck (separate workspace package)
- Suppress excessive stack depth in dev-prerender.ts typed routes
diff --git a/pnpm-workspace.yaml b/pnpm-workspace.yaml
@@ -1,6 +1,23 @@
+catalogMode: prefer
+cleanupUnusedCatalogs: true
+shellEmulator: true
+
+trustPolicy: no-downgrade
+
+trustPolicyExclude:
+  - chokidar@4.0.3
+  - eslint-import-resolver-typescript@3.10.1
+  - undici-types@6.21.0
+  - semver@6.3.1
+  - koa@2.16.3
+  - semver@5.7.2
+  - '@nuxtjs/tailwindcss@7.0.0-beta.1'
+
 packages:
   - playground
   - devtools
+overrides:
+  vite: npm:vite@8.0.1
 catalog:
   '@antfu/eslint-config': ^7.7.3
   '@clack/prompts': ^1.1.0
@@ -90,13 +107,11 @@ catalog:
   vue-tsc: ^3.2.6
   wrangler: ^4.76.0
   yoga-wasm-web: ^0.3.3
-catalogMode: prefer
 catalogs:
   devtools:
     '@iconify-json/simple-icons': ^1.2.75
     json-editor-vue: ^0.18.1
     shiki: ^4.0.2
-cleanupUnusedCatalogs: true
 onlyBuiltDependencies:
   - '@parcel/watcher'
   - '@tailwindcss/oxide'
@@ -106,19 +121,4 @@ onlyBuiltDependencies:
   - sharp
   - unrs-resolver
   - vue-demi
-overrides:
-  vite: npm:vite@8.0.1
-shellEmulator: true
-
-trustPolicy: no-downgrade
-
-trustPolicyExclude:
-  - chokidar@4.0.3
-  - eslint-import-resolver-typescript@3.10.1
-  - undici-types@6.21.0
-  - semver@6.3.1
-  - koa@2.16.3
-  - semver@5.7.2
-  - '@nuxtjs/tailwindcss@7.0.0-beta.1'
-
 trustPolicyIgnoreAfter: 262800
diff --git a/src/runtime/server/og-image/bindings/font-assets/dev-prerender.ts b/src/runtime/server/og-image/bindings/font-assets/dev-prerender.ts
@@ -105,6 +105,7 @@ export async function resolve(event: H3Event, font: FontConfig): Promise<Buffer>
     }
   }
   const fullPath = withBase(path, app.baseURL)
+  // @ts-expect-error excessive stack depth from Nuxt typed routes
   const arrayBuffer = await event.$fetch(fullPath, {
     responseType: 'arrayBuffer',
   }) as ArrayBuffer
diff --git a/src/runtime/server/util/kit.ts b/src/runtime/server/util/kit.ts
@@ -1,8 +1,13 @@
 import type { H3Event } from 'h3'
+import type { NitroRouteRules } from 'nitropack'
 import type { NuxtIslandResponse } from 'nuxt/app'
+import { defu } from 'defu'
+import { useRuntimeConfig } from 'nitropack/runtime'
 import { hash } from 'ohash'
+import { createRouter as createRadixRouter, toRouteMatcher } from 'radix3'
+import { withoutBase, withoutTrailingSlash } from 'ufo'
 
-export { createNitroRouteRuleMatcher, withoutQuery } from 'nuxtseo-shared/server'
+export { withoutQuery } from 'nuxtseo-shared/utils'
 
 export function fetchIsland(e: H3Event, component: string, props: Record<string, any>): Promise<NuxtIslandResponse> {
   const hashId = hash([component, props]).replaceAll('_', '-')
@@ -12,3 +17,20 @@ export function fetchIsland(e: H3Event, component: string, props: Record<string,
     },
   })
 }
+
+export function createNitroRouteRuleMatcher(): ((path: string) => NitroRouteRules) {
+  const { nitro, app } = useRuntimeConfig()
+  const _routeRulesMatcher = toRouteMatcher(
+    createRadixRouter({
+      routes: Object.fromEntries(
+        Object.entries(nitro?.routeRules || {})
+          .map(([path, rules]) => [withoutTrailingSlash(path), rules]),
+      ),
+    }),
+  )
+  return (path: string) => {
+    return defu({}, ..._routeRulesMatcher.matchAll(
+      withoutBase(withoutTrailingSlash(path.split('?')[0]), app.baseURL),
+    ).reverse()) as NitroRouteRules
+  }
+}
diff --git a/tsconfig.json b/tsconfig.json
@@ -1,9 +1,4 @@
 {
   "extends": "./.nuxt/tsconfig.json",
-  "compilerOptions": {
-    "paths": {
-      "#nuxtseo-shared/*": ["./node_modules/nuxtseo-shared/dist/runtime/*"]
-    }
-  },
-  "exclude": ["test/**", "playground/**", "client/**"]
+  "exclude": ["test/**", "playground/**", "client/**", "devtools/**"]
 }

Original file line number	Diff line number	Diff line change
`@@ -105,6 +105,7 @@ export async function resolve(event: H3Event, font: FontConfig): Promise<Buffer>`
`105`	`105`	`}`
`106`	`106`	`}`
`107`	`107`	`const fullPath = withBase(path, app.baseURL)`
	`108`	`+ // @ts-expect-error excessive stack depth from Nuxt typed routes`
`108`	`109`	`const arrayBuffer = await event.$fetch(fullPath, {`
`109`	`110`	`responseType: 'arrayBuffer',`
`110`	`111`	`}) as ArrayBuffer`
Original file line number	Diff line number	Diff line change
`@@ -1,9 +1,4 @@`
`1`	`1`	`{`
`2`	`2`	`"extends": "./.nuxt/tsconfig.json",`
`3`		`- "compilerOptions": {`
`4`		`- "paths": {`
`5`		`- "#nuxtseo-shared/": ["./node_modules/nuxtseo-shared/dist/runtime/"]`
`6`		`- }`
`7`		`- },`
`8`		`- "exclude": ["test/", "playground/", "client/**"]`
	`3`	`+ "exclude": ["test/", "playground/", "client/", "devtools/"]`
`9`	`4`	`}`