Skip to content

fix(mastra): verify cancellation outcomes with aimock - #1086

Merged
blove merged 1 commit into
mainfrom
blove/mastra-cancellation-verification
Sep 9, 2026
Merged

fix(mastra): verify cancellation outcomes with aimock#1086
blove merged 1 commit into
mainfrom
blove/mastra-cancellation-verification

Conversation

@blove

@blove blove commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Cancelling a campsite reservation returned the correct backend decline but replayed an approval message because both outcomes shared one aimock fixture. Reservation continuations now match the actual tool result, and cancellation uses a reply captured from the live model.

The browser tests verify both decisions, their correlated resume commands and backend results, and the visible assistant outcome. The Mastra harness now supports the shared aimock replay/record configuration. Recording instructions require a fresh process per decision because newly recorded responses are cached with broad matchers.

Validation: all six Mastra replay scenarios pass; approval and cancellation each pass against the live model through separate aimock recording processes; the E2E wiring checks pass. An isolated Chrome MCP session also verified live cancellation from the approval dialog through the backend decline to the visible non-booking reply. Code review found no remaining issues. No external reservations are made by this local demonstration tool.

@vercel

vercel Bot commented Sep 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
threadplane Ready Ready Preview Sep 9, 2026 4:39am UTC

Request Review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove
blove enabled auto-merge (squash) September 9, 2026 04:36
@blove
blove merged commit 03b5796 into main Sep 9, 2026
33 checks 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