Skip to content

Code completion triggers in doc comments #5507

Closed

Description

@DanTup

Not sure what's happened here, but when I type full-stops in doc comments, it now triggers code completion (which also contains results) where I don't think it used to.

Image

Activity

  1. added this to the v3.112.0 milestone on May 7, 2025
  2. FMorschel commented on May 7, 2025

    @FMorschel
    Contributor

    Actually, I can trigger auto-completion anywhere, no need for . or anything. The only requirement is having one [] somewhere. It can even be empty.

  3. FMorschel commented on May 8, 2025

    @FMorschel
    Contributor

    I'd like to take a look at this if you don't mind.

  4. FMorschel commented on May 8, 2025

    @FMorschel
    Contributor

    I found the problem already, I'll wrap this up and post the CL here.

  5. DanTup commented on May 8, 2025

    @DanTup
    MemberAuthor

    @FMorschel great, thanks! I hadn't even had chance to figure out if this was in Dart-Code or the server.. sounds like you're way ahead :-)

  6. added
    in editorRelates to code editing or language features
    on May 8, 2025
  7. FMorschel commented on May 8, 2025

    @FMorschel
    Contributor

    This is the original CL that caused the regression https://dart-review.googlesource.com/c/sdk/+/411380. The correct resolution for that problem was worked on at https://dart-review.googlesource.com/c/sdk/+/419422. It was easy for me because I was the one who introduced these tests, and both CLs were mine. I've been working on comment references and completions for a bit.

    This is the new CL that fixes this issue: https://dart-review.googlesource.com/c/sdk/+/427460

  8. DanTup commented on May 12, 2025

    @DanTup
    MemberAuthor

    @FMorschel thanks!

  9. added
    in lsp/analysis serverSomething to be fixed in the Dart analysis server
    relies on sdk changesSomething that requires changes in the Dart/Flutter SDK to ship before it will become available
    on May 12, 2025
  10. FMorschel commented on May 14, 2025

    @FMorschel
    Contributor

    The above CL has landed. This can be closed!

  11. DanTup commented on May 14, 2025

    @DanTup
    MemberAuthor

    Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

in editorRelates to code editing or language featuresin lsp/analysis serverSomething to be fixed in the Dart analysis serveris bugrelies on sdk changesSomething that requires changes in the Dart/Flutter SDK to ship before it will become available

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions