Skip to content

fix(triage): drop apostrophe from MODE text that broke bash quoting#3325

Merged
bokelley merged 1 commit intomainfrom
bokelley/triage-fix-apostrophe
Apr 26, 2026
Merged

fix(triage): drop apostrophe from MODE text that broke bash quoting#3325
bokelley merged 1 commit intomainfrom
bokelley/triage-fix-apostrophe

Conversation

@bokelley
Copy link
Copy Markdown
Contributor

PR #3316 introduced a MODE directive containing PR's head branch. Inside the jq filter (which is wrapped in single quotes for bash), the apostrophe closed the bash string and turned the parenthetical (do not open a new PR) into a subshell, producing syntax error near unexpected token '(' on workflow run 24968657038.

Rephrase to remove the apostrophe. Local repro confirms the payload now generates cleanly.

Same fix needs porting to adcp-client / adcp-client-python / adcp-go.

🤖 Generated with Claude Code

…uote

The PR-feedback MODE directive read "...PR's head branch..." which inside
a single-quoted bash block (the jq filter wrapping) closed the quote at
the apostrophe and made bash interpret the rest as code. Workflow run
24968657038 failed with `syntax error near unexpected token '('` on the
parenthetical that followed.

Rephrase to "the PR head branch — do not open a new PR" so the entire
jq filter stays inside the bash single-quote.

Caught the same morning we shipped the original change. Local repro
confirmed before/after.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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