Fixed recovery of loops

[LanderOtto]

This commit fixes the recovery of workflows which have loops.

Requires PR #822

[codecov]

Codecov Report

❌ Patch coverage is 81.39535% with 48 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.10%. Comparing base (fccb5b2) to head (0a6b388).
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
streamflow/recovery/utils.py	82.48%	18 Missing and 6 partials ⚠️
streamflow/recovery/policy/recovery.py	75.00%	5 Missing and 3 partials ⚠️
streamflow/deployment/connector/kubernetes.py	80.00%	6 Missing and 1 partial ⚠️
streamflow/workflow/step.py	81.81%	2 Missing and 2 partials ⚠️
streamflow/workflow/port.py	86.36%	2 Missing and 1 partial ⚠️
streamflow/cwl/transformer.py	0.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #826      +/-   ##
==========================================
+ Coverage   73.46%   74.10%   +0.63%     
==========================================
  Files          91       91              
  Lines       12257    12374     +117     
  Branches     2160     2180      +20     
==========================================
+ Hits         9005     9170     +165     
+ Misses       2742     2689      -53     
- Partials      510      515       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

In general, the fix is to either (a) reinstate the code as active logic if it is required, or (b) remove it completely if it’s no longer needed. Since we cannot safely change the program’s behavior without broader context or tests, the least risky fix that preserves existing functionality is to delete the commented‑out if block and keep only the descriptive comment explaining the current behavior.

Concretely, in streamflow/recovery/utils.py, inside DirectGraph.remove_nodes, in the loop over succ in self._successors[current], we will remove the two commented lines:

• # if not (self._predecessors[succ].difference(stack))):
• # stack.append(succ)
  leaving the surrounding logic unchanged. No new imports, methods, or definitions are required.