Include the code for the fidget spinner in Codecompanion

[ilan-schemoul]

Documenting ~100 loc that needs to be copy pasted by ever user is not the best. Any change would need to be manually replicated by every user.
Ideally we want native integration (if fidget is available then setup fidget)
At the very least just copy paste the code in a file of the doc to somewhere in the repo so we can just require that file from outside and manually set it up.

[olimorris]

I disagree.

Mandating how my plugin takes over your Neovim UI is selfish and opinionated. Providing you with events and hooks that you can leverage to your desire and providing examples of how is more than reasonable.

If this were VS Code then I would agree.

[ilan-schemoul]

Mandating how my plugin takes over your Neovim UI is selfish and opinionated. Providing you with events and hooks that you can leverage to your desire and providing examples of how is more than reasonable.

That would not be mandating because anyone would still be free to enable/disable integration of spinner (pretty common pattern) (either you provide a file then user need to import it or user pass a bololean to enable in the config fidget spinner).

Also like I don't think anyone really cares that much about how the text inside the spinner that much. I'd assume 99.9% just want visual feedback when LLM is loading. 0.1% (or whatever the percent) just will not opt in for the integration and manually write it