clean: clarify notes for future

Author: AlexandrSuhinin

src/CodexAppServerClient.ts

@@ -77,6 +77,7 @@ export class CodexAppServerClient {
         return await this.sendRequest({ method: "account/read", params: params });
     }
 
+    //TODO create type-safe helper
     async awaitTurnCompleted(): Promise<TurnCompletedNotification> {
         return await new Promise((resolve) => {
             this.connection.onNotification("turn/completed", (event: TurnCompletedNotification) => {
@@ -85,6 +86,7 @@ export class CodexAppServerClient {
         });
     }
 
+    //TODO support removal (leads to duplicated processing of follow-ups)
     onServerNotification(callback: (event: ServerNotification) => void){
         this.notificationHandlers.push(callback);
     }

src/CodexEventHandler.ts

@@ -34,6 +34,13 @@ export class CodexEventHandler {
     }
 
     private async createUpdateEvent(notification: ServerNotification): Promise<UpdateSessionEvent | null> {
+        /*
+        TODO split UpdateSessionEvent to improve completion
+        createUpdateEvent({
+            sessionUpdate: "" , <- completion of UpdateSessionEvent["sessionUpdate"]
+            params: {}, <- quickfix to generate required fields (rest of)
+        });
+         */
         switch (notification.method) {
             case "item/agentMessage/delta":
                 return await this.createTextEvent(notification.params);

src/__tests__/CodexACPAgent/CodexAcpClient.test.ts

@@ -58,7 +58,7 @@ describe('ACP server test', () => {
     });
 
     function loadNotifications(){
-        //TODO load from file
+        //TODO collect logs form dev run and then load them from file to speedup
         const serverNotifications: ServerNotification[] = [
             { method: "item/agentMessage/delta", params: { threadId: "string", turnId: "string", itemId: "string", delta: "He", }},
             { method: "item/agentMessage/delta", params: { threadId: "string", turnId: "string", itemId: "string", delta: "ll", }},