Skip to content

refactor(adapters): consolidate bidirectional adapter files #599

Description

@kcenon

What

The monitoring_system has 8 separate adapter header files for bridging with other systems:

  • common_monitor_adapter.h / common_to_monitoring_adapter.h
  • monitoring_to_common_adapter.h / common_system_adapter.h
  • thread_system_adapter.h / thread_to_monitoring_adapter.h
  • logger_system_adapter.h / logger_to_monitoring_adapter.h

Bidirectional adapters are split into separate files (A→B and B→A), doubling the maintenance surface.

Why

  • 8 adapter files for 3 system integrations is excessive
  • Each system change requires updating up to 4 adapter files
  • Consolidating bidirectional pairs reduces file count and maintenance burden

Where

  • include/kcenon/monitoring/adapters/ — 8 adapter headers

How

Acceptance Criteria

  • Bidirectional adapter pairs merged (e.g., common↔monitoring in one file)
  • Public API remains unchanged (aliases or re-exports if needed)
  • Reduced from 8 to 3-4 adapter files

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/coreCore architecture and infrastructurepriority/mediumMedium priority - Important but not urgentsize/MMedium - 1-3 days of worktype/refactorCode refactoring without functional changes

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions