Skip to content

Remove nested groups in displayDirectories to eliminate redundant directory titles#13997

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/update-file-tree-renderer
Closed

Remove nested groups in displayDirectories to eliminate redundant directory titles#13997
Copilot wants to merge 2 commits intomainfrom
copilot/update-file-tree-renderer

Conversation

Copy link
Contributor

Copilot AI commented Feb 5, 2026

displayDirectories in parse_mcp_gateway_log.cjs was creating nested groups that duplicated directory paths—once in the outer group context and again as a nested group title.

Changes

actions/setup/js/display_file_helpers.cjs

  • Replaced displayDirectory() call with inline iteration
  • Directory paths now render via core.info() instead of core.startGroup()
  • Maintains single outer group without nested directory groups

Test updates

  • Updated assertions in display_file_helpers.test.cjs and parse_mcp_gateway_log.test.cjs to verify directory paths appear in info messages

Before/After

Before:

START GROUP: === Listing All Gateway-Related Files ===
  START GROUP: 📁 Directory: /tmp/gh-aw/mcp-logs
    INFO: gateway.log
  END GROUP
END GROUP

After:

START GROUP: === Listing All Gateway-Related Files ===
INFO: 📁 Directory: /tmp/gh-aw/mcp-logs
  INFO: gateway.log
END GROUP
Original prompt

Update the javascript file tree renderer used in actions/setup/js/parse_mcp_gateway_log.cjs (displayDirectories) to avoid repeating the title of a region inside the region.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…groups

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update javascript file tree renderer to avoid title repetition Remove nested groups in displayDirectories to eliminate redundant directory titles Feb 5, 2026
Copilot AI requested a review from pelikhan February 5, 2026 21:48
@pelikhan pelikhan closed this Feb 5, 2026
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