fix(i18n): respect inline route rules

harlan-zw · harlan-zw · commit 8fea805f0f04 · 2025-04-16T18:57:52.000+10:00
Fixes harlan-zw/nuxt-seo#404
diff --git a/src/runtime/server/composables/getPathRobotConfig.ts b/src/runtime/server/composables/getPathRobotConfig.ts
@@ -8,8 +8,9 @@ import { createNitroRouteRuleMatcher } from '../kit'
 import { getSiteRobotConfig } from './getSiteRobotConfig'
 
 export function getPathRobotConfig(e: H3Event, options?: { userAgent?: string, skipSiteIndexable?: boolean, path?: string }): RobotsContext {
+  const runtimeConfig = useRuntimeConfig(e)
   // has already been resolved
-  const { robotsDisabledValue, robotsEnabledValue, isNuxtContentV2 } = useRuntimeConfig()['nuxt-robots']
+  const { robotsDisabledValue, robotsEnabledValue, isNuxtContentV2 } = runtimeConfig['nuxt-robots']
   if (!options?.skipSiteIndexable) {
     if (!getSiteRobotConfig(e).indexable) {
       return {
@@ -85,7 +86,16 @@ export function getPathRobotConfig(e: H3Event, options?: { userAgent?: string, s
 
   // 3. nitro route rules
   nitroApp._robotsRuleMactcher = nitroApp._robotsRuleMactcher || createNitroRouteRuleMatcher()
-  const routeRules = normaliseRobotsRouteRule(nitroApp._robotsRuleMactcher(path))
+  let routeRulesPath = path
+  // if we're using i18n we need to strip leading prefixes so the rule will match
+  if (runtimeConfig.public?.i18n?.locales) {
+    const { locales } = runtimeConfig.public.i18n
+    const locale = locales.find(l => routeRulesPath.startsWith(`/${l.code}`))
+    if (locale) {
+      routeRulesPath = routeRulesPath.replace(`/${locale.code}`, '')
+    }
+  }
+  const routeRules = normaliseRobotsRouteRule(nitroApp._robotsRuleMactcher(routeRulesPath))
   if (routeRules && (typeof routeRules.allow !== 'undefined' || typeof routeRules.rule !== 'undefined')) {
     return {
       indexable: routeRules.allow,
diff --git a/test/fixtures/i18n/nuxt.config.ts b/test/fixtures/i18n/nuxt.config.ts
@@ -37,5 +37,9 @@ export default defineNuxtConfig({
     ],
   },
 
+  experimental: {
+    inlineRouteRules: true,
+  },
+
   compatibilityDate: '2025-03-13',
 })
diff --git a/test/fixtures/i18n/pages/route-rules.vue b/test/fixtures/i18n/pages/route-rules.vue
@@ -0,0 +1,11 @@
+<script lang="ts" setup>
+import { defineRouteRules } from '#imports'
+
+defineRouteRules({
+  robots: false,
+})
+</script>
+
+<template>
+  <div>route rules</div>
+</template>
diff --git a/test/i18n.test.ts b/test/i18n.test.ts
@@ -37,4 +37,7 @@ describe('i18n', () => {
       # END nuxt-robots"
     `)
   })
+  it('respects route rules', async () => {
+    expect((await $fetch('/en/route-rules/?mockProductionEnv=true')).match(/<meta name="robots" content="([^"]+)">/)[1]).toMatchInlineSnapshot(`"noindex, nofollow"`)
+  })
 })
