feat: improve body parsing logic

Current body parsing logic with trim() + split("###") is too fragile and
pose problems with some body that contains case with ### in the middle
of the line or case with codeblock ``` section.

These case will cause the script to parse these as separate section and
produce wrong outputs and in some case even prints error assuming things
are checkbox and errors out on the concat function.

To make the parsing logic more solid, implement a dedicated function and
parse with this logic:
- We split the body for "\n"
- We ignore codeblock ``` section
- We check "###" only at the start of the line
- We check for "### " (with the space included) as that is the correct
  section Github issue template expects.

With the following change case like:
```
root@OpenWrt:~# cat /boot/config.txt
```

Are correctly parsed as all part of a single section instead of being
wrongly treated as different empty sections.

Signed-off-by: Christian Marangi <[email protected]>

Author: Ansuel

index.js

@@ -111,9 +111,35 @@ async function run(env, body, fs, core) {
     return result
   }
 
-  result = body
-    .trim()
-    .split("###")
+  function parseBody(body) {
+    let result = [];
+    let ignore = false;
+
+    body.split("\n").reduce((str, line, idx, arr) => {
+      // ``` Section must be ignored and not parsed as new section
+      if (line == "```")
+        ignore = !ignore
+
+      // Parse new setion only if they start with ### SPACE
+      if (!ignore && line.startsWith("### ")) {
+        result.push(str.trim())
+
+        return line.replace(/^### /, "")+"\n";
+      }
+
+      str += line + "\n"
+
+      // Push the last string if we are at the end of lines
+      if (arr.length - 1 == idx)
+        result.push(str.trim())
+
+      return str;
+    }, "")
+
+    return result;
+  }
+
+  result = parseBody(body)
     .filter(Boolean)
     .map((line) => {
       return line