Skip to content

fix(skills): avoid treating skill content as a template#2703

Merged
amitksingh1490 merged 4 commits intoantinomyhq:mainfrom
daninfpj:fix/handlebars-skill-parsing
Mar 28, 2026
Merged

fix(skills): avoid treating skill content as a template#2703
amitksingh1490 merged 4 commits intoantinomyhq:mainfrom
daninfpj:fix/handlebars-skill-parsing

Conversation

@daninfpj
Copy link
Copy Markdown
Contributor

@daninfpj daninfpj commented Mar 26, 2026

Fixes #2654

Uses plain string replacement when substituting path placeholders in a skill's command field to avoid treating skill content as a Handlebars template, which would break skills containing {{ }} syntax (e.g. JSX or code examples).

Verification

  • Lint and test pass
  • Ran :ask command with the 12-principles-of-animation skill loaded
  • Created skill with /create-skill and verified path replacement works correctly

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 26, 2026

CLA assistant check
All committers have signed the CLA.

@amitksingh1490 amitksingh1490 self-requested a review March 27, 2026 04:13
@amitksingh1490 amitksingh1490 enabled auto-merge (squash) March 28, 2026 11:28
@amitksingh1490 amitksingh1490 merged commit 394f7bd into antinomyhq:main Mar 28, 2026
9 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.

[Bug]: Failed to load skills — Failed to parse template

3 participants