@scardis/omnifocus-mcp 0.1.1 → 0.1.2

package/src/runtime/jxaShim.ts

@@ -1,55 +0,0 @@
-/**
- * JXA wrapper template. The `__SNIPPET__` placeholder is replaced with the
- * OmniJS snippet to execute. The resulting script is passed to
- * `osascript -l JavaScript` via stdin.
- *
- * This shim:
- *  - Calls Application('OmniFocus').evaluateJavascript() with the snippet
- *  - Wraps in try/catch
- *  - Prints exactly one line of JSON: {ok, data} | {ok: false, error: {...}}
- *
- * The JXA scripting dictionary is used ONLY for evaluateJavascript.
- * No other scripting-dictionary domain methods are invoked here.
- */
-export const JXA_SHIM_TEMPLATE = `
-(function() {
-  var snippet = __SNIPPET__;
-  var app = Application('OmniFocus');
-  app.includeStandardAdditions = true;
-  try {
-    var result = app.evaluateJavascript(snippet);
-    // result is already a JSON string produced by the snippet
-    $.NSFileHandle.fileHandleWithStandardOutput.writeData(
-      $.NSString.alloc.initWithString(result + '\\n')
-        .dataUsingEncoding($.NSUTF8StringEncoding)
-    );
-  } catch(e) {
-    var errName = e.name || 'Error';
-    var errMsg = e.message || String(e);
-    // OmniJS wraps thrown errors; the original name is embedded in the message
-    // as "SomeName: original message". Recover it when the name is generic.
-    var nameMatch = errMsg.match(/^([A-Z]\\w*Error): ([\\s\\S]*)$/);
-    if (nameMatch) { errName = nameMatch[1]; errMsg = nameMatch[2]; }
-    var err = JSON.stringify({
-      ok: false,
-      error: {
-        name: errName,
-        message: errMsg
-      }
-    });
-    $.NSFileHandle.fileHandleWithStandardOutput.writeData(
-      $.NSString.alloc.initWithString(err + '\\n')
-        .dataUsingEncoding($.NSUTF8StringEncoding)
-    );
-  }
-})();
-`;
-
-/**
- * Build the complete JXA script to pass to osascript.
- * The snippet is embedded as a JS string literal via JSON.stringify,
- * so it is safe to contain any characters including quotes, backslashes, etc.
- */
-export function buildJxaScript(snippet: string): string {
-  return JXA_SHIM_TEMPLATE.replace("__SNIPPET__", JSON.stringify(snippet));
-}

package/src/runtime/resultProtocol.ts

@@ -1,62 +0,0 @@
-import { z } from "zod";
-
-const ErrorDetail = z.object({
-  name: z.string(),
-  message: z.string(),
-  stack: z.string().optional(),
-});
-
-const SuccessEnvelope = z.object({
-  ok: z.literal(true),
-  data: z.unknown(),
-});
-
-const ErrorEnvelope = z.object({
-  ok: z.literal(false),
-  error: ErrorDetail,
-});
-
-const Envelope = z.discriminatedUnion("ok", [SuccessEnvelope, ErrorEnvelope]);
-
-export type ResultEnvelope = z.infer<typeof Envelope>;
-
-export class ExecutionError extends Error {
-  readonly errorName: string;
-  readonly remoteStack?: string;
-
-  constructor(detail: z.infer<typeof ErrorDetail>) {
-    super(detail.message);
-    this.name = "ExecutionError";
-    this.errorName = detail.name;
-    this.remoteStack = detail.stack;
-  }
-}
-
-/**
- * Scans stdout for the first JSON-parseable line and validates it as a result
- * envelope. Ignores any preceding non-JSON chatter (e.g. osascript warnings).
- */
-export function parseResultLine(stdout: string): ResultEnvelope {
-  const lines = stdout.split("\n");
-  for (const line of lines) {
-    const trimmed = line.trim();
-    if (!trimmed) continue;
-    let parsed: unknown;
-    try {
-      parsed = JSON.parse(trimmed);
-    } catch {
-      continue; // not JSON, skip
-    }
-    const result = Envelope.safeParse(parsed);
-    if (result.success) {
-      return result.data;
-    }
-    // Parsed as JSON but not a valid envelope — this is a protocol error
-    throw new Error(
-      `Bridge returned JSON that is not a valid result envelope: ${trimmed}`
-    );
-  }
-  throw new Error(
-    `No valid JSON result envelope found in osascript output:\n${stdout}`
-  );
-}

package/src/runtime/snippetLoader.ts

@@ -1,79 +0,0 @@
-import { readFileSync } from "fs";
-import { join, dirname } from "path";
-import { fileURLToPath } from "url";
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const SNIPPETS_DIR = join(__dirname, "..", "snippets");
-
-// Allowlist of valid snippet names. Adding a new snippet requires an explicit
-// entry here, which prevents path-traversal attacks if a snippet name ever
-// reaches this loader from a dynamic source.
-const ALLOWED_SNIPPETS = new Set([
-  "get_folder",
-  "get_project",
-  "get_tag",
-  "get_task",
-  "list_folders",
-  "list_projects",
-  "list_tags",
-  "list_tasks",
-  "resolve_name",
-  "create_task",
-  "edit_task",
-  "complete_task",
-  "drop_task",
-  "delete_task",
-  "create_project",
-  "edit_project",
-  "complete_project",
-  "drop_project",
-  "delete_project",
-  "create_folder",
-  "edit_folder",
-  "delete_folder",
-  "create_tag",
-  "edit_tag",
-  "delete_tag",
-  "move_task",
-  "move_project",
-]);
-
-const cache = new Map<string, string>();
-
-export function loadSnippet(name: string): string {
-  if (!ALLOWED_SNIPPETS.has(name)) {
-    throw new Error(`Unknown snippet: "${name}"`);
-  }
-
-  if (cache.has(name)) {
-    return cache.get(name)!;
-  }
-
-  const filePath = join(SNIPPETS_DIR, `${name}.js`);
-  let content: string;
-  try {
-    content = readFileSync(filePath, "utf-8");
-  } catch {
-    throw new Error(`Snippet "${name}" could not be loaded`);
-  }
-
-  const matches = content.split("__ARGS__").length - 1;
-  if (matches === 0) {
-    throw new Error(
-      `Snippet "${name}" contains no __ARGS__ placeholder. Every snippet must have exactly one.`
-    );
-  }
-  if (matches > 1) {
-    throw new Error(
-      `Snippet "${name}" contains ${matches} __ARGS__ placeholders. Exactly one is required.`
-    );
-  }
-
-  cache.set(name, content);
-  return content;
-}
-
-/** For testing: clear the in-memory cache. */
-export function clearSnippetCache(): void {
-  cache.clear();
-}

package/src/schemas/enums.ts

@@ -1,32 +0,0 @@
-import { z } from "zod";
-
-export const IdSchema = z.string().min(1, "ID must be a non-empty string");
-
-export const EntityType = z.enum([
-  "task",
-  "project",
-  "folder",
-  "tag",
-  "perspective",
-]);
-
-export const ProjectType = z.enum(["parallel", "sequential", "singleActions"]);
-
-export const ProjectStatus = z.enum(["active", "onHold", "done", "dropped"]);
-
-export const TaskStatus = z.enum([
-  "available",
-  "incomplete",
-  "completedByChildren",
-  "complete",
-  "dropped",
-  "dueSoon",
-  "overdue",
-  "flagged",
-  "blocked",
-  "next",
-]);
-
-export const TagStatus = z.enum(["active", "onHold", "dropped"]);
-
-export const FolderStatus = z.enum(["active", "dropped"]);

package/src/schemas/index.ts

@@ -1,38 +0,0 @@
-export {
-  IdSchema,
-  EntityType,
-  ProjectType,
-  ProjectStatus,
-  TaskStatus,
-  TagStatus,
-  FolderStatus,
-} from "./enums.js";
-
-export {
-  RepetitionRuleInput,
-  RepetitionRuleDetail,
-  TaskSummary,
-  ListTasksFilter,
-  TaskDetail,
-  CreateTaskInput,
-  EditTaskInput,
-  MoveTaskInput,
-  ReviewIntervalInput,
-  CreateProjectInput,
-  EditProjectInput,
-  MoveProjectInput,
-  ListProjectsFilter,
-  ProjectSummary,
-  ProjectDetail,
-  ListFoldersFilter,
-  CreateFolderInput,
-  EditFolderInput,
-  FolderSummary,
-  FolderDetail,
-  ListTagsFilter,
-  CreateTagInput,
-  EditTagInput,
-  TagSummary,
-  TagDetail,
-  ResolveCandidate,
-} from "./shapes.js";

package/src/schemas/shapes.ts

@@ -1,263 +0,0 @@
-import { z } from "zod";
-import {
-  IdSchema,
-  ProjectType,
-  ProjectStatus,
-  TaskStatus,
-  TagStatus,
-  FolderStatus,
-  EntityType,
-} from "./enums.js";
-
-// ─── Recurrence ──────────────────────────────────────────────────────────────
-
-const RepetitionFrequency = z.enum(["daily", "weekly", "monthly", "yearly"]);
-const RepetitionMethod = z.enum(["fixed", "dueDate", "start"]);
-const DayOfWeek = z.enum(["sunday", "monday", "tuesday", "wednesday", "thursday", "friday", "saturday"]);
-
-export const RepetitionRuleInput = z.object({
-  frequency: RepetitionFrequency,
-  interval: z.number().int().positive().default(1),
-  daysOfWeek: z.array(DayOfWeek).optional().describe("Only valid when frequency is 'weekly'"),
-  method: RepetitionMethod,
-});
-
-export const RepetitionRuleDetail = z.object({
-  frequency: RepetitionFrequency,
-  interval: z.number(),
-  daysOfWeek: z.array(DayOfWeek).optional(),
-  method: RepetitionMethod,
-});
-
-// ─── Task ────────────────────────────────────────────────────────────────────
-
-export const TaskSummary = z.object({
-  id: IdSchema,
-  name: z.string(),
-  status: TaskStatus,
-  flagged: z.boolean(),
-  containerId: IdSchema.nullable(),
-  containerType: z.enum(["project", "inbox", "task"]).nullable(),
-  dueDate: z.string().datetime().nullable(),
-  tagIds: z.array(IdSchema),
-});
-
-export const ListTasksFilter = z.object({
-  flagged: z.literal(true).optional(),
-  status: z.array(TaskStatus).optional(),
-  tagId: IdSchema.optional(),
-  dueBeforeDate: z.string().datetime().optional(),
-});
-
-export const TaskDetail = z.object({
-  id: IdSchema,
-  name: z.string(),
-  note: z.string(),
-  status: TaskStatus,
-  flagged: z.boolean(),
-  deferDate: z.string().datetime().nullable(),
-  plannedDate: z.string().datetime().nullable(),
-  dueDate: z.string().datetime().nullable(),
-  completionDate: z.string().datetime().nullable(),
-  estimatedMinutes: z.number().nullable(),
-  containerId: IdSchema.nullable(),
-  containerType: z.enum(["project", "inbox", "task"]).nullable(),
-  tagIds: z.array(IdSchema),
-  parentTaskId: IdSchema.nullable(),
-  repetitionRule: RepetitionRuleDetail.nullable(),
-});
-
-export const CreateTaskInput = z
-  .object({
-    name: z.string().min(1),
-    note: z.string().optional(),
-    flagged: z.boolean().optional(),
-    deferDate: z.string().datetime().optional(),
-    plannedDate: z.string().datetime().optional(),
-    dueDate: z.string().datetime().optional(),
-    estimatedMinutes: z.number().int().positive().optional(),
-    projectId: IdSchema.optional(),
-    parentTaskId: IdSchema.optional(),
-    tagIds: z.array(IdSchema).optional(),
-    repetitionRule: RepetitionRuleInput.optional(),
-  })
-  .refine((d) => !(d.projectId && d.parentTaskId), {
-    message: "Provide projectId or parentTaskId, not both",
-  });
-
-export const EditTaskInput = z.object({
-  id: IdSchema,
-  name: z.string().min(1).optional(),
-  note: z.string().optional(),
-  flagged: z.boolean().optional(),
-  deferDate: z.string().datetime().nullable().optional(),
-  plannedDate: z.string().datetime().nullable().optional(),
-  dueDate: z.string().datetime().nullable().optional(),
-  estimatedMinutes: z.number().int().positive().nullable().optional(),
-  tagIds: z.array(IdSchema).optional(),
-  repetitionRule: RepetitionRuleInput.optional(),
-  clearRepetitionRule: z.literal(true).optional().describe("Set to true to clear the task's repetition rule"),
-});
-
-// ─── Project ─────────────────────────────────────────────────────────────────
-
-export const ReviewIntervalInput = z.object({
-  steps: z.number().int().positive(),
-  unit: z.enum(["days", "weeks", "months", "years"]),
-});
-
-export const CreateProjectInput = z.object({
-  name: z.string().min(1),
-  folderId: IdSchema.optional(),
-  note: z.string().optional(),
-  type: ProjectType.optional(),
-  status: z.enum(["active", "onHold"]).optional(),
-  flagged: z.boolean().optional(),
-  deferDate: z.string().datetime().optional(),
-  dueDate: z.string().datetime().optional(),
-  reviewInterval: ReviewIntervalInput.optional(),
-  tagIds: z.array(IdSchema).optional(),
-});
-
-export const EditProjectInput = z.object({
-  id: IdSchema,
-  name: z.string().min(1).optional(),
-  note: z.string().optional(),
-  type: ProjectType.optional(),
-  status: z.enum(["active", "onHold"]).optional(),
-  flagged: z.boolean().optional(),
-  deferDate: z.string().datetime().nullable().optional(),
-  dueDate: z.string().datetime().nullable().optional(),
-  reviewInterval: ReviewIntervalInput.optional(),
-  tagIds: z.array(IdSchema).optional(),
-});
-
-export const MoveTaskInput = z
-  .object({
-    id: IdSchema,
-    projectId: IdSchema.optional(),
-    parentTaskId: IdSchema.optional(),
-  })
-  .refine(
-    (d) => (d.projectId !== undefined) !== (d.parentTaskId !== undefined),
-    { message: "Exactly one of projectId or parentTaskId must be provided" }
-  );
-
-export const MoveProjectInput = z.object({
-  id: IdSchema,
-  folderId: IdSchema.nullable(),
-});
-
-export const ListFoldersFilter = z.object({
-  status: FolderStatus.optional(),
-});
-
-export const ListTagsFilter = z.object({
-  status: TagStatus.optional(),
-});
-
-export const ListProjectsFilter = z.object({
-  status: z.array(ProjectStatus).optional(),
-  folderId: IdSchema.optional(),
-  flagged: z.literal(true).optional(),
-});
-
-export const ProjectSummary = z.object({
-  id: IdSchema,
-  name: z.string(),
-  folderPath: z.string(),
-  folderId: IdSchema.nullable(),
-  status: ProjectStatus,
-  type: ProjectType,
-  flagged: z.boolean(),
-});
-
-export const ProjectDetail = z.object({
-  id: IdSchema,
-  name: z.string(),
-  note: z.string(),
-  folderPath: z.string(),
-  status: ProjectStatus,
-  type: ProjectType,
-  flagged: z.boolean(),
-  deferDate: z.string().datetime().nullable(),
-  dueDate: z.string().datetime().nullable(),
-  completionDate: z.string().datetime().nullable(),
-  reviewInterval: z.string().nullable(),
-  nextReviewDate: z.string().datetime().nullable(),
-  lastReviewDate: z.string().datetime().nullable(),
-  tagIds: z.array(IdSchema),
-});
-
-export const CreateFolderInput = z.object({
-  name: z.string().min(1),
-  parentFolderId: IdSchema.optional(),
-});
-
-export const EditFolderInput = z.object({
-  id: IdSchema,
-  name: z.string().min(1),
-});
-
-// ─── Folder ──────────────────────────────────────────────────────────────────
-
-export const FolderSummary = z.object({
-  id: IdSchema,
-  name: z.string(),
-  path: z.string(),
-  parentId: IdSchema.nullable(),
-  status: FolderStatus,
-});
-
-export const FolderDetail = z.object({
-  id: IdSchema,
-  name: z.string(),
-  path: z.string(),
-  parentId: IdSchema.nullable(),
-  status: FolderStatus,
-  childFolderIds: z.array(IdSchema),
-  projectIds: z.array(IdSchema),
-});
-
-export const CreateTagInput = z.object({
-  name: z.string().min(1),
-  parentTagId: IdSchema.optional(),
-});
-
-export const EditTagInput = z
-  .object({
-    id: IdSchema,
-    name: z.string().min(1).optional(),
-    status: TagStatus.optional(),
-  })
-  .refine((d) => d.name !== undefined || d.status !== undefined, {
-    message: "Provide at least one of name or status",
-  });
-
-// ─── Tag ─────────────────────────────────────────────────────────────────────
-
-export const TagSummary = z.object({
-  id: IdSchema,
-  name: z.string(),
-  path: z.string(),
-  parentId: IdSchema.nullable(),
-  status: TagStatus,
-});
-
-export const TagDetail = z.object({
-  id: IdSchema,
-  name: z.string(),
-  path: z.string(),
-  parentId: IdSchema.nullable(),
-  status: TagStatus,
-  childTagIds: z.array(IdSchema),
-});
-
-// ─── Resolution ──────────────────────────────────────────────────────────────
-
-export const ResolveCandidate = z.object({
-  id: IdSchema,
-  name: z.string(),
-  path: z.string(),
-  type: EntityType,
-});

package/src/server.ts

@@ -1,56 +0,0 @@
-import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
-import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
-import { z } from "zod";
-import { allTools } from "./tools/index.js";
-
-const server = new McpServer({
-  name: "omnifocus-mcp",
-  version: "0.1.0",
-});
-
-for (const tool of allTools) {
-  // Unwrap ZodEffects from .refine() to get the underlying ZodObject.
-  // The MCP SDK's zodToJsonSchema handles ZodObject correctly but not ZodEffects.
-  const baseSchema =
-    tool.inputSchema instanceof z.ZodEffects
-      ? (tool.inputSchema._def.schema as z.AnyZodObject)
-      : (tool.inputSchema as z.AnyZodObject);
-
-  server.registerTool(
-    tool.name,
-    {
-      description: tool.description,
-      inputSchema: baseSchema,
-    },
-    async (args: Record<string, unknown>) => {
-      try {
-        const input = tool.inputSchema.parse(args);
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        const result = await (tool.handler as (input: any) => Promise<unknown>)(input);
-        return {
-          content: [
-            {
-              type: "text" as const,
-              text: JSON.stringify(result, null, 2),
-            },
-          ],
-        };
-      } catch (err) {
-        const message =
-          err instanceof Error ? err.message : String(err);
-        return {
-          content: [
-            {
-              type: "text" as const,
-              text: `Error: ${message}`,
-            },
-          ],
-          isError: true,
-        };
-      }
-    }
-  );
-}
-
-const transport = new StdioServerTransport();
-await server.connect(transport);

package/src/tools/completeProject.ts

@@ -1,21 +0,0 @@
-import { z } from "zod";
-import { runSnippet } from "../runtime/index.js";
-import { IdSchema, ProjectDetail } from "../schemas/index.js";
-
-const CompleteProjectInput = z.object({
-  id: IdSchema.describe("The project's id.primaryKey"),
-});
-
-export async function completeProjectHandler(
-  input: z.infer<typeof CompleteProjectInput>
-): Promise<z.infer<typeof ProjectDetail>> {
-  const raw = await runSnippet("complete_project", { id: input.id });
-  return ProjectDetail.parse(raw);
-}
-
-export const completeProjectTool = {
-  name: "complete_project",
-  description: "Mark a project as done (complete). Returns the updated project detail.",
-  inputSchema: CompleteProjectInput,
-  handler: completeProjectHandler,
-} as const;

package/src/tools/completeTask.ts

@@ -1,23 +0,0 @@
-import { z } from "zod";
-import { runSnippet } from "../runtime/index.js";
-import { IdSchema, TaskDetail } from "../schemas/index.js";
-
-const completeTaskSchema = z.object({
-  id: IdSchema.describe("The task's id.primaryKey"),
-});
-
-export type CompleteTaskInput = z.infer<typeof completeTaskSchema>;
-
-export async function completeTaskHandler(
-  input: CompleteTaskInput
-): Promise<z.infer<typeof TaskDetail>> {
-  const raw = await runSnippet("complete_task", { id: input.id });
-  return TaskDetail.parse(raw);
-}
-
-export const completeTaskTool = {
-  name: "complete_task",
-  description: "Mark a task complete by its stable ID. Returns the updated task detail.",
-  inputSchema: completeTaskSchema,
-  handler: completeTaskHandler,
-} as const;

package/src/tools/createFolder.ts

@@ -1,20 +0,0 @@
-import { z } from "zod";
-import { runSnippet } from "../runtime/index.js";
-import { CreateFolderInput, FolderDetail } from "../schemas/index.js";
-
-export type CreateFolderInputType = z.infer<typeof CreateFolderInput>;
-
-export async function createFolderHandler(
-  input: CreateFolderInputType
-): Promise<z.infer<typeof FolderDetail>> {
-  const raw = await runSnippet("create_folder", input);
-  return FolderDetail.parse(raw);
-}
-
-export const createFolderTool = {
-  name: "create_folder",
-  description:
-    "Create a new OmniFocus folder. Omit parentFolderId to create at the top level; provide parentFolderId to nest it inside an existing folder.",
-  inputSchema: CreateFolderInput,
-  handler: createFolderHandler,
-} as const;

package/src/tools/createProject.ts

@@ -1,20 +0,0 @@
-import { z } from "zod";
-import { runSnippet } from "../runtime/index.js";
-import { CreateProjectInput, ProjectDetail } from "../schemas/index.js";
-
-export type CreateProjectInputType = z.infer<typeof CreateProjectInput>;
-
-export async function createProjectHandler(
-  input: CreateProjectInputType
-): Promise<z.infer<typeof ProjectDetail>> {
-  const raw = await runSnippet("create_project", input);
-  return ProjectDetail.parse(raw);
-}
-
-export const createProjectTool = {
-  name: "create_project",
-  description:
-    "Create a new OmniFocus project. Omit folderId to create at the top level; provide folderId to place it inside a folder. Optionally set type (parallel/sequential/singleActions), status (active/onHold), review interval, and tags.",
-  inputSchema: CreateProjectInput,
-  handler: createProjectHandler,
-} as const;

package/src/tools/createTag.ts

@@ -1,20 +0,0 @@
-import { z } from "zod";
-import { runSnippet } from "../runtime/index.js";
-import { CreateTagInput, TagDetail } from "../schemas/index.js";
-
-export type CreateTagInputType = z.infer<typeof CreateTagInput>;
-
-export async function createTagHandler(
-  input: CreateTagInputType
-): Promise<z.infer<typeof TagDetail>> {
-  const raw = await runSnippet("create_tag", input);
-  return TagDetail.parse(raw);
-}
-
-export const createTagTool = {
-  name: "create_tag",
-  description:
-    "Create a new OmniFocus tag. Omit parentTagId to create at the top level; provide parentTagId to nest it under an existing tag.",
-  inputSchema: CreateTagInput,
-  handler: createTagHandler,
-} as const;