refactor: migrate Davis CoPilot integration to official SDK #153
Conversation
📦 Package.json Changes DetectedThis PR modifies Checklist for Reviewers
Package.json Diffdiff --git a/package.json b/package.json
index 3f2b9e1..dfb80bb 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-copilot": "^1.0.0",
"@dynatrace-sdk/client-platform-management-service": "^1.6.3",
"@dynatrace-sdk/client-query": "^1.18.1",
"@dynatrace-sdk/shared-errors": "^1.0.0",
This comment was automatically generated by the PR Package.json Comment workflow. |
AI Alert This PR has been fully reviewed with the power of GitHub Copilot |
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the Davis CoPilot integration to use the official @dynatrace-sdk/client-davis-copilot package instead of manual API calls. The main objective is to improve reliability and maintainability by replacing custom HTTP requests with standardized SDK methods.
- Migrated from manual HTTP client calls to the official Dynatrace SDK for Davis CoPilot
- Removed extensive manual type definitions in favor of SDK imports
- Added response handling logic for streaming and non-streaming conversation responses
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/capabilities/davis-copilot.ts | Major refactor replacing manual API calls with official SDK client, removing custom type definitions, and adding response handling for conversation API |
| package.json | Added dependency for @dynatrace-sdk/client-davis-copilot package |
| CHANGELOG.md | Added entry documenting the migration to official SDK |
|
@raphael-wigoutschnigg-dt @MrManny please re-review. I've looked into the API and SDK, and removed handling for streaming responses in the chat endpoint. |
Will do! Sorry, was a bit tied up with meetings today, so I tried to delegate this to Copilot for maximum irony. Will probably get to it tomorrow before lunch. |
- Replace manual API calls with @dynatrace-sdk/client-davis-copilot - Simplify imports by removing unused type exports - Keep only Dql2NlResponse export for external compatibility - Improve reliability and maintainability of Davis CoPilot functions - Update function implementations to use PublicClient from official SDK
bf9ceee to
9b45112
Compare
AI Alert This PR has been fully created with the power of GitHub Copilot