notes for BF ME review

Author: bfarmer67

src/Hyperbee.Expressions/Transformation/StateMachineBuilder.cs

@@ -433,7 +433,18 @@ FieldInfo[] fields
     //    return bodyExpressions;
     //}
 
-    private static List<Expression> GetBodyExpressions( Expression jumpTable, StateContext.Scope firstScope ) //BF ME Where does this operation belong? Should we do this for each scope?
+    //BF ME
+    //
+    // This proof-of-concept only merges the top-level scope.
+    //
+    // The final implementation
+    //
+    //  - should merge all scopes
+    //  - should probably be moved to the state-machine (node) optimizer
+    //  - needs to latch in to try-catch transitions
+    //  - should we consider a `BlockNodeExpression` that contains mergeable nodes?
+
+    private static List<Expression> GetBodyExpressions( Expression jumpTable, StateContext.Scope firstScope ) 
     {
         return [jumpTable, MergeNodeExpressions( firstScope.Nodes )];
 

src/Hyperbee.Expressions/Transformation/Transitions/TryCatchTransition.cs

@@ -60,7 +60,7 @@ List<Expression> GetExpressions()
                 )
             };
 
-            body.AddRange( StateScope.Nodes );
+            body.AddRange( StateScope.Nodes ); //BF ME - merge nodes here or add a single `NodeBlockExpression` node that contains the merge reduce
 
             MapCatchBlock( Parent.StateOrder, out var catches, out var switchCases );