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: 82c4930
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: adaa6a1
Choose a head ref
  • 10 commits
  • 17 files changed
  • 4 contributors

Commits on Dec 10, 2025

  1. Correct lower-case issue state (#1567)

    * Correct lower-case issue state
    
    * Uppercase orderBy and direction as well
    
    * Auto-correct invalid enum parameters
    almaleksia authored Dec 10, 2025
    Configuration menu
    Copy the full SHA
    cfd7ef9 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    90a1255 View commit details
    Browse the repository at this point in the history

Commits on Dec 11, 2025

  1. Add support for safe tool renaming (#1563)

    * add suppport for tool aliases for backwards compatibility when tool names change
    
    * cleanup
    
    * log alias usage as warning
    
    * remove comments
    
    * remove mock data
    
    * remove unused code, move deprecated tool aliases to its own file
    
    * remove unused code and add tests
    
    * resolve tool aliases in its own explicit step
    
    * improve logic by returning aliases used in resolvetoolaliases
    
    * remove unused function
    
    * remove comments
    
    * remove comment
    
    * Update pkg/github/deprecated_tool_aliases.go
    
    Co-authored-by: Copilot <[email protected]>
    
    * restore comment
    
    ---------
    
    Co-authored-by: Copilot <[email protected]>
    tommaso-moro and Copilot authored Dec 11, 2025
    Configuration menu
    Copy the full SHA
    94ee074 View commit details
    Browse the repository at this point in the history

Commits on Dec 12, 2025

  1. fix: indent multi-line descriptions in generated docs

    When tool parameter descriptions span multiple lines, the continuation
    lines now receive proper indentation to maintain markdown list formatting.
    
    This fixes the rendering issue where multi-line descriptions would break
    out of the parameter list structure.
    
    Fixes #1494
    SamMorrowDrums committed Dec 12, 2025
    Configuration menu
    Copy the full SHA
    4cbbf6f View commit details
    Browse the repository at this point in the history
  2. fix: update get-me script for go-SDK MCP protocol

    The go-SDK migration changed MCP protocol handling to require proper
    initialization before tool calls. This updates the script to:
    
    - Add initialize request with protocol version and client info
    - Add notifications/initialized notification
    - Add arguments field to tools/call params
    - Keep stdin open with sleep for response
    - Gracefully handle missing jq dependency
    SamMorrowDrums committed Dec 12, 2025
    Configuration menu
    Copy the full SHA
    5da71e3 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    5cd95dc View commit details
    Browse the repository at this point in the history
  4. 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.
    SamMorrowDrums committed Dec 12, 2025
    Configuration menu
    Copy the full SHA
    87ea319 View commit details
    Browse the repository at this point in the history
  5. 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
    SamMorrowDrums committed Dec 12, 2025
    Configuration menu
    Copy the full SHA
    3a1844c View commit details
    Browse the repository at this point in the history
  6. 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
    SamMorrowDrums committed Dec 12, 2025
    Configuration menu
    Copy the full SHA
    e7e2925 View commit details
    Browse the repository at this point in the history
  7. 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.
    SamMorrowDrums committed Dec 12, 2025
    2 Configuration menu
    Copy the full SHA
    adaa6a1 View commit details
    Browse the repository at this point in the history
Loading