Skip to content

fix(eve): defer context during approval resume#599

Open
iroiro147 wants to merge 1 commit into
vercel:mainfrom
iroiro147:fix/linear-approval-context-529
Open

fix(eve): defer context during approval resume#599
iroiro147 wants to merge 1 commit into
vercel:mainfrom
iroiro147:fix/linear-approval-context-529

Conversation

@iroiro147

Copy link
Copy Markdown
Contributor

Summary

  • defer channel context attached to pending tool-approval responses until the next internal step
  • keep the approval-response tool message as the tail of the approval-resume model call so approval collection can run
  • add resolver and harness regressions for context-bearing approval resumes

Refs #529.
Related #460.

Tests

  • pnpm guard:invariants
  • pnpm --filter eve exec vitest run --config vitest.unit.config.ts src/harness/input-requests.test.ts src/harness/tool-loop.test.ts (171 passed)
  • pnpm --filter eve typecheck
  • pnpm --filter eve exec oxlint src/harness/input-requests.ts src/harness/input-requests.test.ts src/harness/tool-loop.ts src/harness/tool-loop.test.ts --deny-warnings
  • pnpm exec oxfmt --check packages/eve/src/harness/input-requests.ts packages/eve/src/harness/input-requests.test.ts packages/eve/src/harness/tool-loop.ts packages/eve/src/harness/tool-loop.test.ts .changeset/linear-approval-context-resume.md
  • 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.

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