LoadingGroups: Remove recursions from function

[anthony-murphy]

This pull request refactors the logic in the isSnapshotFetchRequiredForLoadingGroupId function to improve its efficiency and readability. Instead of using recursion to traverse the snapshot tree, it now uses an explicit stack-based approach for iterative traversal. This change makes the code easier to follow and can help avoid potential stack overflow issues with deeply nested trees.

Snapshot tree traversal improvements:

• Replaced recursive traversal with an iterative approach using a stack (trees) to process each ISnapshotTree node, making the function more efficient and readable.
• Ensured that child trees without a groupId are pushed onto the stack for further evaluation, maintaining the correct logic for determining if a snapshot fetch is required.

[Copilot]

Pull Request Overview

This PR refactors the isSnapshotFetchRequiredForLoadingGroupId function to replace recursive traversal with an iterative stack-based approach for processing snapshot trees. This change improves efficiency and prevents potential stack overflow issues with deeply nested trees.

Key Changes

• Replaced recursive function calls with an iterative approach using an explicit stack
• Maintained the same logical behavior while improving performance and readability
• Eliminated the risk of stack overflow for deeply nested snapshot trees

Comments suppressed due to low confidence (1)

packages/runtime/runtime-utils/src/snapshotUtils.ts:38

• Missing return statement at the end of the function. The iterative implementation needs to return false when no blobs requiring fetch are found after processing all trees in the stack.

	return false;