Skip to content

Conversation

@hawkgs
Copy link
Member

@hawkgs hawkgs commented Oct 24, 2025

Provide system instructions as a separate prompt via the system option parameter of Genkit.generate instead of concatenating them to the main executable prompt.

…em opt param

Provide system instructions as a separate prompt via the `system` option parameter of `Genkit.generate` instead of concatenating them to the main executable prompt.
Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

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

LGTM. Did you try and see if this changes some of the results out of curiosity?

@hawkgs
Copy link
Member Author

hawkgs commented Oct 24, 2025

I've ran just a single eval to test if things are working as intended, so I don't have any objective measurable feedback to provide. I guess we'll need to run more tests to verify if there is a significant impact. Nonetheless, there should be some positive impact given that LLMs should treat system prompts differently.

@devversion devversion merged commit 5975565 into angular:main Oct 24, 2025
3 checks passed
@hawkgs hawkgs deleted the hawkgs/genkit-use-system-prompt branch October 24, 2025 13:24
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