-
Notifications
You must be signed in to change notification settings - Fork 119
feat(sdk): optimize tool definitions and prompts for efficient MCP workflows #2722
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
tupizz
merged 26 commits into
andrii/sd-2451-refactor-mcp-set-up
from
tadeu/sdk-tool-definitions-update
Apr 6, 2026
Merged
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
2f4cbf2
feat(sdk): update tool definitions for efficient multi-block workflows
tupizz 039a036
feat(evals,sdk): add efficient workflow patterns to all agent touchpo…
tupizz dc427eb
feat(evals): enable tool search to reduce token overhead
tupizz 091da29
docs(ai): add markdown insert pattern and formatting guidance
tupizz 2c94377
docs(ai): add efficient patterns to MCP how-to-use guide
tupizz 9f65e2c
fix(evals): remove debug console.log that dumped every SDK message
tupizz 13ba3a6
feat(document-api): add alignment field to StyleApplyStep and StyleAp…
tupizz b67b909
fix(document-api): keep inline required on StyleApplyInput, guard opt…
tupizz 98a1609
feat(document-api): add alignment to format.apply step JSON schema
tupizz 69f45cf
feat(super-editor): support alignment in format.apply mutation step
tupizz 9aa655b
docs(sdk): update tool descriptions to show alignment inside format.a…
tupizz 145693d
feat(document-api): add scope: block to format.apply for full-paragra…
tupizz 64cab87
feat(document-api): allow placement and BlockNodeAddress target for m…
tupizz c3395d6
chore: regenerate SDK artifacts and docs from updated contract
tupizz 1e4e294
feat(evals): add new NDA documents and implement interactive DOCX out…
tupizz 4c04ebd
fix: address PR review — minProperties, RichContentInsertInput type, …
tupizz ee17a46
Revert "fix: address PR review — minProperties, RichContentInsertInpu…
tupizz e95f430
fix(document-api): add minProperties, type export, shared alignment c…
tupizz 16d6a52
docs(sdk): require fontSize on headings after markdown insert
tupizz 8da40b0
docs(sdk): context-driven formatting guidance for markdown inserts
tupizz e61a515
docs(sdk): only set properties explicitly present in document blocks
tupizz f39f127
feat(super-editor): resolve default fontSize in get_content blocks re…
tupizz 739a18f
fix(super-editor): fallback to 10pt default when styles omit fontSize
tupizz 20a1341
fix(super-editor): resolve fontSize per-block via style chain in get_…
tupizz e0b7e23
test(super-editor): add fontSize style chain resolution tests for blo…
tupizz c5a322f
docs(sdk): guide agents to match uppercase title conventions
tupizz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
Binary file not shown.
Binary file not shown.
Binary file not shown.
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The prompt example hard-codes
placement: "end", but insert validation only acceptsbefore,after,insideStart, orinsideEnd;"end"is rejected asINVALID_INPUT. When the benchmark agent follows this example (and the same one inevals/providers/codex-agent.mjs), its first markdown insert fails and burns turns/cost for recoverable errors.Useful? React with 👍 / 👎.