Open
Conversation
- 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
- Add min_column_width floor enforcement in TableConfig - Implement ANSI header row and alternating-row coloring - Fix border variant rendering, correct AsciiGrid corner characters - Add unicode_visual_len and pad_unicode_width for proper display width alignment - Expand TableConfig with pub(crate) accessors for border_variant and color - Add comprehensive test coverage for corner cases, borders, colors, and unicode handling - Update spec.md with rendering feature specifications - Document implementation and pitfalls in development notes
- Add alignment tests for unicode_box, markdown, bordered, and grid tables - Each test ensures all lines display with equal character width - Include comprehensive documentation of root causes and prevention strategies
- Prevent ANSI color bleed in multiline cells by iterating .lines() with per-line RESET - Fix table separator width alignment by delegating to helper functions for consistent padding - Fix column width calculation to use maximum single-line width instead of total string width - Add comprehensive tests for multiline cell rendering, borders, and color handling
- Delete implementation plan for tasks 012-015 after completion - Remove plan directory readme
- Document table rendering improvements (tasks 012-015): min_column_width floor, ANSI coloring, border variants, Unicode display width - Update test organization: 406 tests in `tests/` as standalone binaries, integration feature marked as legacy no-op - Clarify TableConfig API hardening (task 011) and unicode_box() fix for separator rendering - Update version to v0.10.0 and expand version history with detailed task breakdowns
- Add `indent_prefix` builder method and field to ExpandedConfig - Add `postgres_style()` and `property_style()` convenience constructors - Fix ANSI reset placement in multiline cells to prevent background-color bleed - Fix column width calculation to use per-line maximums for multiline cells - Fix table header separator alignment for unicode and markdown borders - Add comprehensive test suites for expanded formatter (546 lines) and indent feature (200 lines) - Update specification with detailed multiline, ANSI reset, and indentation documentation - Bump version to v0.11.0
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
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