diff --git a/.github/workflows/src/context.js b/.github/workflows/src/context.js index 6a88034ea045..c267673eed4b 100644 --- a/.github/workflows/src/context.js +++ b/.github/workflows/src/context.js @@ -25,6 +25,15 @@ export async function extractInputs(github, context, core) { core.info(` payload.action: ${context.payload.action}`); core.info(` payload.workflow_run.event: ${context.payload.workflow_run?.event || "undefined"}`); + core.summary.addHeading("extractInputs", 2); + core.summary.addTable([ + ["eventName", context.eventName], + ["payload.action", context.payload.action || "undefined"], + ["payload.workflow_run.event", context.payload.workflow_run?.event || "undefined"], + ]); + core.summary.write(); + core.setOutput("summary", process.env.GITHUB_STEP_SUMMARY); + // Log full context when debug is enabled. Most workflows should be idempotent and can be re-run // with debug enabled to replay the previous context. if (core.isDebug()) { diff --git a/.github/workflows/src/summarize-checks/summarize-checks.js b/.github/workflows/src/summarize-checks/summarize-checks.js index c2192557ed46..ca3ffc0667a1 100644 --- a/.github/workflows/src/summarize-checks/summarize-checks.js +++ b/.github/workflows/src/summarize-checks/summarize-checks.js @@ -364,9 +364,7 @@ export async function summarizeChecksImpl( core.info(`Handling ${event_name} event for PR #${issue_number} in ${owner}/${repo}.`); const prUrl = `https://github.com/${owner}/${repo}/pull/${issue_number}`; - core.summary.addRaw("PR: "); - core.summary.addLink(prUrl, prUrl); - core.summary.write(); + core.summary.addRaw("PR: ").addLink(prUrl, prUrl).write(); core.setOutput("summary", process.env.GITHUB_STEP_SUMMARY); let labelNames = await getExistingLabels(github, owner, repo, issue_number); @@ -447,8 +445,7 @@ export async function summarizeChecksImpl( core.info( `Updating comment '${NEXT_STEPS_COMMENT_ID}' on ${owner}/${repo}#${issue_number} with body: ${commentBody}`, ); - core.summary.addRaw(`\n${commentBody}\n\n`); - core.summary.write(); + core.summary.addRaw(`\n${commentBody}\n\n`).write(); // this will remain commented until we're comfortable with the change. // await commentOrUpdate( diff --git a/.github/workflows/test/mocks.js b/.github/workflows/test/mocks.js index 2260f5675e44..35d8b2fad776 100644 --- a/.github/workflows/test/mocks.js +++ b/.github/workflows/test/mocks.js @@ -57,8 +57,13 @@ export function createMockCore() { setOutput: vi.fn((name, value) => console.log(`setOutput('${name}', '${value}')`)), setFailed: vi.fn((msg) => console.log(`setFailed('${msg}')`)), summary: { - // eslint-disable-next-line @typescript-eslint/no-unused-vars - addRaw: vi.fn(function (content) { + addHeading: vi.fn(function () { + return this; // Return 'this' for method chaining + }), + addRaw: vi.fn(function () { + return this; // Return 'this' for method chaining + }), + addTable: vi.fn(function () { return this; // Return 'this' for method chaining }), write: vi.fn().mockResolvedValue(undefined),