-
Notifications
You must be signed in to change notification settings - Fork 35
Rename detect-repo-visibility to determine-automatic-lockdown with runtime token check #8798
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…stom token requirement Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…uire custom token Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot The detection of the custom GitHub environment variable needs to be done as a "if" statement in the step |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done in c79b157. The determination step is now always generated when lockdown is not explicitly set, with an Example generated YAML: - name: Determine automatic lockdown mode for GitHub MCP server
id: determine-automatic-lockdown
if: secrets.GH_AW_GITHUB_MCP_SERVER_TOKEN != ''
uses: actions/github-script@... |
|
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done in 3b8861f:
Key documentation changes:
|
Rename detect-repo-visibility to determine-automatic-lockdown with runtime token check
Changes Completed
detect_repo_visibility.cjstodetermine_automatic_lockdown.cjsdetect_repo_visibility.test.cjstodetermine_automatic_lockdown.test.cjsdetectRepoVisibilitytodetermineAutomaticLockdownImplementation Details
The automatic lockdown determination step is now generated whenever:
The step includes a runtime condition that only executes when
GH_AW_GITHUB_MCP_SERVER_TOKENis defined:This ensures that:
Documentation Updates
Updated documentation in:
docs/src/content/docs/guides/security.md- Clarified that automatic determination only occurs whenGH_AW_GITHUB_MCP_SERVER_TOKENis defineddocs/src/content/docs/reference/tools.md- Updated lockdown field documentation to reflect runtime determination behaviorKey changes:
GH_AW_GITHUB_MCP_SERVER_TOKENGITHUB_TOKEN, automatic determination is skippedOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.