Skip to content

Move Token Usage header into step summary details #50126

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.

Sign up for GitHub

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

Merged
pelikhan merged 1 commit into from
Aug 4, 2026
Merged

Move Token Usage header into step summary details #50126

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion actions/setup/js/parse_token_usage.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ function getSummaryTitle() {
* @returns {string}
*/
function buildStepSummarySection(title, markdown) {
return `### ${title}\n\n<details>\n<summary>Per-request AI credits and token totals</summary>\n\n${markdown}</details>\n\n`;
return `<details>\n<summary>${title}</summary>\n\nPer-request AI credits and token totals\n\n${markdown}</details>\n\n`;
}

/**
Expand Down
14 changes: 7 additions & 7 deletions actions/setup/js/parse_token_usage.test.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ describe("parse_token_usage", () => {

await main();

expect(mockCore.summary.addRaw).toHaveBeenCalledWith(expect.stringContaining("### Token Usage"), true);
expect(mockCore.summary.addRaw).toHaveBeenCalledWith(expect.stringContaining("<summary>Token Usage</summary>"), true);
expect(mockCore.summary.addRaw).toHaveBeenCalledWith(expect.stringContaining("| Alias |"), true);
expect(mockCore.summary.write).toHaveBeenCalled();
expect(mockCore.info).toHaveBeenCalledWith(expect.stringContaining("Token usage summary appended"));
Expand Down Expand Up @@ -226,7 +226,7 @@ describe("parse_token_usage", () => {

await main();

expect(mockCore.summary.addRaw).toHaveBeenCalledWith(expect.stringContaining("### Threat Detection Token Usage"), true);
expect(mockCore.summary.addRaw).toHaveBeenCalledWith(expect.stringContaining("<summary>Threat Detection Token Usage</summary>"), true);
});

test("appends token usage section to GITHUB_STEP_SUMMARY when configured", async () => {
Expand All @@ -253,8 +253,8 @@ describe("parse_token_usage", () => {
await main();

const stepSummary = originalReadFileSync(stepSummaryPath, "utf8");
expect(stepSummary).toContain("### Token Usage");
expect(stepSummary).toContain("<summary>Per-request AI credits and token totals</summary>");
expect(stepSummary).toContain("<summary>Token Usage</summary>");
expect(stepSummary).toContain("Per-request AI credits and token totals");
expect(stepSummary).toContain("| ΔAI Credits | AI Credits |");
expect(fs.appendFileSync).toHaveBeenCalledWith(stepSummaryPath, expect.any(String), "utf8");
expect(mockCore.summary.addRaw).not.toHaveBeenCalled();
Expand Down Expand Up @@ -333,7 +333,7 @@ describe("parse_token_usage", () => {
await main();

const summaryCall = mockCore.summary.addRaw.mock.calls[0];
expect(summaryCall[0]).toContain("### Token Usage");
expect(summaryCall[0]).toContain("<summary>Token Usage</summary>");
expectTokenUsageTableRows(summaryCall[0], [
["sonnet46", "100", "200"],
["gpt40", "50", "80"],
Expand Down Expand Up @@ -544,9 +544,9 @@ describe("parse_token_usage", () => {

test("buildStepSummarySection wraps markdown in a heading and details block", () => {
const section = buildStepSummarySection("Token Usage", "| Alias |\n| --- |");
expect(section).toContain("### Token Usage");
expect(section).toContain("<details>");
expect(section).toContain("<summary>Per-request AI credits and token totals</summary>");
expect(section).toContain("<summary>Token Usage</summary>");
expect(section).toContain("Per-request AI credits and token totals");
Comment on lines +548 to +549
});

test("renderTokenTableAsPlainText strips table separator lines and pipes", () => {
Expand Down