oldnews 0.3.0__tar.gz → 0.5.0__tar.gz

{oldnews-0.3.0 → oldnews-0.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: oldnews
-Version: 0.3.0
+Version: 0.5.0
 Summary: A terminal-based client for TheOldReader
 Keywords: atom,client,Google Reader,RSS,TheOldReader,terminal,news-reader,news
 Author: Dave Pearson
@@ -15,8 +15,9 @@ Classifier: Programming Language :: Python :: 3.13
 Classifier: Programming Language :: Python :: 3.14
 Classifier: Typing :: Typed
 Requires-Dist: html-to-markdown>=2.15.0
-Requires-Dist: oldas>=0.3.1
+Requires-Dist: oldas>=0.6.0
 Requires-Dist: textual>=6.3.0
+Requires-Dist: textual-autocomplete>=4.0.6
 Requires-Dist: textual-enhanced>=1.2.0
 Requires-Dist: typedal>=4.2.2
 Requires-Dist: xdg-base-dirs>=6.0.2

{oldnews-0.3.0 → oldnews-0.5.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "oldnews"
-version = "0.3.0"
+version = "0.5.0"
 description = "A terminal-based client for TheOldReader"
 readme = "README.md"
 authors = [
@@ -20,8 +20,9 @@ keywords = [
 ]
 dependencies = [
     "html-to-markdown>=2.15.0",
-    "oldas>=0.3.1",
+    "oldas>=0.6.0",
     "textual>=6.3.0",
+    "textual-autocomplete>=4.0.6",
     "textual-enhanced>=1.2.0",
     "typedal>=4.2.2",
     "xdg-base-dirs>=6.0.2",

{oldnews-0.3.0 → oldnews-0.5.0}/src/oldnews/commands/__init__.py

@@ -3,30 +3,48 @@
 ##############################################################################
 # Local imports.
 from .main import (
+    AddSubscription,
+    Copy,
+    CopyArticleToClipboard,
+    CopyFeedToClipboard,
+    CopyHomePageToClipboard,
     Escape,
+    Information,
     MarkAllRead,
+    MoveSubscription,
     Next,
     NextUnread,
     OpenArticle,
-    OpenOrigin,
+    OpenHomePage,
     Previous,
     PreviousUnread,
     RefreshFromTheOldReader,
+    Remove,
+    Rename,
     ToggleShowAll,
 )
 
 ##############################################################################
 # Exports.
 __all__ = [
+    "AddSubscription",
+    "Copy",
+    "CopyArticleToClipboard",
+    "CopyFeedToClipboard",
+    "CopyHomePageToClipboard",
     "Escape",
+    "Information",
     "MarkAllRead",
     "Next",
     "NextUnread",
+    "MoveSubscription",
     "OpenArticle",
-    "OpenOrigin",
+    "OpenHomePage",
     "Previous",
     "PreviousUnread",
     "RefreshFromTheOldReader",
+    "Rename",
+    "Remove",
     "ToggleShowAll",
 ]
 

{oldnews-0.3.0 → oldnews-0.5.0}/src/oldnews/commands/main.py

@@ -64,8 +64,8 @@ class OpenArticle(Command):
 
 
 ##############################################################################
-class OpenOrigin(Command):
-    """Open the origin for the current article in the web browser"""
+class OpenHomePage(Command):
+    """Open the home page for the current subscription in the web browser"""
 
     BINDING_KEY = "O"
 
@@ -77,4 +77,67 @@ class MarkAllRead(Command):
     BINDING_KEY = "R"
 
 
+##############################################################################
+class CopyHomePageToClipboard(Command):
+    """Copy the URL of the current subscription's home page to the clipboard"""
+
+    BINDING_KEY = "f3"
+
+
+##############################################################################
+class CopyFeedToClipboard(Command):
+    """Copy the URL of the current subscription's feed to the clipboard"""
+
+    BINDING_KEY = "f4"
+
+
+##############################################################################
+class CopyArticleToClipboard(Command):
+    """Copy the URL for the current article to the clipboard"""
+
+    BINDING_KEY = "f5"
+
+
+##############################################################################
+class Copy(Command):
+    """Copy a URL to the clipboard depending on the context"""
+
+    BINDING_KEY = "ctrl+c"
+
+
+##############################################################################
+class AddSubscription(Command):
+    """Add a subscription feed"""
+
+    BINDING_KEY = "plus"
+
+
+##############################################################################
+class Rename(Command):
+    """Rename the current folder or subscription"""
+
+    BINDING_KEY = "f6"
+
+
+##############################################################################
+class Remove(Command):
+    """Remove the current folder or subscription"""
+
+    BINDING_KEY = "delete"
+
+
+##############################################################################
+class MoveSubscription(Command):
+    """Move the current subscription to folder"""
+
+    BINDING_KEY = "f7"
+
+
+##############################################################################
+class Information(Command):
+    """Show low-level information about the selected item"""
+
+    BINDING_KEY = "f8"
+
+
 ### main.py ends here

{oldnews-0.3.0 → oldnews-0.5.0}/src/oldnews/data/__init__.py

@@ -17,6 +17,10 @@ from .local_articles import (
     get_unread_article_ids,
     locally_mark_article_ids_read,
     locally_mark_read,
+    move_subscription_articles,
+    remove_folder_from_articles,
+    remove_subscription_articles,
+    rename_folder_for_articles,
     save_local_articles,
 )
 from .local_folders import get_local_folders, save_local_folders
@@ -43,7 +47,11 @@ __all__ = [
     "locally_mark_read",
     "locally_mark_article_ids_read",
     "LocalUnread",
+    "move_subscription_articles",
     "remember_we_last_grabbed_at",
+    "remove_folder_from_articles",
+    "remove_subscription_articles",
+    "rename_folder_for_articles",
     "reset_data",
     "save_configuration",
     "save_local_articles",

{oldnews-0.3.0 → oldnews-0.5.0}/src/oldnews/data/db.py

@@ -3,14 +3,11 @@
 ##############################################################################
 # Python imports.
 from pathlib import Path
-from typing import Any
 
 ##############################################################################
 # TypeDAL imports.
-from typedal import TypeDAL, TypedField, TypedTable
+from typedal import TypeDAL
 from typedal.config import TypeDALConfig
-from typedal.helpers import get_field
-from typedal.types import Field
 
 ##############################################################################
 # Local imports.
@@ -20,6 +17,7 @@ from .local_folders import LocalFolder
 from .local_subscriptions import LocalSubscription, LocalSubscriptionCategory
 from .locations import data_dir
 from .navigation_state import NavigationState
+from .tools import safely_index
 
 
 ##############################################################################
@@ -32,31 +30,6 @@ def db_file() -> Path:
     return data_dir() / "oldnews.db"
 
 
-##############################################################################
-def _safely_index(
-    table: type[TypedTable], name: str, field: str | Field | TypedField[Any]
-) -> None:
-    """Create an index on a type, but handle errors.
-
-    Args:
-        table: The table to create the index against.
-        name: The name of the index.
-        field: The field to index.
-
-    Notes:
-        From what I can gather TypeDAL *should* only create the index if it
-        doesn't exist. Instead it throws an error if it exists. So here I
-        swallow the `RuntimeError`. Hopefully there is a better way and I've
-        just missed it.
-    """
-    try:
-        table.create_index(
-            name, get_field(field) if isinstance(field, TypedField) else field
-        )
-    except RuntimeError:
-        pass
-
-
 ##############################################################################
 def initialise_database() -> TypeDAL:
     """Create the database.
@@ -73,20 +46,20 @@ def initialise_database() -> TypeDAL:
     dal = TypeDAL(f"sqlite://{db_file()}", folder=data_dir(), config=TypeDALConfig())
 
     dal.define(LocalArticle)
-    _safely_index(LocalArticle, "idx_local_article_article_id", LocalArticle.article_id)
-    _safely_index(
+    safely_index(LocalArticle, "idx_local_article_article_id", LocalArticle.article_id)
+    safely_index(
         LocalArticle,
         "idx_local_article_origin_stream_id",
         LocalArticle.origin_stream_id,
     )
 
     dal.define(LocalArticleCategory)
-    _safely_index(
+    safely_index(
         LocalArticleCategory,
         "idx_local_article_category_article",
         LocalArticleCategory.article,
     )
-    _safely_index(
+    safely_index(
         LocalArticleCategory,
         "idx_local_article_category_category",
         LocalArticleCategory.category,
@@ -97,19 +70,19 @@ def initialise_database() -> TypeDAL:
     dal.define(LocalFolder)
 
     dal.define(LocalSubscription)
-    _safely_index(
+    safely_index(
         LocalSubscription,
         "idx_local_subscription_subscription_id",
         LocalSubscription.subscription_id,
     )
 
     dal.define(LocalSubscriptionCategory)
-    _safely_index(
+    safely_index(
         LocalSubscriptionCategory,
         "idx_local_subscription_category_subscription",
         LocalSubscriptionCategory.subscription,
     )
-    _safely_index(
+    safely_index(
         LocalSubscriptionCategory,
         "idx_local_subscription_category_category_id",
         LocalSubscriptionCategory.category_id,

{oldnews-0.3.0 → oldnews-0.5.0}/src/oldnews/data/last_grab.py

@@ -8,6 +8,10 @@ from datetime import datetime, timezone
 # TypeDAL imports.
 from typedal import TypedTable
 
+##############################################################################
+# Local imports.
+from .tools import commit
+
 
 ##############################################################################
 class LastGrabbed(TypedTable):
@@ -39,10 +43,9 @@ def remember_we_last_grabbed_at(grab_time: datetime | None = None) -> None:
     Note:
         If `grab_time` isn't supplied then it is recorded as now.
     """
-    assert LastGrabbed._db is not None
     LastGrabbed.truncate()
     LastGrabbed.insert(at_time=grab_time or datetime.now(timezone.utc))
-    LastGrabbed._db.commit()
+    commit(LastGrabbed)
 
 
 ### last_grab.py ends here

{oldnews-0.3.0 → oldnews-0.5.0}/src/oldnews/data/local_articles.py

@@ -8,13 +8,17 @@ from typing import Iterable, Iterator, cast
 
 ##############################################################################
 # OldAS imports.
-from oldas import Article, Articles, Folder, State, Subscription
+from oldas import Article, Articles, Folder, Folders, State, Subscription
 from oldas.articles import Alternate, Alternates, Direction, Origin, Summary
 
 ##############################################################################
 # TypeDAL imports.
 from typedal import TypedField, TypedTable, relationship
 
+##############################################################################
+# Local imports.
+from .tools import commit
+
 
 ##############################################################################
 class LocalArticle(TypedTable):
@@ -62,9 +66,8 @@ class LocalArticle(TypedTable):
             category: The category to add.
         """
         if not str(category) in self.categories:
-            assert LocalArticleCategory._db is not None
             LocalArticleCategory.insert(article=self.id, category=str(category))
-            LocalArticleCategory._db.commit()
+            commit(LocalArticleCategory)
 
     def remove_category(self, category: str | State) -> None:
         """Remove a given category from the local article.
@@ -73,12 +76,11 @@ class LocalArticle(TypedTable):
             category: The category to add.
         """
         if str(category) in self.categories:
-            assert LocalArticleCategory._db is not None
             LocalArticleCategory.where(
                 (LocalArticleCategory.article == self.id)
                 & (LocalArticleCategory.category == str(category))
             ).delete()
-            LocalArticleCategory._db.commit()
+            commit(LocalArticleCategory)
 
 
 ##############################################################################
@@ -113,7 +115,6 @@ def save_local_articles(articles: Articles) -> Articles:
     Returns:
         The articles.
     """
-    assert LocalArticle._db is not None
     for article in articles:
         local_article = LocalArticle.update_or_insert(
             LocalArticle.article_id == article.id,
@@ -146,7 +147,7 @@ def save_local_articles(articles: Articles) -> Articles:
                 for alternate in article.alternate
             ]
         )
-    LocalArticle._db.commit()
+    commit(LocalArticle)
     return articles
 
 
@@ -336,7 +337,6 @@ def get_unread_article_ids() -> list[str]:
 ##############################################################################
 def clean_old_read_articles(cutoff: timedelta) -> int:
     """Clean up articles that are older than the given cutoff time."""
-    assert LocalArticle._db is not None
     read = get_local_read_article_ids()
     retire_time = datetime.now() - cutoff
     cleaned = len(
@@ -344,8 +344,77 @@ def clean_old_read_articles(cutoff: timedelta) -> int:
             (LocalArticle.published < retire_time) & LocalArticle.id.belongs(read)
         ).delete()
     )
-    LocalArticle._db.commit()
+    commit(LocalArticle)
     return cleaned
 
 
+##############################################################################
+def rename_folder_for_articles(rename_from: str | Folder, rename_to: str) -> None:
+    """Rename a folder for all articles that are in that folder.
+
+    Args:
+        rename_from: The folder name to rename from.
+        rename_to: The folder name to rename to.
+    """
+    rename_from = Folders.full_id(rename_from)
+    rename_to = Folders.full_id(rename_to)
+    LocalArticleCategory.where(LocalArticleCategory.category == rename_from).update(
+        category=rename_to
+    )
+    commit(LocalArticleCategory)
+
+
+##############################################################################
+def remove_folder_from_articles(folder: str | Folder) -> None:
+    """Remove a folder from being associated with all articles.
+
+    Args:
+        folder: The folder to remove from all articles.
+    """
+    LocalArticleCategory.where(
+        LocalArticleCategory.category == Folders.full_id(folder)
+    ).delete()
+    commit(LocalArticleCategory)
+
+
+##############################################################################
+def move_subscription_articles(
+    subscription: Subscription,
+    from_folder: str | Folder | None,
+    to_folder: str | Folder | None,
+) -> None:
+    """Move the articles of a subscription from one folder to another.
+
+    Args:
+        subscription: The subscription whose articles we should move.
+        from_folder: The folder to move from.
+        to_folder: The folder to move to.
+    """
+    from_folder = (
+        Folders.full_id(from_folder) if from_folder is not None else from_folder
+    )
+    to_folder = Folders.full_id(to_folder) if to_folder is not None else to_folder
+    for article in LocalArticle.where(origin_stream_id=subscription.id).join().select():
+        if from_folder:
+            LocalArticleCategory.where(
+                (LocalArticleCategory.article == article.id)
+                & (LocalArticleCategory.category == from_folder)
+            ).delete()
+            commit(LocalArticleCategory)
+        if to_folder:
+            LocalArticleCategory.insert(article=article.id, category=to_folder)
+            commit(LocalArticleCategory)
+
+
+##############################################################################
+def remove_subscription_articles(subscription: Subscription) -> None:
+    """Remove all the articles associated with the given subscription.
+
+    Args:
+        subscription: The subscription to remove the articles for.
+    """
+    LocalArticle.where(origin_stream_id=subscription.id).delete()
+    commit(LocalArticle)
+
+
 ### local_articles.py ends here

{oldnews-0.3.0 → oldnews-0.5.0}/src/oldnews/data/local_folders.py

@@ -8,6 +8,10 @@ from oldas import Folder, Folders
 # TypeDAL imports.
 from typedal import TypedTable
 
+##############################################################################
+# Local imports.
+from .tools import commit
+
 
 ##############################################################################
 class LocalFolder(TypedTable):
@@ -42,12 +46,11 @@ def save_local_folders(folders: Folders) -> Folders:
     Returns:
         The folders.
     """
-    assert LocalFolder._db is not None
     LocalFolder.truncate()
     LocalFolder.bulk_insert(
         [{"folder_id": folder.id, "sort_id": folder.sort_id} for folder in folders]
     )
-    LocalFolder._db.commit()
+    commit(LocalFolder)
     return folders
 
 

{oldnews-0.3.0 → oldnews-0.5.0}/src/oldnews/data/local_subscriptions.py

@@ -13,6 +13,10 @@ from oldas.subscriptions import Categories, Category
 # TypeDAL imports.
 from typedal import TypedTable
 
+##############################################################################
+# Local imports.
+from .tools import commit
+
 
 ##############################################################################
 class LocalSubscription(TypedTable):
@@ -83,29 +87,33 @@ def save_local_subscriptions(subscriptions: Subscriptions) -> Subscriptions:
     Returns:
         The subscriptions.
     """
-    assert LocalSubscription._db is not None
-    for subscription in subscriptions:
-        LocalSubscriptionCategory.where(subscription=subscription.id).delete()
-        LocalSubscriptionCategory.bulk_insert(
-            [
-                {
-                    "subscription": subscription.id,
-                    "category_id": category.id,
-                    "label": category.label,
-                }
-                for category in subscription.categories
-            ]
-        )
-        LocalSubscription.update_or_insert(
-            LocalSubscription.subscription_id == subscription.id,
-            subscription_id=subscription.id,
-            title=subscription.title,
-            sort_id=subscription.sort_id,
-            first_item_time=subscription.first_item_time,
-            url=subscription.url,
-            html_url=subscription.html_url,
-        )
-    LocalSubscription._db.commit()
+    LocalSubscription.truncate()
+    LocalSubscriptionCategory.truncate()
+    LocalSubscription.bulk_insert(
+        [
+            {
+                "subscription_id": subscription.id,
+                "title": subscription.title,
+                "sort_id": subscription.sort_id,
+                "first_item_time": subscription.first_item_time,
+                "url": subscription.url,
+                "html_url": subscription.html_url,
+            }
+            for subscription in subscriptions
+        ]
+    )
+    LocalSubscriptionCategory.bulk_insert(
+        [
+            {
+                "subscription": subscription.id,
+                "category_id": category.id,
+                "label": category.label,
+            }
+            for subscription in subscriptions
+            for category in subscription.categories
+        ]
+    )
+    commit(LocalSubscription)
     return subscriptions
 
 

{oldnews-0.3.0 → oldnews-0.5.0}/src/oldnews/data/local_unread.py

@@ -2,7 +2,7 @@
 
 ##############################################################################
 # OldAS imports.
-from oldas import Folders, Subscriptions, id_is_a_folder
+from oldas import Folders, Subscriptions, id_is_a_feed
 
 ##############################################################################
 # Local imports.
@@ -41,7 +41,7 @@ def total_unread(unread: LocalUnread) -> int:
     Returns:
         The total unread.
     """
-    return sum(count for category, count in unread.items() if id_is_a_folder(category))
+    return sum(count for category, count in unread.items() if id_is_a_feed(category))
 
 
 ### local_unread.py ends here

{oldnews-0.3.0 → oldnews-0.5.0}/src/oldnews/data/navigation_state.py

@@ -4,6 +4,10 @@
 # TypeDAL imports.
 from typedal import TypedTable
 
+##############################################################################
+# Local imports.
+from .tools import commit
+
 
 ##############################################################################
 class NavigationState(TypedTable):
@@ -33,10 +37,9 @@ def save_navigation_state(state: set[str]) -> None:
     Args:
         state: The state to save.
     """
-    assert NavigationState._db is not None
     NavigationState.truncate()
     NavigationState.bulk_insert([{"expanded_folder_id": folder} for folder in state])
-    NavigationState._db.commit()
+    commit(NavigationState)
 
 
 ### navigation_state.py ends here

oldnews-0.5.0/src/oldnews/data/tools.py

@@ -0,0 +1,45 @@
+"""Database tool functions."""
+
+##############################################################################
+# Python imports.
+from typing import Any
+
+##############################################################################
+# TypeDAL imports.
+from typedal import TypedField, TypedTable
+from typedal.helpers import get_db, get_field
+from typedal.types import Field
+
+
+##############################################################################
+def safely_index(
+    table: type[TypedTable], name: str, field: str | Field | TypedField[Any]
+) -> None:
+    """Create an index on a type, but handle errors.
+
+    Args:
+        table: The table to create the index against.
+        name: The name of the index.
+        field: The field to index.
+
+    Notes:
+        From what I can gather TypeDAL *should* only create the index if it
+        doesn't exist. Instead it throws an error if it exists. So here I
+        swallow the `RuntimeError`. Hopefully there is a better way and I've
+        just missed it.
+    """
+    try:
+        table.create_index(
+            name, get_field(field) if isinstance(field, TypedField) else field
+        )
+    except RuntimeError:
+        pass
+
+
+##############################################################################
+def commit(table: type[TypedTable]) -> None:
+    """Commit changes."""
+    get_db(table()).commit()
+
+
+### tools.py ends here