Backport "Fix undercompilation upon ctor change" to LTS (#20994)

Backports #19911 to the LTS branch.

PR submitted by the release tooling.
[skip ci]

Author: WojciechMazur

Diff for: compiler/src/dotty/tools/dotc/sbt/package.scala

@@ -4,17 +4,23 @@ import dotty.tools.dotc.core.Contexts.Context
 import dotty.tools.dotc.core.Symbols.Symbol
 import dotty.tools.dotc.core.NameOps.stripModuleClassSuffix
 import dotty.tools.dotc.core.Names.Name
+import dotty.tools.dotc.core.Names.termName
 
 inline val TermNameHash = 1987 // 300th prime
 inline val TypeNameHash = 1993 // 301st prime
 inline val InlineParamHash = 1997 // 302nd prime
 
 extension (sym: Symbol)
 
-  def constructorName(using Context) =
-    sym.owner.fullName ++ ";init;"
-
-  /** Mangle a JVM symbol name in a format better suited for internal uses by sbt. */
-  def zincMangledName(using Context): Name =
-    if (sym.isConstructor) constructorName
-    else sym.name.stripModuleClassSuffix
+  /** Mangle a JVM symbol name in a format better suited for internal uses by sbt.
+   *  WARNING: output must not be written to TASTy, as it is not a valid TASTy name.
+   */
+  private[sbt] def zincMangledName(using Context): Name =
+    if sym.isConstructor then
+      // TODO: ideally we should avoid unnecessarily caching these Zinc specific
+      // names in the global chars array. But we would need to restructure
+      // ExtractDependencies caches to avoid expensive `toString` on
+      // each member reference.
+      termName(sym.owner.fullName.mangledString.replace(".", ";").nn ++ ";init;")
+    else
+      sym.name.stripModuleClassSuffix

Diff for: sbt-bridge/test/xsbt/ExtractUsedNamesSpecification.scala

@@ -306,7 +306,7 @@ class ExtractUsedNamesSpecification {
     // All classes extend Object
     "Object",
     // All classes have a default constructor called <init>
-    "java.lang.Object;init;",
+    "java;lang;Object;init;",
     // the return type of the default constructor is Unit
     "Unit"
   )

Diff for: sbt-test/source-dependencies/constructors/A.scala

@@ -1 +1,3 @@
+package example
+
 class A(a: Int)

Diff for: sbt-test/source-dependencies/constructors/B.scala

@@ -1 +1,3 @@
+package example
+
 class B { val y = new A(2) }

Diff for: sbt-test/source-dependencies/constructors/changes/A2.scala

@@ -1 +1,3 @@
+package example
+
 class A(a: String)

Diff for: sbt-test/source-dependencies/constructors/changes/B2.scala

@@ -1 +1,3 @@
+package example
+
 class B { val y = new A("a") }