`createLockFileForStandardQuery` always creates temp lock because `packPacklist` doesn't return lock files

未關閉 適合新手
#4,471 1 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視

還沒有人認領這個 Issue。

評估

難度
2/5
預估耗時
1-3 小時
新手友好度
68/100
Issue 類型
缺陷
描述清晰度
基本清楚
活躍度
冷清
技術堆疊
typescript, vscode
領域
devtools

研究方向

從 src/local-queries/standard-queries.ts 中的 createLockFileForStandardQuery 開始,追蹤 cli.packPacklist(queryPath, false) 的結果以及對 lockFilePath 的檢查。使用包含 codeql-pack.lock.yml 的 pack 驗證行為,然後確保能夠偵測到現有的 lock 檔案,而不是一律建立暫存檔案。

由索引模型根據 Issue 內容生成。

描述

bug VSCode

Describe the bug
In src/local-queries/standard-queries.ts , createLockFileForStandardQuery tries to detect whether a pack already has a codeql-pack.lock.yml by checking the output of cli.packPacklist():

const packContents = await cli.packPacklist(queryPath, false);
const lockFilePath = packContents.find((p) =>
    QLPACK_LOCK_FILENAMES.includes(basename(p)),
  );

However, codeql pack packlist doesn't return codeql-pack.lock.yml. This means lockFilePath is always undefined, so the check for an existing lock file can never succeed.

主要語言
TypeScript
星號
539
分支
240
平均合併
2 天 6 小時
30 天內合併 PR
29

貢獻指南

開啟貢獻指南

從這裡開始

  1. 先讀完整個 Issue,再讀專案的貢獻指南。
  2. 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
  3. Fork 儲存庫,在一個分支上完成修改。
  4. 送出 Pull Request,並在描述裡引用這個 Issue 編號。

github/vscode-codeql 的其他 Issue

查看 github/vscode-codeql 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。