Skip to content

fix(conductor-new-track): unbreak install command in Execute Installation step - #174

Open
xiaolai wants to merge 1 commit into
gemini-cli-extensions:mainfrom
xiaolai:fix/nlpm-new-track-curl-code-block
Open

fix(conductor-new-track): unbreak install command in Execute Installation step#174
xiaolai wants to merge 1 commit into
gemini-cli-extensions:mainfrom
xiaolai:fix/nlpm-new-track-curl-code-block

Conversation

@xiaolai

@xiaolai xiaolai commented Jul 20, 2026

Copy link
Copy Markdown

Automated: drive-by fix from NLPM, an NL artifact linter. Reviewed and reproduced before submission.

Bug: skills/conductor-new-track/SKILL.md line 126 collapses the "exact curl command sequence" into a single unfenced inline-code span ( bash mkdir -p ... curl -sSL ... ) instead of a fenced code block with each command on its own line.

Evidence: Running the literal span as shell fails: bash: line 1: skill_name: No such file or directory (the unescaped <...> placeholders are parsed as redirections since there's no fencing/newline separation). The correctly-formatted version of this identical instruction already exists at skills/conductor-setup/SKILL.md:182-187.

Fix: Reformats the instruction into a fenced ```bash block with mkdir and curl on separate lines, matching conductor-setup/SKILL.md's existing style.

…tion step

The curl install instructions were collapsed into a single unfenced
inline-code span, making the "exact command sequence" the agent is
told to run unusable. conductor-setup/SKILL.md already has the
correctly fenced version of the same instruction; this mirrors it.

Co-Authored-By: Claude Code <noreply@anthropic.com>
@google-cla

google-cla Bot commented Jul 20, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@mahimashanware

Copy link
Copy Markdown
Contributor

Hi, we'd like to submit this PR but it looks like the CLA check is blocked by Claude not having signed it. Can you please remove Claude from your commit?

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.

3 participants