Merge pull request #15565 from ckipp01/bridgeChanges

Author: dwijnand

project/Dependencies.scala

@@ -25,6 +25,6 @@ object Dependencies {
     "com.vladsch.flexmark" % "flexmark-ext-yaml-front-matter" % flexmarkVersion,
   )
 
-  val newCompilerInterface = "org.scala-sbt" % "compiler-interface" % "1.4.3"
+  val newCompilerInterface = "org.scala-sbt" % "compiler-interface" % "1.7.1"
   val oldCompilerInterface = "org.scala-sbt" % "compiler-interface" % "1.3.5"
 }

sbt-bridge/src/dotty/tools/xsbt/DelegatingReporter.java

@@ -39,12 +39,13 @@ public void doReport(Diagnostic dia, Context ctx) {
     StringBuilder rendered = new StringBuilder();
     rendered.append(messageAndPos(dia, ctx));
     Message message = dia.msg();
+    String diagnosticCode = String.valueOf(message.errorId().errorNumber());
     boolean shouldExplain = Diagnostic.shouldExplain(dia, ctx);
     if (shouldExplain && !message.explanation().isEmpty()) {
       rendered.append(explanation(message, ctx));
     }
 
-    delegate.log(new Problem(position, message.msg(), severity, rendered.toString()));
+    delegate.log(new Problem(position, message.msg(), severity, rendered.toString(), diagnosticCode));
   }
 
   private static Severity severityOf(int level) {

sbt-bridge/src/dotty/tools/xsbt/DiagnosticCode.java

@@ -0,0 +1,23 @@
+package dotty.tools.xsbt;
+
+import java.util.Optional;
+
+final public class DiagnosticCode implements xsbti.DiagnosticCode {
+	private final String _code;
+	private final Optional<String> _explanation;
+
+	public DiagnosticCode(String code, Optional<String> explanation) {
+		super();
+		this._code = code;
+		this._explanation = explanation;
+	}
+
+	public String code() {
+		return _code;
+	}
+
+	public Optional<String> explanation() {
+		return _explanation;
+	}
+
+}

sbt-bridge/src/dotty/tools/xsbt/Problem.java

@@ -9,13 +9,15 @@ final public class Problem implements xsbti.Problem {
   private final String _message;
   private final Severity _severity;
   private final Optional<String> _rendered;
+  private final String _diagnosticCode;
 
-  public Problem(Position position, String message, Severity severity, String rendered) {
+  public Problem(Position position, String message, Severity severity, String rendered, String diagnosticCode) {
     super();
     this._position = position;
     this._message = message;
     this._severity = severity;
     this._rendered = Optional.of(rendered);
+    this._diagnosticCode = diagnosticCode;
   }
 
   public String category() {
@@ -38,8 +40,17 @@ public Optional<String> rendered() {
     return _rendered;
   }
 
+  public Optional<xsbti.DiagnosticCode> diagnosticCode() {
+    // NOTE: It's important for compatibility that we only construct a
+    // DiagnosticCode here to maintain compatibility with older versions of
+    // zinc while using this newer version of the compiler. If we would
+    // contstruct it earlier, you'd end up with ClassNotFoundExceptions for
+    // DiagnosticCode.
+    return Optional.of(new DiagnosticCode(_diagnosticCode, Optional.empty()));
+  }
+
   @Override
   public String toString() {
-    return "Problem(" + _position + ", " + _message + ", " + _severity + ", " + _rendered + ")";
+    return "Problem(" + _position + ", " + _message + ", " + _severity + ", " + _rendered + ", " + _diagnosticCode + ")";
   }
 }

sbt-test/compilerReporter/simple/project/Reporter.scala

@@ -36,6 +36,11 @@ object Reporter {
       assert(line.isPresent() == true)
       assert(line.get() == 9)
 
+      val diagnosticCode = mainProblem.diagnosticCode()
+      assert(diagnosticCode.isPresent() == true)
+      val code = diagnosticCode.get()
+      assert(diagnosticCode.get().code() == "6")
+
       val pointer = mainProblem.position().pointer()
       assert(pointer.isPresent() == true)
       assert(pointer.get() == 10)