Skip to content

feat: render demo seed answers with citation chips - #145

Merged
suguanYang merged 3 commits into
stagingfrom
feat/wangbinqi/demo-chat-citation-chips
Aug 20, 2026
Merged

feat: render demo seed answers with citation chips#145
suguanYang merged 3 commits into
stagingfrom
feat/wangbinqi/demo-chat-citation-chips

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary

  • Map demo catalog page_citation_page_number / page_citation_asset_url onto seed chat citations.
  • Keep [[cite:n]] in demo answers (and append markers if an older catalog omits them) so Tesla/SpaceX seed turns render {title}/pN chips and SOURCES page links.

Depends on Knowhere projecting those citation fields on /api/v1/demo/catalog.

Test plan

  • pnpm exec vitest run src/domains/demo/view.test.ts src/integrations/knowhere-demo.test.ts
  • Open a fresh demo workspace and confirm Tesla/SpaceX seed answers show inline {title}/pN chips
  • Click a seed chip and land on that parsed page card

Made with Cursor

@vercel

vercel Bot commented Aug 20, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
knowhere-notebook-staging Ready Ready Preview Aug 20, 2026 4:47pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
knowhere-notebook Ignored Ignored Preview Aug 20, 2026 4:47pm

Request Review

suguanYang and others added 2 commits August 21, 2026 00:28
Map catalog page metadata onto demo chat citations and keep [[cite:n]] in the answer so seed Tesla/SpaceX turns match live {title}/pN chips.

Co-authored-by: Cursor <cursoragent@cursor.com>
Reuse toChatCitationViews so demo Q&A chips get the same title/pN page
numbers as normal retrieval answers.

Co-authored-by: Cursor <cursoragent@cursor.com>
withCitationMarkers appends [[cite:n]] when the catalog answer omits them
so chips can render on guest demo chat.

Co-authored-by: Cursor <cursoragent@cursor.com>
@suguanYang
suguanYang merged commit 8120a22 into staging Aug 20, 2026
7 checks passed
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