Commit Graph

3 Commits

Author SHA1 Message Date
Niv Hertz ec41a51600 AGE-1086 | Create base coding agents skill for cli (#145) 2026-06-18 11:12:23 +00:00
Liran Hason f35a583c30 feat: add cx dashboards replace command (#90) 2026-05-18 09:24:31 +03:00
Niv Hertz bcb3c35644 feat(skills): consolidate telemetry skills + shared references system (#59)
## Summary

- **Collapse 5 single-pillar skills** (`cx-query-logs`,
`cx-query-spans`, `cx-rum`, `cx-dataprime`, `cx-metrics-query`) into a
single `cx-telemetry-querying` gateway skill that covers all telemetry
pillars and loads pillar-specific reference files on demand
- **Introduce `skills/shared/`** as the single source of truth for
language guides and telemetry-pillar how-tos (DataPrime syntax, PromQL
guidelines, logs/spans/metrics/RUM querying)
- **Add `scripts/sync-shared-references.sh`** to distribute shared files
into each consuming skill's `references/` directory as regular file
copies (not symlinks, so `npx skills add coralogix/cx-cli` works
correctly on clients)
- **Update `cx-alerts` and `cx-create-dashboard`** to reference shared
files via local `references/` copies instead of cross-skill name
references
- **Extend `scripts/verify-skills.sh`** with a sync check (check #6)
that fails if any `references/` copy is out of sync with
`skills/shared/`
- **Add `.github/workflows/skills.yml`** CI workflow to validate skill
correctness and reference sync on every PR touching `skills/`
- **Update all developer docs** (`docs/adding-a-skill.md`,
`.claude/skills/add-skill/SKILL.md`, `skills/README.md`, `CLAUDE.md`) to
document the new 8-skill structure and shared references architecture

## Why

Skills were previously cross-referencing each other by skill name (e.g.,
"load `cx-dataprime` skill for syntax help"), which is fragile — if a
skill is renamed or removed, the reference silently breaks. The new
approach uses file-based references: each skill ships its own
`references/` directory with the content it needs, copied from the
canonical `skills/shared/` source by a sync script. CI validates the
copies stay in sync.

## Test plan

- [ ] `bash scripts/verify-skills.sh` — all 8 skills pass, 0 errors
- [ ] `bash scripts/sync-shared-references.sh` — exits 0, no errors
- [ ] No `SKILL.md` references any of the 5 deleted skill names
- [ ] `cx-telemetry-querying` triggers on all former skill trigger
phrases (logs, spans, metrics, RUM, DataPrime)
- [ ] CI workflow (`.github/workflows/skills.yml`) runs and passes on
this PR

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-05-05 09:16:45 +00:00