NotificationsYou must be signed in to change notification settings
Fork906
Star10k

fix(cli): fix prompt issue in mcp configure claude-code#17599

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

johnstcn merged 3 commits intomainfromcj/exp-mcp-update-prompt

Apr 29, 2025

Merged

fix(cli): fix prompt issue in mcp configure claude-code#17599

johnstcn merged 3 commits intomainfromcj/exp-mcp-update-prompt

Apr 29, 2025

Conversation

Copy link

Member

johnstcn commentedApr 29, 2025•
edited
Loading

Updates default Coder prompt.
Skips the directions to report tasks if the pre-requisites are not available (agent token and app slug).
Adds the capability to override the default Coder prompt viaCODER_MCP_CLAUDE_CODER_PROMPT.

fix(cli): fix prompt issue in mcp configure claude-code

63e6379

johnstcn self-assigned this

Apr 29, 2025

johnstcn requested a review fromCopilot

April 29, 2025 10:55

CopilotAI reviewed

Apr 29, 2025

View reviewed changes

Copy link

Contributor

CopilotAI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Pull Request Overview

This PR fixes the prompt generation issue in the “mcp configure claude‐code” CLI command. It updates the default coder prompt text, introduces a new report task prompt that is conditionally included, and revises the associated tests to verify the new behaviors.

Updated expected prompt contents in tests based on agent token and app slug availability.
Introduced a new variable (reportTaskPrompt) and updated function signatures/calls (injectClaudeMD and promptsBlock) to incorporate the report task prompt.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
cli/exp_mcp_test.go	Adjusted test cases with revised expected MD content and string fixes
cli/exp_mcp.go	Updated prompt generation logic to conditionally include the report task prompt