Skip to content

Conversation

@kevinbackhouse
Copy link
Contributor

@kevinbackhouse kevinbackhouse commented Dec 8, 2025

Codespace secrets are not allowed to have a name that starts with "GITHUB".

I ran this command to replace the name everywhere:

find . -type f -exec sed -i 's/GITHUB_PERSONAL_ACCESS_TOKEN/GH_TOKEN/g' {} \

I've made the same change in the other repo: GitHubSecurityLab/seclab-taskflow-agent#114

Copilot AI review requested due to automatic review settings December 8, 2025 16:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR renames the environment variable GITHUB_PERSONAL_ACCESS_TOKEN to GH_TOKEN across the entire codebase to comply with GitHub Codespaces naming restrictions that prohibit secret names starting with "GITHUB".

  • Renamed environment variable from GITHUB_PERSONAL_ACCESS_TOKEN to GH_TOKEN across all MCP servers, toolbox configurations, scripts, and documentation
  • Updated Python code to read the new GH_TOKEN environment variable
  • Updated shell scripts and devcontainer configuration to use the new variable name

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/seclab_taskflows/toolboxes/local_gh_resources.yaml Updated env variable reference from GITHUB_PERSONAL_ACCESS_TOKEN to GH_TOKEN
src/seclab_taskflows/toolboxes/ghsa.yaml Updated env variable reference from GITHUB_PERSONAL_ACCESS_TOKEN to GH_TOKEN
src/seclab_taskflows/toolboxes/gh_file_viewer.yaml Updated env variable reference from GITHUB_PERSONAL_ACCESS_TOKEN to GH_TOKEN
src/seclab_taskflows/toolboxes/gh_code_scanning.yaml Updated env variable reference from GITHUB_PERSONAL_ACCESS_TOKEN to GH_TOKEN
src/seclab_taskflows/toolboxes/gh_actions.yaml Updated env variable reference from GITHUB_PERSONAL_ACCESS_TOKEN to GH_TOKEN
src/seclab_taskflows/mcp_servers/local_gh_resources.py Renamed Python variable and updated all references in authorization headers
src/seclab_taskflows/mcp_servers/gh_file_viewer.py Renamed Python variable and updated all references in authorization headers
src/seclab_taskflows/mcp_servers/gh_code_scanning.py Renamed Python variable and updated all references in authorization headers
src/seclab_taskflows/mcp_servers/gh_actions.py Renamed Python variable and updated all references in authorization headers
src/seclab_taskflows/mcp_servers/codeql_python/README.md Updated documentation to reference GH_TOKEN instead of GITHUB_PERSONAL_ACCESS_TOKEN
src/run_seclab_agent.sh Updated shell script to pass GH_TOKEN environment variable to Docker container
.devcontainer/post-attach.sh Updated Codespaces warning message to reference GH_TOKEN

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kevinbackhouse kevinbackhouse merged commit a725583 into GitHubSecurityLab:main Dec 8, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants