-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Codeium #8780
Add Codeium #8780
Conversation
Please add your package to the "c" repository file. |
Are you sure I need to add it to the "c" file? I'm following the instructions here with the "Manual Hosting" route and it doesn't mention that: https://packagecontrol.io/docs/submitting_a_package#Step_6 |
Oh, sorry, you're proposing a "closed source" package. Any particular reason for that? You don't seem to be monetising it directly. Note that sublime-package files are simply zips that can be extracted. |
|
Not sure whether this will work for package updating. Especially, putting a newer version after a older one. {
"schema_version": "3.0.0",
"packages": [
{
"name": "Codeium",
"releases": [
{
"sublime_text": "*",
"version": "1.2.26",
"url": "https://github.com/Exafunction/codeium-sublime/raw/main/Codeium.sublime-package"
},
{
"version": "1.2.38",
"sublime_text": "*",
"url": "https://github.com/Exafunction/codeium-sublime/raw/main/Codeium.sublime-package"
}
]
}
]
} |
I'd like to do a closed-source package not because I'm worried about monetization or source code exposure but because the package currently only works properly with generated python protobuf files and I did not want to have to commit these to the repo. If the "Manual Hosting" route is not an option and is a blocker for getting this on package control, we can go with the GitHub hosting route instead, just let me know. Re the other comments:
I'll remove these.
Can you clarify what is wrong? Is it just the wrong order? I can fix that, but is there anything else? |
The pre-packaged solution is not a problem in and of itself. It does prevent users from seeing what goes on inside the package (default settings, key bindings, any other inner working), and providing feedback or contributions. As you're currently proposing it, users will only be able to find the package by name directly. There will not be any content for it on the package control site, no description or labels that helps users find the package and understand what it's for. There is not even a readme or anything that will tell users what the package does or how it can be used. Or even so much as a place for users to report issues. Even searching the web for "Exafunction codeium" doesn't tell me much about what it is or what it does. We might go on and review the package contents later, but as it stands, just the proposed entry in the package control registry is not sufficient. |
Installing this in Safe Mode for Sublime and it freezes ST completely. My assumption is that it is because you are downloading the language server in the main thread which blocks the UI. This should likely take advantage of threading or a similar to background this tasks. LSP does a great job at doing this in a none blocking way for its language servers.
|
If it’s “just” a language server, consider depending on the LSP package. |
Iirc, Package Control uses the first match from |
It's not a traditional language server (in particular it doesn't actually implement the language server protocol). Your point about freezing the UI is well-taken though. I'll fix this and let you know once I've done so. Regarding Manual Hosting vs GitHub Hosting, I took a look at the Package Control link for a manually hosted package: https://packagecontrol.io/packages/Tinkerwell If I add labels, a homepage, and a description would that be sufficient? |
Yes, that should work. The labels and description would help users find the package, and I would assume the homepage would cover everything else that users need around how to use the package, reports issues, etc. It would be even better if you could provide links for “readme” and “issues” too of course. |
Adding a readme link is strongly recommended as that is what the packagecontrol.io site will render for the package. Further feedback after taking a look at the source:
|
Alright a list of updates:
If you could give it another round of feedback, that would be much appreciated. |
Create and return |
Also, a quick hint already that this is still an issue. You added a new release triple but all releases point to the exact same upstream URL, making them essentially equivalent. Now, PC doesn't support installing older releases anyway (at the moment), but this could also mean that in the event that you replaced the archive and did not change the version in your Also also, I would also still strongly recommend using a code repository as your basis and building the package archive in an automated fashion, for example using a Github Action, as that will improve the ability to invite contributors and transparency about the package's contents. (Admittedly, that can still be tampered with, but it's a sign of good faith I'd say.) From experience, I can say that helping people with issues in their local setup and a specific package is much easier if the package's contents are readily available for viewing and you don't need to manuall download and extract them. And a last tip, you can still use the |
I looked into |
(Ugh, I didn't mean to close this) |
Hi all, just checking in to see if there's anything else you want me to do here. Thanks for all the feedback you've provided so far! |
Hi, I haven't had the time to look at it again but intend to do so over the weekend. |
Looks good to me. Following are just some comments/recommendations you could also implement but are not required to have this merged. In fact, I'll do so now.
|
Thanks for merging. Do you know when I can expect Package Control to get updated with it? I don't see Codeium in the Channel json yet and I just wanted to make sure that there was nothing else I needed to do. |
It should definitely be there by now, unless there is a problem with the file that I can't spot with my eyes alone. There is no entry in the cache for https://raw.githubusercontent.com/Exafunction/codeium-sublime/main/packages.json. @braver, do you have an idea? |
Nope, sorry, not a clue. We've had cases where @wbond had to go in and manually vet something. Hope it's nothing like that though. |
Hmm, is there any way you could reach out to wbond? I'd really like to resolve this... |
Sorry for the delay in this - hoping to look into it tonight |
Did you get a chance to look into this? |
I’ve spent a couple of times over the past 48 hours debugging but haven’t figured it out yet. I believe it is a bug in the crawler - something to do with the package data |
I see that the crawler is open source, if you can give me a couple pointers I can look into it as well if that's helpful |
The packages.json file at https://raw.githubusercontent.com/Exafunction/codeium-sublime/main/packages.json is missing the It was harder to debug since the package never made in into the DB since it was broken from the start, and thus no page was created upon which to show the error that the date was missing. |
Oops my bad. Thanks so much for looking into this! I've added a |
Yay it worked! https://packagecontrol.io/packages/Codeium |
The package checker never ran for this addition because … it didn't work and we don't know why. The setup is still too fragile but I lack the time to work on it, much like everyone else. (Note: that means running the checker entirely via GitHub Actions to make it easier to debug.) |
As a remainder, I've already done that and I'm using it in the LSP Package Control channel https://github.com/sublimelsp/st-package-reviewer-action But I suppose it would still require time to switch to it, test and all. |
Yes, the tracking issue is #8050 |
My package Codeium briefly was listed as missing because of a bad PR. Could you re-review and approve it? |
Unfortunately, only @wbond can do this. |
Any updates on this? The package can not be installed |
Package should be available again. |
Can confirm, is up and running. Thank you! |
I confirm too, thx for the work |
My package is Codeium, an AI-powered code completion tool.
There are no packages like it in Package Control.