Assume NPM person can be string

hatch_nodejs_version/metadata_source.py

@@ -7,11 +7,11 @@
 import os.path
 import re
 import urllib.parse
-from typing import Any
+from typing import Any, Union
 
 from hatchling.metadata.plugin.interface import MetadataHookInterface
 
-AUTHOR_PATTERN = r"^([^<(]+?)?[ \t]*(?:<([^>(]+?)>)?[ \t]*(?:\(([^)]+?)\)|$)"
+AUTHOR_PATTERN = r"^(?P<name>[^<(]+?)?[ \t]*(?:<(?P<email>[^>(]+?)>)?[ \t]*(?:\((?P<url>[^)]+?)\)|$)"
 REPOSITORY_PATTERN = r"^(?:(gist|bitbucket|gitlab|github):)?(.*?)$"
 REPOSITORY_TABLE = {
     "gitlab": "https://gitlab.com",
@@ -137,19 +137,24 @@ def _parse_bugs(self, bugs: str | dict[str, str]) -> str | None:
         return bugs["url"]
 
     def _parse_person(self, person: dict[str, str]) -> dict[str, str]:
-        if {"url", "email"} & person.keys():
-            result = {"name": person["name"]}
-            if "email" in person:
-                result["email"] = person["email"]
+        result = {}
+        if isinstance(person, dict):
+            if {"url", "email"} & person.keys():
+                result["name"] = person["name"]
+                if "email" in person:
+                    result["email"] = person["email"]
+                return result
+            else:
+                author = person["name"]
         else:
-            match = re.match(AUTHOR_PATTERN, person["name"])
-            if match is None:
-                raise ValueError(f"Invalid author name: {person['name']}")
-            name, email, _ = match.groups()
-            result = {"name": name}
-            if email is not None:
-                result["email"] = email
-
+            author = person
+        match = re.match(AUTHOR_PATTERN, author)
+        if match is None:
+            raise ValueError(f"Invalid author name: {author}")
+        name, email, _ = match.groups()
+        result = {"name": name}
+        if email is not None:
+            result["email"] = email
         return result
 
     def _parse_repository(self, repository: str | dict[str, str]) -> str:

tests/test_metadata_config.py

@@ -2,6 +2,7 @@
 #
 # SPDX-License-Identifier: MIT
 import pytest
+import json
 
 from hatch_nodejs_version.metadata_source import NodeJSMetadataHook
 
@@ -165,3 +166,19 @@ def test_labels(self, project):
         assert urls["the-repository"] == "https://github.com/some/code.git"
         assert urls["the-bug-tracker"] == "https://www.send-help.com"
         assert urls["the-homepage"] == "https://where-the-heart-is.com"
+
+    def test_authors_accepted_as_strings(self, project):
+        original_package_content = json.loads(TRIVIAL_PACKAGE_CONTENTS)
+        updated_package_content = original_package_content.copy()
+        author_as_string = f"{original_package_content['author']['name']} " \
+                           f"<{original_package_content['author']['email']}>"
+        updated_package_content['author'] = author_as_string
+        (project / "pyproject.toml").write_text(TRIVIAL_PYPROJECT_CONTENTS)
+        (project / "package.json").write_text(json.dumps(updated_package_content))
+
+        config = {}
+        metadata = {}
+        metadata_source = NodeJSMetadataHook(project, config=config)
+        metadata_source.update(metadata)
+        assert metadata == TRIVIAL_EXPECTED_METADATA
+