fix: stale PostToolUse notifications and background task output leak#353

Draft

timvisher-dd wants to merge 31 commits intoagentclientprotocol:mainfrom

timvisher-dd:fix/stale-post-tool-hook-notifications

Commits on Mar 30, 2026

fix: replace blocking PostToolUse hook with non-blocking fire-and-stash

timvisher-dd
and
claude
committed
fix: consume SDK internal turns from background tasks before returning prompt

timvisher-dd
and
claude
committed
fix: improve internal turn detection with task_type filter and event loop yield

timvisher-dd
and
claude
committed
test: add coverage for local_agent filtering and async task_notification

timvisher-dd
and
claude
committed
fix: correct misleading "microtask" comment to "macrotask"

timvisher-dd
and
claude
committed
test: add warning path and failed task_notification coverage

timvisher-dd
and
claude
committed
test: add edge case coverage for stopped status, error results, and back-to-back internal turns

timvisher-dd
and
claude
committed
test: replace real setTimeout delays with deterministic microtask flushing

timvisher-dd
and
claude
committed
fix: address codex timer review — microtask flush before advancing and document depth

timvisher-dd
and
claude
committed
style: fix prettier formatting in test files

timvisher-dd
and
claude
committed
fix: lint errors and add bin/test for local CI validation

timvisher-dd
and
claude
committed
fix: type test fixture helpers as any[] to fix tsc build errors

timvisher-dd
and
claude
committed
fix: improve bin/test comment explaining git-tracked-only format check

timvisher-dd
and
claude
committed
refactor: bin/test parses CI workflow YAML instead of hardcoding steps

timvisher-dd
and
claude
committed
fix: resolve all tsc build errors for CI

timvisher-dd
and
claude
committed
docs: annotate minified SDK names with likely original names

timvisher-dd
and
claude
committed
fix: add sdk-tools.js type shim for CI npm ci resolution

timvisher-dd
and
claude
committed
fix: bin/test validates node_modules sync instead of blindly skipping npm ci

timvisher-dd
and
claude
committed
fix: inject user replay into mock query so promptReplayed becomes true

timvisher-dd
and
claude
committed
fix: replace promptReplayed with backgroundInitPending for result detection

timvisher-dd
and
claude
committed
fix: replace peek-once with inactivity poll loop for bg task detection

timvisher-dd
and
claude
committed
test: update bg-task-leak tests for poll loop and add new coverage

timvisher-dd
and
claude
committed
feat: add structured stderr logging to bg-task poll loop

timvisher-dd
and
claude
committed
fix: remove inactivity timeout, keep turn open until tasks resolve

timvisher-dd
and
claude
committed
style: format bg-task-leak test with prettier

timvisher-dd
and
claude
committed
fix: check cancellation before consuming background task results

timvisher-dd
and
claude
committed
docs: add agent-shell rendering continuity and poll harness results

timvisher-dd
and
claude
committed
test: add regression test for cancel during backgroundInitPending

timvisher-dd
and
claude
committed
chore: remove pr.md from tracking

timvisher-dd
and
claude
committed
fix: remove duplicate imports in tools.ts introduced by rebase

timvisher-dd
and
claude
committed
fix: resolve rebase conflicts with upstream session_state_changed and stopReason

timvisher-dd
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: stale PostToolUse notifications and background task output leak#353

fix: stale PostToolUse notifications and background task output leak#353
timvisher-dd wants to merge 31 commits intoagentclientprotocol:mainfrom
timvisher-dd:fix/stale-post-tool-hook-notifications

Commits on Mar 30, 2026

fix: replace blocking PostToolUse hook with non-blocking fire-and-stash

fix: consume SDK internal turns from background tasks before returning prompt

fix: improve internal turn detection with task_type filter and event loop yield

test: add coverage for local_agent filtering and async task_notification

fix: correct misleading "microtask" comment to "macrotask"

test: add warning path and failed task_notification coverage

test: add edge case coverage for stopped status, error results, and back-to-back internal turns

test: replace real setTimeout delays with deterministic microtask flushing

fix: address codex timer review — microtask flush before advancing and document depth

style: fix prettier formatting in test files

fix: lint errors and add bin/test for local CI validation

fix: type test fixture helpers as any[] to fix tsc build errors

fix: improve bin/test comment explaining git-tracked-only format check

refactor: bin/test parses CI workflow YAML instead of hardcoding steps

fix: resolve all tsc build errors for CI

docs: annotate minified SDK names with likely original names

fix: add sdk-tools.js type shim for CI npm ci resolution

fix: bin/test validates node_modules sync instead of blindly skipping npm ci

fix: inject user replay into mock query so promptReplayed becomes true

fix: replace promptReplayed with backgroundInitPending for result detection

fix: replace peek-once with inactivity poll loop for bg task detection

test: update bg-task-leak tests for poll loop and add new coverage

feat: add structured stderr logging to bg-task poll loop

fix: remove inactivity timeout, keep turn open until tasks resolve

style: format bg-task-leak test with prettier

fix: check cancellation before consuming background task results

docs: add agent-shell rendering continuity and poll harness results

test: add regression test for cancel during backgroundInitPending

chore: remove pr.md from tracking

fix: remove duplicate imports in tools.ts introduced by rebase

fix: resolve rebase conflicts with upstream session_state_changed and stopReason