airflow icon indicating copy to clipboard operation
airflow copied to clipboard

added chakra mcp configuration to setup_vscode.py

Open aaron-wolmutt opened this issue 3 weeks ago • 3 comments

added chakra mcp configuration to setup_vscode.py

  • Adds support for mcp.json in the setup_vscode.py script
  • Added docs for the mcp support in vscode

^ Add meaningful description above Read the Pull Request Guidelines for more information. In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed. In case of a new dependency, check compliance with the ASF 3rd Party License Policy. In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

aaron-wolmutt avatar Dec 07 '25 21:12 aaron-wolmutt

I don't think we want to do it. We do not want to tell enyone to use MCP and pre-configure it in their vscode.

If you want maybe you can add a separate script to set it up for those who want to use it explicitly.

I can modify the script to make configuring the chakra mcp server in VS Code optional. I started using the mcp and was thinking it might be useful for other developers.

aaron-wolmutt avatar Dec 09 '25 00:12 aaron-wolmutt

I can modify the script to make configuring the chakra mcp server in VS Code optional. I started using the mcp and was thinking it might be useful for other developers.

Sure. I understand it's useful. But I don't thing making it pretty much "mandatory" for anyone using VSCode is a good idea or something desired. It can be optional if you opt for it . It could be separate script or CLI option in the current script - but making it always configured and installed is just a bad idea.

potiuk avatar Dec 09 '25 13:12 potiuk

I made setting up the chakra ui mcp server in vscode optional.

I also added documentation for the dev tools feature. If other developers are using mcp servers in vscode I think they should contribute it to this script. mcp servers should be always optional, but I would like them to be available in the scripts.

aaron-wolmutt avatar Dec 09 '25 23:12 aaron-wolmutt