Introduce Skills Framework Scaffold

[hharshhsaini]

This PR introduces the foundational Skills Framework scaffold for the Jaeger AI agent. It defines the Skill data model, a YAML-based loader that dynamically discovers skills from a configurable directory without recompilation, and an in-memory registry. No AI/LLM dependencies are introduced in this PR. Follows patterns established by jaegerquery extension.

Which problem is this PR solving?

• Part of [Feature]: GenAI integration with Jaeger #7827

Description of the changes

• Introduces Skill and SkillTool structs.
• Implements a dynamic YAML loader with validation and zap logging for discovering skills.
• Implements an in-memory skills Registry.
• Adds comprehensive unit testing (loader_test.go and registry_test.go) covering valid and fallback scenarios.
• Adds an example testing skill at testdata/skills/critical_path_analyzer.yaml.

How was this change tested?

• Unit tests run using go test -v ./... inside cmd/jaeger/internal/extension/jaegermcp/internal/skills (100% passing tests).
• All lints validated locally via make lint.
• Verified no goroutine leaks with Jaeger's testutils.VerifyGoLeaks.

Checklist

• I have read https://github.com/jaegertracing/jaeger/blob/main/CONTRIBUTING_GUIDELINES.md
• I have signed all commits
• I have added unit tests for the new functionality
• I have run lint and test steps successfully: make lint test

AI Usage in this PR (choose one)

See AI Usage Policy.

• None: No AI tools were used in creating this PR
• Light: AI provided minor assistance (formatting, simple suggestions)
• Moderate: AI helped with code generation or debugging specific parts
• Heavy: AI generated most or all of the code changes

[Copilot]

Pull request overview

Introduces a new internal skills scaffold for the Jaeger MCP extension, including a YAML-based skill definition model, directory loader, and in-memory registry intended to support future agent/skills functionality without LLM dependencies.

Changes:

• Added Skill / SkillTool data model for YAML-backed skill definitions.
• Added LoadSkills YAML loader to discover and validate skills from a directory with zap logging.
• Added an in-memory Registry with unit tests and sample skill YAML in testdata.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
go.mod	Promotes gopkg.in/yaml.v3 to a direct dependency for the new YAML loader.
cmd/jaeger/internal/extension/jaegermcp/internal/skills/skills.go	Defines Skill and SkillTool structs with YAML tags.
cmd/jaeger/internal/extension/jaegermcp/internal/skills/loader.go	Implements directory discovery + YAML parsing + minimal validation for skills.
cmd/jaeger/internal/extension/jaegermcp/internal/skills/registry.go	Adds a mutex-protected in-memory registry for loaded skills.
cmd/jaeger/internal/extension/jaegermcp/internal/skills/loader_test.go	Adds unit tests for basic loader scenarios.
cmd/jaeger/internal/extension/jaegermcp/internal/skills/registry_test.go	Adds unit tests for registry operations.
cmd/jaeger/internal/extension/jaegermcp/internal/skills/package_test.go	Adds leak verification TestMain for the package.
cmd/jaeger/internal/extension/jaegermcp/internal/skills/testdata/skills/critical_path_analyzer.yaml	Adds an example skill YAML for testing and reference.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[yurishkuro]

not accepting PRs without overall design agreed upfront

[hharshhsaini]

not accepting PRs without overall design agreed upfront

@yurishkuro yes sir , I am working on the complete Design of the project , will Raise the PR for it ASAP.