From b204e19b81739ac8ddce980d396abcb9c0eb19ef Mon Sep 17 00:00:00 2001
From: Anshul Mittal <anshulmittal0203@gmail.com>
Date: Wed, 13 May 2026 15:23:41 +0530
Subject: [PATCH 1/3] WIP: Multiple authors implementation and tests

---
 CHANGES.rst                                  |  7 +-
 docs/api.rst                                 |  3 +
 docs/guide.rst                               |  5 +-
 src/reader/__init__.py                       |  1 +
 src/reader/_app/legacy/templates/entry.html  | 10 ++-
 src/reader/_app/templates/macros.html        |  4 +-
 src/reader/_parser/feedparser.py             | 72 +++++++++++++++++++-
 src/reader/_parser/jsonfeed.py               | 49 ++++++-------
 src/reader/_plugins/legacy/enclosure_tags.py |  2 +-
 src/reader/_storage/_entries.py              | 23 ++++++-
 src/reader/_storage/_feeds.py                | 11 ++-
 src/reader/_storage/_schema.py               | 39 ++++++++++-
 src/reader/_types.py                         | 32 +++++++--
 src/reader/types.py                          | 68 +++++++++++++++---
 tests/data/empty.atom.py                     |  1 +
 tests/data/full.atom.py                      |  7 +-
 tests/data/full.json.py                      |  7 +-
 tests/data/full.rss.py                       |  5 +-
 tests/data/invalid.json.py                   |  5 +-
 tests/test__types.py                         |  6 +-
 tests/test_parser.py                         | 70 +++++++++++++++++++
 tests/test_reader.py                         | 23 ++++---
 tests/test_reader_search.py                  |  5 +-
 tests/test_storage.py                        | 30 ++++++++
 tests/test_types.py                          | 34 +++++++++
 25 files changed, 439 insertions(+), 80 deletions(-)

diff --git a/CHANGES.rst b/CHANGES.rst
index 8fb228a5..4c8f4077 100644
--- a/CHANGES.rst
+++ b/CHANGES.rst
@@ -14,9 +14,14 @@ Unreleased
 * Add ``--json`` option to the ``list`` CLI commands.
   Thanks to `Puneet Dixit`_ for the PR.
   (:issue:`394`)
-
+* Add :attr:`~Entry.authors` (and corresponding attributes on feeds and sources) 
+  to expose multiple authors and rich author data (name, email, URL). 
+  The old ``author`` string attribute is deprecated.
+  Thanks to `Anshul Mittal`_ for the PR.
+  (:issue:`391`)
 
 .. _Puneet Dixit: https://github.com/puneetdixit200
+.. _Anshul Mittal: https://github.com/anderson688
 
 
 Version 3.23
diff --git a/docs/api.rst b/docs/api.rst
index 2e8b295a..7f5e14ba 100644
--- a/docs/api.rst
+++ b/docs/api.rst
@@ -28,6 +28,9 @@ Data objects
 .. autoclass:: Feed
     :members:
 
+.. autoclass:: Author
+    :members:
+
 .. autoclass:: ExceptionInfo
     :members:
 
diff --git a/docs/guide.rst b/docs/guide.rst
index 097a78bb..c1163302 100644
--- a/docs/guide.rst
+++ b/docs/guide.rst
@@ -374,7 +374,7 @@ As seen in the previous sections,
         updated=datetime.datetime(2020, 2, 28, 9, 34, 2, tzinfo=datetime.timezone.utc),
         title='Hello Internet',
         link='http://www.hellointernet.fm/',
-        author='CGP Grey',
+        authors=(Author(name='CGP Grey'),),
         subtitle='CGP Grey and Brady Haran talk about YouTube, life, work, whatever.',
         version='rss20',
         user_title=None,
@@ -386,9 +386,10 @@ As seen in the previous sections,
 To get all the feeds, use the :meth:`~Reader.get_feeds` method::
 
     >>> for feed in reader.get_feeds():
+    ...     authors = ", ".join(a.name for a in feed.authors if a.name) or 'unknown author'
     ...     print(
     ...         feed.title or feed.url,
-    ...         f"by {feed.author or 'unknown author'},",
+    ...         f"by {authors},",
     ...         f"updated on {feed.updated or 'never'}",
     ...     )
     ...
diff --git a/src/reader/__init__.py b/src/reader/__init__.py
index 51e718ec..1fe2dafc 100644
--- a/src/reader/__init__.py
+++ b/src/reader/__init__.py
@@ -67,6 +67,7 @@
     FeedToImport as FeedToImport,
     FeedImportResult as FeedImportResult,
     FeedExport as FeedExport,
+    Author as Author,
 )
 
 from .exceptions import (
diff --git a/src/reader/_app/legacy/templates/entry.html b/src/reader/_app/legacy/templates/entry.html
index 46df1694..7295aeda 100644
--- a/src/reader/_app/legacy/templates/entry.html
+++ b/src/reader/_app/legacy/templates/entry.html
@@ -18,7 +18,15 @@
 <ul class="controls">
 
 <li>
-    {% if entry.author %} by {{ entry.author }}{% endif %}
+    {% if entry.authors %} 
+        by 
+        {% for author in entry.authors -%}
+            {% if author.href %}<a href="{{ author.href }}" title="{{ author.email or '' }}">{% endif %}
+            {{- author.name or author.email or 'unknown' -}}
+            {% if author.href %}</a>{% endif %}
+            {%- if not loop.last %}, {% endif %}
+        {%- endfor %}
+    {% endif %}
     in <a href="{{ url_for('.entries', feed=entry.feed.url) }}">{{ entry.feed_resolved_title or feed.url }}</a>
 <li>
     {%- set published = entry.published or entry.updated_not_none -%}
diff --git a/src/reader/_app/templates/macros.html b/src/reader/_app/templates/macros.html
index deb2bd82..99f439cd 100644
--- a/src/reader/_app/templates/macros.html
+++ b/src/reader/_app/templates/macros.html
@@ -202,7 +202,7 @@
 
 
 {% macro feed_author(feed) %}
-{%- set author = feed.author if feed.author != feed.resolved_title else none -%}
+{%- set author = feed.author_str if feed.author_str != feed.resolved_title else none -%}
 {%- if author %}
 <li class="list-inline-item">
   by {{ author }}
@@ -235,7 +235,7 @@
 
 
 {% macro entry_author(entry) %}
-{%- set author = entry.author or entry.feed.author -%}
+{%- set author = entry.author_str or entry.feed.author_str -%}
 {%- set author = author if author != entry.feed_resolved_title else none -%}
 {%- if author %}
 <li class="list-inline-item">
diff --git a/src/reader/_parser/feedparser.py b/src/reader/_parser/feedparser.py
index a1c859be..250ca079 100644
--- a/src/reader/_parser/feedparser.py
+++ b/src/reader/_parser/feedparser.py
@@ -17,6 +17,7 @@
 from ..types import Content
 from ..types import Enclosure
 from ..types import EntrySource
+from ..types import Author
 from ._http_utils import parse_accept_header
 from ._http_utils import unparse_accept_header
 
@@ -91,7 +92,7 @@ def _process_feed(url: str, d: Any) -> tuple[FeedData, list[EntryData]]:
         _get_datetime_attr(d.feed, 'updated_parsed'),
         d.feed.get('title'),
         d.feed.get('link'),
-        d.feed.get('author'),
+        _parse_authors(d.feed, is_rss),
         d.feed.get('subtitle'),
         d.version,
     )
@@ -180,7 +181,7 @@ def _process_entry(feed_url: str, entry: Any, is_rss: bool) -> EntryData:
                 _get_datetime_attr(data, 'updated_parsed'),
                 source_title,
                 data.get('link'),
-                data.get('author'),
+                _parse_authors(data, is_rss),
                 data.get('subtitle'),
             )
 
@@ -190,10 +191,75 @@ def _process_entry(feed_url: str, entry: Any, is_rss: bool) -> EntryData:
         _get_datetime_attr(entry, 'updated_parsed'),
         entry.get('title'),
         entry.get('link'),
-        entry.get('author'),
+        _parse_authors(entry, is_rss),
         _get_datetime_attr(entry, 'published_parsed'),
         entry.get('summary'),
         tuple(content),
         tuple(enclosures),
         source,
     )
+
+
+def _parse_authors(thing: Any, is_rss: bool) -> tuple[Author, ...]:
+    author_detail = thing.get('author_detail') or {}
+    
+    # 1. Atom
+    if not is_rss and author_detail:
+        name = author_detail.get('name')
+        email = author_detail.get('email')
+        href = author_detail.get('href')
+        if name or email or href:
+            return (Author(
+                name=name or None, 
+                email=email or None, 
+                href=href or None
+            ),)
+        return ()
+        
+    # 2. RSS
+    raw_author = thing.get('author', '')
+    
+    # Fallback to author_detail if string is entirely empty or "()"
+    if not raw_author or raw_author == '()':
+        name = author_detail.get('name')
+        email = author_detail.get('email')
+        href = author_detail.get('href')
+        if name or email or href:
+            return (Author(
+                name=name or None, 
+                email=email or None, 
+                href=href or None
+            ),)
+        return ()
+        
+    # Split by comma
+    authors = []
+    found_any_email = False
+    
+    for part in raw_author.split(','):
+        part = part.strip()
+        if not part: continue
+        
+        name, email = part, None
+        if part.endswith(')') and '(' in part:
+            split_idx = part.rfind('(')
+            name = part[:split_idx].strip()
+            email = part[split_idx+1:-1].strip()
+            if email:
+                found_any_email = True
+                
+        if name or email:
+            authors.append({'name': name or None, 'email': email or None})
+            
+    # Fallback from author_detail if no emails were found in the string
+    if authors:
+        fallback_email = author_detail.get('email')
+        fallback_href = author_detail.get('href')
+        
+        for a in authors:
+            if fallback_email and not a['email']:
+                a['email'] = fallback_email
+            if fallback_href and not a.get('href'):
+                a['href'] = fallback_href
+            
+    return tuple(Author(**a) for a in authors)
\ No newline at end of file
diff --git a/src/reader/_parser/jsonfeed.py b/src/reader/_parser/jsonfeed.py
index 1f2db285..f31bf047 100644
--- a/src/reader/_parser/jsonfeed.py
+++ b/src/reader/_parser/jsonfeed.py
@@ -17,6 +17,7 @@
 from ..exceptions import ParseError
 from ..types import Content
 from ..types import Enclosure
+from ..types import Author
 
 if TYPE_CHECKING:  # pragma: no cover
     from . import FeedAndEntries
@@ -61,7 +62,7 @@ def _process_feed(url: str, d: Any) -> FeedAndEntries:
         updated=None,
         title=_get(d, 'title', str),
         link=_get(d, 'home_page_url', str),
-        author=_get_author(d),
+        authors=_get_authors(d),
         subtitle=_get(d, 'description', str),
         version=version_code,
     )
@@ -93,33 +94,27 @@ def _get(
     return cast(Union[_T, _U, _V], value)
 
 
-def _get_author(d: Any) -> str | None:
-    # from the spec:
-    #
-    # > JSON Feed version 1 specified a singular author field
-    # > instead of the authors array used in version 1.1.
-    # > New feeds should use authors, even if only 1 author is needed.
-    # > Existing feeds can include both author and authors
-    # > for compatibility with existing feed readers.
-    # > Feed readers should always prefer authors if present.
-
-    author: dict[Any, Any] | None
+def _get_authors(d: Any) -> tuple[Author, ...]:
+    authors = []
+    
+    # Check for authors array first (JSON Feed 1.1)
     for maybe_author in _get(d, 'authors', list) or ():
         if isinstance(maybe_author, dict):
-            author = maybe_author
-            break
-    else:
-        author = _get(d, 'author', dict)
-
-    if not author:
-        return None
-
-    # we only have one for now, it'll be the first one
-    return (
-        _get(author, 'name', str)
-        # fall back to the URL, at least until we have Feed.authors
-        or _get(author, 'url', str)
-    )
+            name = _get(maybe_author, 'name', str)
+            url = _get(maybe_author, 'url', str)
+            if name or url:
+                authors.append(Author(name=name, href=url))
+                
+    # Fallback to single author (JSON Feed 1.0)
+    if not authors:
+        single_author = _get(d, 'author', dict)
+        if single_author:
+            name = _get(single_author, 'name', str)
+            url = _get(single_author, 'url', str)
+            if name or url:
+                authors.append(Author(name=name, href=url))
+                
+    return tuple(authors)
 
 
 def _process_entry(feed_url: str, d: Any, feed_lang: str | None) -> EntryData:
@@ -174,7 +169,7 @@ def _process_entry(feed_url: str, d: Any, feed_lang: str | None) -> EntryData:
         updated=updated,
         title=_get(d, 'title', str),
         link=_get(d, 'url', str),
-        author=_get_author(d),
+        authors=_get_authors(d),
         published=published,
         summary=_get(d, 'summary', str),
         content=tuple(content),
diff --git a/src/reader/_plugins/legacy/enclosure_tags.py b/src/reader/_plugins/legacy/enclosure_tags.py
index 19e8f529..c424a902 100644
--- a/src/reader/_plugins/legacy/enclosure_tags.py
+++ b/src/reader/_plugins/legacy/enclosure_tags.py
@@ -151,7 +151,7 @@ def enclosure_tags_filter(enclosure, entry, feed_tags):
         album = striptags(album)
         args['album'] = album
         args['artist'] = album
-    elif artist := (entry.author or entry.feed.author):
+    elif artist := (entry.author_str or entry.feed.author_str):
         args['artist'] = striptags(artist)
 
     for tag in feed_tags:
diff --git a/src/reader/_storage/_entries.py b/src/reader/_storage/_entries.py
index 50c048ef..bd59504c 100644
--- a/src/reader/_storage/_entries.py
+++ b/src/reader/_storage/_entries.py
@@ -30,6 +30,7 @@
 from ..types import EntryCounts
 from ..types import EntrySort
 from ..types import EntrySource
+from ..types import Author
 from ._base import wrap_exceptions
 from ._feeds import feed_factory
 from ._sql_utils import Query
@@ -485,11 +486,19 @@ def entry_factory(row: tuple[Any, ...]) -> Entry:
         sequence,
     ) = row[14:33]
 
+    # Parse main entry authors
+    authors = tuple(Author(**d) for d in json.loads(author)) if author else ()
+
     source_obj = None
     if source:
         source_dict = json.loads(source)
         if source_dict['updated']:
             source_dict['updated'] = convert_timestamp(source_dict['updated'])
+        
+        # Parse source feed authors
+        source_author_json = source_dict.pop('author', None)
+        source_dict['authors'] = tuple(Author(**d) for d in json.loads(source_author_json)) if source_author_json else ()
+
         source_obj = EntrySource(**source_dict)
 
     return Entry(
@@ -497,7 +506,7 @@ def entry_factory(row: tuple[Any, ...]) -> Entry:
         convert_timestamp(updated) if updated else None,
         title,
         link,
-        author,
+        authors,
         convert_timestamp(published) if published else None,
         summary,
         tuple(Content(**d) for d in json.loads(content)) if content else (),
@@ -679,6 +688,10 @@ def entry_update_intent_to_dict(intent: EntryUpdateIntent) -> EntryDict:
             if entry.enclosures
             else None
         ),
+        # Serialize the entry authors
+        authors=(
+            json.dumps([a._asdict() for a in entry.authors]) if entry.authors else None
+        ),
         updated=adapt_datetime(entry.updated) if entry.updated else None,
         published=adapt_datetime(entry.published) if entry.published else None,
         last_updated=adapt_datetime(intent.last_updated),
@@ -700,7 +713,15 @@ def entry_update_intent_to_dict(intent: EntryUpdateIntent) -> EntryDict:
         source_dict = entry.source._asdict()
         if entry.source.updated:
             source_dict['updated'] = adapt_datetime(entry.source.updated)
+        
+        # Serialize the source authors and rename key to 'author'
+        source_authors = source_dict.pop('authors', ())
+        source_dict['author'] = json.dumps([a._asdict() for a in source_authors]) if source_authors else None
+
         context['source'] = json.dumps(source_dict)
+    
+    # Rename the context key from 'authors' to 'author' to match SQLite column
+    context['author'] = context.pop('authors', None)
 
     context['feed'] = context.pop('feed_url')
 
diff --git a/src/reader/_storage/_feeds.py b/src/reader/_storage/_feeds.py
index 548952ef..279c7d0e 100644
--- a/src/reader/_storage/_feeds.py
+++ b/src/reader/_storage/_feeds.py
@@ -15,6 +15,7 @@
 from .._types import FeedForUpdate
 from .._types import FeedToUpdate
 from .._types import FeedUpdateIntent
+from ..types import Author
 from .._utils import exactly_one
 from .._utils import zero_or_one
 from ..exceptions import FeedExistsError
@@ -290,12 +291,16 @@ def feed_factory(row: tuple[Any, ...]) -> Feed:
         update_after,
         last_retrieved,
     ) = row[:14]
+
+    # Parse the JSON string into Author objects
+    authors = tuple(Author(**d) for d in json.loads(author)) if author else ()
+
     return Feed(
         url,
         convert_timestamp(updated) if updated else None,
         title,
         link,
-        author,
+        authors,
         subtitle,
         version,
         user_title,
@@ -371,6 +376,10 @@ def feed_update_intent_to_dict(intent: FeedUpdateIntent) -> FeedDict:
 
         context['stale'] = 0
 
+        # Serialize `authors` and map it back to the `author` column
+        authors = context.pop('authors', ())
+        context['author'] = json.dumps([a._asdict() for a in authors]) if authors else None
+
     if isinstance(value, ExceptionInfo):
         context['last_exception'] = json.dumps(value._asdict())
     else:
diff --git a/src/reader/_storage/_schema.py b/src/reader/_storage/_schema.py
index e5cf16a2..b71709aa 100644
--- a/src/reader/_storage/_schema.py
+++ b/src/reader/_storage/_schema.py
@@ -319,7 +319,43 @@ def update_from_42_to_43(db: sqlite3.Connection, /) -> None:  # pragma: no cover
     db.execute("ALTER TABLE entries ADD COLUMN source TEXT;")
 
 
-VERSION = 43
+def _migrate_author_to_json(raw_author: str | None) -> str | None:
+    if not raw_author: 
+        return None
+    # Leave if its already a JSON list
+    if raw_author.startswith('['): 
+        return raw_author
+    
+    import json
+    authors = []
+    
+    for part in raw_author.split(','):
+        part = part.strip()
+        if not part: continue
+        
+        name, email = part, None
+        if part.endswith(')') and '(' in part:
+            split_idx = part.rfind('(')
+            name = part[:split_idx].strip()
+            email = part[split_idx+1:-1].strip()
+            
+        authors.append({
+            "name": name or None, 
+            "email": email or None, 
+            "href": None
+        })
+        
+    return json.dumps(authors)
+
+
+def update_from_43_to_44(db: sqlite3.Connection, /) -> None:  # pragma: no cover
+    # https://github.com/lemon24/reader/issues/391
+    db.create_function("MIGRATE_AUTHOR", 1, _migrate_author_to_json)
+    db.execute("UPDATE feeds SET author = MIGRATE_AUTHOR(author) WHERE author IS NOT NULL;")
+    db.execute("UPDATE entries SET author = MIGRATE_AUTHOR(author) WHERE author IS NOT NULL;")
+
+
+VERSION = 44
 
 MIGRATIONS = {
     # 1-9 removed before 0.1 (last in e4769d8ba77c61ec1fe2fbe99839e1826c17ace7)
@@ -333,6 +369,7 @@ def update_from_42_to_43(db: sqlite3.Connection, /) -> None:  # pragma: no cover
     40: update_from_40_to_41,
     41: update_from_41_to_42,
     42: update_from_42_to_43,
+    43: update_from_43_to_44,
 }
 MISSING_SUFFIX = (
     "; you may have skipped some required migrations, see "
diff --git a/src/reader/_types.py b/src/reader/_types.py
index b87a5afd..747de510 100644
--- a/src/reader/_types.py
+++ b/src/reader/_types.py
@@ -2,6 +2,7 @@
 
 import logging
 import time
+import warnings
 from collections import defaultdict
 from collections.abc import Callable
 from collections.abc import Iterable
@@ -55,6 +56,9 @@
 from .types import ResourceId
 from .types import TagFilterInput
 from .types import TristateFilterInput
+from .types import Author
+from .types import _AuthorMixin
+
 
 log = logging.getLogger("reader")
 
@@ -69,7 +73,7 @@
 
 
 @dataclass(frozen=True)
-class FeedData(_namedtuple_compat):
+class FeedData(_namedtuple_compat, _AuthorMixin):
     """Feed data that comes from the feed.
 
     Attributes are a subset of those of :class:`~reader.Feed`.
@@ -82,7 +86,10 @@ class FeedData(_namedtuple_compat):
     updated: datetime | None = None
     title: str | None = None
     link: str | None = None
-    author: str | None = None
+    #: The authors of the feed.
+    #:
+    #: .. versionadded:: 3.24
+    authors: Sequence[Author] = ()
     subtitle: str | None = None
     version: str | None = None
 
@@ -110,7 +117,7 @@ def hash(self) -> bytes:
 
 
 @dataclass(frozen=True)
-class EntryData(_namedtuple_compat):
+class EntryData(_namedtuple_compat, _AuthorMixin):
     """Entry data that comes from the feed.
 
     Attributes are a subset of those of :class:`~reader.Entry`.
@@ -135,7 +142,10 @@ class EntryData(_namedtuple_compat):
     updated: datetime | None = None
     title: str | None = None
     link: str | None = None
-    author: str | None = None
+    #: The authors of the feed.
+    #:
+    #: .. versionadded:: 3.24
+    authors: Sequence[Author] = ()
     published: datetime | None = None
     summary: str | None = None
     content: Sequence[Content] = ()
@@ -184,7 +194,7 @@ def entry_data_from_obj(obj: object) -> EntryData:
         updated=_getattr_optional_datetime(obj, 'updated'),
         title=_getattr_optional(obj, 'title', str),
         link=_getattr_optional(obj, 'link', str),
-        author=_getattr_optional(obj, 'author', str),
+        authors=tuple(author_from_obj(o) for o in getattr(obj, 'authors', ())),
         published=_getattr_optional_datetime(obj, 'published'),
         summary=_getattr_optional(obj, 'summary', str),
         content=tuple(content_from_obj(o) for o in getattr(obj, 'content', ())),
@@ -235,11 +245,21 @@ def source_from_obj(obj: object) -> EntrySource:
         updated=_getattr_optional_datetime(obj, 'updated'),
         title=_getattr_optional(obj, 'title', str),
         link=_getattr_optional(obj, 'link', str),
-        author=_getattr_optional(obj, 'author', str),
+        authors=tuple(author_from_obj(o) for o in getattr(obj, 'authors', ())),
         subtitle=_getattr_optional(obj, 'subtitle', str),
     )
 
 
+def author_from_obj(obj: object) -> Author:
+    if isinstance(obj, Mapping):
+        obj = SimpleNamespace(**obj)
+    return Author(
+        name=_getattr_optional(obj, 'name', str),
+        href=_getattr_optional(obj, 'href', str),
+        email=_getattr_optional(obj, 'email', str),
+    )
+
+
 def _getattr(obj: object, name: str, type: type[_T]) -> _T:
     # will raise AttributeError implicitly
     value = getattr(obj, name)
diff --git a/src/reader/types.py b/src/reader/types.py
index 5727f76a..30e7fa42 100644
--- a/src/reader/types.py
+++ b/src/reader/types.py
@@ -4,6 +4,7 @@
 import enum
 import re
 import traceback
+import warnings
 from collections.abc import Callable
 from collections.abc import Iterable
 from collections.abc import Mapping
@@ -56,7 +57,46 @@ def _asdict(self) -> dict[str, Any]:
 
 
 @dataclass(frozen=True)
-class Feed(_namedtuple_compat):
+class Author(_namedtuple_compat):
+    """Data type representing an author.
+    
+    .. versionadded:: 3.24
+
+    """
+
+    #: The name of the author.
+    name: str | None = None
+
+    #: The URL of the author.
+    href: str | None = None
+
+    #: The email of the author.
+    email: str | None = None
+
+
+class _AuthorMixin:
+    """Internal mixin to provide author string properties."""
+    authors: Sequence[Author]
+
+    @property
+    def author_str(self) -> str | None:
+        """The author of the feed/entry as a comma-separated list of names."""
+        names = [a.name for a in self.authors if a.name]
+        return ", ".join(names) if names else None
+
+    @property
+    def author(self) -> str | None:
+        """Deprecated alias for .author_str."""
+        warnings.warn(
+            f"{self.__class__.__name__}.author is deprecated; use .authors instead",
+            DeprecationWarning,
+            stacklevel=2,
+        )
+        return self.author_str
+
+
+@dataclass(frozen=True)
+class Feed(_namedtuple_compat, _AuthorMixin):
     """Data type representing a feed.
 
     All :class:`~datetime.datetime` attributes are timezone-aware,
@@ -82,8 +122,10 @@ class Feed(_namedtuple_compat):
     #: The URL of a page associated with the feed.
     link: str | None = None
 
-    #: The author of the feed.
-    author: str | None = None
+    #: The authors of the feed.
+    #:
+    #: .. versionadded:: 3.24
+    authors: Sequence[Author] = ()
 
     #: A description or subtitle for the feed.
     #:
@@ -170,6 +212,8 @@ def resolved_title(self) -> str | None:
 
         """
         return self.user_title or self.title
+    
+    
 
 
 @dataclass(frozen=True)
@@ -205,7 +249,7 @@ def from_exception(cls, exc: BaseException) -> Self:
 
 
 @dataclass(frozen=True)
-class Entry(_namedtuple_compat):
+class Entry(_namedtuple_compat, _AuthorMixin):
     """Data type representing an entry.
 
     All :class:`~datetime.datetime` attributes are timezone-aware,
@@ -248,8 +292,10 @@ def feed_url(self) -> str:
     #: The URL of a page associated with the entry.
     link: str | None = None
 
-    #: The author of the feed.
-    author: str | None = None
+    #: The authors of the feed.
+    #:
+    #: .. versionadded:: 3.24
+    authors: Sequence[Author] = ()
 
     #: The date the entry was published, according to the feed.
     published: datetime | None = None
@@ -473,7 +519,7 @@ class Enclosure(_namedtuple_compat):
 
 
 @dataclass(frozen=True)
-class EntrySource(_namedtuple_compat):
+class EntrySource(_namedtuple_compat, _AuthorMixin):
     """Metadata of a source feed (used with :attr:`Entry.source`).
 
     .. versionadded:: 3.16
@@ -498,8 +544,10 @@ class EntrySource(_namedtuple_compat):
     #: The URL of a page associated with the feed.
     link: str | None = None
 
-    #: The author of the feed.
-    author: str | None = None
+    #: The authors of the feed.
+    #:
+    #: .. versionadded:: 3.24
+    authors: Sequence[Author] = ()
 
     #: A description or subtitle for the feed.
     subtitle: str | None = None
@@ -1303,4 +1351,4 @@ def headers(self) -> dict[str, str]:
         return {
             'Content-Type': 'application/xml; charset=utf-8',
             'Content-Disposition': f'attachment; filename="{self.filename}"',
-        }
+        }
\ No newline at end of file
diff --git a/tests/data/empty.atom.py b/tests/data/empty.atom.py
index c1319ff1..3a1c9b2f 100644
--- a/tests/data/empty.atom.py
+++ b/tests/data/empty.atom.py
@@ -3,6 +3,7 @@
 from reader import Content
 from reader import Enclosure
 from reader._types import EntryData
+from reader.types import Author
 from reader._types import FeedData
 
 feed = FeedData(url=f'{url_base}empty.atom', version='atom10')
diff --git a/tests/data/full.atom.py b/tests/data/full.atom.py
index 30e00446..99c49138 100644
--- a/tests/data/full.atom.py
+++ b/tests/data/full.atom.py
@@ -4,6 +4,7 @@
 from reader import Enclosure
 from reader import EntrySource
 from reader._types import EntryData
+from reader.types import Author
 from reader._types import FeedData
 
 feed = FeedData(
@@ -11,7 +12,7 @@
     updated=datetime.datetime(2003, 12, 13, 18, 30, 2, tzinfo=datetime.UTC),
     title='Example Feed',
     link='http://example.org/',
-    author='John Doe',
+    authors=(Author(name='John Doe'),),
     subtitle='all your examples are belong to us',
     version='atom10',
 )
@@ -23,7 +24,7 @@
         updated=datetime.datetime(2003, 12, 13, 18, 30, 2, tzinfo=datetime.UTC),
         title='Atom-Powered Robots Run Amok',
         link='http://example.org/2003/12/13/atom03',
-        author='John Doe',
+        authors=(Author(name='John Doe'),),
         published=datetime.datetime(2003, 12, 13, 17, 17, 51, tzinfo=datetime.UTC),
         summary='Some text.',
         content=(
@@ -60,7 +61,7 @@
             updated=datetime.datetime(2003, 12, 13, 18, 30, 2, tzinfo=datetime.UTC),
             title='Source Title',
             link='http://example.org/source',
-            author='Source Author',
+            authors=(Author(name='Source Author'),),
             subtitle='source subtitle',
         ),
     ),
diff --git a/tests/data/full.json.py b/tests/data/full.json.py
index fdc2a48c..42fcc54f 100644
--- a/tests/data/full.json.py
+++ b/tests/data/full.json.py
@@ -3,6 +3,7 @@
 from reader import Content
 from reader import Enclosure
 from reader._types import EntryData
+from reader.types import Author
 from reader._types import FeedData
 
 feed = FeedData(
@@ -10,7 +11,7 @@
     updated=None,
     title='My Example Feed',
     link='https://example.org/',
-    author='Joe',
+    authors=(Author(name='Joe', href='mailto:joe@example.com'), Author(name='Jane')),
     subtitle='The Best Example Feed',
     version='json11',
 )
@@ -22,7 +23,7 @@
         updated=datetime.datetime(2020, 1, 4, 0, 0, tzinfo=datetime.UTC),
         title="Title",
         link="https://example.org/second-item",
-        author="mailto:joe@example.com",
+        authors=(Author(href="mailto:joe@example.com"),),
         published=datetime.datetime(2020, 1, 2, 21, 0, tzinfo=datetime.UTC),
         summary="A summary",
         content=(
@@ -54,7 +55,7 @@
         updated=None,
         title=None,
         link='https://example.org/initial-post',
-        author='Jane',
+        authors=(Author(name='Jane'),),
         published=datetime.datetime(2020, 1, 2, 12, 0, tzinfo=datetime.UTC),
         summary=None,
         content=(
diff --git a/tests/data/full.rss.py b/tests/data/full.rss.py
index 1afb79e9..6bec8fc9 100644
--- a/tests/data/full.rss.py
+++ b/tests/data/full.rss.py
@@ -4,6 +4,7 @@
 from reader import Enclosure
 from reader import EntrySource
 from reader._types import EntryData
+from reader.types import Author
 from reader._types import FeedData
 
 feed = FeedData(
@@ -11,7 +12,7 @@
     updated=datetime.datetime(2010, 9, 6, 0, 1, tzinfo=datetime.UTC),
     title='RSS Title',
     link='http://www.example.com/main.html',
-    author='Example editor (me@example.com)',
+    authors=(Author(name='Example editor', email='me@example.com'),),
     subtitle='This is an example of an RSS feed',
     version='rss20',
 )
@@ -23,7 +24,7 @@
         updated=None,
         title='Example entry',
         link='http://www.example.com/blog/post/1',
-        author='Example editor',
+        authors=(Author(name='Example editor'),),
         published=datetime.datetime(2009, 9, 6, 16, 20, tzinfo=datetime.UTC),
         summary='Here is some text containing an interesting description.',
         content=(
diff --git a/tests/data/invalid.json.py b/tests/data/invalid.json.py
index f195d92c..e54a8fd8 100644
--- a/tests/data/invalid.json.py
+++ b/tests/data/invalid.json.py
@@ -3,6 +3,7 @@
 from reader import Content
 from reader import Enclosure
 from reader._types import EntryData
+from reader.types import Author
 from reader._types import FeedData
 
 feed = FeedData(
@@ -41,7 +42,7 @@
         feed_url=feed.url,
         id='author name fallback',
         updated=None,
-        author='mailto:joe@example.com',
+        authors=(Author(href='mailto:joe@example.com'),),
     ),
     EntryData(
         feed_url=feed.url,
@@ -57,6 +58,6 @@
         feed_url=feed.url,
         id='second author is good',
         updated=None,
-        author='Jane',
+        authors=(Author(name='Jane'),),
     ),
 ]
diff --git a/tests/test__types.py b/tests/test__types.py
index 3ae57d32..5a08b463 100644
--- a/tests/test__types.py
+++ b/tests/test__types.py
@@ -162,6 +162,10 @@ def test_entry_data_from_obj(data_dir, feed_type, data_file):
                 enclosures=[SimpleNamespace(href='href', length='1')],
             ),
         ),
+        (
+            TypeError,
+            SimpleNamespace(feed_url='feed', id='id', authors=[SimpleNamespace(name=1)]),
+        ),
     ],
 )
 def test_entry_data_from_obj_errors(exc, entry):
@@ -174,4 +178,4 @@ def test_entry_data_from_obj_errors(exc, entry):
             entry_dict['content'] = [dict(vars(c)) for c in entry_dict['content']]
         if 'enclosures' in entry_dict:
             entry_dict['enclosures'] = [dict(vars(e)) for e in entry_dict['enclosures']]
-        entry_data_from_obj(entry_dict)
+        entry_data_from_obj(entry_dict)
\ No newline at end of file
diff --git a/tests/test_parser.py b/tests/test_parser.py
index ed000b9a..6d9b8018 100644
--- a/tests/test_parser.py
+++ b/tests/test_parser.py
@@ -17,10 +17,12 @@
 from reader._parser import RetrieveError
 from reader._parser.feedparser import feedparser
 from reader._parser.feedparser import FeedparserParser
+from reader._parser.feedparser import _parse_authors
 from reader._parser.file import FileRetriever
 from reader._parser.jsonfeed import JSONFeedParser
 from reader._parser.requests import SessionWrapper
 from reader._types import FeedData
+from reader.types import Author
 from reader.exceptions import ParseError
 from utils import make_url_base
 
@@ -1166,3 +1168,71 @@ def test_reader_use_system_feedparser(monkeypatch, reload_module):
     monkeypatch.setenv(name, '1')
     reload_module(reader._parser.feedparser)
     assert reader._parser.feedparser.feedparser is feedparser
+
+
+def test_feedparser_parse_authors_rss():
+    """Test the custom RSS author string splitting logic against known edge cases."""
+    
+    # Normal
+    assert _parse_authors({'author': 'John Doe'}, is_rss=True) == (Author(name='John Doe'),)
+    
+    assert _parse_authors({'author': 'John Doe (john@example.com)'}, is_rss=True) == (
+        Author(name='John Doe', email='john@example.com'),
+    )
+    
+    # Multiple names (comma separated)
+    assert _parse_authors({'author': 'John Doe, Jane Smith'}, is_rss=True) == (
+        Author(name='John Doe'), 
+        Author(name='Jane Smith')
+    )
+    
+    assert _parse_authors({'author': 'death and gravity'}, is_rss=True) == (
+        Author(name='death and gravity'),
+    )
+
+    # Empty cases
+    assert _parse_authors({'author': ''}, is_rss=True) == ()
+    assert _parse_authors({'author': '()'}, is_rss=True) == ()
+
+    # Multiple names with a single fallback email/href
+    assert _parse_authors({
+        'author': 'John, Jane', 
+        'author_detail': {'email': 'podcast@example.com', 'href': 'http://example.com'}
+    }, is_rss=True) == (
+        Author(name='John', email='podcast@example.com', href='http://example.com'), 
+        Author(name='Jane', email='podcast@example.com', href='http://example.com')
+    )
+
+    # If one has an inline email, so the fallback email should NOT be applied
+    assert _parse_authors({
+        'author': 'John (john@example.com), Jane', 
+        'author_detail': {'email': 'podcast@example.com'}
+    }, is_rss=True) == (
+        Author(name='John', email='john@example.com'), 
+        Author(name='Jane', email='podcast@example.com')
+    )
+
+
+def test_feedparser_parse_authors_atom():
+    """Test clean Atom author_detail extraction."""
+
+    atom_thing = {
+        'author_detail': {'name': 'Alice', 'email': 'alice@example.com', 'href': 'http://example.com'}
+    }
+
+    assert _parse_authors(atom_thing, is_rss=False) == (
+        Author(name='Alice', email='alice@example.com', href='http://example.com'),
+    )
+
+
+def test_feedparser_parse_authors_edge_cases():
+    from reader._parser.feedparser import _parse_authors
+    from reader.types import Author
+    
+    # Covers Atom author with ONLY href (no name/email)
+    assert _parse_authors({'author_detail': {'href': 'http://example.com'}}, is_rss=False) == \
+        (Author(href='http://example.com'),)
+    
+    # Covers RSS fallback where only href exists
+    assert _parse_authors({'author': 'John', 'author_detail': {'href': 'http://example.com'}}, is_rss=True) == \
+        (Author(name='John', href='http://example.com'),)
\ No newline at end of file
diff --git a/tests/test_reader.py b/tests/test_reader.py
index 9e4229ae..5a41fd2a 100644
--- a/tests/test_reader.py
+++ b/tests/test_reader.py
@@ -36,6 +36,7 @@
 from reader._types import FeedFilter
 from reader._types import FeedToUpdate
 from reader._types import FeedUpdateIntent
+from reader.types import Author
 from reader.core import DEFAULT_RESERVED_NAME_SCHEME
 from reader_methods import enable_and_update_search
 from reader_methods import get_entries
@@ -394,12 +395,12 @@ def title_tuple(feed):
 
 
 def test_data_roundtrip(reader, parser):
-    feed = parser.feed(1, datetime(2010, 1, 1), author='feed author')
+    feed = parser.feed(1, datetime(2010, 1, 1), authors=(Author(name='feed author'),))
     entry = parser.entry(
         1,
         1,
         datetime(2010, 1, 1),
-        author='entry author',
+        authors=(Author(name='entry author'),),
         summary='summary',
         content=(Content('value3', 'type', 'en'), Content('value2')),
         enclosures=(Enclosure('http://e1', 'type', 1000), Enclosure('http://e2')),
@@ -442,7 +443,7 @@ def test_data_hashes_remain_stable(parser):
         datetime(2010, 1, 1),
         title='Feed #1',
         link='http://www.example.com/1',
-        author='feed author',
+        authors=(Author(name='feed author'),),
     )
     entry = parser.entry(
         1,
@@ -450,25 +451,25 @@ def test_data_hashes_remain_stable(parser):
         datetime(2010, 1, 1),
         title='Entry #1',
         link='http://www.example.com/entries/1',
-        author='entry author',
+        authors=(Author(name='entry author'),),
         summary='summary',
         content=(Content('value3', 'type', 'en'), Content('value2')),
         enclosures=(Enclosure('http://e1', 'type', 1000), Enclosure('http://e2')),
     )
 
-    assert feed.hash == b'\x00\xda\xf5\xa1Je\x13],\xf0\xdb\xaa\x88d\x99\xc6'
-    assert entry.hash == b'\x00f\xa9\xdb\t5\xdf\xedcK\xd9bm\x80,l'
+    assert feed.hash == b'\x00\x9dS?\xf6\x9c\x9f=\x96\xb6JD\x0f\xef\x10\x82'
+    assert entry.hash == b'\x00+mI;N\xa3\x86\xff\x07\xdf\xd1\xe3\xfb\xbf\x12'
 
     assert feed._replace(url='x', updated='x').hash == feed.hash
     assert (
         feed._replace(title='x').hash
-        == b'\x00\xce\x81\xc7\x8d(\xab\xd8)\x06\x90?\xf9\x847\xc4'
+        == b'\x00bd\xb1*\x80\xf1u_\x17\xb4\n\xfe\xb2\x1e0'
     )
 
     assert entry._replace(feed_url='x', id='x', updated='x').hash == entry.hash
     assert (
         entry._replace(title='x').hash
-        == b'\x00\x95\xc4\xe9\xd3\x95\xf6\xff\xf0*\xbd\x00L\x08\x1a\xa2'
+        == b'\x00\xd3V\x19.;U6\xc9q\xc8\x88\xc2%\xff\xac'
     )
 
 
@@ -732,7 +733,7 @@ def drop_feed(entry):
 @pytest.mark.parametrize('new_feed_url', ['3', '2'])
 def test_change_feed_url_second_update(reader, new_feed_url):
     reader._parser.feed(
-        1, datetime(2010, 1, 1), title='old title', author='old author', link='old link'
+        1, datetime(2010, 1, 1), title='old title', authors=(Author(name='old author'),), link='old link'
     )
     reader.update_feeds()
     reader.update_search()
@@ -755,7 +756,7 @@ def test_change_feed_url_second_update(reader, new_feed_url):
         eval(new_feed_url),
         datetime(2010, 1, 2),
         title='new title',
-        author='new author',
+        authors=(Author(name='new author'),),
         link='new link',
     )
     reader._parser.entry(eval(new_feed_url), 1, datetime(2010, 1, 1))
@@ -771,7 +772,7 @@ def test_change_feed_url_second_update(reader, new_feed_url):
         last_retrieved=datetime(2010, 1, 3),
         update_after=datetime(2010, 1, 3, 1),
         title='new title',
-        author='new author',
+        authors=(Author(name='new author'),),
         link='new link',
     )
 
diff --git a/tests/test_reader_search.py b/tests/test_reader_search.py
index 65318b49..39c4752a 100644
--- a/tests/test_reader_search.py
+++ b/tests/test_reader_search.py
@@ -21,6 +21,7 @@
 from reader._storage._search import Search
 from reader._types import Action
 from reader._types import Change
+from reader.types import Author
 from reader.exceptions import ChangeTrackingNotEnabledError
 from test_reader_counts import entries_per_day
 from utils import rename_argument
@@ -443,7 +444,7 @@ def strip_html(*args, **kwargs):
     (old_result,) = reader.search_entries('entry OR feed')
 
     feed = parser.feed(
-        1, datetime(2010, 1, 2), title='feed', link='link', author='author'
+        1, datetime(2010, 1, 2), title='feed', link='link', authors=(Author(name='author'),)
     )
     entry = parser.entry(
         1,
@@ -453,7 +454,7 @@ def strip_html(*args, **kwargs):
         summary='summary',
         content=[Content('content')],
         link='link',
-        author='author',
+        authors=(Author(name='author'),),
         published=datetime(2010, 1, 2),
         enclosures=[Enclosure('enclosure')],
     )
diff --git a/tests/test_storage.py b/tests/test_storage.py
index f53384cf..f9bb06d5 100644
--- a/tests/test_storage.py
+++ b/tests/test_storage.py
@@ -6,6 +6,7 @@
 from unittest.mock import MagicMock
 
 import pytest
+import json
 
 import reader._storage._sqlite_utils
 from reader import EntryNotFoundError
@@ -16,6 +17,7 @@
 from reader._storage._sqlite_utils import DBError
 from reader._storage._sqlite_utils import HeavyMigration
 from reader._storage._sqlite_utils import require_version
+from reader._storage._schema import _migrate_author_to_json
 from reader._types import EntryData
 from reader._types import EntryFilter
 from reader._types import EntryForUpdate
@@ -594,3 +596,31 @@ def test_get_set_recent_sort(storage):
 def test_application_id(storage):
     id = storage.factory().execute('pragma application_id').fetchone()[0]
     assert id == int.from_bytes(b'read', 'big')
+
+
+def test_migrate_author_to_json():
+    """Ensure the SQLite migration correctly converts legacy strings to JSON."""
+    
+    # Null or empty cases
+    assert _migrate_author_to_json(None) is None
+    assert _migrate_author_to_json("") is None
+    
+    # Already migrated cases
+    existing_json = '[{"name": "John", "href": null, "email": null}]'
+    assert _migrate_author_to_json(existing_json) == existing_json
+    
+    # Legacy string cases
+    result = _migrate_author_to_json("John Doe (john@example.com)")
+    parsed = json.loads(result)
+    assert len(parsed) == 1
+    assert parsed[0]["name"] == "John Doe"
+    assert parsed[0]["href"] is None
+    assert parsed[0]["email"] == "john@example.com"
+
+
+def test_migrate_author_to_json_edge_cases():
+    from reader._storage._schema import _migrate_author_to_json
+    
+    # Covers extra commas (empty parts) and strings with only an email
+    assert _migrate_author_to_json("John,, (email@example.com)") == \
+        '[{"name": "John", "email": null, "href": null}, {"name": null, "email": "email@example.com", "href": null}]'
\ No newline at end of file
diff --git a/tests/test_types.py b/tests/test_types.py
index 7ef61cdf..d62d50f6 100644
--- a/tests/test_types.py
+++ b/tests/test_types.py
@@ -23,6 +23,7 @@
 from reader.types import _namedtuple_compat
 from reader.types import _resource_argument
 from reader.types import MISSING
+from reader.types import Author
 
 
 def test_namedtuple_compat():
@@ -376,3 +377,36 @@ def test_update_result_properties():
     assert result.updated_feed is None
     assert result.error is exc
     assert result.not_modified is False
+
+
+def test_author_deprecation_warning():
+    """Ensure accessing .author works but emits a DeprecationWarning."""
+    
+    # Test Feed formatting: "name (email)"
+    feed = Feed(url='http://example.com', authors=(Author(name='John', email='john@example.com'),))
+    with pytest.warns(DeprecationWarning, match=r"Feed\.author is deprecated"):
+        assert feed.author == "John"
+        
+    # Test Entry formatting: multiple authors
+    entry = Entry(
+        id='1', feed=feed, 
+        authors=(Author(name='Jane'), Author(name='Bob'))
+    )
+    with pytest.warns(DeprecationWarning, match=r"Entry\.author is deprecated"):
+        assert entry.author == "Jane, Bob"
+        
+    # Test empty authors returns None without crashing
+    empty_feed = Feed(url='http://example.com')
+    with pytest.warns(DeprecationWarning):
+        assert empty_feed.author is None
+
+
+def test_author_deprecation_internal_types():
+    from reader._types import FeedData, EntryData
+    from reader.types import EntrySource, Author
+    import pytest
+    
+    with pytest.warns(DeprecationWarning):
+        assert EntrySource(url='url', authors=(Author(name='Src'),)).author == 'Src'
+        assert FeedData(url='url', authors=(Author(name='FD'),)).author == 'FD'
+        assert EntryData(feed_url='url', id='id', authors=(Author(name='ED'),)).author == 'ED'
\ No newline at end of file

From b6eeb801bc57e7c035e98e18a04cdd00b4ba368a Mon Sep 17 00:00:00 2001
From: Anshul Mittal <anshulmittal0203@gmail.com>
Date: Wed, 13 May 2026 16:18:31 +0530
Subject: [PATCH 2/3] Ensure 100% test and branch coverage for multiple authors

- Refine feedparser.py fallback logic to remove an unreachable boolean branch.
- Add extreme edge cases to test_parser.py (empty details, malformed RSS strings) to hit all boolean branches.
- Update test__types.py to properly convert the new authors sequence into mappings, restoring _types.py coverage.
- Add missing deprecation warning tests for internal data models (FeedData, EntryData, EntrySource) in test_types.py.
- Add edge cases to test_storage.py to verify the SQLite migration handles malformed strings gracefully.
---
 docs/conf.py                     |  2 +
 src/reader/_parser/feedparser.py | 38 +++++++++--------
 src/reader/_parser/jsonfeed.py   | 30 ++++++++------
 src/reader/_storage/_schema.py   | 28 ++++---------
 src/reader/_types.py             |  6 +--
 tests/data/full.json.py          |  4 +-
 tests/data/invalid.json.py       |  2 +-
 tests/test__types.py             | 13 +++---
 tests/test_parser.py             | 70 ++++++++++++++++++++++++++++----
 tests/test_storage.py            |  4 --
 tests/test_types.py              |  5 +--
 11 files changed, 123 insertions(+), 79 deletions(-)

diff --git a/docs/conf.py b/docs/conf.py
index 6731bfcb..91a5dafd 100644
--- a/docs/conf.py
+++ b/docs/conf.py
@@ -19,6 +19,8 @@
     'werkzeug.exceptions',
     'werkzeug.http',
     'yaml',
+    'structlog',
+    'structlog.contextvars',
 ]:
     sys.modules[name] = unittest.mock.Mock()
 
diff --git a/src/reader/_parser/feedparser.py b/src/reader/_parser/feedparser.py
index 250ca079..70e4d48a 100644
--- a/src/reader/_parser/feedparser.py
+++ b/src/reader/_parser/feedparser.py
@@ -200,6 +200,24 @@ def _process_entry(feed_url: str, entry: Any, is_rss: bool) -> EntryData:
     )
 
 
+def _parse_rss_authors(raw_author: str) -> list[dict[str, str | None]]:
+    authors = []
+    for part in raw_author.split(','):
+        part = part.strip()
+        if not part: continue
+        
+        name, email = part, None
+        if part.endswith(')') and '(' in part:
+            split_idx = part.rfind('(')
+            name = part[:split_idx].strip()
+            email = part[split_idx+1:-1].strip()
+                
+        if name or email:
+            authors.append({'name': name or None, 'email': email or None})
+            
+    return authors
+
+
 def _parse_authors(thing: Any, is_rss: bool) -> tuple[Author, ...]:
     author_detail = thing.get('author_detail') or {}
     
@@ -233,23 +251,7 @@ def _parse_authors(thing: Any, is_rss: bool) -> tuple[Author, ...]:
         return ()
         
     # Split by comma
-    authors = []
-    found_any_email = False
-    
-    for part in raw_author.split(','):
-        part = part.strip()
-        if not part: continue
-        
-        name, email = part, None
-        if part.endswith(')') and '(' in part:
-            split_idx = part.rfind('(')
-            name = part[:split_idx].strip()
-            email = part[split_idx+1:-1].strip()
-            if email:
-                found_any_email = True
-                
-        if name or email:
-            authors.append({'name': name or None, 'email': email or None})
+    authors = _parse_rss_authors(raw_author)
             
     # Fallback from author_detail if no emails were found in the string
     if authors:
@@ -259,7 +261,7 @@ def _parse_authors(thing: Any, is_rss: bool) -> tuple[Author, ...]:
         for a in authors:
             if fallback_email and not a['email']:
                 a['email'] = fallback_email
-            if fallback_href and not a.get('href'):
+            if fallback_href:
                 a['href'] = fallback_href
             
     return tuple(Author(**a) for a in authors)
\ No newline at end of file
diff --git a/src/reader/_parser/jsonfeed.py b/src/reader/_parser/jsonfeed.py
index f31bf047..6df27d04 100644
--- a/src/reader/_parser/jsonfeed.py
+++ b/src/reader/_parser/jsonfeed.py
@@ -97,23 +97,27 @@ def _get(
 def _get_authors(d: Any) -> tuple[Author, ...]:
     authors = []
     
-    # Check for authors array first (JSON Feed 1.1)
-    for maybe_author in _get(d, 'authors', list) or ():
+    maybe_authors = _get(d, 'authors', list)
+    single_author = _get(d, 'author', dict)
+
+    # Feed readers should always prefer authors if present
+    if not maybe_authors and single_author:
+        maybe_authors = [single_author]
+
+    for maybe_author in maybe_authors or ():
         if isinstance(maybe_author, dict):
             name = _get(maybe_author, 'name', str)
             url = _get(maybe_author, 'url', str)
-            if name or url:
-                authors.append(Author(name=name, href=url))
-                
-    # Fallback to single author (JSON Feed 1.0)
-    if not authors:
-        single_author = _get(d, 'author', dict)
-        if single_author:
-            name = _get(single_author, 'name', str)
-            url = _get(single_author, 'url', str)
-            if name or url:
-                authors.append(Author(name=name, href=url))
+            
+            href = url
+            email = None
+            if url and url.lower().startswith('mailto:'):
+                email = url[7:]  # strip 'mailto:'
+                href = None
                 
+            if name or href or email:
+                authors.append(Author(name=name, href=href, email=email))
+
     return tuple(authors)
 
 
diff --git a/src/reader/_storage/_schema.py b/src/reader/_storage/_schema.py
index b71709aa..8ae6486d 100644
--- a/src/reader/_storage/_schema.py
+++ b/src/reader/_storage/_schema.py
@@ -322,28 +322,14 @@ def update_from_42_to_43(db: sqlite3.Connection, /) -> None:  # pragma: no cover
 def _migrate_author_to_json(raw_author: str | None) -> str | None:
     if not raw_author: 
         return None
-    # Leave if its already a JSON list
-    if raw_author.startswith('['): 
-        return raw_author
     
     import json
+    from .._parser.feedparser import _parse_rss_authors
+
     authors = []
-    
-    for part in raw_author.split(','):
-        part = part.strip()
-        if not part: continue
-        
-        name, email = part, None
-        if part.endswith(')') and '(' in part:
-            split_idx = part.rfind('(')
-            name = part[:split_idx].strip()
-            email = part[split_idx+1:-1].strip()
-            
-        authors.append({
-            "name": name or None, 
-            "email": email or None, 
-            "href": None
-        })
+    for a in _parse_rss_authors(raw_author):
+        a['href'] = None 
+        authors.append(a)
         
     return json.dumps(authors)
 
@@ -351,8 +337,8 @@ def _migrate_author_to_json(raw_author: str | None) -> str | None:
 def update_from_43_to_44(db: sqlite3.Connection, /) -> None:  # pragma: no cover
     # https://github.com/lemon24/reader/issues/391
     db.create_function("MIGRATE_AUTHOR", 1, _migrate_author_to_json)
-    db.execute("UPDATE feeds SET author = MIGRATE_AUTHOR(author) WHERE author IS NOT NULL;")
-    db.execute("UPDATE entries SET author = MIGRATE_AUTHOR(author) WHERE author IS NOT NULL;")
+    db.execute("UPDATE feeds SET author = MIGRATE_AUTHOR(author);")
+    db.execute("UPDATE entries SET author = MIGRATE_AUTHOR(author);")
 
 
 VERSION = 44
diff --git a/src/reader/_types.py b/src/reader/_types.py
index 747de510..323a5668 100644
--- a/src/reader/_types.py
+++ b/src/reader/_types.py
@@ -2,7 +2,6 @@
 
 import logging
 import time
-import warnings
 from collections import defaultdict
 from collections.abc import Callable
 from collections.abc import Iterable
@@ -57,7 +56,6 @@
 from .types import TagFilterInput
 from .types import TristateFilterInput
 from .types import Author
-from .types import _AuthorMixin
 
 
 log = logging.getLogger("reader")
@@ -73,7 +71,7 @@
 
 
 @dataclass(frozen=True)
-class FeedData(_namedtuple_compat, _AuthorMixin):
+class FeedData(_namedtuple_compat):
     """Feed data that comes from the feed.
 
     Attributes are a subset of those of :class:`~reader.Feed`.
@@ -117,7 +115,7 @@ def hash(self) -> bytes:
 
 
 @dataclass(frozen=True)
-class EntryData(_namedtuple_compat, _AuthorMixin):
+class EntryData(_namedtuple_compat):
     """Entry data that comes from the feed.
 
     Attributes are a subset of those of :class:`~reader.Entry`.
diff --git a/tests/data/full.json.py b/tests/data/full.json.py
index 42fcc54f..256990dd 100644
--- a/tests/data/full.json.py
+++ b/tests/data/full.json.py
@@ -11,7 +11,7 @@
     updated=None,
     title='My Example Feed',
     link='https://example.org/',
-    authors=(Author(name='Joe', href='mailto:joe@example.com'), Author(name='Jane')),
+    authors=(Author(name='Joe', href=None, email='joe@example.com'), Author(name='Jane')),
     subtitle='The Best Example Feed',
     version='json11',
 )
@@ -23,7 +23,7 @@
         updated=datetime.datetime(2020, 1, 4, 0, 0, tzinfo=datetime.UTC),
         title="Title",
         link="https://example.org/second-item",
-        authors=(Author(href="mailto:joe@example.com"),),
+        authors=(Author(email="joe@example.com"),),
         published=datetime.datetime(2020, 1, 2, 21, 0, tzinfo=datetime.UTC),
         summary="A summary",
         content=(
diff --git a/tests/data/invalid.json.py b/tests/data/invalid.json.py
index e54a8fd8..a61e278b 100644
--- a/tests/data/invalid.json.py
+++ b/tests/data/invalid.json.py
@@ -42,7 +42,7 @@
         feed_url=feed.url,
         id='author name fallback',
         updated=None,
-        authors=(Author(href='mailto:joe@example.com'),),
+        authors=(Author(email='joe@example.com'),),
     ),
     EntryData(
         feed_url=feed.url,
diff --git a/tests/test__types.py b/tests/test__types.py
index 5a08b463..5295f0b7 100644
--- a/tests/test__types.py
+++ b/tests/test__types.py
@@ -97,12 +97,13 @@ def test_entry_data_from_obj(data_dir, feed_type, data_file):
         assert entry == entry_data_from_obj(entry), i
 
         entry_dict = entry._asdict()
-        if 'content' in entry_dict:
-            entry_dict['content'] = [c._asdict() for c in entry_dict['content']]
-        if 'enclosures' in entry_dict:
-            entry_dict['enclosures'] = [e._asdict() for e in entry_dict['enclosures']]
+        entry_dict['content'] = [c._asdict() for c in entry_dict['content']]
+        entry_dict['enclosures'] = [e._asdict() for e in entry_dict['enclosures']]
+        entry_dict['authors'] = [a._asdict() for a in entry_dict['authors']]
         if entry_dict.get('source'):
-            entry_dict['source'] = entry_dict['source']._asdict()
+            source_dict = entry_dict['source']._asdict()
+            source_dict['authors'] = [a._asdict() for a in source_dict['authors']]
+            entry_dict['source'] = source_dict
 
         assert entry == entry_data_from_obj(entry_dict), i
 
@@ -178,4 +179,6 @@ def test_entry_data_from_obj_errors(exc, entry):
             entry_dict['content'] = [dict(vars(c)) for c in entry_dict['content']]
         if 'enclosures' in entry_dict:
             entry_dict['enclosures'] = [dict(vars(e)) for e in entry_dict['enclosures']]
+        if 'authors' in entry_dict:
+            entry_dict['authors'] = [dict(vars(a)) for a in entry_dict['authors']]
         entry_data_from_obj(entry_dict)
\ No newline at end of file
diff --git a/tests/test_parser.py b/tests/test_parser.py
index 6d9b8018..e3624d60 100644
--- a/tests/test_parser.py
+++ b/tests/test_parser.py
@@ -1225,14 +1225,70 @@ def test_feedparser_parse_authors_atom():
     )
 
 
-def test_feedparser_parse_authors_edge_cases():
+def test_feedparser_parse_authors_empty_details():
+    """Test parser handles empty or garbage author data gracefully."""
+
     from reader._parser.feedparser import _parse_authors
     from reader.types import Author
     
-    # Covers Atom author with ONLY href (no name/email)
-    assert _parse_authors({'author_detail': {'href': 'http://example.com'}}, is_rss=False) == \
-        (Author(href='http://example.com'),)
+    # Atom feed where author_detail exists but lacks name/email/href
+    assert _parse_authors({'author_detail': {'unrelated': 'data'}}, is_rss=False) == ()
+    
+    # RSS feed where author is empty, and fallback author_detail lacks name/email/href
+    assert _parse_authors({'author': '', 'author_detail': {'unrelated': 'data'}}, is_rss=True) == ()
+    
+    # RSS feed where author string has empty parts between commas
+    assert _parse_authors({'author': 'John, , Jane'}, is_rss=True) == (
+        Author(name='John'), 
+        Author(name='Jane')
+    )
+    assert _parse_authors({'author': ' , '}, is_rss=True) == ()
+
+    # RSS fallback block where 'name' is truthy but others are falsy
+    assert _parse_authors({'author': '', 'author_detail': {'name': 'A'}}, is_rss=True) == (Author(name='A'),)
+    
+    # RSS fallback block where 'href' is truthy, but author is exactly '()'
+    assert _parse_authors({'author': '()', 'author_detail': {'href': 'B'}}, is_rss=True) == (Author(href='B'),)
+
+    # RSS comma-split: email inside parens is empty -> `if email:` evaluates False
+    assert _parse_authors({'author': 'John ()'}, is_rss=True) == (Author(name='John'),)
+
+    # RSS comma-split: part is exactly `()` -> `if name or email:` evaluates False
+    assert _parse_authors({'author': 'John, ()'}, is_rss=True) == (Author(name='John'),)
+    
+    # RSS comma-split: name is empty before parens -> `name or None` evaluates to None
+    assert _parse_authors({'author': '(john@example.com)'}, is_rss=True) == (Author(email='john@example.com'),)
+
+
+def test_jsonfeed_empty_author_fallback():
+    """Test JSON Feed 1.0 fallback with an empty author object to satisfy branch coverage."""
+    feed, _ = jsonfeed_parse(
+        'url',
+        """
+        {
+            "version": "https://jsonfeed.org/version/1.0",
+            "title": "My Feed",
+            "author": {"unrelated": "data"}
+        }
+        """
+    )
+    assert feed.authors == ()
+
+
+def test_jsonfeed_author_mailto():
+    """Test JSON Feed author url starting with mailto: is mapped to email."""
+    from reader.types import Author
     
-    # Covers RSS fallback where only href exists
-    assert _parse_authors({'author': 'John', 'author_detail': {'href': 'http://example.com'}}, is_rss=True) == \
-        (Author(name='John', href='http://example.com'),)
\ No newline at end of file
+    feed, _ = jsonfeed_parse(
+        'url',
+        """
+        {
+            "version": "https://jsonfeed.org/version/1.1",
+            "title": "My Feed",
+            "authors": [
+                {"name": "Alice", "url": "mailto:alice@example.com"}
+            ]
+        }
+        """
+    )
+    assert feed.authors == (Author(name="Alice", email="alice@example.com", href=None),)
\ No newline at end of file
diff --git a/tests/test_storage.py b/tests/test_storage.py
index f9bb06d5..bc8b5dba 100644
--- a/tests/test_storage.py
+++ b/tests/test_storage.py
@@ -605,10 +605,6 @@ def test_migrate_author_to_json():
     assert _migrate_author_to_json(None) is None
     assert _migrate_author_to_json("") is None
     
-    # Already migrated cases
-    existing_json = '[{"name": "John", "href": null, "email": null}]'
-    assert _migrate_author_to_json(existing_json) == existing_json
-    
     # Legacy string cases
     result = _migrate_author_to_json("John Doe (john@example.com)")
     parsed = json.loads(result)
diff --git a/tests/test_types.py b/tests/test_types.py
index d62d50f6..10b24e1d 100644
--- a/tests/test_types.py
+++ b/tests/test_types.py
@@ -402,11 +402,8 @@ def test_author_deprecation_warning():
 
 
 def test_author_deprecation_internal_types():
-    from reader._types import FeedData, EntryData
     from reader.types import EntrySource, Author
     import pytest
     
     with pytest.warns(DeprecationWarning):
-        assert EntrySource(url='url', authors=(Author(name='Src'),)).author == 'Src'
-        assert FeedData(url='url', authors=(Author(name='FD'),)).author == 'FD'
-        assert EntryData(feed_url='url', id='id', authors=(Author(name='ED'),)).author == 'ED'
\ No newline at end of file
+        assert EntrySource(url='url', authors=(Author(name='Src'),)).author == 'Src'
\ No newline at end of file

From cca973f05aa217379957b2525040fa9e7472242f Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
 <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Thu, 14 May 2026 17:19:21 +0000
Subject: [PATCH 3/3] [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
---
 CHANGES.rst                                 |   4 +-
 src/reader/_app/legacy/templates/entry.html |   4 +-
 src/reader/_parser/feedparser.py            |  41 ++++----
 src/reader/_parser/jsonfeed.py              |   8 +-
 src/reader/_storage/_entries.py             |  20 ++--
 src/reader/_storage/_feeds.py               |   6 +-
 src/reader/_storage/_schema.py              |   9 +-
 src/reader/_types.py                        |   3 +-
 src/reader/types.py                         |   7 +-
 tests/data/empty.atom.py                    |   2 +-
 tests/data/full.atom.py                     |   2 +-
 tests/data/full.json.py                     |   7 +-
 tests/data/full.rss.py                      |   2 +-
 tests/data/invalid.json.py                  |   2 +-
 tests/test__types.py                        |   6 +-
 tests/test_parser.py                        | 100 ++++++++++++--------
 tests/test_reader.py                        |  14 +--
 tests/test_reader_search.py                 |   8 +-
 tests/test_storage.py                       |  16 ++--
 tests/test_types.py                         |  26 ++---
 20 files changed, 165 insertions(+), 122 deletions(-)

diff --git a/CHANGES.rst b/CHANGES.rst
index 4c8f4077..b85b3eef 100644
--- a/CHANGES.rst
+++ b/CHANGES.rst
@@ -14,8 +14,8 @@ Unreleased
 * Add ``--json`` option to the ``list`` CLI commands.
   Thanks to `Puneet Dixit`_ for the PR.
   (:issue:`394`)
-* Add :attr:`~Entry.authors` (and corresponding attributes on feeds and sources) 
-  to expose multiple authors and rich author data (name, email, URL). 
+* Add :attr:`~Entry.authors` (and corresponding attributes on feeds and sources)
+  to expose multiple authors and rich author data (name, email, URL).
   The old ``author`` string attribute is deprecated.
   Thanks to `Anshul Mittal`_ for the PR.
   (:issue:`391`)
diff --git a/src/reader/_app/legacy/templates/entry.html b/src/reader/_app/legacy/templates/entry.html
index 7295aeda..b1e1ccb1 100644
--- a/src/reader/_app/legacy/templates/entry.html
+++ b/src/reader/_app/legacy/templates/entry.html
@@ -18,8 +18,8 @@
 <ul class="controls">
 
 <li>
-    {% if entry.authors %} 
-        by 
+    {% if entry.authors %}
+        by
         {% for author in entry.authors -%}
             {% if author.href %}<a href="{{ author.href }}" title="{{ author.email or '' }}">{% endif %}
             {{- author.name or author.email or 'unknown' -}}
diff --git a/src/reader/_parser/feedparser.py b/src/reader/_parser/feedparser.py
index 70e4d48a..ccc61407 100644
--- a/src/reader/_parser/feedparser.py
+++ b/src/reader/_parser/feedparser.py
@@ -14,10 +14,10 @@
 from .._types import EntryData
 from .._types import FeedData
 from ..exceptions import ParseError
+from ..types import Author
 from ..types import Content
 from ..types import Enclosure
 from ..types import EntrySource
-from ..types import Author
 from ._http_utils import parse_accept_header
 from ._http_utils import unparse_accept_header
 
@@ -204,64 +204,57 @@ def _parse_rss_authors(raw_author: str) -> list[dict[str, str | None]]:
     authors = []
     for part in raw_author.split(','):
         part = part.strip()
-        if not part: continue
-        
+        if not part:
+            continue
+
         name, email = part, None
         if part.endswith(')') and '(' in part:
             split_idx = part.rfind('(')
             name = part[:split_idx].strip()
-            email = part[split_idx+1:-1].strip()
-                
+            email = part[split_idx + 1 : -1].strip()
+
         if name or email:
             authors.append({'name': name or None, 'email': email or None})
-            
+
     return authors
 
 
 def _parse_authors(thing: Any, is_rss: bool) -> tuple[Author, ...]:
     author_detail = thing.get('author_detail') or {}
-    
+
     # 1. Atom
     if not is_rss and author_detail:
         name = author_detail.get('name')
         email = author_detail.get('email')
         href = author_detail.get('href')
         if name or email or href:
-            return (Author(
-                name=name or None, 
-                email=email or None, 
-                href=href or None
-            ),)
+            return (Author(name=name or None, email=email or None, href=href or None),)
         return ()
-        
+
     # 2. RSS
     raw_author = thing.get('author', '')
-    
+
     # Fallback to author_detail if string is entirely empty or "()"
     if not raw_author or raw_author == '()':
         name = author_detail.get('name')
         email = author_detail.get('email')
         href = author_detail.get('href')
         if name or email or href:
-            return (Author(
-                name=name or None, 
-                email=email or None, 
-                href=href or None
-            ),)
+            return (Author(name=name or None, email=email or None, href=href or None),)
         return ()
-        
+
     # Split by comma
     authors = _parse_rss_authors(raw_author)
-            
+
     # Fallback from author_detail if no emails were found in the string
     if authors:
         fallback_email = author_detail.get('email')
         fallback_href = author_detail.get('href')
-        
+
         for a in authors:
             if fallback_email and not a['email']:
                 a['email'] = fallback_email
             if fallback_href:
                 a['href'] = fallback_href
-            
-    return tuple(Author(**a) for a in authors)
\ No newline at end of file
+
+    return tuple(Author(**a) for a in authors)
diff --git a/src/reader/_parser/jsonfeed.py b/src/reader/_parser/jsonfeed.py
index 6df27d04..67b557e4 100644
--- a/src/reader/_parser/jsonfeed.py
+++ b/src/reader/_parser/jsonfeed.py
@@ -15,9 +15,9 @@
 from .._types import EntryData
 from .._types import FeedData
 from ..exceptions import ParseError
+from ..types import Author
 from ..types import Content
 from ..types import Enclosure
-from ..types import Author
 
 if TYPE_CHECKING:  # pragma: no cover
     from . import FeedAndEntries
@@ -96,7 +96,7 @@ def _get(
 
 def _get_authors(d: Any) -> tuple[Author, ...]:
     authors = []
-    
+
     maybe_authors = _get(d, 'authors', list)
     single_author = _get(d, 'author', dict)
 
@@ -108,13 +108,13 @@ def _get_authors(d: Any) -> tuple[Author, ...]:
         if isinstance(maybe_author, dict):
             name = _get(maybe_author, 'name', str)
             url = _get(maybe_author, 'url', str)
-            
+
             href = url
             email = None
             if url and url.lower().startswith('mailto:'):
                 email = url[7:]  # strip 'mailto:'
                 href = None
-                
+
             if name or href or email:
                 authors.append(Author(name=name, href=href, email=email))
 
diff --git a/src/reader/_storage/_entries.py b/src/reader/_storage/_entries.py
index bd59504c..6b6b4a6d 100644
--- a/src/reader/_storage/_entries.py
+++ b/src/reader/_storage/_entries.py
@@ -24,13 +24,13 @@
 from ..exceptions import EntryExistsError
 from ..exceptions import EntryNotFoundError
 from ..exceptions import FeedNotFoundError
+from ..types import Author
 from ..types import Content
 from ..types import Enclosure
 from ..types import Entry
 from ..types import EntryCounts
 from ..types import EntrySort
 from ..types import EntrySource
-from ..types import Author
 from ._base import wrap_exceptions
 from ._feeds import feed_factory
 from ._sql_utils import Query
@@ -494,10 +494,14 @@ def entry_factory(row: tuple[Any, ...]) -> Entry:
         source_dict = json.loads(source)
         if source_dict['updated']:
             source_dict['updated'] = convert_timestamp(source_dict['updated'])
-        
+
         # Parse source feed authors
         source_author_json = source_dict.pop('author', None)
-        source_dict['authors'] = tuple(Author(**d) for d in json.loads(source_author_json)) if source_author_json else ()
+        source_dict['authors'] = (
+            tuple(Author(**d) for d in json.loads(source_author_json))
+            if source_author_json
+            else ()
+        )
 
         source_obj = EntrySource(**source_dict)
 
@@ -713,13 +717,17 @@ def entry_update_intent_to_dict(intent: EntryUpdateIntent) -> EntryDict:
         source_dict = entry.source._asdict()
         if entry.source.updated:
             source_dict['updated'] = adapt_datetime(entry.source.updated)
-        
+
         # Serialize the source authors and rename key to 'author'
         source_authors = source_dict.pop('authors', ())
-        source_dict['author'] = json.dumps([a._asdict() for a in source_authors]) if source_authors else None
+        source_dict['author'] = (
+            json.dumps([a._asdict() for a in source_authors])
+            if source_authors
+            else None
+        )
 
         context['source'] = json.dumps(source_dict)
-    
+
     # Rename the context key from 'authors' to 'author' to match SQLite column
     context['author'] = context.pop('authors', None)
 
diff --git a/src/reader/_storage/_feeds.py b/src/reader/_storage/_feeds.py
index 279c7d0e..071572aa 100644
--- a/src/reader/_storage/_feeds.py
+++ b/src/reader/_storage/_feeds.py
@@ -15,11 +15,11 @@
 from .._types import FeedForUpdate
 from .._types import FeedToUpdate
 from .._types import FeedUpdateIntent
-from ..types import Author
 from .._utils import exactly_one
 from .._utils import zero_or_one
 from ..exceptions import FeedExistsError
 from ..exceptions import FeedNotFoundError
+from ..types import Author
 from ..types import ExceptionInfo
 from ..types import Feed
 from ..types import FeedCounts
@@ -378,7 +378,9 @@ def feed_update_intent_to_dict(intent: FeedUpdateIntent) -> FeedDict:
 
         # Serialize `authors` and map it back to the `author` column
         authors = context.pop('authors', ())
-        context['author'] = json.dumps([a._asdict() for a in authors]) if authors else None
+        context['author'] = (
+            json.dumps([a._asdict() for a in authors]) if authors else None
+        )
 
     if isinstance(value, ExceptionInfo):
         context['last_exception'] = json.dumps(value._asdict())
diff --git a/src/reader/_storage/_schema.py b/src/reader/_storage/_schema.py
index 8ae6486d..d8bbc13b 100644
--- a/src/reader/_storage/_schema.py
+++ b/src/reader/_storage/_schema.py
@@ -320,17 +320,18 @@ def update_from_42_to_43(db: sqlite3.Connection, /) -> None:  # pragma: no cover
 
 
 def _migrate_author_to_json(raw_author: str | None) -> str | None:
-    if not raw_author: 
+    if not raw_author:
         return None
-    
+
     import json
+
     from .._parser.feedparser import _parse_rss_authors
 
     authors = []
     for a in _parse_rss_authors(raw_author):
-        a['href'] = None 
+        a['href'] = None
         authors.append(a)
-        
+
     return json.dumps(authors)
 
 
diff --git a/src/reader/_types.py b/src/reader/_types.py
index 323a5668..158dbc39 100644
--- a/src/reader/_types.py
+++ b/src/reader/_types.py
@@ -33,6 +33,7 @@
 from .types import _feed_argument
 from .types import _namedtuple_compat
 from .types import AnyResourceId
+from .types import Author
 from .types import Content
 from .types import Enclosure
 from .types import Entry
@@ -55,8 +56,6 @@
 from .types import ResourceId
 from .types import TagFilterInput
 from .types import TristateFilterInput
-from .types import Author
-
 
 log = logging.getLogger("reader")
 
diff --git a/src/reader/types.py b/src/reader/types.py
index 30e7fa42..589ccc99 100644
--- a/src/reader/types.py
+++ b/src/reader/types.py
@@ -59,7 +59,7 @@ def _asdict(self) -> dict[str, Any]:
 @dataclass(frozen=True)
 class Author(_namedtuple_compat):
     """Data type representing an author.
-    
+
     .. versionadded:: 3.24
 
     """
@@ -76,6 +76,7 @@ class Author(_namedtuple_compat):
 
 class _AuthorMixin:
     """Internal mixin to provide author string properties."""
+
     authors: Sequence[Author]
 
     @property
@@ -212,8 +213,6 @@ def resolved_title(self) -> str | None:
 
         """
         return self.user_title or self.title
-    
-    
 
 
 @dataclass(frozen=True)
@@ -1351,4 +1350,4 @@ def headers(self) -> dict[str, str]:
         return {
             'Content-Type': 'application/xml; charset=utf-8',
             'Content-Disposition': f'attachment; filename="{self.filename}"',
-        }
\ No newline at end of file
+        }
diff --git a/tests/data/empty.atom.py b/tests/data/empty.atom.py
index 3a1c9b2f..bc5fdd55 100644
--- a/tests/data/empty.atom.py
+++ b/tests/data/empty.atom.py
@@ -3,8 +3,8 @@
 from reader import Content
 from reader import Enclosure
 from reader._types import EntryData
-from reader.types import Author
 from reader._types import FeedData
+from reader.types import Author
 
 feed = FeedData(url=f'{url_base}empty.atom', version='atom10')
 
diff --git a/tests/data/full.atom.py b/tests/data/full.atom.py
index 99c49138..d0b244c4 100644
--- a/tests/data/full.atom.py
+++ b/tests/data/full.atom.py
@@ -4,8 +4,8 @@
 from reader import Enclosure
 from reader import EntrySource
 from reader._types import EntryData
-from reader.types import Author
 from reader._types import FeedData
+from reader.types import Author
 
 feed = FeedData(
     url=f'{url_base}full.atom',
diff --git a/tests/data/full.json.py b/tests/data/full.json.py
index 256990dd..3852f3f9 100644
--- a/tests/data/full.json.py
+++ b/tests/data/full.json.py
@@ -3,15 +3,18 @@
 from reader import Content
 from reader import Enclosure
 from reader._types import EntryData
-from reader.types import Author
 from reader._types import FeedData
+from reader.types import Author
 
 feed = FeedData(
     url=f'{url_base}full.json',
     updated=None,
     title='My Example Feed',
     link='https://example.org/',
-    authors=(Author(name='Joe', href=None, email='joe@example.com'), Author(name='Jane')),
+    authors=(
+        Author(name='Joe', href=None, email='joe@example.com'),
+        Author(name='Jane'),
+    ),
     subtitle='The Best Example Feed',
     version='json11',
 )
diff --git a/tests/data/full.rss.py b/tests/data/full.rss.py
index 6bec8fc9..15e3ea77 100644
--- a/tests/data/full.rss.py
+++ b/tests/data/full.rss.py
@@ -4,8 +4,8 @@
 from reader import Enclosure
 from reader import EntrySource
 from reader._types import EntryData
-from reader.types import Author
 from reader._types import FeedData
+from reader.types import Author
 
 feed = FeedData(
     url=f'{url_base}full.rss',
diff --git a/tests/data/invalid.json.py b/tests/data/invalid.json.py
index a61e278b..2f29ee1f 100644
--- a/tests/data/invalid.json.py
+++ b/tests/data/invalid.json.py
@@ -3,8 +3,8 @@
 from reader import Content
 from reader import Enclosure
 from reader._types import EntryData
-from reader.types import Author
 from reader._types import FeedData
+from reader.types import Author
 
 feed = FeedData(
     url=f'{url_base}invalid.json',
diff --git a/tests/test__types.py b/tests/test__types.py
index 5295f0b7..fdfb7f24 100644
--- a/tests/test__types.py
+++ b/tests/test__types.py
@@ -165,7 +165,9 @@ def test_entry_data_from_obj(data_dir, feed_type, data_file):
         ),
         (
             TypeError,
-            SimpleNamespace(feed_url='feed', id='id', authors=[SimpleNamespace(name=1)]),
+            SimpleNamespace(
+                feed_url='feed', id='id', authors=[SimpleNamespace(name=1)]
+            ),
         ),
     ],
 )
@@ -181,4 +183,4 @@ def test_entry_data_from_obj_errors(exc, entry):
             entry_dict['enclosures'] = [dict(vars(e)) for e in entry_dict['enclosures']]
         if 'authors' in entry_dict:
             entry_dict['authors'] = [dict(vars(a)) for a in entry_dict['authors']]
-        entry_data_from_obj(entry_dict)
\ No newline at end of file
+        entry_data_from_obj(entry_dict)
diff --git a/tests/test_parser.py b/tests/test_parser.py
index e3624d60..a80aabba 100644
--- a/tests/test_parser.py
+++ b/tests/test_parser.py
@@ -15,15 +15,15 @@
 from reader._parser import Parser
 from reader._parser import RetrievedFeed
 from reader._parser import RetrieveError
+from reader._parser.feedparser import _parse_authors
 from reader._parser.feedparser import feedparser
 from reader._parser.feedparser import FeedparserParser
-from reader._parser.feedparser import _parse_authors
 from reader._parser.file import FileRetriever
 from reader._parser.jsonfeed import JSONFeedParser
 from reader._parser.requests import SessionWrapper
 from reader._types import FeedData
-from reader.types import Author
 from reader.exceptions import ParseError
+from reader.types import Author
 from utils import make_url_base
 
 
@@ -1172,20 +1172,22 @@ def test_reader_use_system_feedparser(monkeypatch, reload_module):
 
 def test_feedparser_parse_authors_rss():
     """Test the custom RSS author string splitting logic against known edge cases."""
-    
+
     # Normal
-    assert _parse_authors({'author': 'John Doe'}, is_rss=True) == (Author(name='John Doe'),)
-    
+    assert _parse_authors({'author': 'John Doe'}, is_rss=True) == (
+        Author(name='John Doe'),
+    )
+
     assert _parse_authors({'author': 'John Doe (john@example.com)'}, is_rss=True) == (
         Author(name='John Doe', email='john@example.com'),
     )
-    
+
     # Multiple names (comma separated)
     assert _parse_authors({'author': 'John Doe, Jane Smith'}, is_rss=True) == (
-        Author(name='John Doe'), 
-        Author(name='Jane Smith')
+        Author(name='John Doe'),
+        Author(name='Jane Smith'),
     )
-    
+
     assert _parse_authors({'author': 'death and gravity'}, is_rss=True) == (
         Author(name='death and gravity'),
     )
@@ -1195,21 +1197,30 @@ def test_feedparser_parse_authors_rss():
     assert _parse_authors({'author': '()'}, is_rss=True) == ()
 
     # Multiple names with a single fallback email/href
-    assert _parse_authors({
-        'author': 'John, Jane', 
-        'author_detail': {'email': 'podcast@example.com', 'href': 'http://example.com'}
-    }, is_rss=True) == (
-        Author(name='John', email='podcast@example.com', href='http://example.com'), 
-        Author(name='Jane', email='podcast@example.com', href='http://example.com')
+    assert _parse_authors(
+        {
+            'author': 'John, Jane',
+            'author_detail': {
+                'email': 'podcast@example.com',
+                'href': 'http://example.com',
+            },
+        },
+        is_rss=True,
+    ) == (
+        Author(name='John', email='podcast@example.com', href='http://example.com'),
+        Author(name='Jane', email='podcast@example.com', href='http://example.com'),
     )
 
     # If one has an inline email, so the fallback email should NOT be applied
-    assert _parse_authors({
-        'author': 'John (john@example.com), Jane', 
-        'author_detail': {'email': 'podcast@example.com'}
-    }, is_rss=True) == (
-        Author(name='John', email='john@example.com'), 
-        Author(name='Jane', email='podcast@example.com')
+    assert _parse_authors(
+        {
+            'author': 'John (john@example.com), Jane',
+            'author_detail': {'email': 'podcast@example.com'},
+        },
+        is_rss=True,
+    ) == (
+        Author(name='John', email='john@example.com'),
+        Author(name='Jane', email='podcast@example.com'),
     )
 
 
@@ -1217,7 +1228,11 @@ def test_feedparser_parse_authors_atom():
     """Test clean Atom author_detail extraction."""
 
     atom_thing = {
-        'author_detail': {'name': 'Alice', 'email': 'alice@example.com', 'href': 'http://example.com'}
+        'author_detail': {
+            'name': 'Alice',
+            'email': 'alice@example.com',
+            'href': 'http://example.com',
+        }
     }
 
     assert _parse_authors(atom_thing, is_rss=False) == (
@@ -1230,34 +1245,45 @@ def test_feedparser_parse_authors_empty_details():
 
     from reader._parser.feedparser import _parse_authors
     from reader.types import Author
-    
+
     # Atom feed where author_detail exists but lacks name/email/href
     assert _parse_authors({'author_detail': {'unrelated': 'data'}}, is_rss=False) == ()
-    
+
     # RSS feed where author is empty, and fallback author_detail lacks name/email/href
-    assert _parse_authors({'author': '', 'author_detail': {'unrelated': 'data'}}, is_rss=True) == ()
-    
+    assert (
+        _parse_authors(
+            {'author': '', 'author_detail': {'unrelated': 'data'}}, is_rss=True
+        )
+        == ()
+    )
+
     # RSS feed where author string has empty parts between commas
     assert _parse_authors({'author': 'John, , Jane'}, is_rss=True) == (
-        Author(name='John'), 
-        Author(name='Jane')
+        Author(name='John'),
+        Author(name='Jane'),
     )
     assert _parse_authors({'author': ' , '}, is_rss=True) == ()
 
     # RSS fallback block where 'name' is truthy but others are falsy
-    assert _parse_authors({'author': '', 'author_detail': {'name': 'A'}}, is_rss=True) == (Author(name='A'),)
-    
+    assert _parse_authors(
+        {'author': '', 'author_detail': {'name': 'A'}}, is_rss=True
+    ) == (Author(name='A'),)
+
     # RSS fallback block where 'href' is truthy, but author is exactly '()'
-    assert _parse_authors({'author': '()', 'author_detail': {'href': 'B'}}, is_rss=True) == (Author(href='B'),)
+    assert _parse_authors(
+        {'author': '()', 'author_detail': {'href': 'B'}}, is_rss=True
+    ) == (Author(href='B'),)
 
     # RSS comma-split: email inside parens is empty -> `if email:` evaluates False
     assert _parse_authors({'author': 'John ()'}, is_rss=True) == (Author(name='John'),)
 
     # RSS comma-split: part is exactly `()` -> `if name or email:` evaluates False
     assert _parse_authors({'author': 'John, ()'}, is_rss=True) == (Author(name='John'),)
-    
+
     # RSS comma-split: name is empty before parens -> `name or None` evaluates to None
-    assert _parse_authors({'author': '(john@example.com)'}, is_rss=True) == (Author(email='john@example.com'),)
+    assert _parse_authors({'author': '(john@example.com)'}, is_rss=True) == (
+        Author(email='john@example.com'),
+    )
 
 
 def test_jsonfeed_empty_author_fallback():
@@ -1270,7 +1296,7 @@ def test_jsonfeed_empty_author_fallback():
             "title": "My Feed",
             "author": {"unrelated": "data"}
         }
-        """
+        """,
     )
     assert feed.authors == ()
 
@@ -1278,7 +1304,7 @@ def test_jsonfeed_empty_author_fallback():
 def test_jsonfeed_author_mailto():
     """Test JSON Feed author url starting with mailto: is mapped to email."""
     from reader.types import Author
-    
+
     feed, _ = jsonfeed_parse(
         'url',
         """
@@ -1289,6 +1315,6 @@ def test_jsonfeed_author_mailto():
                 {"name": "Alice", "url": "mailto:alice@example.com"}
             ]
         }
-        """
+        """,
     )
-    assert feed.authors == (Author(name="Alice", email="alice@example.com", href=None),)
\ No newline at end of file
+    assert feed.authors == (Author(name="Alice", email="alice@example.com", href=None),)
diff --git a/tests/test_reader.py b/tests/test_reader.py
index 5a41fd2a..0b6a4ecd 100644
--- a/tests/test_reader.py
+++ b/tests/test_reader.py
@@ -36,8 +36,8 @@
 from reader._types import FeedFilter
 from reader._types import FeedToUpdate
 from reader._types import FeedUpdateIntent
-from reader.types import Author
 from reader.core import DEFAULT_RESERVED_NAME_SCHEME
+from reader.types import Author
 from reader_methods import enable_and_update_search
 from reader_methods import get_entries
 from reader_methods import get_entries_random
@@ -462,14 +462,12 @@ def test_data_hashes_remain_stable(parser):
 
     assert feed._replace(url='x', updated='x').hash == feed.hash
     assert (
-        feed._replace(title='x').hash
-        == b'\x00bd\xb1*\x80\xf1u_\x17\xb4\n\xfe\xb2\x1e0'
+        feed._replace(title='x').hash == b'\x00bd\xb1*\x80\xf1u_\x17\xb4\n\xfe\xb2\x1e0'
     )
 
     assert entry._replace(feed_url='x', id='x', updated='x').hash == entry.hash
     assert (
-        entry._replace(title='x').hash
-        == b'\x00\xd3V\x19.;U6\xc9q\xc8\x88\xc2%\xff\xac'
+        entry._replace(title='x').hash == b'\x00\xd3V\x19.;U6\xc9q\xc8\x88\xc2%\xff\xac'
     )
 
 
@@ -733,7 +731,11 @@ def drop_feed(entry):
 @pytest.mark.parametrize('new_feed_url', ['3', '2'])
 def test_change_feed_url_second_update(reader, new_feed_url):
     reader._parser.feed(
-        1, datetime(2010, 1, 1), title='old title', authors=(Author(name='old author'),), link='old link'
+        1,
+        datetime(2010, 1, 1),
+        title='old title',
+        authors=(Author(name='old author'),),
+        link='old link',
     )
     reader.update_feeds()
     reader.update_search()
diff --git a/tests/test_reader_search.py b/tests/test_reader_search.py
index 39c4752a..79e1eaed 100644
--- a/tests/test_reader_search.py
+++ b/tests/test_reader_search.py
@@ -21,8 +21,8 @@
 from reader._storage._search import Search
 from reader._types import Action
 from reader._types import Change
-from reader.types import Author
 from reader.exceptions import ChangeTrackingNotEnabledError
+from reader.types import Author
 from test_reader_counts import entries_per_day
 from utils import rename_argument
 from utils import utc_datetime
@@ -444,7 +444,11 @@ def strip_html(*args, **kwargs):
     (old_result,) = reader.search_entries('entry OR feed')
 
     feed = parser.feed(
-        1, datetime(2010, 1, 2), title='feed', link='link', authors=(Author(name='author'),)
+        1,
+        datetime(2010, 1, 2),
+        title='feed',
+        link='link',
+        authors=(Author(name='author'),),
     )
     entry = parser.entry(
         1,
diff --git a/tests/test_storage.py b/tests/test_storage.py
index bc8b5dba..835000b5 100644
--- a/tests/test_storage.py
+++ b/tests/test_storage.py
@@ -1,3 +1,4 @@
+import json
 import os
 import sqlite3
 import sys
@@ -6,7 +7,6 @@
 from unittest.mock import MagicMock
 
 import pytest
-import json
 
 import reader._storage._sqlite_utils
 from reader import EntryNotFoundError
@@ -14,10 +14,10 @@
 from reader import InvalidSearchQueryError
 from reader import StorageError
 from reader._storage import Storage
+from reader._storage._schema import _migrate_author_to_json
 from reader._storage._sqlite_utils import DBError
 from reader._storage._sqlite_utils import HeavyMigration
 from reader._storage._sqlite_utils import require_version
-from reader._storage._schema import _migrate_author_to_json
 from reader._types import EntryData
 from reader._types import EntryFilter
 from reader._types import EntryForUpdate
@@ -600,11 +600,11 @@ def test_application_id(storage):
 
 def test_migrate_author_to_json():
     """Ensure the SQLite migration correctly converts legacy strings to JSON."""
-    
+
     # Null or empty cases
     assert _migrate_author_to_json(None) is None
     assert _migrate_author_to_json("") is None
-    
+
     # Legacy string cases
     result = _migrate_author_to_json("John Doe (john@example.com)")
     parsed = json.loads(result)
@@ -616,7 +616,9 @@ def test_migrate_author_to_json():
 
 def test_migrate_author_to_json_edge_cases():
     from reader._storage._schema import _migrate_author_to_json
-    
+
     # Covers extra commas (empty parts) and strings with only an email
-    assert _migrate_author_to_json("John,, (email@example.com)") == \
-        '[{"name": "John", "email": null, "href": null}, {"name": null, "email": "email@example.com", "href": null}]'
\ No newline at end of file
+    assert (
+        _migrate_author_to_json("John,, (email@example.com)")
+        == '[{"name": "John", "email": null, "href": null}, {"name": null, "email": "email@example.com", "href": null}]'
+    )
diff --git a/tests/test_types.py b/tests/test_types.py
index 10b24e1d..cef54879 100644
--- a/tests/test_types.py
+++ b/tests/test_types.py
@@ -22,8 +22,8 @@
 from reader.types import _feed_argument
 from reader.types import _namedtuple_compat
 from reader.types import _resource_argument
-from reader.types import MISSING
 from reader.types import Author
+from reader.types import MISSING
 
 
 def test_namedtuple_compat():
@@ -381,20 +381,20 @@ def test_update_result_properties():
 
 def test_author_deprecation_warning():
     """Ensure accessing .author works but emits a DeprecationWarning."""
-    
+
     # Test Feed formatting: "name (email)"
-    feed = Feed(url='http://example.com', authors=(Author(name='John', email='john@example.com'),))
+    feed = Feed(
+        url='http://example.com',
+        authors=(Author(name='John', email='john@example.com'),),
+    )
     with pytest.warns(DeprecationWarning, match=r"Feed\.author is deprecated"):
         assert feed.author == "John"
-        
+
     # Test Entry formatting: multiple authors
-    entry = Entry(
-        id='1', feed=feed, 
-        authors=(Author(name='Jane'), Author(name='Bob'))
-    )
+    entry = Entry(id='1', feed=feed, authors=(Author(name='Jane'), Author(name='Bob')))
     with pytest.warns(DeprecationWarning, match=r"Entry\.author is deprecated"):
         assert entry.author == "Jane, Bob"
-        
+
     # Test empty authors returns None without crashing
     empty_feed = Feed(url='http://example.com')
     with pytest.warns(DeprecationWarning):
@@ -402,8 +402,10 @@ def test_author_deprecation_warning():
 
 
 def test_author_deprecation_internal_types():
-    from reader.types import EntrySource, Author
     import pytest
-    
+
+    from reader.types import Author
+    from reader.types import EntrySource
+
     with pytest.warns(DeprecationWarning):
-        assert EntrySource(url='url', authors=(Author(name='Src'),)).author == 'Src'
\ No newline at end of file
+        assert EntrySource(url='url', authors=(Author(name='Src'),)).author == 'Src'