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

feature request: plugin API for extending the fsql abilities #39

Open
hye opened this issue Feb 15, 2018 · 3 comments
Open

feature request: plugin API for extending the fsql abilities #39

hye opened this issue Feb 15, 2018 · 3 comments

Comments

@hye
Copy link

hye commented Feb 15, 2018

Super cool tool you made , thanks @kshvmdn
I would like to see some plugins, (how to extend the fsql)
for example

@kashav
Copy link
Owner

kashav commented Mar 10, 2018

Hey @hye, thanks for the issue! Sorry about the late response – been really busy with school lately.

I want to spend some time thinking about how (and if) a plugin system would integrate with fsql, but I think it's a great idea! It'd be a great way to continue expanding fsql without having to force features upon people. Not to mention, it'd probably make it easier for people to add custom functionality as well.

I'll see if I can come up with something concise/clean/simple, please do let me know if you have any ideas!

@Nexuist
Copy link

Nexuist commented Nov 11, 2021

Hey guys, I just stumbled upon fsql and am enjoying using it so far. Wanted to suggest that instead of a plugin API you should just add an exec() attribute modifier so that we can just write scripts in bash/python/etc to use as plugins. For example, git,exif, and tag are already command line tools so it would be really easy to integrate them with exec(). There's also the possibility of using the output of exec() so that you can e.g. perform HTTP requests or read file contents:

SELECT name FROM . WHERE "tomatoes" IN (SELECT exec("cat", name) FROM .)

This would be the easiest way to greatly increase the usefulness of fsql IMO.

@kashav
Copy link
Owner

kashav commented Jan 30, 2022

@Nexuist, hey, thanks for reviving this.

I'd be happy to review/work with someone on a patch that implements this. I do worry that this is perhaps too powerful and people (me) are going to start cat-ting their whole home directory without realizing it, but I think it could be a fun feature to have, so why not!

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

No branches or pull requests

3 participants