Fix CI failure in Run CI Pipeline / 📊 Coverage Report (run 18681053444)

Author: jdalton

src/normalize.ts

@@ -134,7 +134,7 @@ function qualifiersToEntries(
   if (isObject(rawQualifiers)) {
     // URLSearchParams instances have an "entries" method that returns an iterator
     const rawQualifiersObj = rawQualifiers as QualifiersObject | URLSearchParams
-    const entriesProperty = (rawQualifiersObj as QualifiersObject).entries
+    const entriesProperty = (rawQualifiersObj as QualifiersObject)['entries']
     return typeof entriesProperty === 'function'
       ? (ReflectApply(entriesProperty, rawQualifiersObj, []) as Iterable<
           [string, string]

src/package-url.ts

@@ -99,47 +99,47 @@ class PackageURL {
     rawSubpath: unknown,
   ) {
     const type = isNonEmptyString(rawType)
-      ? (PurlComponent.type?.normalize as ComponentNormalizer)?.(rawType)
+      ? (PurlComponent['type']?.['normalize'] as ComponentNormalizer)?.(rawType)
       : rawType
-    ;(PurlComponent.type?.validate as ComponentValidator)?.(type, true)
+    ;(PurlComponent['type']?.['validate'] as ComponentValidator)?.(type, true)
 
     const namespace = isNonEmptyString(rawNamespace)
-      ? (PurlComponent.namespace?.normalize as ComponentNormalizer)?.(
+      ? (PurlComponent['namespace']?.['normalize'] as ComponentNormalizer)?.(
           rawNamespace,
         )
       : rawNamespace
-    ;(PurlComponent.namespace?.validate as ComponentValidator)?.(
+    ;(PurlComponent['namespace']?.['validate'] as ComponentValidator)?.(
       namespace,
       true,
     )
 
     const name = isNonEmptyString(rawName)
-      ? (PurlComponent.name?.normalize as ComponentNormalizer)?.(rawName)
+      ? (PurlComponent['name']?.['normalize'] as ComponentNormalizer)?.(rawName)
       : rawName
-    ;(PurlComponent.name?.validate as ComponentValidator)?.(name, true)
+    ;(PurlComponent['name']?.['validate'] as ComponentValidator)?.(name, true)
 
     const version = isNonEmptyString(rawVersion)
-      ? (PurlComponent.version?.normalize as ComponentNormalizer)?.(rawVersion)
+      ? (PurlComponent['version']?.['normalize'] as ComponentNormalizer)?.(rawVersion)
       : rawVersion
-    ;(PurlComponent.version?.validate as ComponentValidator)?.(version, true)
+    ;(PurlComponent['version']?.['validate'] as ComponentValidator)?.(version, true)
 
     const qualifiers =
       typeof rawQualifiers === 'string' || isObject(rawQualifiers)
         ? (
-            PurlComponent.qualifiers?.normalize as (
+            PurlComponent['qualifiers']?.['normalize'] as (
               _value: string | QualifiersObject,
             ) => Record<string, string> | undefined
           )?.(rawQualifiers as string | QualifiersObject)
         : rawQualifiers
-    ;(PurlComponent.qualifiers?.validate as ComponentValidator)?.(
+    ;(PurlComponent['qualifiers']?.['validate'] as ComponentValidator)?.(
       qualifiers,
       true,
     )
 
     const subpath = isNonEmptyString(rawSubpath)
-      ? (PurlComponent.subpath?.normalize as ComponentNormalizer)?.(rawSubpath)
+      ? (PurlComponent['subpath']?.['normalize'] as ComponentNormalizer)?.(rawSubpath)
       : rawSubpath
-    ;(PurlComponent.subpath?.validate as ComponentValidator)?.(subpath, true)
+    ;(PurlComponent['subpath']?.['validate'] as ComponentValidator)?.(subpath, true)
 
     this.type = type as string
     this.name = name as string
@@ -156,9 +156,9 @@ class PackageURL {
 
     const typeHelpers = PurlType[type as string]
     if (typeHelpers) {
-      ;(typeHelpers?.normalize as (_purl: PackageURL) => void)?.(this)
+      ;(typeHelpers?.['normalize'] as (_purl: PackageURL) => void)?.(this)
       ;(
-        typeHelpers?.validate as (
+        typeHelpers?.['validate'] as (
           _purl: PackageURL,
           _throws: boolean,
         ) => boolean
@@ -222,19 +222,19 @@ class PackageURL {
       type?: string | undefined
       version?: string | undefined
     } = this
-    /* c8 ignore next - Type encoder uses default PurlComponentEncoder, never returns null/undefined. */ let purlStr = `pkg:${(PurlComponent.type?.encode as ComponentEncoder)?.(type) ?? ''}/`
+    /* c8 ignore next - Type encoder uses default PurlComponentEncoder, never returns null/undefined. */ let purlStr = `pkg:${(PurlComponent['type']?.['encode'] as ComponentEncoder)?.(type) ?? ''}/`
     if (namespace) {
-      /* c8 ignore next - Namespace encoder always returns string, never null/undefined. */ purlStr = `${purlStr}${(PurlComponent.namespace?.encode as ComponentEncoder)?.(namespace) ?? ''}/`
+      /* c8 ignore next - Namespace encoder always returns string, never null/undefined. */ purlStr = `${purlStr}${(PurlComponent['namespace']?.['encode'] as ComponentEncoder)?.(namespace) ?? ''}/`
     }
-    /* c8 ignore next - Name encoder always returns string, never null/undefined. */ purlStr = `${purlStr}${(PurlComponent.name?.encode as ComponentEncoder)?.(name) ?? ''}`
+    /* c8 ignore next - Name encoder always returns string, never null/undefined. */ purlStr = `${purlStr}${(PurlComponent['name']?.['encode'] as ComponentEncoder)?.(name) ?? ''}`
     if (version) {
-      /* c8 ignore next - Version encoder always returns string, never null/undefined. */ purlStr = `${purlStr}@${(PurlComponent.version?.encode as ComponentEncoder)?.(version) ?? ''}`
+      /* c8 ignore next - Version encoder always returns string, never null/undefined. */ purlStr = `${purlStr}@${(PurlComponent['version']?.['encode'] as ComponentEncoder)?.(version) ?? ''}`
     }
     if (qualifiers) {
-      /* c8 ignore next - Qualifiers encoder always returns string, never null/undefined. */ purlStr = `${purlStr}?${(PurlComponent.qualifiers?.encode as ComponentEncoder)?.(qualifiers) ?? ''}`
+      /* c8 ignore next - Qualifiers encoder always returns string, never null/undefined. */ purlStr = `${purlStr}?${(PurlComponent['qualifiers']?.['encode'] as ComponentEncoder)?.(qualifiers) ?? ''}`
     }
     if (subpath) {
-      /* c8 ignore next - Subpath encoder always returns string, never null/undefined. */ purlStr = `${purlStr}#${(PurlComponent.subpath?.encode as ComponentEncoder)?.(subpath) ?? ''}`
+      /* c8 ignore next - Subpath encoder always returns string, never null/undefined. */ purlStr = `${purlStr}#${(PurlComponent['subpath']?.['encode'] as ComponentEncoder)?.(subpath) ?? ''}`
     }
     return purlStr
   }
@@ -278,12 +278,12 @@ class PackageURL {
     // Create a safe object without prototype chain to prevent prototype pollution
     const safeObject: PackageURLObject = {
       __proto__: null,
-      type: parsedRecord.type as string | undefined,
-      namespace: parsedRecord.namespace as string | undefined,
-      name: parsedRecord.name as string | undefined,
-      version: parsedRecord.version as string | undefined,
-      qualifiers: parsedRecord.qualifiers as Record<string, string> | undefined,
-      subpath: parsedRecord.subpath as string | undefined,
+      type: parsedRecord['type'] as string | undefined,
+      namespace: parsedRecord['namespace'] as string | undefined,
+      name: parsedRecord['name'] as string | undefined,
+      version: parsedRecord['version'] as string | undefined,
+      qualifiers: parsedRecord['qualifiers'] as Record<string, string> | undefined,
+      subpath: parsedRecord['subpath'] as string | undefined,
     } as PackageURLObject
 
     return PackageURL.fromObject(safeObject)
@@ -298,12 +298,12 @@ class PackageURL {
     }
     const typedObj = obj as Record<string, unknown>
     return new PackageURL(
-      typedObj.type,
-      typedObj.namespace,
-      typedObj.name,
-      typedObj.version,
-      typedObj.qualifiers,
-      typedObj.subpath,
+      typedObj['type'],
+      typedObj['namespace'],
+      typedObj['name'],
+      typedObj['version'],
+      typedObj['qualifiers'],
+      typedObj['subpath'],
     )
   }
 
@@ -461,13 +461,15 @@ class PackageURL {
       const entries = search.split('&')
       for (let i = 0, { length } = entries; i < length; i += 1) {
         const pairs = entries[i]?.split('=')
-        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,
-        )
+        if (pairs) {
+          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,
+          )
+        }
       }
       // Split the remainder once from right on '?'
       rawQualifiers = searchParams

src/purl-type.ts

@@ -224,7 +224,7 @@ const PurlType = createHelpersNamespaceObject(
       },
       // https://github.com/package-url/purl-spec/blob/master/PURL-TYPES.rst#mlflow
       mlflow(purl: PurlObject) {
-        if (purl.qualifiers?.repository_url?.includes('databricks')) {
+        if (purl.qualifiers?.['repository_url']?.includes('databricks')) {
           lowerName(purl)
         }
         return purl
@@ -306,7 +306,7 @@ const PurlType = createHelpersNamespaceObject(
       // https://github.com/package-url/purl-spec/blob/master/PURL-TYPES.rst#conan
       conan(purl: PurlObject, throws: boolean) {
         if (isNullishOrEmptyString(purl.namespace)) {
-          if (purl.qualifiers?.channel) {
+          if (purl.qualifiers?.['channel']) {
             if (throws) {
               throw new PurlError(
                 'conan requires a "namespace" component when a "channel" qualifier is present',
@@ -542,7 +542,7 @@ const PurlType = createHelpersNamespaceObject(
       swid(purl: PurlObject, throws: boolean) {
         const { qualifiers } = purl
         // SWID requires a tag_id qualifier
-        const tagId = qualifiers?.tag_id
+        const tagId = qualifiers?.['tag_id']
         if (!tagId) {
           if (throws) {
             throw new PurlError('swid requires a "tag_id" qualifier')

src/validate.ts

@@ -170,7 +170,7 @@ function validateQualifiers(
     return false
   }
   const qualifiersObj = qualifiers as QualifiersObject | URLSearchParams
-  const keysProperty = (qualifiersObj as QualifiersObject).keys
+  const keysProperty = (qualifiersObj as QualifiersObject)['keys']
   // type-coverage:ignore-next-line -- TypeScript correctly infers this type through the ternary and cast
   const keysIterable: Iterable<string> =
     // URLSearchParams instances have a "keys" method that returns an iterator