fix(inbox): remove invalid inbox parameter from Task constructor

stephendolan · claude · stephendolan · commit ec889c477d4f · 2025-12-06T10:38:07.000-05:00
The Omni Automation API requires omitting the second argument when creating inbox tasks. Passing `inbox` as second parameter causes task creation to fail. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
diff --git a/src/lib/omnifocus.ts b/src/lib/omnifocus.ts
@@ -417,7 +417,7 @@ export class OmniFocus {
         ${options.project
           ? `const targetProject = findByName(flattenedProjects, "${this.escapeString(options.project)}", "Project");
              const task = new Task("${this.escapeString(options.name)}", targetProject);`
-          : `const task = new Task("${this.escapeString(options.name)}", inbox);`
+          : `const task = new Task("${this.escapeString(options.name)}");`
         }
 
         ${options.note ? `task.note = "${this.escapeString(options.note)}";` : ''}

Original file line number	Diff line number	Diff line change
`@@ -417,7 +417,7 @@ export class OmniFocus {`
`417`	`417`	`${options.project`
`418`	`418`	? `const targetProject = findByName(flattenedProjects, "${this.escapeString(options.project)}", "Project");
`419`	`419`	const task = new Task("${this.escapeString(options.name)}", targetProject);`
`420`		- : `const task = new Task("${this.escapeString(options.name)}", inbox);`
	`420`	+ : `const task = new Task("${this.escapeString(options.name)}");`
`421`	`421`	`}`
`422`	`422`
`423`	`423`	${options.note ? `task.note = "${this.escapeString(options.note)}";` : ''}