Skip to content

fix(sonar): clear develop code smells and harden 100-continue CI waits - #1089

Merged
justcoding121 merged 2 commits into
developfrom
fix/sonar-smells-and-ci-develop
Oct 10, 2026
Merged

justcoding121 merged 2 commits into
developfrom
fix/sonar-smells-and-ci-develop

Conversation

@justcoding121

Copy link
Copy Markdown
Owner

Summary

  • Clear the 5 open Sonar code smells on develop (LooksLikeBotChallenge complexity / Any(), CA1861 static chip arrays).
  • Raise Expect: 100-continue happy-path read timeout (keep a short timeout for the deadlock baseline) so shared CI runners stop failing under load.

Test plan

  • Unit tests for LooksLikeBotChallenge / DecryptFailureLearning
  • HideHostAndProcess inspector test
  • Develop .NET workflow green (build badge)
  • Sonar code_smells measure at 0 on develop

Reduce LooksLikeBotChallenge complexity, prefer static chip arrays in tests, and give Expect: 100-continue happy paths a longer read timeout so shared runners stop timing out under load.
…Windows.

Use Any() for bot-challenge header scan, and raise the large-body tab budget so loaded Windows runners stop failing around 5 s.
@sonarqubecloud

Copy link
Copy Markdown

@justcoding121
justcoding121 merged commit aab2884 into develop Oct 10, 2026
20 checks passed
@justcoding121
justcoding121 deleted the fix/sonar-smells-and-ci-develop branch October 10, 2026 12:56
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.

1 participant