Skip to content

[slides] Fix default toolsets comment in slides#22722

Merged
pelikhan merged 1 commit intomainfrom
slides/fix-toolsets-comment-3238ad0cbcdcf0ea
Mar 24, 2026
Merged

[slides] Fix default toolsets comment in slides#22722
pelikhan merged 1 commit intomainfrom
slides/fix-toolsets-comment-3238ad0cbcdcf0ea

Conversation

@github-actions
Copy link
Contributor

Slide Deck Updates

Changes Made

  • Fixed incorrect comment on toolsets: [default] in the MCP Servers Configuration slide

Content Sources Reviewed

  • Scanned pkg/workflow/github_toolsets.go — authoritative source showing default expands to ["context", "repos", "issues", "pull_requests"]
  • Reviewed docs/src/content/docs/guides/getting-started-mcp.md — confirms users is excluded from default because GitHub Actions tokens don't support user operations
  • Reviewed docs/src/content/docs/reference/github-tools.md for toolset documentation

Factual Error Fixed

The slide previously had:

toolsets: [default]  # context, repos, issues, pull_requests, users

Corrected to:

toolsets: [default]  # context, repos, issues, pull_requests

The users toolset was incorrectly listed. Per the code (DefaultGitHubToolsets) and documentation, the default toolset does not include users because GitHub Actions tokens do not support user operations.

Layout Issues

No Playwright layout issues detected (build tools unavailable in this environment).

Focus Mode

Global Sweep


Verification: Content cross-checked against authoritative Go source code and documentation.

Generated by Slide Deck Maintainer ·

  • expires on Mar 25, 2026, 4:31 PM UTC

The 'default' toolset expands to context, repos, issues, pull_requests.
The 'users' toolset was incorrectly included - it is excluded because
GitHub Actions tokens do not support user operations.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions bot added documentation Improvements or additions to documentation slides labels Mar 24, 2026
@pelikhan pelikhan marked this pull request as ready for review March 24, 2026 16:55
Copilot AI review requested due to automatic review settings March 24, 2026 16:55
@pelikhan pelikhan merged commit 43dc132 into main Mar 24, 2026
1 check passed
@pelikhan pelikhan deleted the slides/fix-toolsets-comment-3238ad0cbcdcf0ea branch March 24, 2026 16:56
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

Updates the slide deck’s GitHub MCP configuration example to correct the inline comment describing what toolsets: [default] expands to.

Changes:

  • Removed users from the slide comment listing the expanded default GitHub toolsets.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

tools:
github:
toolsets: [default] # context, repos, issues, pull_requests, users
toolsets: [default] # context, repos, issues, pull_requests
Copy link

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

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

This slide now states that toolsets: [default] expands to context, repos, issues, pull_requests, but the reference docs currently say the default includes users (see docs/src/content/docs/reference/github-tools.md where it lists Default: context, repos, issues, pull_requests, users). To avoid conflicting guidance, either clarify in the slide that this is the GitHub Actions/action-friendly default, or align the reference documentation with the current DefaultGitHubToolsets behavior.

Suggested change
toolsets: [default] # context, repos, issues, pull_requests
toolsets: [default] # GitHub Actions–friendly default: context, repos, issues, pull_requests

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation slides

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants