Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 30, 2026

ETA doesn't display during initial sync (git clone operations) until 3 repositories complete. Since clones take significantly longer than updates, users wait without feedback.

Changes

  • Adaptive warmup period: Reduced from 3 to 1 repo for initial sync (clones), kept 3 for incremental sync (updates)

    • Detection: checks if s.cloned > 0 to determine sync type
    • Initial sync needs less warmup because each operation takes longer (more stable rate estimate)
    • Incremental sync keeps conservative warmup for accuracy with faster operations
  • Progress calculation fix: Added s.archived to done count (was excluded, causing incorrect progress percentages)

Implementation

// Before: always required 3 samples
minSamples := min(10, s.totalRepos/10)
if minSamples < 3 {
    minSamples = 3
}

// After: adaptive based on sync type
minSamples := 1 // Start with 1 for initial sync
if s.cloned == 0 && done > 0 {
    // Incremental sync - more conservative
    minSamples = min(10, s.totalRepos/10)
    if minSamples < 3 {
        minSamples = 3
    }
}

Result: ETA appears after first clone completes instead of after third.

Original prompt

This section details on the original issue you should resolve

<issue_title>No ETA on initial sync</issue_title>
<issue_description>Initial sync shows no ETA, at least on Windows but possibly on all platforms.</issue_description>

<agent_instructions>There's an ETA string that's supposed to show up when syncing, however at least on the INITIAL sync (eg. git clone/checkout) it does NOT appear at all and this needs to be fixed.</agent_instructions>

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


💡 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 2 commits January 30, 2026 08:26
Co-authored-by: Dids <3455449+Dids@users.noreply.github.com>
Co-authored-by: Dids <3455449+Dids@users.noreply.github.com>
@Dids Dids marked this pull request as ready for review January 30, 2026 08:32
Copilot AI changed the title [WIP] Fix missing ETA on initial sync process Fix missing ETA during initial sync by reducing warmup period for clones Jan 30, 2026
Copilot AI requested a review from Dids January 30, 2026 08:33
@Dids Dids merged commit 38e9777 into master Jan 30, 2026
8 checks passed
@Dids Dids deleted the copilot/fix-initial-sync-eta branch January 30, 2026 08:34
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.

No ETA on initial sync

2 participants