fix(insights): use Date input, remove isCaseManagement, deduplicate impl

- Change startTime/endTime from number to Date in ElementCountByStatusOptions
  SDK converts to Unix ms internally via .getTime()
- Remove isCaseManagement flag — endpoint is identical for both services
- CaseInstancesService delegates to ProcessInstancesService (no duplication)
- Update oauth-scopes.md with full scope list (Insights.RealTimeData, Insights, PIMS, OR.Folders.Read)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: ninja-shreyash

docs/oauth-scopes.md

@@ -83,7 +83,7 @@ This page lists the specific OAuth scopes required in external app for each SDK
 | `cancel()` | `PIMS` |
 | `pause()` | `PIMS` |
 | `resume()` | `PIMS` |
-| `getElementCountByStatus()` | `Insights.RealTimeData` |
+| `getElementCountByStatus()` | `Insights.RealTimeData` `Insights` `PIMS` `OR.Folders.Read` |
 
 ## Maestro Cases
 
@@ -104,7 +104,7 @@ This page lists the specific OAuth scopes required in external app for each SDK
 | `getExecutionHistory()` | `PIMS` |
 | `getStages()` | `PIMS OR.Execution.Read` |
 | `getActionTasks()` | `OR.Tasks` or `OR.Tasks.Read` |
-| `getElementCountByStatus()` | `Insights.RealTimeData` |
+| `getElementCountByStatus()` | `Insights.RealTimeData` `Insights` `PIMS` `OR.Folders.Read` |
 
 ## Conversational Agent
 

package-lock.json

@@ -288,7 +288,6 @@ export interface CaseInstancesServiceModel {
    *
    * Returns per-element execution counts (success, fail, terminated, paused, in-progress) and
    * duration percentile metrics (min, max, avg, p50, p95, p99) for BPMN elements within a case.
-   * Internally sets `isCaseManagement: true` to filter for case-specific data.
    *
    * @param options - Options containing processKey, packageId, time range, and version
    * @returns Promise resolving to an array of {@link ElementCountByStatus}
@@ -298,8 +297,8 @@ export interface CaseInstancesServiceModel {
    * const elements = await caseInstances.getElementCountByStatus({
    *   processKey: '<processKey>',
    *   packageId: '<packageId>',
-   *   startTime: Date.now() - 7 * 24 * 60 * 60 * 1000,
-   *   endTime: Date.now(),
+   *   startTime: new Date('2026-04-01'),
+   *   endTime: new Date(),
    *   version: '1.0.1'
    * });
    *

src/models/maestro/case-instances.models.ts

@@ -41,10 +41,10 @@ export interface ElementCountByStatusOptions {
   processKey: string;
   /** Package identifier */
   packageId: string;
-  /** Start time as Unix timestamp in milliseconds */
-  startTime: number;
-  /** End time as Unix timestamp in milliseconds */
-  endTime: number;
+  /** Start of the time range to query */
+  startTime: Date;
+  /** End of the time range to query */
+  endTime: Date;
   /** Package version to filter by */
   version: string;
 }

src/models/maestro/insights.types.ts

@@ -271,8 +271,8 @@ export interface ProcessInstancesServiceModel {
    * const elements = await processInstances.getElementCountByStatus({
    *   processKey: '<processKey>',
    *   packageId: '<packageId>',
-   *   startTime: Date.now() - 7 * 24 * 60 * 60 * 1000,
-   *   endTime: Date.now(),
+   *   startTime: new Date('2026-04-01'),
+   *   endTime: new Date(),
    *   version: '1.0.1'
    * });
    *

src/models/maestro/process-instances.models.ts

@@ -16,6 +16,7 @@ import {
   ElementCountByStatus,
   ElementCountByStatusOptions
 } from '../../../models/maestro';
+import { ProcessInstancesService } from '../processes/process-instances';
 import { TaskGetResponse } from '../../../models/action-center';
 import {
   CaseJsonResponse
@@ -45,6 +46,7 @@ import { TaskGetAllOptions } from '../../../models/action-center';
 
 export class CaseInstancesService extends BaseService implements CaseInstancesServiceModel {
   private taskService: TaskService;
+  private processInstancesService: ProcessInstancesService;
 
   /**
    * Creates an instance of the Case Instances service.
@@ -54,6 +56,7 @@ export class CaseInstancesService extends BaseService implements CaseInstancesSe
   constructor(instance: IUiPath) {
     super(instance);
     this.taskService = new TaskService(instance);
+    this.processInstancesService = new ProcessInstancesService(instance);
   }
 
   /**
@@ -174,23 +177,7 @@ export class CaseInstancesService extends BaseService implements CaseInstancesSe
    */
   @track('CaseInstances.GetElementCountByStatus')
   async getElementCountByStatus(options: ElementCountByStatusOptions): Promise<ElementCountByStatus[]> {
-    const requestBody = {
-      commonParams: {
-        processKey: options.processKey,
-        packageId: options.packageId,
-        startTime: options.startTime,
-        endTime: options.endTime,
-        version: options.version,
-        isCaseManagement: true
-      }
-    };
-
-    const response = await this.post<ElementCountByStatus[]>(
-      MAESTRO_ENDPOINTS.INSIGHTS.ELEMENT_COUNT_BY_STATUS,
-      requestBody
-    );
-
-    return response.data;
+    return this.processInstancesService.getElementCountByStatus(options);
   }
 
   /**

src/services/maestro/cases/case-instances.ts

@@ -216,8 +216,8 @@ export class ProcessInstancesService extends BaseService implements ProcessInsta
       commonParams: {
         processKey: options.processKey,
         packageId: options.packageId,
-        startTime: options.startTime,
-        endTime: options.endTime,
+        startTime: options.startTime.getTime(),
+        endTime: options.endTime.getTime(),
         version: options.version
       }
     };

src/services/maestro/processes/process-instances.ts

@@ -276,8 +276,8 @@ describe.each(modes)('Maestro Case Instances - Integration Tests [%s]', (mode) =
       const result = await caseInstances.getElementCountByStatus({
         processKey: instance.processKey,
         packageId: instance.packageId,
-        startTime: Date.now() - 30 * 24 * 60 * 60 * 1000,
-        endTime: Date.now(),
+        startTime: new Date(Date.now() - 30 * 24 * 60 * 60 * 1000),
+        endTime: new Date(),
         version: instance.packageVersion
       });
 

tests/integration/shared/maestro/case-instances.integration.test.ts

@@ -313,8 +313,8 @@ describe.each(modes)('Maestro Process Instances - Integration Tests [%s]', (mode
       const result = await processInstances.getElementCountByStatus({
         processKey: instance.processKey,
         packageId: instance.packageId,
-        startTime: Date.now() - 30 * 24 * 60 * 60 * 1000,
-        endTime: Date.now(),
+        startTime: new Date(Date.now() - 30 * 24 * 60 * 60 * 1000),
+        endTime: new Date(),
         version: instance.packageVersion
       });
 

tests/integration/shared/maestro/process-instances.integration.test.ts

@@ -870,47 +870,23 @@ describe('CaseInstancesService', () => {
     const options = {
       processKey: MAESTRO_TEST_CONSTANTS.CASE_PROCESS_KEY,
       packageId: MAESTRO_TEST_CONSTANTS.CASE_PACKAGE_ID,
-      startTime: 1770748200000,
-      endTime: 1773253799999,
+      startTime: new Date('2026-04-01T00:00:00Z'),
+      endTime: new Date('2026-05-01T00:00:00Z'),
       version: MAESTRO_TEST_CONSTANTS.PACKAGE_VERSION
     };
 
-    it('should return element count by status for case instances with isCaseManagement flag', async () => {
+    it('should delegate to ProcessInstancesService and return results', async () => {
       mockApiClient.post.mockResolvedValue(mockElementCountByStatusResponse);
 
       const result = await service.getElementCountByStatus(options);
 
-      expect(mockApiClient.post).toHaveBeenCalledWith(
-        MAESTRO_ENDPOINTS.INSIGHTS.ELEMENT_COUNT_BY_STATUS,
-        {
-          commonParams: {
-            processKey: MAESTRO_TEST_CONSTANTS.CASE_PROCESS_KEY,
-            packageId: MAESTRO_TEST_CONSTANTS.CASE_PACKAGE_ID,
-            startTime: 1770748200000,
-            endTime: 1773253799999,
-            version: MAESTRO_TEST_CONSTANTS.PACKAGE_VERSION,
-            isCaseManagement: true
-          }
-        },
-        {}
-      );
-
       expect(result).toHaveLength(2);
       expect(result[0].elementId).toBe('Event_start');
       expect(result[0].successCount).toBe(2);
       expect(result[1].failCount).toBe(3);
       expect(result[1].inProgressCount).toBe(1);
     });
 
-    it('should always include isCaseManagement: true in the request body', async () => {
-      mockApiClient.post.mockResolvedValue([]);
-
-      await service.getElementCountByStatus(options);
-
-      const callArgs = mockApiClient.post.mock.calls[0];
-      expect(callArgs[1].commonParams.isCaseManagement).toBe(true);
-    });
-
     it('should handle API errors', async () => {
       const error = new Error(TEST_CONSTANTS.ERROR_MESSAGE);
       mockApiClient.post.mockRejectedValue(error);