fix(cli): correct logic around CODER_MCP_APP_STATUS_SLUG (#17391)

johnstcn · web-flow · commit 2d2c9bda9899 · 2025-04-14T16:24:02.000+01:00
Past me was not smart.
diff --git a/cli/exp_mcp.go b/cli/exp_mcp.go
@@ -411,7 +411,7 @@ func mcpServerHandler(inv *serpent.Invocation, client *codersdk.Client, instruct
 	} else {
 		cliui.Warnf(inv.Stderr, "CODER_AGENT_TOKEN is not set, task reporting will not be available")
 	}
-	if appStatusSlug != "" {
+	if appStatusSlug == "" {
 		cliui.Warnf(inv.Stderr, "CODER_MCP_APP_STATUS_SLUG is not set, task reporting will not be available.")
 	} else {
 		clientCtx = toolsdk.WithWorkspaceAppStatusSlug(clientCtx, appStatusSlug)

Original file line number	Diff line number	Diff line change
`@@ -411,7 +411,7 @@ func mcpServerHandler(inv serpent.Invocation, client codersdk.Client, instruct`
`411`	`411`	`} else {`
`412`	`412`	`cliui.Warnf(inv.Stderr, "CODER_AGENT_TOKEN is not set, task reporting will not be available")`
`413`	`413`	`}`
`414`		`- if appStatusSlug != "" {`
	`414`	`+ if appStatusSlug == "" {`
`415`	`415`	`cliui.Warnf(inv.Stderr, "CODER_MCP_APP_STATUS_SLUG is not set, task reporting will not be available.")`
`416`	`416`	`} else {`
`417`	`417`	`clientCtx = toolsdk.WithWorkspaceAppStatusSlug(clientCtx, appStatusSlug)`