Skip to content

chore!: remove the coder_report_task MCP tool - #30742

Merged
matifali merged 1 commit into
mainfrom
remove-report-task-tool
Oct 14, 2026
Merged

matifali merged 1 commit into
mainfrom
remove-report-task-tool

Conversation

@coder

@coder coder Bot commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Coder Tasks have been removed from coder/coder, so the coder_report_task MCP tool and the status reporting pipeline behind it no longer do anything useful. This PR removes the tool and the code that only existed to support it.

Removed

  • codersdk/toolsdk: ToolNameReportTask, ReportTaskArgs, ReportTask, Deps.report, WithTaskReporter, and the All entry, plus their tests.
  • coderd/mcp: the remote MCP exclusion for coder_report_task.
  • coder exp mcp server: the task reporter, the AgentAPI screen watcher, the agent socket connection, and the coder_report_task skip logic. The server now requires an authenticated user client, because every remaining tool needs one.
  • coder exp mcp configure claude-code: the mcp__coder__coder_report_task allowed tool, the injected coder_report_task CLAUDE.md prompt, the report tasks properly history line, the default system prompt asking the agent to send task status updates, and the CODER_MCP_APP_STATUS_SLUG / CODER_MCP_AI_AGENTAPI_URL env written to the Claude MCP config.
  • cli/cliutil/queue.go: only used by the reporter.
  • github.com/coder/agentapi-sdk-go (and transitive tmaxmax/go-sse) via go mod tidy.
  • The "task reporting" mention in docs/ai-coder/mcp-server.md.

Breaking: removed coder exp flags and env

coder exp is experimental. Any scripts that still pass these flags will now fail with an unknown flag error. Env vars that are no longer read are ignored.

Command Flag Env
coder exp mcp server --app-status-slug CODER_MCP_APP_STATUS_SLUG
coder exp mcp server --ai-agentapi-url CODER_MCP_AI_AGENTAPI_URL
coder exp mcp server --socket-path CODER_AGENT_SOCKET_PATH (still used by coder agent and coder exp sync)
coder exp mcp configure claude-code --claude-app-status-slug CODER_MCP_APP_STATUS_SLUG
coder exp mcp configure claude-code --ai-agentapi-url CODER_MCP_AI_AGENTAPI_URL

--claude-system-prompt no longer has a default.

Intentionally kept
  • Workspace app status reporting (UpdateAppStatus on the agent API and agent socket, agentsdk.PatchAppStatus / ProtoFromPatchAppStatus, the deprecated PATCH /workspaceagents/me/app-status endpoint): still served by coderd and the agent and part of versioned protocols. With this change, nothing in the CLI calls the agent socket UpdateAppStatus client anymore. Removing it would need a separate protocol change.
  • report_tasks in coderd/templatebuilder/modules/{aider,amazon-q}: these catalog entries mirror the input variables of pinned coder/registry module versions, and the behavior lives in those modules. Dropping the variable here would not disable it, because amazon-q defaults it to true. The registry modules need a follow-up instead.
  • UserClientOptional on toolsdk tools: other tools still set it, and remote MCP still uses it.
  • No CLI golden files or generated CLI docs changed, because coder exp commands are hidden.

Verification

  • go build ./...
  • go test ./codersdk/toolsdk/... ./coderd/mcp/... ./cli/cliutil/...
  • go test ./cli -run 'TestExpMcp|TestCommandHelp'
  • golangci-lint run ./cli/... ./codersdk/toolsdk/... ./coderd/mcp/... (0 issues)

Generated by Coder Agents.

Coder Tasks have been removed, so the coder_report_task tool and the
status reporting pipeline behind it no longer serve a purpose.

Remove the tool from toolsdk, the task reporter, AgentAPI screen
watcher, and agent socket wiring from `coder exp mcp server`, and the
report-task prompt, allowed tool, and env from
`coder exp mcp configure claude-code`. Drop the now-unused cliutil
queue and the agentapi-sdk-go dependency.

BREAKING CHANGE: `coder exp mcp server` no longer accepts
--app-status-slug, --ai-agentapi-url, or --socket-path, and
`coder exp mcp configure claude-code` no longer accepts
--claude-app-status-slug or --ai-agentapi-url.
@coder coder Bot assigned matifali Oct 14, 2026
@github-actions github-actions Bot added release/breaking This label is applied to PRs to detect breaking changes as part of the release process community Pull Requests and issues created by the community. labels Oct 14, 2026
@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Docs preview

Check off each page once it's been reviewed. If a page changes in a later push, its checkbox clears automatically so it gets a fresh look. Pages not yet wired into the docs navigation aren't listed here.

@coder
coder Bot marked this pull request as ready for review October 14, 2026 08:35
@coder
coder Bot requested a review from nickvigilante as a code owner October 14, 2026 08:35
@matifali matifali removed the community Pull Requests and issues created by the community. label Oct 14, 2026
@linear-code

linear-code Bot commented Oct 14, 2026

Copy link
Copy Markdown

CLI-61

@matifali
matifali merged commit 69fab7d into main Oct 14, 2026
64 of 65 checks passed
@matifali
matifali deleted the remove-report-task-tool branch October 14, 2026 11:35
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release/breaking This label is applied to PRs to detect breaking changes as part of the release process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants