[Bug]: Addon Not Working

[TurtleP]

How are you using the lua-language-server?

Visual Studio Code Extension (sumneko.lua)

Which OS are you using?

Windows

What is the issue affecting?

Annotations, Libraries

Expected Behaviour

The addon's definitions and comments should be displayed.

Actual Behaviour

The library's comments above functions are being displayed.

Reproduction steps

1. Download and add this for an addon:

   • batteries.zip

2. Download batteries and place them inside of libraries/batteries:
   

3. Add require("libraries.batteries"):export() to the top of your lua file

4. Hovering :export() shows the library's documented info and not from the addon

Additional Notes

I might have the addon set up incorrectly, but I am genuinely unsure. I've looked at a few of the addon samples and instructions, but can't seem to entirely figure it out properly. My settings.json for the files I'm testing with looks like the following:

{
    "Lua.workspace.library": [
        "G:/GitHub/Lua/batteries/library/batteries"
    ]
}

Log File

No response

[carsakiller]

Hello 👋

Try G:/GitHub/Lua/batteries/library for the library path

[TurtleP]

Doesn't seem to fix the issue, either.

[carsakiller]

Can you provide more details about how your workspace looks, please?

Your screenshot shows the definition files in the addon. How does the workspace where you are trying to enable them look? Do you have any global settings? Can you provide a screenshot of the documentation that you are currently receiving?

There is a lot that can go wrong, so there is a lot to cover 🙂

[TurtleP]

The screenshot I provided is how my workspace looks. I'm trying to test that the addon is working there separately to get the definitions. I don't think I have any global settings. I'm not even receiving any documentation at all.

[carsakiller]

So you are trying to enable the library while the library workspace is open?

[TurtleP]

Yeah, pretty much. I'm not sure why it doesn't enable even though I ask for it to.

[carsakiller]

Setting workspace.library to G:/GitHub/Lua/batteries/libraries, assuming that points to libraries/ in your screenshot should work, looks like the path was just wrong. You may also need to set diagnostics.libraryFiles.

[TurtleP]

From what I understand, workspace.libraries points to where the definitions of the library, not the library itself. I could be wrong, of course, but if I wanted to point it to the library itself, that path would be G:\GitHub\LOVE\tests\batteries\libraries. As a result, my current settings.json looks like:

{
    "Lua.diagnostics.libraryFiles": "Enable",
    "Lua.workspace.library": [
        "G:/GitHub/Lua/batteries/library",
        "${3rd}/love2d/library"
    ],
    "Lua.runtime.special": {
        "love.filesystem.load": "loadfile"
    }
}

and still no definitions from the stuff pops up.

[carsakiller]

I see, batteries can either be required in pieces or exported entirely into the global scope. I am not sure how this could be documented to support both methods, I only know how to document either a global or module.

[TurtleP]

I would think that just having the documentation set up would be fine, no? Because realistically if you require the library and export globally or in pieces, it should have all the proper definitions when the settings are applied.

require("libraries.batteries"):export() -- should show up as how I documented it
assert:some(...) -- should be documented
("something"):find("thing") -- should be documented

local stringx = require("libraries.batteries.stringx")
stringx.find(...) -- should be documented

[carsakiller]

I was able to get it to work by not using :export():

local assert = require("libraries.batteries.assert")

I don't know how to make the language server understand that :export() instead includes it globally

[TurtleP]

Hm.. interesting. Maybe that's a feature that would be needed to be added?

[TurtleP]

I was able to get it to work by not using :export():

local assert = require("libraries.batteries.assert")

I don't know how to make the language server understand that :export() instead includes it globally

What does your config look like? I tried this and I'm not getting the documentation that I wrote out, only the comments that semi-document from the library itself, still.

according to documentation, this should look like:

{
    "Lua.diagnostics.libraryFiles": "Enable",
    "Lua.workspace.userThirdParty": [
        "G:/GitHub/Lua"
    ],
    "Lua.workspace.library": [
        "G:/GitHub/Lua/batteries/library",
        "${3rd}/love2d/library"
    ],
    "Lua.runtime.special": {
        "love.filesystem.load": "loadfile"
    }
}

but this also doesn't work, see my comment above

[carsakiller]

Hm yeah, looks like I only have the basic documentation, not your custom stuff. I'm not sure how it would work with :export(), but all I can really suggest is to look at the built-in libraries to see if those help.

[TurtleP]

Yeah I tried not using :export() and still only get the basic documentation.

I've been looking at all the other examples as well, but there's really nothing different on how I'm doing this. Maybe somehow the language server is taking the comments from the library as priority? That would be weird, though..