Fix 6 SonarCloud code smell issues (S927, S3267, S3776, S3261)

[Copilot]

Resolves six open SonarCloud code smells spanning parameter naming, loop simplification, cognitive complexity, and an empty namespace.

Changes

• S927 — parameter name mismatch (SarifReporter.cs, SonarReporter.cs): Renamed override parameter reportResults → results to match the ReporterBase declaration.

• S3267 — loop simplification (SlnxFile.cs): Replaced manual foreach + inner variable extraction with a LINQ projection:

  // Before
  foreach (var entry in solution.AllFiles())
  {
      var path = entry.Path;
      if (string.IsNullOrEmpty(path)) continue;
      ...
  }
  
  // After
  foreach (var path in solution.AllFiles()
      .Where(e => !string.IsNullOrEmpty(e.Path))
      .Select(e => e.Path))
  { ... }

• S3267 — loop simplification (RequiredFilesChecker.cs): Moved the if (!Contains(...)) guard into a .Where(...) clause on the loop source.

• S3776 — cognitive complexity (SlnxCollector.cs): Extracted per-file processing from CollectAsync (complexity 20) into a new ProcessFileAsync private method, bringing CollectAsync well under the allowed threshold of 15.

• S3261 — empty namespace (SonarRuleSeverity.cs): Removed the empty namespace declaration; file now contains only the explanatory comments.

Original prompt

Doel

Los de volgende 6 openstaande SonarCloud code smell issues op in één PR:

---

1. S927 — SarifReporter.cs regel 15

Bericht: Rename parameter reportResults to results to match the base class declaration.

ReporterBase.WriteReportAsync declareert de parameter als results. De override in SarifReporter gebruikt reportResults.

Fix: Hernoem de parameter reportResults naar results in de methode-signatuur én in de body van WriteReportAsync in src/SLNX-validator.Core/SarifReporting/SarifReporter.cs.

---

2. S927 — SonarReporter.cs regel 10

Bericht: Rename parameter reportResults to results to match the base class declaration.

Zelfde probleem als hierboven maar in src/SLNX-validator.Core/SonarQubeReporting/SonarReporter.cs.

Fix: Hernoem de parameter reportResults naar results in de methode-signatuur én in de body.

---

3. S3267 — SlnxFile.cs regel 108

Bericht: Loop should be simplified by calling Select(entry => entry.Path)

In ComputeAbsoluteFiles staat:

foreach (var entry in solution.AllFiles())
{
    var path = entry.Path;
    ...
}

Fix: Gebruik Select om entry.Path direct te projecteren:

foreach (var path in solution.AllFiles().Select(entry => entry.Path))

---

4. S3776 — SlnxCollector.cs regel 11

Bericht: Refactor this method to reduce its Cognitive Complexity from 20 to the 15 allowed.

De CollectAsync methode in src/SLNX-validator/SlnxCollector.cs is te complex.

Fix: Extraheer de binnenste logica voor het verwerken van een enkel bestand naar een aparte private methode, bijv. ProcessFileAsync. De foreach-lus in CollectAsync roept dan ProcessFileAsync aan. Dit verlaagt de cognitive complexity van CollectAsync significant.

---

5. S3267 — RequiredFilesChecker.cs regel 38

Bericht: Loops should be simplified using the "Where" LINQ method

In CheckInSlnx:

foreach (var requiredPath in requiredAbsolutePaths)
{
    if (!slnxFile.Files.Contains(requiredPath, StringComparer.OrdinalIgnoreCase))
    {
        ...
    }
}

Fix: Gebruik LINQ Where om de conditie te verplaatsen:

foreach (var requiredPath in requiredAbsolutePaths.Where(p => !slnxFile.Files.Contains(p, StringComparer.OrdinalIgnoreCase)))
{
    ...
}

---

6. S3261 — SonarRuleSeverity.cs regel 3

Bericht: Remove this empty namespace.

src/SLNX-validator.Core/SonarQubeReporting/SonarRuleSeverity.cs bevat alleen een comment en een lege namespace:

// This file is intentionally empty. SonarRuleSeverity has been renamed to RuleSeverity.
// See Core/Reporting/RuleSeverity.cs
namespace JulianVerdurmen.SlnxValidator.Core.SonarQubeReporting;

Fix: Verwijder de namespace declaratie (laat alleen de comments staan, of verwijder het hele bestand als er geen referenties meer zijn).

---

Algemene opmerkingen

• Zorg dat alle bestaande tests nog steeds groen zijn.
• Geen functionele gedragswijzigingen — alleen code quality verbeteringen.

The following is the prior conversation context from the user's chat exploration (may be truncated):

Assistant: Helaas kan ik de SonarCloud-link niet direct openen of uitlezen — ik heb geen toegang tot externe webservices zoals SonarCloud.

Wat ik wel kan doen:

1. Kopieer de issues uit SonarCloud en plak ze hier, dan kan ik helpen ze op te lossen.
2. Maak een PR in 304NotModified/SLNX-validator als je weet welke code-issues aangepakt moeten worden.
3. Onderzoek de codebase als je me vertelt wat voor soort problemen SonarCloud heeft gevonden (bijv. code smells, bugs, security vulnerabilities).

Wat is het makkelijkst voor jou?

User: Heb je een export url?

Assistant: Ja! SonarCloud heeft een publieke REST API. Je kunt de issues ophalen via:

https://sonarcloud.io/api/issues/search?componentKeys=slnx-validator&issueStatuses=OPEN,CONFIRMED

Plak die URL in je browser (of stuur me de JSON-output), dan kan ik de issues analyseren en helpen oplossen.

Als het project privé is, heb je een token nodig. Die kun je aanmaken op: sonarcloud.io/account/security

Probeer het eens en plak de output hier!

User: {"total":7,"p":1,"ps":100,"paging":{"pageIndex":1,"pageSize":100,"total":7},"effortTotal":58,"debtTotal":58,"issues":[{"key":"AZ1puUd_P77SLJn5J6UV","rule":"csharpsquid:S927","severity":"CRITICAL","component":"slnx-validator:src/SLNX-validator.Core/SarifReporting/SarifReporter.cs","project":"slnx-validator","line":15,"hash":"247cfac9bd39dfac92b3b3a004be4845","textRange":{"startLine":15,"endLine":15,"startOffset":62,"endOffset":75},"flows":[],"status":"OPEN","message":"Rename parameter \u0027reportResults\u0027 to \u0027results\u0027 to match the base class declaration.","effort":"10min","debt":"10min","tags":["suspicious"],"creationDate":"2026-04-06T18:13:31+0000","updateDate":"2026-04-06T22:46:01+0000","type":"CODE_SMELL","organization":"304notmodifie...

This pull request was created from Copilot chat.