Renames the callback & remove redundant checks

src/node/services/API/workspace-watcher.ts

@@ -27,20 +27,14 @@ export class NodeDevWorkspaceWatcher implements IDevWorkspaceWatcher {
         const path = `/apis/${devWorkspaceApiGroup}/${devworkspaceVersion}/watch/namespaces/${namespace}/devworkspaces`;
 
         return this.customObjectWatch.watch(path, {}, (type: string, devworkspace: IDevWorkspace) => {
-            const status = devworkspace!.status!.phase;
             const workspaceId = devworkspace!.status!.devworkspaceId;
-            if (devworkspace.kind !== 'DevWorkspace' || !workspaceId || !status) {
-                return;
-            }
 
             if (type === 'ADDED') {
                 callbacks.onAdded(devworkspace);
             } else if (type === 'MODIFIED') {
-                callbacks.onStatusChange({ workspaceId, status });
+                callbacks.onModified(devworkspace);
             } else if (type === 'DELETED') {
-                if (status === 'Terminating') {
-                    callbacks.onDeleted(workspaceId);
-                }
+                callbacks.onDeleted(workspaceId);
             } else {
                 callbacks.onError(`Error: Unknown type '${type}'.`);
             }

src/node/services/helpers/index.ts

@@ -34,6 +34,6 @@ export async function findApi(apisApi: k8s.ApisApi, apiName: string, version?: s
     }
 }
 
-export async function delay(ms = 500): Promise<void> {
+export async function delay(ms: number = 500): Promise<void> {
     await new Promise<void>(resolve => setTimeout(resolve, ms));
-};
+}

src/types.ts

@@ -17,7 +17,7 @@ export interface IDevWorkspaceClient {
 }
 
 export type IDevWorkspaceCallbacks = {
-    onStatusChange: (statusUpdate: { status: string; workspaceId: string }) => void;
+    onModified: (workspace: IDevWorkspace) => void;
     onDeleted: (workspaceId: string) => void;
     onAdded: (workspace: IDevWorkspace) => void;
     onError: (error: string) => void;