Summary

Decouple network_system's monitoring integration by replacing direct monitoring_system dependency with EventBus-based metric publishing using common_system interfaces.

5W1H Specification

• Who: network_system maintainers
• What: Replace direct monitoring_system dependency with IMetricCollector interface + EventBus pattern
• Where: include/kcenon/network/integration/, monitoring_integration.h
• When: After common_system#234 is merged (IMetricCollector interface) ✅ MERGED
• Why:
  • Current KCENON_WITH_MONITORING_SYSTEM creates optional bidirectional dependency
  • Bidirectional dependencies complicate build order and isolated testing
  • EventBus pattern enables metric emission without compile-time coupling
• How:
  1. Use common_system IMetricCollector for metric emission
  2. Optionally publish NetworkMetricEvent via EventBus for external consumers
  3. Remove compile-time monitoring_system dependency

Priority

HIGH - Resolves bidirectional dependency with monitoring_system

Sub-Issues

• refactor: Define NetworkMetricEvent struct for EventBus-based metric publishing #343 - Define NetworkMetricEvent struct for EventBus-based metric publishing
• refactor: Implement EventBus-based metric publishing in network_metrics #344 - Implement EventBus-based metric publishing in network_metrics
• refactor: Remove KCENON_WITH_MONITORING_SYSTEM compile-time dependency #345 - Remove KCENON_WITH_MONITORING_SYSTEM compile-time dependency
• refactor: Update CMake to remove monitoring_system link dependency #346 - Update CMake to remove monitoring_system link dependency
• docs: Update documentation for EventBus-based monitoring #347 - Update documentation for EventBus-based monitoring

Current State Analysis

Current:
network_system ──KCENON_WITH_MONITORING_SYSTEM──> monitoring_system
monitoring_system ──MONITORING_HAS_NETWORK_SYSTEM──> network_system
                    ↑ Bidirectional risk ↑

Target:
network_system ──> common_system (IMetricCollector, EventBus)
                         ↑
monitoring_system ──────┘ (subscribes to events)

Tasks (Now tracked in sub-issues)

• Wait for common_system#234 (IMetricCollector interface) to merge
• refactor: Define NetworkMetricEvent struct for EventBus-based metric publishing #343 - Define NetworkMetricEvent struct in include/kcenon/network/events/
• refactor: Implement EventBus-based metric publishing in network_metrics #344 - Implement EventBus-based metric publishing
• refactor: Remove KCENON_WITH_MONITORING_SYSTEM compile-time dependency #345 - Remove KCENON_WITH_MONITORING_SYSTEM compile-time dependency
• refactor: Update CMake to remove monitoring_system link dependency #346 - Update CMake configuration
• docs: Update documentation for EventBus-based monitoring #347 - Update documentation and migration guide

Acceptance Criteria

• network_system builds without monitoring_system installed
• Metrics still flow to monitoring_system when both are present (via EventBus)
• No KCENON_WITH_MONITORING_SYSTEM guards in installed headers
• Integration tests pass in isolation

Dependencies

• Blocked by: deps: Add unified metric collection interface (IMetricCollector) common_system#234 (IMetricCollector interface) ✅ CLOSED
• Enables: Clean bidirectional decoupling with monitoring_system

Parent Epic

• Epic: Cross-Module Circular Dependency Resolution common_system#232

Related Issues

• Switch integration flags to KCENON_* #335 (KCENON integration flags)