Fix ESLint comment position issue

Author: jdalton

CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## [1.1.3](https://github.com/SocketDev/socket-packageurl-js/releases/tag/v1.1.3) - 2025-09-29
+
+### Fixed
+- Fixed tsgo transpilation bug that produced incorrect `exports.encodeComponent = void 0;` output
+
 ## [1.1.2](https://github.com/SocketDev/socket-packageurl-js/releases/tag/v1.1.2) - 2025-09-27
 
 ### Changed

src/package-url.ts

@@ -324,7 +324,7 @@ class PackageURL {
         const afterColon = purlStr.slice(colonIndex + 1)
         const trimmedAfterColon = trimLeadingSlashes(afterColon)
         url = new URL(`${beforeColon}:${trimmedAfterColon}`)
-        /* c8 ignore next 4 -- V8 coverage sees multiple branch paths in ternary that can't all be tested. */maybeUrlWithAuth =
+        /* c8 ignore next 4 -- V8 coverage sees multiple branch paths in ternary that can't all be tested. */ maybeUrlWithAuth =
           afterColon.length === trimmedAfterColon.length
             ? url
             : new URL(purlStr)
@@ -335,7 +335,9 @@ class PackageURL {
       }
     }
     // The scheme is a constant with the value "pkg".
-    /* c8 ignore next -- Tested: colonIndex === -1 (url undefined) case, but V8 can't see both branches. */if (url?.protocol !== 'pkg:') {
+    /* c8 ignore next -- Tested: colonIndex === -1 (url undefined) case, but V8 can't see both branches. */ if (
+      url?.protocol !== 'pkg:'
+    ) {
       throw new PurlError('missing required "pkg" scheme component')
       /* c8 ignore next -- Unreachable code after throw. */
     }
@@ -361,12 +363,11 @@ class PackageURL {
     let rawVersion: string | undefined
     // Both branches of this ternary are tested, but V8 reports phantom branch combinations
     /* c8 ignore start -- npm vs non-npm path logic both tested but V8 sees extra branches. */
+    // Deviate from the specification to handle a special npm purl type case for
+    // pnpm ids such as 'pkg:npm/[email protected]([email protected]([email protected]))([email protected])'.
     let atSignIndex =
       rawType === 'npm'
-        // Deviate from the specification to handle a special npm purl type case for
-        ?
-          // pnpm ids such as 'pkg:npm/[email protected]([email protected]([email protected]))([email protected])'.
-          pathname.indexOf('@', firstSlashIndex + 2)
+        ? pathname.indexOf('@', firstSlashIndex + 2)
         : pathname.lastIndexOf('@')
     /* c8 ignore stop */
     // When a forward slash ('/') is directly preceding an '@' symbol,
@@ -418,7 +419,10 @@ class PackageURL {
         const value = decodePurlComponent('qualifiers', pairs.at(1) ?? '')
         // Use URLSearchParams#append to preserve plus signs.
         // https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams#preserving_plus_signs
-        /* c8 ignore next -- URLSearchParams.append has internal V8 branches we can't control. */searchParams.append(pairs[0]!, value)
+        /* c8 ignore next -- URLSearchParams.append has internal V8 branches we can't control. */ searchParams.append(
+          pairs[0]!,
+          value,
+        )
       }
       // Split the remainder once from right on '?'.
       rawQualifiers = searchParams

src/purl-type.ts

@@ -423,9 +423,11 @@ const PurlType = createHelpersNamespaceObject(
             }
             return false
           }
-          if (/[~'!()*]/.test(name)) {if (throws) {
+          if (/[~'!()*]/.test(name)) {
+            if (throws) {
               throw new PurlError(
-                `npm "name" component can not contain special characters ("~'!()*")`,)
+                `npm "name" component can not contain special characters ("~'!()*")`,
+              )
             }
             return false
           }

src/validate.ts

@@ -158,7 +158,8 @@ function validateStrings(
     return true
   }
   if (throws) {
-    throw new PurlError(`"'${name}" must be a string`)}
+    throw new PurlError(`"'${name}" must be a string`)
+  }
   return false
 }
 

test/package-url.test.mts

@@ -771,7 +771,8 @@ describe('PackageURL', () => {
         const isBuiltin = npmBuiltinNames.includes(id)
         const isMixedCased = /[A-Z]/.test(id)
         const containsIllegalCharacters = /[~'!()*]/.test(id)
-        expect(isBuiltin || isMixedCased || containsIllegalCharacters,
+        expect(
+          isBuiltin || isMixedCased || containsIllegalCharacters,
           `assert for ${legacyName}`,
         )
       }
@@ -1924,7 +1925,7 @@ describe('PackageURL', () => {
             name: 'パッケージ',
             expectedError:
               /npm "name" component can only contain URL-friendly characters/,
-          // Non-ASCII characters
+            // Non-ASCII characters
           },
         ]
 
@@ -2628,11 +2629,11 @@ describe('PackageURL', () => {
         )
         expect(formatPurlErrorMessage('[ message')).toBe(
           'Invalid purl: [ message',
-        // After Z
+          // After Z
         )
         expect(formatPurlErrorMessage('@ message')).toBe(
           'Invalid purl: @ message',
-        // Before A
+          // Before A
         )
 
         // Test normalize.js line 7 - namespace filter