create-screenfix 0.1.0 → 0.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-screenfix",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "Screenshot capture tool for Claude Code via MCP",
   "bin": {
     "create-screenfix": "./bin/create-screenfix.js"
@@ -26,8 +26,9 @@
   ],
   "repository": {
     "type": "git",
-    "url": "https://github.com/anthropics/screenfix.git"
+    "url": "https://github.com/sukhdeepbangar/screenfix.git"
   },
+  "homepage": "https://sukhdeepbangar.github.io/screenfix",
   "license": "MIT",
   "dependencies": {
     "commander": "^11.0.0"

package/python/screenfix/annotation_window.py

@@ -38,7 +38,7 @@ _current_controller = None
 
 def save_screenshot(temp_path: str) -> str:
     """Save screenshot from temp location to final location."""
-    save_dir = Path(config.save_directory)
+    save_dir = Path(config.save_directory).resolve()  # Absolute path
     save_dir.mkdir(parents=True, exist_ok=True)
 
     timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
@@ -46,7 +46,7 @@ def save_screenshot(temp_path: str) -> str:
     final_path = save_dir / filename
 
     shutil.move(temp_path, final_path)
-    return str(final_path)
+    return str(final_path)  # Returns absolute path
 
 
 def cleanup_temp_file(temp_path: str):

package/python/screenfix/mcp_server.py

@@ -1,21 +1,16 @@
-"""MCP server for ScreenFix."""
+"""MCP server for ScreenFix - daemon control only."""
 
 import asyncio
-import base64
 import json
 import os
 import subprocess
 import sys
 import time
 from pathlib import Path
-from typing import Any
 
 from mcp.server import Server
 from mcp.server.stdio import stdio_server
-from mcp.types import Tool, TextContent, ImageContent
-
-from .config import config
-from .task_tracker import get_tasks, get_pending_tasks, mark_task_complete
+from mcp.types import Tool, TextContent
 
 
 STATE_FILE = Path.home() / ".config" / "screenfix" / "state.json"
@@ -45,30 +40,6 @@ def get_daemon_state() -> dict:
         return {"running": False, "listening": False}
 
 
-def get_screenshots() -> list[dict]:
-    """Get list of screenshots."""
-    save_dir = Path(config.save_directory)
-    if not save_dir.exists():
-        return []
-
-    screenshots = []
-    for f in save_dir.glob("*.png"):
-        screenshots.append({
-            "path": str(f),
-            "filename": f.name,
-            "modified": f.stat().st_mtime,
-        })
-
-    screenshots.sort(key=lambda x: x["modified"], reverse=True)
-    return screenshots
-
-
-def get_last_screenshot() -> dict | None:
-    """Get the most recent screenshot."""
-    screenshots = get_screenshots()
-    return screenshots[0] if screenshots else None
-
-
 def start_daemon() -> tuple[bool, str]:
     """Start the daemon as a background process."""
     state = get_daemon_state()
@@ -137,72 +108,13 @@ def create_server() -> Server:
             ),
             Tool(
                 name="get_status",
-                description="Get ScreenFix status",
+                description="Get ScreenFix daemon status",
                 inputSchema={"type": "object", "properties": {}, "required": []},
             ),
-            Tool(
-                name="get_last_screenshot",
-                description="Get the most recent screenshot with its image and related task",
-                inputSchema={
-                    "type": "object",
-                    "properties": {
-                        "include_image": {
-                            "type": "boolean",
-                            "description": "Include image data",
-                            "default": True,
-                        }
-                    },
-                    "required": [],
-                },
-            ),
-            Tool(
-                name="list_screenshots",
-                description="List all screenshots",
-                inputSchema={
-                    "type": "object",
-                    "properties": {
-                        "limit": {"type": "integer", "default": 10}
-                    },
-                    "required": [],
-                },
-            ),
-            Tool(
-                name="get_tasks",
-                description="Get tasks from tasks.md",
-                inputSchema={
-                    "type": "object",
-                    "properties": {
-                        "pending_only": {"type": "boolean", "default": False}
-                    },
-                    "required": [],
-                },
-            ),
-            Tool(
-                name="complete_task",
-                description="Mark a task as complete",
-                inputSchema={
-                    "type": "object",
-                    "properties": {
-                        "task_text": {"type": "string", "description": "Task text to mark complete"}
-                    },
-                    "required": ["task_text"],
-                },
-            ),
-            Tool(
-                name="read_screenshot",
-                description="Read a specific screenshot by path",
-                inputSchema={
-                    "type": "object",
-                    "properties": {
-                        "path": {"type": "string", "description": "Path to screenshot"}
-                    },
-                    "required": ["path"],
-                },
-            ),
         ]
 
     @server.call_tool()
-    async def call_tool(name: str, arguments: dict[str, Any]) -> list[TextContent | ImageContent]:
+    async def call_tool(name: str, arguments: dict) -> list[TextContent]:
 
         if name == "start_daemon":
             success, message = start_daemon()
@@ -214,89 +126,15 @@ def create_server() -> Server:
 
         elif name == "get_status":
             state = get_daemon_state()
-            screenshots = get_screenshots()
-            tasks = get_pending_tasks()
-
             status = "Running" if state.get("running") else "Not running"
             text = f"""ScreenFix Status:
 - Daemon: {status}
-- Screenshots: {len(screenshots)}
-- Pending tasks: {len(tasks)}
 
-Use Cmd+Ctrl+Shift+4 to capture (instant, no delay)"""
+Use Cmd+Ctrl+Shift+4 to capture (instant, no delay)
+Screenshots saved to: ./screenfix/screenshots/
+Tasks saved to: ./screenfix/tasks/screenfix-tasks.md"""
             return [TextContent(type="text", text=text)]
 
-        elif name == "get_last_screenshot":
-            include_image = arguments.get("include_image", True)
-            screenshot = get_last_screenshot()
-
-            if not screenshot:
-                return [TextContent(type="text", text="No screenshots found. Use Cmd+Ctrl+Shift+4 to capture.")]
-
-            result = [TextContent(type="text", text=f"Screenshot: {screenshot['path']}")]
-
-            if include_image and os.path.exists(screenshot["path"]):
-                with open(screenshot["path"], "rb") as f:
-                    image_data = base64.standard_b64encode(f.read()).decode("utf-8")
-                result.append(ImageContent(type="image", data=image_data, mimeType="image/png"))
-
-            tasks = get_tasks()
-            for task in tasks:
-                if task.get("screenshot") == screenshot["path"]:
-                    result.append(TextContent(
-                        type="text",
-                        text=f"\nTask: {task['text']}\nStatus: {'Done' if task['completed'] else 'Pending'}"
-                    ))
-                    break
-
-            return result
-
-        elif name == "list_screenshots":
-            limit = arguments.get("limit", 10)
-            screenshots = get_screenshots()[:limit]
-
-            if not screenshots:
-                return [TextContent(type="text", text="No screenshots found.")]
-
-            lines = ["Screenshots:"]
-            for i, s in enumerate(screenshots, 1):
-                lines.append(f"{i}. {s['filename']}")
-
-            return [TextContent(type="text", text="\n".join(lines))]
-
-        elif name == "get_tasks":
-            pending_only = arguments.get("pending_only", False)
-            tasks = get_pending_tasks() if pending_only else get_tasks()
-
-            if not tasks:
-                return [TextContent(type="text", text="No tasks found.")]
-
-            lines = ["Tasks:"]
-            for i, task in enumerate(tasks, 1):
-                status = "[x]" if task["completed"] else "[ ]"
-                lines.append(f"{i}. {status} {task['text']}")
-
-            return [TextContent(type="text", text="\n".join(lines))]
-
-        elif name == "complete_task":
-            task_text = arguments.get("task_text", "")
-            if mark_task_complete(task_text):
-                return [TextContent(type="text", text=f"Completed: {task_text}")]
-            return [TextContent(type="text", text=f"Task not found: {task_text}")]
-
-        elif name == "read_screenshot":
-            path = arguments.get("path", "")
-            if not path or not os.path.exists(path):
-                return [TextContent(type="text", text=f"Not found: {path}")]
-
-            with open(path, "rb") as f:
-                image_data = base64.standard_b64encode(f.read()).decode("utf-8")
-
-            return [
-                TextContent(type="text", text=f"Screenshot: {path}"),
-                ImageContent(type="image", data=image_data, mimeType="image/png"),
-            ]
-
         return [TextContent(type="text", text=f"Unknown tool: {name}")]
 
     return server

package/templates/commands/screenfix-list-tasks.md

@@ -1,6 +1,8 @@
 ---
-allowed-tools: mcp__screenfix__get_tasks, mcp__screenfix__complete_task
-description: View and manage ScreenFix tasks
+allowed-tools: Read, Glob
+description: View ScreenFix tasks and screenshots
 ---
 
-Get all tasks using `mcp__screenfix__get_tasks`. Show pending tasks and their screenshots.
+1. Read the tasks file at `./screenfix/tasks/screenfix-tasks.md`
+2. Use Glob to find screenshots: `./screenfix/screenshots/*.png`
+3. Show pending tasks (lines with `- [ ]`) and their associated screenshots

package/templates/commands/screenfix-start.md

@@ -3,6 +3,6 @@ allowed-tools: mcp__screenfix__start_daemon, mcp__screenfix__get_status
 description: Start the ScreenFix daemon
 ---
 
-Start the ScreenFix daemon using `mcp__screenfix__start_daemon`. After starting, check the status.
+Start the ScreenFix daemon using `mcp__screenfix__start_daemon`. After starting, check the status with `mcp__screenfix__get_status`.
 
 Once running, use **Cmd+Ctrl+Shift+4** to capture screenshots instantly (no delay).

package/templates/commands/screenfix-status.md

@@ -1,6 +1,6 @@
 ---
 allowed-tools: mcp__screenfix__get_status
-description: Check ScreenFix status
+description: Check ScreenFix daemon status
 ---
 
 Check the current status of ScreenFix using `mcp__screenfix__get_status`.

package/templates/commands/screenfix-tasks-next.md

@@ -1,14 +1,15 @@
 ---
-allowed-tools: mcp__screenfix__get_tasks, mcp__screenfix__get_last_screenshot, mcp__screenfix__read_screenshot, mcp__screenfix__complete_task, Read, Edit, Write, Bash, Glob, Grep
+allowed-tools: Read, Edit, Write, Bash, Glob, Grep
 description: Execute the next pending ScreenFix task
 ---
 
-Get all pending tasks using `mcp__screenfix__get_tasks` with `pending_only: true`.
+1. Read the tasks file at `./screenfix/tasks/screenfix-tasks.md`
+2. Find the FIRST pending task (line with `- [ ]`)
 
-Take only the FIRST pending task:
-1. Read the associated screenshot using `mcp__screenfix__read_screenshot`
+For that task:
+1. The task line contains the screenshot path - read that screenshot using the Read tool
 2. Analyze the screenshot and the task instructions
 3. Implement the fix described in the instructions
-4. Mark the task complete using `mcp__screenfix__complete_task`
+4. Mark the task complete by editing the tasks file: change `- [ ]` to `- [x]`
 
 Execute one task at a time.

package/templates/commands/screenfix-tasks-yolo.md

@@ -1,14 +1,15 @@
 ---
-allowed-tools: mcp__screenfix__get_tasks, mcp__screenfix__get_last_screenshot, mcp__screenfix__read_screenshot, mcp__screenfix__complete_task, Read, Edit, Write, Bash, Glob, Grep
+allowed-tools: Read, Edit, Write, Bash, Glob, Grep
 description: Execute all pending ScreenFix tasks automatically
 ---
 
-Get all pending tasks using `mcp__screenfix__get_tasks` with `pending_only: true`.
+1. Read the tasks file at `./screenfix/tasks/screenfix-tasks.md`
+2. Find all pending tasks (lines with `- [ ]`)
 
 For each pending task:
-1. Read the associated screenshot using `mcp__screenfix__read_screenshot`
+1. The task line contains the screenshot path - read that screenshot using the Read tool
 2. Analyze the screenshot and the task instructions
 3. Implement the fix described in the instructions
-4. Mark the task complete using `mcp__screenfix__complete_task`
+4. Mark the task complete by editing the tasks file: change `- [ ]` to `- [x]`
 
 Execute all tasks without asking for confirmation. Just do it.