From d7f616b9dfd1b184de398cf80765b69cc6a853dd Mon Sep 17 00:00:00 2001 From: Jon Lipsky Date: Wed, 6 May 2026 11:33:47 -0700 Subject: [PATCH 1/2] test: v1 validation probe for Claude path --- .github/sonar-fix-config.yml | 2 +- .github/workflows/sonar-fix.yml | 2 +- src/main/java/clipper2/SonarFixProbeV1Claude.java | 8 ++++++++ 3 files changed, 10 insertions(+), 2 deletions(-) create mode 100644 src/main/java/clipper2/SonarFixProbeV1Claude.java diff --git a/.github/sonar-fix-config.yml b/.github/sonar-fix-config.yml index 5f33c7c..c3f58f9 100644 --- a/.github/sonar-fix-config.yml +++ b/.github/sonar-fix-config.yml @@ -1,4 +1,4 @@ # Per-repo override: route to Codex instead of the central default's Claude. # Everything else (severities, rules allow/deny, etc.) inherits from # config/default.yml in the central sonar-fix repo at run time. -agent: codex +agent: claude diff --git a/.github/workflows/sonar-fix.yml b/.github/workflows/sonar-fix.yml index 069bf6c..74486c8 100644 --- a/.github/workflows/sonar-fix.yml +++ b/.github/workflows/sonar-fix.yml @@ -344,7 +344,7 @@ jobs: name: "Fix" needs: detect-and-resolve if: needs.detect-and-resolve.outputs.should_run == 'true' - uses: jon-lipsky-sonarsource/sonar-fix/.github/workflows/fix.yml@main + uses: jon-lipsky-sonarsource/sonar-fix/.github/workflows/fix.yml@v1 with: sonar-project-key: ${{ vars.SONAR_PROJECT_KEY }} sonar-org: ${{ vars.SONAR_ORG || vars.COPILOT_MCP_SONAR_ORG }} diff --git a/src/main/java/clipper2/SonarFixProbeV1Claude.java b/src/main/java/clipper2/SonarFixProbeV1Claude.java new file mode 100644 index 0000000..446f477 --- /dev/null +++ b/src/main/java/clipper2/SonarFixProbeV1Claude.java @@ -0,0 +1,8 @@ +package clipper2; + +public class SonarFixProbeV1Claude { + public void run() { + // TODO: remove this probe + int unused = 42; // S1481: unused variable + } +} From cade28ffd8ac914bd1d58a6b311f3f97b26305ef Mon Sep 17 00:00:00 2001 From: "claude[bot]" <41898282+claude[bot]@users.noreply.github.com> Date: Wed, 6 May 2026 18:42:01 +0000 Subject: [PATCH 2/2] fix: resolve SonarQube issues (automated) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fixes: - java:S1135 in src/main/java/clipper2/SonarFixProbeV1Claude.java:5 — removed TODO comment - java:S1854 in src/main/java/clipper2/SonarFixProbeV1Claude.java:6 — removed useless assignment - java:S1481 in src/main/java/clipper2/SonarFixProbeV1Claude.java:6 — removed unused local variable Verified clean via Agentic Analysis (0 new issues in modified files). Note: pre-existing java:S120 (package naming) remains — project-wide issue not introduced by this fix. --- src/main/java/clipper2/SonarFixProbeV1Claude.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/main/java/clipper2/SonarFixProbeV1Claude.java b/src/main/java/clipper2/SonarFixProbeV1Claude.java index 446f477..a7ba8f3 100644 --- a/src/main/java/clipper2/SonarFixProbeV1Claude.java +++ b/src/main/java/clipper2/SonarFixProbeV1Claude.java @@ -2,7 +2,6 @@ public class SonarFixProbeV1Claude { public void run() { - // TODO: remove this probe - int unused = 42; // S1481: unused variable + // Intentionally empty probe method } }