omnifocus-mcp: Skill for Claude Code

OmniFocus Integration Testing

Unit tests in this project mock AppleScript execution. This is fast but dangerous — mocked tests cannot catch AppleScript-level bugs. Integration tests run against a real OmniFocus instance and have caught critical bugs that unit tests missed.

Why Integration Tests Matter Here

The elifintervalDays story: A variable naming typo in AppleScript went undetected by 400+ unit tests because they mocked run_applescript(). The AppleScript was syntactically valid Python-side, but at runtime, AppleScript interpreted elifintervalDays as something unexpected. Only integration tests against real OmniFocus caught it.

Classes of bugs only integration tests catch:

• Variable naming conflicts with OmniFocus properties (see applescript-omnifocus skill)
• AppleScript syntax that's valid but produces wrong results
• OmniFocus version-specific behavior changes
• Timing issues with AppleScript execution
• Rich text handling inconsistencies
• Recurring task completion behavior differences

Test Database Setup

Integration tests use a separate OmniFocus database to prevent production data corruption.

Initial Setup

# Create test database in OmniFocus
# File > New Database > name it "OmniFocus-TEST"

# Set environment variables
export OMNIFOCUS_TEST_MODE=true
export OMNIFOCUS_TEST_DATABASE=OmniFocus-TEST.ofocus

# Generate test fixtures
./scripts/setup_test_database.sh

Switching Databases

# Switch to test database (sets env vars, switches OmniFocus)
./scripts/local/switch_to_test_db.sh

# Switch back to production
./scripts/local/switch_to_prod_db.sh

Cleaning Up

# Remove test data (keeps database)
./scripts/cleanup_test_data.sh

Running Integration Tests

# All integration tests
make test-integration

# Specific test file
pytest tests/test_integration_real.py -v

# E2E tests (full MCP stack)
make test-e2e
pytest tests/test_e2e_real.py -v

Prerequisites:

• OmniFocus must be running
• Test database must be active (check with ./scripts/local/switch_to_test_db.sh)
• Test fixtures must be loaded