Summary

Consolidate multiple buffering implementations in monitoring_system to simplify the codebase for C++20 module migration, following Kent Beck's "Fewest Elements" principle.

Parent Epic

Part of kcenon/common_system#256 (C++20 Module Migration - Cross-System Preparation)

Current State

Buffer Implementation Inventory

Analysis Results

Identified Issues

C++20 Module Impact

Clear separation between public and internal APIs needed for:

export module kcenon.monitoring;
export import :core;       // thread_local_buffer, central_collector
export import :buffers;    // time_series_buffer (public)
// Internal: ring_buffer, buffering_strategy, buffer_manager

Sub-Issues

• [REFACTOR] Consolidate load_average_history with time_series_buffer #311 - Consolidate load_average_history with time_series_buffer
• [REFACTOR] Make buffering_strategy use ring_buffer #312 - Make buffering_strategy use ring_buffer
• [REFACTOR] Clarify public vs internal buffer APIs #313 - Clarify public vs internal buffer APIs

Tasks

• Review thread_local_buffer usage and necessity → Keep (used by central_collector)
• Review time_series_buffer usage and necessity → Keep (used by system_resource_collector)
• Review buffer_manager and buffering_strategy overlap → Keep both (different purposes)
• Identify unused or redundant buffer implementations → load_average_history is redundant
• Create consolidation plan → Sub-issues created
• Implement changes with deprecation warnings
• Update documentation

Acceptance Criteria

• Clear separation between public and internal buffer APIs
• No redundant buffer implementations
• Unified buffering interface where applicable
• All tests pass
• No breaking changes to public API

Related

• C++20 Module Migration Epic ([EPIC] C++20 Module Migration - Cross-System Preparation common_system#256)
• Kent Beck "Fewest Elements" principle