Skip to content

Fix YAML heredoc break from unindented FOOTER lines#303

Merged
jth-nw merged 2 commits intomainfrom
dev
Feb 24, 2026
Merged

Fix YAML heredoc break from unindented FOOTER lines#303
jth-nw merged 2 commits intomainfrom
dev

Conversation

@jth-nw
Copy link
Collaborator

@jth-nw jth-nw commented Feb 24, 2026

The triple-quoted Python string for FOOTER had lines at column 0, which caused YAML to truncate the run block early and bash could never find the PYTHON_EOF terminator. Replaced with string concatenation so every line stays inside the YAML indent level.

Generated with AI

james-haytko_nwx and others added 2 commits February 24, 2026 12:36
The triple-quoted Python string for FOOTER had lines at column 0,
which caused YAML to truncate the run block early and bash could
never find the PYTHON_EOF terminator. Replaced with string
concatenation so every line stays inside the YAML indent level.

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
When reviewing the full document, Claude fixes preexisting issues
on lines outside the PR diff. GitHub rejects the entire review
with HTTP 422 if any suggestion references a line not in the diff.

Fix: parse the PR diff (with --unified=10 context) to build the
set of valid (file, line) pairs before posting. Suggestions for
out-of-diff lines are skipped; they still appear in the prose
summary and can be applied via @claude.

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
@jth-nw jth-nw merged commit fc28e14 into main Feb 24, 2026
6 of 8 checks passed
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