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

New Plugin: AI #482

Open
wants to merge 7 commits into
base: plugin_api_v2
Choose a base branch
from

Conversation

shamanicvocalarts
Copy link

AI is a Flow Launcher plugin for querying LLM from Flow Launcher via OpenRouter

AI is a Flow Launcher plugin for querying LLM from Flow Launcher via OpenRouter
Copy link
Contributor

@cibere cibere left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, and thanks for the plugin submission. If you don't mind, I have a couple of changes that I would like you to make.

  1. Could you add the json file extension to the file you made in this PR?
  2. It seems like you just copy pasted the contents from your plugin.json file. However, data provided in this repository is slightly different than the data in your plugin.json file. Here is a template that you can use:
{
  "ID": "Unique GUID from your plugin.json",  
  "Name": "Plugin name",
  "Description": "Short description",
  "Author": "Author",
  "Version": "Version from your plugin.json",
  "Language": "Programming language",
  "Website": "Plugin website",
  "UrlDownload": "URL to download",
  "UrlSourceCode": "URL to source code",
  "IcoPath": "Plugin icon image's CDN URL, e.g. https://cdn.jsdelivr.net/gh/Flow-Launcher/Flow.Launcher/Plugins/Flow.Launcher.Plugin.Explorer/Images/explorer.png"
}
  1. Could you change the name to something more descriptive? Perhaps OpenRouter AI?
  2. Could you add the api key to the settings template? Supporting the env variable is fine, however it would be best to have a unified settings experience between all plugins if possible. There is a password field that would work well here.

@shamanicvocalarts
Copy link
Author

Hi, happy to make those changes, sorry for 1 & 2 my bad 😅.

for name, I am happy to change it , though I am intending to support more than openrouter for platform/features in the future with this plugin.
open-webui for instance , for opening initial query in a chat window .
open-webui & ollama for embeddings & knowledge collections .

maybe a name like AI assistant may be more suitable?

happy to add API key field back in

@cibere
Copy link
Contributor

cibere commented Mar 10, 2025

for name, I am happy to change it , though I am intending to support more than openrouter for platform/features in the future with this plugin. open-webui for instance , for opening initial query in a chat window . open-webui & ollama for embeddings & knowledge collections .

maybe a name like AI assistant may be more suitable?

Perhaps, though the only reason I asked for the change was to help differentiate the AI plugins (there are so many different ai models that the name AI could be confusing/misleading). If you are planning to add those other models though, then I'm unsure what a good name would be.

@shamanicvocalarts
Copy link
Author

for name, I am happy to change it , though I am intending to support more than openrouter for platform/features in the future with this plugin. open-webui for instance , for opening initial query in a chat window . open-webui & ollama for embeddings & knowledge collections .
maybe a name like AI assistant may be more suitable?

Perhaps, though the only reason I asked for the change was to help differentiate the AI plugins (there are so many different ai models that the name AI could be confusing/misleading). If you are planning to add those other models though, then I'm unsure what a good name would be.

to clear up any potential confusion, OpenRouter is not a model, instead it aggregates many API/models - so out of the box plugin already supports 100s of different models, via Openrouter you can query openAI, Anthropic, Deepseek & many others. this was why AI felt suitable to me as a name to begin with, as this plugin gives you access to 90% of llm models available on the market.

updated URL, Updated name, added back in API key and API key tetss to main plugin
… to AI-Assistant-8226842d-f2e2-4159-940c-966666ce7b44.json
…ssistant-8226842d-f2e2-4159-940c-966666ce7b44.json
Copy link
Contributor

@cibere cibere left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, however it still needs to be manually tested with flow

@shamanicvocalarts
Copy link
Author

Nice one thanks @cibere , let me know if there's anything I need to do on my side. & sorry for excessive commits, I'm quite new to PR process

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.

2 participants