patch: Remove get_ownership tool as the oauth scopes are gone

Author: christian-kreuzberger-dtx

CHANGELOG.md

@@ -6,6 +6,11 @@
 
 - Added `status` parameter to `list_problems` tool to filter problems by status (ACTIVE, CLOSED, or ALL).
 - Added `timeframe` parameter to `list_problems` tool to support flexible time ranges (e.g., "12h", "24h", "7d", "30d"). Default: "24h".
+- Removed `get_ownership` tool as it no longer works with OAuth Clients.
+
+### Scopes
+
+- Removed `settings:objects:read` and `environment-api:entities:read` scopes, as they are no longer required
 
 ## 0.12.0
 

README.md

@@ -330,10 +330,10 @@ Depending on the features you are using, the following scopes are needed:
 **Available for both Platform Tokens and OAuth Clients:**
 
 - `app-engine:apps:run` - needed for almost all tools
-- `environment-api:entities:read` - for retrieving ownership details from monitored entities (_currently not available for Platform Tokens_)
 - `automation:workflows:read` - read Workflows
 - `automation:workflows:write` - create and update Workflows
 - `automation:workflows:run` - run Workflows
+- `app-settings:objects:read` - read app-settings - needed for `send_slack_message` tool to read connection details from App-Settings
 - `storage:buckets:read` - needed for `execute_dql` tool to read all system data stored on Grail
 - `storage:logs:read` - needed for `execute_dql` tool to read logs for reliability guardian validations
 - `storage:metrics:read` - needed for `execute_dql` tool to read metrics for reliability guardian validations
@@ -350,14 +350,11 @@ Depending on the features you are using, the following scopes are needed:
 - `davis-copilot:nl2dql:execute` - execute Davis Copilot Natural Language (NL) to DQL skill
 - `davis-copilot:dql2nl:execute` - execute DQL to Natural Language (NL) skill
 - `email:emails:send` - needed for `send_email` tool to send emails
-- `settings:objects:read` - needed for reading ownership information and Guardians (SRG) from settings
 
 **Notes**:
 
-- Please ensure that `settings:objects:read` is used, and _not_ the similarly named scope `app-settings:objects:read`.
 - Versions before 0.12.0 required the scope `app-engine:functions:run`, which is no longer required.
-
-**Important**: Some features requiring `environment-api:entities:read` will not work with Platform Tokens.
+- Versions before 0.13.0 required the scopes `settings:objects:read` and `environment-api:entities:read`, which are no longer required.
 
 ## ✨ Example prompts ✨
 

src/capabilities/get-ownership-information.ts

@@ -24,7 +24,6 @@ import { getPackageJsonVersion } from './utils/version';
 import { createDtHttpClient } from './authentication/dynatrace-clients';
 import { listVulnerabilities } from './capabilities/list-vulnerabilities';
 import { listProblems } from './capabilities/list-problems';
-import { getOwnershipInformation } from './capabilities/get-ownership-information';
 import { getEventsForCluster } from './capabilities/get-events-for-cluster';
 import { createWorkflowForProblemNotification } from './capabilities/create-workflow-for-problem-notification';
 import { updateWorkflow } from './capabilities/update-workflow';
@@ -90,8 +89,6 @@ const allRequiredScopes = scopesBase.concat([
 
   // Settings and configuration scopes
   'app-settings:objects:read', // Read app settings objects
-  'settings:objects:read', // Read settings objects
-  'environment-api:entities:read', // Read entities via environment API
 
   // Davis CoPilot scopes
   'davis-copilot:nl2dql:execute', // Convert natural language to DQL
@@ -1031,28 +1028,6 @@ const main = async () => {
     },
   );
 
-  tool(
-    'get_ownership',
-    'Get detailed Ownership information for one or multiple entities on Dynatrace',
-    {
-      entityIds: z.string().optional().describe('Comma separated list of entityIds'),
-    },
-    {
-      readOnlyHint: true,
-    },
-    async ({ entityIds }) => {
-      const dtClient = await createAuthenticatedHttpClient(
-        scopesBase.concat('environment-api:entities:read', 'settings:objects:read'),
-      );
-      console.error(`Fetching ownership for ${entityIds}`);
-      const ownershipInformation = await getOwnershipInformation(dtClient, entityIds);
-      console.error(`Done!`);
-      let resp = 'Ownership information:\n';
-      resp += JSON.stringify(ownershipInformation);
-      return resp;
-    },
-  );
-
   tool(
     'reset_grail_budget',
     'Reset the Grail query budget after it was exhausted, allowing new queries to be executed. This clears all tracked bytes scanned in the current session.',