Bricks: split PackageResolver.MaterializeTwinAsync
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Open Gaya/MASS4.Bricks/PackageResolver.Twins.cs and read MaterializeTwinAsync and its callees. Apply the pending cached-bake fix first, then introduce a private value holding the parameters and move the index/cached/published path into a separate async step. 'Done' is the full suite green with no test changes and each changed method at complexity ≤8 and CRAP ≤15.
索引モデルが issue の本文から書いたものです。
説明
Current State
PackageResolver.MaterializeTwinAsync (Gaya/MASS4.Bricks/PackageResolver.Twins.cs) scored CRAP 32 in the store contract client batch (96a5587): one async body reads the lock, the package store, the registry index and the variant choice. The pending cached-bake fix splits out CachedTwin, CheckedTwinAsync and AvailableTwinAsync, which brings it to complexity 14 at the 15 limit, with AvailableTwinAsync taking eleven parameters. Branches from static-lambda caching, the strict-lock exit, the index fetch and the cached path still add up in one state machine.
Tasks
- Carry the source id, ranges, requester, depth, existing lock and locked bake in one private value instead of separate parameters
- Keep the strict-lock exit in a synchronous
MaterializeTwinAsyncand move the index/cached/published path into its own async step - Pass that value to
AvailableTwinAsyncinstead of its parameter list
Acceptance Criteria
- Behavior unchanged: no test edited, full suite green
- Every changed method has complexity ≤ 8 and CRAP ≤ 15
Related
- Lands after the cached/locked bake compatibility and advisory fix
- 主要言語
- C#
- スター
- 1
- フォーク
- 1
- 平均マージ
- 1時間 46分
- マージ済み PR（30日）
- 41
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
MASS4ORG/Turian のほかの issue
-
c:assets p:low size:XS t:task
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
c:engine p:medium size:S t:enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
メンテナーはふだん 1 日以内に返信
-
c:gaya p:high size:XS t:bug
難易度 3/5 1〜2日 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
c:build c:editor p:medium size:XL t:feature
難易度 5/5 1週間以上 初心者へのやさしさ 5/100
メンテナーはふだん 1 日以内に返信
-
c:build p:medium size:XL t:refactor
難易度 5/5 1週間以上 初心者へのやさしさ 8/100
メンテナーはふだん 1 日以内に返信
MASS4ORG/Turian の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
unoplatform/uno.check#591 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
weaviate/csharp-client#378 ·
-
agentic-workflows automated testing
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
Azure-Samples/Legacy-Modernization-Agents#194 ·
メンテナーはふだん 1 日以内に返信
-
area-encryption area-security p1-recommended technical-debt
難易度 2/5 1時間未満 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
bug flaky small
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
hexide-io/HexIDE#1906 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信