From 315342e16d8ac8afe67222176e417ea02e415407 Mon Sep 17 00:00:00 2001
From: Maksym Vlasov <MaxymVlasov@users.noreply.github.com>
Date: Tue, 10 Sep 2024 20:53:39 +0300
Subject: [PATCH] fix(`terraform_docs`): Restore
 `--hook-config=--add-to-existing-file` default behavior. Regression from
 1.94.0. (#716)

---------

Co-authored-by: George L. Yermulnik <yz@yz.kiev.ua>
---
 hooks/terraform_docs.sh | 44 ++++++++++++++++++++++++++++-------------
 1 file changed, 30 insertions(+), 14 deletions(-)

diff --git a/hooks/terraform_docs.sh b/hooks/terraform_docs.sh
index 4c402549e..435190a32 100755
--- a/hooks/terraform_docs.sh
+++ b/hooks/terraform_docs.sh
@@ -260,24 +260,40 @@ function terraform_docs {
 
     replace_old_markers "$output_file"
 
-    #
-    # If `--add-to-existing-file=true` set, check is in file exist "hook markers",
-    # and if not - append "hook markers" to the end of file.
-    #
-    if $add_to_existing; then
-      HAVE_MARKER=$(grep -o "$insertion_marker_begin" "$output_file" || exit 0)
-
-      if [ ! "$HAVE_MARKER" ]; then
-        # Use of insertion markers, where addToExisting=true, with no markers in the existing file
-        echo "$insertion_marker_begin" >> "$output_file"
-        echo "$insertion_marker_end" >> "$output_file"
-      fi
-    fi
-
     if [[ "$terraform_docs_awk_file" == "0" ]]; then
+      #? TF 0.12+ and terraform-docs 0.12.0+
+
+      #
+      # If `--add-to-existing-file=false` (default behavior), check if "hook markers" exist in file,
+      # and, if not, skip execution to avoid addition of terraform-docs section, as
+      # terraform-docs in 'inject' mode adds markers by default if they are not present
+      #
+      if [[ $add_to_existing == false ]]; then
+        have_marker=$(grep -o "$insertion_marker_begin" "$output_file") || unset have_marker
+        [[ ! $have_marker ]] && continue
+      fi
       # shellcheck disable=SC2086
       terraform-docs --output-mode="$output_mode" --output-file="$output_file" $tf_docs_formatter $args ./ > /dev/null
+
     else
+      #? TF 0.12+ and terraform-docs < 0.8
+      #? Yes, we don't cover case of TF 0.12+ and terraform-docs 0.8-0.11
+      #? but I probably just drop this section in next release of the hook,
+      #? as there's no sense to support hacks for tool versions which were released more than 3 years ago
+
+      #
+      # If `--add-to-existing-file=true` set, check if "hook markers" exist in file,
+      # and, if not, append "hook markers" to the end of the file.
+      #
+      if [[ $add_to_existing == true ]]; then
+        have_marker=$(grep -o "$insertion_marker_begin" "$output_file") || unset have_marker
+
+        if [[ ! $have_marker ]]; then
+          # Use of insertion markers, when "add_to_existing=true" with no markers in the existing file
+          echo "$insertion_marker_begin" >> "$output_file"
+          echo "$insertion_marker_end" >> "$output_file"
+        fi
+      fi
       # Can't append extension for mktemp, so renaming instead
       local tmp_file_docs
       tmp_file_docs=$(mktemp "${TMPDIR:-/tmp}/terraform-docs-XXXXXXXXXX")