Skip to content

fix(eve): merge system instructions before model calls#597

Open
iroiro147 wants to merge 1 commit into
vercel:mainfrom
iroiro147:fix/merge-system-instructions-596
Open

fix(eve): merge system instructions before model calls#597
iroiro147 wants to merge 1 commit into
vercel:mainfrom
iroiro147:fix/merge-system-instructions-596

Conversation

@iroiro147

Copy link
Copy Markdown
Contributor

Summary

  • merge assembled system instructions into a single system message before constructing the model call
  • keep session history, dynamic instruction lifecycle, and array-form user content unchanged
  • preserve Anthropic direct-cache provider options by applying the cache marker before merging

Fixes #596.

Tests

  • pnpm --filter eve exec vitest run --config vitest.unit.config.ts src/harness/tool-loop.test.ts (148 passed)
  • pnpm --filter eve run typecheck
  • pnpm --filter eve exec oxlint src/harness/tool-loop.ts src/harness/tool-loop.test.ts src/harness/instrumentation-runtime-context.ts src/public/instrumentation/index.ts
  • pnpm exec oxfmt --check packages/eve/src/harness/tool-loop.ts packages/eve/src/harness/tool-loop.test.ts packages/eve/src/harness/instrumentation-runtime-context.ts packages/eve/src/public/instrumentation/index.ts .changeset/merge-system-instructions.md
  • pnpm guard:invariants
  • 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.

Multi-system-message prompt assembly breaks Fireworks (opaque 500), merge system messages at the model-call boundary

2 participants