Surface raw HTML content through translation pair pipeline for allowH…#740
Open
dadukhankevin wants to merge 5 commits intomainfrom
Open
Surface raw HTML content through translation pair pipeline for allowH…#740dadukhankevin wants to merge 5 commits intomainfrom
dadukhankevin wants to merge 5 commits intomainfrom
Conversation
…tmlPredictions The allowHtmlPredictions toggle was incomplete: HTML was stripped at SQLite index time but the raw content (already stored in s_raw_content/t_raw_content columns) was never surfaced to the prompt builder. Now rawContent flows through MinimalCellResult so buildFewShotExamplesText can use HTML-preserving examples when the toggle is on. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Restructure system message: consolidate 12 appended instructions into focused paragraphs, reduce noise and redundancy - Simplify HTML preservation: always instruct model to preserve HTML from source, toggle only controls whether HTML is present in examples/context - Fix temperature passthrough: always send configured temperature instead of silently dropping it for the default model - Remove redundant "Instructions" header from user message - Add diagnostic logging for system message and few-shot example HTML Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Keep raw source HTML in the current-task prompt when HTML predictions are enabled while still using sanitized text for example search. Add regression coverage to ensure the prompt preserves source markup and spacing boundaries. Made-with: Cursor
…ions - Adjusted assertions to verify that the system message includes the target language "fr" or "French". - Updated format instructions check to ensure it mentions HTML/formatting handling instead of just plain text when HTML is disabled.
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
2 participants
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.
…tmlPredictions
The allowHtmlPredictions toggle was incomplete: HTML was stripped at SQLite index time but the raw content (already stored in s_raw_content/t_raw_content columns) was never surfaced to the prompt builder. Now rawContent flows through MinimalCellResult so buildFewShotExamplesText can use HTML-preserving examples when the toggle is on.