gnd: scaffolding follow-ups from #6660 review
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 48/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- rust, solidity
- 領域
- blockchain, tooling
調査の方向性
scaffold/schema.rs と codegen/abi.rs から始め、重複している整数幅のカットオフと、イベント名およびパラメータ名の追加パスを追跡します。issue に記載されている生成スキーマと ABI getter の動作を比較します。Int8 のマッピングが一貫して機能し、列挙された各衝突が生成名の重複なしで処理され、解析できない schema.graphql に対して警告が出れば完了です。
索引モデルが issue の本文から書いたものです。
説明
Follow-ups from the review of #6660.
Int8 support (comment)
Map mid-size Solidity ints (signed 33-64 bits, unsigned 25-63) to Int8 instead of BigInt. The schema scaffold and the ABI codegen need to change together: the generated getter would chain .toBigInt().toI64() since ethereum.Value has no i64 accessor. The entity side already supports Int8. Before adding the band, consolidate the int width cutoff into one shared helper — it's currently duplicated in scaffold/schema.rs and ~8 places in codegen/abi.rs.
Duplicate-name corner cases (comment)
All of these produce a duplicate name in the generated output, so the build fails:
add: an entity renamed on collision isn't checked against the other events in the same batch. ExistingTransfer+ ABI withTransferandTokenTransfer→TokenTransferdeclared twice.- Two params that sanitize to the same field name aren't deduped (e.g. two unnamed params both become
value). The accessor side already dedupes (param0,param1); the field side should too. - The overload suffix can clash with a real event:
Transfer,Transfer,Transfer1→ twoTransfer1s. Bump the counter until the name is free. - An unparseable schema.graphql silently disables collision detection in
add. Should log a warning.
- 主要言語
- Rust
- スター
- 3.2k
- フォーク
- 1.1k
- 平均マージ
- 4日 1時間
- マージ済み PR(30日)
- 1
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
graphprotocol/graph-node のほかの issue
-
current: include emits an all-null bucket for dimensionless aggregations, nulling the whole response オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
graphprotocol/graph-node#6719 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
graphprotocol/graph-node#6673 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
graphprotocol/graph-node#6650 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
graphprotocol/graph-node#6722 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
graphprotocol/graph-node#6721 ·
graphprotocol/graph-node の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
bug good first issue package: quic
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
-
`dora trace view` sends a non-canonical full UUID as-is, so a valid trace ID shows "No spans found" オープンcli coordinator rust
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
area: tasks enhancement good first issue help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
Jason-jo17/Polybench#15 · コメント 1 件 ·