Rewrite KotlinP so it does not use visitors from kotlinx-metadata anymore;

fix bug with missing moduleName.

libraries/kotlinx-metadata/src/kotlinx/metadata/nodes.kt

@@ -1036,7 +1036,7 @@ class KmEffect(
  */
 @ExperimentalContracts
 @OptIn(DeprecatedVisitor::class)
-class KmEffectExpression : KmEffectExpressionVisitor() {
+class  KmEffectExpression : KmEffectExpressionVisitor() {
     /**
      * Effect expression flags, consisting of [Flag.EffectExpression] flags.
      */

libraries/tools/kotlinp/src/org/jetbrains/kotlin/kotlinp/Kotlinp.kt

@@ -5,13 +5,11 @@
 
 package org.jetbrains.kotlin.kotlinp
 
-import kotlinx.metadata.DeprecatedVisitor
 import kotlinx.metadata.InconsistentKotlinMetadataException
 import kotlinx.metadata.jvm.KotlinClassMetadata
 import kotlinx.metadata.jvm.KotlinModuleMetadata
 import java.io.File
 
-@OptIn(DeprecatedVisitor::class)
 class Kotlinp(private val settings: KotlinpSettings) {
     internal fun renderClassFile(classFile: KotlinClassMetadata?): String =
         when (classFile) {

libraries/tools/kotlinp/testData/ContextReceivers.txt

@@ -40,6 +40,8 @@ package {
   context(B)
   public final val h: kotlin/Int
     public final /* non-default */ get
+
+  // module name: test-module
 }
 // META-INF/test-module.kotlin_module
 // ------------------------------------------

libraries/tools/kotlinp/testData/Contracts.txt

@@ -63,6 +63,8 @@ package {
     contract {
       returns(true) implies (p#0 != null)
     }
+
+  // module name: test-module
 }
 // META-INF/test-module.kotlin_module
 // ------------------------------------------

libraries/tools/kotlinp/testData/IntersectionTypeInLambdaLiteralAndDelegatedProperty.txt

@@ -48,6 +48,8 @@ package {
   // local final /* delegated */ var d: kotlin/Any
   //   local final get
   //   local final set
+
+  // module name: test-module
 }
 // IntersectionTypeInLambdaLiteralAndDelegatedPropertyKt$test$1.class
 // ------------------------------------------

libraries/tools/kotlinp/testData/Lambda.txt

@@ -4,6 +4,8 @@ package {
 
   // signature: test()V
   public final fun test(): kotlin/Unit
+
+  // module name: test-module
 }
 // LambdaKt$test$f$1.class
 // ------------------------------------------

libraries/tools/kotlinp/testData/LocalDelegatedProperties.txt

@@ -69,6 +69,8 @@ package {
   // local delegated property #2
   // local final /* delegated */ val local3: kotlin/CharSequence?
   //   local final get
+
+  // module name: test-module
 }
 // META-INF/test-module.kotlin_module
 // ------------------------------------------

libraries/tools/kotlinp/testData/MultiFileClass.txt

@@ -11,6 +11,8 @@ package {
   // field: x:I
   public final const val x: kotlin/Int /* = ... */
     public final get
+
+  // module name: test-module
 }
 // META-INF/test-module.kotlin_module
 // ------------------------------------------

libraries/tools/kotlinp/testData/OptionalAnnotation.txt

@@ -4,6 +4,8 @@ package {
 
   // signature: ok()V
   public final fun ok(): kotlin/Unit
+
+  // module name: test-module
 }
 // META-INF/test-module.kotlin_module
 // ------------------------------------------

libraries/tools/kotlinp/testData/SimplePackage.txt

@@ -14,6 +14,8 @@ package {
   public typealias F<T#0 /* T */, T#1 /* U */> = kotlin/collections/Map<T#0, kotlin/Function1<kotlin/text/StringBuilder^, T#1?>> /* = kotlin/collections/Map<T#0, kotlin/Function1<java/lang/StringBuilder /* = kotlin/text/StringBuilder^ */, T#1?>> */
 
   public typealias G<T#0 /* S */> = F^<kotlin/collections/List<T#0>, kotlin/collections/Set<T#0>> /* = kotlin/collections/Map<kotlin/collections/List<T#0>, kotlin/Function1<java/lang/StringBuilder /* = kotlin/text/StringBuilder^ */, kotlin/collections/Set<T#0>?>> */
+
+  // module name: test-module
 }
 // META-INF/test-module.kotlin_module
 // ------------------------------------------

libraries/tools/kotlinp/testData/SyntheticClass.txt

@@ -4,6 +4,8 @@ package {
 
   // signature: f(Lkotlin/annotation/AnnotationTarget;)Ljava/lang/String;
   public final fun f(a: kotlin/annotation/AnnotationTarget): kotlin/String?
+
+  // module name: test-module
 }
 // SyntheticClassKt$WhenMappings.class
 // ------------------------------------------

libraries/tools/kotlinp/testData/VersionRequirement.txt

@@ -35,6 +35,8 @@ package {
 
   // requires API version 1.1.0 (level=HIDDEN, errorCode=314)
   public typealias Typealias = kotlin/String /* = kotlin/String */
+
+  // module name: test-module
 }
 // META-INF/test-module.kotlin_module
 // ------------------------------------------

libraries/tools/kotlinp/testData/localClasses/AnonymousObject.txt

@@ -4,6 +4,8 @@ package {
 
   // signature: test()Ljava/lang/Runnable;
   public final fun test(): java/lang/Runnable
+
+  // module name: test-module
 }
 // AnonymousObjectKt$test$1.class
 // ------------------------------------------

libraries/tools/kotlinp/testData/localClasses/DeepInnerLocalChain.txt

@@ -4,6 +4,8 @@ package {
 
   // signature: test()V
   public final fun test(): kotlin/Unit
+
+  // module name: test-module
 }
 // DeepInnerLocalChainKt$test$Local.class
 // ------------------------------------------

libraries/tools/kotlinp/testData/localClasses/LocalClassInSignature.txt

@@ -4,6 +4,8 @@ package {
 
   // signature: test()V
   public final fun test(): kotlin/Unit
+
+  // module name: test-module
 }
 // LocalClassInSignatureKt$test$Local.class
 // ------------------------------------------