Skip to content

fix: resolve duplicate skill name brand-guidelines (B7, B8)#533

Open
xiaolai wants to merge 1 commit intodavila7:mainfrom
xiaolai:fix/nlpm-brand-guidelines-duplicate-name
Open

fix: resolve duplicate skill name brand-guidelines (B7, B8)#533
xiaolai wants to merge 1 commit intodavila7:mainfrom
xiaolai:fix/nlpm-brand-guidelines-duplicate-name

Conversation

@xiaolai
Copy link
Copy Markdown

@xiaolai xiaolai commented Apr 22, 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

Two skills in cli-tool/components/skills/business-marketing/ both declare name: brand-guidelines in their YAML frontmatter:

  • brand-guidelines-anthropic/SKILL.md
  • brand-guidelines-community/SKILL.md

In Claude Code's skill registry, the name field is the install key. When a user installs both skills, the second one silently overwrites the first, meaning one skill is permanently inaccessible regardless of which path it was installed from.

Fix

Rename each skill's name field to match its directory name:

File Before After
brand-guidelines-anthropic/SKILL.md name: brand-guidelines name: brand-guidelines-anthropic
brand-guidelines-community/SKILL.md name: brand-guidelines name: brand-guidelines-community

The skill in enterprise-communication/brand-guidelines/SKILL.md already has a unique directory and is not affected.

Why it matters

Users who install both skills from this repo get only one of them — with no error or warning. Renaming them to unique keys ensures both are independently installable and accessible by name.

The fix is minimal (two one-line frontmatter changes) and does not touch any skill content.


Summary by cubic

Fixes a silent install conflict by giving two brand-guidelines skills unique name keys so both can be installed and used independently.

  • Area: components (cli-tool/components/)
  • Change: Renamed frontmatter name to brand-guidelines-anthropic and brand-guidelines-community in their SKILL.md files
  • Why: Prevents key collision where one skill overwrote the other during install
  • No new components; no docs/components.json regeneration needed
  • No new environment variables or secrets

Written for commit 3fcf87f. Summary will update on new commits.

brand-guidelines-anthropic and brand-guidelines-community both declared
`name: brand-guidelines`, causing a silent install conflict where the
second skill overwrites the first in Claude Code's skill registry.

Renamed to match their directory names: brand-guidelines-anthropic and
brand-guidelines-community respectively.

Co-Authored-By: Claude Code <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 22, 2026

@claude is attempting to deploy a commit to the Daniel Avila's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the review-pending Component PR awaiting maintainer review label Apr 22, 2026
@github-actions
Copy link
Copy Markdown
Contributor

👋 Thanks for contributing, @xiaolai!

This PR touches cli-tool/components/** and has been marked review-pending.

What happens next

  1. 🤖 Automated security audit runs and posts results on this PR.
  2. 👀 Maintainer review — a human reviewer validates the component with the component-reviewer agent (format, naming, security, clarity).
  3. Merge — once approved, your PR is merged to main.
  4. 📦 Catalog regeneration — the component catalog is rebuilt automatically.
  5. 🚀 Live on aitmpl.com — your component appears on the website after deploy.

While you wait

  • Check the Security Audit comment below for any issues to fix.
  • Make sure your component follows the contribution guide.

This is an automated message. No action is required from you right now — a maintainer will review soon.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-pending Component PR awaiting maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants