Enhance args.sh pattern collection logic

Add better handling for non-option arguments in pattern collection:
* feat: Added check to skip script name as pattern
* refactor: Improved conditionals for collecting pathspec

Author: Test

src/args.sh

@@ -233,19 +233,26 @@ parse_args() {
         print_debug "Debug: No target specified, defaulting to current working tree."
         GIV_REVISION="--current"
     fi
-    
-    print_debug "Parsing revision"
-    # 3. Collect all non-option args as pattern (until first option or end)
-    while [ $# -gt 0 ] && [ "${1#-}" = "$1" ]; do
-        # If the first argument is a pattern, collect it
-        print_debug "Collecting pattern: $1"
-        if [ -z "${GIV_PATHSPEC}" ]; then
-            GIV_PATHSPEC="$1"
-        else
-            GIV_PATHSPEC="${GIV_PATHSPEC} $1"
-        fi
-        shift
-    done
+
+        print_debug "Parsing revision"
+        print_debug "${@}"
+        # 3. Collect all non-option args as pattern (until first option or end)
+        # Only collect pathspec if there are non-option args left AND they are not files like the script itself
+        while [ $# -gt 0 ] && [ "${1#-}" = "$1" ]; do
+            # Avoid setting pathspec to the script name itself (e.g., install.sh)
+            if [ "$1" = "$(basename "$0")" ]; then
+                print_debug "Skipping script name argument: $1"
+                shift
+                continue
+            fi
+            print_debug "Collecting pattern: $1"
+            if [ -z "${GIV_PATHSPEC}" ]; then
+                GIV_PATHSPEC="$1"
+            else
+                GIV_PATHSPEC="${GIV_PATHSPEC} $1"
+            fi
+            shift
+        done
 
     print_debug "Target and pattern parsed: ${GIV_REVISION}, ${GIV_PATHSPEC}"
 

tests/test_commands.bats

@@ -69,11 +69,12 @@ setup() {
   generate_response() {
     print_debug "Mock generate_response called with args: $*"
     echo "RESP"
+    cat "$1" || true
   }
   export -f generate_response
 
   # make helper stubs
-  build_history() { printf "HIST:%s\n" "$2" >"$1"; }
+  #build_history() { printf "HIST:%s\n" "$2" >"$1"; }
   # generate_response() { echo "RESP"; }
   portable_mktemp() { mktemp; }
   get_project_version() { echo "1.2.3"; }
@@ -102,6 +103,7 @@ teardown() {
 #----------------------------------------
 @test "cmd_message with no id errors" {
   echo "some working changes" >"$REPO/file.txt"
+  export GIV_DEBUG="true"
   run cmd_message ""
   assert_success
   assert_output --partial "RESP"
@@ -110,6 +112,8 @@ teardown() {
   echo "change" >"$REPO/file.txt"
   run cmd_message "--current"
   assert_success
+  assert_output --partial "file.txt"
+  assert_output --partial "+change"
 }
 @test "cmd_message single-commit prints message" {
   run git -C "$REPO" rev-parse HEAD~1 # ensure HEAD~1 exists