* Fix up jxa script syntax errors

* Add jxa unittests, which are checked via Github Actions CI

* Enable CI checks for all golang unit tests for consistency

Co-authored by: Claude Code v2.0.24 + Claude Sonnet 4.5 with the prompts:

```
Debug why the scripts under @scripts/ crash with errors such as `Failed to list tasks:
  failed to execute list_tasks.jxa: exit status 1 -
  /opt/homebrew/share/mcp-omnifocus/list_tasks.jxa:13:14: script error: Expected end of
  line, etc. but found “(”. (-2741)`
```

```
Add a github actions check to verify the syntax of all jxa scripts during CI
```

Author: conallob

.github/workflows/build.yml

@@ -40,3 +40,9 @@ jobs:
               exit 1
             fi
           done
+
+      - name: Validate JXA script syntax
+        run: make validate-jxa
+
+      - name: Run Go tests
+        run: go test ./... -v

CLAUDE.md

@@ -66,14 +66,21 @@ scripts/                    # JXA automation scripts
 - `./bin/mcp-omnifocus` - Run the compiled binary
 - `./bin/mcp-omnifocus -scripts /path/to/scripts` - Run with explicit scripts path
 
+### Testing
+- `make test` - Run all tests (includes Go tests and JXA validation)
+- `make validate-jxa` - Validate JXA script syntax only
+- `go test ./... -v` - Run Go tests only
+
 ### Testing JXA Scripts
 Scripts can be tested independently:
 ```bash
-osascript scripts/list_projects.jxa
-osascript scripts/list_tasks.jxa
-osascript scripts/create_task.jxa '{"name":"Test Task"}'
+osascript -l JavaScript scripts/list_projects.jxa
+osascript -l JavaScript scripts/list_tasks.jxa
+osascript -l JavaScript scripts/create_task.jxa '{"name":"Test Task"}'
 ```
 
+**Important:** Always use the `-l JavaScript` flag when testing JXA scripts with `osascript` to ensure proper syntax parsing.
+
 ## MCP Tools
 
 All tools are registered in `cmd/mcp-omnifocus/main.go` in the `registerTools()` function.
@@ -194,6 +201,16 @@ To test the MCP server locally:
 3. Check that JXA scripts have execute permissions (`chmod +x scripts/*.jxa`)
 4. Ensure OmniFocus is running and has projects/tasks to query
 
+## Continuous Integration
+
+The project uses GitHub Actions for CI/CD. The build workflow (`.github/workflows/build.yml`) runs on every push and pull request to `main`:
+
+1. **Build** - Compiles the Go binary
+2. **Validate JXA Syntax** - Uses `osacompile` to check all JXA scripts for syntax errors without executing them
+3. **Run Tests** - Executes all Go tests
+
+The JXA syntax validation ensures that script syntax errors are caught early, before they cause runtime failures. This is particularly important because the `-l JavaScript` flag must be used when calling `osascript` for proper syntax parsing.
+
 ## Release Process
 
 Releases are automated using GitHub Actions and GoReleaser v2.

Makefile

@@ -1,4 +1,4 @@
-.PHONY: build clean install test run
+.PHONY: build clean install test run validate-jxa
 
 # Build the MCP server
 build:
@@ -24,3 +24,29 @@ run:
 # Build for release
 release:
 	CGO_ENABLED=0 go build -ldflags="-s -w" -o bin/mcp-omnifocus ./cmd/mcp-omnifocus
+
+# Validate JXA script syntax
+validate-jxa:
+	@echo "Validating JXA script syntax..."
+	@FAILED=0; \
+	for script in scripts/*.jxa; do \
+		echo "Checking syntax: $$script"; \
+		if osacompile -l JavaScript -o /tmp/test.scpt "$$script" 2>&1; then \
+			echo "✓ $$script: syntax OK"; \
+			rm -f /tmp/test.scpt; \
+		else \
+			echo "✗ $$script: syntax error detected"; \
+			FAILED=1; \
+		fi; \
+	done; \
+	if [ $$FAILED -eq 1 ]; then \
+		echo ""; \
+		echo "JXA syntax validation failed"; \
+		exit 1; \
+	fi; \
+	echo ""; \
+	echo "All JXA scripts passed syntax validation"
+
+# Run all tests
+test: validate-jxa
+	go test ./...

internal/omnifocus/client.go

@@ -182,7 +182,8 @@ func isValidScriptsDir(dir string) bool {
 func (c *Client) executeJXA(scriptName string, args ...string) ([]byte, error) {
 	scriptPath := filepath.Join(c.scriptsDir, scriptName)
 
-	cmdArgs := append([]string{scriptPath}, args...)
+	// Build command arguments: -l JavaScript, script path, then script arguments
+	cmdArgs := append([]string{"-l", "JavaScript", scriptPath}, args...)
 	cmd := exec.Command("osascript", cmdArgs...)
 
 	output, err := cmd.CombinedOutput()

scripts/complete_task.jxa

@@ -11,12 +11,18 @@ function run(argv) {
     const doc = app.defaultDocument;
     const taskId = argv[0];
 
-    const tasks = doc.flattenedTasks.whose({id: taskId});
-    if (tasks.length === 0) {
+    // Find task by ID
+    const allTasks = doc.flattenedTasks();
+    let task = null;
+    for (let i = 0; i < allTasks.length; i++) {
+        if (allTasks[i].id() === taskId) {
+            task = allTasks[i];
+            break;
+        }
+    }
+    if (!task) {
         return JSON.stringify({error: 'Task not found'});
     }
-
-    const task = tasks[0];
     task.markComplete();
 
     return JSON.stringify({

scripts/create_task.jxa

@@ -15,7 +15,15 @@ function run(argv) {
 
     if (taskData.projectId) {
         // Add to specific project
-        const project = doc.flattenedProjects.whose({id: taskData.projectId})[0];
+        // Find project by ID
+        const allProjects = doc.flattenedProjects();
+        let project = null;
+        for (let i = 0; i < allProjects.length; i++) {
+            if (allProjects[i].id() === taskData.projectId) {
+                project = allProjects[i];
+                break;
+            }
+        }
         if (!project) {
             return JSON.stringify({error: 'Project not found'});
         }
@@ -47,9 +55,17 @@ function run(argv) {
     // Add tags
     if (taskData.tags && taskData.tags.length > 0) {
         taskData.tags.forEach(tagName => {
-            const tags = doc.flattenedTags.whose({name: tagName});
-            if (tags.length > 0) {
-                task.addTag(tags[0]);
+            // Find tag by name
+            const allTags = doc.flattenedTags();
+            let foundTag = null;
+            for (let i = 0; i < allTags.length; i++) {
+                if (allTags[i].name() === tagName) {
+                    foundTag = allTags[i];
+                    break;
+                }
+            }
+            if (foundTag) {
+                task.addTag(foundTag);
             } else {
                 // Create tag if it doesn't exist
                 const newTag = doc.Tag({name: tagName});

scripts/list_tasks.jxa

@@ -9,7 +9,15 @@ function run(argv) {
 
     let tasks;
     if (projectId) {
-        const project = doc.flattenedProjects.whose({id: projectId})[0];
+        // Find project by ID
+        const allProjects = doc.flattenedProjects();
+        let project = null;
+        for (let i = 0; i < allProjects.length; i++) {
+            if (allProjects[i].id() === projectId) {
+                project = allProjects[i];
+                break;
+            }
+        }
         if (!project) {
             return JSON.stringify({error: 'Project not found'});
         }

scripts/update_task.jxa

@@ -15,13 +15,19 @@ function run(argv) {
         return JSON.stringify({error: 'Task ID required'});
     }
 
-    const tasks = doc.flattenedTasks.whose({id: updateData.id});
-    if (tasks.length === 0) {
+    // Find task by ID
+    const allTasks = doc.flattenedTasks();
+    let task = null;
+    for (let i = 0; i < allTasks.length; i++) {
+        if (allTasks[i].id() === updateData.id) {
+            task = allTasks[i];
+            break;
+        }
+    }
+    if (!task) {
         return JSON.stringify({error: 'Task not found'});
     }
 
-    const task = tasks[0];
-
     // Update properties
     if (updateData.name !== undefined) {
         task.name = updateData.name;