Commit 5749645
committed
feat!: remove -DisableTagSorting parameter and always enable intelligent tag temporal sorting (#6)
BREAKING CHANGE: Remove -DisableTagSorting parameter and always enable intelligent tag temporal sorting
This major simplification removes 195+ lines of conditional logic (26% code reduction)
and ensures optimal behavior by default. API Compatible Tags can now be listed in any
order as the script always uses actual git tag dates for chronological sorting.
## Breaking Changes
- Remove -DisableTagSorting parameter completely
- Eliminate all legacy manual ordering code paths
- API Compatible Tags no longer require manual temporal ordering
## Code Simplifications
- Sort-TagsByDate: Always perform temporal sorting when tag dates available
- Get-TagUnion: Always use temporal sorting, remove complex fallback logic
- Update-RepositoryDictionary: Single code path for both Strict/Permissive modes
- Remove $script:EnableTagSorting variable and all conditional logic
## Functions Modified
- Sort-TagsByDate: Remove conditional EnableTagSorting checks (-8 lines)
- Get-TagUnion: Remove manual ordering fallback logic (-65 lines)
- Update-RepositoryDictionary: Simplify Strict mode sections (-45 lines)
- Update-RepositoryDictionary: Simplify Permissive mode sections (-35 lines)
- Show-Summary: Remove conditional tag sorting status display (-12 lines)
- Main execution: Simplify startup logging (-8 lines)
- Parameter definitions: Remove DisableTagSorting param (-4 lines)
- Documentation: Clean up help text and comments (-18 lines)
## Enhanced Logging
- Add consistent Info-level start/end messages for all dependency processing depths
- Add depth 0 start/end messages: "Starting/Completed depth 0 processing"
- Enhanced depth completion statistics showing repositories examined, dependency files found, and new repositories checked out
- Add final completion message: "Recursive processing complete - no more nested dependencies found"
- Improve depth transition messages: "Processing nested dependencies from X repositories (moving to depth Y)"
- Fix depth numbering bug: correctly progress from depth 0 → 1 → 2 → 3 instead of 0 → 2 → 3 → 4
## Benefits
- Always optimal behavior: No configuration needed for intelligent tag sorting
- Simplified maintenance: API Compatible Tags can be in any order
- Better performance: Optimized algorithms always used
- Cleaner codebase: Single execution path eliminates branching overhead
- Enhanced reliability: Eliminates possibility of manual ordering errors
- Improved user experience: Zero configuration required for best behavior
- Better visibility: Clear logging of recursive dependency processing flow with accurate depth tracking
## Migration
- JSON files: No changes required - existing configurations work unchanged
- Command line: Remove any -DisableTagSorting parameter usage
- Behavior: Automatic improvement - always uses chronological intelligence
- Logging: Enhanced depth tracking provides better visibility into recursive processing
## Compatibility
- All existing JSON configuration files work without modification
- API Compatible Tags can now be listed in any order
- Same core functionality with always-optimal behavior
- Performance equal or better than previous optimized path
- Improved logging provides better troubleshooting capabilities
Closes #61 parent 53475e5 commit 5749645
3 files changed
+297
-484
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
8 | 55 | | |
9 | 56 | | |
10 | 57 | | |
| |||
0 commit comments