increase length of codefence to 8 because it's enough to render code …

…fence that includes code fence in most cases

internal/terraform/plan.go

@@ -1,11 +1,9 @@
 package terraform
 
 import (
-	"bufio"
 	"encoding/json"
 	"fmt"
 	"io"
-	"regexp"
 	"strings"
 	"text/template"
 
@@ -33,7 +31,9 @@ const planTemplateBody = `### {{len .CreatedAddresses}} to add, {{len .UpdatedAd
 {{if .ResourceChanges -}}
 <details><summary>Change details</summary>
 {{ range .ResourceChanges }}
-{{.GetUnifiedDiffString}}
+{{codeFence}}diff
+# {{.ResourceChange.Type}}.{{.ResourceChange.Name}} {{.HeaderSuffix}}
+{{.GetUnifiedDiffString}}{{codeFence}}
 {{end}}
 </details>
 {{end}}`
@@ -69,10 +69,8 @@ func (r ResourceChangeData) GetUnifiedDiffString() (string, error) {
 	if err != nil {
 		return "", fmt.Errorf("failed to create diff: %w", err)
 	}
-	comment := fmt.Sprintf("# %s.%s %s", r.ResourceChange.Type, r.ResourceChange.Name, r.HeaderSuffix())
-	fence := r.codeFence(diffText)
 
-	return fence + "diff\n" + comment + "\n" + diffText + fence, nil
+	return diffText, nil
 }
 
 func (r ResourceChangeData) HeaderSuffix() string {
@@ -89,23 +87,13 @@ func (r ResourceChangeData) HeaderSuffix() string {
 	return ""
 }
 
-var codeFenceRegexp = regexp.MustCompile(`\s*(` + "`" + `{3,})`)
-
-func (r ResourceChangeData) codeFence(diffText string) string {
-	count := 3
-
-	scanner := bufio.NewScanner(strings.NewReader(diffText))
-	for scanner.Scan() {
-		values := codeFenceRegexp.FindStringSubmatch(scanner.Text())
-		if values != nil && len(values[1]) >= 3 {
-			count = len(values[1]) + 1
-		}
-	}
-	return strings.Repeat("`", count)
-}
-
 func (plan *PlanData) Render(w io.Writer) error {
-	planTemplate, err := template.New("plan").Parse(planTemplateBody)
+	funcMap := template.FuncMap{
+		"codeFence": func() string {
+			return "````````"
+		},
+	}
+	planTemplate, err := template.New("plan").Funcs(funcMap).Parse(planTemplateBody)
 	if err != nil {
 		return fmt.Errorf("invalid template text: %w", err)
 	}

test/testdata/all_types_mixed/expected.md

@@ -9,7 +9,7 @@
     - random_id.test4
 <details><summary>Change details</summary>
 
-```diff
+````````diff
 # env_variable.test2 will be updated in-place
 @@ -1,5 +1,5 @@
  {
@@ -18,9 +18,9 @@
 +  "name": "test2_changed",
    "value": ""
  }
-```
+````````
 
-```diff
+````````diff
 # env_variable.test3 will be destroyed
 @@ -1,5 +1 @@
 -{
@@ -29,18 +29,18 @@
 -  "value": ""
 -}
 +null
-```
+````````
 
-```diff
+````````diff
 # env_variable.test5 will be created
 @@ -1 +1,3 @@
 -null
 +{
 +  "name": "test5"
 +}
-```
+````````
 
-```diff
+````````diff
 # random_id.test4 will be replaced
 @@ -1,10 +1,5 @@
  {
@@ -54,6 +54,6 @@
    "keepers": null,
    "prefix": null
  }
-```
+````````
 
 </details>

test/testdata/aws_sample/expected.md

@@ -10,7 +10,7 @@
     - aws_security_group.admin
 <details><summary>Change details</summary>
 
-```diff
+````````diff
 # aws_instance.test will be destroyed
 @@ -1,92 +1 @@
 -{
@@ -106,9 +106,9 @@
 -  ]
 -}
 +null
-```
+````````
 
-```diff
+````````diff
 # aws_route_table.public-route will be created
 @@ -1 +1,22 @@
 -null
@@ -134,19 +134,19 @@
 +  "timeouts": null,
 +  "vpc_id": "vpc-0c08ee65bf93a360f"
 +}
-```
+````````
 
-```diff
+````````diff
 # aws_route_table_association.puclic-a will be created
 @@ -1 +1,4 @@
 -null
 +{
 +  "gateway_id": null,
 +  "subnet_id": "subnet-0342dca4d2a611266"
 +}
-```
+````````
 
-```diff
+````````diff
 # aws_security_group.admin will be replaced
 @@ -1,6 +1,5 @@
  {
@@ -177,9 +177,9 @@
    "timeouts": null,
    "vpc_id": "vpc-0c08ee65bf93a360f"
  }
-```
+````````
 
-```diff
+````````diff
 # aws_subnet.public-a will be updated in-place
 @@ -18,10 +18,10 @@
    "owner_id": "999999999999",
@@ -194,6 +194,6 @@
    },
    "timeouts": null,
    "vpc_id": "vpc-0c08ee65bf93a360f"
-```
+````````
 
 </details>

test/testdata/iam_policy/expected.md

@@ -3,7 +3,7 @@
     - aws_iam_policy.test_policy
 <details><summary>Change details</summary>
 
-```diff
+````````diff
 # aws_iam_policy.test_policy will be updated in-place
 @@ -12,7 +12,8 @@
        "Action": [
@@ -15,6 +15,6 @@
        ],
        "Resource": "*"
      }
-```
+````````
 
 </details>

test/testdata/include_code_fence/expected.md

@@ -3,7 +3,7 @@
     - env_variable.test1
 <details><summary>Change details</summary>
 
-````diff
+````````diff
 # env_variable.test1 will be updated in-place
 @@ -7,7 +7,7 @@
    ",
@@ -14,6 +14,6 @@
    ```
 
    ",
-````
+````````
 
 </details>

test/testdata/single_add/expected.md

@@ -3,13 +3,13 @@
     - null_resource.foo
 <details><summary>Change details</summary>
 
-```diff
+````````diff
 # null_resource.foo will be created
 @@ -1 +1,3 @@
 -null
 +{
 +  "triggers": null
 +}
-```
+````````
 
 </details>

test/testdata/single_change/expected.md

@@ -3,7 +3,7 @@
     - env_variable.test1
 <details><summary>Change details</summary>
 
-```diff
+````````diff
 # env_variable.test1 will be updated in-place
 @@ -1,5 +1,5 @@
  {
@@ -12,6 +12,6 @@
 +  "name": "test1_changed",
    "value": ""
  }
-```
+````````
 
 </details>

test/testdata/single_destroy/expected.md

@@ -3,14 +3,14 @@
     - null_resource.foo
 <details><summary>Change details</summary>
 
-```diff
+````````diff
 # null_resource.foo will be destroyed
 @@ -1,4 +1 @@
 -{
 -  "id": "7047514762471223910",
 -  "triggers": null
 -}
 +null
-```
+````````
 
 </details>

test/testdata/single_replace/expected.md

@@ -3,7 +3,7 @@
     - random_id.test
 <details><summary>Change details</summary>
 
-```diff
+````````diff
 # random_id.test will be replaced
 @@ -1,10 +1,5 @@
  {
@@ -17,6 +17,6 @@
    "keepers": null,
    "prefix": null
  }
-```
+````````
 
 </details>