Skip to content

Define v1 write contract and mutation checklist#38

Merged
deverman merged 3 commits into
masterfrom
issue-28-write-contract
Jun 1, 2026
Merged

Define v1 write contract and mutation checklist#38
deverman merged 3 commits into
masterfrom
issue-28-write-contract

Conversation

@deverman

Copy link
Copy Markdown
Owner

Summary

Define the v1 write contract and mutation change checklist before implementing write tools.

Change

  • add docs/omni-automation-write-contract.md
  • add docs/mutation-change-checklist.md
  • lock the v1 mutation surface for tasks and projects
  • document homogeneous bulk, ID-only targeting, preview/verify, cache invalidation, and out-of-scope areas

Why

Issue #28 is the write-side equivalent of the existing query contract. It removes API-shape ambiguity before mutation work starts and keeps future implementation aligned with documented Omni Automation APIs.

Validation

  • swift test

Notes

swift test is currently blocked by the existing swift-sdk compiler failure in .build/checkouts/swift-sdk/Sources/MCP/Base/Transports/NetworkTransport.swift on this machine/toolchain. This PR changes docs only.

Copy link
Copy Markdown
Owner Author

Merge-order note: this PR is intentionally docs-only and should merge after PR #39. PR #39 updates swift-sdk to 0.12.0 so swift test passes again on Swift 6.3.1 / Xcode 26.4.1. Keep this PR scoped to the write-contract docs; do not fold the dependency update into it.

@deverman

Copy link
Copy Markdown
Owner Author

Dependency follow-up: PR #39 has merged, this branch has been updated from origin/master, and swift test now passes cleanly on Swift 6.3.1 / Xcode 26.4.1. The validation blocker mentioned in the original PR body is no longer present.

@deverman

deverman commented Jun 1, 2026

Copy link
Copy Markdown
Owner Author

Architecture/stack update: PR #40 has been retargeted to this branch, so this write contract PR is now the mechanical base for the mutation implementation stack.

Current intended order: #38 -> #40 -> #41 -> #42 -> #43 -> #44 -> #46.

@deverman

deverman commented Jun 1, 2026

Copy link
Copy Markdown
Owner Author

Updated #38 on top of current master after #37 merged.

Current diff is docs-only:

  • docs/mutation-change-checklist.md
  • docs/omni-automation-write-contract.md

Validation:

  • git diff --check
  • swift test (68 tests passed; existing MCP SDK deprecation warnings only)

Merge order note: this should merge after #37 and before #40.

@deverman deverman marked this pull request as ready for review June 1, 2026 11:39
@deverman deverman merged commit 9a69581 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