Uncomment filter

Author: nojaf

analysis/src/CompletionBackEnd.ml

@@ -1124,7 +1124,7 @@ and getCompletionsForContextPath ~debug ~full ~opens ~rawOpens ~pos ~env ~exact
               ~pos ~scope ~debug ~prefix ~env ~rawOpens ~full completionPath
             |> TypeUtils.filterPipeableFunctions ~env ~full ~synthetic
                  ~targetTypeId:mainTypeId
-            (* |> List.filter (fun (c : Completion.t) ->
+            |> List.filter (fun (c : Completion.t) ->
                    (* If we're completing from the current module then we need to care about scope.
                       This is automatically taken care of in other cases. *)
                    if isFromCurrentModule then
@@ -1138,7 +1138,7 @@ and getCompletionsForContextPath ~debug ~full ~opens ~rawOpens ~pos ~env ~exact
                               | _ -> false)
                        |> Option.is_some
                      | _ -> false
-                   else true) *)
+                   else true)
         in
         (* Extra completions can be drawn from the @editor.completeFrom attribute. Here we
            find and add those completions as well. *)

tests/analysis_tests/tests/src/expected/DotPipeCompletionSpec.res.txt

@@ -275,18 +275,6 @@ Path DotPipeCompletionSpec.SomeOtherModule.
     "tags": [],
     "detail": "string",
     "documentation": {"kind": "markdown", "value": "```rescript\nnname: string\n```\n\n```rescript\ntype typeOutsideModule = {nname: string}\n```"}
-  }, {
-    "label": "->outOfScope",
-    "kind": 12,
-    "tags": [],
-    "detail": "typeOutsideModule => typeOutsideModule",
-    "documentation": null,
-    "sortText": "outOfScope",
-    "insertText": "->outOfScope",
-    "additionalTextEdits": [{
-      "range": {"start": {"line": 80, "character": 6}, "end": {"line": 80, "character": 7}},
-      "newText": ""
-      }]
   }, {
     "label": "->doWithTypeOutsideModule",
     "kind": 12,
@@ -299,18 +287,6 @@ Path DotPipeCompletionSpec.SomeOtherModule.
       "range": {"start": {"line": 80, "character": 6}, "end": {"line": 80, "character": 7}},
       "newText": ""
       }]
-  }, {
-    "label": "->cc",
-    "kind": 12,
-    "tags": [],
-    "detail": "typeOutsideModule => typeOutsideModule",
-    "documentation": null,
-    "sortText": "cc",
-    "insertText": "->cc",
-    "additionalTextEdits": [{
-      "range": {"start": {"line": 80, "character": 6}, "end": {"line": 80, "character": 7}},
-      "newText": ""
-      }]
   }, {
     "label": "->SomeOtherModule.getNName",
     "kind": 12,
@@ -513,18 +489,6 @@ Path DotPipeCompletionSpec.SomeOtherModule.
     "tags": [],
     "detail": "string",
     "documentation": {"kind": "markdown", "value": "```rescript\nnname: string\n```\n\n```rescript\ntype typeOutsideModule = {nname: string}\n```"}
-  }, {
-    "label": "->outOfScope",
-    "kind": 12,
-    "tags": [],
-    "detail": "typeOutsideModule => typeOutsideModule",
-    "documentation": null,
-    "sortText": "outOfScope",
-    "insertText": "->outOfScope",
-    "additionalTextEdits": [{
-      "range": {"start": {"line": 101, "character": 6}, "end": {"line": 101, "character": 7}},
-      "newText": ""
-      }]
   }, {
     "label": "->doWithTypeOutsideModule",
     "kind": 12,
@@ -537,18 +501,6 @@ Path DotPipeCompletionSpec.SomeOtherModule.
       "range": {"start": {"line": 101, "character": 6}, "end": {"line": 101, "character": 7}},
       "newText": ""
       }]
-  }, {
-    "label": "->cc",
-    "kind": 12,
-    "tags": [],
-    "detail": "typeOutsideModule => typeOutsideModule",
-    "documentation": null,
-    "sortText": "cc",
-    "insertText": "->cc",
-    "additionalTextEdits": [{
-      "range": {"start": {"line": 101, "character": 6}, "end": {"line": 101, "character": 7}},
-      "newText": ""
-      }]
   }, {
     "label": "->SomeOtherModule.getNName",
     "kind": 12,