From 552fe0fb2448bb0f078a1db47ab8555eddef18ca Mon Sep 17 00:00:00 2001
From: Dominik Przybyl <przybyl.dominik@gmail.com>
Date: Tue, 31 Oct 2023 12:35:40 +0100
Subject: [PATCH 1/2] added empty list and empty structure handling

---
 app/aem/core/src/main/antlr/ApmLang.g4           | 16 +++++++++++-----
 .../cognifide/apm/core/grammar/ScriptRunner.java |  4 ++++
 .../apm/core/grammar/ScriptRunnerTest.groovy     |  4 +++-
 app/aem/core/src/test/resources/define.apm       |  4 ++++
 4 files changed, 22 insertions(+), 6 deletions(-)

diff --git a/app/aem/core/src/main/antlr/ApmLang.g4 b/app/aem/core/src/main/antlr/ApmLang.g4
index 9b7c6df5..23ddda54 100644
--- a/app/aem/core/src/main/antlr/ApmLang.g4
+++ b/app/aem/core/src/main/antlr/ApmLang.g4
@@ -33,7 +33,7 @@ name
     ;
 
 privilegeName
-    : IDENTIFIER ':' IDENTIFIER
+    : IDENTIFIER COLON IDENTIFIER
     ;
 
 path
@@ -42,7 +42,7 @@ path
     ;
 
 array
-    : ARRAY_BEGIN arrayValue (',' arrayValue)* ARRAY_END
+    : ARRAY_BEGIN (arrayValue (COMMA arrayValue)*)? ARRAY_END
     ;
 
 arrayValue
@@ -53,11 +53,11 @@ arrayValue
     ;
 
 structure
-    : STRUCTURE_BEGIN structureEntry (',' structureEntry)* STRUCTURE_END
+    : STRUCTURE_BEGIN (structureEntry (COMMA structureEntry)*)? STRUCTURE_END
     ;
 
 structureEntry
-    : structureKey ':' structureValue
+    : structureKey COLON structureValue
     ;
 
 structureKey
@@ -156,7 +156,7 @@ flag
     ;
 
 body
-    : BLOCK_BEGIN command+ BLOCK_END
+    : BLOCK_BEGIN command* BLOCK_END
     ;
 
 /*
@@ -176,6 +176,12 @@ STRUCTURE_BEGIN
 STRUCTURE_END
     : '}'
     ;
+COMMA
+    : ','
+    ;
+COLON
+    : ':'
+    ;
 BLOCK_BEGIN
     : 'begin'
     | 'BEGIN'
diff --git a/app/aem/core/src/main/java/com/cognifide/apm/core/grammar/ScriptRunner.java b/app/aem/core/src/main/java/com/cognifide/apm/core/grammar/ScriptRunner.java
index f74d1eb9..0232050c 100644
--- a/app/aem/core/src/main/java/com/cognifide/apm/core/grammar/ScriptRunner.java
+++ b/app/aem/core/src/main/java/com/cognifide/apm/core/grammar/ScriptRunner.java
@@ -145,6 +145,10 @@ public Status visitRequireVariable(RequireVariableContext ctx) {
     public Status visitForEach(ForEachContext ctx) {
       List<Map<String, ApmType>> values = readValues(ctx);
       ListIterator<Map<String, ApmType>> iterator = values.listIterator();
+      if (!iterator.hasNext() && shouldVisitNextChild()) {
+        String key = ctx.IDENTIFIER().toString();
+        progress(ctx, Status.SKIPPED, "for-each", String.format("%s is always empty", key));
+      }
       while (iterator.hasNext() && shouldVisitNextChild()) {
         try {
           int index = iterator.nextIndex();
diff --git a/app/aem/core/src/test/groovy/com/cognifide/apm/core/grammar/ScriptRunnerTest.groovy b/app/aem/core/src/test/groovy/com/cognifide/apm/core/grammar/ScriptRunnerTest.groovy
index b9414da6..e7ef01b7 100644
--- a/app/aem/core/src/test/groovy/com/cognifide/apm/core/grammar/ScriptRunnerTest.groovy
+++ b/app/aem/core/src/test/groovy/com/cognifide/apm/core/grammar/ScriptRunnerTest.groovy
@@ -92,7 +92,9 @@ class ScriptRunnerTest extends Specification {
                      "Executing command SHOW \"t\"",
                      "Executing command SHOW [3, \"ab\"]",
                      "Executing command SHOW [\"a\", \"b\", \"c\", \"d\", 1, 2]",
-                     "Executing command SHOW [\n\t[\"a\", \"b\"],\n\t[\"c\", \"d\"]\n]"]
+                     "Executing command SHOW [\n\t[\"a\", \"b\"],\n\t[\"c\", \"d\"]\n]",
+                     "Executing command SHOW []",
+                     "Executing command SHOW {}"]
     }
 
     def "run macro"() {
diff --git a/app/aem/core/src/test/resources/define.apm b/app/aem/core/src/test/resources/define.apm
index 4074e82e..5b4ecab9 100644
--- a/app/aem/core/src/test/resources/define.apm
+++ b/app/aem/core/src/test/resources/define.apm
@@ -43,6 +43,8 @@ DEFINE tab3 [['a', 'b'], ['c', 'd']]
 DEFINE obj {x:'a', y:1, z:['c', 1], 't':'t'}
 DEFINE tabEnum [a, b, "c", "d", 1, 2]
 DEFINE tab4 [1 + 2, "a" + "b"]
+DEFINE emptyList []
+DEFINE emptyMap {}
 SHOW $a
 SHOW $b
 SHOW $tab1
@@ -60,3 +62,5 @@ SHOW ${obj[t]}
 SHOW $tab4
 SHOW $tabEnum
 SHOW $tab3
+SHOW $emptyList
+SHOW $emptyMap

From d1450f1555f39393b248c8c23125b11b042bdedf Mon Sep 17 00:00:00 2001
From: Dominik Przybyl <przybyl.dominik@gmail.com>
Date: Tue, 31 Oct 2023 23:48:51 +0100
Subject: [PATCH 2/2] revert some changes

---
 app/aem/core/src/main/antlr/ApmLang.g4 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/aem/core/src/main/antlr/ApmLang.g4 b/app/aem/core/src/main/antlr/ApmLang.g4
index 23ddda54..38e5ef09 100644
--- a/app/aem/core/src/main/antlr/ApmLang.g4
+++ b/app/aem/core/src/main/antlr/ApmLang.g4
@@ -156,7 +156,7 @@ flag
     ;
 
 body
-    : BLOCK_BEGIN command* BLOCK_END
+    : BLOCK_BEGIN command+ BLOCK_END
     ;
 
 /*