91b6b31
fix(mcp): pass down mcp name to logger (#1078)
Click to expand commit body
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>
Carlos Alexandro Becker
created
96af782
Revert "refactor(tidy): remove nested if and else block"
Click to expand commit body
This reverts commit a6e99fff52b4ab105df91f595a6e94ff3e8bd2cb.
kujtimiihoxha
created
5811c4a
refactor(tidy): remove nested if and else block
d66dfa2
fix(lsp): use csync for lsp clients (#1073)
Click to expand commit body
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>
6c380d4
feat: add attribution settings to config and bash tool (#1025)
Click to expand commit body
* feat: add attribution settings to config and bash tool
* fix(readme): move ephemeral data block back to cfg section
Closes: #445
Amolith
created
b00ffcc
feat: LSP implementation using x/powernap (#1011)
Click to expand commit body
* feat: LSP implementation using x/powernap
Replace custom LSP client implementation with the `charmbracelet/x/powernap` library, significantly reducing codebase complexity by removing ~12,000 lines of custom LSP protocol handling code. This migration introduces:
- **Simplified client architecture**: Replace custom LSP transport, protocol, and method handling with powernap's battle-tested implementation
- **Enhanced workspace support**: Add workspace markers and inactive LSP server detection capabilities from powernap
- **Improved configuration**: Integrate powernap's default LSP server configurations and settings
- **Reduced maintenance burden**: Remove custom protocol definitions, JSON marshaling, and transport layer code
- **More features**: e.g. workspace root markers, starting LSP servers only where these files exist
The refactor maintains existing functionality while leveraging powernap's robust LSP client foundation, making the codebase more maintainable and feature-rich.
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
chore: updates
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: code review
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* revert: unwanted change
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: improve code
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: schema
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: improve merge
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: unneeded changes
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: several fixes
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: more cleanup
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: use csync
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: did close
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: remove unused code
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: cleanup
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: clean
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: cleanup
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* test: fix
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* refactor: improve func
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
* fix: http client debug
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>
Carlos Alexandro Becker
and
Andrey Nering
created
898f1ee
fix(config): look for more than just crush.md
Click to expand commit body
Crush used to show the initialization popup even when the project uses
something other than `CRUSH.md`. Crush already has support for loading
those other context files, like `CLAUDE.md` or `AGENTS.md`, so this just
changes the popup check logic to look for the same set it loads
Co-Authored-By: Crush <crush@charm.land>
there no reason for that to be there.
with it removed, go run ./... will work :)
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Carlos Alexandro Becker
created
480dc02
chore(legal): @Grin1024 has signed the CLA
Charm
created
3d79848
chore(legal): @WhiskeyJack96 has signed the CLA