added functions to make the string composition a bit nicier. still ugly imo

NodeApp/src/helpers/AutoCompletionHelper.ts

@@ -121,26 +121,21 @@ function flatten(tree: CmdNode): Array<CmdNode> {
     }
 }
 
-function addWords(cmd: CmdNode, prefix: string): string {
-    return `${prefix}                words="${cmd.children.map(c => c.name).join(" ")} --help -h"\n`
-}
-
 function printWordsGrandParents(node: CmdNode): string {
     let parentWords = ""
-    let prefix = ""
+    let ident = 3
     if (node.parent !== undefined) {
-        parentWords =
-            `            case "\${grand_parent}" in
-                ${node.parent})\n`
-        prefix = "    "
-    }
-    parentWords += addWords(node, prefix)
+        parentWords += openCase(ident, 'grand_parent')
+        ident += 1
+        parentWords += addLine(ident, `${node.parent})`)
+        ident += 1
+    }
+    parentWords += addLine(ident, `words="${node.children.map(c => c.name).join(" ")} --help -h"`)
+    ident -= 1
+    parentWords += addLine(ident, ';;')
     if (node.parent !== undefined) {
-        parentWords +=
-            `                ;;
-                *)
-                ;;
-            esac\n`
+        ident -= 1
+        parentWords += `${closeCase(ident)}`
     }
     return parentWords
 }
@@ -164,37 +159,46 @@ function buildCmdNode(root: Command): CmdNode {
     return createCmdNode(root.name(), root.commands.map(subCmd => getSubCmdNode(subCmd, root)).concat(getOptions(root)))
 }
 
+function openCase(identLevel: number, pattern: string): string {
+    return `${'    '.repeat(identLevel)}case "\${${pattern}}" in\n`
+}
+
+function closeCase(identLevel: number): string {
+    return `${'    '.repeat(identLevel + 1)}*)\n`
+        + `${'    '.repeat(identLevel + 1)};;\n`
+        + `${'    '.repeat(identLevel)}esac\n`
+}
+
+function addLine(identLevel: number, pattern: string): string {
+    return `${'    '.repeat(identLevel)}${pattern}\n`
+}
+
 export function getBashCompletion(root: Command, filename: string) {
     const tree = buildCmdNode(root)
 
     let data =
-        `#/usr/bin/env bash\nfunction _dojo_completions()
-{
-    latest="\${COMP_WORDS[$COMP_CWORD]}"
-    parent="\${COMP_WORDS[$COMP_CWORD - 1]}"
-    grand_parent="\${COMP_WORDS[$COMP_CWORD - 2]}"
-    words=""
-    case "\${parent}" in\n`
+        `${addLine(0, '#/usr/bin/env bash\nfunction _dojo_completions()')}`
+        + `${addLine(0, '{')}${addLine(1, 'latest="${COMP_WORDS[$COMP_CWORD]}"')}`
+        + `${addLine(1, 'parent="${COMP_WORDS[$COMP_CWORD - 1]}"')}`
+        + `${addLine(1, 'grand_parent="${COMP_WORDS[$COMP_CWORD - 2]}"')}`
+        + `${addLine(1, 'words=""')}${openCase(1, "parent")}`
     const commands = Array.from(new Set(flatten(tree).filter(cmd => !isLeaf(cmd)).map(cmd => cmd.name))).map(name => search(tree, name))
     for (const node of commands) {
         const cmd = node[0]
-        data += `        ${cmd.name})\n`
+        data += addLine(2, `${cmd.name})`)
         if (cmd !== undefined && node.length == 1) {
-            data += addWords(cmd, "")
+            data += addLine(3, `words="${cmd.children.map(c => c.name).join(" ")} --help -h"`)
         } else if (cmd !== undefined && node.length > 1) {
             node.forEach(n => data += printWordsGrandParents(n))
         }
-        data += "        ;;\n"
+        data += addLine(2, ';;')
     }
     data +=
-        `        *)
-        ;;
-    esac
-    COMPREPLY=($(compgen -W "$words" -- $latest))
-    return 0
-}
-complete -F _dojo_completions dojo
-`
+        `${closeCase(1)}`
+        + `${addLine(1, 'COMPREPLY=($(compgen -W "$words" -- $latest))')}`
+        + `${addLine(1, 'return 0')}`
+        + `${addLine(0, '}')}`
+        + `${addLine(0, 'complete -F _dojo_completions dojo')}`
 
     writeFileSync(filename, data);
 }