`createLockFileForStandardQuery` always creates temp lock because `packPacklist` doesn't return lock files
还没有人认领这个 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 内容生成。
描述
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 天 18 小时
- 30 天内合并 PR
- 37
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
github/vscode-codeql 的其他 Issue
-
bug enhancement VSCode
难度 2/5 1-3 小时 新手友好度 68/100
github/vscode-codeql#592 · 2 条评论 ·
-
enhancement VSCode
难度 3/5 1-2 天 新手友好度 55/100
github/vscode-codeql#4432 · 1 条评论 ·
-
bug VSCode
难度 3/5 1-2 天 新手友好度 52/100
github/vscode-codeql#4356 ·
-
bug VSCode
难度 3/5 1-2 天 新手友好度 64/100
github/vscode-codeql#4339 · 1 条评论 ·
-
CLI test failure 未关闭cli-test-failure
难度 3/5 1-2 天 新手友好度 32/100
github/vscode-codeql#4330 · 4 条评论 ·
查看 github/vscode-codeql 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 65/100
-
难度 2/5 1-3 小时 新手友好度 75/100
-
bug v2
难度 2/5 1-3 小时 新手友好度 75/100
modelcontextprotocol/inspector#2458 · 1 条评论 ·
-
难度 1/5 1 小时以内 新手友好度 75/100
railmapgen/rmp-gallery#4068 ·
-
Mend: dependency security vulnerability status: needs triage 🕵️♀️
难度 2/5 1-3 小时 新手友好度 70/100
carbon-design-system/ibm-products#9907 ·