refactor: narrow lookahead cleanup to grammar changes

Signed-off-by: 付典 <fudianchn@gmail.com>

Author: fudianchn

.github/workflows/ci.yml

@@ -45,28 +45,6 @@ jobs:
       - name: Check for grammar ambiguities
         run: ./gradlew checkGrammarAmbiguity --no-daemon
 
-      - name: Restore and save lookahead facts
-        uses: actions/cache@v5
-        with:
-          path: build/lookahead-cache/
-          key: lookahead-facts-${{ runner.os }}-${{ hashFiles('src/test/java/net/sf/jsqlparser/parser/audit/**', 'build.gradle', 'gradle/wrapper/gradle-wrapper.properties') }}-${{ github.sha }}
-          restore-keys: |
-            lookahead-facts-${{ runner.os }}-${{ hashFiles('src/test/java/net/sf/jsqlparser/parser/audit/**', 'build.gradle', 'gradle/wrapper/gradle-wrapper.properties') }}-
-
-      - name: Report changed lookahead contexts
-        env:
-          LOOKAHEAD_BASE: ${{ github.event.pull_request.base.sha || github.event.before || github.sha }}
-        run: ./gradlew lookaheadAudit --no-daemon -PlookaheadBase="$LOOKAHEAD_BASE"
-
-      - name: Upload lookahead review report
-        if: always()
-        uses: actions/upload-artifact@v5
-        with:
-          name: lookahead-review
-          path: build/reports/lookahead/
-          retention-days: 7
-          if-no-files-found: ignore
-
       - name: Upload grammar warning report
         if: failure()
         uses: actions/upload-artifact@v5
@@ -238,4 +216,4 @@ jobs:
 
       - name: Deploy to GitHub Pages
         id: deployment
-        uses: actions/deploy-pages@v5
+        uses: actions/deploy-pages@v5

README.md

@@ -142,10 +142,17 @@ and missing syntax gets added on demand — [open an issue](https://github.com/J
 | **ksqlDB windows** | JOIN `WITHIN`, window `GRACE PERIOD`, and `EMIT CHANGES`/`FINAL` |
 | **ClickHouse column selection** | `COLUMNS('regexp')` select items with chained `APPLY`, `EXCEPT`, and `REPLACE` transformers |
 | **DML** | `INSERT` · `UPDATE` · `UPSERT` · `MERGE` · `DELETE` · `TRUNCATE TABLE` |
+| **Oracle RETURNING** | Named output binds (`INTO :name`), JDBC output parameters (`INTO ?`), and `BULK COLLECT INTO` |
 | **DDL** | `CREATE …` · `ALTER …` · `DROP …` |
+| **Oracle vector types** | `VECTOR(*, *)` · `VECTOR(*, FLOAT32)` · `VECTOR(3, *)`, including dense and sparse storage formats |
 | **PostgreSQL ownership** | `DROP OWNED` · `REASSIGN OWNED` |
 | **MySQL table maintenance** | `ANALYZE` · `CHECK` · `OPTIMIZE` · `REPAIR TABLE` |
+| **MySQL account management** | `CREATE`/`ALTER`/`RENAME USER` · `CREATE ROLE` · `SET ROLE` · `SET DEFAULT ROLE` · `SET PASSWORD` |
+| **Tablespaces** | MySQL and PostgreSQL `CREATE TABLESPACE` · `ALTER TABLESPACE` |
 | **PostgreSQL RLS** | `CREATE POLICY` · `ALTER TABLE … ENABLE`/`DISABLE`/`FORCE`/`NO FORCE ROW LEVEL SECURITY` |
+| **Routine changes** | `ALTER FUNCTION` · `ALTER PROCEDURE` · `ALTER ROUTINE` · PostgreSQL `ALTER TRIGGER` |
+| **MySQL metadata statements** | `SHOW [FULL] COLUMNS`/`FIELDS` · `SHOW TABLE STATUS` · `SHOW CREATE TABLE`/`VIEW`/`PROCEDURE`/`FUNCTION`/`TRIGGER`/`EVENT`/`DATABASE`/`USER` |
+| **PostgreSQL administration** | `CREATE`/`ALTER EVENT TRIGGER` · `REINDEX` · `SECURITY LABEL` |
 | **Informix constraints** | `ALTER TABLE … ADD CONSTRAINT` with trailing constraint names for primary, unique, foreign and check constraints; enable with `parser.withDialect(Dialect.INFORMIX)` |
 | **Salesforce SOQL** | `INCLUDES` · `EXCLUDES` |
 

build.gradle

@@ -265,7 +265,7 @@ tasks.register('xmldoc', Javadoc) {
 }
 
 test {
-    environment 'EXPORT_TEST_TO_FILE', 'False'
+    environment = [ 'EXPORT_TEST_TO_FILE': 'False' ]
     useJUnitPlatform()
 
     // set heap size for the test JVM(s)
@@ -846,24 +846,3 @@ tasks.register('checkGrammarAmbiguity') {
 tasks.named('check') {
     dependsOn('checkGrammarAmbiguity')
 }
-
-// Report base/working-tree context changes without making historical lookahead debt a new gate.
-// The analyzer lives in test sources: no audit classes or dependencies enter the library JAR.
-tasks.register('lookaheadAudit', JavaExec) {
-    group = 'verification'
-    description = 'Reports changed lookahead decisions and stale evidence against a Git base.'
-    dependsOn(testClasses)
-    classpath = sourceSets.test.runtimeClasspath
-    mainClass = 'net.sf.jsqlparser.parser.audit.LookaheadAudit'
-    javaLauncher = javaToolchains.launcherFor(java.toolchain)
-    jvmArgs '-Xss4m', '-Xmx1g', '-Dfile.encoding=UTF-8'
-    workingDir = projectDir
-    doFirst {
-        args = [projectDir.absolutePath,
-                providers.gradleProperty('lookaheadBase').getOrElse('HEAD'),
-                layout.buildDirectory.dir('reports/lookahead').get().asFile.absolutePath,
-                file('config/lookahead-evidence.tsv').absolutePath,
-                providers.gradleProperty('lookaheadFailOnFindings').getOrElse('false'),
-                providers.gradleProperty('lookaheadFull').getOrElse('false')]
-    }
-}

config/lookahead-evidence.tsv

@@ -18,12 +18,16 @@ public class DateUnitExpression extends ASTNodeAccessImpl implements Expression
 
     private final DateUnit type;
 
+    private final String text;
+
     public DateUnitExpression(DateUnit type) {
         this.type = Objects.requireNonNull(type);
+        this.text = type.toString();
     }
 
     public DateUnitExpression(String DateUnitStr) {
         this.type = Objects.requireNonNull(DateUnit.from(DateUnitStr));
+        this.text = DateUnitStr;
     }
 
     public DateUnit getType() {
@@ -38,7 +42,7 @@ public <T, S> T accept(ExpressionVisitor<T> expressionVisitor, S context) {
 
     @Override
     public String toString() {
-        return type.toString();
+        return text;
     }
 
     public enum DateUnit {

src/main/java/net/sf/jsqlparser/expression/DateUnitExpression.java

@@ -436,12 +436,11 @@ public StringBuilder appendObject(StringBuilder builder) {
     @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.NPathComplexity"})
     public StringBuilder appendObject(StringBuilder builder,
             Consumer<Expression> expressionPrinter) {
-        builder.append("JSON_OBJECT( ");
+        builder.append("JSON_OBJECT(");
+        int start = builder.length();
         int i = 0;
         for (JsonKeyValuePair keyValuePair : keyValuePairs) {
-            if (i > 0) {
-                builder.append(", ");
-            }
+            builder.append(i > 0 ? ", " : " ");
             keyValuePair.appendTo(builder, expressionPrinter);
             i++;
         }
@@ -453,7 +452,8 @@ public StringBuilder appendObject(StringBuilder builder,
         appendUniqueKeys(builder);
         appendReturningClause(builder, true);
 
-        builder.append(" ) ");
+        // An empty argument list renders as JSON_OBJECT()
+        builder.append(builder.length() > start ? " )" : ")");
 
         return builder;
     }
@@ -495,20 +495,18 @@ public StringBuilder appendArray(StringBuilder builder) {
     @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.NPathComplexity"})
     public StringBuilder appendArray(StringBuilder builder,
             Consumer<Expression> expressionPrinter) {
-        builder.append("JSON_ARRAY( ");
+        builder.append("JSON_ARRAY(");
         int i = 0;
 
         for (JsonFunctionExpression expr : expressions) {
-            if (i > 0) {
-                builder.append(", ");
-            }
+            builder.append(i > 0 ? ", " : " ");
             expr.appendTo(builder, expressionPrinter);
             i++;
         }
 
         appendOnNullType(builder);
         appendReturningClause(builder, true);
-        builder.append(") ");
+        builder.append(')');
 
         return builder;
     }