cfc05d6f5e## Summary `unionCapabilities` merges `Experimental` map entries from inner variant servers but skips the `Extensions` field entirely. This means any extension registered via `ServerCapabilities.AddExtension()` on an inner server (e.g. `io.modelcontextprotocol/skills`) is silently dropped from the front proxy's `InitializeResult`. - Add the same first-wins merge logic for `Extensions` that already exists for `Experimental` - Bump `go-sdk` to v1.4.1 which introduced the `Extensions` field on `ServerCapabilities` - Add unit test for `unionCapabilities` extensions merging - Add integration test verifying extensions survive the variant path end-to-end ## Test plan - [x] `TestUnionCapabilities_MergesExtensions` — verifies first-wins merge semantics for extensions - [x] `TestIntegration_ExtensionsAdvertised` — verifies extensions registered on inner servers appear in the client's `InitializeResult` - [x] All existing tests pass (27/27) Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
80 lines
No EOL
2.1 KiB
YAML
80 lines
No EOL
2.1 KiB
YAML
name: Go CI
|
|
on:
|
|
push:
|
|
branches: [main]
|
|
paths:
|
|
- 'go/**'
|
|
- '.github/workflows/go.yml'
|
|
pull_request:
|
|
paths:
|
|
- 'go/**'
|
|
- '.github/workflows/go.yml'
|
|
workflow_dispatch:
|
|
|
|
permissions:
|
|
contents: read
|
|
|
|
jobs:
|
|
go-lint:
|
|
name: "Linting"
|
|
runs-on: ubuntu-latest
|
|
steps:
|
|
- name: Check out code
|
|
uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
|
|
- name: Set up Go
|
|
uses: actions/setup-go@7a3fe6cf4cb3a834922a1244abfce67bcef6a0c5 # v6.2.0
|
|
with:
|
|
go-version: "^1.24"
|
|
- name: Check formatting
|
|
run: |
|
|
cd go/sdk
|
|
unformatted=$(gofmt -l .)
|
|
if [ -n "$unformatted" ]; then
|
|
echo "The following files are not properly formatted:"
|
|
echo "$unformatted"
|
|
exit 1
|
|
fi
|
|
echo "All Go files are properly formatted"
|
|
- name: Run Go vet
|
|
run: |
|
|
cd go/sdk
|
|
go vet ./...
|
|
- name: Run staticcheck
|
|
uses: dominikh/staticcheck-action@288b4e28bae83c59f35f73651aeb5cab746a06fc # v1.4.0
|
|
with:
|
|
install-go: false
|
|
version: "v0.6.1"
|
|
working-directory: go/sdk
|
|
|
|
go-test:
|
|
name: "Unit Tests"
|
|
runs-on: ubuntu-latest
|
|
strategy:
|
|
matrix:
|
|
go: ["1.24", "1.25"]
|
|
steps:
|
|
- name: Check out code
|
|
uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
|
|
- name: Set up Go
|
|
uses: actions/setup-go@7a3fe6cf4cb3a834922a1244abfce67bcef6a0c5 # v6.2.0
|
|
with:
|
|
go-version: ${{ matrix.go }}
|
|
- name: Test
|
|
run: |
|
|
cd go/sdk
|
|
go test -v ./...
|
|
|
|
go-race-test:
|
|
name: "Race Detection"
|
|
runs-on: ubuntu-latest
|
|
steps:
|
|
- name: Check out code
|
|
uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
|
|
- name: Set up Go
|
|
uses: actions/setup-go@7a3fe6cf4cb3a834922a1244abfce67bcef6a0c5 # v6.2.0
|
|
with:
|
|
go-version: "1.24"
|
|
- name: Test with -race
|
|
run: |
|
|
cd go/sdk
|
|
go test -v -race ./... |