chore: apply lint fixes

danielroe · TheAlexLichter · danielroe · commit 221ce99ebde8 · 2025-06-03T10:12:53.000+01:00
Co-authored-by: Alexander Lichter &lt;github@lichter.io&gt;
diff --git a/packages/nuxt/src/app/components/route-provider.ts b/packages/nuxt/src/app/components/route-provider.ts
@@ -36,7 +36,7 @@ export const defineRouteProvider = (name = 'RouteProvider') => defineComponent({
       onMounted(() => {
         nextTick(() => {
           if (['#comment', '#text'].includes(vnode?.el?.nodeName)) {
-            const filename = (vnode?.type as any).__file
+            const filename = (vnode?.type as any)?.__file
             console.warn(`[nuxt] \`${filename}\` does not have a single root node and will cause errors when navigating between routes.`)
           }
         })
diff --git a/packages/nuxt/src/app/entry.ts b/packages/nuxt/src/app/entry.ts
@@ -49,7 +49,7 @@ if (import.meta.client) {
     import.meta.webpackHot.accept()
   }
 
-  // eslint-disable-next-line
+  // eslint-disable-next-line prefer-const
   let vueAppPromise: Promise<App<Element>>
 
   entry = async function initApp () {
diff --git a/packages/nuxt/src/pages/module.ts b/packages/nuxt/src/pages/module.ts
@@ -200,7 +200,7 @@ export default defineNuxtModule({
             // way to add a route without a file, which must be possible
             const route = addedPagePaths.has(absolutePagePath)
               ? parent
-              : /^\//.test(page.path)
+              : page.path[0] === '/'
                 // @ts-expect-error TODO: either fix types upstream or figure out another
                 // way to add a route without a file, which must be possible
                 ? rootPage.insert(page.path, page.file)
diff --git a/packages/nuxt/test/nitro/render-index.ts b/packages/nuxt/test/nitro/render-index.ts
@@ -7,7 +7,7 @@ const nitroApp = useNitroApp()
 
 async function renderIndex () {
   const text = await (nitroApp as NitroApp).localFetch('/', {}).then(r => r.text())
-  // eslint-disable-next-line
+  // eslint-disable-next-line no-console
   console.log(text)
 }
 
diff --git a/packages/vite/src/runtime/vite-node.mjs b/packages/vite/src/runtime/vite-node.mjs
@@ -11,7 +11,7 @@ const runner = createRunner()
 /** @type {(ssrContext: import('#app').NuxtSSRContext) => Promise<any>} */
 let render
 
-/** @param ssrContext {import('#app').NuxtSSRContext} */
+/** @param {import('#app').NuxtSSRContext} ssrContext */
 export default async (ssrContext) => {
   // Workaround for stub mode
   // https://github.com/nuxt/framework/pull/3983
@@ -74,16 +74,16 @@ function createRunner () {
 }
 
 /**
- * @param errorData {any}
- * @param id {string}
+ * @param {any} errorData
+ * @param {string} id
  */
 function formatViteError (errorData, id) {
   const errorCode = errorData.name || errorData.reasonCode || errorData.code
   const frame = errorData.frame || errorData.source || errorData.pluginCode
 
-  /** @param locObj {{ file?: string, id?: string, url?: string }} */
+  /** @param {{ file?: string, id?: string, url?: string }} locObj */
   const getLocId = (locObj = {}) => locObj.file || locObj.id || locObj.url || id || ''
-  /** @param locObj {{ line?: string, column?: string }} */
+  /** @param {{ line?: string, column?: string }} locObj */
   const getLocPos = (locObj = {}) => locObj.line ? `${locObj.line}:${locObj.column || 0}` : ''
   const locId = getLocId(errorData.loc) || getLocId(errorData.location) || getLocId(errorData.input) || getLocId(errorData)
   const locPos = getLocPos(errorData.loc) || getLocPos(errorData.location) || getLocPos(errorData.input) || getLocPos(errorData)
diff --git a/test/fixtures/basic-types/types.ts b/test/fixtures/basic-types/types.ts
@@ -175,7 +175,7 @@ describe('middleware', () => {
     definePageMeta({
       validate: async () => {
         await new Promise(resolve => setTimeout(resolve, 1000))
-        // eslint-disable-next-line
+        // eslint-disable-next-line no-constant-condition
         if (0) {
           return createError({
             statusCode: 404,

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ export const defineRouteProvider = (name = 'RouteProvider') => defineComponent({`
`36`	`36`	`onMounted(() => {`
`37`	`37`	`nextTick(() => {`
`38`	`38`	`if (['#comment', '#text'].includes(vnode?.el?.nodeName)) {`
`39`		`- const filename = (vnode?.type as any).__file`
	`39`	`+ const filename = (vnode?.type as any)?.__file`
`40`	`40`	console.warn(`[nuxt] \`${filename}\` does not have a single root node and will cause errors when navigating between routes.`)
`41`	`41`	`}`
`42`	`42`	`})`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ if (import.meta.client) {`
`49`	`49`	`import.meta.webpackHot.accept()`
`50`	`50`	`}`
`51`	`51`
`52`		`- // eslint-disable-next-line`
	`52`	`+ // eslint-disable-next-line prefer-const`
`53`	`53`	`let vueAppPromise: Promise<App<Element>>`
`54`	`54`
`55`	`55`	`entry = async function initApp () {`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ const nitroApp = useNitroApp()`
`7`	`7`
`8`	`8`	`async function renderIndex () {`
`9`	`9`	`const text = await (nitroApp as NitroApp).localFetch('/', {}).then(r => r.text())`
`10`		`- // eslint-disable-next-line`
	`10`	`+ // eslint-disable-next-line no-console`
`11`	`11`	`console.log(text)`
`12`	`12`	`}`
`13`	`13`