Enhance Project Metadata Handling

Add metadata replacement functionality
feat: Added replace_metadata function to process GIV_METADATA_ environment variables
fix: Corrected token replacement pipeline in build_prompt function
docs: Updated shellcheck directives for project provider scripts

Author: Test

src/llm.sh

@@ -279,6 +279,29 @@ replace_tokens() {
     ' "$@"
 }
 
+# replace_metadata: Replaces placeholders in stdin with values from GIV_METADATA_ environment variables.
+#
+# Usage:
+#   replace_metadata < summary_file > output_file
+replace_metadata() {
+  awk '
+    BEGIN {
+      ORS = ""
+    }
+    {
+      line = $0
+      for (v in ENVIRON) {
+        if (v ~ /^GIV_METADATA_/) {
+          name = substr(v, 14)
+          gsub("\\[" name "\\]", ENVIRON[v], line)
+        }
+      }
+      print line "\n"
+    }
+  '
+}
+
+
 # build_prompt: fill a prompt template with tokens and diff content
 #
 # Usage:
@@ -378,12 +401,12 @@ build_prompt() {
     export GIV_TOKEN_EXAMPLE="${example:-${GIV_TOKEN_EXAMPLE}}"
     export GIV_TOKEN_RULES="${rules:-${GIV_TOKEN_RULES}}"
 
-    # combine template + instructions, then replace tokens using diff as summary
+    # combine template + instructions, then replace metadata and tokens using diff as summary
     {
         cat "${template_file}"
         printf '\nOutput just the final content—no extra commentary or code fencing. '
         printf 'Use only information contained in this prompt and the summaries provided above.\n'
-    } | replace_tokens "${summary_file}"
+    }  | replace_tokens "${summary_file}"| replace_metadata "${summary_file}"
 
     return
 }

src/project/metadata.sh

@@ -16,9 +16,11 @@ metadata_init() {
     # -------------------------
     print_debug "Determining project provider for type: ${GIV_PROJECT_TYPE:-auto}"
     if [ "${GIV_PROJECT_TYPE}" = "custom" ]; then
+        # shellcheck disable=SC1091
         [ -f "${GIV_HOME}/project_provider.sh" ] && . "${GIV_HOME}/project_provider.sh"
     elif [ "${GIV_PROJECT_TYPE}" = "auto" ]; then
         for f in "${GIV_LIB_DIR}/project/providers"/*.sh; do
+            # shellcheck disable=SC1090
             [ -f "$f" ] && . "$f"
         done
         for fn in $(set | awk -F'=' '/^provider_.*_detect=/ { sub("()","",$1); print $1 }'); do
@@ -28,6 +30,7 @@ metadata_init() {
             fi
         done
     else
+        # shellcheck disable=SC1090
         . "${GIV_LIB_DIR}/project/providers/provider_${GIV_PROJECT_TYPE}.sh" || return 1
         DETECTED_PROVIDER="provider_${GIV_PROJECT_TYPE}_detect"
     fi
@@ -93,6 +96,7 @@ metadata_init() {
     # Export for current shell
     # -------------------------
     set -a
+    # shellcheck disable=SC1090
     . "$ENV_FILE"
     set +a
 }

templates/release_notes_prompt.md

@@ -2,6 +2,13 @@
 
 [SUMMARY]
 
+## Project Overview
+
+[TITLE]
+[AUTHOR]
+[URL]
+[DESCRIPTION]
+
 ## Instructions
 
 Write one comprehensive release notes document [PROJECT_TITLE] [VERSION] from the provided change summaries, following these guidelines:

tests/replace_tokens.bats

@@ -16,6 +16,7 @@ export GIV_HOME="$BATS_TEST_DIRNAME/.giv"
 export GIV_TMP_DIR="$BATS_TEST_DIRNAME/.giv/.tmp"
 
 setup() {
+    mkdir -p "$GIV_TMP_DIR"
     TMPDIR_REPO="$(mktemp -d -p "$GIV_TMP_DIR")"
     cd "$TMPDIR_REPO"
     rm -f input.md
@@ -116,20 +117,20 @@ EOF
     assert_output ""
 }
 
-@test "fails if template missing" {
+@test "build_prompt fails if template missing" {
     run build_prompt --template missing.md --summary diff.txt
     [ "$status" -ne 0 ]
     assert_output "template file not found: missing.md"
 }
 
-@test "fails if diff missing" {
+@test "build_prompt fails if diff missing" {
     write_file template.md "Hello"
     run build_prompt --template template.md --summary missing.txt
     [ "$status" -ne 0 ]
     assert_output "diff file not found: missing.txt"
 }
 
-@test "injects summary into prompt" {
+@test "build_prompt injects summary into prompt" {
     write_file template.md \
         "Summary:" \
         "[SUMMARY]" \
@@ -151,7 +152,7 @@ Output just the final content—no extra commentary or code fencing. Use only in
 EOF
 }
 
-@test "replaces all optional tokens" {
+@test "build_prompt replaces all optional tokens" {
     write_file template.md \
         "Project: [PROJECT_TITLE]" \
         "Version: [VERSION]" \

tests/test_build_prompt.bats

@@ -0,0 +1,29 @@
+#!/usr/bin/env bats
+
+load 'test_helper/bats-support/load'
+load 'test_helper/bats-assert/load'
+load "$BATS_TEST_DIRNAME/../src/config.sh"
+load "$BATS_TEST_DIRNAME/../src/system.sh"
+load "$BATS_TEST_DIRNAME/../src/project.sh"
+load "$BATS_TEST_DIRNAME/../src/llm.sh"
+
+setup() {
+  export GIV_METADATA_TITLE="Test Title"
+  export GIV_METADATA_DESCRIPTION="Test Description"
+  export GIV_METADATA_VERSION="1.0.0"
+  echo "[TITLE] - [DESCRIPTION] - [VERSION]" > template.txt
+  echo "Summary content here." > summary.txt
+}
+
+teardown() {
+  unset GIV_METADATA_TITLE
+  unset GIV_METADATA_DESCRIPTION
+  unset GIV_METADATA_VERSION
+  rm -f template.txt summary.txt
+}
+
+@test "build_prompt calls replace_metadata and replace_tokens" {
+  run build_prompt --template template.txt --summary summary.txt
+  assert_success
+  assert_output "Test Title - Test Description - 1.0.0\nOutput just the final content—no extra commentary or code fencing. Use only information contained in this prompt and the summaries provided above."
+}

tests/test_replace_metadata.bats

@@ -0,0 +1,40 @@
+#!/usr/bin/env bats
+
+load 'test_helper/bats-support/load'
+load 'test_helper/bats-assert/load'
+load "$BATS_TEST_DIRNAME/../src/config.sh"
+load "$BATS_TEST_DIRNAME/../src/system.sh"
+load "$BATS_TEST_DIRNAME/../src/llm.sh"
+export GIV_TMP_DIR="$BATS_TEST_DIRNAME/.giv/.tmp"
+
+setup() {
+  export GIV_METADATA_TITLE="Test Title"
+  export GIV_METADATA_DESCRIPTION="Test Description"
+  export GIV_METADATA_VERSION="1.0.0"
+  export GIV_DEBUG="true"
+  export -p GIV_METADATA_TITLE
+  export -p GIV_METADATA_DESCRIPTION
+  export -p GIV_METADATA_VERSION
+  cd "$GIV_TMP_DIR"
+}
+
+teardown() {
+  unset GIV_METADATA_TITLE
+  unset GIV_METADATA_DESCRIPTION
+  unset GIV_METADATA_VERSION
+}
+
+@test "replace_metadata replaces placeholders with GIV_METADATA_ variables" {
+  echo "[TITLE] - [DESCRIPTION] - [VERSION]" > input.txt
+  run env GIV_METADATA_TITLE="Test Title" GIV_METADATA_DESCRIPTION="Test Description" GIV_METADATA_VERSION="1.0.0" bash -c "cat input.txt | source $BATS_TEST_DIRNAME/../src/llm.sh; replace_metadata"
+  assert_success
+  assert_output "Test Title - Test Description - 1.0.0"
+}
+
+@test "replace_metadata handles missing variables gracefully" {
+  echo "[TITLE] - [MISSING] - [VERSION]" > input.txt
+  run env GIV_METADATA_TITLE="Test Title" GIV_METADATA_DESCRIPTION="Test Description" GIV_METADATA_VERSION="1.0.0" bash -c "cat input.txt | source $BATS_TEST_DIRNAME/../src/llm.sh; replace_metadata"
+  assert_success
+  assert_output "Test Title - [MISSING] - 1.0.0"
+}
+