Skip to content

docs: update go sdk examples#1393

Open
caarlos0 wants to merge 1 commit intogithub:stagedfrom
caarlos0:update-go-sdk-examples
Open

docs: update go sdk examples#1393
caarlos0 wants to merge 1 commit intogithub:stagedfrom
caarlos0:update-go-sdk-examples

Conversation

@caarlos0
Copy link
Copy Markdown

@caarlos0 caarlos0 commented Apr 14, 2026

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

Updated the Go copilot-sdk examples to the latest version, as well as changed the models to latest versions.
Updated the copilot sdk skill as well, as it the MCP configuration in there was incorrect.
Also removed a seemingly dangling go.sum file.


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.

Copilot AI review requested due to automatic review settings April 14, 2026 14:01
@caarlos0 caarlos0 requested a review from aaronpowell as a code owner April 14, 2026 14:01
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ 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 1393 --base staged

@caarlos0 caarlos0 changed the base branch from main to staged April 14, 2026 14:10
@caarlos0 caarlos0 requested a review from dvelton as a code owner April 14, 2026 14:10
@caarlos0 caarlos0 force-pushed the update-go-sdk-examples branch from 8f5e9c2 to aa8acb0 Compare April 14, 2026 14:11
Copy link
Copy Markdown
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

Updates the Go copilot-sdk cookbook examples to align with newer SDK APIs and newer model identifiers, and removes an unused Go checksum file.

Changes:

  • Update example session lifecycle calls (e.g., Destroy()Disconnect()) and event handling to newer SDK types.
  • Refresh model names used across Go recipes and their rendered markdown docs.
  • Remove a dangling cookbook/copilot-sdk/go.sum file.

Reviewed changes

Copilot reviewed 14 out of 15 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
cookbook/copilot-sdk/go/recipe/ralph-loop.go Updates model name and session/event APIs for the runnable recipe.
cookbook/copilot-sdk/go/recipe/pr-visualization.go Updates model name and event handling to newer typed event data.
cookbook/copilot-sdk/go/recipe/persisting-sessions.go Updates model name, session cleanup, and ListSessions call signature.
cookbook/copilot-sdk/go/recipe/multiple-sessions.go Updates models and session cleanup calls.
cookbook/copilot-sdk/go/recipe/managing-local-files.go Updates model, session cleanup, and typed event handling.
cookbook/copilot-sdk/go/recipe/error-handling.go Updates model, session cleanup, and typed message extraction.
cookbook/copilot-sdk/go/recipe/accessibility-report.go Updates session config/event typing and MCP server configuration approach.
cookbook/copilot-sdk/go/ralph-loop.md Keeps documentation in sync with updated recipe APIs/models.
cookbook/copilot-sdk/go/pr-visualization.md Keeps documentation in sync with updated recipe APIs/models.
cookbook/copilot-sdk/go/persisting-sessions.md Keeps documentation in sync with updated recipe APIs/models and ListSessions signature.
cookbook/copilot-sdk/go/multiple-sessions.md Keeps documentation in sync with updated recipe APIs/models.
cookbook/copilot-sdk/go/managing-local-files.md Keeps documentation in sync with updated recipe APIs/models and typed event data.
cookbook/copilot-sdk/go/error-handling.md Keeps documentation in sync with updated recipe APIs/models and typed message extraction.
cookbook/copilot-sdk/go/accessibility-report.md Keeps documentation in sync with updated recipe APIs/models and MCP server configuration.
cookbook/copilot-sdk/go.sum Removes an unused checksum file in a directory without a Go module.

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 force-pushed the update-go-sdk-examples branch from aa8acb0 to 1ef5f71 Compare April 14, 2026 14:49
@github-actions
Copy link
Copy Markdown
Contributor

🔍 Skill Validator Results

⛔ Findings need attention

Scope Checked
Skills 1
Agents 1
Total 2
Severity Count
--- ---:
❌ Errors 1
⚠️ Warnings 1
ℹ️ Advisories 0

Summary

Level Finding
[copilot-sdk] SKILL.md body is 910 lines — maximum is 500. Move detailed reference material to separate files.
Full validator output ```text Found 1 skill(s) [copilot-sdk] 📊 copilot-sdk: 5,749 BPE tokens [chars/4: 6,042] (comprehensive ✗), 63 sections, 41 code blocks ❌ [copilot-sdk] SKILL.md body is 910 lines — maximum is 500. Move detailed reference material to separate files. [copilot-sdk] ⚠ Skill is 5,749 BPE tokens (chars/4 estimate: 6,042) — "comprehensive" skills hurt performance by 2.9pp on average. Consider splitting into 2–3 focused skills. {Ansi.Red}Skill spec conformance failures — fix the errors above.{Ansi.Reset} ```
> **Note:** The validator returned a non-zero exit code. Please review the findings above before merge.

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.

2 participants