fix: handle scoped packages named v

Author: trivikr

server/utils/parse-package-params.ts

@@ -6,9 +6,10 @@ export function parsePackageParams(segments: string[]): {
   rawPackageName: string
   rawVersion: string | undefined
 } {
-  const vIndex = segments.indexOf('v')
+  const packageSegmentCount = segments[0]?.startsWith('@') ? 2 : 1
+  const vIndex = packageSegmentCount
 
-  if (vIndex !== -1 && vIndex < segments.length - 1) {
+  if (segments[vIndex] === 'v' && vIndex < segments.length - 1) {
     return {
       rawPackageName: segments.slice(0, vIndex).join('/'),
       rawVersion: segments.slice(vIndex + 1).join('/'),

shared/utils/parse-package-param.ts

@@ -37,9 +37,10 @@ export interface ParsedPackageParams {
  */
 export function parsePackageParam(pkgParam: string): ParsedPackageParams {
   const segments = pkgParam.split('/')
-  const vIndex = segments.indexOf('v')
+  const packageSegmentCount = segments[0]?.startsWith('@') ? 2 : 1
+  const vIndex = packageSegmentCount
 
-  if (vIndex !== -1 && vIndex < segments.length - 1) {
+  if (segments[vIndex] === 'v' && vIndex < segments.length - 1) {
     return {
       packageName: segments.slice(0, vIndex).join('/'),
       version: segments[vIndex + 1],

test/unit/server/utils/parse-package-params.spec.ts

@@ -49,5 +49,14 @@ describe('parsePackageParams', () => {
         rawVersion: '1.0.0',
       })
     })
+
+    it('parses scoped package names whose package segment is literally v', () => {
+      const segments = ['@scope', 'v', 'v', '1.2.3']
+      const result = parsePackageParams(segments)
+      expect(result).toEqual({
+        rawPackageName: '@scope/v',
+        rawVersion: '1.2.3',
+      })
+    })
   })
 })

test/unit/shared/utils/parse-package-param.spec.ts

@@ -85,6 +85,15 @@ describe('parsePackageParam', () => {
         rest: [],
       })
     })
+
+    it('parses scoped package names whose package segment is literally v', () => {
+      const result = parsePackageParam('@scope/v/v/1.2.3/dist/index.js')
+      expect(result).toEqual({
+        packageName: '@scope/v',
+        version: '1.2.3',
+        rest: ['dist', 'index.js'],
+      })
+    })
   })
 
   describe('edge cases', () => {