[Bug] Inconsistent dataSuffix hex stripping: slice(2) used in some examples but not others
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 1/5
- 見積もり時間
- 1時間未満
- 初心者へのやさしさ
- 78/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 静か
- 技術スタック
- javascript
調査の方向性
tmp-builder-codes-outline.mdx を開き、"5. Minimal example (EOA),"、"A. EOA Transactions,"、"B. Smart Account / ERC-4337." における dataSuffix の処理を比較してください。各例で既存の 16 進文字列に一貫した形でサフィックスが追加されていることを確認し、その後、ドキュメントの例が引き続き有効であることを検証してください。
索引モデルが issue の本文から書いたものです。
説明
Description
In tmp-builder-codes-outline.mdx, there is an inconsistency in how
dataSuffix is appended to calldata across examples.
Some examples strip the 0x prefix:
data: tx.data + suffix.slice(2)
While others append directly without stripping:
tx.data = tx.data + dataSuffix
userOp.callData = userOp.callData + dataSuffix
Location
tmp-builder-codes-outline.mdx:
- Section "5. Minimal example (EOA)" → uses
suffix.slice(2) - Section "A. EOA Transactions" → uses
tx.data + dataSuffix(no slice) - Section "B. Smart Account / ERC-4337" → uses
userOp.callData + dataSuffix(no slice)
Impact
- Developers copying different examples will get inconsistent behavior
- Appending
0xprefix mid-calldata will produce malformed transactions - Could cause silent transaction failures or wrong attribution
Suggested Fix
Standardize all examples to consistently use suffix.slice(2) when
appending to existing hex strings, or use a proper concat utility:
data: tx.data + dataSuffix.slice(2)
- 主要言語
- JavaScript
- スター
- 337
- フォーク
- 798
- 平均マージ
- 19時間 12分
- マージ済み PR（30日）
- 60
環境構築
このプロジェクトの環境構築ファイルはまだ確認していません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
base/docs のほかの issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 95/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 95/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 95/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
bug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
amponce/archive-movie-browser#354 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
saayam-for-all/webapp#1870 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
Imageomics/OpenCite#66 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
chr15m/twiiit.com#20 ·
-
documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
antropos17/Aegis#629 ·
メンテナーはふだん 4 日以内に返信