[6.4.0] Include name in repr of exported rules (bazelbuild#19229)

For an exported rule `foo_library`, `repr(foo_library)` now evaluates to
`<rule foo_library>`, not `<rule>`, matching the behavior of native
rules more closely.

Fixes bazelbuild#17483

Closes bazelbuild#18392.

Commit
bazelbuild@002490b

PiperOrigin-RevId: 538458291
Change-Id: I331955655756a3c235be0a93f1394716ddf9849d

Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>

src/main/java/com/google/devtools/build/lib/analysis/starlark/StarlarkRuleClassFunctions.java

@@ -977,7 +977,11 @@ public boolean isExported() {
 
     @Override
     public void repr(Printer printer) {
-      printer.append("<rule>");
+      if (isExported()) {
+        printer.append("<rule ").append(getRuleClass().getName()).append(">");
+      } else {
+        printer.append("<rule>");
+      }
     }
 
     @Override

src/test/java/com/google/devtools/build/lib/starlark/StarlarkStringRepresentationsTest.java

@@ -268,7 +268,7 @@ public void testStringRepresentations_functions() throws Exception {
   @Test
   public void testStringRepresentations_rules() throws Exception {
     setBuildLanguageOptions("--experimental_builtins_injection_override=+cc_library");
-    assertStringRepresentation("native.cc_library", "<rule>");
+    assertStringRepresentation("native.cc_library", "<rule cc_library>");
     assertStringRepresentation("def f(): pass", "rule(implementation=f)", "<rule>");
   }