Skip to content

Comments

fix(cli): Skip existing skills instead of failing when adding multiple#35

Merged
gricha merged 1 commit intomainfrom
fix/skip-existing-skills-on-add
Feb 23, 2026
Merged

fix(cli): Skip existing skills instead of failing when adding multiple#35
gricha merged 1 commit intomainfrom
fix/skip-existing-skills-on-add

Conversation

@gricha
Copy link
Member

@gricha gricha commented Feb 23, 2026

When running dotagents add <source> skill-a skill-b and one of the skills
already exists in agents.toml, the command would abort entirely without
adding any of the remaining skills.

Now it skips existing skills with a yellow warning and continues adding the
rest. Only errors if all specified skills already exist. This matches the
behavior already used in the interactive multi-select path.

Agent transcript: https://claudescope.sentry.dev/share/D429kDAB1WPC1fgfnVDPBwB2FWRgbuSnC3BpW2DVxyo

When adding multiple skills via `dotagents add`, if some already exist
in agents.toml, skip them with a warning instead of aborting the entire
command. Only errors if all specified skills already exist.

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

Agent transcript: https://claudescope.sentry.dev/share/vTCSYkbTh8oQFf1snloZsgu1DVzcSH8X8vI9JNNpnHI
@vercel
Copy link

vercel bot commented Feb 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dotagents Ready Ready Preview, Comment Feb 23, 2026 11:39pm

Request Review

@gricha gricha marked this pull request as ready for review February 23, 2026 23:40
@gricha gricha merged commit 008602f into main Feb 23, 2026
15 checks passed
@gricha gricha deleted the fix/skip-existing-skills-on-add branch February 23, 2026 23:40
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