Refactor session creation and imports in SKILL.md

[jamesmontemagno]

Refactor session creation in Python and Go examples to use keyword arguments for clarity. Update imports to match new structure.

Pull Request Checklist

• I have read and followed the CONTRIBUTING.md guidelines.
• I have read and followed the Guidance for submissions involving paid services.
• My contribution adds a new instruction, prompt, agent, skill, or workflow file in the correct directory.
• The file follows the required naming convention.
• The content is clearly structured and follows the example format.
• I have tested my instructions, prompt, agent, skill, or workflow with GitHub Copilot.
• I have run npm start and verified that README.md is up to date.
• I am targeting the staged branch for this pull request.

---

Description

---

Type of Contribution

• New instruction file.
• New prompt file.
• New agent file.
• New plugin.
• New skill file.
• New agentic workflow.
• Update to existing instruction, prompt, agent, plugin, skill, or workflow.
• Other (please specify):

---

Additional Notes

---

By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

[github-actions]

⚠️ This PR targets main, but PRs should target staged.

The main branch is auto-published from staged and should not receive direct PRs.
Please close this PR and re-open it against the staged branch.

You can change the base branch using the Edit button at the top of this PR,
or run: gh pr edit 1323 --base staged

[Copilot]

Pull request overview

Updates the Copilot SDK skill documentation to reflect newer session-creation patterns (Python keyword arguments; Go context.Context) and expands MCP server configuration examples.

Changes:

• Refactors Python create_session examples to use keyword arguments and updates PermissionHandler imports.
• Refactors Go examples to pass context.Context into Start, CreateSession, and SendAndWait.
• Adds new MCP Server Integration documentation including local filesystem (stdio) and remote HTTP + headers examples across languages.

Show a summary per file

File	Description
skills/copilot-sdk/SKILL.md	Updates Python/Go session creation examples and adds expanded MCP server integration snippets.

Copilot's findings

Comments suppressed due to low confidence (7)

skills/copilot-sdk/SKILL.md:680

• The Go remote MCP server snippet uses quoted keys ("type", "url", "headers", "tools") inside the copilot.MCPServerConfig literal, which is invalid Go syntax for a struct literal and won’t compile. Use the MCPServerConfig field names (e.g., Type, URL, Headers, Tools) consistent with other Go examples in this repo (e.g., plugins/copilot-sdk/skills/copilot-sdk/SKILL.md).

    MCPServers: map[string]copilot.MCPServerConfig{
        "github": {
            "type": "http",
            "url":  "https://api.githubcopilot.com/mcp/",
            "headers": map[string]string{
                "Authorization": "Bearer " + os.Getenv("GITHUB_TOKEN"),
            },
            "tools": []string{"*"},

skills/copilot-sdk/SKILL.md:182

• The Python example imports PermissionHandler from copilot.session, but the rest of the repository’s Python Copilot SDK examples import it from copilot. To avoid conflicting guidance and potential import errors, align this import with the established pattern or update related docs simultaneously.

from copilot import CopilotClient
from copilot.session import PermissionHandler
from copilot.generated.session_events import SessionEventType

skills/copilot-sdk/SKILL.md:309

• This Python snippet imports PermissionHandler from copilot.session, which differs from other Python Copilot SDK documentation in this repo that imports it from copilot. Please align to a single import path (and update other docs if the module layout changed).

from copilot import CopilotClient
from copilot.session import PermissionHandler
from copilot.tools import define_tool

skills/copilot-sdk/SKILL.md:494

• This Python snippet imports PermissionHandler from copilot.session, but other Python Copilot SDK examples in the repo import it from copilot. For consistency (and to avoid broken copy/paste), align the import path or update the other documentation files in the same PR.

from copilot import CopilotClient
from copilot.session import PermissionHandler
from copilot.tools import define_tool

skills/copilot-sdk/SKILL.md:202

• This example calls await session.send_and_wait("Tell me a short joke"), but other Python Copilot SDK examples in this repo pass MessageOptions or a dict with a prompt key. If send_and_wait does not accept a plain string, this snippet will be incorrect—please align with the repo’s documented usage or update the other docs to match the new API.

    await session.send_and_wait("Tell me a short joke")

skills/copilot-sdk/SKILL.md:342

• This snippet uses await session.send_and_wait("What's the weather like in Seattle and Tokyo?"). Other Python Copilot SDK docs in this repo use MessageOptions(prompt=...) or { "prompt": ... } for send_and_wait. If string prompts aren’t supported, update this to the documented shape (or update the rest of the docs to match).

    await session.send_and_wait("What's the weather like in Seattle and Tokyo?")

skills/copilot-sdk/SKILL.md:539

• The interactive loop calls await session.send_and_wait(user_input) with a raw string. To keep the documentation consistent (and to match the rest of the repo’s Python examples), pass MessageOptions or a dict containing the prompt field unless the SDK explicitly supports string prompts.

        await session.send_and_wait(user_input)

• Files reviewed: 1/1 changed files
• Comments generated: 3

[Copilot]

The Go MCPServers example uses quoted keys (e.g., "type", "command") inside a copilot.MCPServerConfig composite literal. In Go, struct literal field keys must be identifiers (e.g., Type, Command), so this snippet will not compile as written. Update the config literal to use the actual MCPServerConfig struct fields (matching the existing Type/URL usage elsewhere) or explicitly make the value a map type if that’s the intended API.

This issue also appears on line 673 of the same file.

Suggested change

	"type": "stdio",
	"command": "npx",
	"args": []string{"-y", "@modelcontextprotocol/server-filesystem", "."},
	"tools": []string{"*"},
	Type: "stdio",
	Command: "npx",
	Args: []string{"-y", "@modelcontextprotocol/server-filesystem", "."},
	Tools: []string{"*"},

[Copilot]

This skill switches the Python import to from copilot.session import PermissionHandler, but other Copilot SDK Python docs/instructions in this repo consistently import PermissionHandler from copilot (e.g., cookbook/copilot-sdk/python/error-handling.md and instructions/copilot-sdk-python.instructions.md). Unless the package structure has changed everywhere, this example is likely to break or create inconsistency—consider keeping from copilot import PermissionHandler here or updating the rest of the repo docs in the same PR.

This issue also appears in the following locations of the same file:

• line 180
• line 307
• line 492

Suggested change

	from copilot import CopilotClient
	from copilot.session import PermissionHandler
	from copilot import CopilotClient, PermissionHandler

[Copilot]

session.send_and_wait() is shown being called with a bare string prompt. Elsewhere in this repo’s Python Copilot SDK docs, send_and_wait is called with a MessageOptions instance or a dict containing {"prompt": ...} (e.g., cookbook/copilot-sdk/python/error-handling.md). If the API still expects MessageOptions/dict, this example will fail—consider using the established call shape for consistency.

This issue also appears in the following locations of the same file:

• line 202
• line 342
• line 539

Suggested change

	response = await session.send_and_wait("What is 2 + 2?")
	response = await session.send_and_wait({"prompt": "What is 2 + 2?"})

[aaronpowell]

It looks like you've incorrectly branched from the main branch not staged, and as a result all the materialised plugins are included in this PR.

You can attempt to fix this with a rebase:

git fetch origin staged
git rebase --onto origin/staged origin/main <branch name>
git push --force-with-lease

If that does not resolve it, you can run npm run plugin:clean which will delete the materialised plugins and you can commit that change.