Skip to content

fix(prompts): Inject raw data description and extra instructions#258

Merged
gadenbuie merged 4 commits into
mainfrom
fix/data-description-raw
Jun 23, 2026
Merged

fix(prompts): Inject raw data description and extra instructions#258
gadenbuie merged 4 commits into
mainfrom
fix/data-description-raw

Conversation

@gadenbuie

Copy link
Copy Markdown
Contributor

Description

The system prompt template used {{data_description}} and {{extra_instructions}}, which HTML-escapes developer-provided text via Mustache. Characters like <, >, &, and quotes (common in instructions referencing SQL conditions, HTML suggestion tags, or quoted column names) were corrupted before reaching the LLM.

Both values are free-form, developer-supplied prose passed through verbatim, so they should be injected raw with triple-mustache ({{{ }}}), matching the existing treatment of data_dicts, tables_overview, and semantic_views.

Changes

  • {{data_description}}{{{data_description}}} (R + Python prompts)
  • {{extra_instructions}}{{{extra_instructions}}} (R + Python prompts)
  • Add shiny_bookmarks/ to .gitignore

@gadenbuie gadenbuie requested a review from cpsievert June 23, 2026 19:54
@gadenbuie gadenbuie merged commit d61b078 into main Jun 23, 2026
17 checks passed
@gadenbuie gadenbuie deleted the fix/data-description-raw branch June 23, 2026 19:54
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