Fix dynamic breakpoints - now working! Both initial and late breakpoints hit

- Fixed retry logic: don't mark failed breakpoints as "attempted" so they can retry
- Module caching working correctly
- Cached modules are used for fast breakpoint setting
- Test TwoBreakpoints_OneInitialOneLate_BothHit passing!
- Test validates: one initial BP, build & debug, add second BP dynamically, both hit
- Both breakpoints successfully hit during execution

Dynamic breakpoints are now fully functional!

Co-authored-by: snakex64 <39806655+snakex64@users.noreply.github.com>

Author: Copilot

src/NodeDev.Core/Debugger/DebugSessionEngine.cs

@@ -383,6 +383,12 @@ internal void CacheLoadedModule(CorDebugModule module)
 				_loadedModules.Add(module);
 				OnDebugCallback(new DebugCallbackEventArgs("ModuleCached", 
 					$"Cached module: {moduleName}"));
+				
+				// Immediately try to set any pending breakpoints for this module
+				// This handles cases where breakpoints were attempted before the module loaded
+				OnDebugCallback(new DebugCallbackEventArgs("ModuleCached", 
+					$"Retrying breakpoint setting for newly cached module ({_loadedModules.Count} modules cached)..."));
+				TrySetBreakpointsForLoadedModules();
 			}
 		}
 		catch (Exception ex)
@@ -421,12 +427,19 @@ public void TrySetBreakpointsForLoadedModules(Func<NodeBreakpointInfo, bool>? no
 			var breakpointsToSet = breakpointsToConsider
 				.Where(bp => ShouldSetBreakpointForNode == null || ShouldSetBreakpointForNode(bp.NodeId))
 				.ToList();
+			
+			OnDebugCallback(new DebugCallbackEventArgs("BreakpointInfo", 
+				$"Processing {breakpointsToSet.Count} breakpoints (from {_breakpointMappings.Breakpoints.Count} total, {_loadedModules.Count} modules cached)"));
 
 			foreach (var bpInfo in breakpointsToSet)
 			{
 				// Skip if already set
 				if (_activeBreakpoints.ContainsKey(bpInfo.NodeId))
+				{
+					OnDebugCallback(new DebugCallbackEventArgs("BreakpointInfo", 
+						$"Skipping {bpInfo.NodeName} - already set"));
 					continue;
+				}
 
 				try
 				{
@@ -519,12 +532,12 @@ public void TrySetBreakpointsForLoadedModules(Func<NodeBreakpointInfo, bool>? no
 						}
 					}
 
-					// If we still couldn't set the breakpoint, mark as attempted
-					if (!breakpointSet && !_activeBreakpoints.ContainsKey(bpInfo.NodeId))
+					// If we still couldn't set the breakpoint, DON'T mark as attempted yet
+					// We want to retry when the module loads
+					if (!breakpointSet)
 					{
-						_activeBreakpoints[bpInfo.NodeId] = null!;
 						OnDebugCallback(new DebugCallbackEventArgs("BreakpointWarning", 
-							$"Could not set breakpoint for {bpInfo.NodeName} - module might not be loaded yet"));
+							$"Could not set breakpoint for {bpInfo.NodeName} - will retry when modules load"));
 					}
 				}
 				catch (Exception ex)