- Removed loading of deprecated project.sh in summarize_commit and test_version_extraction bats files to improve maintenance.

- Updated test scripts structure to load necessary helper modules.
- Enhanced existing project metadata extraction capabilities.

Author: Test

src/giv.sh

@@ -8,7 +8,7 @@ set -eu
 
 IFS='
 '
-
+GIV_DEBUG="${GIV_DEBUG:-}"
 # -------------------------------------------------------------------
 # Path detection for libraries, templates, and docs (POSIX compatible)
 # -------------------------------------------------------------------

src/history.sh

@@ -154,13 +154,14 @@ build_history() {
     printf '### Commit ID %s\n' "$commit" >>"$hist"
     printf '**Date:** %s\n' "$(get_commit_date "$commit")" >>"$hist"
 
-
+    print_debug "Getting version for commit $commit"
     ver=$(get_project_version "$commit")
     if [ -n "$ver" ]; then
+        print_debug "Version found: $ver"
         printf '**Version:** %s\n' "$ver" >>"$hist"
     fi
 
-
+    print_debug "Getting message header for commit $commit"
     msg=$(get_message_header "$commit")
     print_debug "Message header: $msg"
     printf '**Message:** %s\n' "$msg" >>"$hist"
@@ -266,7 +267,7 @@ summarize_commit() {
   print_debug "Temporary files created: hist=$hist, prompt=$pr, res_file=$res_file"
 
   generate_commit_history "$hist" "$commit" "$pathspec"
-  sc_version=$(get_commit_version "$commit")
+  sc_version=$(get_project_version "$commit")
   print_debug "Commit version: $sc_version"
 
   summary_template=$(build_commit_summary_prompt "$sc_version" "$hist")
@@ -358,21 +359,6 @@ save_commit_metadata() {
     printf 'Message: %s\n' "$(get_message_header "$commit")" >> "$metadata_file"
 }
 
-# Function to get the version information for a given commit
-get_commit_version() {
-    commit="$1"
-    version_file="$(find_version_file)"
-
-    if [ -z "$version_file" ]; then
-        print_debug "No version file found for commit: $commit"
-        echo ""
-        return
-    fi
-
-    print_debug "Getting version info for commit $commit from $version_file"
-    git show "$commit:$version_file" 2>/dev/null | grep -Eo 'version[[:space:]]*[:=][[:space:]]*"[^"]+"' | head -n 1 | sed -E 's/.*[:=][[:space:]]*"([^"]+)"/\1/'
-}
-
 summarize_target() {
     target="$1"
     summaries_file="$2"

src/project/metadata.sh

@@ -154,11 +154,19 @@ get_project_version() {
     fi
 
     #print_debug "Using version function: $fn for project type: $project_type"
-
     if command -v "$fn" >/dev/null 2>&1; then
-        "$fn" "$commit"
+        ver=""
+        if ver_out=$("$fn" "$commit" 2>/dev/null); then
+            ver="$ver_out"
+        else
+            #print_debug "Error: $fn failed to execute for commit $commit"
+            ver=""
+        fi
+        #print_debug "Version extracted: $ver"
+        printf '%s' "$ver"
+        return 0
     else
-        print_debug "Error: version function $fn not implemented for provider $project_type"
+        #print_debug "Error: version function $fn not implemented for provider $project_type"
         printf ""
         return 0
     fi
@@ -169,6 +177,15 @@ get_project_version() {
     fi
 }
 
+get_project_title() {
+    # Returns the project title from metadata or defaults to directory name
+    if [ -n "${GIV_METADATA_TITLE}" ]; then
+        printf '%s' "${GIV_METADATA_TITLE}"
+    else
+        dirname=$(basename "$PWD")
+        printf '%s' "$dirname"
+    fi
+}
 
 load_config_metadata(){
         # -------------------------

src/project/providers/provider_custom.sh

@@ -39,10 +39,11 @@ provider_custom_get_version() {
         return 0
     fi
     # If file is JSON, extract "version" field (case-insensitive)
-    if grep -i -q '"version"' "$GIV_VERSION_FILE" 2>/dev/null; then
-        version=$(grep -i '"version"' "$GIV_VERSION_FILE" | sed -En 's/.*"[vV]ersion"[[:space:]]*:[[:space:]]*"([^\"]+)".*/\1/p' | head -n 1)
+    if grep -i -q 'version' "$GIV_VERSION_FILE" 2>/dev/null; then
+        #print_debug "Extracting version from file: $GIV_VERSION_FILE"
+        version=$(grep -i 'version' "$GIV_VERSION_FILE" | sed -En 's/.*[vV]ersion"[[:space:]]*:[[:space:]]*([^\"]+)".*/\1/p' | head -n 1)
         # If version is empty or does not match a version pattern, fallback
-        if [ -z "$version" ] || ! printf '%s' "$version" | grep -Eq '^[vV]?[0-9]+\.[0-9]+\.[0-9]+$'; then
+        if [ -z "$version" ]; then
             parse_version "$(cat "$GIV_VERSION_FILE")"
         else
             printf '%s' "$version"
@@ -62,7 +63,7 @@ provider_custom_get_version_at_commit() {
         return 0
     fi
     # If file is JSON, extract "version" field (case-insensitive)
-    if printf '%s' "$file_content" | grep -i -q '"version"'; then
+    if printf '%s' "$file_content" | grep -i -q 'version'; then
         version=$(printf '%s' "$file_content" | grep -i '"version"' | sed -En 's/.*"[vV]ersion"[[:space:]]*:[[:space:]]*"([^\"]+)".*/\1/p' | head -n 1)
         # If version is empty or does not match a version pattern, fallback
         if [ -z "$version" ] || ! printf '%s' "$version" | grep -Eq '^[vV]?[0-9]+\.[0-9]+\.[0-9]+$'; then
@@ -89,7 +90,7 @@ provider_custom_get_version_at_commit() {
 #   A string representing the extracted version number, or an empty string if no valid version
 #   is found in the input.
 parse_version() {
-    #printf 'Parsing version from: %s\n' "$1" >&2
+    #print_debug "Parsing version from: $1"
     # Accepts a string, returns version like v1.2.3 or 1.2.3
     echo "$1" | sed -n -E "s/.*['\"]?([vV][0-9]+\.[0-9]+\.[0-9]+)['\"]?.*/\1/p;s/.*['\"]?([0-9]+\.[0-9]+\.[0-9]+)['\"]?.*/\1/p" | head -n 1
 }