Skip to content

fix: make root skills/ a real directory so Gemini CLI picks up skills#41

Open
gkarthi-signoz wants to merge 1 commit into
SigNoz:mainfrom
gkarthi-signoz:gemini-skills
Open

fix: make root skills/ a real directory so Gemini CLI picks up skills#41
gkarthi-signoz wants to merge 1 commit into
SigNoz:mainfrom
gkarthi-signoz:gemini-skills

Conversation

@gkarthi-signoz

@gkarthi-signoz gkarthi-signoz commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

The Gemini extension installed fine, but none of the skills were discovered. The root cause: skills/ was a symlink to plugins/signoz/skills. Gemini's installer clones the repo to a temp directory and copies it into ~/.gemini/extensions/, rewriting symlinks to absolute paths pointing at that temp clone — which is then deleted. The installed extension ended up with a dangling skills symlink, so skill discovery found nothing.

Changes

Replaced the root skills/ symlink with a real copy of plugins/signoz/skills/ (12 skills). plugins/signoz/skills/ remains the source of truth.
Extended the auto CalVer bump workflow to rsync plugins/signoz/skills/ → skills/ whenever the signoz plugin changes, so the two copies can't drift. The sync lands in the same auto-bump commit.
Documented the convention in CONTRIBUTING.md: never edit root skills/ directly.

Testing

gemini extensions validate . passes.
Installed from local path: ~/.gemini/extensions/signoz/skills/ now contains real skill directories (previously a dangling symlink), and /skills list shows all 12 skills.

@gkarthi-signoz gkarthi-signoz changed the title fix: replace skills symlink with real copy for Gemini make root skills/ a real directory so Gemini CLI picks up skills Jun 10, 2026
@gkarthi-signoz gkarthi-signoz changed the title make root skills/ a real directory so Gemini CLI picks up skills fix: make root skills/ a real directory so Gemini CLI picks up skills Jun 10, 2026
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