Skip to content

Adapt Notebook to SDK result-layout parsed storage#138

Merged
suguanYang merged 14 commits into
stagingfrom
fix/wangbinqi/sdk-result-layout-notebook
Jul 7, 2026
Merged

Adapt Notebook to SDK result-layout parsed storage#138
suguanYang merged 14 commits into
stagingfrom
fix/wangbinqi/sdk-result-layout-notebook

Conversation

@suguanYang

@suguanYang suguanYang commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary:

  • add page-asset display/read flows and SDK-first chunk/page-asset routes for parsed documents
  • align BlobParsedDocumentStorage with the SDK object-path ParsedDocumentStorage interface and result-relative asset paths
  • make page-mode source detection/display lazy and tolerant of unavailable remote/display reads
  • keep the app/page route export surface valid for Next generated type checks
  • install published @ontos-ai/knowhere-sdk@2.1.2
  • validate chat citation refs against resolved evidence source metadata and avoid fallback source chips from unselected retrieval results

SDK note:

  • @ontos-ai/knowhere-sdk@2.1.2 is installed from the npm registry and locked in pnpm-lock.yaml.

Tests:

  • pnpm exec vitest run src/agent-harness/validator.test.ts src/domains/chat/index.test.ts
  • pnpm exec vitest run src/agent-harness/runtime.test.ts src/agent-harness/knowhere-text.test.ts
  • pnpm exec vitest run src/domains/chat/service.test.ts
  • pnpm test
  • pnpm run typecheck
  • pnpm lint
  • git diff --check

@vercel

vercel Bot commented Jul 7, 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 7, 2026 8:58pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
knowhere-notebook Ignored Ignored Preview Jul 7, 2026 8:58pm

Request Review

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