Skip to content

Conversation

@kevinbackhouse
Copy link
Contributor

The environment variable was renamed in GitHubSecurityLab/seclab-taskflow-agent#104.

Copilot AI review requested due to automatic review settings December 4, 2025 09:46
Copilot finished reviewing on behalf of kevinbackhouse December 4, 2025 09:49
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 COPILOT_TOKEN to AI_API_TOKEN across the codebase to align with changes made in PR #104. The rename affects fallback authentication logic in MCP server files and Docker/devcontainer configuration.

  • Updates four MCP server Python files to use AI_API_TOKEN as the fallback when GITHUB_PERSONAL_ACCESS_TOKEN is not set
  • Updates shell scripts to pass the renamed environment variable to Docker containers and check for it in devcontainer setup

Reviewed changes

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

Show a summary per file
File Description
src/seclab_taskflows/mcp_servers/local_gh_resources.py Updates fallback token environment variable from COPILOT_TOKEN to AI_API_TOKEN
src/seclab_taskflows/mcp_servers/gh_file_viewer.py Updates fallback token environment variable from COPILOT_TOKEN to AI_API_TOKEN
src/seclab_taskflows/mcp_servers/gh_code_scanning.py Updates fallback token environment variable from COPILOT_TOKEN to AI_API_TOKEN
src/seclab_taskflows/mcp_servers/gh_actions.py Updates fallback token environment variable from COPILOT_TOKEN to AI_API_TOKEN
src/run_seclab_agent.sh Updates Docker run command to pass AI_API_TOKEN instead of COPILOT_TOKEN
.devcontainer/post-attach.sh Updates Codespaces secret validation to check for AI_API_TOKEN instead of COPILOT_TOKEN

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

@p-
Copy link
Contributor

p- commented Dec 4, 2025

@m-y-mo / @kevinbackhouse given the naming I think it would make sense to remove those fallbacks GITHUB_PERSONAL_ACCESS_TOKEN = os.getenv('AI_API_TOKEN') (an AI token could be a PAT, but doesn't have to be)

@kevinbackhouse
Copy link
Contributor Author

@m-y-mo / @kevinbackhouse given the naming I think it would make sense to remove those fallbacks GITHUB_PERSONAL_ACCESS_TOKEN = os.getenv('AI_API_TOKEN') (an AI token could be a PAT, but doesn't have to be)

I've pushed a commit with this change.

@kevinbackhouse kevinbackhouse merged commit eadd066 into GitHubSecurityLab:main Dec 4, 2025
6 checks passed
@kevinbackhouse kevinbackhouse deleted the rename-env-var branch December 4, 2025 17:10
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