Thomas Kosiewski d1595781e1 fix: fix nil pointer dereference in ReportTask (#19045) ...

This pull request addresses a bug related to a nil pointer dereference
in the task reporting functionality.

### Bug Fixes and Error Handling:

* Updated `RegisterTools` in `mcp.go` to skip registering the
`ReportTask` tool in the remote MCP context when a task reporter is not
configured, preventing potential nil pointer dereference panics.
* Added a check in `toolsdk.go` to ensure task reporting dependencies
are available before invoking the reporter, returning an appropriate
error if not.

### Test Coverage:

* Added `TestReportTaskNilPointerDeref` in `toolsdk_test.go` to verify
that the system does not panic when task reporting dependencies are
missing and instead returns a clear error message.
* Added `TestReportTaskWithReporter` in `toolsdk_test.go` to validate
correct behavior when a task reporter is configured, ensuring the
handler processes the request as expected.

Signed-off-by: Thomas Kosiewski <tk@coder.com>

2025-07-28 11:23:29 +02:00

..

mcp_e2e_test.go

feat: implement MCP HTTP server endpoint with authentication (#18670)

2025-07-03 19:27:41 +02:00

mcp_test.go

feat: implement MCP HTTP server endpoint with authentication (#18670)

2025-07-03 19:27:41 +02:00

mcp.go

fix: fix nil pointer dereference in ReportTask (#19045)

2025-07-28 11:23:29 +02:00