perf: set appropiate/missing lazy loading events

[manuuurino]

📑 Description

Noticed some lazy loading improvements from my personal config. These were the one i am using, sorry but i cannot check all plugins, i hope i can improve the startup/running performance a tiny bit further.

[github-actions]

Review Checklist

Does this PR follow the [Contribution Guidelines](development guidelines)? Following is a partial checklist:

Proper conventional commit scoping:

• If you are adding a new plugin, the scope would be the name of the category it is being added into. ex. feat(utility): added noice.nvim plugin

• If you are modifying a pre-existing plugin or pack, the scope would be the name of the plugin folder. ex. fix(noice-nvim): fix LSP handler error

• Pull request title has the appropriate conventional commit type and scope where the scope is the name of the pre-existing directory in the project as described above

• README is properly formatted and uses fenced in links with <url> unless they are inside a [title](url)

• Proper usage of opts table rather than setting things up with the config function.

[Uzaaft]

@mehalter

[luxus]

will it not get loaded when using the mappings?

[mehalter]

A few of these need to be reverted because they are significantly less performant. Other than that, thanks for the recommendations!

EDIT: realized lazy = true is no longer default, so what most of these actually just need is lazy = true since they are all lazy loaded based on module.

[mehalter]

Suggested change

	event = "VeryLazy",
	lazy = true,

This should not be lazy loaded on event. It is lazy loaded on module and keybinding. We should make it lazy = true.

[mehalter]

Suggested change

	event = "User AstroFile",
	event = { "User AstroFile", "InsertEnter" },

This lets it work on new files as well

[mehalter]

Suggested change

	event = "User AstroLspSetup",
	lazy = true,

This lazy loading is handled by the autocmd and should not be loaded on event. This is less performant. It should just be lazy = true.

[mehalter]

Suggested change

	event = "User AstroFile",
	ft = { "yaml", "json" },
	lazy = true,

this should be reverted. This is more performant as is. It is lazy loaded based on module only when needed.

[mehalter]

I went in and resolved the performance issues. Thanks!