fix(worker): gate mcp cleanup diagnostics

[zebbern]

Summary

• gate successful MCP cleanup status diagnostics behind workflowDiagnosticLog
• keep warning/error output and explicit SKIP_CONTAINER_CLEANUP=true debug instructions unchanged
• add a regression test covering default-quiet cleanup behavior

Verification

• red test first: bun test worker/src/temporal/activities/__tests__/mcp.activity.test.ts failed with 3 console.log calls
• bun test worker/src/temporal/activities/__tests__/mcp.activity.test.ts
• bun test worker/src/temporal/activities
• git diff --check
• bun run lint
• bun run typecheck
• bun run test
• bunx tsc --build worker --force && bun run typecheck after full test cleanup
• git push -u origin codex/quiet-mcp-cleanup-diagnostics pre-push hook

[Copilot]

Pull request overview

This PR reduces log noise from MCP cleanup by routing successful cleanup diagnostics through workflowDiagnosticLog (tied to SENTRIS_DEBUG_WORKFLOW), while keeping warning/error logs and the explicit SKIP_CONTAINER_CLEANUP=true debugging instructions unchanged.

Changes:

• Replaced successful MCP cleanup console.log statements with workflowDiagnosticLog in cleanupRunResourcesActivity.
• Added a regression test ensuring default behavior does not emit successful cleanup diagnostics to console.log.

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.activity.ts	Gates successful MCP cleanup diagnostics behind workflowDiagnosticLog while leaving warnings/errors and SKIP-cleanup instructions as-is.
worker/src/temporal/activities/__tests__/mcp.activity.test.ts	Adds regression coverage verifying successful cleanup does not write to console.log by default.

---

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