Add macOS CI and initial tests

.github/workflows/tests.yml

@@ -0,0 +1,26 @@
+name: macOS Tests
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+
+jobs:
+  test:
+    runs-on: macos-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v5
+        with:
+          python-version: '3.11'
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install -r requirements-dev.txt
+          pip install .
+      - name: Lint
+        run: |
+          ruff check .
+          black --check .
+      - name: Test
+        run: pytest -q

pyproject.toml

@@ -13,6 +13,10 @@ package-mode = true
 include = ["src/nodetool/package_metadata/nodetool-apple.json"]
 repository = "https://github.com/nodetool-ai/nodetool-apple"
 
+[tool.ruff]
+[tool.ruff.lint]
+extend-ignore = ["F401", "F841"]
+
 [tool.poetry.dependencies]
 python = "^3.11"
 nodetool-core = { git = "https://github.com/nodetool-ai/nodetool-core.git", rev = "main" }

src/nodetool/nodes/apple/dictionary.py

@@ -31,7 +31,9 @@ def is_cacheable(cls) -> bool:
 
     async def process(self, context: ProcessingContext) -> list[str]:
         if not IS_MACOS:
-            raise NotImplementedError("Dictionary functionality is only available on macOS")
+            raise NotImplementedError(
+                "Dictionary functionality is only available on macOS"
+            )
         if not self.term:
             return []
 

src/nodetool/nodes/apple/messages.py

@@ -30,7 +30,9 @@ def is_cacheable(cls) -> bool:
 
     async def process(self, context: ProcessingContext):
         if not IS_MACOS:
-            raise NotImplementedError("Messages functionality is only available on macOS")
+            raise NotImplementedError(
+                "Messages functionality is only available on macOS"
+            )
         text_content = escape_for_applescript(self.text)
         recipient = escape_for_applescript(self.recipient)
 

src/nodetool/nodes/apple/notes.py

@@ -13,6 +13,7 @@
 
 export_notes_script = Path(__file__).parent / "exportnotes.applescript"
 
+
 def escape_for_applescript(text: str) -> str:
     """Escape special characters for AppleScript strings."""
     # First escape backslashes, then quotes
@@ -33,6 +34,7 @@ class CreateNote(BaseNode):
     - Create documentation or records
     - Save workflow outputs as notes
     """
+
     title: str = Field(default="", description="Title of the note")
     body: str = Field(default="", description="Content of the note")
     folder: str = Field(default="Notes", description="Notes folder to save to")
@@ -68,10 +70,14 @@ async def process(self, context: ProcessingContext):
         except subprocess.CalledProcessError as e:
             raise Exception(f"Failed to create note: {e.stderr}")
 
+
 class ReadNotes(BaseNode):
     """Read notes from Apple Notes via AppleScript"""
+
     note_limit: int = Field(default=10, description="Maximum number of notes to export")
-    note_limit_per_folder: int = Field(default=10, description="Maximum notes per folder")
+    note_limit_per_folder: int = Field(
+        default=10, description="Maximum notes per folder"
+    )
 
     @classmethod
     def is_cacheable(cls) -> bool:

src/nodetool/nodes/apple/screen.py

@@ -24,7 +24,9 @@ class CaptureScreen(BaseNode):
 
     async def process(self, context: ProcessingContext) -> ImageRef:
         if not IS_MACOS:
-            raise NotImplementedError("Screen capture functionality is only available on macOS")
+            raise NotImplementedError(
+                "Screen capture functionality is only available on macOS"
+            )
         main_display = Quartz.CGMainDisplayID()  # type: ignore
 
         # If region is specified, capture that region, otherwise capture full screen

tests/test_escape_for_applescript.py

@@ -0,0 +1,12 @@
+from nodetool.nodes.apple.notes import escape_for_applescript
+
+
+def test_escape_quotes_and_backslashes():
+    text = 'Hello "World" \\ Test'
+    expected = 'Hello \\"World\\" \\\\ Test'
+    assert escape_for_applescript(text) == expected
+
+
+def test_escape_newlines():
+    text = "line1\nline2"
+    assert escape_for_applescript(text) == "line1\\nline2"

tests/test_nodes_cacheable.py

@@ -0,0 +1,17 @@
+import pytest
+from nodetool.nodes.apple import calendar, notes, messages, speech, dictionary
+
+CASES = [
+    (calendar.CreateCalendarEvent, False),
+    (calendar.ListCalendarEvents, False),
+    (notes.CreateNote, False),
+    (notes.ReadNotes, False),
+    (messages.SendMessage, False),
+    (speech.SayText, False),
+    (dictionary.SearchDictionary, True),
+]
+
+
+@pytest.mark.parametrize("node_cls,expected", CASES)
+def test_node_is_cacheable(node_cls, expected):
+    assert node_cls.is_cacheable() is expected