Skip to content

Align remote source processed row state#128

Merged
suguanYang merged 1 commit into
stagingfrom
fix/wangbinqi/remote-source-processed-default
Jul 5, 2026
Merged

Align remote source processed row state#128
suguanYang merged 1 commit into
stagingfrom
fix/wangbinqi/remote-source-processed-default

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary:

  • Show unlocalized remote sources as Processed without chunk counts.
  • Default remote source rows to included and let ready remote rows use the normal checkbox toggle path.
  • Decode unchecked knowhere-doc source IDs into retrieval exclude document IDs.

Verification:

  • pnpm test src/components/source-row.test.ts src/domains/sources/route-service.test.ts src/domains/chat/index.test.ts
  • pnpm typecheck (passes; prints existing Effect.void diagnostic in src/domains/sources/counts.ts)

@vercel

vercel Bot commented Jul 5, 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, Comment Jul 5, 2026 10:08am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
knowhere-notebook Ignored Ignored Preview Jul 5, 2026 10:08am

Request Review

@suguanYang
suguanYang merged commit b17f8b9 into staging Jul 5, 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