Variable array handling

Author: MrMineO5

src/control/LiveInstrumentRemote.ts

@@ -179,6 +179,7 @@ export default class LiveInstrumentRemote {
 
                     let promises = [];
                     for (let variable of result) {
+                        if (!variable.value.objectId) continue;
                         promises.push(this.getVariable(variable.value.objectId, newRemainingDepth)
                             .then(res => variable.value.value = res));
                     }
@@ -313,7 +314,7 @@ export default class LiveInstrumentRemote {
     test() {
         let instrument = new LiveBreakpoint();
         instrument.id = randomUUID();
-        instrument.location = {source: "test/javascript/test.js", line: 5};
+        instrument.location = {source: "test/javascript/test.js", line: 8};
         instrument.hitLimit = 1;
         instrument.applyImmediately = true;
         instrument.applied = false;

src/util/VariableUtil.ts

@@ -22,6 +22,10 @@ namespace VariableUtil {
         return variables;
     }
 
+    export function encodeVariables(variables: Runtime.PropertyDescriptor[]) {
+        return variables.map(encodeVariable);
+    }
+
     export function encodeVariable(variable: Runtime.PropertyDescriptor) {
         if (!variable.value) {
             return JSON.stringify({
@@ -51,7 +55,8 @@ namespace VariableUtil {
         };
         obj[variable.name] = "";
         if (variable.value.value) {
-            if (variable.value.type !== 'object') { // TODO: Handle arrays
+            // Arrays are also objects, so no special handling is necessary
+            if (variable.value.type !== 'object') {
                 obj[variable.name] = value;
             } else {
                 obj[variable.name] = variable.value.value.reduce((acc, v) => {

test/javascript/test.js

@@ -1,6 +1,10 @@
 const SourcePlusPlus = require("../../dist/SourcePlusPlus.js");
 SourcePlusPlus.start({}, true).then(() => {
+    let testarray = ["this", "is", "an", "array"];
+
     console.log("test");
 
     SourcePlusPlus.liveInstrumentRemote.test();
+
+    let i = 2;
 });