Fix typos and broken numbering in the summarizer prompt - #175
Merged
Conversation
Reported by @bhavyakeerthi3 in #157, verified against current main: adn -> and infromation -> information (twice; once in the system prompt, once in the human message template) retining -> retaining drived -> derived (twice, in "Reactome-drived" and "UniProt-drived") Also "key biological terms terminology" -> "key biological terminology", and a doubled space. The instruction list numbered 1,2,3,4,5 then 4 again, then 5,6,6 -- so three of nine items shared a number with an earlier one. Renumbered 1-9. This is prompt text the model reads, and a list that contradicts itself about its own ordering is not a good instruction to follow. Checked the rest of src/ for the same misspellings; there are none. This is the Cross-Database Prototype profile, which config_default.yml does not enable, so no user sees these today. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reported by @bhavyakeerthi3 in #157, verified against current
main.Typos
Also
key biological terms terminology->key biological terminology, and a doubled space.Broken numbering
The instruction list ran
1,2,3,4,5then 4 again, then5,6,6— three of nine items shared a number with an earlier one. Renumbered 1–9.This is prompt text the model reads. A list that contradicts itself about its own ordering is a poor instruction to follow, and "4" appearing twice makes the second one ambiguous to reference.
Scope
Checked the rest of
src/for the same misspellings — there are none.This is the Cross-Database Prototype profile, which
config_default.ymldoes not enable, so no user sees these today. Worth fixing before that profile is revived rather than after.Taken from #157; the rest of that PR (wiring the dead
perform_web_searchroute) is real but sits in the same disabled profile and is tracked separately.🤖 Generated with Claude Code