create-loopengine 0.1.68 → 0.1.70

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-loopengine",
-  "version": "0.1.68",
+  "version": "0.1.70",
   "description": "Scaffold a new loopengine agent project — a runnable HTTP server with one example agent, in your own repo.",
   "license": "MIT",
   "type": "module",

package/template/adapters/http.ts

@@ -130,12 +130,11 @@ import {
   setAgentApprovers,
   listAgentSlackMembers,
   replyPrivatelyInSlack,
+  slackArgs,
   type MessagingDecision,
   type DecisionOutcome,
   type MessagingDeps,
   handleMessagingAction,
-  agentConfirmTools,
-  setAgentConfirmTools,
   openSlackAnswerModal,
   parseSlackInteraction,
   replyInSlack,
@@ -1291,28 +1290,6 @@ async function handleApproversPut(req: IncomingMessage, res: ServerResponse, age
   }
 }
 
-// Messaging tab → Confirm-required tools: tools whose approval by reply
-// needs a Confirm click. Admin login needed, like the approvers.
-async function handleConfirmToolsPut(req: IncomingMessage, res: ServerResponse, agentName: string): Promise<void> {
-  const entry = getEntry(agentName)
-  if (!entry) {
-    res.writeHead(404, { 'content-type': 'application/json' }).end(JSON.stringify({ error: `unknown agent '${agentName}'` }))
-    return
-  }
-  if (!secretsProtected) {
-    res.writeHead(403, { 'content-type': 'application/json' }).end(JSON.stringify({ error: 'Changing Confirm-required tools needs LOOPENGINE_ADMIN_AUTH set on the server.' }))
-    return
-  }
-  const body = await readJsonBody(req)
-  try {
-    const tools = setAgentConfirmTools(entry.config, body.tools)
-    res.writeHead(200, { 'content-type': 'application/json' }).end(JSON.stringify({ tools }))
-  } catch (err) {
-    const status = err instanceof NotifierFileError ? 400 : 500
-    res.writeHead(status, { 'content-type': 'application/json' }).end(JSON.stringify({ error: err instanceof Error ? err.message : String(err) }))
-  }
-}
-
 async function handleSlackMembers(res: ServerResponse, agentName: string): Promise<void> {
   const entry = getEntry(agentName)
   if (!entry) {
@@ -2316,7 +2293,12 @@ async function resolvePendingApproval(res: ServerResponse, pendingId: string, bo
             sessionId: checkpoint.sessionId,
             env: createAgentEnv(entry.config.name, agentDirFor(entry.config)),
           })
-          resultBlock = { type: 'tool_result', tool_use_id: item.toolUseId, content: JSON.stringify(output), is_error: false }
+          // The agent's own history still shows the arguments it asked
+          // for, so say when the approver changed them; otherwise its
+          // reply describes what it asked for, not what ran.
+          const changed = editedArgs && JSON.stringify(editedArgs) !== JSON.stringify(item.args)
+          const content = changed ? `The approver changed the arguments before it ran: ${JSON.stringify(editedArgs)}. Result: ${JSON.stringify(output)}` : JSON.stringify(output)
+          resultBlock = { type: 'tool_result', tool_use_id: item.toolUseId, content, is_error: false }
         } catch (err) {
           resultBlock = { type: 'tool_result', tool_use_id: item.toolUseId, content: `ERROR: ${err}`, is_error: true }
         }
@@ -2812,7 +2794,7 @@ async function handleSlackInteraction(req: IncomingMessage, res: ServerResponse,
   let ran = ''
   if (interaction.type === 'decide' && interaction.decision === 'approve') {
     const item = (await checkpoints.get(interaction.pendingId))?.outstanding[interaction.pendingId]
-    if (item) ran = `\nRan \`${item.tool}\` with \`${JSON.stringify(item.args)}\`.`
+    if (item) ran = `\nRan \`${item.tool}\` with:\n${slackArgs(item.args, (await resolveAgentTool(entry.config, item.tool))?.input_schema)}`
   }
   const capture = captureResponse()
   if (interaction.type === 'decide') await resolvePendingApproval(capture.res, interaction.pendingId, { decision: interaction.decision })
@@ -2875,7 +2857,6 @@ function slackMessagingDeps(agentName: string, entry: RegistryEntry, botToken: s
     getHistory: (checkpoint) => sessions.getHistory(`${checkpoint.tenant}:${environment}:${checkpoint.agent}:${checkpoint.sessionId}`),
     plan: process.env.LOOPENGINE_PLAN,
     approvers: () => agentApprovers(entry.config),
-    confirmTools: () => agentConfirmTools(entry.config),
     decide: decideFromMessaging,
     toolSchema: async (toolName) => (await resolveAgentTool(entry.config, toolName))?.input_schema,
     // The same checks the console's approve-with-edits applies, plus the
@@ -3469,11 +3450,6 @@ const server = createServer(async (req, res) => {
       await handleApproversPut(req, res, decodeURIComponent(approversMatch[1]))
       return
     }
-    const confirmToolsMatch = req.method === 'PUT' && pathname.match(/^\/agents\/([^/]+)\/notifier\/confirm-tools$/)
-    if (confirmToolsMatch) {
-      await handleConfirmToolsPut(req, res, decodeURIComponent(confirmToolsMatch[1]))
-      return
-    }
     const slackMembersMatch = req.method === 'GET' && pathname.match(/^\/agents\/([^/]+)\/notifier\/slack-members$/)
     if (slackMembersMatch) {
       await handleSlackMembers(res, decodeURIComponent(slackMembersMatch[1]))

package/template/package.json

@@ -9,7 +9,7 @@
     "cli": "tsx adapters/cli.ts"
   },
   "dependencies": {
-    "loopengine": "^0.1.84",
+    "loopengine": "^0.1.86",
     "actauth": "^0.0.12",
     "tsx": "^4.19.0",
     "typescript": "^5.7.0"