actions/claude-code-action
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

refactor: rename ACTIONS_TOKEN to DEFAULT_WORKFLOW_TOKEN (#385)

Browse Source 
Updated all references from ACTIONS_TOKEN to DEFAULT_WORKFLOW_TOKEN to match
the naming convention used in action.yml where the GitHub token is passed as
DEFAULT_WORKFLOW_TOKEN environment variable.
This commit is contained in:
Ashwin Bhat 2025-08-02 21:26:52 -07:00 committed by GitHub
parent d829b4d14b
commit d66adfb7fa
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 14 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/mcp/install-mcp-server.ts
View File

@ -118,7 +118,7 @@ export async function prepareMcpConfig(
if (context.isPR && hasActionsReadPermission) { if (context.isPR && hasActionsReadPermission) {
// Verify the token actually has actions:read permission // Verify the token actually has actions:read permission
const actuallyHasPermission = await checkActionsReadPermission( const actuallyHasPermission = await checkActionsReadPermission(
process.env.ACTIONS_TOKEN || "", process.env.DEFAULT_WORKFLOW_TOKEN || "",
owner, owner,
repo, repo,
); );
@ -138,7 +138,7 @@ export async function prepareMcpConfig(
], ],
env: { env: {
// Use workflow github token, not app token // Use workflow github token, not app token
GITHUB_TOKEN: process.env.ACTIONS_TOKEN, GITHUB_TOKEN: process.env.DEFAULT_WORKFLOW_TOKEN,
REPO_OWNER: owner, REPO_OWNER: owner,
REPO_NAME: repo, REPO_NAME: repo,
PR_NUMBER: context.entityNumber?.toString() || "", PR_NUMBER: context.entityNumber?.toString() || "",

24
test/install-mcp-server.test.ts
View File

@ -547,8 +547,8 @@ describe("prepareMcpConfig", () => {
}); });
test("should include github_ci server when context.isPR is true and actions:read permission is granted", async () => { test("should include github_ci server when context.isPR is true and actions:read permission is granted", async () => {
const oldEnv = process.env.ACTIONS_TOKEN; const oldEnv = process.env.DEFAULT_WORKFLOW_TOKEN;
process.env.ACTIONS_TOKEN = "workflow-token"; process.env.DEFAULT_WORKFLOW_TOKEN = "workflow-token";
const contextWithPermissions = { const contextWithPermissions = {
...mockPRContext, ...mockPRContext,
@ -575,7 +575,7 @@ describe("prepareMcpConfig", () => {
expect(parsed.mcpServers.github_ci.env.PR_NUMBER).toBe("456"); expect(parsed.mcpServers.github_ci.env.PR_NUMBER).toBe("456");
expect(parsed.mcpServers.github_file_ops).toBeDefined(); expect(parsed.mcpServers.github_file_ops).toBeDefined();
process.env.ACTIONS_TOKEN = oldEnv; process.env.DEFAULT_WORKFLOW_TOKEN = oldEnv;
}); });
test("should not include github_ci server when context.isPR is false", async () => { test("should not include github_ci server when context.isPR is false", async () => {
@ -595,8 +595,8 @@ describe("prepareMcpConfig", () => {
}); });
test("should not include github_ci server when actions:read permission is not granted", async () => { test("should not include github_ci server when actions:read permission is not granted", async () => {
const oldTokenEnv = process.env.ACTIONS_TOKEN; const oldTokenEnv = process.env.DEFAULT_WORKFLOW_TOKEN;
process.env.ACTIONS_TOKEN = "workflow-token"; process.env.DEFAULT_WORKFLOW_TOKEN = "workflow-token";
const result = await prepareMcpConfig({ const result = await prepareMcpConfig({
githubToken: "test-token", githubToken: "test-token",
@ -612,12 +612,12 @@ describe("prepareMcpConfig", () => {
expect(parsed.mcpServers.github_ci).not.toBeDefined(); expect(parsed.mcpServers.github_ci).not.toBeDefined();
expect(parsed.mcpServers.github_file_ops).toBeDefined(); expect(parsed.mcpServers.github_file_ops).toBeDefined();
process.env.ACTIONS_TOKEN = oldTokenEnv; process.env.DEFAULT_WORKFLOW_TOKEN = oldTokenEnv;
}); });
test("should parse additional_permissions with multiple lines correctly", async () => { test("should parse additional_permissions with multiple lines correctly", async () => {
const oldTokenEnv = process.env.ACTIONS_TOKEN; const oldTokenEnv = process.env.DEFAULT_WORKFLOW_TOKEN;
process.env.ACTIONS_TOKEN = "workflow-token"; process.env.DEFAULT_WORKFLOW_TOKEN = "workflow-token";
const contextWithPermissions = { const contextWithPermissions = {
...mockPRContext, ...mockPRContext,
@ -644,12 +644,12 @@ describe("prepareMcpConfig", () => {
expect(parsed.mcpServers.github_ci).toBeDefined(); expect(parsed.mcpServers.github_ci).toBeDefined();
expect(parsed.mcpServers.github_ci.env.GITHUB_TOKEN).toBe("workflow-token"); expect(parsed.mcpServers.github_ci.env.GITHUB_TOKEN).toBe("workflow-token");
process.env.ACTIONS_TOKEN = oldTokenEnv; process.env.DEFAULT_WORKFLOW_TOKEN = oldTokenEnv;
}); });
test("should warn when actions:read is requested but token lacks permission", async () => { test("should warn when actions:read is requested but token lacks permission", async () => {
const oldTokenEnv = process.env.ACTIONS_TOKEN; const oldTokenEnv = process.env.DEFAULT_WORKFLOW_TOKEN;
process.env.ACTIONS_TOKEN = "invalid-token"; process.env.DEFAULT_WORKFLOW_TOKEN = "invalid-token";
const contextWithPermissions = { const contextWithPermissions = {
...mockPRContext, ...mockPRContext,
@ -677,6 +677,6 @@ describe("prepareMcpConfig", () => {
), ),
); );
process.env.ACTIONS_TOKEN = oldTokenEnv; process.env.DEFAULT_WORKFLOW_TOKEN = oldTokenEnv;
}); });
}); });