Add .md/.yml/.yaml/.toml file preview support and remove gateway.log inlining

[Copilot]

The file display helper only rendered .json, .jsonl, .log, and .txt files. Markdown and configuration files were skipped. Additionally, parse_mcp_gateway_log.cjs contained outdated code that inlined gateway.log content when gateway.md was present.

Changes

• display_file_helpers.cjs: Added .md, .yml, .yaml, .toml to displayExtensions array
• parse_mcp_gateway_log.cjs: Removed 15 lines that read and output gateway.log when gateway.md exists
• Tests: Updated to verify new file types render and removed test for deleted behavior

Before:

const displayExtensions = [".json", ".jsonl", ".log", ".txt"];

After:

const displayExtensions = [".json", ".jsonl", ".log", ".txt", ".md", ".yml", ".yaml", ".toml"];

Original prompt

Reference: https://github.com/github/gh-aw/actions/runs/21672687909/job/62484453714#step:36:1

The JavaScript that generate a preview of files should also render .md , .yml .yaml, .toml files

See report

Remove code that tries to inline gateway.log, it is outdated.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

Extends the GitHub Actions file preview helper to render additional human-readable file types (Markdown and common config formats) and removes outdated behavior that inlined gateway.log when gateway.md is present.

Changes:

• Added .md, .yml, .yaml, and .toml to the set of file extensions whose contents are printed in log groups.
• Removed the gateway.log inlining path when gateway.md exists, writing gateway.md directly to the step summary.
• Updated tests to cover the newly supported file types and to validate removal of the legacy gateway.log inlining behavior.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
actions/setup/js/display_file_helpers.cjs	Expands supported preview extensions to include Markdown/YAML/TOML.
actions/setup/js/display_file_helpers.test.cjs	Adds/adjusts unit tests to verify content rendering for the new file types and unsupported-extension behavior.
actions/setup/js/parse_mcp_gateway_log.cjs	Removes outdated logic that read/printed gateway.log when gateway.md is available.
actions/setup/js/parse_mcp_gateway_log.test.cjs	Updates tests to reflect the new gateway.md-only behavior and .md preview support.

Comments suppressed due to low confidence (1)

actions/setup/js/parse_mcp_gateway_log.test.cjs:315

• Restoring the original fs functions and deleting global.core happens inside the try block. If an assertion throws, these globals/mocks may leak into subsequent tests. Move the restore/cleanup into the finally block (or use vi.spyOn + vi.restoreAllMocks) so it always runs.

        // Restore original functions
        fs.existsSync = originalExistsSync;
        fs.readFileSync = originalReadFileSync;
        delete global.core;

---

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

[Copilot]

The fs.readFileSync mock calls fs.readFileSync(gatewayMdPath, ...) inside its own mock implementation, which will recurse indefinitely because fs.readFileSync has already been replaced. Use the saved originalReadFileSync (or another unmocked reference) to read the temp file content.

Suggested change

	return fs.readFileSync(gatewayMdPath, encoding);
	return originalReadFileSync(gatewayMdPath, encoding);