Skip to content

test: cover referenced backend skill reads without an executor - #27404

Merged
jif-oai merged 1 commit into
mainfrom
jif/external-plugins-5
Jun 11, 2026
Merged

test: cover referenced backend skill reads without an executor#27404
jif-oai merged 1 commit into
mainfrom
jif/external-plugins-5

Conversation

@jif-oai

@jif-oai jif-oai commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Why

PR #27388 lets models read child resources referenced by backend plugin skills without an executor. The integration fixture should prove that real flow: the injected SKILL.md advertises a child skill:// resource, and skills.read resolves that exact resource through the backend provider.

This is stacked on #27388.

What changed

  • Adds a child-resource link to the backend skill fixture and asserts that it reaches model context.
  • Tightens the end-to-end skills test around skills.list followed by skills.read for the referenced resource.
  • Splits the existing app-server mcpResource/read coverage into a focused test so the generic RPC path remains covered independently.

Validation

  • Adds app-server integration coverage for both the referenced backend skill resource and the generic MCP resource read path.

@jif-oai jif-oai changed the title test: add e2e test test: cover referenced backend skill reads without an executor Jun 10, 2026
@jif-oai
jif-oai force-pushed the jif/external-plugins-4b branch from cb32cc8 to 74066fb Compare June 11, 2026 09:44
@jif-oai
jif-oai requested a review from a team as a code owner June 11, 2026 09:44
Base automatically changed from jif/external-plugins-4b to main June 11, 2026 10:38
@jif-oai
jif-oai force-pushed the jif/external-plugins-5 branch from 1191e92 to 68dc3b7 Compare June 11, 2026 10:48
@jif-oai
jif-oai merged commit dd46ef1 into main Jun 11, 2026
30 checks passed
@jif-oai
jif-oai deleted the jif/external-plugins-5 branch June 11, 2026 10:56
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant