Skip to content

fix: route multi-turn follow-up turns with session-aware context#1755

Open
Deepak8858 wants to merge 1 commit intovllm-project:mainfrom
Deepak8858:fix/issue-1751-session-aware-context
Open

fix: route multi-turn follow-up turns with session-aware context#1755
Deepak8858 wants to merge 1 commit intovllm-project:mainfrom
Deepak8858:fix/issue-1751-session-aware-context

Conversation

@Deepak8858
Copy link
Copy Markdown

Fixes #1751

Populates SelectionContext.SessionID, SelectionContext.UserID, and SelectionContext.ConversationHistory from chat completions and Response API flows to ensure multi-turn selectors receive proper session-aware context.

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 10, 2026

Deploy Preview for vllm-semantic-router ready!

Name Link
🔨 Latest commit ef0b7b8
🔍 Latest deploy log https://app.netlify.com/projects/vllm-semantic-router/deploys/69d93b9431664b0008eb14f3
😎 Deploy Preview https://deploy-preview-1755--vllm-semantic-router.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

👥 vLLM Semantic Team Notification

The following members have been identified for the changed files in this PR and have been automatically assigned:

📁 src/semantic-router

Owners: @rootfs, @Xunzhuo, @szedan-rh, @yehuditkerido, @abdallahsamabd, @asaadbalum, @liavweiss, @noalimoy
Files changed:

  • src/semantic-router/pkg/extproc/req_filter_classification.go
  • src/semantic-router/pkg/extproc/req_filter_classification_runtime.go

vLLM

🎉 Thanks for your contributions!

This comment was automatically generated based on the OWNER files in the repository.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

✅ Supply Chain Security Report — All Clear

Scanner Status Findings
AST Codebase Scan (Py, Go, JS/TS, Rust) 27 finding(s) — MEDIUM: 21 · LOW: 6
AST PR Diff Scan No issues detected
Regex Fallback Scan No issues detected

Scanned at 2026-04-10T18:04:34.550Z · View full workflow logs

@rootfs
Copy link
Copy Markdown
Collaborator

rootfs commented Apr 10, 2026

@Deepak8858 can you sign DCO and fix the precommit error?

 vllm-project#1751)

Signed-off-by: Deepak Singh <deepak.singh@example.com>
@Deepak8858 Deepak8858 force-pushed the fix/issue-1751-session-aware-context branch from be9b7e0 to ef0b7b8 Compare April 10, 2026 18:04
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.

bug: route multi-turn follow-up turns with session-aware context instead of isolated last-turn classification

9 participants