microsoft / microsoft/hve-core

feat(skills): bind performance-slo-planner SLIs to telemetry-foundations vocabulary

Open
#2,432 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement needs-triage skills
Dominant language
Python
Stars
1.5k
Forks
301
Avg merge
3d 3h
Merged PRs (30d)
92

Description

## Summary

Follow-up from the #2420 review (F1, @WilliamBerryiii). Bind the SLIs in `performance-slo-planner` to the shared `telemetry-foundations` vocabulary instead of free-form metric/trace names.

## Proposed changes

- SLIs cite the `telemetry-foundations` conventions (latency = histogram in seconds; `..` metric names; bounded cardinality).
- Add an instrument/unit/metric-name column to the Observability hooks output so each SLI carries its telemetry contract.
- Add `**/.copilot-tracking/performance-plans/**` to the `telemetry-overlay` applyTo glob to mandate the vocabulary.
- For AI-backed journeys, name model latency / refusal rate / accuracy-drift.

## Context

Deferred from #2420 to keep that PR focused on the mechanical and scoping fixes; this meaningfully expands scope.

## Acceptance criteria

- `SKILL.md` references the telemetry-foundations vocabulary for SLIs, and the observability hooks table carries instrument/unit/metric-name.
- The `telemetry-overlay` glob includes `performance-plans`.
- `npm run validate:skills` and `npm run lint:frontmatter` pass; plugins regenerated.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the performance-slo-planner SKILL.md, the telemetry-overlay applyTo configuration, and the generated plugin files. Compare the existing telemetry-foundations conventions and the observability hooks table, then run npm run validate:skills and npm run lint:frontmatter. Done means the SLI telemetry contract, performance-plans glob, and regenerated plugins satisfy the listed acceptance criteria.

Written by the indexing model from the issue text.

Assessment

Tech stack
markdown
Domain
developer-experience, tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.