Install function erases existing repo data via _repo_add

[calebj]

Invoking downloader._repo_add(name, url) on an existing repo clears Installer's data on that repo, causing it to lose track of all installed cogs for purposes of updates. See here.

[orels1]

Thanks for the heads-up. I’ll try to fix that asap

[orels1]

So as I understand it - I should use list_cogs to check if the cog is available. and if so - install it with install or if not - use _repo_add as before

[calebj]

Perhaps. Personally, I think that it would be best to invoke the appropriate methods directly instead of going through ctx.invoke. But as long as it checks that it isn't overwriting anything, any approach that works is better than what it does now.