Skip to content

Conversation

@mattdholloway
Copy link
Contributor

@mattdholloway mattdholloway commented Jan 2, 2026

Summary

This pull request updates the way documentation is generated for available GitHub toolsets. The change is that the generated documentation now only includes tools that are normally available to users, rather than polluting the documentation by listing all possible tools. This results in not including those that are disabled or behind feature flags.

Why

What changed

Documentation and Tool Availability Updates:

  • Removed documentation for the following GitHub Actions tools from README.md: actions_get, actions_list, actions_run_trigger, and get_job_logs, as these are no longer available to regular users. [1] [2]
  • Updated the tool inventory build process in generate_docs.go to include only tools available to regular users by specifying the toolset filter WithToolsets([]string{"all"}).

MCP impact

  • No tool or API changes
  • Tool schema or behavior changed
  • New tool added

Prompts tested (tool changes only)

Security / limits

  • No security or limits impact
  • Auth / permissions considered
  • Data exposure, filtering, or token/size limits considered

Tool renaming

  • I am renaming tools as part of this PR (e.g. a part of a consolidation effort)
    • I have added the new tool aliases in deprecated_tool_aliases.go
  • I am not renaming tools as part of this PR

Note: if you're renaming tools, you must add the tool aliases. For more information on how to do so, please refer to the official docs.

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed
  • Updated (README / docs / examples)

@mattdholloway mattdholloway changed the title exclude tools requiring ff from docs Exclude tools requiring ff from docs Jan 2, 2026
@mattdholloway mattdholloway changed the title Exclude tools requiring ff from docs Exclude tools requiring feature flag from docs Jan 2, 2026
Copy link
Collaborator

@SamMorrowDrums SamMorrowDrums left a comment

Choose a reason for hiding this comment

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

The outcome is correct, but I think we could use existing filters to do the filtering, by providing an always false feature flag checker.

@mattdholloway mattdholloway marked this pull request as ready for review January 2, 2026 13:44
@mattdholloway mattdholloway requested a review from a team as a code owner January 2, 2026 13:44
Copilot AI review requested due to automatic review settings January 2, 2026 13:44
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 PR modifies the documentation generation logic to exclude tools that require feature flags (marked with FeatureFlagEnable) from the generated README.md documentation, as these tools are not available to regular users. The change ensures that consolidated actions tools, which are behind a feature flag, don't appear in the public documentation.

Key Changes

  • Modified documentation generation to use AvailableTools() with feature flag filtering instead of AllTools()
  • Explicitly set all toolsets to be enabled during documentation generation
  • Tools requiring feature flags (FeatureFlagEnable) are now excluded from the README

Reviewed changes

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

File Description
cmd/github-mcp-server/generate_docs.go Updated inventory building to explicitly enable all toolsets and changed from AllTools() to AvailableTools() to respect feature flag filtering
README.md Removed documentation for 4 consolidated actions tools (actions_get, actions_list, actions_run_trigger, get_job_logs) that require the FeatureFlagConsolidatedActions feature flag

mattdholloway and others added 2 commits January 2, 2026 13:47
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

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.

4 participants