fix: make AUTHOR_REGEX less restrictive (#517)

src/poetry/core/packages/package.py

@@ -1,6 +1,5 @@
 from __future__ import annotations
 
-import re
 import warnings
 
 from typing import TYPE_CHECKING
@@ -13,6 +12,7 @@
 from poetry.core.constraints.version.exceptions import ParseConstraintError
 from poetry.core.packages.dependency_group import MAIN_GROUP
 from poetry.core.packages.specification import PackageSpecification
+from poetry.core.utils.patterns import AUTHOR_REGEX
 from poetry.core.version.exceptions import InvalidVersionError
 
 
@@ -32,10 +32,6 @@
 
     T = TypeVar("T", bound="Package")
 
-AUTHOR_REGEX = re.compile(
-    r"(?u)^(?P<name>[- .,\w\d'’\"():&]+)(?: <(?P<email>.+?)>)?$"  # noqa: RUF001
-)
-
 
 class Package(PackageSpecification):
     AVAILABLE_PYTHONS: ClassVar[set[str]] = {

src/poetry/core/utils/patterns.py

@@ -3,6 +3,8 @@
 import re
 
 
+AUTHOR_REGEX = re.compile(r"(?u)^(?P<name>[^<>]+)(?: <(?P<email>.+?)>)?$")
+
 wheel_file_re = re.compile(
     r"""^(?P<namever>(?P<name>.+?)(-(?P<ver>\d.+?))?)
         ((-(?P<build>\d.*?))?-(?P<pyver>.+?)-(?P<abi>.+?)-(?P<plat>.+?)

tests/packages/test_package.py

@@ -83,6 +83,8 @@ def test_package_authors_invalid() -> None:
         ("John-Paul: Doe", None),
         ("John-Paul: Doe", "[email protected]"),
         ("John Doe the 3rd", "[email protected]"),
+        ("FirstName LastName [email protected]", None),
+        ("Surname, Given Name [Department]", None),
     ],
 )
 def test_package_authors_valid(name: str, email: str | None) -> None:
@@ -98,12 +100,9 @@ def test_package_authors_valid(name: str, email: str | None) -> None:
     ("name",),
     [
         ("<[email protected]>",),
-        ("[email protected]",),
+        ("<[email protected]",),
+        ("[email protected]>",),
         ("<John Doe",),
-        ("John? Doe",),
-        ("Jane+Doe",),
-        ("~John Doe",),
-        ("John~Doe",),
     ],
 )
 def test_package_author_names_invalid(name: str) -> None: