Skip to content
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

RuneGPT Submission #7459

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

RuneGPT Submission #7459

wants to merge 6 commits into from

Conversation

Kamran14
Copy link

RuneGPT connects with LLM providers to help RuneLite users achieve their goal. With stats preloaded, each response is tailored to the user providing a unique experience.

Currently it only supports Gemini which is available to all users free of charge.
Time constraint

@runelite-github-app
Copy link

runelite-github-app bot commented Feb 18, 2025

@Kamran14
Copy link
Author

Will update once build failures are addressed.

@iProdigy iProdigy added the waiting for author waiting for the pr author to make changes or respond to questions label Feb 18, 2025
@runelite-github-app runelite-github-app bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Feb 19, 2025
@iProdigy iProdigy added the waiting for author waiting for the pr author to make changes or respond to questions label Feb 19, 2025
@runelite-github-app runelite-github-app bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Feb 19, 2025
@Kamran14
Copy link
Author

Failures have been addressed and fixed in new commit.

@iProdigy iProdigy added the waiting for author waiting for the pr author to make changes or respond to questions label Feb 19, 2025
@iProdigy
Copy link
Member

also https://github.com/polyphilz/scapegpt already exists; does this need to be a separate plugin (rather than contributing improvements to the existing plugin)

@runelite-github-app runelite-github-app bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Mar 3, 2025
use latest commit && add warning
@Kamran14
Copy link
Author

Kamran14 commented Mar 3, 2025

  • Fixed mentioned issues
  • config option was introduced while testing api key, forgot to remove
  • compressed img to 18kb, let me know if this is sufficient
  • added warning

regarding existing plugin;

  • ScapeGPT exists but is not a functional plugin as it hasn't been working for months possibly a year now based on user reviews and my own test
  • RuneGPT is open source and time to introduce new features and test end-to-end is fairly easy compared to ScapeGPT due to it having a requirement of a third party server
  • ScapeGPT's third party has not been active for a while so likely the author has abandoned the project

@YvesW
Copy link
Member

YvesW commented Mar 3, 2025

You should create an issue and/or a PR to know for sure as described in the take-over policy. Also, we have a take-over policy for this specific reason.

https://github.com/runelite/runelite/wiki/Plugin-takeover-policy

@Kamran14
Copy link
Author

Kamran14 commented Mar 3, 2025

iProdigy had submitted a PR on that repo 2 weeks ago and there has been no activity from the author; polyphilz/scapegpt#2

  • I'd rather not take over the plugin as that would require me to spin up an instance and maintain that for the backend, and likely will have to pay down the road as well
  • The approaches these two plugins take is quite different and I'd have to change most of the logic that ScapeGPT currently holds
  • ScapeGPT only works with openai currently and requires payment to use, which is against the model I'm trying to use for RuneGPT where players bring their own key

All in all, taking over ScapeGPT isn’t ideal since it would require me to maintain a backend (with potential future costs), its approach differs significantly from RuneGPT's, and it relies solely on OpenAI with a paid model, whereas RuneGPT lets players bring their own API key via Gemini which is free.

@raiyni
Copy link
Member

raiyni commented Mar 3, 2025

We aren't asking you to never change the code of the plugin you takeover.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants