From 78996f849d60aae81060e36ebbb8ad2f6ea01217 Mon Sep 17 00:00:00 2001
From: Nilay Majorwar <nilaymajorwar@gmail.com>
Date: Sun, 28 May 2023 19:47:56 +0530
Subject: [PATCH] Fix Chef parser throwing at section end

---
 languages/chef/parser/index.ts | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/languages/chef/parser/index.ts b/languages/chef/parser/index.ts
index db68daf..1bfa10a 100644
--- a/languages/chef/parser/index.ts
+++ b/languages/chef/parser/index.ts
@@ -94,7 +94,7 @@ const parseEmptyLine = (
 
 /** Parse the stack for method instructions section */
 const parseRecipeComments = (stack: CodeStack): void => {
-  while (stack[stack.length - 1].line.trim() !== "") stack.pop();
+  while (stack[stack.length - 1]?.line.trim() !== "") stack.pop();
 };
 
 /** Parse the stack for the header of ingredients section */
@@ -266,6 +266,7 @@ const parseRecipe = (
   parseEmptyLine(stack, lastCharRange);
 
   // Check if exists and parse recipe comments
+  assertCodeStackNotEmpty(stack, lastCharRange);
   if (stack[stack.length - 1].line.trim() !== "Ingredients.") {
     parseRecipeComments(stack);
     parseEmptyLine(stack, lastCharRange);
@@ -277,12 +278,14 @@ const parseRecipe = (
   parseEmptyLine(stack, lastCharRange);
 
   // Check if exists and parse cooking time
+  assertCodeStackNotEmpty(stack, lastCharRange);
   if (stack[stack.length - 1].line.trim().startsWith("Cooking time: ")) {
     parseCookingTime(stack);
     parseEmptyLine(stack, lastCharRange);
   }
 
   // Check if exists and parse oven temperature
+  assertCodeStackNotEmpty(stack, lastCharRange);
   if (stack[stack.length - 1].line.trim().startsWith("Pre-heat oven ")) {
     parseOvenSetting(stack);
     parseEmptyLine(stack, lastCharRange);
@@ -349,3 +352,16 @@ const popCodeStack = (
   const line = trim ? item.line.trim() : item.line;
   return { line, row: item.row };
 };
+
+/**
+ * Utility to assert that given code stack is not empty.
+ * If it is, throws a ParseError for unexpected EOF for the given DocumentRange.
+ * @param stack Code stack to assert on
+ * @param range DocumentRange to throw ParseError on
+ */
+const assertCodeStackNotEmpty = (
+  stack: CodeStack,
+  range: DocumentRange
+): void => {
+  if (stack.length == 0) throw new ParseError("Unexpected EOF", range);
+};