Skip to content

refactor: Remove KCENON_WITH_MONITORING_SYSTEM compile-time dependency #345

Description

@kcenon

Summary

Remove all KCENON_WITH_MONITORING_SYSTEM compile-time guards and migrate to runtime EventBus pattern.

Parent Issue

Depends On

Files to Modify

  • include/kcenon/network/config/feature_flags.h
  • include/kcenon/network/integration/monitoring_integration.h
  • src/integration/monitoring_integration.cpp
  • include/kcenon/network/core/network_context.h
  • src/core/network_context.cpp

Tasks

  • Remove KCENON_WITH_MONITORING_SYSTEM definition from feature_flags.h
  • Remove #if KCENON_WITH_MONITORING_SYSTEM guards from monitoring_integration.h/cpp
  • Remove monitoring_system_adapter class (replaced by EventBus)
  • Update network_context to always use monitoring_integration_manager
  • Keep basic_monitoring as fallback for direct metric logging

Acceptance Criteria

  • No KCENON_WITH_MONITORING_SYSTEM references in source files
  • network_system builds without monitoring_system installed
  • Metrics flow via EventBus for external consumers

Metadata

Metadata

Assignees

No one assigned

    Labels

    architectureArchitectural changes and designrefactoringCode refactoring and improvements

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions