Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds comprehensive documentation for the defaultResults field in Virtual MCP Server composite tools. The defaultResults field provides fallback values when workflow steps are skipped due to conditional logic or continue-on-error behavior, addressing a critical gap in the documentation (issue #489).
Changes:
- Added a new "Default step outputs" section with detailed explanations, examples, and validation behavior
- Added cross-reference tips in the "Steps" and "Error handling" sections to help users discover the feature
- Fixed pre-existing markdownlint errors (heading style and JSX comment syntax)
danbarr
requested changes
Feb 2, 2026
Collaborator
danbarr
left a comment
danbarr
left a comment
There was a problem hiding this comment.
Some minor style/consistency notes but overall LGTM.
I reviewed from a style POV, but I'm not able to adequately review the technical accuracy. Will leave that to others.
jerm-dro
reviewed
Feb 3, 2026
Contributor
jerm-dro
left a comment
jerm-dro
left a comment
There was a problem hiding this comment.
One minor nit, but the content looks great to me
Add comprehensive documentation for the defaultResults field in Virtual
MCP Server composite tools. This feature provides fallback values when
workflow steps are skipped due to conditional logic or continue-on-error
behavior.
Changes:
- Add "Default step outputs" section to composite tools guide
- Document when defaultResults are required (conditional steps and
onError.action: continue)
- Include configuration examples for both use cases
- Document validation behavior and error messages
- Add cross-references from "Steps" and "Error handling" sections to
help users discover the feature
- Fix pre-existing markdownlint errors (heading style, JSX comment)
Resolves: #489
yrobla
force-pushed
the
issue-489
branch
2 times, most recently
from
8cd7a34 to
75a287d
Compare
danbarr
approved these changes
Feb 3, 2026
jerm-dro
approved these changes
Feb 4, 2026
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.
Description
Add comprehensive documentation for the
defaultResultsfield in Virtual MCP Servercomposite tools. This field provides fallback values when workflow steps are skipped due to
conditional logic (
condition: false) or continue-on-error behavior (onError.action: continue).The new "Default step outputs" section covers:
scenarios
feature when they need it
Also fixed pre-existing markdownlint errors (heading style and JSX comment syntax).
Type of change
Related issues/PRs
Resolves: #489
Screenshots
None needed (text documentation only).
Submitter checklist
Content and formatting
Navigation
Not applicable - updated existing page, no new pages added or navigation changes.