Either use FirstVariableId or get rid of it. We'll continue to use it for now.

Author: rkeithhill

src/PowerShellEditorServices/Debugging/DebugService.cs

@@ -306,8 +306,7 @@ private async Task ClearBreakpointsInFile(ScriptFile scriptFile)
 
         private async Task FetchStackFramesAndVariables()
         {
-            // Avoid using 0 as it indicates a variable node with no children.
-            this.nextVariableId = 1;
+            this.nextVariableId = VariableDetailsBase.FirstVariableId;
             this.variables = new List<VariableDetailsBase>();
 
             // Create a dummy variable for index 0, should never see this.

src/PowerShellEditorServices/Debugging/VariableDetailsBase.cs

@@ -12,7 +12,8 @@ namespace Microsoft.PowerShell.EditorServices
     public abstract class VariableDetailsBase
     {
         /// <summary>
-        /// Provides a constant that is used as the starting variable ID for all
+        /// Provides a constant that is used as the starting variable ID for all.
+        /// Avoid 0 as it indicates a variable node with no children.
         /// variables.
         /// </summary>
         public const int FirstVariableId = 1;