diff --git a/README.md b/README.md index 91da9a7..22cdc8e 100644 --- a/README.md +++ b/README.md @@ -23,6 +23,11 @@ python task.py list # Complete a task python task.py done 1 + +# Output JSON for automation +python task.py add "Buy groceries" --json +python task.py list --json +python task.py done 1 --json ``` ## Testing diff --git a/commands/add.py b/commands/add.py index 1b1a943..91e9986 100644 --- a/commands/add.py +++ b/commands/add.py @@ -19,7 +19,7 @@ def validate_description(description): return description.strip() -def add_task(description): +def add_task(description, json_output=False): """Add a new task.""" description = validate_description(description) @@ -31,7 +31,11 @@ def add_task(description): tasks = json.loads(tasks_file.read_text()) task_id = len(tasks) + 1 - tasks.append({"id": task_id, "description": description, "done": False}) + task = {"id": task_id, "description": description, "done": False} + tasks.append(task) tasks_file.write_text(json.dumps(tasks, indent=2)) - print(f"Added task {task_id}: {description}") + if json_output: + print(json.dumps({"task": task, "message": "Task added"})) + else: + print(f"Added task {task_id}: {description}") diff --git a/commands/done.py b/commands/done.py index c9dfd42..42c6006 100644 --- a/commands/done.py +++ b/commands/done.py @@ -17,11 +17,14 @@ def validate_task_id(tasks, task_id): return task_id -def mark_done(task_id): +def mark_done(task_id, json_output=False): """Mark a task as complete.""" tasks_file = get_tasks_file() if not tasks_file.exists(): - print("No tasks found!") + if json_output: + print(json.dumps({"error": "No tasks found"})) + else: + print("No tasks found!") return tasks = json.loads(tasks_file.read_text()) @@ -31,7 +34,13 @@ def mark_done(task_id): if task["id"] == task_id: task["done"] = True tasks_file.write_text(json.dumps(tasks, indent=2)) - print(f"Marked task {task_id} as done: {task['description']}") + if json_output: + print(json.dumps({"task": task, "message": "Task marked done"})) + else: + print(f"Marked task {task_id} as done: {task['description']}") return - print(f"Task {task_id} not found") + if json_output: + print(json.dumps({"error": f"Task {task_id} not found"})) + else: + print(f"Task {task_id} not found") diff --git a/commands/list.py b/commands/list.py index 714315d..fadd2d3 100644 --- a/commands/list.py +++ b/commands/list.py @@ -18,18 +18,28 @@ def validate_task_file(): return tasks_file -def list_tasks(): +def list_tasks(json_output=False): """List all tasks.""" # NOTE: No --json flag support yet (feature bounty) tasks_file = validate_task_file() if not tasks_file: - print("No tasks yet!") + if json_output: + print(json.dumps({"tasks": []})) + else: + print("No tasks yet!") return tasks = json.loads(tasks_file.read_text()) if not tasks: - print("No tasks yet!") + if json_output: + print(json.dumps({"tasks": []})) + else: + print("No tasks yet!") + return + + if json_output: + print(json.dumps({"tasks": tasks})) return for task in tasks: diff --git a/task.py b/task.py index 53cc8ed..841467f 100644 --- a/task.py +++ b/task.py @@ -25,22 +25,25 @@ def main(): # Add command add_parser = subparsers.add_parser("add", help="Add a new task") add_parser.add_argument("description", help="Task description") + add_parser.add_argument("--json", action="store_true", help="Output JSON") # List command list_parser = subparsers.add_parser("list", help="List all tasks") + list_parser.add_argument("--json", action="store_true", help="Output JSON") # Done command done_parser = subparsers.add_parser("done", help="Mark task as complete") done_parser.add_argument("task_id", type=int, help="Task ID to mark done") + done_parser.add_argument("--json", action="store_true", help="Output JSON") args = parser.parse_args() if args.command == "add": - add_task(args.description) + add_task(args.description, args.json) elif args.command == "list": - list_tasks() + list_tasks(args.json) elif args.command == "done": - mark_done(args.task_id) + mark_done(args.task_id, args.json) else: parser.print_help() diff --git a/test_task.py b/test_task.py index ba98e43..d72c4f1 100644 --- a/test_task.py +++ b/test_task.py @@ -4,6 +4,8 @@ import pytest from pathlib import Path from commands.add import add_task, validate_description +from commands.list import list_tasks +from commands.done import mark_done from commands.done import validate_task_id @@ -28,3 +30,20 @@ def test_validate_task_id(): with pytest.raises(ValueError): validate_task_id(tasks, 99) + + +def test_json_output_for_task_commands(tmp_path, monkeypatch, capsys): + """All task commands should emit parseable JSON when requested.""" + monkeypatch.setattr(Path, "home", lambda: tmp_path) + + add_task("Write tests", json_output=True) + added = json.loads(capsys.readouterr().out) + assert added["task"] == {"id": 1, "description": "Write tests", "done": False} + + list_tasks(json_output=True) + listed = json.loads(capsys.readouterr().out) + assert listed["tasks"] == [added["task"]] + + mark_done(1, json_output=True) + done = json.loads(capsys.readouterr().out) + assert done["task"] == {"id": 1, "description": "Write tests", "done": True}