GitHub App: PR overview comment re-edited on every push with no content change
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 68/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- typescript
- Bereich
- backend
Rechercherichtung
Beginne damit, das Rendering und den Aktualisierungspfad des PR-Übersichtskommentars der GitHub App zu lokalisieren, und vergleiche dann den gerenderten Inhalt mit dem bestehenden Kommentar, wobei scan-run IDs in dashboard URLs ignoriert werden. Als erledigt gilt dies, wenn Pushes mit einem unveränderten Dependency-Diff den Kommentar unverändert lassen und doppelte Benachrichtigungen sowie issue_comment.edited webhooks vermeiden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- TypeScript
- Sterne
- 320
- Forks
- 66
- Ø Merge
- 2 Std. 21 Min.
- Gemergte PRs (30 T.)
- 30
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus SocketDev/socket-cli
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
SocketDev/socket-cli#1160 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
SocketDev/socket-cli#1547 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
SocketDev/socket-cli#1525 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
SocketDev/socket-cli#1517 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
SocketDev/socket-cli#1497 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in SocketDev/socket-cli
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
rossoctl/serverless-harness#428 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
prime-radiant-inc/evener#3726 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
FuRongJun-1999/dsh-memory#56 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug via-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
pingdotgg/t3code#15682 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
openwatersio/slackwater.xyz#152 ·
Maintainer antworten meist innerhalb von 1 Tag