[Bug] Inconsistent dataSuffix hex stripping: slice(2) used in some examples but not others
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Tranquilo
- Stack tecnológico
- javascript
- Área
- blockchain, documentation
Línea de trabajo
Abre tmp-builder-codes-outline.mdx y compara el manejo de dataSuffix en "5. Minimal example (EOA)," "A. EOA Transactions," y "B. Smart Account / ERC-4337." Comprueba que cada ejemplo añada el sufijo de forma coherente a una cadena hexadecimal existente y, después, verifica que los ejemplos de la documentación sigan siendo válidos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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)
- Lenguaje dominante
- JavaScript
- Estrellas
- 343
- Forks
- 818
- Merge medio
- 1 d 4 h
- PR fusionados (30 d)
- 63
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de base/docs
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
Issue on docsAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
Los mantenedores suelen responder en 1 día
-
Broken redirect: /wallet-app/:slug* catch-all leads to 404Posiblemente ocupada @forumevi la tomó hace 18 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
Los mantenedores suelen responder en 1 día
-
docs(builder-codes): base.dev links use HTTP instead of HTTPSPosiblemente ocupada @xam-dev-ux la tomó hace 38 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
base/docs#1910 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Issues similares
-
bug user-priority/P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
t8y2/dbx#11718 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug confirmed perf
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
videojs/video.js#9400 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
agent/scanner bug hive/hosted-available-lke648397-260827-5n31
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
rescript-lang/rescript#8765 ·
Los mantenedores suelen responder en 1 día
-
[workshop-sim] Improve: Add a required self-check gate to the agentic-workflows concept pageAbiertofeedback simulation workshop
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
githubnext/gh-aw-workshop#4417 ·
Los mantenedores suelen responder en 1 día