Skip to content

fix(eve): prioritize current hook deliveries over stale backlog - #598

Closed
iroiro147 wants to merge 2 commits into
vercel:mainfrom
iroiro147:fix/drop-stale-buffered-deliveries-547
Closed

fix(eve): prioritize current hook deliveries over stale backlog#598
iroiro147 wants to merge 2 commits into
vercel:mainfrom
iroiro147:fix/drop-stale-buffered-deliveries-547

Conversation

@iroiro147

Copy link
Copy Markdown
Contributor

Summary

  • split session delivery backlog into turn-owned, current-hook, and replaced-hook queues
  • classify ready hook reads before selecting the next delivery, so current live input runs before older replaced-hook backlog
  • preserve turn-control cancellation/rebuffer ordering and keep the active hook current when a candidate rekey claim fails

Refs #547.

Tests

  • pnpm --filter eve exec vitest run --config vitest.unit.config.ts src/execution/session-delivery-hook.test.ts src/execution/workflow-entry.test.ts src/execution/turn-control-receiver.test.ts src/execution/turn-dispatch.test.ts
  • pnpm --filter eve exec vitest run --config vitest.integration.config.ts src/execution/session-delivery-hook.integration.test.ts
  • pnpm --filter eve typecheck
  • pnpm changeset status --since origin/main
  • git diff --check

Signed-off-by: iroiro147 <sarthak.singh@juspay.in>
@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

@iroiro147 is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

Comment thread packages/eve/src/execution/session-delivery-hook.ts Outdated
Signed-off-by: iroiro147 <sarthak.singh@juspay.in>
@iroiro147

Copy link
Copy Markdown
Contributor Author

Superseded by verified rebased replacement #1402 (rebuilt on current main, MERGEABLE): #1402 — closing this stale conflicting PR in favor of the new one.

@iroiro147 iroiro147 closed this Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant