Add comprehensive observability analysis rules and DQL patterns

[dt-benedict]

This PR adds comprehensive observability analysis capabilities to the Dynatrace MCP agent rules:

Key Additions

• observabilityProblems.md: Complete problem investigation workflow with 6-phase systematic approach
• Enhanced DynatraceQueryLanguage.md: Critical DQL syntax corrections and best practices
• Updated README.md: Structured analysis modes and workflow documentation

Features

• Real-world investigation examples with precise root cause analysis
• Problem → Span → Log correlation patterns
• Remediation-focused DQL patterns for security and compliance monitoring
• Complete semantic field discovery for entity relationships
• Business logic error detection patterns

Impact

• Enables comprehensive problem resolution with precise technical details
• Provides systematic investigation framework for development teams
• Improves DQL query accuracy with verified syntax patterns"

[MrManny]

Going to ask Co-Pilot for a review; haven't done that yet. Let's see what happens!

[Copilot]

Pull Request Overview

This PR adds comprehensive observability analysis capabilities to the Dynatrace MCP agent rules, transforming it from a security-focused toolkit into a complete observability platform integration. The additions enable systematic problem investigation, distributed tracing analysis, and log correlation with precise root cause identification capabilities.

Key Changes

• Complete problem investigation framework with 6-phase systematic workflow for incident analysis
• Enhanced DQL patterns with critical syntax corrections and semantic field discovery capabilities
• Distributed tracing analysis with precise exception tracking and performance correlation

Reviewed Changes

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

Show a summary per file

File	Description
observabilityProblems.md	Complete problem investigation framework with real-world examples, semantic field discovery, and systematic 6-phase analysis workflow
README.md	Updated architecture overview with 9 rule files, workflow connectivity map, and comprehensive analysis modes documentation
DynatraceSpanAnalysis.md	New distributed tracing analysis guide with exception details, error correlation, and span-log-problem integration patterns
DynatraceSecurityCompliance.md	Enhanced with semantic compliance fields, evidence-based investigation patterns, and runtime protection analysis
DynatraceQueryLanguage.md	Critical syntax corrections, semantic field discovery patterns, and entity relationship mapping capabilities
DynatraceMcpIntegration.md	Updated problem analysis workflow integration and added comprehensive SRE/GitOps automation patterns
DynatraceLogAnalysis.md	New comprehensive log analysis guide with business logic error detection, deployment correlation, and trace correlation patterns

[dt-benedict]

I fixed the identified issues 👍 Surprisingly good feedback!

[MrManny]

Co-Pilot found some good issues.

[MrManny]

I think there's still the four backticks in observabilityProblems.md@L332. I'll go ahead and apply the suggestion from Co-Pilot.

[dynatrace-cla-bot]

All committers have signed the CLA.

[MrManny]

I think there's still the four backticks in observabilityProblems.md@L332. I'll go ahead and apply the suggestion from Co-Pilot.

Or, alternatively, let me break the build 🫠

[dt-benedict]

Weird, I had deleted the 4 backticks from my file. Well, now it's failing for prettier again. I'll have to have a look later.

[MrManny]

Weird, I had deleted the 4 backticks from my file. Well, now it's failing for prettier again.

Mea culpa. I'll quickly run prettier and push the change. Or, alternatively, drop the commit with the applied suggestion

[MrManny]

Weird, I had deleted the 4 backticks from my file. Well, now it's failing for prettier again. I'll have to have a look later.

I am so sorry. GitHub won't let me do things to this PR. 🙈
Can you drop the Co-Pilot suggestion and force-push with lease please?

git reset --hard HEAD~1
git push --force-with-lease

[MrManny]

Hmm. Is it just me or is the DS_Store still part of this PR?

[MrManny]

Thank you :)
LGTM! Any other comments from your side, @christian-kreuzberger-dtx?

[christian-kreuzberger-dtx]

LGTM

[christian-kreuzberger-dtx]

Inclusive Language check is failing because it detects Mastercard as something that's not inclusive enough... Because of Master. 🤦
I'll try to get it unblocked.