From 2cc9d5c19ed369102f8ed5d3cd2ba887ed953db1 Mon Sep 17 00:00:00 2001 From: kewton Date: Thu, 8 Oct 2026 21:54:15 +0900 Subject: [PATCH] =?UTF-8?q?fix(session):=20self=5Fresume=5Fpending=20?= =?UTF-8?q?=E3=82=92=E9=80=81=E3=82=8B=20source=20=E3=81=AE=E6=B3=A8?= =?UTF-8?q?=E8=A8=98=E3=82=92=20antigravity=20=E3=81=A8=20claude=20?= =?UTF-8?q?=E3=81=AB=E5=90=88=E3=82=8F=E3=81=9B=E3=82=8B=20(#3449)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit コメントと it の題名だけの変更。振る舞いは変えない。 - agent-event-dedup.ts の releaseStopClaims のコメントを「antigravity (#2614) と claude (#3430)」に修正 - capabilities.test.ts の it の名前を変更（中身の期待値 ['claude','antigravity'] は不変） 旧: names exactly one source whose stop can say it will resume by itself (#2614) 新: names the sources whose stop can say it will resume by itself: antigravity (#2614) and claude (#3430) あわせて it 内コメントの「title's one」の言及を直した 対になる場所の探索 (grep -rn "self_resume_pending|stopReportsSelfResume|self-resume|self resume" src docs): - agent-event-types.ts の宣言コメント、capabilities.test.ts の冒頭コメント、docs/module-reference.md は既に claude (#3430) を含む → 直さない - agent-event-turn.ts:457 の「antigravity's self-resume post_tool_use」はターン開始の話で antigravity 固有のため正しい → 直さない - docs の ja/en: 「antigravity だけ」と読める記述は無かった 本文に無い指摘: なし Co-Authored-By: Claude Sonnet 5.5 --- changelog.d/3449.md | 2 ++ src/lib/session/agent-event-dedup.ts | 4 ++-- tests/unit/hooks/sources/capabilities.test.ts | 4 ++-- 3 files changed, 6 insertions(+), 4 deletions(-) create mode 100644 changelog.d/3449.md diff --git a/changelog.d/3449.md b/changelog.d/3449.md new file mode 100644 index 000000000..fea4c9e97 --- /dev/null +++ b/changelog.d/3449.md @@ -0,0 +1,2 @@ + +- **fix(session): self_resume_pending を送る source の注記を antigravity と claude に合わせる** (#3449): releaseStopClaims のコメントと capabilities テストの題名が「antigravity だけ」と読めたのを直した（振る舞いは不変） diff --git a/src/lib/session/agent-event-dedup.ts b/src/lib/session/agent-event-dedup.ts index 53d137b1a..6f0498f2d 100644 --- a/src/lib/session/agent-event-dedup.ts +++ b/src/lib/session/agent-event-dedup.ts @@ -380,8 +380,8 @@ export function shortSessionTag(sessionId: string): string { /** * Forget every `stop` one session of one instance has claimed (Issue #3289). * - * Every subtype: antigravity posts `stop` / `self_resume_pending` (#2614) - * beside the plain one. + * Every subtype: antigravity (#2614) and claude (#3430) post `stop` / + * `self_resume_pending` beside the plain one. */ function releaseStopClaims(composite: string, sessionId: string): void { releaseClaims(composite, ['stop'], sessionId); diff --git a/tests/unit/hooks/sources/capabilities.test.ts b/tests/unit/hooks/sources/capabilities.test.ts index 65c58668b..fe2a6239a 100644 --- a/tests/unit/hooks/sources/capabilities.test.ts +++ b/tests/unit/hooks/sources/capabilities.test.ts @@ -331,11 +331,11 @@ describe('[#1924] AgentSourceCapabilities — the table of §4 D3', () => { expect(scraperOnly).toEqual(['gemini', 'copilot']); }); - it('names exactly one source whose stop can say it will resume by itself (#2614)', () => { + it('names the sources whose stop can say it will resume by itself: antigravity (#2614) and claude (#3430)', () => { // agy is the one tool whose `Stop` payload is known to state it (`fullyIdle`) // and whose hook passes it on. Issue #3430 added claude, which reads it off // its transcript rather than the payload (only an empty `background_tasks` - // has been captured) — so the title's "one" now reads "one per mechanism". + // has been captured) — so the list is one source per mechanism. const selfResuming = Object.keys(TABLE).filter((id) => TABLE[id].stopReportsSelfResume); expect(selfResuming).toEqual(['claude', 'antigravity']); });