Don't add document divider on empty snippet file

AndreasArvidsson · Jun 16, 2024 · 5dea94a · 5dea94a
1 parent af6bbdb
commit 5dea94a
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 9 deletions.
diff --git a/package.json b/package.json
@@ -2,7 +2,7 @@
     "name": "andreas-talon",
     "displayName": "Andreas Talon",
     "description": "VSCode extension used by Talon Voice",
-    "version": "3.49.2",
+    "version": "3.50.0",
     "publisher": "AndreasArvidsson",
     "license": "MIT",
     "main": "./out/extension.js",

diff --git a/src/language/SnippetFormatter.ts b/src/language/SnippetFormatter.ts
@@ -16,13 +16,12 @@ class SnippetFormatter {
 
     getText(text: string): string {
         const eol = this.eol;
-        return (
-            parseSnippetFile(text)
-                .map((s) => this.getDocumentText(s))
-                // Remove empty documents
-                .filter(Boolean)
-                .join(`${eol}---${eol}${eol}`) + `${eol}---${eol}`
-        );
+        const result = parseSnippetFile(text)
+            .map((s) => this.getDocumentText(s))
+            // Remove empty documents
+            .filter(Boolean)
+            .join(`${eol}---${eol}${eol}`);
+        return result ? result + `${eol}---${eol}` : "";
     }
 
     private getDocumentText(document: SnippetDocument): string {

diff --git a/src/test/snippetFormatter.test.ts b/src/test/snippetFormatter.test.ts
@@ -55,7 +55,7 @@ baz
 `
     },
     {
-        title: "Empty document",
+        title: "Empty snippet document",
         pre: `\
 name: test
 ---
@@ -75,6 +75,11 @@ test
 ---
 `
     },
+    {
+        title: "Empty file",
+        pre: "",
+        post: ""
+    },
     {
         title: "Large file",
         pre: `\