Skip to content

feat: archive remote Knowhere sources and filter by client origin - #146

Merged
suguanYang merged 2 commits into
stagingfrom
feat/wangbinqi/remote-archive-origin-filter
Aug 20, 2026
Merged

feat: archive remote Knowhere sources and filter by client origin#146
suguanYang merged 2 commits into
stagingfrom
feat/wangbinqi/remote-archive-origin-filter

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary

  • PATCH /api/sources/knowhere-doc:… now archives the Knowhere document instead of 404ing; if a matching Notebook sources row exists it is soft-deleted and its blob is cleaned up.
  • Remote listing and chat listDocuments only show documents with created_by_client / createdByClient in {notebook, cli, mcp}. Untagged leftovers (e.g. staging dummy.pdf) stay in Knowhere but no longer appear in the sidebar.

Test plan

  • On staging Notebook, confirm untagged/dummy remotes no longer appear in Sources.
  • Confirm a CLI or MCP remote still appears and can be deleted (PATCH 200, document archived in Knowhere).
  • Confirm a local Notebook upload still deletes as before (Knowhere archive + local row gone).
  • Confirm Official Library hide still works for catalog demo ids.

Made with Cursor

Remote-only sidebar ids now call Knowhere archive (and clean a matching local row if one exists). Listing and chat only show remotes created by notebook, CLI, or MCP.

Co-authored-by: Cursor <cursoragent@cursor.com>
@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 11:57am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
knowhere-notebook Ignored Ignored Preview Aug 20, 2026 11:57am

Request Review

Only drop singleton page cards that repeat the same page number. Multi-page section chunks that share a boundary page stay visible.

Co-authored-by: Cursor <cursoragent@cursor.com>
@suguanYang
suguanYang merged commit f2f8bd2 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