Skip to content

Conversation

Copy link

Copilot AI commented Jan 16, 2026

The updateAllMarketplaces function terminated on first failure, preventing remaining marketplaces from updating and never printing the summary message.

Changes:

  • Refactored error handling: runGitCommand now throws instead of calling process.exit(1)
  • Batch operation resilience: updateAllMarketplaces catches errors per-marketplace, continues processing, and reports summary:
    Error updating beta: Failed to clone/update marketplace: git pull failed
    Updated marketplace: gamma
    
    Update complete: 2 succeeded, 1 failed
    
  • Consistent error propagation: Single operations (addMarketplace) handle errors and exit; batch-called operations (updateMarketplace) throw for caller handling
  • Test coverage: Added test verifying all marketplaces are attempted despite failures

Bonus fix: Removed duplicate await saveConfig({ line in plugin.ts from previous commit.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: scaryrawr <661373+scaryrawr@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP Address feedback on plugin commands PR Handle marketplace update failures gracefully in batch operations Jan 16, 2026
Copilot AI requested a review from scaryrawr January 16, 2026 01:51
@scaryrawr scaryrawr closed this Jan 16, 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