Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: github/github-mcp-server
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 311d581
Choose a base ref
...
head repository: github/github-mcp-server
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 9426075
Choose a head ref
  • 18 commits
  • 22 files changed
  • 14 contributors

Commits on Dec 19, 2025

  1. Add aliases for new actions tools (#1652)

    * add aliases for new actions tools
    
    * generate docs
    tommaso-moro authored Dec 19, 2025
    Configuration menu
    Copy the full SHA
    63c7db0 View commit details
    Browse the repository at this point in the history
  2. Add raw client error annotation and annotate GetFileContents (#1570)

    * Add raw client error annotation and annotate GetFileContents
    
    * Track response.
    
    * add raw errors to context
    
    * add raw api error test
    
    * Update pkg/errors/error.go
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Add blank line after Error() method for readability
    
    * add NewGitHubRawAPIErrorResponse back
    
    ---------
    
    Co-authored-by: Matt Holloway <mattdholloway@pm.me>
    Co-authored-by: Matt Holloway <mattdholloway@github.com>
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    4 people authored Dec 19, 2025
    Configuration menu
    Copy the full SHA
    b79d126 View commit details
    Browse the repository at this point in the history

Commits on Dec 22, 2025

  1. feat: Add DestructiveHint to delete_project_item tool annotation

    Add DestructiveHint: true to the delete_project_item tool to be
    consistent with other delete operations (delete_file and
    delete_workflow_run_logs) that properly indicate destructive behavior.
    
    This helps LLMs better understand that this tool permanently removes
    data and should be used with appropriate caution.
    
    Co-Authored-By: Claude <noreply@anthropic.com>
    2 people authored and SamMorrowDrums committed Dec 22, 2025
    Configuration menu
    Copy the full SHA
    844137c View commit details
    Browse the repository at this point in the history
  2. chore: Update toolsnap for delete_project_item with destructiveHint

    Add destructiveHint: true to the snapshot file to match the code change.
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    
    Co-Authored-By: Claude <noreply@anthropic.com>
    2 people authored and SamMorrowDrums committed Dec 22, 2025
    Configuration menu
    Copy the full SHA
    6c0bbb8 View commit details
    Browse the repository at this point in the history
  3. get_file_contents fetch refs improvements (#1655)

    * get_file_contents improvements
    
    * Return custom errors when main ref is supplied
    
    * Remove test for short sha
    
    * Apply Copilot suggestion
    almaleksia authored Dec 22, 2025
    Configuration menu
    Copy the full SHA
    2f31c15 View commit details
    Browse the repository at this point in the history
  4. remove experiments toolset

    tommaso-moro authored and SamMorrowDrums committed Dec 22, 2025
    Configuration menu
    Copy the full SHA
    b5be18c View commit details
    Browse the repository at this point in the history
  5. build(deps): bump docker/setup-buildx-action from 3.11.1 to 3.12.0

    Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 3.11.1 to 3.12.0.
    - [Release notes](https://github.com/docker/setup-buildx-action/releases)
    - [Commits](docker/setup-buildx-action@e468171...8d2750c)
    
    ---
    updated-dependencies:
    - dependency-name: docker/setup-buildx-action
      dependency-version: 3.12.0
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    dependabot[bot] authored and SamMorrowDrums committed Dec 22, 2025
    Configuration menu
    Copy the full SHA
    f9ef72f View commit details
    Browse the repository at this point in the history
  6. feat: auto-fix license files on PRs and improve CI reliability (#1583)

    * fix: handle architecture-specific license differences
    
    The licenses script now:
    - Generates separate license reports per GOOS/GOARCH combination
    - Groups identical reports together (comma-separated arch names)
    - Adds a Table of Contents at the top of each platform file
    - Handles cases where different architectures have different dependencies
      (e.g., x/sys/unix vs x/sys/windows, mousetrap on Windows only)
    
    This addresses the issue discovered in cli/cli where some deps changed
    which changed the mod graph for different GOARCH and affected the
    exported licenses because go-licenses tries to find common ancestors.
    
    * fix: make license script portable and deterministic
    
    Address review feedback:
    - Remove bash 4.0+ associative array requirement for macOS compatibility
    - Add cross-platform hash function (md5sum on Linux, md5 on macOS)
    - Ensure deterministic iteration order using sorted groups file
    - Add better error handling for failed go-licenses commands
    - Fix grammar: 'architecture(s)' -> 'architectures'
    - Add documentation for third-party/ being a union of all architectures
    - Use file-based state instead of associative arrays for portability
    
    * fix: update licenses-check to use new architecture-aware format
    
    - Check now regenerates using ./script/licenses and compares
    - Add GOROOT/PATH setup in CI to fix go-licenses module info errors
    - Check both license files AND third-party directory for changes
    - See: google/go-licenses#244
    
    * fix: use LC_ALL=C for consistent sorting across systems
    
    The sort command uses locale-specific ordering which can differ between
    systems. Use LC_ALL=C to ensure consistent ordering in CI and locally.
    
    * feat: auto-fix license files on PRs and improve CI reliability
    
    Changes:
    - Pin go-licenses version in CI for reproducibility (commit 5348b744)
    - Add GOROOT/PATH setup for 'Package does not have module info' fix
    - Update license-check.yml to auto-fix and push to PR branches
    - Add CI=true env var to use pinned go-licenses version
    - Add dependabot exclusion from auto-fix workflow
    - Add code-scanning exclusion for third-party files
    
    * feat: auto-close PRs that only needed license updates
    
    After the bot pushes license fixes, check if the PR now only contains
    license file changes. If so, close it automatically with a comment
    explaining that the license updates are complete.
    
    This prevents stale PRs from accumulating when someone creates a PR
    just to fix licenses, or when all other changes were already merged
    to the base branch.
    
    * feat: auto-create/manage license fix PRs for failing PRs
    
    Creates stacked PRs to fix license issues:
    - Detects when a PR needs license updates
    - Creates child PR: main <- PR:feature <- PR:license-fix
    - Tracks PRs with metadata and hash of license changes
    - Auto-closes if user fixes licenses manually
    - Auto-closes and recreates if dependencies change
    - Prevents multiple fix PRs for same base PR
    
    Rules:
    - Only targets PRs against main (not stacked PRs)
    - Only runs on ready-for-review PRs (not drafts)
    - Skips bots and forks
    - Hash-based detection avoids unnecessary work
    
    * fix: allow auto-fix workflow to run on dependabot PRs
    
    Dependabot PRs frequently need license updates and can't be merged until
    fixed. The auto-fix workflow helps by creating a child PR with the
    license changes, making it easy to merge both together.
    
    * fix: address Copilot review comments
    
    - Remove dependabot exclusion (we want to support dependabot PRs)
    - Comment indentation already fixed
    - CI env var already set for reproducibility
    
    * refactor: move base branch filter to on: block
    
    Moved the 'targets main' check from job if: to workflow on.pull_request.branches.
    This prevents the workflow from even triggering for PRs targeting other branches,
    saving CI resources.
    
    Draft check is implicit in the types list (opened + ready_for_review).
    Fork check must stay in if: condition (can't be filtered in on: block).
    
    * refactor: merge auto-fix into license-check workflow
    
    Combines both workflows into one with two jobs:
    1. license-check: Checks licenses, fails if needed, sets outputs
    2. auto-create-fix-pr: Creates child PR if needed (only for non-forks)
    
    Benefits:
    - Single workflow file, easier to maintain
    - Check fails (blocks merge) while still creating helpful fix PR
    - Fork detection in first job, second job skips for forks
    - Hash-based tracking prevents duplicate PRs
    
    * refactor: simplify license-check to auto-commit approach
    
    Much simpler workflow:
    1. Always try to auto-commit fix directly to PR branch
    2. If push fails (fork without permissions), comment once with instructions
    3. Don't create child PRs - just fix in place or give instructions
    4. Only comment if not already commented (prevent spam)
    5. Always fail check if licenses need updating
    
    Benefits:
    - Much simpler - single job
    - No child PR management complexity
    - Clear UX: either fixed or instructed
    - Works for all PRs (internal/fork/dependabot)
    SamMorrowDrums authored Dec 22, 2025
    Configuration menu
    Copy the full SHA
    4f064a3 View commit details
    Browse the repository at this point in the history
  7. Fix YAML syntax errors in license-check workflow

    - Use separate -m flags for multi-line git commit message
    - Add proper indentation to template literal content to fix YAML parsing
    
    Co-authored-by: SamMorrowDrums <4811358+SamMorrowDrums@users.noreply.github.com>
    Copilot and SamMorrowDrums committed Dec 22, 2025
    Configuration menu
    Copy the full SHA
    2bfe467 View commit details
    Browse the repository at this point in the history
  8. build(deps): bump github.com/google/jsonschema-go from 0.3.0 to 0.4.2

    Bumps [github.com/google/jsonschema-go](https://github.com/google/jsonschema-go) from 0.3.0 to 0.4.2.
    - [Release notes](https://github.com/google/jsonschema-go/releases)
    - [Commits](google/jsonschema-go@v0.3.0...v0.4.2)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/google/jsonschema-go
      dependency-version: 0.4.2
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    dependabot[bot] authored and SamMorrowDrums committed Dec 22, 2025
    Configuration menu
    Copy the full SHA
    6a3c306 View commit details
    Browse the repository at this point in the history
  9. chore: regenerate license files

    Auto-generated by license-check workflow
    github-actions[bot] authored and SamMorrowDrums committed Dec 22, 2025
    Configuration menu
    Copy the full SHA
    e51e6b1 View commit details
    Browse the repository at this point in the history
  10. build(deps): bump github.com/modelcontextprotocol/go-sdk

    Bumps [github.com/modelcontextprotocol/go-sdk](https://github.com/modelcontextprotocol/go-sdk) from 1.2.0-pre.1 to 1.2.0.
    - [Release notes](https://github.com/modelcontextprotocol/go-sdk/releases)
    - [Commits](modelcontextprotocol/go-sdk@v1.2.0-pre.1...v1.2.0)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/modelcontextprotocol/go-sdk
      dependency-version: 1.2.0
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    dependabot[bot] authored and SamMorrowDrums committed Dec 22, 2025
    Configuration menu
    Copy the full SHA
    6d1ce24 View commit details
    Browse the repository at this point in the history
  11. chore: regenerate license files

    Auto-generated by license-check workflow
    github-actions[bot] authored and SamMorrowDrums committed Dec 22, 2025
    Configuration menu
    Copy the full SHA
    b28d7fb View commit details
    Browse the repository at this point in the history
  12. Use typed IconTheme constants from Go SDK

    Replace string conversions with mcp.IconThemeLight and mcp.IconThemeDark
    constants to match the SDK's typed IconTheme field. This fixes the CI build
    errors where string literals were being used instead of the proper IconTheme type.
    
    Co-authored-by: SamMorrowDrums <4811358+SamMorrowDrums@users.noreply.github.com>
    Copilot and SamMorrowDrums committed Dec 22, 2025
    Configuration menu
    Copy the full SHA
    8ac1eb4 View commit details
    Browse the repository at this point in the history

Commits on Dec 23, 2025

  1. Update PR template to include tool renaming section (#1657)

    * update pr template
    
    * fix
    tommaso-moro authored Dec 23, 2025
    Configuration menu
    Copy the full SHA
    95a637e View commit details
    Browse the repository at this point in the history
  2. build(deps): bump github.com/spf13/cobra from 1.10.1 to 1.10.2

    Bumps [github.com/spf13/cobra](https://github.com/spf13/cobra) from 1.10.1 to 1.10.2.
    - [Release notes](https://github.com/spf13/cobra/releases)
    - [Commits](spf13/cobra@v1.10.1...v1.10.2)
    
    ---
    updated-dependencies:
    - dependency-name: github.com/spf13/cobra
      dependency-version: 1.10.2
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    dependabot[bot] authored and SamMorrowDrums committed Dec 23, 2025
    Configuration menu
    Copy the full SHA
    a1bc8b6 View commit details
    Browse the repository at this point in the history
  3. chore: regenerate license files

    Auto-generated by license-check workflow
    github-actions[bot] authored and SamMorrowDrums committed Dec 23, 2025
    Configuration menu
    Copy the full SHA
    7e32623 View commit details
    Browse the repository at this point in the history
  4. Fallback to default branch in get_file_contents when main doesn't exi…

    …st (#1669)
    
    * Fallback to default branch in get_file_contents when main doesn't exist
    
    * Addressing review comments
    almaleksia authored Dec 23, 2025
    Configuration menu
    Copy the full SHA
    9426075 View commit details
    Browse the repository at this point in the history
Loading