Skip to content

Implement set_tasks_completion lifecycle flow#42

Merged
deverman merged 5 commits into
masterfrom
issue-31-set-tasks-completion
Jun 1, 2026
Merged

Implement set_tasks_completion lifecycle flow#42
deverman merged 5 commits into
masterfrom
issue-31-set-tasks-completion

Conversation

@deverman

Copy link
Copy Markdown
Owner

Summary:

Validation:

  • node --check Plugin/FocusRelayBridge.omnijs/Resources/BridgeLibrary.js
  • swift test
  • ./scripts/install-plugin.sh
  • swift run focusrelay bridge-health-check
  • swift run focusrelay get-task e-Wn5k27h6g --fields id,name,completed,completionDate
  • swift run focusrelay set-tasks-completion e-Wn5k27h6g --state completed --verify --return-fields id,name,completed,completionDate
  • swift run focusrelay set-tasks-completion e-Wn5k27h6g --state active --verify --return-fields id,name,completed,completionDate

Notes:

@deverman

Copy link
Copy Markdown
Owner Author

Merge-order note: this PR is intentionally stacked on #41 and should not merge before #41 lands. After #41 merges, retarget or rebase this branch onto master before final review and merge.

@deverman

Copy link
Copy Markdown
Owner Author

Merge-order note: the follow-up work for issue #32 is being stacked on top of this PR because task move and reparent behavior reuses the same task mutation surface. Merge #42 before the future #32 PR.

@deverman deverman changed the base branch from issue-30-update-tasks to master June 1, 2026 15:08
@deverman

deverman commented Jun 1, 2026

Copy link
Copy Markdown
Owner Author

Updated #42 on top of current master after #41 merged, resolved squash-merge conflicts, and retargeted the PR base to master.

Validation:

  • node --check Plugin/FocusRelayBridge.omnijs/Resources/BridgeLibrary.js
  • swift test (79 tests passed; existing MCP SDK deprecation warnings only)
  • git diff --check
  • ./scripts/install-plugin.sh; fully restarted OmniFocus
  • ./.build/debug/focusrelay bridge-health-check
  • ./.build/debug/focusrelay set-tasks-completion --help
  • Preview-only live check: ./.build/debug/focusrelay set-tasks-completion ixKfgse5Mw- --state completed --preview-only --verify --return-fields id,name,completed,completionDate
  • Real live lifecycle check: completed ixKfgse5Mw- with --verify, then restored it with --state active --verify
  • Confirmed restore with ./.build/debug/focusrelay get-task ixKfgse5Mw- --fields id,name,completed,completionDate
  • swift run focusrelay benchmark-gate-check --tool task-counts
  • swift run focusrelay benchmark-gate-check --tool list-tasks
  • swift run focusrelay benchmark-gate-check --tool project-counts

Merge order note: this should merge after #41 and before #43.

@deverman deverman marked this pull request as ready for review June 1, 2026 15:17
@deverman deverman merged commit 13b5d2e into master Jun 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant