Skip to content
This repository was archived by the owner on Jul 16, 2026. It is now read-only.

Commit f845d61

Browse files
LiangNiangclaude
andcommitted
docs(gateway): explain why recap notify catch is split from outer catch
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent d02f62c commit f845d61

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

packages/core/src/gateway/gateway.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -338,6 +338,9 @@ export class Gateway {
338338
`[gateway] auto-new: recap archived old=${oldRoute.id}, recapId=${entry.id}`,
339339
);
340340
const text = formatRecapNotice(incoming.channelType, entry.result.heading);
341+
// Inner catch is intentionally separate from the outer .catch:
342+
// recap is already persisted to route.recaps[]; push failure must
343+
// not be logged as "recap failed" or it loses the distinction.
341344
try {
342345
await this.pushMessage(incoming.channelType, incoming.channelId, text);
343346
} catch (err) {

0 commit comments

Comments
 (0)