Remove duplicate filename display in displayFileContent#13999
Merged
Conversation
- Remove duplicate filename display before content group - Include file size in the collapsible group title instead - Update special case messages to include filename - Update all tests to check startGroup calls for filename - All 18 tests passing Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update displayFileContent to avoid filename duplication
Remove duplicate filename display in displayFileContent
Feb 5, 2026
pelikhan
approved these changes
Feb 5, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes duplicate filename display in the displayFileContent function. Previously, the function displayed filenames twice: once via core.info() and again as the collapsible group title. The changes consolidate all filename displays into a single location - either in the startGroup() call for displayable files, or in a single core.info() call for special cases (empty files, unsupported types, or files too large to display).
Changes:
- Removed the duplicate
core.info()line that displayed file size information before content display - Updated
startGroup()call to include both filename and size in the title - Consolidated special case messages (empty, too large, unsupported types) to include filename in a single info line
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| actions/setup/js/display_file_helpers.cjs | Removed duplicate filename display; consolidated filename and size into startGroup() title; updated special case messages to include filename |
| actions/setup/js/display_file_helpers.test.cjs | Updated all tests to verify filename appears in startGroup() calls instead of separate info() calls; updated special case test expectations |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This was referenced Feb 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
displayFileContentfunction displayed filenames twice: once as an info line and again as the collapsible group title.Changes
startGroup()call onlycore.info()line that duplicated the filenameBefore
After
Special cases now format consistently:
${fileName} (empty file),${fileName} (content not displayed for .pdf files), etc.Original prompt
💡 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.