Casework: description validation accepts two request entities sharing one requestRoute
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
この issue は crates/registry-casework-breg/src/config.rs にあり、具体的には validate_description 関数に関するものです。まず、既存の duplicate-entity テストを調べて、検証パターンを理解してください。関数を変更して、requestEntity に加えて requestRoute も追跡するようにし、重複したルート用の新しいユニットテストを追加してください。修正が機能することを確認するために、既存のテストを実行してください。
索引モデルが issue の本文から書いたものです。
説明
Problem
validate_description in crates/registry-casework-breg/src/config.rs refuses a v1alpha2 source description that lists the same requestEntity twice, but it never compares requestRoute across entries. A description that gives two different request entities the same route therefore passes caseworkctl check, caseworkctl package, and runtime input validation (validate_description_input).
BregAdapter::new requires routes to be unique, so the same project is refused only when the runtime constructs the source adapter. An offline-approved package is undeployable.
caseworkctl source add does not produce this shape, since it copies each route from bregctl explain change-requests. It is reachable through a hand-edited or otherwise supplied description.
Expected
The description decoder refuses a repeated requestRoute in the same place it refuses a repeated requestEntity, so the offline checks and adapter construction agree.
Fix sketch
- Track the routes seen while building
by_entityinvalidate_description, and returnSourceAdapterError::Invalidon a repeat. - Add a unit test beside the existing duplicate-entity description test: two entries with distinct entities and one shared route are refused.
Found in review of #1355, after the related policy-level entity uniqueness check landed there.
- 主要言語
- Rust
- スター
- 2
- フォーク
- 0
- 平均マージ
- 5時間 6分
- マージ済み PR（30日）
- 170
環境構築
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
registrystack/registry-stack のほかの issue
-
agent-ready area:platform bug criticality:p3 rust triage:needs-implementation
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
registrystack/registry-stack#1583 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
docs(breg): configure/breg.mdx lists stages for a request_lifecycle when, which bregctl refusesオープンagent-ready area:breg area:docs criticality:p3 documentation triage:needs-implementation
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
registrystack/registry-stack#1574 ·
メンテナーはふだん 1 日以内に返信
-
area:docs area:release criticality:p2 documentation triage:needs-implementation
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
registrystack/registry-stack#1474 ·
メンテナーはふだん 1 日以内に返信
-
agent-ready area:evidence bug criticality:p3 triage:needs-implementation
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
registrystack/registry-stack#1449 ·
メンテナーはふだん 1 日以内に返信
-
agent-ready area:breg bug criticality:p3 triage:needs-implementation
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
registrystack/registry-stack#1405 ·
メンテナーはふだん 1 日以内に返信
registrystack/registry-stack の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
trailofbits/dylint#2107 ·
メンテナーはふだん 1 日以内に返信
-
area:cli bug good first issue priority:medium
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
メンテナーはふだん 1 日以内に返信
-
arrays_zip with two same-named inputs fails with "ArrowArray struct has 2 children (expected 1)"オープンbug requires-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
apache/datafusion-comet#6251 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
bug false-positive harper-core linting
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
Automattic/harper#4471 ·
メンテナーはふだん 1 日以内に返信