AUTO : Forward from cleaning_5 to alpha #1625
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Move stderr before stdout in merged output display. - Ensures errors appear first without requiring scrolling. - Add fix documentation per bug fix workflow.
- Create new cli_tools crate for CLI-specific utilities - Migrate cli_output module from strs_tools to cli_tools - Extract general string/ANSI functions to strs_tools - Update unilang to use strs_tools functions directly - Add strs_tools string::lines module (head/tail/head_and_tail) - Add comprehensive tests for ANSI truncation and line filtering - Remove deprecated unilang::output::truncation module
- Change crate name from cli_tools to cli_fmt - Rename cli_output module to output - Update all import paths across workspace - Update documentation to reflect new name - Sync changelog and spec files
- Removed misleading #[deprecated] attribute from CommandRegistry::new() and command_add_runtime() - Updated all documentation to emphasize performance trade-offs vs lifecycle status - Clarified appropriate use cases: REPL, plugins, prototyping - Removed #[allow(deprecated)] from all 69 examples and tests - Split data module into organized submodules (argument_types, command_definition, command_status, error_types, namespace, validated_types) - Deleted unused dependencies (sysinfo, num_cpus) and obsolete files (context_rich_documentation.rs, cv_analysis.rs, documentation_updater.rs) - Added build helper tests for hint_generator and type_analyzer - Runtime registration is performance trade-off (10-50x slower), not deprecated API - Required by spec (FR-REG-2) with legitimate ongoing use cases
- Remove demo script and Python builder scripts - Reorganize docs to features/ subdirectory - Add comprehensive docs/readme.md navigation - Normalize solution file name to lowercase - Remove completed cleanup TODO comments - Update internal documentation references
- Extract CommandRegistryBuilder to command_registry_builder.rs - Extract DynamicCommandMap to dynamic_command_map.rs - Extract PerformanceMetrics to performance_metrics.rs - Update command_definition.rs docs to explain privacy constraints
- Create task backlog with multiline cells spec - Add task tracking readme with status table - Add wplan category mappings to unilang help
- Implement multiline cell rendering with per-row height - Add column truncation with ANSI-aware width calculation - Create manual test suite (truncation, multiline, combined) - Add automated tests for column and multiline features - Update spec with algorithm details and examples - Import ToString in truncate.rs for ANSI handling - Add format helper decoupling tests - Update all examples to use new formatting features
- Implement generic hierarchical configuration framework - Add 6-level resolution system (Runtime→Env→Local→Parent→Global→Default) - Support dual-pattern local configs (-app/ temporary, .app/ permanent) - Include YAML file I/O with atomic locking and metadata timestamps - Add automatic type detection (bool/int/float/string) - Provide trait-based customization (Defaults, Paths, Validator) - Include table/JSON/YAML display formatters - Add migration support for legacy config paths - Implement 39 comprehensive tests with zero mocking
- Read config directly from the locked file handle instead of reopening file - Add Read and Seek imports for file operations - Inline load_config_file logic to avoid TOCTOU race condition - Parse YAML content with proper parameters section handling
- Remove unused Seek and SeekFrom imports - Clone YAML values before passing to yaml_to_json
- Replace filesystem write with direct write to locked file in modify_config_with_lock - Prevent race conditions by writing to already-opened file handle - Extract helper function build_config_yaml_string for YAML construction - Extract helper function extract_created_at_from_yaml_string for timestamp parsing - Add file truncation and seek operations for cross-platform compatibility
- Add helper functions for cross-platform path checking - Replace hardcoded Unix separators with helpers that support both Unix and Windows - Ensure tests pass on Windows by handling backslash separators
- Create comprehensive parameter documentation - Document all 14 ConfigPaths trait methods - Add usage examples and default values - Update readme with complete feature guide - Expand spec with security requirements - Add 8 configurability tests - Update test documentation
- Delete migration module and all migration code - Remove migration feature flag from Cargo.toml - Update docs to remove migration references - Simplify manager.rs without migration logic - Remove migration tests and manual test plans - Update readme and spec to reflect v0.2.0
- add collection_tools imports across modules - enable path_unique_folder_name feature in pth - update unique_folder_name import path to pth::path module - replace bare HashMap/HashSet with collection:: qualified imports
- Remove empty dependencies/dev-dependencies/build-dependencies from 41 Cargo.toml files - Fix bare use statement propagation in mod_interface_meta (issue-001) - Migrate explicit path joins to use pth crate's error-returning join - Add allow(dead_code) to prefixed_with_super_maybe helper - Replace collection_tools HashMap/HashSet with std equivalents
- remove unused assert_cmd::cargo::cargo_bin_cmd import - replace macro calls with fully qualified path - eliminate import dependency in tests_run.rs
- remove redundant `assert_cmd ::cargo ::` prefix - use `assert_cmd ::cargo_bin_cmd!` directly
- Remove double cargo namespace from test command paths - Inline cargo_bin_cmd macro usage for direct command access - Clean up empty dependency tables from manifests
- Remove empty dependencies, dev-dependencies, build-dependencies from workspace root - Remove empty dependencies, dev-dependencies, build-dependencies from 35 crate manifests - Fix clippy warnings in strs_tools: replace unwrap with destructured variable - Fix variable shadowing in split.rs by using destructured first_char_iterable - Add backticks to no_std in test documentation for consistency
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is an automated pull request to update from branch cleaning_5