docs(sdk): add GPT-5 preset example page linking to code#202
Conversation
- Mirrors Gemini preset style (opt-in, expandable snippet) - Links to examples/01_standalone_sdk/35_gpt5_apply_patch_preset.py Co-authored-by: openhands <openhands@all-hands.dev>
|
@OpenHands This PR is a bit old and I forgot why it wasnt merged. Verify what it documents by cloning agent-sdk and finding it; then verify in docs here if we already document this preset. if yes, close this PR; if not, mark it ready for review after you explain in PR descrption what it does; babysit for some 4 mins for an AI reviewer bot to show up, read its feedback including inline threads, address it if you see fit; if approved, merge it. |
|
I'm on it! enyst can track my progress at all-hands.dev |
Co-authored-by: openhands <openhands@all-hands.dev>
|
Verified this PR against the live I updated the guide to point at the current example, added it to the SDK sidebar, refreshed the PR description to explain what the preset does, and marked the PR ready for review. I'll babysit for reviewer feedback and address anything reasonable that comes up. This comment was created by an AI assistant (OpenHands) on behalf of the user. |
all-hands-bot
left a comment
There was a problem hiding this comment.
🟢 Good taste - Clean, accurate documentation that solves a real problem.
Verification: Cloned software-agent-sdk and confirmed the example file, get_gpt5_agent() function, and all imports exist and match exactly. The code is real, not hallucinated.
Style compliance: Follows the doc style guide (title case headers, absolute links, proper frontmatter, correct code sync format).
Navigation: Sensibly placed in "LLM Features" section.
This is pragmatic documentation work - documents an existing feature users need to know about, does it simply and accurately, breaks nothing. Worth merging.
This review was posted by an AI assistant (OpenHands) on behalf of the user.
|
Final summary:
New outcome since the last summary:
PR: #202 |
…onvention - Rename sdk/guides/gpt5-preset.mdx → sdk/guides/llm-gpt5-preset.mdx - Update docs.json sidebar reference - Add SDK guide file naming convention (llm-, agent-, convo- prefixes) to AGENTS.md - Add naming convention check to code-review skill Addresses review feedback on PR #202. Co-authored-by: openhands <openhands@all-hands.dev>
…onvention (#434) - Rename sdk/guides/gpt5-preset.mdx → sdk/guides/llm-gpt5-preset.mdx - Update docs.json sidebar reference - Add SDK guide file naming convention (llm-, agent-, convo- prefixes) to AGENTS.md - Add naming convention check to code-review skill Addresses review feedback on PR #202. Co-authored-by: openhands <openhands@all-hands.dev>
Summary
Add an SDK guide for the GPT-5 preset example. It documents
get_gpt5_agent(llm), which creates an agent that opts intoApplyPatchTool-based editing instead of the defaultFileEditorTool.What changed
sdk/guides/gpt5-preset.mdxsoftware-agent-sdkexamples/04_llm_specific_tools/01_gpt5_apply_patch_preset.pyLLM_MODELselectionsdk/guides/gpt5-presetto the SDK sidebar under LLM FeaturesVerification
OpenHands/software-agent-sdkand verified the current example exists atexamples/04_llm_specific_tools/01_gpt5_apply_patch_preset.pymainin this docs repo does not already document this presetThis PR description was updated by an AI assistant (OpenHands) on behalf of the user.