Correct stale MCP-deploy guidance for Python/JS data apps#85
Draft
claude[bot] wants to merge 2 commits into
Draft
Correct stale MCP-deploy guidance for Python/JS data apps#85claude[bot] wants to merge 2 commits into
claude[bot] wants to merge 2 commits into
Conversation
The dataapp-development skill told agents that Python/JS data apps cannot be deployed via MCP and to fall back to kbagent / customer git. The Keboola MCP server now ships modify_python_js_data_app and deploy_data_app, which create/modify/deploy Python/JS apps via a managed-git draft->promote flow. Document that flow as the path MCP clients (including Kai, which has no kbagent) use, and scope the kbagent/customer-git path to CLI-only or customer-managed-repo cases. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T9Mn2fJt3LMVtmwnvkScD6
| For `authentication_type` defaults and the OIDC-downgrade footgun on updates, see [authentication.md](authentication.md) §MCP defaults. | ||
|
|
||
| **Limitations:** Streamlit type only. No Git deployment mode via MCP. No Python/JS type via MCP today (planned — see [python-js-apps.md](python-js-apps.md) "Deployment via MCP — PLACEHOLDER"). | ||
| **Limitations:** On a truly filesystem-less client, Streamlit type only. The MCP server *does* deploy Python/JS apps via a managed-git draft→promote flow (`modify_python_js_data_app` + `deploy_data_app`), but that flow requires the agent to run git itself (clone/commit/push — MCP never runs git for you), so it needs a filesystem and belongs to Path B rather than Path A. No Streamlit Git deployment mode via MCP (Code mode only). |
Collaborator
There was a problem hiding this comment.
I don't really get this @claude and I don't think it's true. Clearly, this wasn't cross-verified well enough with the MCP server and kai-agent backend.
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.
Requested by Jordan Burger · Slack thread
What
The
dataapp-developmentskill (build-time vendored into Keboola's Kai agent) told the agent that Python/JS data apps cannot be deployed via MCP and to use the kbagent CLI / customer-managed git instead.Before: an MCP-based client (Kai, which has no kbagent) reads the skill and concludes its only deployment path is unavailable — even though the tooling to deploy ships.
After: the skill documents the Keboola MCP server's supported flow for Python/JS data apps (
modify_python_js_data_app+deploy_data_app, managed-git draft→promote), presents it as the path MCP clients use, and scopes the kbagent/customer-git path to standalone/local scenarios rather than presenting it as the only option.How
Corrected the stale claims in
plugins/dataapp-developer/skills/dataapp-development/(deployment-paths.md, python-js-apps.md, and related references), matched the corrected flow to the shipped docstrings in keboola/mcp-servertools/data_apps.py, and fixeddeploy_data_appexamples to include the requiredactionargument. This addresses the highest-severity finding from the 2026-07-09 Kai prompt audit (related governance RFC: keboola/mcp-server#618). Docs-only change.🤖 Generated with Claude Code
https://claude.ai/code/session_01T9Mn2fJt3LMVtmwnvkScD6
Generated by Claude Code