Skip to content

Improve post MCP installation message - #1822

Merged
lionello merged 1 commit into
mainfrom
kevin/mcpMessage
Jan 20, 2026
Merged

Improve post MCP installation message#1822
lionello merged 1 commit into
mainfrom
kevin/mcpMessage

Conversation

@KevyVo

@KevyVo KevyVo commented Jan 20, 2026

Copy link
Copy Markdown
Contributor

Description

Currently, the CLI prints the message:
“Run defang mcp setup to install the Defang MCP server.”
This leaves users a bit confused about what to do next. Adding more context would help better guide them through the next steps.

Linked Issues

#1746

Checklist

  • I have performed a self-review of my code
  • I have added appropriate tests
  • I have updated the Defang CLI docs and/or README to reflect my changes, if necessary

Summary by CodeRabbit

  • Documentation
    • Enhanced post-installation instructions to guide users on using the Defang MCP server with their choice of coding agents or the built-in agent option.

✏️ Tip: You can customize this high-level summary in your review settings.

@KevyVo KevyVo self-assigned this Jan 20, 2026
@coderabbitai

coderabbitai Bot commented Jan 20, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

The installation script gains two additional user-guidance messages displayed in interactive terminal sessions. These messages inform users about using the Defang MCP server with various coding agents and the option to run Defang's built-in agent, with no changes to control flow or error handling.

Changes

Cohort / File(s) Summary
User Guidance Messages
src/bin/install
Added two echo lines in the final user instructions block (TTY-only): clarifies MCP server compatibility with coding agents and promotes built-in agent option

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • jordanstephens
  • lionello

Poem

🐰 A rabbit hops with glee so bright,
Two helpful lines now shine in sight!
"Use any agent, or ours instead,"
The script now guides where users tread! 🌟

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: adding improved post-installation messaging to guide users on MCP server usage.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@KevyVo KevyVo linked an issue Jan 20, 2026 that may be closed by this pull request
@lionello
lionello merged commit 4821345 into main Jan 20, 2026
15 checks passed
@lionello
lionello deleted the kevin/mcpMessage branch January 20, 2026 20:26
KevyVo added a commit that referenced this pull request Jan 21, 2026
lionello added a commit that referenced this pull request Jan 21, 2026
* add warning for AWS_ACCESS_KEY and AWS_PROFILE

* fix test

* add warning to AWS env var

* Apply suggestions from code review

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

* remove check in stack

* Use assert

* Use assert

* fix: don't prompt in CI (#1815)

* fix: don't prompt in CI

Fixes #1814

* Apply suggestion from @lionello

[no ci]

* improve post MCP message (#1822)

* spacing

---------

Co-authored-by: Lio李歐 <lionello@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Post-install MCP Server setup is confusing

2 participants