refactor(agent): remove unused Client.SDK() and Socket() accessors

[aashishrajdev]

Summary

Removes two exported accessors on the agent's Docker client that have no callers anywhere in the repo, tests included. SDK() handed out the raw Docker client, letting callers bypass the per-call timeouts and error wrapping that Service enforces. Socket() was the only reader of the socket field, so the field is removed too.

Type of change

• Bug fix
• New feature
• Refactor / cleanup
• Docs
• Infrastructure / CI
• Breaking change

Areas touched

• apps/server (NestJS)
• apps/web (React / Vite)
• apps/agent (Go)
• packages/protocol (shared WebSocket contracts)
• infrastructure / .github/workflows
• docs

Related issues

Closes #171

How to test

1. Confirm no callers remain (expect no output):

   grep -rn "\.SDK()\|\.Socket()" --include=*.go .

[aashishrajdev]

The failing check formatting step is pre-existing on develop: internal/communication/client.go has a whitespace-only line from e857fd0 (PR #155), which landed right after the gofmt check was added in #154. Every Agent run on develop since then fails the same way.

Added the one-line gofmt fix here so CI is green. It is the only file in apps/agent that gofmt rejects.

[rodriguecyber]

The failing check formatting step is pre-existing on develop: internal/communication/client.go has a whitespace-only line from e857fd0 (PR #155), which landed right after the gofmt check was added in #154. Every Agent run on develop since then fails the same way.

Added the one-line gofmt fix here so CI is green. It is the only file in apps/agent that gofmt rejects.

please look to the CI checks are failing

[aashishrajdev]

@rodriguecyber the ci is green now you can merge