Skip to content

K2-287: add monitoring-advisor skill#34

Open
andyliucoder wants to merge 2 commits intomainfrom
aliu/k2-287-monitoring-advisor-skill
Open

K2-287: add monitoring-advisor skill#34
andyliucoder wants to merge 2 commits intomainfrom
aliu/k2-287-monitoring-advisor-skill

Conversation

@andyliucoder
Copy link
Copy Markdown
Collaborator

@andyliucoder andyliucoder commented Apr 10, 2026

Summary

  • Add monitoring-advisor skill that ports the interactive coverage agent flow to a Claude skill using public MCP tools
  • Supports warehouse discovery → use-case analysis → coverage gap detection → monitor creation, with importance-based fallback when no use cases exist
  • Delegates to monitor-creation skill for the actual monitor creation workflow (DRY); keeps only coverage-specific guidance (use-case tag asset_selection, description guidelines)
  • Makes aggregate_time_field optional in metric monitor reference — supports whole-table scans for tables without timestamp columns

Closes K2-287

Port interactive coverage agent flow to a Claude skill using public MCP
tools. Includes warehouse discovery, use-case analysis, coverage gap
detection, and monitor creation with cross-references to monitor-creation
skill for parameter docs. Also makes aggregate_time_field optional in
metric monitor reference.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@linear
Copy link
Copy Markdown

linear bot commented Apr 10, 2026

Address PR feedback: remove duplicated monitor type table, reference
file listing, confirmation/presentation steps. Instead delegate to
the monitor-creation skill for the full creation workflow. Keep only
coverage-specific guidance (use-case tag asset_selection, description
guidelines, audience pre-fetching).

Co-Authored-By: Claude Opus 4.6 <[email protected]>
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.

2 participants