Skip to content

fix(mcp): clarify page asset reads#113

Merged
suguanYang merged 3 commits into
mainfrom
fix/wangbinqi/mcp-page-assets-guidance
Jul 7, 2026
Merged

fix(mcp): clarify page asset reads#113
suguanYang merged 3 commits into
mainfrom
fix/wangbinqi/mcp-page-assets-guidance

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary

  • Default SDK MCP knowhere_read_chunks display reads to assetUrlPolicy: "durable" when callers do not provide a policy, while preserving explicit policies and leaving targeted reads unset.
  • Clarify that stdio file parse paths are local to the MCP server process, and document page screenshots under chunk metadata.pageAssets instead of image chunks.
  • Add a patch changeset for @ontos-ai/knowhere-mcp.

Public API impact: no input schema change; this is an MCP wrapper default and documentation update.

Verification

  • pnpm --filter @ontos-ai/knowhere-mcp test:ci -- src/tests/mcp.test.ts
  • pnpm --filter @ontos-ai/knowhere-mcp typecheck
  • pnpm --filter @ontos-ai/knowhere-mcp format:check
  • pnpm --filter @ontos-ai/knowhere-mcp lint (passes with existing no-console warnings in src/stdio.ts)
  • pnpm --filter @ontos-ai/knowhere-mcp build
  • git diff --check

Checklist

  • Tests were added or updated when behavior changed
  • Public docs or examples were updated when needed
  • A changeset was added when a release should be published

@suguanYang suguanYang force-pushed the fix/wangbinqi/mcp-page-assets-guidance branch from 383302f to e2bc39a Compare July 7, 2026 15:41
@suguanYang suguanYang merged commit df7d85b into main Jul 7, 2026
4 checks passed
@github-actions github-actions Bot mentioned this pull request Jul 7, 2026
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