feat: add skillDirectories and disabledSkills to all SDKs#57
Merged
Conversation
Expose skill configuration options from CLI server to SDK clients: - skillDirectories: directories to load skills from - disabledSkills: list of skill names to disable Updated SDKs: - Node.js: SessionConfig, ResumeSessionConfig types and client - Go: SessionConfig, ResumeSessionConfig structs and client - .NET: SessionConfig, ResumeSessionConfig classes and client - Python: SessionConfig, ResumeSessionConfig TypedDicts and client
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request adds skill configuration options (skillDirectories and disabledSkills) to all SDK clients, exposing these options from the CLI server to SDK consumers. The changes are consistent across all four SDK implementations (Node.js, Go, .NET, and Python).
Changes:
- Added
skillDirectoriesanddisabledSkillsconfiguration fields toSessionConfigandResumeSessionConfigtypes in all SDKs - Updated client implementations to serialize and send these fields to the server
- Updated Node.js package dependency to @github/copilot version 0.0.388-1
- Removed playground linting references from justfile
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| python/copilot/types.py | Added skill_directories and disabled_skills fields to SessionConfig and ResumeSessionConfig TypedDicts |
| python/copilot/client.py | Added serialization logic to send skillDirectories and disabledSkills in create_session and resume_session methods |
| nodejs/src/types.ts | Added skillDirectories and disabledSkills optional fields to SessionConfig interface and included them in ResumeSessionConfig Pick type |
| nodejs/src/client.ts | Added skillDirectories and disabledSkills to request payloads in createSession and resumeSession methods |
| nodejs/package.json | Updated @github/copilot dependency from ^0.0.387 to ^0.0.388-1 |
| nodejs/package-lock.json | Updated lockfile to reflect new @github/copilot version and platform-specific binaries |
| go/types.go | Added SkillDirectories and DisabledSkills fields to SessionConfig and ResumeSessionConfig structs with documentation |
| go/client.go | Added serialization logic to send skillDirectories and disabledSkills in CreateSession and ResumeSessionWithOptions methods |
| dotnet/src/Types.cs | Added SkillDirectories and DisabledSkills properties to SessionConfig and ResumeSessionConfig classes with XML documentation |
| dotnet/src/Client.cs | Updated CreateSessionRequest and ResumeSessionRequest records to include SkillDirectories and DisabledSkills parameters |
| justfile | Removed playground linting commands from lint-nodejs target |
Files not reviewed (1)
- nodejs/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
SteveSandersonMS
approved these changes
Jan 21, 2026
This was referenced Jan 22, 2026
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.
Expose skill configuration options from CLI server to SDK clients:
Updated SDKs: