Fixed a bunch of issues with the extension

Author: timfennis

ext/andy-cpp/CHANGELOG.md

@@ -1,12 +1,21 @@
 # Change Log
 
-## [v0.0.6] (2024-10-11)
+## [v0.1.0] (2024-12-30)
+
+This release probably still has bugs but it seems to display all 2024 Advent of Code puzzles correctly,
+which is the point of the 0.1.0 release.
+
+- Fixed detection of lots of operators like `<` and `>`
+- `%=` and `%%=` are detected correctly
+- Identify variables and punctuation like `,` and `:`
+
+## [v0.0.6] (2024-12-11)
 
 - Fixed some issues with arbitrary base literals not getting parsed correctly
 - Correctly detect augmented assignment operators like `++=`
 - Detect `<>` and `++` as special non arithmetic operators
 
-## [v0.0.5] (2024-10-11)
+## [v0.0.5] (2024-12-11)
 
 - Fixed an issue with `.` operator's capture groups incorrectly parsing methods as operators
 

ext/andy-cpp/build.sh

@@ -1,4 +1,3 @@
 #!/bin/fish
 
-
 docker run -v (pwd):/build -w /build -it timfennis/vsce package

ext/andy-cpp/package.json

@@ -7,7 +7,7 @@
     "url": "https://github.com/timfennis/andy-cpp.git"
   },
   "publisher": "TimFennis",
-  "version": "0.0.6",
+  "version": "0.1.0",
   "engines": {
     "vscode": "^1.85.0"
   },

ext/andy-cpp/syntaxes/andy-cpp.tmLanguage.json

@@ -36,6 +36,12 @@
         {
           "include": "#keywords"
         },
+        {
+          "include": "#variables"
+        },
+        {
+          "include": "#punctuation"
+        },
         {
           "include": "#comment"
         }
@@ -133,6 +139,11 @@
           "match": "\\b([0-9][0-9_]*)\\b",
           "name": "constant.numeric.decimal.andy-cpp"
         },
+        {
+          "comment": "Infinity",
+          "match": "\\bInf\\b",
+          "name": "constant.numeric.infinity.andy-cpp"
+        },
         {
           "comment": "booleans",
           "name": "constant.language.bool.andy-cpp",
@@ -216,7 +227,7 @@
         {
           "comment": "augmented assignment operators",
           "name": "keyword.operator.assignment.andy-cpp",
-          "match": "((\\+\\+?|-|\\*|/|\\|\\^|&|\\||<<|>>|~|<>)=)"
+          "match": "((\\+\\+?|-|\\*|/|\\|\\^|&|\\||%%?|<<|>>|~|<>)=)"
         },
         {
           "comment": "function augmented assignment opreators",
@@ -229,20 +240,20 @@
           "match": "(?<![<>])=(?!=|>)"
         },
         {
-          "comment": "comparison operators",
-          "name": "keyword.operator.comparison.andy-cpp",
-          "match": "(>=<|<=>|=(=)?(?!>)|!=|<=|(?<!=)>=)"
-        },
-        {
-          "comment": "math operators",
+          "comment": "concatenation operators",
           "name": "keyword.operator.other.andy-cpp",
           "match": "(\\+\\+|<>)"
         },
         {
-          "comment": "math operators",
+          "comment": "binary operators",
           "name": "keyword.operator.arithmetic.andy-cpp",
           "match": "(\\+|\\-|\\*|/(?!\\/)|\\\\|\\^|\\%%?|>>|<<|&(?!&)|\\|(?!\\|)|~)"
         },
+        {
+          "comment": "comparison operators",
+          "name": "keyword.operator.comparison.andy-cpp",
+          "match": "(>=<|<=>|=(=)?(?!>)|!=|<=|(?<!=)>=|>|<)"
+        },
         {
           "name": "punctuation.semicolon.andy-cpp",
           "match": ";"
@@ -282,6 +293,11 @@
         }
       },
       "patterns": [
+        {
+          "comment": "Key value separator",
+          "name": "punctuation.separator.dictionary.andy-cpp",
+          "match": ":"
+        },
         {
           "include": "#expression"
         }
@@ -345,6 +361,24 @@
           "name": "constant.character.escape.andy-cpp"
         }
       }
+    },
+    "variables": {
+      "patterns": [
+        {
+          "comment": "variables",
+          "name": "variable.other.andy-cpp",
+          "match": "\\b[a-z_][a-z0-9_]*\\b"
+        }
+      ]
+    },
+    "punctuation": {
+      "patterns": [
+        {
+          "comment": "comma",
+          "name": "punctuation.comma.andy-cpp",
+          "match": ","
+        }
+      ]
     }
   },
   "scopeName": "source.andy-cpp"

src/interpreter/evaluate.rs

@@ -590,6 +590,7 @@ pub(crate) fn evaluate_expression(
                             lhs_expr.span.merge(index_expr.span),
                         )?;
 
+                        // TODO: This borrow_mut can fail, handle it better!!
                         dict.borrow_mut().insert(key, default_value.clone());
 
                         Ok(default_value)