Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
24b9cbb
refactor(security): migrate code_scanning, secret_scanning, dependabo…
SamMorrowDrums Dec 13, 2025
53f502c
refactor(discussions): migrate to NewTool pattern
SamMorrowDrums Dec 13, 2025
027c5a6
Refactor security_advisories tools to use NewTool pattern
SamMorrowDrums Dec 13, 2025
dbc73d5
refactor: convert projects, labels, and dynamic_tools to NewTool pattern
SamMorrowDrums Dec 13, 2025
1ba8bff
Add --features CLI flag for feature flag support
SamMorrowDrums Dec 13, 2025
59a32bb
Add validation tests for tools, resources, and prompts metadata
SamMorrowDrums Dec 13, 2025
fc8a843
Fix default toolsets behavior when not in dynamic mode
SamMorrowDrums Dec 13, 2025
48a674d
refactor: address PR review feedback for toolsets
SamMorrowDrums Dec 14, 2025
71b5f58
refactor: Apply HandlerFunc pattern to resources for stateless NewToo…
SamMorrowDrums Dec 14, 2025
edc99de
refactor: simplify ForMCPRequest switch cases
SamMorrowDrums Dec 14, 2025
f1b364d
refactor(generate_docs): use strings.Builder and AllTools() iteration
SamMorrowDrums Dec 14, 2025
d2bdbcf
feat(toolsets): add AvailableToolsets() with exclude filter
SamMorrowDrums Dec 14, 2025
83bbe87
refactor(generate_docs): hoist success logging to generateAllDocs
SamMorrowDrums Dec 14, 2025
b907604
refactor: consolidate toolset validation into ToolsetGroup
SamMorrowDrums Dec 14, 2025
02c9863
refactor: rename toolsets package to registry with builder pattern
SamMorrowDrums Dec 15, 2025
d73d98b
fix: remove unnecessary type arguments in helper_test.go
SamMorrowDrums Dec 15, 2025
a0fee3c
fix: restore correct behavior for --tools and --toolsets flags
SamMorrowDrums Dec 15, 2025
c57bae0
Move labels tools to issues toolset
SamMorrowDrums Dec 15, 2025
3a614e7
Restore labels toolset with get_label in both issues and labels
SamMorrowDrums Dec 15, 2025
a735a82
Fix instruction generation and capability advertisement
SamMorrowDrums Dec 15, 2025
34db085
Add tests for dynamic toolset management tools
SamMorrowDrums Dec 15, 2025
eb10ce0
Advertise all capabilities in dynamic toolsets mode
SamMorrowDrums Dec 15, 2025
956a9cd
Improve conformance test with dynamic tool calls and JSON normalization
SamMorrowDrums Dec 15, 2025
14d6a9e
Add conformance-report to .gitignore
SamMorrowDrums Dec 15, 2025
236ace1
Add conformance test CI workflow
SamMorrowDrums Dec 15, 2025
e335203
Add map indexes for O(1) lookups in Registry
SamMorrowDrums Dec 15, 2025
881d2bb
perf(registry): O(1) HasToolset lookup via pre-computed set
SamMorrowDrums Dec 15, 2025
fe4f78e
simplify: remove lazy toolsByName map - not needed for actual use cases
SamMorrowDrums Dec 15, 2025
1953945
Add generic tool filtering mechanisms to registry package
Copilot Dec 16, 2025
bc198ee
docs: improve filter evaluation order and FilteredTools documentation
SamMorrowDrums Dec 16, 2025
4bfc643
Refactor GenerateToolsetsHelp() to use strings.Builder pattern
Copilot Dec 15, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
69 changes: 69 additions & 0 deletions .github/workflows/conformance.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,69 @@
name: Conformance Test

on:
pull_request:

permissions:
contents: read

jobs:
conformance:
runs-on: ubuntu-latest

steps:
- name: Check out code
uses: actions/checkout@v6
with:
# Fetch full history to access merge-base
fetch-depth: 0

- name: Set up Go
uses: actions/setup-go@v6
with:
go-version-file: "go.mod"

- name: Download dependencies
run: go mod download

- name: Run conformance test
id: conformance
run: |
# Run conformance test, capture stdout for summary
script/conformance-test > conformance-summary.txt 2>&1 || true

# Output the summary
cat conformance-summary.txt

# Check result
if grep -q "RESULT: ALL TESTS PASSED" conformance-summary.txt; then
echo "status=passed" >> $GITHUB_OUTPUT
else
echo "status=differences" >> $GITHUB_OUTPUT
fi

- name: Generate Job Summary
run: |
# Add the full markdown report to the job summary
echo "# MCP Server Conformance Report" >> $GITHUB_STEP_SUMMARY
echo "" >> $GITHUB_STEP_SUMMARY
echo "Comparing PR branch against merge-base with \`origin/main\`" >> $GITHUB_STEP_SUMMARY
echo "" >> $GITHUB_STEP_SUMMARY

# Extract and append the report content (skip the header since we added our own)
tail -n +5 conformance-report/CONFORMANCE_REPORT.md >> $GITHUB_STEP_SUMMARY

echo "" >> $GITHUB_STEP_SUMMARY
echo "---" >> $GITHUB_STEP_SUMMARY
echo "" >> $GITHUB_STEP_SUMMARY

# Add interpretation note
if [ "${{ steps.conformance.outputs.status }}" = "passed" ]; then
echo "✅ **All conformance tests passed** - No behavioral differences detected." >> $GITHUB_STEP_SUMMARY
else
echo "⚠️ **Differences detected** - Review the diffs above to ensure changes are intentional." >> $GITHUB_STEP_SUMMARY
echo "" >> $GITHUB_STEP_SUMMARY
echo "Common expected differences:" >> $GITHUB_STEP_SUMMARY
echo "- New tools/toolsets added" >> $GITHUB_STEP_SUMMARY
echo "- Tool descriptions updated" >> $GITHUB_STEP_SUMMARY
echo "- Capability changes (intentional improvements)" >> $GITHUB_STEP_SUMMARY
fi
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,5 @@ bin/
# binary
github-mcp-server

.history
.history
conformance-report/
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -384,6 +384,7 @@ You can also configure specific tools using the `--tools` flag. Tools can be use
- Tools, toolsets, and dynamic toolsets can all be used together
- Read-only mode takes priority: write tools are skipped if `--read-only` is set, even if explicitly requested via `--tools`
- Tool names must match exactly (e.g., `get_file_contents`, not `getFileContents`). Invalid tool names will cause the server to fail at startup with an error message
- When tools are renamed, old names are preserved as aliases for backward compatibility. See [Deprecated Tool Aliases](docs/deprecated-tool-aliases.md) for details.

### Using Toolsets With Docker

Expand Down Expand Up @@ -459,7 +460,6 @@ The following sets of tools are available:
| `code_security` | Code security related tools, such as GitHub Code Scanning |
| `dependabot` | Dependabot tools |
| `discussions` | GitHub Discussions related tools |
| `experiments` | Experimental features that are not considered stable yet |
| `gists` | GitHub Gist related tools |
| `git` | GitHub Git API related tools for low-level Git operations |
| `issues` | GitHub Issues related tools |
Expand Down
Loading
Loading