fix parsing bug for .npmrc files with comments

Author: odanado

packages/nuxi/package.json

@@ -48,6 +48,7 @@
     "chokidar": "^4.0.3",
     "citty": "^0.1.6",
     "clipboardy": "^4.0.0",
+    "confbox": "^0.2.2",
     "consola": "^3.4.2",
     "defu": "^6.1.4",
     "fuse.js": "^7.1.0",

packages/nuxi/src/commands/module/_utils.ts

@@ -1,3 +1,4 @@
+import { parseINI } from 'confbox'
 import { $fetch } from 'ofetch'
 import { readPackageJSON } from 'pkg-types'
 import { coerce, satisfies } from 'semver'
@@ -126,20 +127,23 @@ export async function getNuxtVersion(cwd: string) {
 }
 
 export function getRegistryFromContent(content: string, scope: string | null) {
-  if (scope) {
-    const scopedRegex = new RegExp(`^${scope}:registry=(.+)$`, 'm')
-    const scopedMatch = content.match(scopedRegex)?.[1]
-    if (scopedMatch) {
-      return scopedMatch.trim()
+  try {
+    const npmConfig = parseINI<Record<string, string | undefined>>(content)
+
+    if (scope) {
+      const scopeKey = `${scope}:registry`
+      if (npmConfig[scopeKey]) {
+        return npmConfig[scopeKey].trim()
+      }
     }
-  }
 
-  // If no scoped registry found or no scope provided, look for the default registry
-  const defaultRegex = /^\s*registry=(.+)$/m
-  const defaultMatch = content.match(defaultRegex)?.[1]
-  if (defaultMatch) {
-    return defaultMatch.trim()
-  }
+    if (npmConfig.registry) {
+      return npmConfig.registry.trim()
+    }
 
-  return null
+    return null
+  }
+  catch {
+    return null
+  }
 }

packages/nuxi/test/unit/commands/module/_utils.test.ts

@@ -45,4 +45,14 @@ some-other-setting=value
     expect(getRegistryFromContent('', null)).toBeNull()
     expect(getRegistryFromContent('', '@myorg')).toBeNull()
   })
+
+  it('extracts registry from line with comments', () => {
+    const content = `
+registry=https://registry.npmjs.org/ # with comment
+@myorg:registry=https://my-registry.org/ # another comment
+    `
+
+    expect(getRegistryFromContent(content, null)).toBe('https://registry.npmjs.org/')
+    expect(getRegistryFromContent(content, '@myorg')).toBe('https://my-registry.org/')
+  })
 })