feat: replace glean-cli-shared with root glean-cli discovery skill#80
Open
steve-calvert-glean wants to merge 2 commits intomainfrom
Open
feat: replace glean-cli-shared with root glean-cli discovery skill#80steve-calvert-glean wants to merge 2 commits intomainfrom
steve-calvert-glean wants to merge 2 commits intomainfrom
Conversation
Replace the separate glean-cli-shared skill with a unified glean-cli
root skill that combines agent discovery with auth, global flags, and
shared reference content. This solves the chicken-and-egg problem where
agents didn't know the Glean CLI existed unless explicitly told.
The root skill's description is tuned for broad trigger matching
("internal docs", "company information", "people", "policies") so
agents activate it on enterprise knowledge queries without the user
needing to mention the CLI.
Changes:
- Replace sharedTmpl/writeSharedSkill with rootTmpl/writeRootSkill
- Root skill outputs to glean-cli/ (not glean-cli-shared/)
- Per-command skills now reference ../glean-cli/SKILL.md
- Delete glean-cli-shared directory
- Update README skill table
- Anchor .gitignore binary patterns with / to avoid matching skills/
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Derive skillPrefix from rootSkillName and pass both through template data so no hardcoded "glean-cli" strings remain outside the constant definitions. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
glean-cli-sharedskill with a unifiedglean-cliroot skill that combines agent discovery with auth, global flags, and shared reference content../glean-cli/SKILL.mdinstead of../glean-cli-shared/SKILL.md.gitignorebinary patterns with/to avoid matching the newskills/glean-cli/directoryTest plan
go build ./...passesgo run . generate-skills --output-dir skills/producesglean-cli/SKILL.md(notglean-cli-shared/)../glean-cli/SKILL.mdglean-cli-sharedorglean-shared🤖 Generated with Claude Code