Skip to content

Conversation

@friggeri
Copy link
Collaborator

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

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
@friggeri friggeri requested a review from a team as a code owner January 20, 2026 18:32
Copilot AI review requested due to automatic review settings January 20, 2026 18:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 skillDirectories and disabledSkills configuration fields to SessionConfig and ResumeSessionConfig types 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.

@friggeri friggeri enabled auto-merge January 21, 2026 00:03
@friggeri friggeri added this pull request to the merge queue Jan 21, 2026
Merged via the queue into main with commit 2415f6f Jan 21, 2026
22 checks passed
@friggeri friggeri deleted the frg/skills branch January 21, 2026 00:31
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.

3 participants