feat(claude): remove coder_report_task tool call message

[35C4n0r]

Closes: #155

• This PR introduces a method for claude to remove the coder's report task tool call.

[github-actions]

✅ Preview binaries are ready!

To test with modules: agentapi_version = "agentapi_157" or download from: https://github.com/coder/agentapi/releases/tag/agentapi_157

[mafredri]

If I understand correctly, this will remove tool calls entirely from message history as well? Would it be valuable to keep them around, but as a different type of message, or under a new endpoint?

For debuggability, I could see them being useful.

[35C4n0r]

If I understand correctly, this will remove tool calls entirely from message history as well? Would it be valuable to keep them around, but as a different type of message, or under a new endpoint?

For debuggability, I could see them being useful.

Makes sense to keep them around for debugging, imo we can just log them for now - this should be enough for our usecase as the logs are stored in agentapi-start.log in our workspaces.
WDYT @mafredri

[mafredri]

Makes sense to keep them around for debugging, imo we can just log them for now - this should be enough for our usecase as the logs are stored in agentapi-start.log in our workspaces. WDYT @mafredri

Reasonable 👍🏻

[mafredri]

One last concern, but otherwise LGTM 👍🏻

[mafredri]

Is it possible that ● coder - coder_report_task (MCP) is the last entry, and there won't be a next message with ●? In which I assume we'd remove all the remaining lines?

Asking because I don't see handling for this case (unless I'm just missing it).

[35C4n0r]

Is it possible that ● coder - coder_report_task (MCP) is the last entry, and there won't be a next message with ●? In which I assume we'd remove all the remaining lines?

@mafredri
Added the logic to handle this case. This handles the case where this is the last thing claude outputs is the tool call, and it also acts as a failsafe if the next new message is not prefixed with ●.
I'm checking for a newline after the tool-call and not directly using len(lines) in above case, because I never want to run into this: In which I assume we'd remove all the remaining lines?

[matifali]

Thanks for the quick fix here Jay.
Let's update Claude's code module to use this new version when it lands.
@david-fraley can inform the customer