closes #806

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Co-authored-by: taigr <taigr@users.noreply.github.com>

This ensure we'll collect `zsh` instead of `/bin/zsh` or
`/opt/homebrew/bin/zsh`, for example.

Add pretty-printed JSON parameters to MCP tool permission dialogs so
users can read more of the content.

Co-Authored-By: Crush <crush@charm.land>

This change extends the provider configuration to allow users to define custom providers with `type: "gemini"`.

This enables connecting to any Gemini-compatible API by specifying its `base_url` and `api_key` within the `providers` section of `crush.json`. It supports complex setups, such as using a local proxy or a model-balancing service.

This mimics the behavior of Claude Code and allows folks who use
Crush from within a terminal emulator that captures the escape key
(like Nvim's built in terminal emulator) to use it.

* fix: disable providers

if we remove then from the list, they'll still show up because they
  won't get merged with catwalk providers later on.

closes #1037

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

- Replace mcpClientTools with mcpClient2Tools for direct tool storage
- Consolidate tool updates in updateMcpTools function
- Improve error handling by removing client on connection errors
- Remove redundant client deletion in getTools function

- Remove globalCtx field from agent struct
- Refactor setupEvents to accept context parameter directly

* fix(mcp): improve timeout errors

refs #970

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix: improve

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

---------

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix(lsp): load defaults by name or command name

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix(lsp): simplify

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

---------

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

refs #970

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

closes #382

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix(lsp): powernap load by name

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix(mcp): expand command

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix(lsp): expand command

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

---------

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

- windows/arm/v7 was marked broken by the Go team
- add `-trimpath` to trim the `/home/etc` from logs

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

This commit ensures that the "Yes" and "No" options in the init screen
get properly selected when navigating with the keyboard.

Related: https://github.com/charmbracelet/crush/issues/1048

The summarization dialog was getting stuck in "Generating..." state due to two issues:
- Agent was formatting nil error values instead of actual response errors
- Dialog only handled summarize events but not error events, leaving it in loading state

💖 Generated with Crush

Co-authored-by: Crush <crush@charm.land>

improved zig support into https://github.com/charmbracelet/x/pull/567

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

The environment variable indicated to configure the Azure AI endpoint
did not match the value in [Catwalk][].

[Catwalk]: https://github.com/charmbracelet/catwalk/blob/9d9a86e3a4aec20b8874175c115a3533d5083b54/internal/providers/configs/azure.json#L6

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

This will make the logs more useful. Right now it would simply print the
MCP's library logs without any additional context, so you wouldn't know
which MCP is causing issues.

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

This reverts commit a6e99fff52b4ab105df91f595a6e94ff3e8bd2cb.

The map was being passed down everywhere, but the locking mechanism only
ever lived in `app.go`, which might cause concurrent access issues.

This changes it to a `*csync.Map`.

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* feat(lsp): remove internal watcher

It was only ever useful if the user edits files through their editor,
but we don't really need it assuming we only care about edits done by
Crush itself.

Basically, we lose that functionality, but save a bunch of file
descriptors and have improved perf.

Damn good deal if you ask me.

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix: cleanup

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix: remove rlimit

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix: more cleanup

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

---------

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

Co-authored-by: Andrey Nering <andreynering@users.noreply.github.com>