Skip to content

Apply textwrap.dedent to multi-line prompt strings#130

Open
nikivanstein wants to merge 5 commits intomainfrom
claude/dedent-multiline-prompts-tLCzs
Open

Apply textwrap.dedent to multi-line prompt strings#130
nikivanstein wants to merge 5 commits intomainfrom
claude/dedent-multiline-prompts-tLCzs

Conversation

@nikivanstein
Copy link
Copy Markdown
Collaborator

Prevents accidental indentation from being sent to the LLM when
prompt strings are defined inside indented code blocks. Also fixes
a concrete bug in diff_output_format_prompt where the first content
line had 8 spurious leading spaces.

https://claude.ai/code/session_015FyjKEfZyut3DuZcKQNPQe

Prevents accidental indentation from being sent to the LLM when
prompt strings are defined inside indented code blocks. Also fixes
a concrete bug in diff_output_format_prompt where the first content
line had 8 spurious leading spaces.

https://claude.ai/code/session_015FyjKEfZyut3DuZcKQNPQe
claude added 4 commits April 14, 2026 06:56
Removed the task-specific second sentence about metaheuristics so the
default role prompt works across different problem domains without
users needing to override it.

https://claude.ai/code/session_015FyjKEfZyut3DuZcKQNPQe
- black reformatted llamea/llamea.py after the dedent changes
- CLAUDE.md documents the repo layout, dev workflow (uv sync/black/isort/pytest),
  prompt architecture, and code conventions for future sessions

https://claude.ai/code/session_015FyjKEfZyut3DuZcKQNPQe
isort must run before black; running black first can cause black to
fail or produce conflicts with isort's output.

https://claude.ai/code/session_015FyjKEfZyut3DuZcKQNPQe
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 14, 2026

Codecov Report

❌ Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
llamea/llamea.py 85.71% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

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