Skip to content

Commit 2e2971c

Browse files
dfa1claude
andcommitted
chore(sonar): suppress S106 in CLI and code-gen entry points
The 43 S106 "use a logger instead of System.out/err" smells are all in CLI commands, the TUI terminal drivers, and the proto-gen Main — programs whose stdout/stderr IS the intended output. Suppress with class-level @SuppressWarnings("java:S106") rather than swapping in a logger. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 6a78dbf commit 2e2971c

11 files changed

Lines changed: 11 additions & 0 deletions

File tree

cli/src/main/java/io/github/dfa1/vortex/cli/CountCommand.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import java.nio.file.Files;
77
import java.nio.file.Path;
88

9+
@SuppressWarnings("java:S106") // CLI command: stdout is the intended output channel
910
final class CountCommand {
1011

1112
private CountCommand() {

cli/src/main/java/io/github/dfa1/vortex/cli/FilterCommand.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
import java.util.Arrays;
2626
import java.util.List;
2727

28+
@SuppressWarnings("java:S106") // CLI command: stdout is the intended output channel
2829
final class FilterCommand {
2930

3031
private FilterCommand() {

cli/src/main/java/io/github/dfa1/vortex/cli/InspectCommand.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
import java.nio.file.Files;
1212
import java.nio.file.Path;
1313

14+
@SuppressWarnings("java:S106") // CLI command: stdout is the intended output channel
1415
final class InspectCommand {
1516

1617
private InspectCommand() {

cli/src/main/java/io/github/dfa1/vortex/cli/SchemaCommand.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
import java.util.List;
1111
import java.util.Locale;
1212

13+
@SuppressWarnings("java:S106") // CLI command: stdout is the intended output channel
1314
final class SchemaCommand {
1415

1516
private SchemaCommand() {

cli/src/main/java/io/github/dfa1/vortex/cli/SelectCommand.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
import java.util.Arrays;
1313
import java.util.List;
1414

15+
@SuppressWarnings("java:S106") // CLI command: stdout is the intended output channel
1516
final class SelectCommand {
1617

1718
private SelectCommand() {

cli/src/main/java/io/github/dfa1/vortex/cli/StatsCommand.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
import java.util.List;
1111
import java.util.Map;
1212

13+
@SuppressWarnings("java:S106") // CLI command: stdout is the intended output channel
1314
final class StatsCommand {
1415

1516
private StatsCommand() {

cli/src/main/java/io/github/dfa1/vortex/cli/TuiCommand.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import java.io.PrintStream;
1010
import java.util.Optional;
1111

12+
@SuppressWarnings("java:S106") // CLI command: stdout is the intended output channel
1213
final class TuiCommand {
1314

1415
private TuiCommand() {

cli/src/main/java/io/github/dfa1/vortex/cli/VortexCli.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
/// Entry point for the Vortex command-line tool.
88
///
99
/// Exit codes: see [ExitStatus].
10+
@SuppressWarnings("java:S106") // CLI entry point: stdout is the intended output channel
1011
public final class VortexCli {
1112

1213
static {

cli/src/main/java/io/github/dfa1/vortex/cli/tui/term/PosixTerminal.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
/// On entry: saves the original `termios`, switches to alt screen, hides
2222
/// the cursor. On [#close()]: restores everything, even on exceptions, via a
2323
/// shutdown hook that fires if the caller skips try-with-resources.
24+
@SuppressWarnings("java:S106") // terminal driver: writes control sequences straight to stdout
2425
public final class PosixTerminal implements Terminal {
2526

2627
private static final int STDIN_FD = 0;

cli/src/main/java/io/github/dfa1/vortex/cli/tui/term/WindowsTerminal.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
///
2323
/// `GetConsoleScreenBufferInfo` drives [#size()]; we report the visible
2424
/// window rect, not the scrollback buffer.
25+
@SuppressWarnings("java:S106") // terminal driver: writes control sequences straight to stdout
2526
public final class WindowsTerminal implements Terminal {
2627

2728
private static final long STD_INPUT_HANDLE = -10L;

0 commit comments

Comments
 (0)