LuaLS / LuaLS/lua-language-server

Returning a value as a class doesnt set the fields

オープン
#2,321 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
Lua
スター
4.4k
フォーク
442
PR マージ指標
30日以内にマージされた PR はありません

説明

### 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, Type Checking

### Expected Behaviour

`value` be member of `MyClass`

### Actual Behaviour

![grafik](https://github.com/LuaLS/lua-language-server/assets/15840247/d0ede492-0113-4d95-beb5-391a77508f8f)
`value` is not a member

### Reproduction steps

```lua
local function myConstructor()
---@class MyClass
return { value = true }
end

local o = myConstructor()
print(o.value)
```

### Additional Notes

This should also work with a `setmetatable` variant:

```lua
---@class MyClass
local MyClass = {}
MyClass.__index = MyClass

local function myConstructor()
---@class MyClass
return setmetatable({ value = true }, MyClass)
end

local o = myConstructor()
print(o.value)
```

It does work with this workarround:

```lua
local function myConstructor()
---@class MyClass
local o = setmetatable({ value = true }, MyClass)
return o
end
```

### Log File

_No response_

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、issue にある 2 つのコンストラクターの例を再現し、language server が返されるテーブル上の @class アノテーションをどのように解釈するかを追跡します。これを、ローカル変数を使った動作する回避策と比較し、直接の return と setmetatable による return の両方を対象とする回帰テストを追加します。型チェック中に値が MyClass のメンバーとして認識されれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
lua
領域
devtools
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。