Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix unused orphanDeleted variable in mid-stream catch block

Browse Source 
The variable was assigned but never read since fallback delivery is
intentionally unconditional in this path. Fixes lint error.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Raul 2026-03-10 08:48:08 +02:00 committed by Vincent Koc
parent f2e0997ede
commit bef0b8c8bb
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/slack/monitor/message-handler/dispatch.ts
View File

@ -311,11 +311,7 @@ export async function dispatchPreparedSlackMessage(prepared: PreparedSlackMessag
// If startStream already created a Slack message, delete it to prevent
// the orphaned stream message from persisting alongside the fallback.
// Only deliver the fallback when deletion succeeds (or no orphan
// existed) — if deletion fails the stream message is still visible, so
// sending a normal reply would recreate the duplicate this PR prevents.
const orphanedTs = streamSession?.streamMessageTs;
let orphanDeleted = !orphanedTs; // trivially "deleted" if nothing to delete
if (orphanedTs) {
try {
await ctx.app.client.chat.delete({
@ -324,7 +320,6 @@ export async function dispatchPreparedSlackMessage(prepared: PreparedSlackMessag
ts: orphanedTs,
});
logVerbose(`slack-stream: deleted orphaned stream message ${orphanedTs}`);
orphanDeleted = true;
} catch (deleteErr) {
logVerbose(
`slack-stream: failed to delete orphaned stream message ${orphanedTs}: ${String(deleteErr)}`,