v27: MCP integration and documentation#27
Conversation
BaseAgent.send_message() now logs warning if namespace not configured, making silent failures visible. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Agents now report status='EXITED' before disconnecting from ActiveMQ. This distinguishes clean exits from crashes (which leave status='OK' with stale heartbeat). Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Test Coverage Summary |
There was a problem hiding this comment.
Pull request overview
This PR adds agent lifecycle tracking and namespace configuration warnings to improve monitoring and multi-tenant deployments. The changes focus on workflow agent improvements from v26, though the PR description mentions MCP integration and documentation that are not visible in the provided diff.
Key changes:
- Exit status reporting when agents shut down for better lifecycle tracking
- Namespace configuration warnings to guide proper multi-tenant setup
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| else: | ||
| logging.warning( | ||
| f"Sending message without namespace (msg_type={message_body.get('msg_type', 'unknown')}). " | ||
| "Configure namespace in testbed.toml to enable namespace filtering." | ||
| ) |
There was a problem hiding this comment.
This warning will be logged on every message sent when namespace is not configured. For deployments that intentionally don't use namespaces (which is valid since config_path is optional), this creates excessive log noise. Consider either: (1) logging this once during initialization instead of on every send_message call, (2) changing to logging.debug() or logging.info() level since it's informational rather than a warning about a problem, or (3) adding a flag to suppress repeated warnings.
Summary
Changes
/swf-monitor/mcp/Test plan
🤖 Generated with Claude Code