cli-todo-jd 0.2.0__tar.gz → 0.2.1__tar.gz

{cli_todo_jd-0.2.0 → cli_todo_jd-0.2.1}/PKG-INFO

@@ -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.0 → cli_todo_jd-0.2.1}/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-0.2.0 → cli_todo_jd-0.2.1}/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 → cli_todo_jd-0.2.1}/cli_todo_jd.egg-info/PKG-INFO

@@ -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.0 → cli_todo_jd-0.2.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "cli-todo-jd"
-version = "0.2.0"
+version = "0.2.1"
 description = "Add your description here"
 readme = "README.md"
 requires-python = ">=3.10"
@@ -23,7 +23,7 @@ todo = "cli_todo_jd.cli_entry:app"
 
 
 [tool.bumpversion]
-current_version = "0.2.0"
+current_version = "0.2.1"
 parse = "(?P<major>\\d+)\\.(?P<minor>\\d+)\\.(?P<patch>\\d+)"
 serialize = ["{major}.{minor}.{patch}"]
 search = "{current_version}"