From 31ca165499a4b2a537d1907f641e507abe14cddf Mon Sep 17 00:00:00 2001 From: Nazim Date: Thu, 26 Mar 2026 16:23:32 +0000 Subject: [PATCH] fix: use DELETE (done) instead of PATCH (read) for GitHub notifications MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PATCH only marks as read — notification stays in inbox. DELETE marks as done — removes from inbox entirely. --- bin/argus | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/bin/argus b/bin/argus index c4177c6..f94a03d 100755 --- a/bin/argus +++ b/bin/argus @@ -652,16 +652,16 @@ def cmd_notif_pull(args) -> None: else: kept_count += 1 - # Mark as read on GitHub (prevents re-fetching ~5000 stale notifications) + # Mark as done on GitHub (DELETE removes from inbox entirely) if not args.dry_run and backend == "gh": try: subprocess.run( - ["gh", "api", "--method", "PATCH", + ["gh", "api", "--method", "DELETE", f"notifications/threads/{notif_id}"], capture_output=True, text=True, check=False ) except Exception: - pass # Best-effort — don't fail the pull if marking read fails + pass # Best-effort — don't fail the pull if marking done fails if not args.dry_run: # Record poll