Fix #5237: Do not list <xyz> in scala packege completions

Author: nicolasstucki

compiler/src/dotty/tools/dotc/interactive/Completion.scala

@@ -140,7 +140,7 @@ object Completion {
     private[this] val completions = new RenameAwareScope
 
     /**
-     * Return the list of symbols that shoudl be included in completion results.
+     * Return the list of symbols that should be included in completion results.
      *
      * If several symbols share the same name, the type symbols appear before term symbols inside
      * the same `Completion`.
@@ -252,7 +252,10 @@ object Completion {
       (
            (mode.is(Mode.Term) && sym.isTerm)
         || (mode.is(Mode.Type) && (sym.isType || sym.isStable))
-      )
+      ) &&
+      (sym ne defn.RepeatedParamClass) &&
+      (sym ne defn.ByNameParamClass2x) &&
+      (sym ne defn.EqualsPatternClass)
 
     /**
      * Find all the members of `site` that are accessible and which should be included in `info`.

compiler/test/dotty/tools/repl/TabcompleteTests.scala

@@ -90,4 +90,10 @@ class TabcompleteTests extends ReplTest {
       val expected = List("Renamed")
       assertEquals(expected, tabComplete("val foo: Rena"))
     }
+
+  @Test def importScala = fromInitialState { implicit s =>
+    val comp = tabComplete("import scala.")
+    // check that there are no special symbols leaked: <byname>, <special-ops>, ...
+    assertEquals(comp.find(_.startsWith("<")), None)
+  }
 }