Skip to content

fix: remove stale template variable from slides SKILL.md#281

Open
xiaolai wants to merge 1 commit intonextlevelbuilder:mainfrom
xiaolai:fix/nlpm-stale-template-variable
Open

fix: remove stale template variable from slides SKILL.md#281
xiaolai wants to merge 1 commit intonextlevelbuilder:mainfrom
xiaolai:fix/nlpm-stale-template-variable

Conversation

@xiaolai
Copy link
Copy Markdown

@xiaolai xiaolai commented Apr 26, 2026

Automated audit: This PR was generated by NLPM, a natural language programming linter, running via claude-code-action. Please evaluate the diff on its merits.

Bug

Line 14 of .claude/skills/slides/SKILL.md contains the raw template placeholder <args>$ARGUMENTS</args> that was never substituted during generation. This renders as literal XML-like text in the skill body. When Claude Code loads the skill, this tag is included as-is in the context, which can confuse the LLM into treating the text as a special tag or trying to substitute $ARGUMENTS itself.

Fix

Removed the unsubstituted <args>$ARGUMENTS</args> line. Arguments are already available to the LLM via the invocation context — no explicit placeholder is needed in the skill body.

The <args>$ARGUMENTS</args> placeholder at line 14 was never substituted
during template generation and rendered as literal XML-like text in the
skill body, potentially confusing LLM consumers.

Co-Authored-By: Claude Code <[email protected]>
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