【架构优化】从 runtime.installNPMCLI 中拆出 npm 参数拼装 npmCLIInstallArgs 与安装结果启用 activateNPMStage
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 30/100
Rechercherichtung
Start in internal/runtime/cli_ondemand.go, where installNPMCLI mixes directory setup, npm argument assembly and stage activation. Check open pull request #487 first, since work on this split may already be under way. Done means npmCLIInstallArgs and activateNPMStage exist as private helpers with unit tests, installNPMCLI keeps its signature, and go test ./internal/runtime/ passes.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
现状问题
internal/runtime/cli_ondemand.go 的 installNPMCLI 约 45 行，把三件事写在一个函数里：
- 准备安装目录与临时 stage 目录；
- 拼装
npm install参数（--engine-strict/--engine-strict=false依RelaxEngines切换）并执行； - 校验 stage 内可执行文件、替换
current目录、返回最终路径。
参数拼装是纯逻辑，却只能通过伪造 npm 可执行文件的集成测试间接覆盖；启用安装结果的逻辑也无法单独测试。
重构方案
- 拆出纯函数
npmCLIInstallArgs(stage string, spec onDemandCLI) []string，负责参数拼装； - 拆出
activateNPMStage(parent, stage, binary string) (string, error)，负责校验可执行文件、替换current并返回路径； installNPMCLI只保留目录准备、执行 npm 与调用上面两个函数，签名与调用方不变；参数顺序、报错文字逐字保持一致。
影响范围
仅 internal/runtime/cli_ondemand.go，包内私有函数，不影响对外接口和行为。
验证方式
- 新增单元测试覆盖
npmCLIInstallArgs（严格/放宽 engines 两种）与activateNPMStage（成功替换旧 current、缺可执行文件报错）； - 现有
cli_ondemand_test.go集成测试保持通过； gofmt、go vet ./internal/runtime/、go test ./internal/runtime/。
- Vorherrschende Sprache
- Go
- Sterne
- 141
- Forks
- 28
- Ø Merge
- 1 T. 15 Std.
- Gemergte PRs (30 T.)
- 113
Entwicklungsumgebung
- Enthält ein Dockerfile oder eine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Tencent/LoopForge
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag
-
[Bug]: 官方镜像未安装 wecom-cli，企业微信通知报「通知服务未安装 wecom-cli」Evtl. vergeben @newexploreor hat das vor 1 Tag übernommen. Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Tencent/LoopForge#156 · 1 Kommentar · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
【架构优化】合并 gitwork/workbranch.go 中 push/pull 重复的默认提交身份注入为 withFallbackIdentityEvtl. vergeben @fx-kurumi hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 40/100
Maintainer antworten meist innerhalb von 1 Tag
-
【架构优化】从 githubCombinedChecks 中拆出 commit status/check runs 读取与整体状态汇总 githubChecksOverallEvtl. vergeben @fx-kurumi hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 18/100
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in Tencent/LoopForge
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
bazel-contrib/rules_go#4784 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Уязвимые компоненты в GoOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
POSIdev-community/aictl#40 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
nebari-dev/nebari-infrastructure-core#774 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
oracle/terraform-provider-oci#2662 ·
-
[BUG]: A pre-hashed ACL password in uppercase hex fails ACL LOAD and crash-loops restarted podsOffenbug needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
valkey-io/valkey-operator#644 ·
Maintainer antworten meist innerhalb von 3 Tagen