Fix error messages to follow standard: no periods at end

Author: jdalton

src/package-url.ts

@@ -254,27 +254,27 @@ class PackageURL {
    */
   static fromJSON(json: unknown): PackageURL {
     if (typeof json !== 'string') {
-      throw new Error('JSON string argument is required.')
+      throw new Error('JSON string argument is required')
     }
 
     // Size limit: 1MB to prevent memory exhaustion.
     const MAX_JSON_SIZE = 1024 * 1024
     if (json.length > MAX_JSON_SIZE) {
       throw new Error(
-        `JSON string exceeds maximum size limit of ${MAX_JSON_SIZE} bytes.`,
+        `JSON string exceeds maximum size limit of ${MAX_JSON_SIZE} bytes`,
       )
     }
 
     let parsed
     try {
       parsed = JSON.parse(json)
     } catch (e) {
-      throw new Error('Invalid JSON string.', { cause: e })
+      throw new Error('Invalid JSON string', { cause: e })
     }
 
     // Validate parsed result is an object.
     if (!parsed || typeof parsed !== 'object' || Array.isArray(parsed)) {
-      throw new Error('JSON must parse to an object.')
+      throw new Error('JSON must parse to an object')
     }
 
     // Create a safe object without prototype chain to prevent prototype pollution.
@@ -296,7 +296,7 @@ class PackageURL {
    */
   static fromObject(obj: unknown): PackageURL {
     if (!isObject(obj)) {
-      throw new Error('Object argument is required.')
+      throw new Error('Object argument is required')
     }
     const typedObj = obj as Record<string, unknown>
     return new PackageURL(

test/json-export.test.mts

@@ -243,16 +243,16 @@ describe('PackageURL JSON/dict export', () => {
     it('should validate input and throw appropriate errors', () => {
       // Test non-object inputs
       expect(() => PackageURL.fromObject('not an object')).toThrow(
-        'Object argument is required.',
+        'Object argument is required',
       )
       expect(() => PackageURL.fromObject(null)).toThrow(
-        'Object argument is required.',
+        'Object argument is required',
       )
       expect(() => PackageURL.fromObject(undefined)).toThrow(
-        'Object argument is required.',
+        'Object argument is required',
       )
       expect(() => PackageURL.fromObject(123)).toThrow(
-        'Object argument is required.',
+        'Object argument is required',
       )
 
       // Test validation errors
@@ -293,26 +293,26 @@ describe('PackageURL JSON/dict export', () => {
     it('should validate input and throw appropriate errors', () => {
       // Test non-string inputs
       expect(() => PackageURL.fromJSON(123)).toThrow(
-        'JSON string argument is required.',
+        'JSON string argument is required',
       )
       expect(() => PackageURL.fromJSON(null)).toThrow(
-        'JSON string argument is required.',
+        'JSON string argument is required',
       )
       expect(() => PackageURL.fromJSON(undefined)).toThrow(
-        'JSON string argument is required.',
+        'JSON string argument is required',
       )
       expect(() => PackageURL.fromJSON({})).toThrow(
-        'JSON string argument is required.',
+        'JSON string argument is required',
       )
 
       // Test invalid JSON
       expect(() => PackageURL.fromJSON('invalid json')).toThrow(
-        'Invalid JSON string.',
+        'Invalid JSON string',
       )
       expect(() => PackageURL.fromJSON('{"type":"npm","name"}')).toThrow(
-        'Invalid JSON string.',
+        'Invalid JSON string',
       )
-      expect(() => PackageURL.fromJSON('')).toThrow('Invalid JSON string.')
+      expect(() => PackageURL.fromJSON('')).toThrow('Invalid JSON string')
 
       // Test validation of created PackageURL
       expect(() =>