Skip to content

作答时把图内联成 base64,不再把 S3 链接交给模型 - #180

Merged
EricNGOntos merged 2 commits into
mainfrom
feat/wuchengke/compose-inline-images
Sep 20, 2026
Merged

EricNGOntos merged 2 commits into
mainfrom
feat/wuchengke/compose-inline-images

Conversation

@EricNGOntos

Copy link
Copy Markdown
Contributor

Summary

  • compose 把图嵌回占位符,拉成字节后交给 DeepSeek V4.1 Flash(Gateway 编成 base64),不再让模型去访问 S3 签名链。
  • 图/表链接打不开时清掉占位符并打警告,不中断作答。

Test plan

  • vitest compose / runtime / route-service
  • tsc --noEmit
  • CI lint / typecheck / test / build

Made with Cursor

Place figures at the chunk placeholder, skip unreachable image/table URLs with a warning, and keep the default chat model on DeepSeek V4.1 Flash.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Sep 20, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
knowhere-notebook-staging Ready Ready Preview Sep 20, 2026 2:07pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
knowhere-notebook Ignored Ignored Preview Sep 20, 2026 2:07pm UTC

Request Review

Co-authored-by: Cursor <cursoragent@cursor.com>
@EricNGOntos
EricNGOntos merged commit 3b6ac15 into main Sep 20, 2026
7 checks passed
@EricNGOntos
EricNGOntos deleted the feat/wuchengke/compose-inline-images branch September 20, 2026 14:09

This branch was successfully deployed

1 active deployment
Preview – knowhere-notebook-staging — 06375b77 Deployed Sep 20, 2026 by vercel[bot]
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