Repository navigation
Fix orchestrated email review flow and Cosmos response contracts - #1751
Merged
Paul Lizer (paullizer) merged 1 commit intoOct 11, 2026
Conversation
Normalize operation receipts before strict validation, preserve saved action linkage through orchestration, and present review/send outcomes inline. Add SDK-shaped storage doubles, a reviewed boundary inventory, and offline contract CI coverage. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Paul Lizer (paullizer)
merged commit Oct 11, 2026
198e514
into
microsoft:paullizer-react-v2-ui
12 checks passed
15 of 24 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
An orchestrated email request could create its Outlook draft and pending action, then report draft creation as failed because receipt persistence rejected a CosmosDict response. Missing orchestration-to-message linkage also placed the review card above the conversation, making preparation, review, and send outcomes confusing.
Approach
Scope and compatibility
Version advances to 0.261.324. Existing conversations and pending actions are not migrated, relinked, rewritten, or replayed. Delivery policies and strict domain validators remain unchanged. No real mail was sent and no deployment was performed.
The branch includes React V2 base commit
add94ccfe, pulled before implementation. The target advanced with mobile changes during validation; those later changes have not been merged into this branch.Validation
.\.venv\pinned\Scripts\python.exe scripts\check_cosmos_response_boundaries.py --run-contracts: 567 passed plus 231 subtests, using pinned Cosmos 4.9.0 and Semantic Kernel 1.39.4 with network access blocked.node --experimental-strip-types --test --test-reporter=dot .\functional_tests\test_v2_m365_pending_actions_logic.mjs .\functional_tests\test_v2_m365_pending_actions_store.mjs .\functional_tests\test_v2_m365_pending_actions_stream.mjs .\functional_tests\test_v2_orchestration_streaming_surface.mjs: 119 passed..\.venv\Scripts\python.exe -m pytest .\ui_tests\test_v2_m365_pending_action_cards.py -q --tb=short: 14 passed, desktop/mobile and keyboard review with mocked sends..\.venv\Scripts\python.exe -m pytest .\functional_tests\test_v2_m365_pending_actions.py .\functional_tests\test_docs_app_surface_coverage.py .\functional_tests\test_docs_site_quality.py -q --tb=short: 27 passed.npm --prefix application\v2_ui run build: passed; Vite reports its existing large-chunk advisory.git diff --check: passed.Wider optional workflow probes exposed existing cold-import and stale retry/edit expectations, reproduced independently of SDK-shaped defaults and documented in the Cosmos compatibility fix notes. These unrelated behaviors were not changed.