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

support rendering prompty in importTemplate #786

Merged
merged 2 commits into from
Oct 18, 2024
Merged

support rendering prompty in importTemplate #786

merged 2 commits into from
Oct 18, 2024

Conversation

pelikhan
Copy link
Member

@pelikhan pelikhan commented Oct 18, 2024


Pull Request Description

Summary of Changes

  • System Role Integration 🛠️

    • Added support for system role in various components.
    • Introduced systemText variable to handle system messages.
    • Updated message handling to include system messages.
  • New Interfaces and Functions 🆕

    • Created PromptSystemNode interface for system role nodes.
    • Added createSystemNode function to generate system role nodes.
  • Visitor and Renderer Updates 🔄

    • Enhanced PromptNodeVisitor to support system nodes.
    • Modified visitNode and renderPromptNode functions to process system nodes.
    • Adjusted resolvePromptNode to handle system nodes.
  • Prompt Template Enhancements 📜

    • Updated promptyToGenAIScript to support system role.
    • Enhanced createChatTurnGenerationContext to handle system role messages.
    • Added role property to WriteTextOptions for specifying message roles.
  • Bug Fixes and Minor Improvements 🐛

    • Implemented various minor fixes and improvements to ensure seamless integration of the system role.

These changes aim to enhance the flexibility and functionality of the prompt handling system by introducing a new system role, allowing for more structured and organized message processing.

generated by prd

@pelikhan pelikhan merged commit 5d645b3 into main Oct 18, 2024
7 of 10 checks passed
@pelikhan pelikhan deleted the system-mg branch October 18, 2024 19:23
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.

1 participant