From f9745528f3ee6c37ac2870d4f42bdbc2a7d25ab1 Mon Sep 17 00:00:00 2001 From: Jon Lipsky Date: Wed, 6 May 2026 11:37:31 -0700 Subject: [PATCH 1/2] test: v1 validation probe for Codex path --- .github/workflows/sonar-fix.yml | 2 +- src/main/java/clipper2/SonarFixProbeV1Codex.java | 8 ++++++++ 2 files changed, 9 insertions(+), 1 deletion(-) create mode 100644 src/main/java/clipper2/SonarFixProbeV1Codex.java 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/SonarFixProbeV1Codex.java b/src/main/java/clipper2/SonarFixProbeV1Codex.java new file mode 100644 index 0000000..dca7aa4 --- /dev/null +++ b/src/main/java/clipper2/SonarFixProbeV1Codex.java @@ -0,0 +1,8 @@ +package clipper2; + +public class SonarFixProbeV1Codex { + public void run() { + // TODO: remove this probe + int unused = 42; // S1481: unused variable + } +} From 014518f4258afa72b7e4c880afc51d21dd3d6798 Mon Sep 17 00:00:00 2001 From: jon-lipsky-sonarsource <190663086+jon-lipsky-sonarsource@users.noreply.github.com> Date: Wed, 6 May 2026 18:42:08 +0000 Subject: [PATCH 2/2] fix: resolve SonarQube issues (automated) Fixes: - java:S1135 in src/main/java/clipper2/SonarFixProbeV1Codex.java:5 - removed TODO probe comment - java:S1854 in src/main/java/clipper2/SonarFixProbeV1Codex.java:6 - removed useless assignment - java:S1481 in src/main/java/clipper2/SonarFixProbeV1Codex.java:6 - removed unused local variable Verified via Agentic Analysis on modified file; requested issues are clean. Existing java:S120 package naming issue remains out of scope. Skipped: - None --- src/main/java/clipper2/SonarFixProbeV1Codex.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/main/java/clipper2/SonarFixProbeV1Codex.java b/src/main/java/clipper2/SonarFixProbeV1Codex.java index dca7aa4..a80cd09 100644 --- a/src/main/java/clipper2/SonarFixProbeV1Codex.java +++ b/src/main/java/clipper2/SonarFixProbeV1Codex.java @@ -2,7 +2,6 @@ public class SonarFixProbeV1Codex { public void run() { - // TODO: remove this probe - int unused = 42; // S1481: unused variable + // Intentionally empty validation probe. } }