fix(worker): gate mcp discovery readiness diagnostics

[zebbern]

Summary

• route MCP discovery STDIO readiness progress diagnostics through workflowDiagnosticLog
• expand MCP discovery diagnostics tests to cover cache writes and successful stdio readiness without console.log noise
• keep warning/error paths and Docker cleanup behavior unchanged

Verification

• bun test worker/src/temporal/activities/tests/mcp-discovery.activity.test.ts
• bun test worker/src/temporal/activities
• bun test worker
• bun run typecheck
• bun run lint
• bun run test

[Copilot]

Pull request overview

This PR reduces noisy MCP discovery progress output by routing successful STDIO readiness diagnostics through the existing workflowDiagnosticLog gate (controlled by SENTRIS_DEBUG_WORKFLOW), and extends the worker activity tests to verify both cache writes and that successful readiness does not emit console.log output by default.

Changes:

• Replace console.log calls in waitForContainerReady with workflowDiagnosticLog to gate successful readiness/progress diagnostics.
• Expand MCP discovery activity tests to cover Redis cache writes and to assert no default console.log noise on successful STDIO readiness/tool discovery.
• Add module mocks in tests to safely exercise docker cleanup behavior without invoking real Docker.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
worker/src/temporal/activities/mcp-discovery.activity.ts	Gates readiness/progress logs by switching to workflowDiagnosticLog in the container readiness polling loop.
worker/src/temporal/activities/__tests__/mcp-discovery.activity.test.ts	Adds mocks + new assertions ensuring successful diagnostics stay silent by default and cache writes are exercised.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.