refactor(next): comprehensive SonarLint cleanup and zero-defect quality pass (#578) - #583
Merged
Conversation
…ty pass (#578) - Clean up temporal literals (SimpleDate, SimpleDateTime, SimpleDuration, SimpleTime) by removing unused 3-arg constructors and S1172 suppressions - Remove all 19 non-standard NullableProblems suppressions across 12 files - Add missing @OverRide annotations in Literal (S1161) - Precompile regex pattern in RDFXMLStatementEmitter (S9142) - Simplify AST visitor lambdas in DatasetClauseAst and DescribeQueryAst (S1602) - Remove redundant self-assignment in PrefixDeclarationAst (S4165) - Fix Javadoc comment syntax in RDFaParser (S9355) - Remove empty package data/impl/io/jsonld (S4032) - Align exception hierarchy depth suppressions with S110 in query/api/exception - Add SonarLint extraction and headless LSP CLI tools in tools/ - Verify 100% zero defects on SonarLint across core.next, 10/10 boundary tests, and 0 W3C regressions
|
Test Results 424 files 424 suites 20s ⏱️ Results for commit 7fbe45a. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #578
Summary
Comprehensive SonarLint cleanup and zero-defect quality pass across the entire
fr.inria.corese.core.nextmodule.Changes
fr.inria.corese.core.nextusing live headless SonarLint LSP runner (tools/sonarlint_extract.py).@Override, unused constructors, container membership regex precompilation, lambda simplification, redundant self-assignments, Javadoc comment syntax).@SuppressWarnings("NullableProblems")across 12 files.data/impl/io/jsonld.tools/sonarlint_extract.pyandtools/extract_sonar_alerts.pyto automate Sonar extraction via IDE and headless LSP.CoreseGraphStatementStore.java.