Skip to content

Add onboarding welcome action for [ONBOARD] issues#2997

Merged
yunjchoi merged 6 commits into
mainfrom
add-onboarding-welcome-action
Jul 13, 2026
Merged

Add onboarding welcome action for [ONBOARD] issues#2997
yunjchoi merged 6 commits into
mainfrom
add-onboarding-welcome-action

Conversation

@yunjchoi

@yunjchoi yunjchoi commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Adds a GitHub Action that welcomes new contributors when an Azure MCP onboarding issue is filed.

Trigger: issues (opened, labeled), gated on both the onboarding and Azure.Mcp.Server labels applied by the Azure MCP - Service Onboarding Request template. Requiring both labels scopes this to Azure MCP only — the generic New Microsoft MCP Server Onboarding Request template also uses onboarding but pairs it with Template.Mcp.Server, so it will not trigger this action.

Behavior: Posts a comment pointing contributors to the CONTRIBUTING.md guide and encouraging them to try the @onboarding Copilot agent. A hidden marker prevents duplicate comments across the multiple label events.

Permissions: issues: write only.

Comments on onboarding-labeled issues to point new contributors to CONTRIBUTING.md and the @onboarding Copilot agent. Requires both 'onboarding' and 'Azure.Mcp.Server' labels so the generic MCP onboarding template does not trigger it.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 6, 2026 19:35
@yunjchoi yunjchoi requested review from a team as code owners July 6, 2026 19:35
@github-project-automation github-project-automation Bot moved this to Untriaged in Azure MCP Server Jul 6, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a GitHub Actions workflow that automatically welcomes new contributors when an Azure MCP onboarding issue is created (or later labeled), posting a single guidance comment with links to contributing docs and the @onboarding Copilot agent.

Changes:

  • Add an issues-triggered workflow (opened, labeled) gated on both onboarding and Azure.Mcp.Server labels.
  • Post a welcome comment via actions/github-script, with an HTML marker to prevent duplicate comments across multiple label events.

Invoking Livetests

Copilot submitted PRs are not trustworthy by default. Users with write access to the repo need to validate the contents of this PR before leaving a comment with the text /azp run mcp - pullrequest - live. This will trigger the necessary livetest workflows to complete required validation.

@jongio jongio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One version consistency note flagged inline.

Comment thread .github/workflows/onboarding-welcome.yml Outdated
Co-authored-by: Jon Gallant <2163001+jongio@users.noreply.github.com>
@github-project-automation github-project-automation Bot moved this from Untriaged to In Progress in Azure MCP Server Jul 7, 2026
@yunjchoi yunjchoi enabled auto-merge (squash) July 7, 2026 17:18

@jongio jongio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incremental review of the v7 to v8 fix. The repo's other workflows pin actions/github-script by SHA at v9. One inline suggestion below.

Comment thread .github/workflows/onboarding-welcome.yml Outdated

@jongio jongio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logic, permissions, and duplicate-prevention all look correct. The label gating (onboarding + Azure.Mcp.Server) properly scopes to Azure MCP onboarding only, and the hidden marker prevents re-posting.

The SHA-pinning concern is already covered by existing feedback. One minor suggestion below about redundant workflow runs.

Comment thread .github/workflows/onboarding-welcome.yml
yunjchoi and others added 2 commits July 13, 2026 13:42
Co-authored-by: Jon Gallant <2163001+jongio@users.noreply.github.com>
Co-authored-by: Jon Gallant <2163001+jongio@users.noreply.github.com>

@jongio jongio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One blocking issue from the latest push. The rest of the changes (SHA-pinning github-script and adding concurrency to dedupe the multiple labeled events) look right.

  • .github/workflows/onboarding-welcome.yml: the jobs: block lost its welcome: job id and has a duplicated runs-on, so it no longer parses as a valid job. Details inline.

Comment thread .github/workflows/onboarding-welcome.yml Outdated
Co-authored-by: Jon Gallant <2163001+jongio@users.noreply.github.com>

@jongio jongio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incremental review of the latest push. The welcome: job id is restored, which fixes half of what I flagged, but the duplicate runs-on is still present so the workflow still won't parse. One inline note.

Comment thread .github/workflows/onboarding-welcome.yml Outdated
Co-authored-by: Jon Gallant <2163001+jongio@users.noreply.github.com>

@jongio jongio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed the latest commit drops the duplicate runs-on, so .github/workflows/onboarding-welcome.yml now parses as a single valid welcome job (checked with a YAML parse). The concurrency dedupe on the issue number and the SHA-pinned github-script@3a2844b... (v9) are both in place, and the label gate still requires both onboarding and Azure.Mcp.Server. Nothing outstanding from me on this workflow.

@yunjchoi yunjchoi merged commit c68bfa8 into main Jul 13, 2026
15 checks passed
@yunjchoi yunjchoi deleted the add-onboarding-welcome-action branch July 13, 2026 21:55
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Azure MCP Server Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants