refactor(host): explicit host for configuration reads and one entered runtime store
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 25/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- node.js, typescript
調査の方向性
Start with the root platform-runtime-* files, platform packages, and defineRuntimeSlots modules; trace the remaining commandHostEnvironment() calls and runtime slot handles. Read the follow-ups #3763 and #3772 and the holistic review on #3747 for decisions already made. Done means configuration reads use the runtime's explicit host, one runtime-owned store is entered and disposed, and the daemon host test plus two-runtime isolation tests pass.
索引モデルが issue の本文から書いたものです。
説明
Part of #3747. Follow-up to #3763 and #3772, which leave two pieces deferred in their PR bodies.
1. Narrow commandHostEnvironment() to command execution
About 60 call sites remain in the root platform-runtime-* files and the platform packages. They read configuration (toolchain and SDK paths, caches, path resolution) through the ambient, AsyncLocalStorage-bound commandHostEnvironment(). The holistic review on the stack decided:
- Configuration reads take the host explicitly. Pass it as a parameter, through the runtime deps, or through a port bound at construction.
- The ambient lookup is only for host-kit command execution and diagnostics.
Done when:
- Every non-test call of
commandHostEnvironment()outside host-kit's command execution and diagnostics is replaced by an explicit host. - A test on the daemon route proves configuration is read from the runtime's host and not from the process, when the two differ.
2. One entered runtime store
#3772 moved per-runtime state onto defineRuntimeSlots. Each module still exposes its own handle, collected into runtimeStates.
Required:
- Collapse them into one store, entered once per runtime.
- Modules declare slots; the runtime owns entry and disposal.
Done when:
- No module-level handle or
runtimeStatesaggregation remains. - The two-runtime isolation tests (Apple runner, Android helper, PNG worker, daemon modules) still pass.
Ordering
- #3757 (
agent-device/host) does not depend on this issue. - #3753 (lint gate) should land after part 1, so the gate can also forbid
commandHostEnvironment()outside the command-execution boundary.
- 主要言語
- TypeScript
- スター
- 5k
- フォーク
- 334
- 平均マージ
- 12時間 13分
- マージ済み PR（30日）
- 723
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
callstack/agent-device のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 87/100
callstack/agent-device#3819 ·
メンテナーはふだん 1 日以内に返信
-
macOS helper audio-probe always reports -90 dBFS: AVAudioPCMBuffer frameLength set after the copy対応中かも @jipeng6036-del が 1 日前に担当しました。 オープンbug ready-for-agent
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
callstack/agent-device#3771 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
callstack/agent-device#1869 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
callstack/agent-device#3832 ·
メンテナーはふだん 1 日以内に返信
-
Android smoke: first fixture capture stalls before any readable snapshot対応中かも @thymikee が今日担当しました。 オープンbug needs-triage
callstack/agent-device#3831 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
callstack/agent-device の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
getsentry/sentry-conventions#732 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
status:in-linear status:team-assigned team:nodes
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
motiondivision/motion#3955 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
equinor/prisma-decision-web#919 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 83/100