search

yaroslavyaroslav / OpenAI-sublime-text

First class Sublime Text AI assistant with GPT-4o and llama.cpp support!
MIT License
130 stars 11 forks source link

Key Bindings can not be set #3

Closed eusonlito closed 1 year ago

eusonlito commented 1 year ago

When you click the Preferences > Package Settings > OpenAI completion > Key Bindings menu option the console show the error:

error: The settings file "res://Packages/openAI/Default.sublime-keymap" could not be opened

You have defined the key bindings base file here:

https://github.com/yaroslavyaroslav/OpenAI-sublime-text/blob/develop/Main.sublime-menu#L27

But the base_file Default.sublime-keymap not exists in the repository.

rchl commented 1 year ago

Also, after adding that file, I would suggest to include a suggested key binding there but commented out. Then the user can easily copy it over to user key bindings and enable. Another reason this is useful is because user doesn't know what command and arguments to use for the key binding since it's not documented. So maybe include all three different key bindings with different mode arguments.

yaroslavyaroslav commented 1 year ago

It was, since it's working for me, but I guess it has gone somewhere within package publication process. I'll move it back shortly, and punch update.

yaroslavyaroslav commented 1 year ago

It could worth it. I'm not promise to provide full command list, but I'll do so with the bare minimum for sure.

raineorshine commented 1 year ago

I ran into the same error with v1.1.2. I had to manually add Packages/User/openAI.sublime-settings. It still gives the error when I try to access the Settings through the menu, but at least it reads my API key.

yaroslavyaroslav commented 1 year ago

@raineorshine I see that it's different one. The original was not about sublime-settings, but about sublime-keymap which was really dropped from the project of the review stage.

Could you please open new issue, and provide some details, like: Sublime version, OS version, error log if any.

raineorshine commented 1 year ago

I get the same error for both settings and keymap. I'll open an another issue when I'm back at my computer!