Remove `ToolType` from `Workload` struct and delete all references

[amirejaz]

We will remove the ToolType column in PR, but it is still part of the Workload struct. ToolType always points to either "mcp" (for container workloads) or "remote" (for remote workloads). It was initially created to differentiate between mcp and other tool types, but MCP seems to stand out without any other type. We can safely remove this field and all its references.

Scope of changes:

• Remove ToolType field from core.Workload struct
• Remove LabelToolType and GetToolType from labels package
• Simplify shouldSkipWorkload logic in client manager (currently checks ToolType != "mcp" && !Remote, can be simplified to just check Remote field)
• Remove ToolType from list command output
• Remove ToolType from backend metadata (or keep hardcoded "mcp" in VMCP if needed for external systems)
• Update tests and API documentation

[deepika1214]

Hi @amirejaz, I would like to work on this issue. Can you please assign it to me!

[carlos-gn]

hey @eleftherias

[eleftherias]

Thanks @carlos-gn, I've assigned the issue to you.

[eleftherias]

Hey @deepika1214, thanks for offering, we already had @carlos-gn volunteer to work on this earlier today in #2917 (comment), so I've assigned it to him. We will have some new "good first issues" on Monday, there might be something there that's interesting to you.

[deepika1214]

Sure, Thanks for letting me know!