Skip to content

Commit 306aa1b

Browse files
Merge branch 'main' into circleci-project-setup
2 parents 299376a + b01f7f5 commit 306aa1b

File tree

196 files changed

+21717
-11152
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

196 files changed

+21717
-11152
lines changed

.github/prompts/bug-report-review.prompt.yml

Lines changed: 18 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,26 +5,38 @@ messages:
55
66
Your job is to analyze bug reports and assess their completeness.
77
8+
**CRITICAL: Detect unfilled templates**
9+
- Flag issues containing unmodified template text like "A clear and concise description of what the bug is"
10+
- Flag placeholder values like "Type this '...'" or "View the output '....'" that haven't been replaced
11+
- Flag generic/meaningless titles (e.g., random words, test content)
12+
- These are ALWAYS "Missing Details" even if the template structure is present
13+
814
Analyze the issue for these key elements:
9-
1. Clear description of the problem
15+
1. Clear description of the problem (not template text)
1016
2. Affected version (from running `docker run -i --rm ghcr.io/github/github-mcp-server ./github-mcp-server --version`)
11-
3. Steps to reproduce the behavior
12-
4. Expected vs actual behavior
17+
3. Steps to reproduce the behavior (actual steps, not placeholders)
18+
4. Expected vs actual behavior (real descriptions, not template text)
1319
5. Relevant logs (if applicable)
1420
1521
Provide ONE of these assessments:
1622
1723
### AI Assessment: Ready for Review
18-
Use when the bug report has most required information and can be triaged by a maintainer.
24+
Use when the bug report has actual information in required fields and can be triaged by a maintainer.
1925
2026
### AI Assessment: Missing Details
21-
Use when critical information is missing (no reproduction steps, no version info, unclear problem description).
27+
Use when:
28+
- Template text has not been replaced with actual content
29+
- Critical information is missing (no reproduction steps, no version info, unclear problem description)
30+
- The title is meaningless or spam-like
31+
- Placeholder text remains in any section
32+
33+
When marking as Missing Details, recommend adding the "waiting-for-reply" label.
2234
2335
### AI Assessment: Unsure
2436
Use when you cannot determine the completeness of the report.
2537
2638
After your assessment header, provide a brief explanation of your rating.
27-
If details are missing, note which specific sections need more information.
39+
If details are missing, be specific about which sections contain template text or need actual information.
2840
- role: user
2941
content: "{{input}}"
3042
model: openai/gpt-4o-mini

.github/prompts/default-issue-review.prompt.yml

Lines changed: 28 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,24 +5,47 @@ messages:
55
66
Your job is to analyze new issues and help categorize them.
77
8+
**CRITICAL: Detect invalid or incomplete submissions**
9+
- Flag issues with unmodified template text (e.g., "A clear and concise description...")
10+
- Flag placeholder values that haven't been replaced (e.g., "Type this '...'", "....", "XXX")
11+
- Flag meaningless, spam-like, or test titles (e.g., random words, nonsensical content)
12+
- Flag empty or nearly empty issues
13+
- These are ALWAYS "Missing Details" or "Invalid" depending on severity
14+
815
Analyze the issue to determine:
9-
1. Is this a bug report, feature request, question, or something else?
10-
2. Is the issue clear and well-described?
16+
1. Is this a bug report, feature request, question, documentation issue, or something else?
17+
2. Is the issue clear and well-described with actual content (not template text)?
1118
3. Does it contain enough information for maintainers to act on?
19+
4. Is this potentially spam, a test issue, or completely invalid?
1220
1321
Provide ONE of these assessments:
1422
1523
### AI Assessment: Ready for Review
16-
Use when the issue is clear, well-described, and contains enough context for maintainers to understand and act on it.
24+
Use when the issue is clear, well-described with actual content, and contains enough context for maintainers to understand and act on it.
1725
1826
### AI Assessment: Missing Details
19-
Use when the issue is unclear, lacks context, or needs more information to be actionable.
27+
Use when:
28+
- Template text has not been replaced with actual content
29+
- The issue is unclear or lacks context
30+
- Critical information is missing to make it actionable
31+
- The title is vague but the issue seems legitimate
32+
33+
When marking as Missing Details, recommend adding the "waiting-for-reply" label.
34+
35+
### AI Assessment: Invalid
36+
Use when:
37+
- The issue appears to be spam or test content
38+
- The title is completely meaningless and body has no useful information
39+
- This doesn't relate to the GitHub MCP Server project at all
40+
41+
When marking as Invalid, recommend adding the "invalid" label and consider closing.
2042
2143
### AI Assessment: Unsure
2244
Use when you cannot determine the nature or completeness of the issue.
2345
2446
After your assessment header, provide a brief explanation including:
25-
- What type of issue this appears to be (bug, feature request, question, etc.)
47+
- What type of issue this appears to be (bug, feature request, question, invalid, etc.)
48+
- Which specific sections contain template text or need actual information
2649
- What additional information might be helpful if any
2750
- role: user
2851
content: "{{input}}"

.github/workflows/code-scanning.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,10 @@ jobs:
3535
category: /language:go
3636
build-mode: autobuild
3737
runner: '["ubuntu-22.04"]'
38+
- language: javascript
39+
category: /language:javascript
40+
build-mode: none
41+
runner: '["ubuntu-22.04"]'
3842
steps:
3943
- name: Checkout repository
4044
uses: actions/checkout@v6
@@ -74,6 +78,18 @@ jobs:
7478
go-version: ${{ fromJSON(steps.resolve-environment.outputs.environment).configuration.go.version }}
7579
cache: false
7680

81+
- name: Set up Node.js
82+
if: matrix.language == 'go' || matrix.language == 'javascript'
83+
uses: actions/setup-node@v4
84+
with:
85+
node-version: "20"
86+
cache: "npm"
87+
cache-dependency-path: ui/package-lock.json
88+
89+
- name: Build UI
90+
if: matrix.language == 'go'
91+
run: script/build-ui
92+
7793
- name: Autobuild
7894
uses: github/codeql-action/autobuild@v4
7995

.github/workflows/docker-publish.yml

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ jobs:
4646
# https://github.com/sigstore/cosign-installer
4747
- name: Install cosign
4848
if: github.event_name != 'pull_request'
49-
uses: sigstore/cosign-installer@faadad0cce49287aee09b3a48701e75088a2c6ad #v4.0.0
49+
uses: sigstore/cosign-installer@ba7bc0a3fef59531c69a25acd34668d6d3fe6f22 #v4.1.0
5050
with:
5151
cosign-release: "v2.2.4"
5252

@@ -70,7 +70,7 @@ jobs:
7070
# https://github.com/docker/metadata-action
7171
- name: Extract Docker metadata
7272
id: meta
73-
uses: docker/metadata-action@c299e40c65443455700f0fdfc63efafe5b349051 # v5.10.0
73+
uses: docker/metadata-action@030e881283bb7a6894de51c315a6bfe6a94e05cf # v6.0.0
7474
with:
7575
images: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}
7676
tags: |
@@ -94,14 +94,20 @@ jobs:
9494

9595
- name: Inject go-build-cache
9696
uses: reproducible-containers/buildkit-cache-dance@5b81f4d29dc8397a7d341dba3aeecc7ec54d6361 # v3.3.0
97+
uses: reproducible-containers/buildkit-cache-dance@1b8ab18fbda5ad3646e3fcc9ed9dd41ce2f297b4 # v3.3.2
9798
with:
98-
cache-source: go-build-cache
99+
cache-map: |
100+
{
101+
"go-build-cache/apk": "/var/cache/apk",
102+
"go-build-cache/pkg": "/go/pkg/mod",
103+
"go-build-cache/build": "/root/.cache/go-build"
104+
}
99105
100106
# Build and push Docker image with Buildx (don't push on PR)
101107
# https://github.com/docker/build-push-action
102108
- name: Build and push Docker image
103109
id: build-and-push
104-
uses: docker/build-push-action@263435318d21b8e681c14492fe198d362a7d2c83 # v6.18.0
110+
uses: docker/build-push-action@d08e5c354a6adb9ed34480a06d141179aa583294 # v7.0.0
105111
with:
106112
context: .
107113
push: ${{ github.event_name != 'pull_request' }}

.github/workflows/docs-check.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,16 @@ jobs:
1616
- name: Checkout code
1717
uses: actions/checkout@v6
1818

19+
- name: Set up Node.js
20+
uses: actions/setup-node@v4
21+
with:
22+
node-version: "20"
23+
cache: "npm"
24+
cache-dependency-path: ui/package-lock.json
25+
26+
- name: Build UI
27+
run: script/build-ui
28+
1929
- name: Set up Go
2030
uses: actions/setup-go@v6
2131
with:

.github/workflows/go.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,17 @@ jobs:
2525
- name: Check out code
2626
uses: actions/checkout@v6
2727

28+
- name: Set up Node.js
29+
uses: actions/setup-node@v4
30+
with:
31+
node-version: "20"
32+
cache: "npm"
33+
cache-dependency-path: ui/package-lock.json
34+
35+
- name: Build UI
36+
shell: bash
37+
run: script/build-ui
38+
2839
- name: Set up Go
2940
uses: actions/setup-go@v6
3041
with:
@@ -34,6 +45,7 @@ jobs:
3445
run: go mod tidy -diff
3546

3647
- name: Run unit tests
48+
shell: bash
3749
run: script/test
3850

3951
- name: Build

.github/workflows/goreleaser.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,16 @@ jobs:
1616
- name: Check out code
1717
uses: actions/checkout@v6
1818

19+
- name: Set up Node.js
20+
uses: actions/setup-node@v4
21+
with:
22+
node-version: "20"
23+
cache: "npm"
24+
cache-dependency-path: ui/package-lock.json
25+
26+
- name: Build UI
27+
run: script/build-ui
28+
1929
- name: Set up Go
2030
uses: actions/setup-go@v6
2131
with:

.github/workflows/license-check.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,16 @@ jobs:
3232
GH_TOKEN: ${{ github.token }}
3333
run: gh pr checkout ${{ github.event.pull_request.number }}
3434

35+
- name: Set up Node.js
36+
uses: actions/setup-node@v4
37+
with:
38+
node-version: "20"
39+
cache: "npm"
40+
cache-dependency-path: ui/package-lock.json
41+
42+
- name: Build UI
43+
run: script/build-ui
44+
3545
- name: Set up Go
3646
uses: actions/setup-go@v6
3747
with:

.github/workflows/lint.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,18 @@ jobs:
1414
runs-on: ubuntu-latest
1515
steps:
1616
- uses: actions/checkout@v6
17+
- uses: actions/setup-node@v4
18+
with:
19+
node-version: "20"
20+
cache: "npm"
21+
cache-dependency-path: ui/package-lock.json
22+
- name: Build UI
23+
run: script/build-ui
1724
- uses: actions/setup-go@v6
1825
with:
19-
go-version: stable
26+
go-version: '1.25'
2027
- name: golangci-lint
2128
uses: golangci/golangci-lint-action@v9
2229
with:
23-
version: v2.5
30+
# sync with script/lint
31+
version: v2.9

.github/workflows/mcp-diff.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,14 @@ jobs:
1919
with:
2020
fetch-depth: 0
2121

22+
- name: Set up Node.js
23+
uses: actions/setup-node@v4
24+
with:
25+
node-version: '20'
26+
27+
- name: Build UI
28+
run: script/build-ui
29+
2230
- name: Run MCP Server Diff
2331
uses: SamMorrowDrums/mcp-server-diff@v2.3.5
2432
with:

0 commit comments

Comments
 (0)