Repository navigation
Conversation
Deploying ext-skills with
|
| Latest commit: |
87cd708
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://d7f1f8d1.ext-skills.pages.dev |
| Branch Preview URL: | https://fix-156-request-params-types.ext-skills.pages.dev |
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The proposal correctly follows repository governance and resolves the reported type inconsistencies.
0 open findings
What changed in this PR
Proposes a type-safe correction to the next Skills specification revision.
Changes:
- Adds named request-parameter interfaces requiring
_meta. - Aligns request inheritance with the base schema.
- Updates the referenced base-type list.
| File | Description |
|---|---|
docs/decisions.md |
Records the proposed schema-rendering correction and rationale. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
sambhav
force-pushed
the
fix/156-request-params-types
branch
from
October 11, 2026 12:40
daca994 to
163c540
Compare
sambhav
force-pushed
the
fix/156-request-params-types
branch
from
October 11, 2026 12:43
163c540 to
7457ff9
Compare
Named params for skills/get and resources/directory/read, cacheable directory results, *ResultResponse types, optional _meta on Skill, full _meta in request examples. Records the decision and re-anchors index.md. Fixes #156 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01M8b7Vuu2jRefMhoZB3BrrX
sambhav
force-pushed
the
fix/156-request-params-types
branch
from
October 11, 2026 12:49
7457ff9 to
87cd708
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Fixes #156.
Aligns
specification/stable/skills.mdxwith the2026-07-28base schema and records the decision.skills/get,resources/directory/read: named params interfaces, so_metais requiredReadResourceDirectoryResultextendsCacheableResult(breaking:ttlMs,cacheScoperequired)*ResultResponsetypes for each method; optional_metaonSkill_metaWhy this lands in stable
These are corrections to the published page rather than new design:
2026-07-28schema, andReadResourceDirectoryRequestfails to type-check against it (skills.mdx: ReadResourceDirectoryRequest does not type-check against the 2026-07-28 base schema #156)._metawas already required. The page's text says every request MUST include it. Only the types and examples disagreed.skills/listandskills/getcacheable so that every result type in the extension stays uniform with the base protocol;resources/directory/readwas the one it didn't catch. This is the only change that adds a requirement on servers, and it affects an optional method behinddirectoryRead: true.Corrections like these have landed on stable in place before (#139, and #147 after SEP-2640 went Final), so I've followed that route rather than opening a
draft/revision.index.mdexcerpt ranges are re-anchored to the new line numbers.Verification
All interfaces and all 16 examples type-check under
--strictagainst the base schema;docs:buildpasses.🤖 Generated with Claude Code
https://claude.ai/code/session_01M8b7Vuu2jRefMhoZB3BrrX
Generated by Claude Code