cli-todo-jd 0.2.0__py3-none-any.whl → 0.2.1__py3-none-any.whl

cli_todo_jd/cli_entry.py

@@ -37,8 +37,43 @@ def add(
 @app.command(name="list")
 def list_(
     filepath: Path = typer.Option(Path(".todo_list.db"), "--filepath", "-f"),
+    show_all: bool = typer.Option(
+        False, "--all", "-a", help="Show all todos (open + done)."
+    ),
+    show_done: bool = typer.Option(
+        False, "--done", "-d", help="Show only completed todos."
+    ),
+    show_open: bool = typer.Option(
+        False, "--open", "-o", help="Show only open todos (default)."
+    ),
 ) -> None:
-    list_items_on_list(filepath)
+    """List todos.
+
+    Examples
+    --------
+    - todo list
+    - todo list --done
+    - todo list --all
+    - todo list -a
+    """
+
+    # Choose filter. If nothing specified, default to open.
+    # If the user specifies multiple flags, error out.
+    flags = [show_all, show_done, show_open]
+    if sum(1 for f in flags if f) > 1:
+        raise typer.BadParameter(
+            "Use only one of: --all / -a, --done / -d, --open / -o"
+        )
+
+    if show_all:
+        show = "all"
+    elif show_done:
+        show = "done"
+    else:
+        # default is open (or explicit --open)
+        show = "open"
+
+    list_items_on_list(filepath, show=show)
 
 
 @app.command()
@@ -80,7 +115,7 @@ def done(
     filepath: Path = typer.Option(Path(".todo_list.db"), "--filepath", "-f"),
 ) -> None:
     mark_item_as_done(index, filepath)
-    list_(filepath=filepath)
+    list_items_on_list(filepath=filepath, show="all")
 
 
 @app.command()
@@ -89,7 +124,7 @@ def not_done(
     filepath: Path = typer.Option(Path(".todo_list.db"), "--filepath", "-f"),
 ) -> None:
     mark_item_as_not_done(index, filepath)
-    list_(filepath=filepath)
+    list_items_on_list(filepath=filepath, show="all")
 
 
 def parser_optional_args(parser: ArgumentParser):

cli_todo_jd/main.py

@@ -45,13 +45,51 @@ class TodoApp:
         print(f'Added todo: "{item}"')
         self._check_and_load_todos(self.file_path_to_db)
 
-    def list_todos(self) -> None:
+    def list_todos(self, *, show: str = "open") -> None:
+        """List todos.
+
+        Parameters
+        ----------
+        show:
+            "open" (default), "done", or "all".
+        """
+        show = (show or "open").lower()
+        if show not in {"open", "done", "all"}:
+            print("Error: show must be one of: open, done, all")
+            return
+
         # Always read fresh so output reflects the DB
         self._check_and_load_todos(self.file_path_to_db)
         if not self.todos:
             print("No todos found.")
             return
-        self._table_print()
+
+        if show == "all":
+            self._table_print(title="Todos")
+            return
+
+        # Filter in-memory to keep this change minimal. (You can later filter in SQL.)
+        filtered_todos: list[str] = []
+        filtered_status: list[int] = []
+        for todo, done in zip(self.todos, self.status, strict=False):
+            if show == "open" and not done:
+                filtered_todos.append(todo)
+                filtered_status.append(done)
+            elif show == "done" and done:
+                filtered_todos.append(todo)
+                filtered_status.append(done)
+
+        if not filtered_todos:
+            print("No todos found.")
+            return
+
+        original_todos, original_status = self.todos, self.status
+        try:
+            self.todos, self.status = filtered_todos, filtered_status
+            title = "Open todos" if show == "open" else "Completed todos"
+            self._table_print(title=title)
+        finally:
+            self.todos, self.status = original_todos, original_status
 
     def remove_todo(self, index: int) -> None:
         # Maintain current UX: index refers to the displayed (1-based) ordering.
@@ -281,7 +319,7 @@ def create_list(file_path_to_db: str = "./.todo_list.db"):
 
     Parameters
     ----------
-    file_path_to_json : str, optional
+    file_path_to_db : str, optional
         The file path to the JSON file for storing todos, by default "./.todo_list.db"
 
     Returns
@@ -309,17 +347,18 @@ def add_item_to_list(item: str, filepath: str):
     app.list_todos()
 
 
-def list_items_on_list(filepath: str):
-    """
-    List all items in the todo list.
+def list_items_on_list(filepath: str, show: str = "open"):
+    """List items in the todo list.
 
     Parameters
     ----------
-    filepath : str
-        The file path to the JSON file for storing todos.
+    filepath:
+        The SQLite database path.
+    show:
+        "open" (default), "done", or "all".
     """
     app = create_list(file_path_to_db=filepath)
-    app.list_todos()
+    app.list_todos(show=show)
 
 
 def remove_item_from_list(index: int, filepath: str):
@@ -388,7 +427,7 @@ def cli_menu(filepath="./.todo_list.db"):
             item = questionary.text("Enter the todo item:").ask()
             app.add_todo(item)
         elif action == "List todos":
-            app.list_todos()
+            app.list_todos(show="all")
         elif action == "Update todo status":
             if not app.todos:
                 print("No todos to update.")
@@ -398,7 +437,7 @@ def cli_menu(filepath="./.todo_list.db"):
                 choices=["<Back>"] + app.todos,
             ).ask()
 
-            if todo_choice == "<Back>":
+            if todo_choice == "<Back>" or todo_choice is None:
                 continue
 
             todo_index = app.todos.index(todo_choice) + 1
@@ -407,13 +446,13 @@ def cli_menu(filepath="./.todo_list.db"):
                 choices=["Done", "Not Done", "<Back>"],
             ).ask()
 
-            if status_choice == "<Back>":
+            if status_choice == "<Back>" or status_choice is None:
                 continue
             elif status_choice == "Done":
                 app.mark_as_done(todo_index)
             elif status_choice == "Not Done":
                 app.mark_as_not_done(todo_index)
-            app.list_todos()
+            app.list_todos(show="all")
         elif action == "Remove todo":
             if not app.todos:
                 print("No todos to remove.")
@@ -423,7 +462,7 @@ def cli_menu(filepath="./.todo_list.db"):
                 choices=["<Back>"] + app.todos,
             ).ask()
 
-            if todo_choice == "<Back>":
+            if todo_choice == "<Back>" or todo_choice is None:
                 continue
 
             todo_to_remove = app.todos.index(todo_choice) + 1

{cli_todo_jd-0.2.0.dist-info → cli_todo_jd-0.2.1.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cli-todo-jd
-Version: 0.2.0
+Version: 0.2.1
 Summary: Add your description here
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown

cli_todo_jd-0.2.1.dist-info/RECORD

@@ -0,0 +1,11 @@
+cli_todo_jd/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+cli_todo_jd/cli_entry.py,sha256=PiwgYwT5OTvY0ZyPIxEjBrvdNgEQzx5awgqcKXh1AUE,3875
+cli_todo_jd/main.py,sha256=cqjwHSg4l0aH4aLHyVxOHW8WlRiXp0dB4s8PjhIRxao,15437
+cli_todo_jd/storage/__init__.py,sha256=u0jMfDuUIEy9mor1dVeIiAE0Cap6FL77tW9GlsyskYU,210
+cli_todo_jd/storage/migrate.py,sha256=Ij_0OwTvibow79KVilf2O2nfMuohj0raarVV7OcjwbY,3063
+cli_todo_jd/storage/schema.py,sha256=r5BTtcRn8J72_b8NJlryYnd_aiuy0y00eX01QavKE98,1824
+cli_todo_jd-0.2.1.dist-info/METADATA,sha256=YGHa7H0kv4b8aoNPBj_lgKcF6TMyoiDUGhWEhjbKCkY,2982
+cli_todo_jd-0.2.1.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+cli_todo_jd-0.2.1.dist-info/entry_points.txt,sha256=BIfrMKcC340A79aXHg34nnhiDsXh7c9hPck1k-Rb28c,95
+cli_todo_jd-0.2.1.dist-info/top_level.txt,sha256=hOnYr7w1JdQs6MlD1Uzjt24Ca8nvriOWNNq6NaqgHqM,12
+cli_todo_jd-0.2.1.dist-info/RECORD,,

cli_todo_jd-0.2.0.dist-info/RECORD

@@ -1,11 +0,0 @@
-cli_todo_jd/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-cli_todo_jd/cli_entry.py,sha256=sKp-aIWVldH8DiXv22bokjlIiJcAZshKHdLEqjM09HA,2877
-cli_todo_jd/main.py,sha256=AN22ojW0iIXArJQmtEjQ1jSdAb7Z-xr8i4NIEE4xAQ0,13965
-cli_todo_jd/storage/__init__.py,sha256=u0jMfDuUIEy9mor1dVeIiAE0Cap6FL77tW9GlsyskYU,210
-cli_todo_jd/storage/migrate.py,sha256=Ij_0OwTvibow79KVilf2O2nfMuohj0raarVV7OcjwbY,3063
-cli_todo_jd/storage/schema.py,sha256=r5BTtcRn8J72_b8NJlryYnd_aiuy0y00eX01QavKE98,1824
-cli_todo_jd-0.2.0.dist-info/METADATA,sha256=rra5GQZYrK8xCKX3pUaW0wnmw_pujlKR8bpOIVH-a1U,2982
-cli_todo_jd-0.2.0.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
-cli_todo_jd-0.2.0.dist-info/entry_points.txt,sha256=BIfrMKcC340A79aXHg34nnhiDsXh7c9hPck1k-Rb28c,95
-cli_todo_jd-0.2.0.dist-info/top_level.txt,sha256=hOnYr7w1JdQs6MlD1Uzjt24Ca8nvriOWNNq6NaqgHqM,12
-cli_todo_jd-0.2.0.dist-info/RECORD,,