Skip to content

upgrade_feature#63

Merged
stacksjb merged 2 commits into
mainfrom
list_sort
Jul 4, 2026
Merged

upgrade_feature#63
stacksjb merged 2 commits into
mainfrom
list_sort

Conversation

@stacksjb

@stacksjb stacksjb commented Jul 3, 2026

Copy link
Copy Markdown
Owner

Add new features! yay.

Copilot AI review requested due to automatic review settings July 3, 2026 12:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR expands the CLI menu’s functionality and robustness by adding config-path overrides, CSV export support, stronger config validation, and some refactors that improve testability and menu handling.

Changes:

  • Added --config/-c support to load/create a config from a user-supplied path, plus a CLI test for missing path handling.
  • Added CSV export support (with roundtrip test coverage) and improved import flow/cancellation handling.
  • Introduced validate_config for semantic validation and refactored menu selection parsing (main/edit menus) and command execution (injectable executor for tests).

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/assert_cmd.rs Adds integration coverage for --config requiring a path and returning exit code 2 with a specific stderr message.
src/utils.rs Introduces an injectable CommandExecutor abstraction and a run_command_with entrypoint to improve testability.
src/menu_main.rs Refactors main menu selection handling into parsed choices and improves error reporting when command execution fails.
src/menu_edit.rs Refactors edit menu options into enums, adds CSV export option, and centralizes save/validation flow.
src/main.rs Moves to async main, adds --config/-c parsing, and awaits the async menu.
src/csv.rs Adds CSV export and a write_commands_to_csv helper, improves prompt error handling, and adds sorting of discoverable CSV files.
src/config.rs Adds ensure_config_file_path, atomic-ish config writes via temp+rename, and new validate_config checks with tests.
README.md Documents the newly supported features (CSV export and alternate config path).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@stacksjb stacksjb merged commit 3176bcb into main Jul 4, 2026
13 checks passed
@stacksjb stacksjb deleted the list_sort branch July 4, 2026 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants