cliclaw 1.0.22 → 1.0.23

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cliclaw",
-  "version": "1.0.22",
+  "version": "1.0.23",
   "description": "Telegram bot bridging AI CLIs (Claude Code, Codex) to Forum Topics",
   "main": "index.ts",
   "scripts": {

package/src/agents/codex.ts

@@ -199,11 +199,14 @@ function getOrStart(appSessionId: string): ProtoSession {
 
 // ─── public API ──────────────────────────────────────────────────────────────
 export async function askCodex(
-  session: { id: string; codexThreadId?: string },
-  userMessage: string,
-  _onNewThreadId?: (id: string) => void
+  session:         { id: string; codexThreadId?: string },
+  userMessage:     string,
+  _onNewThreadId?: (id: string) => void,
+  approvalHandler?: (callId: string, commandStr: string) => Promise<boolean>
 ): Promise<{ text: string; usage?: TokenUsage }> {
   const ps = getOrStart(session.id)
+  // Update handler every call (chat context may have changed)
+  if (approvalHandler !== undefined) ps.approvalHandler = approvalHandler
   const msgId = randomUUID()
 
   return new Promise((resolve, reject) => {
@@ -237,15 +240,6 @@ export async function askCodex(
   )
 }
 
-/** Attach a per-session handler that is called whenever codex wants to run a command. */
-export function setCodexApprovalHandler(
-  appSessionId: string,
-  handler: (callId: string, commandStr: string) => Promise<boolean>
-): void {
-  const ps = protoSessions.get(appSessionId)
-  if (ps) ps.approvalHandler = handler
-}
-
 export function killCodexSession(appSessionId: string) {
   const ps = protoSessions.get(appSessionId)
   if (ps) {

package/src/config.ts

@@ -5,6 +5,7 @@ import { homedir } from 'os'
 
 export type AgentName = 'claude' | 'codex'
 export type PermissionMode = 'auto' | 'session' | 'ask'
+export type CodexApproval = 'ask' | 'allow'
 
 export interface Config {
   TELEGRAM_BOT_TOKEN: string
@@ -12,6 +13,7 @@ export interface Config {
   FORUM_GROUP_ID: string
   TELEGRAM_ADMIN_IDS: string[]
   PERMISSION_MODE: PermissionMode
+  CODEX_APPROVAL: CodexApproval
   availableAgents: AgentName[]
 }
 
@@ -59,6 +61,9 @@ export function loadConfig(): Config {
     rawMode === 'session' ? 'session' :
     rawMode === 'ask'     ? 'ask'     : 'auto'
 
+  const rawApproval = (process.env.CODEX_APPROVAL || 'ask').toLowerCase()
+  const codexApproval: CodexApproval = rawApproval === 'allow' ? 'allow' : 'ask'
+
   const config: Config = {
     TELEGRAM_BOT_TOKEN: process.env.TELEGRAM_BOT_TOKEN || '',
     DATA_DIR:           process.env.DATA_DIR || join(USER_DIR, 'data'),
@@ -66,6 +71,7 @@ export function loadConfig(): Config {
     TELEGRAM_ADMIN_IDS: (process.env.TELEGRAM_ADMIN_IDS || '')
       .split(',').map(id => id.trim()).filter(Boolean),
     PERMISSION_MODE:    permMode,
+    CODEX_APPROVAL:     codexApproval,
     availableAgents:    checkAgents(),
   }
 

package/src/handlers/messages.ts

@@ -3,7 +3,7 @@ import type { Bot, Context, Api } from 'grammy'
 import type { Storage, Session } from '../storage'
 import type { Config } from '../config'
 import { askClaude, type TokenUsage } from '../agents/claude'
-import { askCodex, setCodexApprovalHandler } from '../agents/codex'
+import { askCodex } from '../agents/codex'
 import { mdToTg, splitHtml } from '../utils/markdown'
 import { getLang, t } from '../i18n'
 import { registerApproval, respondApproval } from './approvals'
@@ -102,26 +102,24 @@ export function registerMessageHandler(bot: Bot<Context>, storage: Storage, conf
 
     storage.addMessage(chatId, session.id, 'user', text)
 
-    // ── Codex approval handler ──────────────────────────────────────────────
-    if (session.model === 'codex') {
-      setCodexApprovalHandler(session.id, (callId, cmdStr) => {
-        const approvalId = randomUUID()
-        const label = `🔧 <b>Codex quer executar:</b>\n<code>${escapeHtml(cmdStr)}</code>`
-        ctx.api.sendMessage(chatId, label, {
-          ...replyOpts,
-          parse_mode: 'HTML',
-          reply_markup: {
-            inline_keyboard: [[
-              { text: '✅ Aprovar', callback_data: `capprove:${approvalId}` },
-              { text: '❌ Negar',   callback_data: `cdeny:${approvalId}` },
-            ]],
-          },
-        }).catch(() => {})
-        return new Promise<boolean>((resolve) => {
-          registerApproval(approvalId, resolve)
-        })
-      })
-    }
+    // ── build codex approval handler (only if CODEX_APPROVAL=ask) ──────────
+    const codexApprovalHandler = (session.model === 'codex' && config.CODEX_APPROVAL === 'ask')
+      ? (callId: string, cmdStr: string) => {
+          const approvalId = randomUUID()
+          ctx.api.sendMessage(chatId,
+            `🔧 <b>Codex quer executar:</b>\n<code>${escapeHtml(cmdStr)}</code>`,
+            {
+              ...replyOpts,
+              parse_mode: 'HTML',
+              reply_markup: { inline_keyboard: [[
+                { text: '✅ Aprovar', callback_data: `capprove:${approvalId}` },
+                { text: '❌ Negar',   callback_data: `cdeny:${approvalId}` },
+              ]] },
+            }
+          ).catch(() => {})
+          return new Promise<boolean>((resolve) => registerApproval(approvalId, resolve))
+        }
+      : undefined
 
     let response: string
     try {
@@ -131,7 +129,8 @@ export function registerMessageHandler(bot: Bot<Context>, storage: Storage, conf
           (id) => storage.setClaudeSessionId(chatId, session!.id, id))
       } else {
         result = await askCodex(session, text,
-          (id) => storage.setCodexThreadId(chatId, session!.id, id))
+          (id) => storage.setCodexThreadId(chatId, session!.id, id),
+          codexApprovalHandler)
       }
       response = (isFirstMessage && result.usage) ? result.text + formatUsage(result.usage) : result.text
     } catch (err: any) {