remove grouping by operation id

Author: Zee2413

src/handlers/StackHandler.ts

@@ -1,4 +1,3 @@
-import { OperationEvent } from '@aws-sdk/client-cloudformation';
 import { ErrorCodes, RequestHandler, ResponseError } from 'vscode-languageserver';
 import { ArtifactExporter } from '../artifactexporter/ArtifactExporter';
 import { TopLevelSection } from '../context/ContextType';
@@ -464,22 +463,8 @@ export function describeEventsHandler(
                 NextToken: params.nextToken,
             });
 
-            const operations = new Map<string, OperationEvent[]>();
-            for (const event of response.OperationEvents ?? []) {
-                const opId = event.OperationId ?? 'unknown';
-                const existing = operations.get(opId);
-                if (existing) {
-                    existing.push(event);
-                } else {
-                    operations.set(opId, [event]);
-                }
-            }
-
             return {
-                operations: [...operations.entries()].map(([operationId, events]) => ({
-                    operationId,
-                    events: events.sort((a, b) => (b.Timestamp?.getTime() ?? 0) - (a.Timestamp?.getTime() ?? 0)),
-                })),
+                events: response.OperationEvents ?? [],
                 nextToken: response.NextToken,
             };
         } catch (error) {

src/stacks/StackRequestType.ts

@@ -127,13 +127,8 @@ export type DescribeEventsParams = {
     nextToken?: string;
 };
 
-export type StackOperationGroup = {
-    operationId: string;
-    events: OperationEvent[];
-};
-
 export type DescribeEventsResult = {
-    operations: StackOperationGroup[];
+    events: OperationEvent[];
     nextToken?: string;
 };
 

tst/unit/handlers/StackHandler.test.ts

@@ -927,7 +927,7 @@ describe('StackActionHandler', () => {
     });
 
     describe('describeEventsHandler', () => {
-        it('should fetch and group events by operation ID', async () => {
+        it('should return flat events from API', async () => {
             mockComponents.cfnService.describeEvents.resolves({
                 OperationEvents: [
                     { EventId: '1', OperationId: 'op1', Timestamp: new Date('2024-01-01') },
@@ -940,9 +940,8 @@ describe('StackActionHandler', () => {
             const handler = describeEventsHandler(mockComponents);
             const result = (await handler({ stackName: 'test-stack' }, CancellationToken.None)) as DescribeEventsResult;
 
-            expect(result.operations).toHaveLength(2);
-            expect(result.operations[0].operationId).toBe('op1');
-            expect(result.operations[0].events).toHaveLength(2);
+            expect(result.events).toHaveLength(3);
+            expect(result.events[0].EventId).toBe('1');
         });
 
         it('should pass all parameters to API', async () => {