Addresses feedback from PR #366 review: adds comprehensive test coverage for positional argument handling and ensures message validation is always enabled when a file path is provided.

Changes

• Test Coverage: Added TestPositionalArgumentFeature class with 5 tests covering:

  • Positional argument without --message flag
  • Positional argument with --message flag
  • Positional argument with other check flags (--branch)
  • Invalid commit message validation via positional argument
  • Non-existent file fallback behavior

• Logic Simplification: Refactored positional argument handling in main.py (lines 319-325) to unconditionally enable message checking when commit_msg_file is provided, eliminating conditional logic that could cause files to be silently ignored.

• Imperative Verb: Added "refactor" to IMPERATIVES set for conventional commit validation.

Test Results

$ pytest tests/main_test.py::TestPositionalArgumentFeature -v
collected 5 items

test_positional_arg_without_message_flag PASSED
test_positional_arg_with_message_flag PASSED  
test_positional_arg_with_branch_flag PASSED
test_positional_arg_invalid_commit PASSED
test_positional_arg_nonexistent_file PASSED

All new tests pass. One pre-existing test failure in test_main_with_message_empty_string_no_stdin_with_git is unrelated to these changes (mock configuration issue).

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.