sandbox: delimit custom SandboxAgent instructions in rendered prompt#3044
Closed
joaquinhuigomez wants to merge 1 commit intoopenai:mainfrom
Closed
sandbox: delimit custom SandboxAgent instructions in rendered prompt#3044joaquinhuigomez wants to merge 1 commit intoopenai:mainfrom
joaquinhuigomez wants to merge 1 commit intoopenai:mainfrom
Conversation
build_sandbox_instructions concatenated the caller's instructions between the SDK base prompt (which ends with a 'Tool Guidelines > Shell commands' section) and the per-capability fragments (which add more shell guidance). The rendered prompt made the app-specific instructions look nested inside the SDK's shell-commands section. Wrap the additional instructions in a 'Agent instructions' markdown heading so they read as a standalone section delimited from the SDK-generated guidance above and below. Fixes openai#3043
Member
|
Thanks for sending this. Closing it in favor of #3047 |
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.
build_sandbox_instructionsconcatenated the caller's instructions between the SDK base prompt (which ends with a Tool Guidelines > Shell commands section) and the per-capability fragments (which add more shell guidance). In the rendered prompt the app-specific instructions read as if they're nested inside the SDK's shell-commands section.Wrap the additional instructions in an
# Agent instructionsmarkdown heading so they appear as a standalone, clearly delimited section between the SDK base and the capability fragments.Fixes #3043