Skip to content

feat: replace glean-cli-shared with root glean-cli discovery skill#80

Merged
steve-calvert-glean merged 2 commits intomainfrom
scalvert/root-discovery-skill-v2
Apr 7, 2026
Merged

feat: replace glean-cli-shared with root glean-cli discovery skill#80
steve-calvert-glean merged 2 commits intomainfrom
scalvert/root-discovery-skill-v2

Conversation

@steve-calvert-glean
Copy link
Copy Markdown
Collaborator

Summary

  • Replaces 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
  • 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")
  • Per-command skills now reference ../glean-cli/SKILL.md instead of ../glean-cli-shared/SKILL.md
  • Anchors .gitignore binary patterns with / to avoid matching the new skills/glean-cli/ directory

Test plan

  • go build ./... passes
  • go run . generate-skills --output-dir skills/ produces glean-cli/SKILL.md (not glean-cli-shared/)
  • All per-command skills reference ../glean-cli/SKILL.md
  • No remaining references to glean-cli-shared or glean-shared
  • Root skill description triggers on enterprise knowledge queries

🤖 Generated with Claude Code

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) <noreply@anthropic.com>
@steve-calvert-glean steve-calvert-glean added the enhancement New feature or request label Apr 6, 2026
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) <noreply@anthropic.com>
@steve-calvert-glean steve-calvert-glean merged commit 3f137e4 into main Apr 7, 2026
7 checks passed
@steve-calvert-glean steve-calvert-glean deleted the scalvert/root-discovery-skill-v2 branch April 7, 2026 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants