promnesia 1.3.20241021__py3-none-any.whl → 1.4.20250909__py3-none-any.whl

promnesia/sources/github.py

@@ -1,6 +1,7 @@
 '''
 Uses [[https://github.com/karlicoss/HPI][HPI]] github module
 '''
+
 from __future__ import annotations
 
 # Note: requires the 'mistletoe' module if you enable render_markdown
@@ -16,7 +17,9 @@ def index(*, render_markdown: bool = False) -> Results:
             from .markdown import TextParser, extract_from_text
         except ImportError as import_err:
             logger.exception(import_err)
-            logger.critical("Could not import markdown module to render github body markdown. Try 'python3 -m pip install mistletoe'")
+            logger.critical(
+                "Could not import markdown module to render github body markdown. Try 'python3 -m pip install mistletoe'"
+            )
             render_markdown = False
 
     for e in events():

promnesia/sources/guess.py

@@ -22,14 +22,17 @@ def index(path: PathIsh, *args, **kwargs) -> Iterable[Extraction]:
     ps = str(path)
     # TODO better url detection
 
-    index_: Any # meh
+    index_: Any  # meh
     if is_git_repo(ps):
         from . import vcs
+
         index_ = vcs.index
     elif is_website(ps):
         from . import website
+
         index_ = website.index
     else:
         from . import auto
+
         index_ = auto.index
     yield from index_(path, *args, **kwargs)

promnesia/sources/hackernews.py

@@ -21,9 +21,7 @@ def index() -> Results:
             title = item.title
         elif item.text_html:
             title = item.text_html
-            title = textwrap.shorten(
-                    title, width=79, placeholder="…",
-                    break_long_words=True)
+            title = textwrap.shorten(title, width=79, placeholder="…", break_long_words=True)
         # The locator is always the HN story. If the story is a link (as
         # opposed to a text post), we insert a visit such that the link
         # will point back to the corresponding HN story.
@@ -33,8 +31,8 @@ def index() -> Results:
             urls.append(item.url)
         for url in urls:
             yield Visit(
-                    url=url,
-                    dt=item.created,
-                    locator=loc,
-                    context=title,
+                url=url,
+                dt=item.created,
+                locator=loc,
+                context=title,
             )

promnesia/sources/hpi.py

@@ -8,4 +8,6 @@ try:
     import my  # noqa: F401
 except ImportError as e:
     logger.exception(e)
-    logger.critical("Failed during 'import my'. You probably need to install & configure HPI package first (see 'https://github.com/karlicoss/HPI/blob/master/doc/SETUP.org')")
+    logger.critical(
+        "Failed during 'import my'. You probably need to install & configure HPI package first (see 'https://github.com/karlicoss/HPI/blob/master/doc/SETUP.org')"
+    )

promnesia/sources/html.py

@@ -7,7 +7,7 @@ from __future__ import annotations
 from collections.abc import Iterator
 from pathlib import Path
 
-from bs4 import BeautifulSoup
+from bs4 import BeautifulSoup, Tag
 
 from promnesia.common import Loc, PathIsh, Results, Visit, file_mtime
 
@@ -25,11 +25,13 @@ def extract_urls_from_html(s: str) -> Iterator[Url]:
     """
     soup = BeautifulSoup(s, 'lxml')
     for a in soup.find_all('a'):
+        assert isinstance(a, Tag), a  # make mypy happy
         href = a.attrs.get('href')
         if href is None or ('://' not in href):
             # second condition means relative link
             continue
-        text = a.text
+        assert isinstance(href, str), href  # make mypy happy
+        text: str = a.text
         yield (href, text)
 
 

promnesia/sources/instapaper.py

@@ -1,6 +1,7 @@
 '''
 Uses HPI [[https://github.com/karlicoss/HPI/blob/master/doc/MODULES.org#myinstapaper][instapaper]] module
 '''
+
 from promnesia.common import Loc, Results, Visit
 
 

promnesia/sources/markdown.py

@@ -4,10 +4,10 @@ from collections.abc import Iterator
 from pathlib import Path
 from typing import NamedTuple
 
-import mistletoe  # type: ignore
-import mistletoe.block_token as BT  # type: ignore
-from mistletoe.html_renderer import HTMLRenderer  # type: ignore
-from mistletoe.span_token import AutoLink, Link  # type: ignore
+import mistletoe  # type: ignore[import-untyped]
+import mistletoe.block_token as BT  # type: ignore[import-untyped]
+from mistletoe.html_renderer import HTMLRenderer  # type: ignore[import-untyped]
+from mistletoe.span_token import AutoLink, Link  # type: ignore[import-untyped]
 
 from promnesia.common import (
     Extraction,

promnesia/sources/org.py

@@ -4,7 +4,7 @@ import re
 from collections.abc import Iterable, Iterator
 from datetime import datetime
 from pathlib import Path
-from typing import NamedTuple, Optional, cast
+from typing import NamedTuple, cast
 
 import orgparse
 from orgparse.date import OrgDate, gene_timestamp_regex
@@ -25,6 +25,8 @@ from promnesia.common import (
 UPDATE_ORGPARSE_WARNING = 'WARNING: please update orgparse version to a more recent (pip3 install -U orgparse)'
 
 _warned = False
+
+
 def warn_old_orgparse_once() -> Iterable[Exception]:
     global _warned
     if _warned:
@@ -45,6 +47,7 @@ CREATED_RGX = re.compile(gene_timestamp_regex(brtype='inactive'), re.VERBOSE)
 ** subnote
 """
 
+
 class Parsed(NamedTuple):
     dt: datetime | None
     heading: str
@@ -56,14 +59,14 @@ def _parse_node(n: OrgNode) -> Parsed:
 
     heading = n.get_heading('raw')
     pp = n.properties
-    createds = cast(Optional[str], pp.get('CREATED', None))
+    createds = cast(str | None, pp.get('CREATED', None))
     if createds is None:
         # TODO replace with 'match', but need to strip off priority etc first?
         # see _parse_heading in orgparse
         # todo maybe use n.get_timestamps(inactive=True, point=True)? only concern is that it's searching in the body as well?
         m = CREATED_RGX.search(heading)
         if m is not None:
-            createds = m.group(0) # could be None
+            createds = m.group(0)  # could be None
             # todo a bit hacky..
             heading = heading.replace(createds + ' ', '')
     if createds is not None:
@@ -72,7 +75,11 @@ def _parse_node(n: OrgNode) -> Parsed:
             dt = None
         else:
             [odt] = OrgDate.list_from_str(createds)
-            dt = odt.start
+            start = odt.start
+            if not isinstance(start, datetime):  # could be date
+                dt = datetime.combine(start, datetime.min.time())  # meh, but the best we can do?
+            else:
+                dt = start
     else:
         dt = None
     return Parsed(dt=dt, heading=heading)
@@ -160,7 +167,7 @@ def extract_from_file(fname: PathIsh) -> Results:
 
         (parsed, node) = wr
         dt = parsed.dt
-        assert dt is not None # shouldn't be because of fallback
+        assert dt is not None  # shouldn't be because of fallback
         for r in iter_org_urls(node):
             # TODO get body recursively? not sure
             try:
@@ -170,7 +177,7 @@ def extract_from_file(fname: PathIsh) -> Results:
                 ctx = parsed.heading + tagss + '\n' + get_body_compat(node)
             except Exception as e:
                 yield e
-                ctx = 'ERROR' # TODO more context?
+                ctx = 'ERROR'  # TODO more context?
 
             if isinstance(r, Url):
                 yield Visit(
@@ -178,9 +185,11 @@ def extract_from_file(fname: PathIsh) -> Results:
                     dt=dt,
                     locator=Loc.file(
                         fname,
-                        line=getattr(node, 'linenumber', None), # make it defensive so it works against older orgparse (pre 0.2)
+                        line=getattr(
+                            node, 'linenumber', None
+                        ),  # make it defensive so it works against older orgparse (pre 0.2)
                     ),
                     context=ctx,
                 )
-            else: # error
+            else:  # error
                 yield r

promnesia/sources/plaintext.py

@@ -3,7 +3,7 @@ from __future__ import annotations
 from functools import lru_cache
 from pathlib import Path
 
-from promnesia.common import PathIsh, _is_windows, get_logger, get_tmpdir
+from promnesia.common import PathIsh, _is_windows
 
 # https://linux-and-mac-hacks.blogspot.co.uk/2013/04/use-grep-and-regular-expressions-to.html
 _URL_REGEX = r'\b(https?|ftp|file)://[-A-Za-z0-9+&@#/%?=~_|!:,.;]*[-A-Za-z0-9+&@#/%=~_|]'
@@ -17,6 +17,7 @@ if _is_windows:
 @lru_cache
 def _has_grep() -> bool:
     import shutil
+
     return shutil.which('grep') is not None
 
 
@@ -25,9 +26,9 @@ Command = list[str]
 
 _GREP_ARGS: Command = [
     '--color=never',
-    '-H', # always show filename TODO not sure if works on osx
-    '-n', # print line numbers (to restore context)
-    '-I', # ignore binaries
+    '-H',  # always show filename TODO not sure if works on osx
+    '-n',  # print line numbers (to restore context)
+    '-I',  # ignore binaries
 ]
 
 if not _is_windows:
@@ -36,6 +37,7 @@ if not _is_windows:
         '--exclude-dir=".git"',
     ]
 
+
 # NOTE: grep/findstr exit with code 1 on no matches...
 # we hack around it in shellcmd module (search 'grep')
 def _grep(*, paths: list[str], recursive: bool) -> Command:
@@ -43,11 +45,12 @@ def _grep(*, paths: list[str], recursive: bool) -> Command:
         'grep',
         *(['-r'] if recursive else []),
         *_GREP_ARGS,
-        '-E', # 'extended' syntax
+        '-E',  # 'extended' syntax
         _URL_REGEX,
         *paths,
     ]
 
+
 def _findstr(*, path: str, recursive: bool) -> Command:
     return [
         'findstr',
@@ -85,18 +88,18 @@ def _extract_from_file(path: str) -> Command:
 def extract_from_path(path: PathIsh) -> Command:
     pp = Path(path)
 
-    tdir = get_tmpdir()
-
-    logger = get_logger()
-    if pp.is_dir(): # TODO handle archives here???
+    if pp.is_dir():  # TODO handle archives here???
         return _extract_from_dir(str(pp))
 
-    if any(pp.suffix == ex for ex in (
+    if any(
+        pp.suffix == ex
+        for ex in (
             '.xz',
             '.bz2',
             '.gz',
             '.zip',
-    )):
+        )
+    ):
         # todo should be debug?
         # or should delete it completely, feels like unpacking archives here is a bit too much
         raise RuntimeError(f"Archives aren't supported yet: {path}")

promnesia/sources/pocket.py

@@ -16,7 +16,8 @@ def index() -> Results:
         loc = Loc.make(title=title, href=a.pocket_link)
         # Add a reverse locator so that the Promnesia browser extension shows a
         # link on the Pocket page back to the original URL.
-        loc_rev = Loc.make(title=title, href=a.url)
+        # FIXME need to actually use it
+        _loc_rev = Loc.make(title=title, href=a.url)
         hls = a.highlights
         excerpt = a.json.get('excerpt', None)
         if len(hls) == 0:

promnesia/sources/reddit.py

@@ -21,6 +21,7 @@ def index(*, render_markdown: bool = False, renderer: type[RedditRenderer] | Non
     except ModuleNotFoundError as e:
         if "No module named 'my.reddit.all'" in str(e):
             import warnings
+
             warnings.warn("DEPRECATED/reddit: Using an old version of HPI, please update")
             from my.reddit import comments, saved, submissions, upvoted
         else:
@@ -69,6 +70,7 @@ class RedditRenderer:
         self._parser_cls = None
         try:
             from .markdown import TextParser, extract_from_text
+
             self._link_extractor = extract_from_text
             self._parser_cls = TextParser
         except ImportError as import_err:
@@ -78,11 +80,12 @@ class RedditRenderer:
             # only send error if the user is trying to enable this feature
             if render_markdown:
                 logger.exception(import_err)
-                logger.critical("Could not import markdown module to render reddit markdown. Try 'python3 -m pip install mistletoe'")
+                logger.critical(
+                    "Could not import markdown module to render reddit markdown. Try 'python3 -m pip install mistletoe'"
+                )
             render_markdown = False  # force to be false, couldn't import
         self.render_markdown = render_markdown
 
-
     def _from_comment(self, i: Comment) -> Results:
         locator = Loc.make(
             title='Reddit comment',
@@ -90,7 +93,6 @@ class RedditRenderer:
         )
         yield from self._from_common(i, locator=locator)
 
-
     def _from_submission(self, i: Submission) -> Results:
         locator = Loc.make(
             title=f'Reddit submission: {i.title}',
@@ -98,7 +100,6 @@ class RedditRenderer:
         )
         yield from self._from_common(i, locator=locator)
 
-
     def _from_upvote(self, i: Upvote) -> Results:
         locator = Loc.make(
             title='Reddit upvote',
@@ -106,7 +107,6 @@ class RedditRenderer:
         )
         yield from self._from_common(i, locator=locator)
 
-
     def _from_save(self, i: Save) -> Results:
         locator = Loc.make(
             title='Reddit save',
@@ -114,7 +114,6 @@ class RedditRenderer:
         )
         yield from self._from_common(i, locator=locator)
 
-
     # to allow for possible subclassing by the user?
     def _render_body(self, text: str) -> str:
         if self.render_markdown and self._parser_cls is not None:
@@ -122,7 +121,6 @@ class RedditRenderer:
         else:
             return text
 
-
     def _from_common(self, i: RedditBase, locator: Loc) -> Results:
         urls = [i.url]
         # TODO this should belong to HPI.. fix permalink handling I guess
@@ -170,4 +168,3 @@ class RedditRenderer:
                     locator=locator,
                 )
                 emitted.add(res.url)
-

promnesia/sources/roamresearch.py

@@ -7,6 +7,7 @@ from promnesia.common import Loc, Results, Visit, extract_urls
 
 def index() -> Results:
     import my.roamresearch as RR
+
     roam = RR.roam()
     for node in roam.traverse():
         yield from _collect(node)
@@ -14,7 +15,7 @@ def index() -> Results:
 
 def _collect(node: 'RoamNode') -> Results:
     title = node.title
-    body  = node.body or ''
+    body = node.body or ''
     if title is None:
         # most notes don't have title, so we just take the first line instead..
         lines = body.splitlines(keepends=True)
@@ -46,4 +47,5 @@ import typing
 
 if typing.TYPE_CHECKING:
     import my.roamresearch as RR
+
     RoamNode = RR.Node

promnesia/sources/rss.py

@@ -2,17 +2,16 @@
 Uses [[https://github.com/karlicoss/HPI][HPI]] for RSS data.
 '''
 
-from datetime import datetime
-
-import pytz
+from datetime import datetime, timezone
 
 from promnesia.common import Loc, Results, Visit
 
 # arbitrary,  2011-11-04 00:05:23.283+00:00
-default_datetime = datetime.fromtimestamp(1320365123, tz=pytz.utc)
+default_datetime = datetime.fromtimestamp(1320365123, tz=timezone.utc)
 # TODO FIXME allow for visit not to have datetime?
 # I.e. even having context is pretty good!
 
+
 def index() -> Results:
     from my.rss.all import subscriptions
 
@@ -22,6 +21,6 @@ def index() -> Results:
         yield Visit(
             url=feed.url,
             dt=feed.created_at or default_datetime,
-            context='RSS subscription', # TODO use 'provider', etc?
+            context='RSS subscription',  # TODO use 'provider', etc?
             locator=locator,
         )

promnesia/sources/shellcmd.py

@@ -19,7 +19,6 @@ from promnesia.common import (
     _is_windows,
     extract_urls,
     file_mtime,
-    get_system_tz,
     now_tz,
 )
 
@@ -37,8 +36,6 @@ def index(command: str | Sequence[PathIsh]) -> Results:
         cmds = ' '.join(map(str, command))
         cmd = command
 
-    tz = get_system_tz()
-
     # ugh... on windows grep does something nasty? e.g:
     # grep --color=never -r -H -n -I -E http 'D:\\a\\promnesia\\promnesia\\tests\\testdata\\custom'
     # D:\a\promnesia\promnesia\tests\testdata\custom/file1.txt:1:Right, so this points at http://google.com
@@ -55,9 +52,9 @@ def index(command: str | Sequence[PathIsh]) -> Results:
             fname = None
             lineno = None
         else:
-            fname  = m.group(1)
+            fname = m.group(1)
             lineno = int(m.group(2))
-            line   = m.group(3)
+            line = m.group(3)
 
         if fname is not None and needs_windows_grep_patching:
             fname = fname.replace('/', os.sep)
@@ -86,7 +83,7 @@ def index(command: str | Sequence[PathIsh]) -> Results:
 
     r = run(cmd, stdout=PIPE, check=False)
     if r.returncode > 0:
-        if not (cmd[0] in {'grep', 'findstr'} and r.returncode == 1): # ugh. grep returns 1 on no matches...
+        if not (cmd[0] in {'grep', 'findstr'} and r.returncode == 1):  # ugh. grep returns 1 on no matches...
             r.check_returncode()
     output = r.stdout
     assert output is not None

promnesia/sources/signal.py

@@ -1,6 +1,7 @@
 """
 Collects visits from Signal Desktop's encrypted SQLIite db(s).
 """
+
 from __future__ import annotations
 
 # Functions get their defaults from module-data.
@@ -17,17 +18,15 @@ from collections.abc import Iterable, Iterator, Mapping
 from contextlib import contextmanager
 from pathlib import Path
 from textwrap import dedent, indent
-from typing import Any, Union
+from typing import Any
 
 from ..common import Loc, PathIsh, Results, Visit, extract_urls, from_epoch
 
-PathIshes = Union[PathIsh, Iterable[PathIsh]]
-
 
 def index(
     *db_paths: PathIsh,
     http_only: bool = False,
-    locator_schema: str="editor",
+    locator_schema: str = "editor",
     append_platform_path: bool = False,
     override_key: str | None = None,
 ) -> Results:
@@ -51,8 +50,7 @@ def index(
         otherwise, this same key is used for harvesting all db-files.
     """
     logger.debug(
-        "http_only?(%s), locator_schema?(%s), append_platform_path?(%s), "
-        "overide_key given?(%s), db_paths: %s",
+        "http_only?(%s), locator_schema?(%s), append_platform_path?(%s), overide_key given?(%s), db_paths: %s",
         http_only,
         locator_schema,
         append_platform_path,
@@ -171,7 +169,10 @@ def _expand_path(path_pattern: PathIsh) -> Iterable[Path]:
 
     Expansion code adapted from https://stackoverflow.com/a/51108375/548792
     to handle also degenerate cases (``'', '.', '/'``):
+    """
 
+    # NOTE: suppressing doctest from github actions
+    """
     >>> str(next(iter(_get_files('/'))))
     '/'
 
@@ -194,7 +195,7 @@ def _expand_path(path_pattern: PathIsh) -> Iterable[Path]:
     return path.glob(str(Path(*parts))) if parts else [path]
 
 
-def _expand_paths(paths: PathIshes) -> Iterable[Path]:
+def _expand_paths(paths: PathIsh | Iterable[PathIsh]) -> Iterable[Path]:
     if _is_pathish(paths):
         paths = [paths]  # type: ignore[list-item]
     return [pp.resolve() for p in paths for pp in _expand_path(p)]  # type: ignore[union-attr]
@@ -214,7 +215,10 @@ def collect_db_paths(*db_paths: PathIsh, append: bool = False) -> Iterable[Path]
         one or more pathish
 
     Note: needed `append` here, to resolve paths.
+    """
 
+    # NOTE: suppressing doctest from running on Github actions
+    """
     >>> bool(collect_db_paths())  # my home-path
     True
     >>> collect_db_paths(None)
@@ -239,8 +243,7 @@ def collect_db_paths(*db_paths: PathIsh, append: bool = False) -> Iterable[Path]
             plat_paths = platform_db_paths[platform_name]
         except LookupError as le:
             raise ValueError(
-                f"Unknown platform({platform_name}!"
-                f"\n  Expected one of {list(platform_db_paths.keys())}."
+                f"Unknown platform({platform_name}!\n  Expected one of {list(platform_db_paths.keys())}."
             ) from le
 
         if db_paths and append:
@@ -325,9 +328,7 @@ def connect_db(
             )
             sql = "\n".join(sql_cmds)
             cmd = [sqlcipher_exe, str(db_path)]
-            logger.debug(
-                "Decrypting db '%s' with cmd: %s <<<EOF\n%s\nEOF", db_path, cmd, sql
-            )
+            logger.debug("Decrypting db '%s' with cmd: %s <<<EOF\n%s\nEOF", db_path, cmd, sql)
             try:
                 sbp.run(
                     cmd,
@@ -358,12 +359,11 @@ def connect_db(
         yield db
     finally:
         try:
-            if db:
+            if db is not None:
                 db.close()
         finally:
             if decrypted_file and decrypted_file.exists():
                 try:
-
                     logger.debug("Deleting temporary decrypted db: %s", decrypted_file)
                     decrypted_file.unlink()
                 except Exception as ex:

promnesia/sources/smscalls.py

@@ -10,7 +10,6 @@ def index() -> Results:
     from my.smscalls import messages
 
     for m in messages():
-
         if isinstance(m, Exception):
             yield m
             continue

promnesia/sources/stackexchange.py

@@ -16,7 +16,7 @@ def index() -> Results:
             yield Visit(
                 url=v.link,
                 dt=v.when,
-                context='voted', # todo use the votetype? although maybe worth ignoring downvotes
+                context='voted',  # todo use the votetype? although maybe worth ignoring downvotes
                 # or, downvotes could have 'negative' ranking or something
-                locator=Loc.make(title='voted', href=v.link)
+                locator=Loc.make(title='voted', href=v.link),
             )

promnesia/sources/takeout.py

@@ -35,13 +35,15 @@ def index() -> Results:
         logger.exception(ex)
         yield ex
 
-        warnings.warn("Please set up my.google.takeout.parser module for better takeout support. Falling back to legacy implementation.")
+        warnings.warn(
+            "Please set up my.google.takeout.parser module for better takeout support. Falling back to legacy implementation."
+        )
 
         from . import takeout_legacy
+
         yield from takeout_legacy.index()
         return
 
-
     _seen: set[str] = {
         # these are definitely not useful for promnesia
         'Location',
@@ -52,10 +54,13 @@ def index() -> Results:
     imported_yt_csv_models = False
     try:
         from google_takeout_parser.models import CSVYoutubeComment, CSVYoutubeLiveChat
+
         imported_yt_csv_models = True
     except ImportError:
         # warn user to upgrade google_takeout_parser
-        warnings.warn("Please upgrade google_takeout_parser (`pip install -U google_takeout_parser`) to support the new format for youtube comments")
+        warnings.warn(
+            "Please upgrade google_takeout_parser (`pip install -U google_takeout_parser`) to support the new format for youtube comments"
+        )
         CSVYoutubeComment = YoutubeCSVStub  # type: ignore[misc,assignment]
         CSVYoutubeLiveChat = YoutubeCSVStub  # type: ignore[misc,assignment]
 
@@ -130,16 +135,12 @@ def index() -> Results:
         elif isinstance(e, LikedYoutubeVideo):
             # TODO not sure if desc makes sense here since it's not user produced data
             # it's just a part of video meta?
-            yield Visit(
-                url=e.link, dt=e.dt, context=e.desc, locator=Loc(title=e.title, href=e.link)
-            )
+            yield Visit(url=e.link, dt=e.dt, context=e.desc, locator=Loc(title=e.title, href=e.link))
         elif isinstance(e, YoutubeComment):
             for url in e.urls:
                 # todo: use url_metadata to improve locator?
                 # or maybe just extract first sentence?
-                yield Visit(
-                    url=url, dt=e.dt, context=e.content, locator=Loc(title=e.content, href=url)
-                )
+                yield Visit(url=url, dt=e.dt, context=e.content, locator=Loc(title=e.content, href=url))
         elif imported_yt_csv_models and isinstance(e, CSVYoutubeComment):
             contentJSON = e.contentJSON
             content = reconstruct_comment_content(contentJSON, format='text')
@@ -152,12 +153,8 @@ def index() -> Results:
                 continue
             context = f"Commented on {e.video_url}"
             for url in links:
-                yield Visit(
-                    url=url, dt=e.dt, context=content, locator=Loc(title=context, href=url)
-                )
-            yield Visit(
-                url=e.video_url, dt=e.dt, context=content, locator=Loc(title=context, href=e.video_url)
-            )
+                yield Visit(url=url, dt=e.dt, context=content, locator=Loc(title=context, href=url))
+            yield Visit(url=e.video_url, dt=e.dt, context=content, locator=Loc(title=context, href=e.video_url))
         elif imported_yt_csv_models and isinstance(e, CSVYoutubeLiveChat):
             contentJSON = e.contentJSON
             content = reconstruct_comment_content(contentJSON, format='text')
@@ -170,12 +167,8 @@ def index() -> Results:
                 continue
             context = f"Commented on livestream {e.video_url}"
             for url in links:
-                yield Visit(
-                    url=url, dt=e.dt, context=content, locator=Loc(title=context, href=url)
-                )
-            yield Visit(
-                url=e.video_url, dt=e.dt, context=content, locator=Loc(title=context, href=e.video_url)
-            )
+                yield Visit(url=url, dt=e.dt, context=content, locator=Loc(title=context, href=url))
+            yield Visit(url=e.video_url, dt=e.dt, context=content, locator=Loc(title=context, href=e.video_url))
         else:
             yield from warn_once_if_not_seen(e)