docs: document extending a built-in agent kit - #25748
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
The change removes a redundant setup.install block from the built-in Claude kit example and adds a clear, accurate paragraph explaining what the abbreviated spec omits and directing custom-agent authors to the full guide. The new text is concise, correctly scoped, and free of style-guide violations. No actionable issues found.
The revised built-in Claude example still described an incomplete spec that readers could not reuse. Replace it and the related fork example with a working extends-based variant that restores approval prompts. Co-Authored-By: Claude <noreply@anthropic.com>
dvdksn
changed the title
The kits overview repeated the complete extends example from the dedicated examples page. Keep the runnable example in one place and link to it from the conceptual overview. Co-Authored-By: Claude <noreply@anthropic.com>
The kits overview retained a full Ruff example after the sandbox example moved to the dedicated examples page. Keep both kit-type sections conceptual and link to the canonical mixin patterns. Co-Authored-By: Claude <noreply@anthropic.com>
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
The changes are well-scoped and match the existing document's character. The kits.md overview sections are trimmed to conceptual prose and link to canonical runnable examples in kit-examples.md. The fork example is cleanly updated to use extends: claude with only an entrypoint override. No style violations, no missing redirects, no front-matter issues, no broken anchors.
The overview omitted MCP integration and described inherited volumes differently from the detailed example. Use the same verified inheritance list on both pages. Co-Authored-By: Codex <noreply@openai.com>
3 participants
Summary
Keep the mixin and sandbox-kit overview sections conceptual and link them to canonical runnable examples. Update the built-in agent fork example to use
extends: claude, inheriting Docker's full Claude integration while overriding only the entrypoint to use manual permission mode.@netlify /ai/sandboxes/customize/kits/
Closes docker/sbx-releases#150
Generated by Codex