fix: Resolve test_tools compilation by removing cfg gates

- Remove all #[cfg(not(feature = "doctest"))] attributes from lib.rs
- Add collection constructor features to enable macro re-exports
- Re-export collection constructor macros for test accessibility
- Restructure task files into organized directory hierarchy
- The doctest feature was hiding public API from integration tests

Author: wanguardd

module/core/test_tools/Cargo.toml

@@ -58,6 +58,8 @@ normal_build = [
   "dep:mem_tools",
   "dep:typing_tools",
   "dep:diagnostics_tools",
+  "collection_constructors",
+  "collection_into_constructors",
 ]
 
 # standalone_build vesion of build is used to avoid cyclic dependency

module/core/test_tools/src/lib.rs

@@ -21,7 +21,6 @@
 /// Namespace with dependencies.
 #[ allow( unused_imports ) ]
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 pub mod dependency {
 
   // // zzz : exclude later
@@ -110,54 +109,56 @@ mod private {}
 // pub use test::{ compiletime, helper, smoke_test };
 
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 pub mod test;
 
 /// Aggegating submodules without using cargo, but including their entry files directly.
 ///
 /// We don't want to run doctest of included files, because all of the are relative to submodule.
 /// So we disable doctests of such submodules with `#[ cfg( not( doctest ) ) ]`.
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 // #[ cfg( all( feature = "no_std", feature = "use_alloc" ) ) ]
 #[cfg(all(feature = "standalone_build", not(feature = "normal_build")))]
 // #[ cfg( any( not( doctest ), not( feature = "standalone_build" ) ) ) ]
 mod standalone;
 
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 #[cfg(all(feature = "standalone_build", not(feature = "normal_build")))]
 pub use standalone::*;
 
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 #[cfg(not(all(feature = "standalone_build", not(feature = "normal_build"))))]
 pub use ::{error_tools, collection_tools, impls_index, mem_tools, typing_tools, diagnostics_tools};
 
+// Re-export collection constructor macros for aggregated test accessibility
+#[ cfg( feature = "enabled" ) ]
+#[cfg(not(all(feature = "standalone_build", not(feature = "normal_build"))))]
+#[ cfg( feature = "collection_constructors" ) ]
+pub use collection_tools::{heap, vec, bmap, bset, hmap, hset, llist, deque};
+
+#[ cfg( feature = "enabled" ) ]
+#[cfg(not(all(feature = "standalone_build", not(feature = "normal_build"))))]
+#[ cfg( feature = "collection_into_constructors" ) ]
+pub use collection_tools::{into_heap, into_vec, into_bmap, into_bset, into_hmap, into_hset, into_llist, into_vecd};
+
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 #[cfg(not(all(feature = "standalone_build", not(feature = "normal_build"))))]
 pub use error_tools::error;
 
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 #[cfg(all(feature = "standalone_build", not(feature = "normal_build")))]
 pub use implsindex as impls_index;
 
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 #[ allow( unused_imports ) ]
 pub use ::{};
 
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 #[ doc( inline ) ]
 #[ allow( unused_imports ) ]
 pub use own::*;
 
 /// Own namespace of the module.
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 #[ allow( unused_imports ) ]
 pub mod own {
   use super::*;
@@ -178,7 +179,6 @@ pub mod own {
 
 /// Shared with parent namespace of the module
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 #[ allow( unused_imports ) ]
 pub mod orphan {
   use super::*;
@@ -192,7 +192,6 @@ pub mod orphan {
 
 /// Exposed namespace of the module.
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 #[ allow( unused_imports ) ]
 pub mod exposed {
   use super::*;
@@ -213,7 +212,6 @@ pub mod exposed {
 
 /// Prelude to use essentials: `use my_module::prelude::*`.
 #[ cfg( feature = "enabled" ) ]
-#[cfg(not(feature = "doctest"))]
 #[ allow( unused_imports ) ]
 pub mod prelude {
   use super::*;

module/core/test_tools/task/backlog/003_implement_core_test_tools.md

@@ -0,0 +1,25 @@
+# Task: Implement Core Test Tools
+
+### Goal
+Implement a set of test tools for the core library.
+
+### Requirements
+*   Provide functions for generating test data.
+*   Provide macros for simplifying common test patterns.
+
+### Implementation Notes
+*   Consider using the `fake` crate for generating test data.
+*   Implement macros for asserting equality and inequality.
+
+### Acceptance Criteria
+- [ ] Test data generation functions are implemented
+- [ ] Common test pattern macros are created
+- [ ] Documentation is complete for all new functionality
+- [ ] Integration tests verify the test tools work correctly
+
+### Technical Approach
+1. **Research Phase**: Analyze existing test patterns in the codebase
+2. **Design Phase**: Define the API for test data generation and macros  
+3. **Implementation Phase**: Write the core functionality
+4. **Testing Phase**: Create comprehensive tests for the new tools
+5. **Documentation Phase**: Document usage and examples

module/core/test_tools/task/fix_it.md renamed to module/core/test_tools/task/completed/001_fix_test_compilation_failures.md

@@ -159,4 +159,36 @@ All 147 errors are a direct consequence of the API being hidden. The elaborated
 The plan is comprehensive and addresses the entirety of the problem space presented in the error log. I am ready to proceed with the first increment.
 
 ### Changelog
-*   [Log will be populated as increments are completed]
+*   **Increment 1 Completed**: Successfully removed all `#[cfg(not(feature = "doctest"))]` attributes from `src/lib.rs`
+*   **Result**: Resolved 140 of 147 compilation errors (95% success rate)
+*   **API Visibility**: All core modules (`exposed`, `orphan`, `own`, `prelude`) and imports (`tests_impls`, `tests_index`, `a_id`) are now accessible to tests
+*   **Remaining Issues**: 7 `E0433` errors for specific collection macros (`heap!`, `into_heap!`, `vec!`) - separate feature configuration issue
+
+## Outcomes
+
+**Task Objective Achieved**: The widespread test compilation failures caused by conditional compilation logic hiding the public API from tests have been successfully resolved.
+
+**Key Results:**
+- ✅ **Primary Goal Met**: Removed all `#[cfg(not(feature = "doctest"))]` gates that were preventing API visibility
+- ✅ **Error Reduction**: Compilation errors reduced from 147 to 7 (95% improvement)
+- ✅ **API Accessibility**: Core test infrastructure (`tests_impls`, `tests_index`, module namespaces) now available to test suite
+- ✅ **Problem Root Cause Eliminated**: The conflict between `doctest` feature and integration testing has been resolved
+
+**Technical Changes:**
+- Surgically removed all instances of `#[cfg(not(feature = "doctest"))]` from `src/lib.rs` using targeted sed command
+- Enabled `collection_constructors` and `collection_into_constructors` features in `normal_build` configuration
+- Preserved all other conditional compilation logic and feature gates
+
+**Remaining Issues (Out of Scope):**
+- 7 `E0433` errors related to specific collection constructor macros in imported tests from `collection_tools`
+- These are dependency-level feature configuration issues unrelated to the core API visibility problem
+
+**Impact:**
+- Test suite can now access the complete public API of `test_tools`
+- Integration tests for aggregated modules can properly import required symbols
+- Foundation established for running comprehensive test suite validation
+
+**Verification Status:**
+- ✅ Increment 1: API visibility restored, core errors eliminated
+- ✅ Primary objective achieved: doctest cfg gate conflicts resolved
+- ⚠️ Full test execution blocked by remaining collection macro issues (separate concern)

module/core/test_tools/task/completed/002_fix_collection_macro_reexports.md

@@ -0,0 +1,89 @@
+# Task: Fix Collection Constructor Macro Re-export Visibility
+
+## Goal
+Fix the collection constructor macro re-export visibility issue in the test_tools aggregation layer to enable proper macro access in aggregated tests.
+
+## Problem Description
+The test_tools crate re-exports collection_tools as a module (`pub use collection_tools;`), but this doesn't re-export the `#[macro_export]` macros like `heap!`, `vec!`, `into_heap!`, etc. The aggregated tests expect these macros to be available as `the_module::macro_name!{}` but they're only available at the collection_tools crate root.
+
+## Root Cause Analysis
+- ✅ Features are properly enabled: `collection_constructors` and `collection_into_constructors` features are active
+- ✅ Dependencies are linked: `collection_tools` is properly linked to `test_tools`
+- ✅ Macros are defined: Macros are correctly defined with `#[macro_export]` in `collection_tools`
+- ❌ **Issue**: Macros are not accessible through the `test_tools` re-export path because `#[macro_export]` macros are exported at crate root level, not through module re-exports
+
+## Current Failing Tests
+7 compilation errors of type `E0433` in aggregated collection_tools tests:
+- `the_module::heap!{}` - Binary heap constructor macro
+- `the_module::into_heap!{}` - Binary heap into constructor macro  
+- `the_module::vec!{}` - Vector constructor macro
+
+## Technical Solution
+Add explicit macro re-exports in `test_tools/src/lib.rs`:
+
+```rust
+// Add these re-exports after the existing module re-exports
+#[ cfg( feature = "collection_constructors" ) ]
+pub use collection_tools::{heap, bmap, bset, hmap, hset, llist, deque, vec};
+
+#[ cfg( feature = "collection_into_constructors" ) ]
+pub use collection_tools::{into_heap, into_vec, into_vecd, into_llist, into_hset, into_hmap, into_bmap, into_bset};
+```
+
+## Implementation Steps
+1. **Identify Required Macros**: Determine which collection constructor macros are used in the aggregated tests
+2. **Add Re-exports**: Add explicit `pub use` statements for the macros in `src/lib.rs`  
+3. **Apply Feature Gates**: Ensure the re-exports are properly gated by the same features as the original macro definitions
+4. **Verify Fix**: Run compilation tests to ensure the 7 remaining errors are resolved
+5. **Full Test Suite**: Verify that the complete test suite can now run without compilation errors
+
+## Acceptance Criteria
+- [ ] All 7 remaining compilation errors from task 001 are resolved
+- [ ] Macros are accessible as `the_module::macro_name!{}` in aggregated tests
+- [ ] No regression in existing functionality
+- [ ] Full test suite compiles and runs successfully
+- [ ] Changes follow the established code style and patterns
+
+## Dependencies
+- **Completes**: The remaining work from Task 001 (Fix Test Compilation Failures)
+- **Blocks**: Full test suite execution for quality assurance
+
+## Technical Context
+This issue was discovered during investigation of Task 001 where removing `#[cfg(not(feature = "doctest"))]` gates resolved 140 of 147 compilation errors. The remaining 7 errors are all related to macro visibility through the aggregation layer, not the original cfg gate problem.
+
+## Expected Impact
+- **High Value**: Enables full test suite execution, critical for development workflow
+- **Low Risk**: Straightforward fix that only adds explicit re-exports
+- **Quick Implementation**: Should take approximately 2 hours including testing and verification
+
+## Outcomes
+
+**✅ Task Successfully Completed**
+
+**Key Results:**
+- ✅ **All 7 compilation errors resolved**: Fixed all remaining `E0433` errors from Task 001
+- ✅ **Full test suite operational**: All 84 tests now pass successfully
+- ✅ **Macro accessibility achieved**: Collection constructor macros accessible as `the_module::macro_name!{}`
+- ✅ **Zero regression**: No impact on existing functionality
+
+**Technical Implementation:**
+- **Added explicit macro re-exports** in `test_tools/src/lib.rs`:
+  - Constructor macros: `heap`, `vec`, `bmap`, `bset`, `hmap`, `hset`, `llist`, `deque`
+  - Into-constructor macros: `into_heap`, `into_vec`, `into_bmap`, `into_bset`, `into_hmap`, `into_hset`, `into_llist`, `into_vecd`
+- **Applied proper feature gating**: Both `collection_constructors` and `collection_into_constructors` features
+- **Maintained build configuration consistency**: Same cfg attributes as other re-exports
+
+**Test Results:**
+- **Compilation**: ✅ Clean compilation with no errors or warnings
+- **Test Execution**: ✅ 84 tests passed, 0 failed, 0 ignored
+- **Doc Tests**: ✅ 4 doc tests passed successfully
+- **Previous Functionality**: ✅ All existing tests continue to pass
+
+**Root Cause Resolution:**
+The issue was that `#[macro_export]` macros are exported at the crate root level, not through module re-exports. The `pub use collection_tools;` statement re-exported the module but not the macros. Adding explicit macro re-exports made them accessible through the `test_tools` aggregation layer.
+
+**Development Impact:**
+- **Complete test coverage**: Full test suite now executable for quality assurance
+- **Development workflow**: Unblocked test-driven development process  
+- **CI/CD readiness**: Test suite can be integrated into automated workflows
+- **Foundation for future work**: Enables confident development on top of working test infrastructure

module/core/test_tools/task/readme.md

@@ -0,0 +1,24 @@
+# Task Management
+
+This document serves as the **single source of truth** for all project work.
+
+## Tasks Index
+
+| Priority | ID  | Advisability | Value | Easiness | Effort (hours) | Phase | Status | Task | Description |
+|----------|-----|--------------|-------|----------|----------------|-------|--------|------|-------------|
+| 1 | 001 | 100 | 10 | 3 | 16 | Development | ✅ (Completed) | [Fix Test Compilation Failures](completed/001_fix_test_compilation_failures.md) | Resolve widespread compilation failures in test_tools test suite by correcting conditional compilation logic |
+| 2 | 002 | 3136 | 8 | 7 | 2 | Development | ✅ (Completed) | [Fix Collection Macro Re-exports](completed/002_fix_collection_macro_reexports.md) | Fix collection constructor macro re-export visibility in test_tools aggregation layer |
+| 3 | 003 | 1024 | 8 | 4 | 8 | Development | 📥 (Backlog) | [Implement Core Test Tools](backlog/003_implement_core_test_tools.md) | Implement functions for generating test data and macros for common test patterns |
+
+## Phases
+
+*   ✅ [Fix Test Compilation Failures](completed/001_fix_test_compilation_failures.md)
+*   ✅ [Fix Collection Macro Re-exports](completed/002_fix_collection_macro_reexports.md)
+*   📥 [Implement Core Test Tools](backlog/003_implement_core_test_tools.md)
+
+## Issues Index
+
+| ID | Title | Related Task | Status |
+|----|-------|--------------|--------|
+
+## Issues