Skip to content

perf(core): do not require calling setup(), add lazy initialization - #1413

Merged
deathbeam merged 1 commit into
CopilotC-Nvim:mainfrom
deathbeam:add-lazy-init
Sep 13, 2025
Merged

perf(core): do not require calling setup(), add lazy initialization#1413
deathbeam merged 1 commit into
CopilotC-Nvim:mainfrom
deathbeam:add-lazy-init

Conversation

@deathbeam

Copy link
Copy Markdown
Collaborator

Refactored plugin initialization to use lazy loading via metatable, setting an initialized flag and moving setup logic to init(). Replaced add_providers with set_providers for clarity. Improved health check to verify initialization. Cleaned up chat state handling and logging logic for better reliability and maintainability.

@deathbeam deathbeam added enhancement New feature or request refactor labels Sep 13, 2025
Refactored plugin initialization to use lazy loading via metatable,
setting an `initialized` flag and moving setup logic to `init()`.
Replaced `add_providers` with `set_providers` for clarity. Improved
health check to verify initialization. Cleaned up chat state handling
and logging logic for better reliability and maintainability.

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
@deathbeam deathbeam changed the title refactor(core): do not require calling setup(), add lazy initialization perf(core): do not require calling setup(), add lazy initialization Sep 13, 2025
@deathbeam
deathbeam merged commit c15f65e into CopilotC-Nvim:main Sep 13, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant