Skip to content

Reduce cognitive complexity in ParseWorkflowTaskPlaceholders#589

Merged
ziagham merged 1 commit intoflowsynx:masterfrom
mohin-io:issue-587-reduce-task-complexity
Oct 27, 2025
Merged

Reduce cognitive complexity in ParseWorkflowTaskPlaceholders#589
ziagham merged 1 commit intoflowsynx:masterfrom
mohin-io:issue-587-reduce-task-complexity

Conversation

@mohin-io
Copy link
Copy Markdown
Contributor

@mohin-io mohin-io commented Oct 26, 2025

Closes #587.

Summary

  • Extracted parsing responsibilities into focused helpers for top-level fields, parameters, approvals, and dependencies
  • Preserved plugin context handling while reducing branching inside ParseWorkflowTaskPlaceholders
  • Added concise XML documentation to describe the new helpers and align with project style

Testing

  • not run (dotnet CLI unavailable locally: dotnet test)

@mohin-io mohin-io requested review from a team as code owners October 26, 2025 20:59
@sonarqubecloud
Copy link
Copy Markdown

@ziagham ziagham merged commit 72ba185 into flowsynx:master Oct 27, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor: Reduce Cognitive Complexity in ParseWorkflowTaskPlaceholders method

2 participants