Skip to content

feat: add logging to file#2947

Open
teto wants to merge 2 commits intoyetone:mainfrom
teto:teto/add-logging-pr
Open

feat: add logging to file#2947
teto wants to merge 2 commits intoyetone:mainfrom
teto:teto/add-logging-pr

Conversation

@teto
Copy link
Contributor

@teto teto commented Jan 30, 2026

avante is a complex program, especially as lua doesn't enforce strong typing. As I work on it, I find myself adding log statements to understand what's going on.
The current backend for logs is not practical as it's intrusive, generates a popup everytime, and it doesn't distinguish between userfacing and dev-related logs, as error/warn just rely on vim.notify.

This introduces a cheap logger as I cant find any I am really confident with on luarocks.org and I dont want to add a dependency I am not confident about.

avante is a complex program, especially as lua doesn't enforce strong
typing. As I work on it, I find myself adding log statements to
understand what's going on. 
The current backend for logs is not practical as it's intrusive,
generates a popup everytime, and it doesn't distinguish between
userfacing and dev-related logs, as error/warn just rely on vim.notify.

This introduces a cheap logger as I cant find any I am really confident
with on luarocks.org and I dont want to add a dependency I am not
confident about.

[pre-commit.ci lite] apply automatic fixes
@teto teto force-pushed the teto/add-logging-pr branch from 668a3b3 to d826cf1 Compare February 5, 2026 17:59
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.

1 participant