Update: decky-download-all to v1.1.0#1026
Update: decky-download-all to v1.1.0#1026bentemple wants to merge 1 commit intoSteamDeckHomebrew:mainfrom
Conversation
Issues FoundNo issues with your PR description were found. Next Steps
Thank you for your contribution! If you need any help, please reach out on our Discord server. ❤️ |
55c3d97 to
d2dd1c7
Compare
|
Be me: The changes are slightly more than they would have to be, but I left the logging (from web->python so I can just debug with ssh) in because felt nice to keep that for anyone using it -- in-case they would want logs for some reason. Also makes debugging future issues a lot easier. |
|
Also, re generative AI: If majority means I initially started with claude to make this change, initially, then yes. If majority means I just let claude do whatever it wanted and then didn't review, edit, and test the code, and redirect claude when it decided to make a shitty change, then no, I didn't. I have gone through the changes, verified them, understood them and edited them as-needed This isn't an AI slop PR. I think one of the worst things AI does is make too many changes at once. I added, and then left in the logging stuff as kind of a completeness thing because it felt like something I would want as an end-user. (i.e. as a user, I would kind of expect the logging of a plugin to include the actions it takes, but I could make this PR super lean by not including all of main, but instead just the change required to fix 3.8+) And then it will make debugging future issues easier, but if that code being included causes any hesitation, I'm happy to split it out and trim this down to exactly the change required to fix 3.8+ |
Fixes download-all not working on SteamOS 3.8+ due to the new format associated with controlling downloads on remote machines
d2dd1c7 to
e8be63f
Compare
|
Also, re the submitting plugins: When I first read this, I had understood this to mean any plugin that uses AI (leveraging AI, in other words), but re-reading in conjunction with this new check-box added (between my initial plugin and this update) I'm not so sure, that's the case. and it is in-fact a commitment that no plugin can use generative AI to help build it. I certainly don't disagree with the sentiment here. generative AI is definitely and 100% a direct result of GPL code, and therefore all AI should be GPL, imo, including anything written by AI. Anyway, I like my plugin. I think it's simple, focused, and solves an annoying problem. But if it needs to just be for me, then so be it. I don't want to break any community rules here, just trying to make useful shit for myself, and then because I love OSS taking the time to craft it to a quality bar that I think is high enough to share with others as well. |
|
Our main concerns are making sure all code is secure, tested, and properly licensed (given you cannot license AI-generated code unless under certain circumstances per US copyright law). Since this is a fairly small change, I don't think this is an issue. |
Got it. Appreciate the reply. Yeah by the logic presented here IMO, the only ethical application of an llm IS in producing public GPL code. Which this is of course under the GPL license. Thank you for the response. I definitely did not slop this out and one of the best defenses against AI slop, IMO, is atomic commits which require splitting and carefully reviewing every line changed. Which I did here (as I will always do when leveraging an LLM) I really like how Jellyfin has clearly defined their AI policy. Would love to see some better clarity around the policy here. Sorry, for the slight freak-out. was just my 2AM brain going: did I completely mis-read this policy etc... |
mdeguzis
left a comment
There was a problem hiding this comment.
No major issues seen in code submission. Auto builds (https://github.com/bentemple/decky-download-all/actions) look clean.
mdeguzis
left a comment
There was a problem hiding this comment.
Your LICENSE is incorrect. Please see the license of the plugin template repo and replace "Hypothetical Plugin Developer".
Update decky-download-all to v1.1.0
Fixes download-all not working on SteamOS 3.8+ due to the new format associated with controlling downloads on remote machines
Fixes bentemple/decky-download-all#1
Task Checklist
Developer
-- I did use claude to help make these changes, I'm not a web-dev by trade, so it helps me bridge that gap (Java, C++, C, Kotlin, Python are my main native languages), but I have reviewed, and modified all of the code that was changed. See below comment on this. These are crafted commits, that I made. Not AI slop, but I don't really know the intention behind this check-box, but I do want to disclose the use.
Plugin
Backend
Community
Testing