Skip to content

Tighten meal write contracts - #2

Merged
colinds merged 2 commits into
colin/write-safetyfrom
colin/meal-contracts
Aug 25, 2026
Merged

colinds merged 2 commits into
colin/write-safetyfrom
colin/meal-contracts

Conversation

@colinds

@colinds colinds commented Aug 24, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • require exact (menuId, itemId) write identity and positive source-piece ownership
  • keep client-side validation limited to request construction, ownership, selections, and the local preview ceiling
  • add Forkable's dietary advisory, direct billing values, and per-owned-order delivery status
  • cover the write handlers with a fake Forkable client; tests make no live writes

This is an intentional MCP contract update. Agents should re-read the tool schemas; there are no
legacy item-id-only aliases.

Verification

  • bun run check
  • packaged smoke test with Bun and Node

Stack: 3 of 4. Depends on #1; followed by #3.

@colinds
colinds force-pushed the colin/meal-contracts branch from 1c96f84 to a3f0f0b Compare August 24, 2026 21:46
@colinds
colinds marked this pull request as ready for review August 24, 2026 22:35
@colinds
colinds force-pushed the colin/meal-contracts branch from 0fe0261 to 3742103 Compare August 24, 2026 22:55
@colinds
colinds force-pushed the colin/meal-contracts branch 2 times, most recently from c70b484 to e4bf118 Compare August 24, 2026 23:47
@colinds
colinds force-pushed the colin/meal-contracts branch from e4bf118 to 27411ff Compare August 25, 2026 00:01
@colinds
colinds force-pushed the colin/meal-contracts branch from 27411ff to 8da3295 Compare August 25, 2026 00:11
@colinds
colinds force-pushed the colin/meal-contracts branch from 8da3295 to 361c816 Compare August 25, 2026 00:35
@colinds
colinds merged commit acfeeca into main Aug 25, 2026
4 checks passed
@colinds
colinds deleted the colin/meal-contracts branch August 25, 2026 00:42
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