Skip to content

refactor(compat): add container namespace migration infrastructure (Issue #224)#237

Merged
kcenon merged 1 commit into
mainfrom
refactor/issue-224-prepare-container-namespace-migration
Feb 14, 2026
Merged

refactor(compat): add container namespace migration infrastructure (Issue #224)#237
kcenon merged 1 commit into
mainfrom
refactor/issue-224-prepare-container-namespace-migration

Conversation

@kcenon

@kcenon kcenon commented Feb 13, 2026

Copy link
Copy Markdown
Owner

Closes #224

Summary

  • Created centralized container_aliases.h for the upcoming container_module -> kcenon::container namespace migration
  • Completed Phase 1 (audit) and Phase 2 (migration infrastructure) of the issue scope
  • Phase 3 (actual namespace replacement) remains blocked until container_system v2.0 enforces the new namespace

Phase 1: Audit Results

Pattern Count Files
container_module:: (total) 368 46 .h/.cpp files
container_module:: (modules) 16 2 .cppm files
container_module::value_container 365 45 files
container_module::value_types 1 1 file
#include <core/container.h> 4 4 header files
container_module::value_variant 0 0 files
container_module::serialization_format 0 0 files
value_container_ptr (deprecated) 0 0 files

Total: ~384 references across 48 files

Phase 2: Migration Infrastructure

New file: include/kcenon/messaging/compat/container_aliases.h

Provides centralized aliases:

namespace kcenon::messaging::compat {
    namespace container_ns = container_module;  // -> kcenon::container
    using container_data = container_ns::value_container;  // -> message_buffer
    using container_data_ptr = std::shared_ptr<container_data>;
    using container_value_types = container_ns::value_types;
}

Migration Strategy

When container_system v2.0 enforces the new namespace:

  1. Update container_aliases.h: change container_module -> kcenon::container
  2. Update container_aliases.h: change value_container -> message_buffer
  3. Incrementally replace container_module:: references with aliases across codebase (Phase 3)

Phase 3: Blocked

Test Plan

  • New header compiles without errors (header-only, includes <core/container.h>)
  • No existing code modified (zero regression risk)
  • CI build passes on all platforms

…sue #224)

Create centralized type and namespace aliases in preparation for the
upcoming container_system namespace migration:
  container_module -> kcenon::container (namespace)
  value_container  -> message_buffer    (class rename)

Phase 1 audit results (368 references across 48 files):
- container_module::value_container: 365 refs in 45 files
- container_module::value_types: 1 ref in 1 file
- #include <core/container.h>: 4 header files
- C++20 module (.cppm) refs: 16 refs in 2 files

Phase 2 deliverable:
- include/kcenon/messaging/compat/container_aliases.h
- Namespace alias: container_ns = container_module
- Type aliases: container_data, container_data_ptr, container_value_types
- When upstream enforces new namespace, update only this file
@kcenon kcenon merged commit 2fc614a into main Feb 14, 2026
12 checks passed
@kcenon kcenon deleted the refactor/issue-224-prepare-container-namespace-migration branch February 14, 2026 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] Prepare for container_module to kcenon::container namespace migration

1 participant