Skip to content
This repository has been archived by the owner on Aug 11, 2024. It is now read-only.

Re-implement basic prompts with Tracery #1090

Merged
merged 2 commits into from
Jan 31, 2024
Merged

Conversation

gordonbrander
Copy link
Collaborator

@gordonbrander gordonbrander commented Jan 30, 2024

This give us a bit more flexibility in how we generate these prompts, creating opportunities to generate prompts that are cheap yet feel more organic. Also, it will likely come in useful when crafting AI prompts in the future.

  • Ported prompts over to Tracery Grammar
  • Introduced PromptService.
    • PromptService is an actor. This is currently not important, but matches our other services and could allow us to introduce thread-safe state in future. It also forces us to interact with the methods over an async boundary, meaning we could introduce things like HTTP API calls to AI in future.

About the Tracery microlibrary:

This give us a bit more flexibility in how we generate these prompts,
and will likely come in useful when crafting AI prompts in future.
@gordonbrander
Copy link
Collaborator Author

@bfollington did our swift lint settings change recently? I'm getting failures from SwiftLint. However, they seem to be coming from files I did not change.

Copy link
Collaborator

@bfollington bfollington left a comment

Choose a reason for hiding this comment

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

LGTM!

@bfollington bfollington merged commit 0d6bd89 into main Jan 31, 2024
1 of 2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants