Skip to content

feat: add additional_instructions parameter to GeoFilterParser#87

Merged
fredj merged 1 commit intomainfrom
additional_instructions
Apr 22, 2026
Merged

feat: add additional_instructions parameter to GeoFilterParser#87
fredj merged 1 commit intomainfrom
additional_instructions

Conversation

@fredj
Copy link
Copy Markdown
Contributor

@fredj fredj commented Apr 21, 2026

Allows callers to inject custom rules (region-specific endonyms, domain aliases, organization-specific place names) as a dedicated system message positioned after the main system prompt and before the few-shot examples, without forking the default prompt.

zurfjereluhmie
zurfjereluhmie previously approved these changes Apr 22, 2026
Copy link
Copy Markdown
Member

@zurfjereluhmie zurfjereluhmie left a comment

Choose a reason for hiding this comment

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

LGTM, a few minor suggestions below.

Comment thread tests/test_parser_streaming.py Outdated
Comment thread etter/prompts.py Outdated
Allows callers to inject custom rules (region-specific endonyms, domain
aliases, organization-specific place names) as a dedicated system message
positioned after the main system prompt and before the few-shot examples,
without forking the default prompt.
@fredj fredj merged commit 5a8c872 into main Apr 22, 2026
4 checks passed
@fredj fredj deleted the additional_instructions branch April 22, 2026 07:30
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