[feat] Hide the Deploy button in agent playgrounds - #5941
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe playground header no longer shows a Deploy button for agent playgrounds or while agent status loads. Eligible non-embedded, non-evaluator, non-agent workflows retain the icon-only Deploy button. ChangesAgent playground deployment
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: 66a7c9cb-f88c-43be-8bc9-f89ba792686d
📒 Files selected for processing (1)
web/oss/src/components/Playground/Components/PlaygroundVariantConfig/assets/PlaygroundVariantConfigHeader.tsx
Railway Preview Environment
Updated at 2026-08-11T20:07:32.160Z |
Context
The agent playground showed two publish actions side by side: Commit and a labeled Deploy button. For agents, Deploy is not part of the workflow; Commit is the only publish action that matters there. Mahmoud decided to remove Deploy from the agent playground while keeping it for classic apps.
Changes
The Deploy button in the playground config header now renders only when the entity is not an agent. The check reuses
isAgentEffective, the signal the same file already uses to switch the header into agent mode. Since Deploy no longer renders for agents, the agent-only label/type/size override on the button was dead code and is removed; classic apps keep the icon-only Deploy exactly as before. The Commit button is untouched in both modes.Before: agent playground header shows Commit plus a labeled "Deploy" button.
After: agent playground header shows Commit only. Classic completion/chat playgrounds keep their Deploy button unchanged.
Other
DeployVariantButtonrender sites were checked and left alone: the Registry variants table is not a playground surface, and the embedded drawer path already hides Deploy unconditionally.Tests / notes
tsc --noEmiton web/oss: clean apart from one pre-existing unrelated error inagenta-entities/src/session/api/api.ts.next lint --fixon the touched file: clean.What to QA