Skip to content

Conversation

@kyle-rader-msft
Copy link
Contributor

Support passing in a token via environment variable

#608

Associated Risks

There are very few risks here, as taking in an EntraID token inherently means another tool has already handled authentication and we now have a short-lived token.

PR Checklist

  • I have read the contribution guidelines
  • I have read the code of conduct guidelines
  • Title of the pull request is clear and informative.
  • 👌 Code hygiene
  • 🔭 Telemetry - N/A
  • 📄 Documentation: CLI Help Text, Getting Started, and FAQ Updated.
  • 🛡️ N/A -

🧪 How did you test it?

I've manually tested by

  • using azureauth ado token to acquire an ADO access token
  • export it as an env var and launch the MCP server via
    ADO_MCP_AUTH_TOKEN=$(azureauth ado token) npm start -- Office -a envvar
    

@kyle-rader-msft kyle-rader-msft requested a review from a team as a code owner November 4, 2025 18:44
@codecov-commenter
Copy link

codecov-commenter commented Nov 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@4c0f9d0). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #661   +/-   ##
=======================================
  Coverage        ?   96.47%           
=======================================
  Files           ?       16           
  Lines           ?     1249           
  Branches        ?      246           
=======================================
  Hits            ?     1205           
  Misses          ?       17           
  Partials        ?       27           
Flag Coverage Δ
unittests 96.47% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@danhellem
Copy link
Contributor

Sorry @kyle-rader-msft but we need folks to follow the process outlined in https://github.com/microsoft/azure-devops-mcp/blob/main/CONTRIBUTING.md

Please create an Issue first before creating a PR. I will going to close this PR for the time being.

@danhellem danhellem closed this Nov 4, 2025
@danhellem danhellem added the Do Not Merge ❌ do not merge this pull request label Nov 4, 2025
@danhellem danhellem reopened this Nov 4, 2025
@danhellem danhellem requested a review from aaudzei November 4, 2025 18:51
@danhellem danhellem added Needs Review 👓 needs review by the product team and removed Do Not Merge ❌ do not merge this pull request labels Nov 4, 2025
@danhellem
Copy link
Contributor

@aaudzei assigning to you to review per Teams chat

@danhellem danhellem linked an issue Nov 4, 2025 that may be closed by this pull request
@kyle-rader-msft kyle-rader-msft force-pushed the user/kyrader/add-support-access-token-via-env-var branch from 22947c6 to 9c4dce0 Compare November 4, 2025 19:39
@kyle-rader-msft kyle-rader-msft force-pushed the user/kyrader/add-support-access-token-via-env-var branch from f43d029 to d2e6fd3 Compare November 5, 2025 18:21
@aaudzei aaudzei merged commit 0823e3e into microsoft:main Nov 6, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review 👓 needs review by the product team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pass through EntraID token via Env Var

4 participants