Skip to content

Conversation

@christian-kreuzberger-dtx
Copy link
Collaborator

@christian-kreuzberger-dtx christian-kreuzberger-dtx commented Oct 29, 2025

Added Davis Analyzer tools. To try it out, I did the following prompts:

Can you fetch memory consumption metric from Dynatrace, split by service?

followed by

can you take a look at memory consumption of ? Tell me how it looked like in the past 14d

and followed by

Can you predict how this is going to look for the next 14d?

which lead to list_davis_analyzers and execute_davis_analyzer being executed :)

image

Alternatively, all in one prompt:

Please take a look at memory consumption of the kafka service for the past 7 days, 14 days, and 1 month.
Furthermore, predict memory consumption for the next 7 days.

image

@github-actions
Copy link

github-actions bot commented Oct 29, 2025

📦 Package.json Changes Detected

This PR modifies package.json. Please review the following changes carefully, and tick the following checklist boxes:

Checklist for Reviewers

  • Verify that new and/or updated dependencies are necessary and from trusted sources
  • Review any script changes for (security) implications
  • Verify whether a new version should be released after merging the PR

Package.json Diff

diff --git a/package.json b/package.json
index 7751ddf..c2e24c7 100644
--- a/package.json
+++ b/package.json
@@ -48,6 +48,7 @@
   "license": "MIT",
   "dependencies": {
     "@dynatrace-sdk/client-automation": "^5.3.0",
+    "@dynatrace-sdk/client-davis-analyzers": "^1.9.8",
     "@dynatrace-sdk/client-davis-copilot": "^1.0.0",
     "@dynatrace-sdk/client-platform-management-service": "^1.6.3",
     "@dynatrace-sdk/client-query": "^1.18.1",

This comment was automatically generated by the PR Package.json Comment workflow.

@MrManny MrManny requested a review from Copilot November 4, 2025 13:25
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for Davis Analyzers to the Dynatrace MCP Server, enabling users to list and execute various types of analyzers (forecast, anomaly detection, correlation) programmatically.

Key changes:

  • Introduces two new tools: list_davis_analyzers and execute_davis_analyzer
  • Adds required OAuth scopes for Davis Analyzer operations
  • Implements a polling mechanism for long-running analyzer executions

Reviewed Changes

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

Show a summary per file
File Description
src/index.ts Adds import for Davis Analyzer capabilities, registers two new tools, and includes required OAuth scopes
src/capabilities/davis-analyzers.ts Implements core Davis Analyzer functionality including listing and execution with polling support
package.json Adds dependency on @dynatrace-sdk/client-davis-analyzers version ^1.9.7
package-lock.json Updates lock file with resolved version 1.9.8 of the Davis Analyzers client
README.md Documents the two new OAuth scopes required for Davis Analyzer functionality
CHANGELOG.md Documents the new tools and scopes in the Unreleased Changes section

@christian-kreuzberger-dtx christian-kreuzberger-dtx force-pushed the feature/davis-analyzers branch 2 times, most recently from 408f238 to 12412fe Compare November 6, 2025 15:48
@christian-kreuzberger-dtx christian-kreuzberger-dtx force-pushed the feature/davis-analyzers branch 5 times, most recently from d9e6fbc to 2a40111 Compare November 21, 2025 09:55
@christian-kreuzberger-dtx christian-kreuzberger-dtx marked this pull request as ready for review November 21, 2025 10:01
Copy link
Collaborator

@MrManny MrManny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. My only concern is overdocumenting. I flagged one instance, but I see a few others as well, e.g.:

/**
 * Execute a Davis Analyzer
 */
export async function executeDavisAnalyzer(

Feels a bit redundant, since the function name already says exactly what it does and the comment adds nothing on top of it. What are your thoughts?

chore: added integration-tests for davis analyzers

chore: Improve implementation
@christian-kreuzberger-dtx christian-kreuzberger-dtx merged commit be78de2 into main Nov 24, 2025
8 checks passed
@christian-kreuzberger-dtx christian-kreuzberger-dtx deleted the feature/davis-analyzers branch November 24, 2025 08:31
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.

3 participants