Skip to content

Clarify progress total represents all entries, not just changed files - #249

Draft
moshfeu with Copilot wants to merge 10 commits into
masterfrom
copilot/add-status-indication-ui
Draft

Clarify progress total represents all entries, not just changed files#249
moshfeu with Copilot wants to merge 10 commits into
masterfrom
copilot/add-status-indication-ui

Conversation

Copilot AI commented Feb 13, 2026

Copy link
Copy Markdown
Contributor

Users misunderstood the progress counter to show only changed files. The resultBuilder callback in dir-compare fires for every entry pair (equal, distinct, left-only, right-only), so total count represents all comparison work.

Changes

Updated progress messages for clarity:

// Before
progress.report({ message: 'Counting entries...' });
progress.report({ message: `Found ${totalEntries} entries, comparing...` });

// After  
progress.report({ message: 'Counting all entries...' });
progress.report({ message: `Found ${totalEntries} entries (all files/folders), comparing...` });

Example

Comparing two folders with 100 files each (95 identical, 5 different):

  • Before: User sees "200 entries" and wonders why so many when only 5 changed
  • After: User sees "200 entries (all files/folders)" and understands all files are examined

This accurately represents the work: the comparison examines every file regardless of whether it differs.

Original prompt

This section details on the original issue you should resolve

<issue_title>Add status/progress indication</issue_title>
<issue_description>I am often diffing very large project folders. The diff can take a long term to generate. My biggest frustration with the process is that the extension offers no indication of its status/progress. I'll often start a diff, it'll seem that nothing is happening, and then the diff will pop up a few minutes later. This leads to confusion, needless retries, etc. Please add some sofrt of status indication somewhere in the UI.</issue_description>

<agent_instructions>Use the extension's existing loader ui infrastructure </agent_instructions>

Comments on the Issue (you are @copilot in this section)

@moshfeu Hi @nabelekt, thanks for the suggestion I'm looking into it to see if it's possible since the extension fully relays on the great library _dir-compare_'s for the comparison @moshfeu Huge delay, sorry 😅 When I re-read it now, I'm not sure why I said that it might not be possible. If I understand correctly you asked to add a loader when the comparison starts. If so (as asked in #243), I'm on it now @moshfeu I remember now why I said that it might not possible and I was referring to the "progress" request - to present the progress of the comparison process but showing a loaded it is possible

💡 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.

Copilot AI and others added 3 commits February 13, 2026 22:18
Co-authored-by: moshfeu <3723951+moshfeu@users.noreply.github.com>
…rings

Co-authored-by: moshfeu <3723951+moshfeu@users.noreply.github.com>
Co-authored-by: moshfeu <3723951+moshfeu@users.noreply.github.com>
Copilot AI changed the title [WIP] Add status/progress indication for diff process Add progress indication for folder comparison operations Feb 13, 2026
Copilot AI requested a review from moshfeu February 13, 2026 22:23
Copilot AI and others added 2 commits February 13, 2026 22:30
Co-authored-by: moshfeu <3723951+moshfeu@users.noreply.github.com>
Co-authored-by: moshfeu <3723951+moshfeu@users.noreply.github.com>
Copilot AI changed the title Add progress indication for folder comparison operations Add real-time progress tracking for folder comparisons Feb 13, 2026
Co-authored-by: moshfeu <3723951+moshfeu@users.noreply.github.com>
Copilot AI changed the title Add real-time progress tracking for folder comparisons Encapsulate progress UI in comparison functions Feb 13, 2026
Copilot AI and others added 2 commits February 13, 2026 22:50
Co-authored-by: moshfeu <3723951+moshfeu@users.noreply.github.com>
Co-authored-by: moshfeu <3723951+moshfeu@users.noreply.github.com>
Copilot AI changed the title Encapsulate progress UI in comparison functions Add real-time progress tracking with percentage for folder comparisons Feb 13, 2026
Co-authored-by: moshfeu <3723951+moshfeu@users.noreply.github.com>
Copilot AI changed the title Add real-time progress tracking with percentage for folder comparisons Clarify progress total represents all entries, not just changed files Feb 14, 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.

Add status/progress indication

2 participants