Completion not showing top level functions

[shinayser]

The http library has a put method, but it doesn't show up on the completion window:

If I scroll down the entire completion window, the put method is not even there.
This is the same for other methods:

[shinayser]

Another one for the Completion Overlord @pq !

[pq]

Is this one consistent for you?

In a quick attempt I wasn't able to repro:

Does it reproduce in a file in the same project w/ no other imports?

[shinayser]

I think that there are two differences in your test:

• You are using Flutter, I was using a standalone CLI application
• You are using VsCode (or am I wrong?)

I just tried with a new project and still not working. Please follow my steps:

• On Intellij, click File -> New -> Project...
• Select Dart -> Console Application. Name it and press ok.
• Open the project, and go to the pubspec and add http: any on dependencies.
• Open the file main.dart located on the bin. It containes the main method.
• Add import 'package:http/http.dart'; on the top of the file.
• Try to put, post, get inside the main method.
• Doesn't work.

[pq]

Hmmm. So I am using IntelliJ. Following your steps w/ a sample console app works for me too:

(Dart VM version: 2.8.0-edge.e84bea25df23ea5f149fc808a6423c7500b9cace)

🤔

What Dart SDK are you using?

[shinayser]

Dart VM version: 2.8.0-dev.0.0 (Wed Dec 11 10:31:10 2019 +0100) on "windows_x64"

[shinayser]

Ok, I just updated my sdk. Now I have:

Dart VM version: 2.8.0-dev.8.0 (Thu Feb 6 15:47:55 2020 +0100) on "windows_x64"

And still no completion =(

[shinayser]

I am a stupid person. Sorry for taking your time @pq I just found out what was going wrong...

This stupid guy here added http depency to the environment instead of dependencies:

But that raised one question:

Why it wasn't being auto completed, but could be imported normally? 🤔

[pq]

Amazing! (And a relief since I couldn't repro on windows either! ;))

Why it wasn't being auto completed, but could be imported normally?

I believe http comes in as a transitive dependency of test. You can confirm this by removing the dependency on test, upgrading and you'll see a squiggle on the import:

As it happens we've got a lively thread going about how we can improve diagnostics (and support generally) for transitive dependencies here dart-lang/linter#29. Feel free to chime in!

All of that said, it's also weird that we are not flagging http as an invalid environment in your pubspec. That just seems wrong? (fyi @bwilkerson)

[bwilkerson]

We currently do a fairly limited job of validating the pubspec.yaml file. This would be a good thing to check if we ever enhance that.

[shinayser]

Great! Good to see that my stupidness can be of value to enhance the tooling =)

And thanks @pq for your explanation about the http!

[shinayser]

I created a new thread for trackign this issue. Thanks everytone and sorry for my mistake!

[pq]

No worries. Thanks for the follow-up!