Skip to content

fix: correct environment variable name for MCP app status slug #17948

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

ThomasK33
Copy link
Member

@ThomasK33 ThomasK33 commented May 20, 2025

Fixed environment variable name for app status slug in Claude MCP configuration from CODER_MCP_CLAUDE_APP_STATUS_SLUG to CODER_MCP_APP_STATUS_SLUG to maintain consistency with other MCP environment variables.

This also caused the User level Claude.md to not contain instructions to report its progress, so it did not receive status reports.

Change-Id: If8242755e1de90b6aa7c5d3413c3812217a5e536
Signed-off-by: Thomas Kosiewski <tk@coder.com>
@ThomasK33 ThomasK33 self-assigned this May 20, 2025
Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ThomasK33 ThomasK33 requested a review from johnstcn May 20, 2025 17:24
@ThomasK33 ThomasK33 marked this pull request as ready for review May 20, 2025 17:25
Copy link
Member

@johnstcn johnstcn left a comment

Choose a reason for hiding this comment

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

nice find!

@bpmct
Copy link
Member

bpmct commented May 20, 2025

Does this break our modules at all?

Copy link
Member Author

On the contrary, this fixes it.

All the docs mentioned the CODER_MCP_APP_STATUS_SLUG variable, and the module used that one too. We just happened to accidentally add that CLAUDE in between it in this flag.

@ThomasK33 ThomasK33 merged commit b551a06 into main May 20, 2025
41 checks passed
Copy link
Member Author

Merge activity

@ThomasK33 ThomasK33 deleted the thomask33/05-20-fix_correct_environment_variable_name_in_claude_code_mcp_configuration branch May 20, 2025 17:35
@bpmct
Copy link
Member

bpmct commented May 20, 2025

/cherry-pick release/2.22

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request May 20, 2025
Fixed environment variable name for app status slug in Claude MCP configuration from `CODER_MCP_CLAUDE_APP_STATUS_SLUG` to `CODER_MCP_APP_STATUS_SLUG` to maintain consistency with other MCP environment variables.

This also caused the User level Claude.md to not contain instructions to report its progress, so it did not receive status reports.
bpmct pushed a commit that referenced this pull request May 20, 2025
…y-pick #17948) (#17950)

Cherry-picked fix: correct environment variable name for MCP app status
slug (#17948)

Fixed environment variable name for app status slug in Claude MCP
configuration from `CODER_MCP_CLAUDE_APP_STATUS_SLUG` to
`CODER_MCP_APP_STATUS_SLUG` to maintain consistency with other MCP
environment variables.

This also caused the User level Claude.md to not contain instructions to
report its progress, so it did not receive status reports.

Co-authored-by: Thomas Kosiewski <tk@coder.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.

3 participants