@gmb/bitmark-parser-generator 3.11.0 → 3.13.0

package/dist/esm/generator/bitmark/BitmarkGenerator.js

@@ -29,6 +29,9 @@ const DEFAULT_OPTIONS = {
 };
 /**
  * Generate bitmark markup from a bitmark AST
+ *
+ * NOTE: Newlines - a newline is written BEFORE each content that requries a newline
+ *
  */
 class BitmarkGenerator extends AstWalkerGenerator {
     /**
@@ -51,10 +54,9 @@ class BitmarkGenerator extends AstWalkerGenerator {
         super();
         this.ast = new Ast();
         // State
+        this.firstBit = true;
         this.hasCardSet = false;
         this.hasFooter = false;
-        this.skipNLBetweenBitsValue = false;
-        this.wroteSomething = false;
         this.inTag = true;
         // Keep TS happy
         this.inTag;
@@ -127,10 +129,9 @@ class BitmarkGenerator extends AstWalkerGenerator {
         this.writer.closeSync();
     }
     resetState() {
+        this.firstBit = true;
         this.hasCardSet = false;
         this.hasFooter = false;
-        this.skipNLBetweenBitsValue = false;
-        this.wroteSomething = false;
         this.inTag = true;
         this.printed = false;
     }
@@ -147,12 +148,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
     // Non-Terminal nodes (branches)
     //
     // bitmark
-    // bitmarkAst -> bits
-    between_bits(_node, _left, _right, _route) {
-        this.writeNL();
-        this.writeNL();
-        this.writeNL();
-    }
     // bitmarkAst -> bits -> bitsValue
     enter_bitsValue(node, _route) {
         const bit = node.value;
@@ -160,6 +155,12 @@ class BitmarkGenerator extends AstWalkerGenerator {
         const bitResourcesConfig = Config.getBitResourcesConfig(bit.bitType, bit.resourceType);
         this.hasCardSet = this.haveValidCardSet(bit);
         this.hasFooter = this.haveValidFooter(bit);
+        // Separate the bits with 3 newlines
+        if (!this.firstBit) {
+            this.writeNL();
+            this.writeNL();
+            this.writeNL();
+        }
         // Write the bit tag opening
         this.writeOPD(bit.bitLevel);
         if (bit.isCommented)
@@ -195,22 +196,12 @@ class BitmarkGenerator extends AstWalkerGenerator {
             this.writeBreakscapedTagString(resourceType);
         }
         this.writeCL();
-        this.writeNL();
+        // this.writeNL();
         // Continue traversal
         return true;
     }
-    between_bitsValue(node, left, right, route) {
-        // The following keys are combined with other keys so don't need newlines
-        const noNlKeys = [
-            NodeType.bitType,
-            NodeType.textFormat,
-            NodeType.level,
-            NodeType.progress,
-            NodeType.toc,
-            NodeType.referenceEnd,
-            NodeType.labelFalse,
-        ];
-        this.writeNlBetween(node, left, right, route, noNlKeys);
+    exit_bitsValue(_node, _route) {
+        this.firstBit = false;
     }
     // bitmarkAst -> bits -> bitsValue -> internalComment
     enter_internalComment(node, route) {
@@ -221,14 +212,12 @@ class BitmarkGenerator extends AstWalkerGenerator {
             return true;
         for (let i = 0; i < internalComment.length; i++) {
             const comment = internalComment[i];
-            const last = i === internalComment.length - 1;
+            this.writeNL();
             this.writeProperty('internalComment', comment, {
                 format: PropertyFormat.trimmedString,
                 single: false,
                 ignoreEmpty: true,
             });
-            if (!last)
-                this.writeNL();
         }
         // Stop traversal of this branch
         return false;
@@ -242,13 +231,18 @@ class BitmarkGenerator extends AstWalkerGenerator {
             return true;
         const bit = parent === null || parent === void 0 ? void 0 : parent.value;
         if (bit) {
-            if (value != '')
+            const haveTrue = value != '';
+            const haveFalse = bit.labelFalse && bit.labelFalse[0] != '';
+            if (haveTrue || haveFalse) {
+                this.writeNL();
+            }
+            if (haveTrue)
                 this.writeProperty(PropertyTag.labelTrue, value, {
                     format: PropertyFormat.trimmedString,
                     single: true,
                     ignoreEmpty: true,
                 });
-            if (bit.labelFalse && bit.labelFalse[0] != '')
+            if (haveFalse)
                 this.writeProperty(PropertyTag.labelFalse, bit.labelFalse, {
                     format: PropertyFormat.trimmedString,
                     single: true,
@@ -258,6 +252,10 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Stop traversal of this branch
         return false;
     }
+    enter_labelFalse(_node, _route) {
+        // Handled above in enter_labelTrue(), but this function needed to block automatic property handling
+        return true;
+    }
     // bitmarkAst -> bits -> bitsValue -> imageSource
     enter_imageSource(node, route) {
         const imageSource = node.value;
@@ -266,6 +264,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.bitsValue)
             return true;
         const { url, mockupId, size, format, trim } = imageSource;
+        this.writeNL();
         this.writeProperty('imageSource', url, {
             format: PropertyFormat.trimmedString,
             single: true,
@@ -308,6 +307,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.bitsValue)
             return true;
         const { technicalTerm, lang } = nodeValue;
+        this.writeNL();
         this.writeProperty('technicalTerm', technicalTerm, {
             format: PropertyFormat.trimmedString,
             single: true,
@@ -331,6 +331,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.bitsValue)
             return true;
         const { servings, unit, unitAbbr, decimalPlaces, disableCalculation, hint } = nodeValue;
+        this.writeNL();
         this.writeProperty('servings', servings, {
             format: PropertyFormat.trimmedString,
             single: true,
@@ -384,6 +385,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.bitsValue)
             return true;
         const { name, title, avatarImage } = person;
+        this.writeNL();
         this.writeProperty('person', name, {
             format: PropertyFormat.trimmedString,
             single: true,
@@ -402,6 +404,89 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Stop traversal of this branch
         return false;
     }
+    // bitmarkAst -> bits -> bitsValue -> cardNode -> definitions -> definitionsValue -> term -> text
+    // bitmarkAst -> bits -> bitsValue -> cardNode -> definitions -> definitionsValue -> definition -> text
+    // bitmarkAst -> bits -> bitsValue -> cardNode -> definitions -> definitionsValue -> alternativeDefinitions
+    // -> alternativeDefinitionsValue -> text
+    // bitmarkAst -> bits -> bitsValue -> cardNode -> flashcards -> flashcardsValue -> question -> text
+    // bitmarkAst -> bits -> bitsValue -> cardNode -> flashcards -> flashcardsValue -> answer -> text
+    // bitmarkAst -> bits -> bitsValue -> cardNode -> flashcards -> flashcardsValue -> alternativeAnswers ->
+    // -> alternativeAnswersValue -> text
+    enter_text(node, route) {
+        const parent = this.getParentNode(route);
+        if (!parent ||
+            (parent.key !== NodeType.term &&
+                parent.key !== NodeType.definition &&
+                parent.key !== NodeType.alternativeDefinitionsValue &&
+                parent.key !== NodeType.question &&
+                parent.key !== NodeType.answer &&
+                parent.key !== NodeType.alternativeAnswersValue)) {
+            return true;
+        }
+        if (node.value) {
+            this.writeNL();
+            this.textGenerator.generateSync(node.value, TextFormat.bitmarkMinusMinus);
+        }
+        // Stop traversal of this branch
+        return false;
+    }
+    leaf_text(node, route) {
+        const parent = this.getParentNode(route);
+        if (!parent ||
+            (parent.key !== NodeType.term &&
+                parent.key !== NodeType.definition &&
+                parent.key !== NodeType.alternativeDefinitionsValue &&
+                parent.key !== NodeType.question &&
+                parent.key !== NodeType.answer &&
+                parent.key !== NodeType.alternativeAnswersValue &&
+                parent.key !== NodeType.reason)) {
+            return true;
+        }
+        if (StringUtils.isString(node.value)) {
+            const str = node.value;
+            this.writeNL();
+            this.write(Breakscape.breakscape(str, {
+                textFormat: TextFormat.text,
+            }));
+        }
+        // Stop traversal of this branch
+        return false;
+    }
+    // bitmarkAst -> bits -> bitsValue -> * -> term -> icon
+    // bitmarkAst -> bits -> bitsValue -> * -> definition -> icon
+    // bitmarkAst -> bits -> bitsValue -> * -> alternativeDefinitionsValue -> icon
+    enter_icon(node, route) {
+        const resource = node.value;
+        const parent = this.getParentNode(route);
+        if (!parent)
+            return true;
+        if (parent.key !== NodeType.term &&
+            parent.key !== NodeType.definition &&
+            parent.key !== NodeType.alternativeDefinitionsValue &&
+            parent.key !== NodeType.question &&
+            parent.key !== NodeType.answer &&
+            parent.key !== NodeType.alternativeAnswersValue) {
+            // Continue traversal of this branch
+            return true;
+        }
+        // This is a resource, so handle it with the common code
+        this.writeNL();
+        this.writeResource(ResourceTag.icon, resource.src);
+        // this.writePropertyStyleResource(ResourceTag.icon, resource as ResourceJson);
+        // Continue traversal of this branch (for the chained properties)
+        return true;
+    }
+    leaf_iconValue(node, _route) {
+        // Handle as a standard icon property
+        this.writeNL();
+        this.writeProperty('icon', node.value, {
+            format: PropertyFormat.trimmedString,
+            single: true,
+            ignoreEmpty: true,
+        });
+        // Stop traversal of this branch
+        return false;
+    }
     // bitmarkAst -> bits -> bitsValue -> ratingLevelStart
     enter_ratingLevelStart(node, route) {
         this.enterRatingLevelStartEndCommon(node, route);
@@ -423,6 +508,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
             return true;
         const { level, label } = n;
         const levelKey = node.key === NodeType.ratingLevelStart ? PropertyTag.ratingLevelStart : PropertyTag.ratingLevelEnd;
+        this.writeNL();
         this.writeProperty(levelKey, level, {
             format: PropertyFormat.trimmedString,
             single: true,
@@ -452,6 +538,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
             return true;
         const { mark, color, emphasis } = markConfig;
         if (mark) {
+            this.writeNL();
             this.writeProperty('mark', mark, {
                 format: PropertyFormat.trimmedString,
                 single: true,
@@ -471,74 +558,79 @@ class BitmarkGenerator extends AstWalkerGenerator {
                     ignoreEmpty: true,
                 });
             }
-            this.writeNL();
         }
         // Stop traversal of this branch
         return false;
     }
     // bitmarkAst -> bits -> bitsValue -> partialAnswer
     enter_partialAnswer(node, _route) {
-        this.writeProperty('partialAnswer', node.value, {
-            format: PropertyFormat.trimmedString,
-            single: true,
-            ignoreEmpty: true,
-        });
+        if (node.value) {
+            this.writeNL();
+            this.writeProperty('partialAnswer', node.value, {
+                format: PropertyFormat.trimmedString,
+                single: true,
+                ignoreEmpty: true,
+            });
+        }
         // Stop traversal of this branch
         return false;
     }
     // bitmarkAst -> bits -> bitsValue -> questions -> questionsValue -> partialAnswer
     leaf_partialAnswer(node, _route) {
-        this.writeProperty('partialAnswer', node.value, {
-            format: PropertyFormat.trimmedString,
-            single: true,
-            ignoreEmpty: true,
-        });
+        if (node.value) {
+            this.writeNL();
+            this.writeProperty('partialAnswer', node.value, {
+                format: PropertyFormat.trimmedString,
+                single: true,
+                ignoreEmpty: true,
+            });
+        }
         // Stop traversal of this branch
         return false;
     }
     // bitmarkAst -> bits -> bitsValue -> sampleSolution
     // bitmarkAst -> bits -> bitsValue -> questions -> questionsValue -> sampleSolution
     enter_sampleSolution(node, _route) {
-        this.writeProperty('sampleSolution', node.value, {
-            format: PropertyFormat.trimmedString,
-            single: true,
-            ignoreEmpty: true,
-        });
+        if (node.value) {
+            this.writeNL();
+            this.writeProperty('sampleSolution', node.value, {
+                format: PropertyFormat.trimmedString,
+                single: true,
+                ignoreEmpty: true,
+            });
+        }
         // Stop traversal of this branch
         return false;
     }
     leaf_sampleSolution(node, _route) {
-        this.writeProperty('sampleSolution', node.value, {
-            format: PropertyFormat.trimmedString,
-            single: true,
-            ignoreEmpty: true,
-        });
+        if (node.value) {
+            this.writeNL();
+            this.writeProperty('sampleSolution', node.value, {
+                format: PropertyFormat.trimmedString,
+                single: true,
+                ignoreEmpty: true,
+            });
+        }
         // Stop traversal of this branch
         return false;
     }
     // bitmarkAst -> bits -> bitsValue -> reasonableNumOfChars
     // bitmarkAst -> bits -> bitsValue -> questions -> questionsValue -> reasonableNumOfChars
     leaf_reasonableNumOfChars(node, _route) {
+        this.writeNL();
         this.writeProperty('reasonableNumOfChars', node.value, {
             format: PropertyFormat.trimmedString,
             single: true,
             ignoreEmpty: true,
         });
     }
-    // bitmarkAst -> bits -> bitsValue -> questions -> questionsValue -> additionalSolutions
-    between_additionalSolutions(_node, _left, _right, route) {
-        // Ignore values that are not at the bit level as they might be handled elsewhere
-        const parent = this.getParentNode(route);
-        if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.questionsValue)
-            return;
-        this.writeNL();
-    }
     // bitmarkAst -> bits -> bitsValue -> questions -> questionsValue -> additionalSolutions -> additionalSolutionsValue
     leaf_additionalSolutionsValue(node, route) {
         // Ignore values that are not at the bit level as they might be handled elsewhere
         const parent = this.getParentNode(route, 2);
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.questionsValue)
             return;
+        this.writeNL();
         this.writeProperty('additionalSolutions', node.value, {
             format: PropertyFormat.trimmedString,
             single: false,
@@ -557,6 +649,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
             return true; // Will be handled by pageNumber
         if (!this.isEmptyText((_c = parent === null || parent === void 0 ? void 0 : parent.value) === null || _c === void 0 ? void 0 : _c.marginNumber))
             return true; // Will be handled by marginNumber
+        this.writeNL_IfNotChain(route);
         this.writeOPC();
         this.textGenerator.generateSync(item, TextFormat.bitmarkMinusMinus);
         this.writeCL();
@@ -573,6 +666,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
             return true; // Will be handled by pageNumber
         if (!this.isEmptyText((_b = parent === null || parent === void 0 ? void 0 : parent.value) === null || _b === void 0 ? void 0 : _b.marginNumber))
             return true; // Will be handled by marginNumber
+        this.writeNL_IfNotChain(route);
         this.writeOPC();
         this.textGenerator.generateSync((_d = (_c = parent === null || parent === void 0 ? void 0 : parent.value) === null || _c === void 0 ? void 0 : _c.item) !== null && _d !== void 0 ? _d : '', TextFormat.bitmarkMinusMinus);
         this.writeCL();
@@ -590,6 +684,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
             return false; // Ignore empty
         if (!this.isEmptyText((_a = parent === null || parent === void 0 ? void 0 : parent.value) === null || _a === void 0 ? void 0 : _a.marginNumber))
             return true; // Will be handled by marginNumber
+        this.writeNL_IfNotChain(route);
         this.writeOPC();
         this.textGenerator.generateSync((_c = (_b = parent === null || parent === void 0 ? void 0 : parent.value) === null || _b === void 0 ? void 0 : _b.item) !== null && _c !== void 0 ? _c : '', TextFormat.bitmarkMinusMinus);
         this.writeCL();
@@ -608,6 +703,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         const parent = this.getParentNode(route);
         if (this.isEmptyText(marginNumber))
             return false; // Ignore empty
+        this.writeNL_IfNotChain(route);
         this.writeOPC();
         this.textGenerator.generateSync((_b = (_a = parent === null || parent === void 0 ? void 0 : parent.value) === null || _a === void 0 ? void 0 : _a.item) !== null && _b !== void 0 ? _b : '', TextFormat.bitmarkMinusMinus);
         this.writeCL();
@@ -630,7 +726,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.bitsValue && (parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.cardBitsValue)
             return true;
         this.inTag = false;
-        // always write a NL before the body content if there is any?
+        // Always write a NL before the body content if there is any (see Handle body)
         const body = node.value;
         const textFormat = this.getTextFormat(route);
         const isBitmarkText = textFormat === TextFormat.bitmarkPlusPlus || textFormat === TextFormat.bitmarkMinusMinus;
@@ -640,7 +736,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
             if (Array.isArray(json) || ObjectUtils.isObject(json)) {
                 const text = JSON.stringify(json, null, this.prettifySpace);
                 if (text) {
-                    this.writeNL();
                     this.writePlainTextDivider();
                     this.writeNL();
                     this.write(Breakscape.breakscape(text, {
@@ -661,27 +756,15 @@ class BitmarkGenerator extends AstWalkerGenerator {
         }
         else {
             // handle plain text
-            this.writeNL();
             this.writePlainTextDivider();
             this.writeNL();
             const s = (StringUtils.isString(body.body) ? body.body : '');
             this.write(Breakscape.breakscape(`${s}`, {
                 textFormat: TextFormat.text,
             }));
-            this.writeNL();
         }
         // Stop traversal of this branch
         return false;
-        // if ((body.body && body.body.length > 0) || body.bodyJson) {
-        //   this.writeNL();
-        //   // Write the plain text divider if not bitmark++/-- format
-        //   const textFormat = this.getTextFormat(route);
-        //   const isBitmarkText = textFormat === TextFormat.bitmarkPlusPlus || textFormat === TextFormat.bitmarkMinusMinus;
-        //   if (!isBitmarkText) {
-        //     this.writePlainTextDivider();
-        //     this.writeNL();
-        //   }
-        // }
     }
     exit_body(_node, _route) {
         this.inTag = true;
@@ -747,6 +830,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
             const isBitmarkText = textFormat === TextFormat.bitmarkPlusPlus || textFormat === TextFormat.bitmarkMinusMinus;
             if (isBitmarkText) {
                 // handle bitmark text
+                this.writeNL();
                 this.write('==== footer ====');
                 this.writeNL();
                 // The text generator will write to the writer
@@ -841,7 +925,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
         if (!this.isCardAllowed(route))
             return true;
         this.writeCardSetStart();
-        this.writeNL();
         // Continue traversal
         return true;
     }
@@ -849,41 +932,27 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Ignore cards if not allowed
         if (!this.isCardAllowed(route))
             return;
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
     }
     exit_cardNode(_node, route) {
         // Ignore cards if not allowed
         if (!this.isCardAllowed(route))
             return;
-        this.writeNL();
         this.writeCardSetEnd();
         if (this.options.cardSetVersion === CardSetVersion.v1) {
-            this.writeNL();
+            // this.writeNL();
         }
     }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> cardBitsValue
-    between_cardBitsValue(_node, _left, _right, _route) {
-        this.writeNL();
-    }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> elements
     enter_elements(_node, _route) {
         //
     }
     between_elements(_node, _left, _right, _route) {
-        this.writeNL();
         this.writeCardSetVariantDivider();
-        this.writeNL();
-    }
-    exit_elements(_node, _route) {
-        //
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> flashcards
     between_flashcards(_node, _left, _right, _route) {
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> flashcards -> flashcardsValue
     between_flashcardsValue(_node, _left, right, route) {
@@ -892,67 +961,19 @@ class BitmarkGenerator extends AstWalkerGenerator {
         if (!this.isCardAllowed(route))
             return;
         if (right.key === NodeType.answer) {
-            this.writeNL();
             this.writeCardSetSideDivider();
-            this.writeNL();
         }
         else if (right.key === NodeType.alternativeAnswers && ((_a = right.value) === null || _a === void 0 ? void 0 : _a.length) !== 0) {
-            this.writeNL();
             this.writeCardSetVariantDivider();
-            this.writeNL();
         }
     }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> flashcards -> flashcardsValue -> answer
-    enter_answer(node, route) {
-        // Ignore responses that are not at the flashcardsValue level as they are handled elsewhere
-        const parent = this.getParentNode(route);
-        if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.flashcardsValue)
-            return true;
-        if (node.value) {
-            this.textGenerator.generateSync(node.value, TextFormat.bitmarkMinusMinus);
-        }
-        // Stop traversal of this branch
-        return false;
-    }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> flashcards -> flashcardsValue -> alternativeAnswers
     between_alternativeAnswers(_node, _route) {
-        this.writeNL();
         this.writeCardSetVariantDivider();
-        this.writeNL();
-    }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> flashcards -> flashcardsValue -> alternativeAnswers -> alternativeAnswersValue
-    enter_alternativeAnswersValue(node, route) {
-        // Ignore responses that are not at the alternativeAnswers level as they are handled elsewhere
-        const parent = this.getParentNode(route);
-        if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.alternativeAnswers)
-            return true;
-        if (node.value) {
-            // this.writeBreakscapedTagString(node.value);
-            this.textGenerator.generateSync(node.value, TextFormat.bitmarkMinusMinus);
-        }
-        // Stop traversal of this branch
-        return false;
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> definitions -> definitionsValue -> alternativeDefintions
     between_alternativeDefinitions(_node, _route) {
-        this.writeNL();
         this.writeCardSetVariantDivider();
-        this.writeNL();
-    }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> definitions -> definitionsValue -> alternativeDefinitions -> alternativeDefinitionsValue
-    enter_alternativeDefinitionsValue(node, route) {
-        // Ignore responses that are not at the alternativeAnswers level as they are handled elsewhere
-        const parent = this.getParentNode(route);
-        if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.alternativeDefinitions)
-            return true;
-        if (node.value) {
-            // this.writeNL();
-            // this.writeCardSetVariantDivider();
-            // this.writeNL();
-            this.textGenerator.generateSync(node.value, TextFormat.bitmarkMinusMinus);
-        }
-        // Stop traversal of this branch
-        return false;
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> statements
     enter_statements(_node, _route) {
@@ -961,17 +982,13 @@ class BitmarkGenerator extends AstWalkerGenerator {
     between_statements(_node, _left, _right, route) {
         const isTrueFalse1 = this.isOfBitType(route, BitType.trueFalse1);
         if (!isTrueFalse1) {
-            this.writeNL();
             this.writeCardSetCardDivider();
         }
-        this.writeNL();
-    }
-    exit_statements(_node, _route) {
-        //
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> statements -> statementsValue
     enter_statementsValue(node, _route) {
         const statement = node.value;
+        this.writeNL();
         if (statement.isCorrect) {
             this.writeOPP();
         }
@@ -983,19 +1000,13 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Continue traversal
         return true;
     }
-    // bitmarkAst -> bits -> bitsValue -> choices
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> quizzes -> quizzesValue -> choices
-    between_choices(_node, _left, _right, _route) {
-        this.writeNL();
-    }
-    exit_choices(_node, _route) {
-        this.writeNL();
-    }
     // bitmarkAst -> bits -> bitsValue -> choices -> choicesValue
     // bitmarkAst -> bits -> bitsValue -> cardNode -> quizzes -> quizzesValue -> choices -> choicesValue
+    // bitmarkAst -> bits -> bitsValue -> cardNode -> feedbacks -> feedbacksValue -> choices -> choicesValue
     enter_choicesValue(node, _route) {
         const choice = node.value;
-        if (choice.isCorrect) {
+        this.writeNL();
+        if (choice.isCorrect || choice.requireReason) {
             this.writeOPP();
         }
         else {
@@ -1006,18 +1017,11 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Continue traversal
         return true;
     }
-    // bitmarkAst -> bits -> bitsValue -> responses
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> quizzes -> quizzesValue -> responses
-    between_responses(_node, _left, _right, _route) {
-        this.writeNL();
-    }
-    exit_responses(_node, _route) {
-        this.writeNL();
-    }
     // bitmarkAst -> bits -> bitsValue -> responses -> responsesValue
     // bitmarkAst -> bits -> bitsValue -> cardNode -> quizzes -> quizzesValue -> responses -> responsesValue
     enter_responsesValue(node, _route) {
         const response = node.value;
+        this.writeNL();
         if (response.isCorrect) {
             this.writeOPP();
         }
@@ -1029,23 +1033,22 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Continue traversal
         return true;
     }
+    // bitmarkAst -> bits -> bitsValue -> cardNode -> feedbacks
+    between_feedbacks(_node, _left, _right, _route) {
+        this.writeCardSetCardDivider();
+    }
+    // bitmarkAst -> bits -> bitsValue -> cardNode -> feedbacks -> feedbacksValue -> reason
+    enter_reason(_node, _route) {
+        this.writeCardSetSideDivider();
+        // Continue traversal
+        return true;
+    }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> quizzes
     enter_quizzes(_node, _route) {
         //
     }
     between_quizzes(_node, _left, _right, _route) {
-        // this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
-    }
-    exit_quizzes(_node, _route) {
-        //
-    }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> quizzes -> quizzesValue
-    between_quizzesValue(_node, _left, right, _route) {
-        if (right.key === NodeType.choices || right.key === NodeType.responses) {
-            this.writeNL();
-        }
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> heading
     enter_heading(_node, _route) {
@@ -1055,12 +1058,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Ignore cards if not allowed
         if (!this.isCardAllowed(route))
             return;
-        this.writeNL();
         this.writeCardSetSideDivider();
-        this.writeNL();
-    }
-    exit_heading(_node, _route) {
-        //
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> heading -> forValues
     enter_forValues(_node, _route) {
@@ -1070,33 +1068,20 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Ignore cards if not allowed
         if (!this.isCardAllowed(route))
             return;
-        this.writeNL();
         this.writeCardSetSideDivider();
-        this.writeNL();
-    }
-    exit_forValues(_node, _route) {
-        //
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> pairs
     enter_pairs(_node, _route) {
         //
     }
     between_pairs(_node, _left, _right, _route) {
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
-    }
-    exit_pairs(_node, _route) {
-        //
-    }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> pairs -> pairsValue
-    between_pairsValue(_node, _left, _right, _route) {
-        //
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> pairs -> pairsValue -> keyAudio
     enter_keyAudio(node, _route) {
         const resource = node.value;
         // This is a resource, so handle it with the common code
+        this.writeNL();
         this.writeResource(ResourceTag.audio, resource.src);
         // Stop traversal of this branch
         return false;
@@ -1105,6 +1090,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
     enter_keyImage(node, _route) {
         const resource = node.value;
         // This is a resource, so handle it with the common code
+        this.writeNL();
         this.writeResource(ResourceTag.image, resource.src);
         // Stop traversal of this branch
         return false;
@@ -1114,16 +1100,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         //
     }
     between_matrix(_node, _left, _right, _route) {
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
-    }
-    exit_matrix(_node, _route) {
-        //
-    }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> matrix -> matrixValue
-    between_matrixValue(_node, _left, _right, _route) {
-        //
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> pairs -> pairsValue -> values
     // bitmarkAst -> bits -> bitsValue -> cardNode -> matrix -> matrixValue -> cells -> cellsValue -> values
@@ -1131,17 +1108,13 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Ignore cards if not allowed
         if (!this.isCardAllowed(route))
             return;
-        this.writeNL();
         this.writeCardSetSideDivider();
-        this.writeNL();
     }
     between_values(_node, _left, _right, route) {
         // Ignore cards if not allowed
         if (!this.isCardAllowed(route))
             return;
-        this.writeNL();
         this.writeCardSetVariantDivider();
-        this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> pronunciationTable
     between_pronunciationTable(_node, _left, _right, route) {
@@ -1151,9 +1124,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Ignore cards if not allowed
         if (!this.isCardAllowed(route))
             return;
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> table
     between_table(_node, _left, _right, route) {
@@ -1163,9 +1134,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Ignore cards if not allowed
         if (!this.isCardAllowed(route))
             return;
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> table -> data
     // bitmarkAst -> bits -> bitsValue -> cardNode -> pronunciationTable -> data
@@ -1173,9 +1142,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         const parent = this.getParentNode(route);
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.table && (parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.pronunciationTable)
             return;
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> table -> columns
     // bitmarkAst -> bits -> bitsValue -> cardNode -> captionDefinitionList -> columns
@@ -1187,19 +1154,19 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Ignore cards if not allowed
         if (!this.isCardAllowed(route))
             return;
-        this.writeNL();
         this.writeCardSetSideDivider();
-        this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> table -> columns -> columnsValue
     // bitmarkAst -> bits -> bitsValue -> cardNode -> captionDefinitionList -> columns -> columnsValue
     leaf_columnsValue(node, _route) {
+        this.writeNL();
         this.writeOPHASH();
         if (node.value)
             this.writeBreakscapedTagString(node.value);
         this.writeCL();
     }
     enter_columnsValue(node, _route) {
+        this.writeNL();
         this.writeOPHASH();
         if (node.value) {
             this.textGenerator.generateSync(node.value, TextFormat.bitmarkMinusMinus);
@@ -1215,15 +1182,14 @@ class BitmarkGenerator extends AstWalkerGenerator {
         // Ignore cards if not allowed
         if (!this.isCardAllowed(route))
             return;
-        this.writeNL();
         this.writeCardSetSideDivider();
-        this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> table -> data -> dataValue -> dataValueValue
     leaf_dataValueValue(node, route) {
         const parent = this.getParentNode(route, 3);
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.table)
             return;
+        this.writeNL();
         this.write(node.value);
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> table -> data -> dataValue -> dataValueValue
@@ -1239,17 +1205,18 @@ class BitmarkGenerator extends AstWalkerGenerator {
                 // Pronunciation Table
                 const cell = node.value;
                 if (cell.title) {
+                    this.writeNL();
                     this.writeOP();
                     this.writeHash();
                     this.textGenerator.generateSync(cell.title, TextFormat.bitmarkMinusMinus);
                     this.writeCL();
-                    this.writeNL();
                 }
                 if (cell.audio) {
-                    this.writeResource(ResourceTag.audio, cell.audio.src);
                     this.writeNL();
+                    this.writeResource(ResourceTag.audio, cell.audio.src);
                 }
                 if (cell.body) {
+                    this.writeNL();
                     this.textGenerator.generateSync(cell.body, textFormat);
                 }
                 // Stop traversal of this branch
@@ -1257,6 +1224,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
             }
             else {
                 // Table
+                this.writeNL();
                 this.textGenerator.generateSync(node.value, textFormat);
             }
         }
@@ -1267,38 +1235,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         const parent = this.getParentNode(route);
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.cardNode)
             return;
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
-    }
-    // // bitmarkAst -> bits -> bitsValue -> cardNode -> captionDefinitionList -> definitions
-    // protected between_definitions(_node: NodeInfo, _left: NodeInfo, _right: NodeInfo, route: NodeInfo[]): void {
-    //   const parent = this.getParentNode(route);
-    //   if (parent?.key !== NodeType.captionDefinitionList) return;
-    //   this.writeNL();
-    //   this.writeCardSetCardDivider();
-    //   this.writeNL();
-    // }
-    // // bitmarkAst -> bits -> bitsValue -> cardNode -> captionDefinitionList -> definitions -> definitionsValue
-    // protected between_definitionsValue(_node: NodeInfo, _left: NodeInfo, _right: NodeInfo, route: NodeInfo[]): void {
-    //   const parent = this.getParentNode(route);
-    //   if (parent?.key !== NodeType.definitions) return;
-    // // Ignore cards if not allowed
-    // if (!this.isCardAllowed(route)) return;
-    //   this.writeNL();
-    //   this.writeCardSetSideDivider();
-    //   this.writeNL();
-    // }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> definitions -> definitionsValue -> term
-    enter_term(node, route) {
-        const parent = this.getParentNode(route);
-        if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.definitionsValue)
-            return true;
-        if (node.value) {
-            this.textGenerator.generateSync(node.value, TextFormat.bitmarkMinusMinus);
-        }
-        // Stop traversal of this branch
-        return false;
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> captionDefinitionList -> definitions -> definitionsValue -> term
     leaf_term(node, route) {
@@ -1306,28 +1243,19 @@ class BitmarkGenerator extends AstWalkerGenerator {
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.definitionsValue)
             return true;
         if (node.value) {
+            this.writeNL();
             this.write(node.value);
         }
         // Stop traversal of this branch
         return false;
     }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> definitions -> definitionsValue -> definition
-    enter_definition(node, route) {
-        const parent = this.getParentNode(route);
-        if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.definitionsValue)
-            return true;
-        if (node.value) {
-            this.textGenerator.generateSync(node.value, TextFormat.bitmarkMinusMinus);
-        }
-        // Stop traversal of this branch
-        return false;
-    }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> captionDefinitionList -> definitions -> definitionsValue -> definition
     leaf_definition(node, route) {
         const parent = this.getParentNode(route);
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.definitionsValue)
             return true;
         if (node.value) {
+            this.writeNL();
             this.write(node.value);
         }
         // Stop traversal of this branch
@@ -1339,9 +1267,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         const parent = this.getParentNode(route);
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.cardNode && (parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.captionDefinitionList)
             return;
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> definitions -> definitionsValue
     between_definitionsValue(_node, _left, right, route) {
@@ -1353,75 +1279,37 @@ class BitmarkGenerator extends AstWalkerGenerator {
         if (!this.isCardAllowed(route))
             return;
         if (right.key === NodeType.definition) {
-            this.writeNL();
             this.writeCardSetSideDivider();
-            this.writeNL();
         }
         else if (right.key === NodeType.alternativeDefinitions && ((_a = right.value) === null || _a === void 0 ? void 0 : _a.length) > 0) {
-            this.writeNL();
             this.writeCardSetVariantDivider();
-            this.writeNL();
         }
     }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> captionDefinitionList -> definitions -> definitionsValue
-    // protected between_definitionsValue(_node: NodeInfo, _left: NodeInfo, right: NodeInfo, route: NodeInfo[]): void {
-    //   const parent = this.getParentNode(route);
-    //   if (parent?.key !== NodeType.definitions) return;
-    // // Ignore cards if not allowed
-    // if (!this.isCardAllowed(route)) return;
-    //   if (right.key === NodeType.definition) {
-    //     this.writeNL();
-    //     this.writeCardSetSideDivider();
-    //     this.writeNL();
-    //   } else if (right.key === NodeType.alternativeDefinitions && right.value?.length > 0) {
-    //     this.writeNL();
-    //     this.writeCardSetVariantDivider();
-    //     this.writeNL();
-    //   }
-    // }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> questions
     enter_questions(_node, _route) {
         //
     }
     between_questions(_node, _left, _right, _route) {
         this.writeCardSetCardDivider();
-        this.writeNL();
-    }
-    exit_questions(_node, _route) {
-        //
-    }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> questions -> questionsValue
-    between_questionsValue(node, left, right, route) {
-        // The following keys are combined with other keys so don't need newlines
-        const noNlKeys = [
-        //
-        ];
-        this.writeNlBetween(node, left, right, route, noNlKeys);
-    }
-    exit_questionsValue(_node, _route) {
-        this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> ingredients
     enter_ingredients(_node, _route) {
         //
     }
     between_ingredients(_node, _left, _right, _route) {
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
-    }
-    exit_ingredients(_node, _route) {
-        //
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> ingredients -> ingredientsValue
     enter_ingredientsValue(node, _route) {
         const ingredient = node.value;
         if (ingredient.title != null) {
+            this.writeNL();
             this.writeOPHASH();
             this.writeBreakscapedTagString(ingredient.title);
             this.writeCL();
-            this.writeNL();
+            // this.writeNL();
         }
+        this.writeNL();
         // [+] / [-]
         if (ingredient.checked) {
             this.writeOPP();
@@ -1465,8 +1353,10 @@ class BitmarkGenerator extends AstWalkerGenerator {
                 ignoreEmpty: true,
             });
         // item
-        if (ingredient.item != null)
+        if (ingredient.item != null) {
+            this.writeNL();
             this.write(ingredient.item);
+        }
         // Stop traversal of this branch
         return false;
     }
@@ -1476,35 +1366,20 @@ class BitmarkGenerator extends AstWalkerGenerator {
     }
     between_botResponses(_node, _left, _right, _route) {
         this.writeCardSetCardDivider();
-        this.writeNL();
-    }
-    exit_botResponses(_node, _route) {
-        //
-    }
-    // bitmarkAst -> bits -> bitsValue -> cardNode -> botResponses -> botResponsesValue
-    between_botResponsesValue(_node, _left, _right, _route) {
-        this.writeNL();
-    }
-    exit_botResponsesValue(_node, _route) {
-        this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> cardBits
     enter_cardBits(_node, _route) {
         //
     }
     between_cardBits(_node, _left, _right, _route) {
-        this.writeNL();
         this.writeCardSetCardDivider();
-        this.writeNL();
-    }
-    exit_cardBits(_node, _route) {
-        //
     }
     // bitmarkAst -> bits -> bitsValue -> cardNode -> botResponses -> botResponsesValue -> response
     leaf_response(node, route) {
         const parent = this.getParentNode(route);
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.botResponsesValue)
             return;
+        this.writeNL();
         this.writeOPB();
         this.writeBreakscapedTagString(node.value);
         this.writeCL();
@@ -1514,6 +1389,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         const parent = this.getParentNode(route);
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.botResponsesValue)
             return;
+        this.writeNL();
         this.writeProperty('reaction', node.value, {
             format: PropertyFormat.trimmedString,
             single: true,
@@ -1527,6 +1403,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
             return;
         const feeback = node.value;
         if (feeback) {
+            this.writeNL();
             this.write(feeback);
         }
     }
@@ -1534,12 +1411,13 @@ class BitmarkGenerator extends AstWalkerGenerator {
     enter_imagePlaceholder(node, _route) {
         const resource = node.value;
         // This is a resource, so handle it with the common code
+        this.writeNL();
         this.writePropertyStyleResource(node.key, resource);
         // Continue traversal
         return true;
     }
     exit_imagePlaceholder(_node, _route) {
-        this.writeNL();
+        // this.writeNL();
     }
     // bitmarkAst -> bits -> bitsValue -> posterImage
     // bitmarkAst -> bits -> bitsValue -> resource -> * -> posterImage
@@ -1638,9 +1516,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
     enter_book(_node, _route) {
         // Block standard property handling
     }
-    between_book(_node, _route) {
-        this.writeNL();
-    }
     enter_bookValue(node, _route) {
         const book = node.value;
         // const parent = this.getParentNode(route);
@@ -1711,10 +1586,11 @@ class BitmarkGenerator extends AstWalkerGenerator {
     //  * -> itemLead --> item
     //  * -> itemLead --> lead
     //  * -> hint
-    enter_hint(node, _route) {
+    enter_hint(node, route) {
         const value = node.value;
         const text = value;
         if (!this.isEmptyText(text)) {
+            this.writeNL_IfNotChain(route);
             this.writeOPQ();
             this.textGenerator.generateSync(text, TextFormat.bitmarkMinusMinus);
             this.writeCL();
@@ -1723,10 +1599,11 @@ class BitmarkGenerator extends AstWalkerGenerator {
         return false;
     }
     // bitmarkAst -> bits -> bitsValue ->  * -> instruction
-    enter_instruction(node, _route) {
+    enter_instruction(node, route) {
         const value = node.value;
         const text = value;
         if (!this.isEmptyText(text)) {
+            this.writeNL_IfNotChain(route);
             this.writeOPB();
             this.textGenerator.generateSync(text, TextFormat.bitmarkMinusMinus);
             this.writeCL();
@@ -1736,6 +1613,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
     }
     // bitmarkAst -> bits -> bitsValue ->  * -> lang
     enter_lang(node, _route) {
+        this.writeNL();
         this.writeProperty('lang', node.value, {
             format: PropertyFormat.boolean,
             single: true,
@@ -1744,6 +1622,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
     }
     // bitmarkAst -> bits -> bitsValue ->  * -> refAuthor
     enter_refAuthor(node, _route) {
+        this.writeNL();
         this.writeProperty('refAuthor', node.value, {
             format: PropertyFormat.trimmedString,
             single: false,
@@ -1752,6 +1631,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
     }
     // bitmarkAst -> bits -> bitsValue ->  * -> refBookTitle
     enter_refBookTitle(node, _route) {
+        this.writeNL();
         this.writeProperty('refBookTitle', node.value, {
             format: PropertyFormat.trimmedString,
             single: true,
@@ -1760,6 +1640,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
     }
     // bitmarkAst -> bits -> bitsValue ->  * -> refPublisher
     enter_refPublisher(node, _route) {
+        this.writeNL();
         this.writeProperty('refPublisher', node.value, {
             format: PropertyFormat.trimmedString,
             single: false,
@@ -1776,6 +1657,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
     }
     // bitmarkAst -> bits -> bitsValue ->  * -> citationStyle
     enter_citationStyle(node, _route) {
+        this.writeNL();
         this.writeProperty('citationStyle', node.value, {
             format: PropertyFormat.trimmedString,
             single: true,
@@ -1792,6 +1674,8 @@ class BitmarkGenerator extends AstWalkerGenerator {
         const example = (_a = parent === null || parent === void 0 ? void 0 : parent.value.example) !== null && _a !== void 0 ? _a : null;
         // const __isDefaultExample = parent?.value.__isDefaultExample ?? false;
         if (example != null && example !== '') {
+            // Write a newline if not in a chain
+            this.writeNL_IfNotChain(route);
             this.writeOPA();
             this.writeString('example');
             this.writeColon();
@@ -1821,6 +1705,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
     // bitmarkAst -> bits -> bitsValue -> elements -> elementsValue
     leaf_elementsValue(node, _route) {
         if (node.value) {
+            this.writeNL();
             this.writeBreakscapedTagString(node.value);
         }
     }
@@ -1866,18 +1751,21 @@ class BitmarkGenerator extends AstWalkerGenerator {
     // bitmarkAst -> bits -> bitsValue ->  * -> isCorrect
     // bitmarkAst -> bits -> bitsValue -> heading -> forKeys
     leaf_forKeys(node, _route) {
+        this.writeNL();
         this.writeOPHASH();
         this.writeBreakscapedTagString(node.value);
         this.writeCL();
     }
     // bitmarkAst -> bits -> bitsValue -> heading -> forValues
     leaf_forValues(node, _route) {
+        this.writeNL();
         this.writeOPHASH();
         this.writeBreakscapedTagString(node.value);
         this.writeCL();
     }
     // bitmarkAst -> bits -> bitsValue -> heading -> forValuesValue
     leaf_forValuesValue(node, _route) {
+        this.writeNL();
         this.writeOPHASH();
         this.writeBreakscapedTagString(node.value);
         this.writeCL();
@@ -1886,6 +1774,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
     // bitmarkAst -> bits -> bitsValue -> matrix -> matrixValue -> key
     leaf_key(node, _route) {
         if (node.value) {
+            this.writeNL();
             this.writeBreakscapedTagString(node.value);
         }
     }
@@ -1893,6 +1782,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
     // bitmarkAst -> bits -> bitsValue -> matrix -> matrixValue -> cells -> cellsValue -> values -> valuesValue
     leaf_valuesValue(node, _route) {
         if (node.value) {
+            this.writeNL();
             this.writeBreakscapedTagString(node.value);
         }
     }
@@ -1904,23 +1794,10 @@ class BitmarkGenerator extends AstWalkerGenerator {
         if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.questionsValue && (parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.flashcardsValue)
             return;
         if (node.value) {
+            this.writeNL();
             this.writeBreakscapedTagString(node.value);
-            // this.writeNL();
         }
     }
-    enter_question(node, route) {
-        // Ignore responses that are not at the questionsValue level as they are handled elsewhere
-        const parent = this.getParentNode(route);
-        if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.questionsValue && (parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.flashcardsValue)
-            return true;
-        if (node.value) {
-            this.textGenerator.generateSync(node.value, TextFormat.bitmarkMinusMinus);
-            // this.writeString(node.value);
-            // this.writeNL();
-        }
-        // Stop traversal of this branch
-        return false;
-    }
     // bitmarkAst -> bits -> bitsValue -> statements -> text
     // bitmarkAst -> bits -> bitsValue -> resource -> ...
     // bitmarkAst -> bits -> bitsValue -> resource -> posterImage -> ...
@@ -2025,8 +1902,9 @@ class BitmarkGenerator extends AstWalkerGenerator {
             ignoreFalse: true,
         });
     }
-    enter_caption(node, _route) {
+    enter_caption(node, route) {
         const value = node.value;
+        this.writeNL_IfNotChain(route);
         this.writeProperty('caption', value, {
             format: PropertyFormat.bitmarkMinusMinus,
             single: true, // ??
@@ -2089,17 +1967,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
     //
     // Resources
     //
-    // bitmarkAst -> bits -> bitsValue -> resources
-    between_resources(_node, _left, _right, _route) {
-        this.writeNL();
-    }
-    exit_resources(_node, _left, _right, _route) {
-        this.writeNL();
-    }
-    // bitmarkAst -> bits -> bitsValue -> resourcesValue
-    between_resourcesValue(_node, _left, _right, _route) {
-        this.writeNL();
-    }
     //
     // Generated Node Handlers
     //
@@ -2134,6 +2001,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
                 // url / src / href / app
                 const url = resource.url || resource.src || resource.body || '';
                 // Write the resource
+                this.writeNL();
                 this.writeResource(type, url);
                 // Continue traversal
                 return true;
@@ -2146,14 +2014,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
     /**
      * Generate the handlers for properties, as they are mostly the same, but not quite
      */
-    // protected enter_labelTrue(node: NodeInfo, _route: NodeInfo[],
-    // ): void {
-    //   const bit = parent?.value as Bit;
-    //   if (bit) {
-    //     this.writeProperty('labelTrue', node.value ?? '', true);
-    //     this.writeProperty('labelFalse', bit.labelFalse ?? '', true);
-    //   }
-    // }
     generatePropertyHandlers() {
         var _a;
         const propertiesConfig = Config.getRawPropertiesConfig();
@@ -2180,6 +2040,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
                 if ((parent === null || parent === void 0 ? void 0 : parent.key) !== NodeType.bitsValue)
                     return;
                 // Write the property
+                this.writeNL(); // Only if NOT in chain (how do we know this?)
                 this.writeProperty(propertyConfig.tag, node.value, {
                     format: (_a = propertyConfig.format) !== null && _a !== void 0 ? _a : PropertyFormat.trimmedString,
                     single: (_b = propertyConfig.single) !== null && _b !== void 0 ? _b : false,
@@ -2192,35 +2053,44 @@ class BitmarkGenerator extends AstWalkerGenerator {
             this[enterFuncName] = this[enterFuncName].bind(this);
         }
     }
-    writeNlBetween(node, left, _right, _route, 
-    // The following keys are combined with other keys so don't need newlines
-    noNlKeys) {
-        const bit = node.value;
-        if (bit.book) {
-            // If the book node exists, remove the newline caused by reference as it will be bound to book
-            noNlKeys.push(NodeType.reference);
-        }
-        // Check if a no newline key is to the left in this 'between' callback
-        const noNl = (() => {
-            if (!this.wroteSomething || this.skipNLBetweenBitsValue) {
-                return true;
-            }
-            for (const keyType of noNlKeys) {
-                if (left.key === keyType /*|| right.key === keyType*/)
-                    return true;
-            }
-            return false;
-        })();
-        if (!noNl) {
-            this.writeNL();
-        }
-        this.skipNLBetweenBitsValue = false;
-        this.wroteSomething = false;
-    }
     // END NODE HANDLERS
     //
     // UTILITY FUNCTIONS
     //
+    /**
+     * Check if in a chain.
+     *
+     * Return false if at the root of the bit or a card bit, otherwise true.
+     * This is useful to determine if a newline should be written before certain properties.
+     *
+     * @param route
+     * @returns
+     */
+    isChain(route) {
+        const parent = this.getParentNode(route);
+        // Root of bit
+        if ((parent === null || parent === void 0 ? void 0 : parent.key) === NodeType.bitsValue)
+            return false;
+        // Root of card bits
+        if ((parent === null || parent === void 0 ? void 0 : parent.key) === NodeType.cardBitsValue)
+            return false;
+        if ((parent === null || parent === void 0 ? void 0 : parent.key) === NodeType.feedbacksValue)
+            return false;
+        if ((parent === null || parent === void 0 ? void 0 : parent.key) === NodeType.quizzesValue)
+            return false;
+        if ((parent === null || parent === void 0 ? void 0 : parent.key) === NodeType.pairsValue)
+            return false;
+        if ((parent === null || parent === void 0 ? void 0 : parent.key) === NodeType.matrixValue)
+            return false;
+        if ((parent === null || parent === void 0 ? void 0 : parent.key) === NodeType.definitionsValue)
+            return false;
+        if ((parent === null || parent === void 0 ? void 0 : parent.key) === NodeType.questionsValue)
+            return false;
+        // Root of sub-card bits
+        if ((parent === null || parent === void 0 ? void 0 : parent.key) === NodeType.reason)
+            return false;
+        return true;
+    }
     haveValidCardSet(bit) {
         const bitConfig = Config.getBitConfig(bit.bitType);
         if (!bitConfig)
@@ -2352,17 +2222,20 @@ class BitmarkGenerator extends AstWalkerGenerator {
         this.write('#');
     }
     writePlainTextDivider() {
+        this.writeNL();
         this.write('==== text ====');
     }
     writeCardSetStart() {
+        this.writeNL();
         if (this.options.cardSetVersion === CardSetVersion.v1) {
-            this.write('\n===');
+            this.write('===');
         }
         else {
-            this.write('\n====');
+            this.write('====');
         }
     }
     writeCardSetEnd() {
+        this.writeNL();
         if (this.options.cardSetVersion === CardSetVersion.v1) {
             this.write('===');
         }
@@ -2371,6 +2244,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         }
     }
     writeCardSetCardDivider() {
+        this.writeNL();
         if (this.options.cardSetVersion === CardSetVersion.v1) {
             this.write('===');
         }
@@ -2379,6 +2253,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         }
     }
     writeCardSetSideDivider() {
+        this.writeNL();
         if (this.options.cardSetVersion === CardSetVersion.v1) {
             this.write('==');
         }
@@ -2387,6 +2262,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
         }
     }
     writeCardSetVariantDivider() {
+        this.writeNL();
         if (this.options.cardSetVersion === CardSetVersion.v1) {
             this.write('--');
         }
@@ -2394,6 +2270,11 @@ class BitmarkGenerator extends AstWalkerGenerator {
             this.write('++');
         }
     }
+    writeNL_IfNotChain(route) {
+        if (!this.isChain(route)) {
+            this.writeNL();
+        }
+    }
     writeNL() {
         if (this.options.debugGenerationInline) {
             this.write('\\n');
@@ -2413,7 +2294,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
             this.writeColon();
             this.writeBreakscapedTagString(src);
             if (resource.type === ResourceTag.article) {
-                this.writeNL();
+                // this.writeNL();
             }
             this.writeCL();
         }
@@ -2427,7 +2308,7 @@ class BitmarkGenerator extends AstWalkerGenerator {
             this.writeColon();
             this.writeBreakscapedTagString(value);
             if (type === ResourceTag.article) {
-                this.writeNL();
+                // this.writeNL();
             }
             this.writeCL();
         }
@@ -2435,7 +2316,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
     writeProperty(name, values, options) {
         var _a;
         let valuesArray;
-        let wroteSomething = false;
         if (values !== undefined) {
             const isBitmarkText = options.format === PropertyFormat.bitmarkMinusMinus || options.format === PropertyFormat.bitmarkPlusPlus;
             if (isBitmarkText) {
@@ -2447,7 +2327,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
                 this.writeColon();
                 this.textGenerator.generateSync(values, (_a = TextFormat.fromValue(options.format)) !== null && _a !== void 0 ? _a : TextFormat.bitmarkMinusMinus);
                 this.writeCL();
-                wroteSomething = true;
             }
             else {
                 // Write any other property type
@@ -2460,7 +2339,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
                 if (valuesArray.length > 0) {
                     if (options.single)
                         valuesArray = valuesArray.slice(valuesArray.length - 1);
-                    let propertyIndex = 0;
                     for (const val of valuesArray) {
                         if (val !== undefined) {
                             if (options.ignoreFalse && val === false)
@@ -2469,23 +2347,17 @@ class BitmarkGenerator extends AstWalkerGenerator {
                                 continue;
                             if (options.ignoreEmpty && val === '')
                                 continue;
-                            if (propertyIndex > 0)
-                                this.writeNL();
                             this.writeOPA();
                             this.writeBreakscapedTagString(name);
                             this.writeColon();
                             this.writeBreakscapedTagString(`${val}`);
                             this.writeCL();
-                            wroteSomething = true;
-                            propertyIndex++;
+                            // propertyIndex++;
                         }
                     }
                 }
             } // isBitmarkText
         }
-        if (!wroteSomething) {
-            this.skipNLBetweenBitsValue = true;
-        }
     }
     writeInlineDebug(key, state) {
         let tag = key;
@@ -2553,8 +2425,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
      * @param value - The string value to be written.
      */
     write(value) {
-        if (value)
-            this.wroteSomething = true;
         this.writer.write(value);
         return this;
     }
@@ -2563,8 +2433,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
      * @param value - The line to write. When omitted, only the endOfLineString is written.
      */
     writeLine(value) {
-        if (value)
-            this.wroteSomething = true;
         this.writer.writeLine(value);
         return this;
     }
@@ -2574,8 +2442,6 @@ class BitmarkGenerator extends AstWalkerGenerator {
      * @param delimiter - An optional delimiter to be written at the end of each line, except for the last one.
      */
     writeLines(values, delimiter) {
-        if (values.length > 0 && values.reduce((acc, v) => (v ? true : acc), false))
-            this.wroteSomething = true;
         this.writer.writeLines(values, delimiter);
         return this;
     }