Skip to content

Commit e99ff09

Browse files
elifarleyclaude
andcommitted
Fix tree renderer default sort behavior
- Change effective_sort logic from `if sort_spec is not None` to `if sort_spec` - This ensures both None and empty string trigger default mtime descending sort (-d) - Aligns tree renderer with flat renderer behavior - Fixes test_tree_sibling_sort_by_name_reverse which expects default mtime descending 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 3896cc0 commit e99ff09

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/cedarmapper/render/tree.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ def render_tree(
5151

5252
# prepare comparator: default to '-d' (mtime desc) if user didn't supply a sort
5353
comparator = None
54-
effective_sort = sort_spec if sort_spec is not None else "-d"
54+
effective_sort = sort_spec if sort_spec else "-d"
5555
specs = parse_sort_spec(effective_sort)
5656
if specs:
5757
comparator = make_comparator(specs)

0 commit comments

Comments
 (0)