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

Created a new Package for espanso - OASIS-IS Prompt Framework #123

Merged
merged 10 commits into from
Aug 20, 2024

Conversation

Ryfter
Copy link
Contributor

@Ryfter Ryfter commented Jul 29, 2024

This is a mega-prompt framework form for prompting in ChatGPT, Claude and other chatbots. It has two modes.

  1. Using it in a URL bar or any text field it creates a prompt that will run in chatGPT. This is the base version that allows users to get the full prompt in the form of a URL. This only works for ChatGPT.
  2. This is meant to be used in any prompt box. It does not include the URL, but just the text for the prompt.

…omting in ChatGPT, Claude and other chatbots.
@smeech
Copy link
Collaborator

smeech commented Jul 29, 2024

That's an impressive piece of work, which passes all the automated checks.

I am interested to note that although the indentation of the triggers as a whole, and also the first replace: block, doesn't follow our convention, and sets off a lot of linter warnings here, it still works. It's good to question these things!

No script:, shell:, cmd: or malicious code.

specification: is empty in both triggers - just checking this is correct, and I take it :poasis doesn't need a model: section?

There's a lot of duplicated code between the two expansions, which could be shared using Global Variables or Anchors and Aliases? I've put a suggestion here for your interest. As I say in the comments, it's not pretty!

@Ryfter
Copy link
Contributor Author

Ryfter commented Jul 30, 2024

Thank you for your comments. I'll dig into it, and optimize it. I spent most of Sunday just getting that far. I appreciate you taking the time to give me feedback.

:Oasisform creates a prompt as a URL. You can see it in action better in the spreadsheet I created for it: http://3dmkf.com/pace-oasis You fill out the fields in E2 to E8. I converted that into an espanso form. It inserts the model as part of the URL.

The other one :poasis , the p stands for prompt. It is meant to go into the prompt. So, in ChatGPT, you choose the model from the dropdown in the top left of the chat screen.

Ryfter added 3 commits July 30, 2024 16:50
Cleaned it up a bit, but it is still kind of a beast.  I probably need to optimize the variables a bit, since I basically have two of the same prompt.  The url prompt creates a URL for prompting, so it uses %0A to add the line break.  The boxprompt gets pasted in as text and it uses \n for newline.  The box prompt also doesn't define the model, so that is not in that one.  

urlprompt uses :oasis
boxpromt uses :poasis
Added a little more clarity to the instructions.
Made some minor updates to the readme and tweaked the package some based on recommendations.
@smeech
Copy link
Collaborator

smeech commented Jul 30, 2024

Thank you. That's neat.

formo1 throws linter errors here, but works nevertheless.

To make it more tidy, I was playing around with trying to separate the Interpretation values list into an echo: global_var, but hadn't managed to make it work.

Just one critical problem - the path version (currently 0.1.0) has to match the version: value in _manifest.yml (now 0.1.1), I'm afraid, before the CI will accept it.

Once this is fixed, and you're happy with the package, I'll merge it.

A few more typos, by line number:
5. Evaluate
6. available
8. will
17. & 30. additional

@smeech smeech changed the title Created a new Pakcage for espanso - OASIS-IS Prompt Framework Created a new Package for espanso - OASIS-IS Prompt Framework Jul 30, 2024
Ryfter added 3 commits August 20, 2024 11:05
Updated readme that should fix any errors.
This should hopefully fix any errors.
Updated Manifest for updated OASIS-IS framework files.
@smeech
Copy link
Collaborator

smeech commented Aug 20, 2024

Thank you! That now passes all the CI checks.

There are still a few typos - you might wish to correct the one at the top of README.md at least, as it's the one likely to be seen (the ones in the package.yml comments, less so)!

Let me know when you're happy, and I'll hit the Merge button.

Ryfter added 3 commits August 20, 2024 11:40
Running Spell check to reduce embarrassment.
Run through spellcheck.
@smeech smeech merged commit 5f085a6 into espanso:main Aug 20, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants