GitHub App: PR overview comment re-edited on every push with no content change
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 68/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- typescript
- Área
- backend
Línea de trabajo
Comienza localizando el renderizado y la ruta de actualización del comentario de resumen de PR de la GitHub App; después, compara el cuerpo renderizado con el comentario existente, ignorando los scan-run IDs en las dashboard URLs. Se considera completado cuando los pushes con un diff de dependencias sin cambios dejan el comentario intacto y evitan notificaciones duplicadas y issue_comment.edited webhooks.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The Socket GitHub App's PR overview comment (<!-- overview-comment -->) is edited in place after every push, even when the dependency diff is completely unchanged. On one PR I observed 8 edits in ~50 minutes; diffing the bodies, the only change each time was the diff-scan/<uuid> in the dashboard links — same package, same scores, same table, byte-identical otherwise.
Each edit re-notifies subscribers and fires issue_comment.edited webhooks, so unchanged scans produce pure notification churn for humans and bots watching the PR.
Request: skip the comment edit when the rendered body (ignoring scan-run IDs in URLs) is identical to what is already posted. The new scan ID doesn't need to reach the comment — the dashboard already has it
- Lenguaje dominante
- TypeScript
- Estrellas
- 317
- Forks
- 65
- Merge medio
- 1 h 26 min
- PR fusionados (30 d)
- 30
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 SocketDev/socket-cli
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
SocketDev/socket-cli#1160 · 1 comentario ·
-
[fuzz] fuzz-js job failed Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
SocketDev/socket-cli#1547 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
SocketDev/socket-cli#1525 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
SocketDev/socket-cli#1517 ·
-
Add homebrew distribution Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
SocketDev/socket-cli#1497 · 1 reacción ·
Todos los issues de SocketDev/socket-cli
Issues similares
-
Type/Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
OpenNSW/nsw-srilanka#497 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
0xMiden/bridge-portal#132 ·
-
react-doctor severity:warning tech-debt
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
digidem/comapeo-cloud-app#403 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100