@axhub/genie 0.1.5 → 0.1.7

package/server/openai-codex.js

@@ -140,7 +140,7 @@ function transformCodexEvent(event) {
     case 'thread.started':
       return {
         type: 'thread_started',
-        threadId: event.id
+        threadId: event.thread_id || event.id
       };
 
     case 'error':
@@ -157,6 +157,42 @@ function transformCodexEvent(event) {
   }
 }
 
+/**
+ * Extract thread id from thread.started events (SDK uses thread_id)
+ * @param {object} event
+ * @returns {string|null}
+ */
+function getThreadIdFromEvent(event) {
+  const threadId = event?.thread_id || event?.id;
+  return typeof threadId === 'string' && threadId.trim() ? threadId.trim() : null;
+}
+
+/**
+ * Extract text-bearing item info from Codex item events.
+ * Only agent_message and reasoning currently stream textual deltas.
+ * @param {object} item
+ * @returns {{itemId: string, itemType: string, text: string, isReasoning: boolean}|null}
+ */
+function getTextItemInfo(item) {
+  if (!item || (item.type !== 'agent_message' && item.type !== 'reasoning')) {
+    return null;
+  }
+
+  const itemId = typeof item.id === 'string' && item.id.trim()
+    ? item.id.trim()
+    : null;
+  if (!itemId) {
+    return null;
+  }
+
+  return {
+    itemId,
+    itemType: item.type,
+    text: typeof item.text === 'string' ? item.text : '',
+    isReasoning: item.type === 'reasoning'
+  };
+}
+
 /**
  * Map permission mode to Codex SDK options
  * @param {string} permissionMode - 'default', 'acceptEdits', or 'bypassPermissions'
@@ -195,7 +231,8 @@ export async function queryCodex(command, options = {}, ws) {
     cwd,
     projectPath,
     model,
-    permissionMode = 'default'
+    permissionMode = 'default',
+    modelReasoningEffort
   } = options;
 
   const workingDirectory = cwd || projectPath || process.cwd();
@@ -205,6 +242,7 @@ export async function queryCodex(command, options = {}, ws) {
   let thread;
   let currentSessionId = sessionId;
   let resolvedSessionIdFromEvent = null;
+  const streamedTextByItemId = new Map();
 
   const syncResolvedSessionId = (broadcastUpdate = false) => {
     const resolvedThreadId = typeof thread?.id === 'string' && thread.id.trim()
@@ -252,7 +290,8 @@ export async function queryCodex(command, options = {}, ws) {
       skipGitRepoCheck: true,
       sandboxMode,
       approvalPolicy,
-      model
+      model,
+      ...(modelReasoningEffort ? { modelReasoningEffort } : {})
     };
 
     // Start or resume thread
@@ -289,8 +328,11 @@ export async function queryCodex(command, options = {}, ws) {
     syncResolvedSessionId(true);
 
     for await (const event of streamedTurn.events) {
-      if (event?.type === 'thread.started' && typeof event.id === 'string' && event.id.trim()) {
-        resolvedSessionIdFromEvent = event.id.trim();
+      if (event?.type === 'thread.started') {
+        const eventThreadId = getThreadIdFromEvent(event);
+        if (eventThreadId) {
+          resolvedSessionIdFromEvent = eventThreadId;
+        }
       }
 
       syncResolvedSessionId(true);
@@ -301,8 +343,61 @@ export async function queryCodex(command, options = {}, ws) {
         break;
       }
 
-      if (event.type === 'item.started' || event.type === 'item.updated') {
-        continue;
+      const isItemEvent = event.type === 'item.started' || event.type === 'item.updated' || event.type === 'item.completed';
+      if (isItemEvent) {
+        const textItemInfo = getTextItemInfo(event.item);
+
+        if (textItemInfo) {
+          const previousText = streamedTextByItemId.get(textItemInfo.itemId) || '';
+          const nextText = textItemInfo.text || '';
+
+          let delta = '';
+          if (nextText.startsWith(previousText)) {
+            delta = nextText.slice(previousText.length);
+          } else if (nextText !== previousText) {
+            // Fallback when text is rewritten rather than appended
+            delta = nextText;
+          }
+
+          streamedTextByItemId.set(textItemInfo.itemId, nextText);
+
+          if (delta) {
+            sendMessage(ws, {
+              type: 'codex-response',
+              data: {
+                type: 'item_delta',
+                itemType: textItemInfo.itemType,
+                itemId: textItemInfo.itemId,
+                isReasoning: textItemInfo.isReasoning,
+                delta
+              },
+              sessionId: currentSessionId
+            });
+          }
+
+          if (event.type === 'item.completed') {
+            sendMessage(ws, {
+              type: 'codex-response',
+              data: {
+                type: 'item_done',
+                itemType: textItemInfo.itemType,
+                itemId: textItemInfo.itemId,
+                isReasoning: textItemInfo.isReasoning,
+                content: nextText
+              },
+              sessionId: currentSessionId
+            });
+            streamedTextByItemId.delete(textItemInfo.itemId);
+          }
+
+          continue;
+        }
+
+        // Non-text items can be noisy during started/updated phases.
+        // Keep existing behavior: emit them when completed.
+        if (event.type !== 'item.completed') {
+          continue;
+        }
       }
 
       const transformed = transformCodexEvent(event);