@inquiryon/openclaw-amp-governance 1.0.4 → 1.0.6

package/index.js

@@ -1,4 +1,5 @@
 import * as fs from 'fs';
+import { exec } from 'child_process';
 
 console.log('[AMP Governance] Plugin module loaded — Phase 4.');
 
@@ -23,6 +24,9 @@ try {
 // Module-level instance cache so we only init once per process lifetime
 let _instanceId = null;
 
+// Last known sender — populated by message_received, used for HITL notifications
+let _lastSender = null;  // { from: string, channelId: string }
+
 function readSession() {
   try {
     return JSON.parse(fs.readFileSync(SESSION_FILE, 'utf-8'));
@@ -111,6 +115,25 @@ async function ampLog(instanceId, message, level = 'INFO') {
   }
 }
 
+// ── PROACTIVE NOTIFICATIONS ──────────────────────────────────────────────────
+
+/**
+ * Send a proactive WhatsApp message to the user without waiting for the agent
+ * to finish. Uses the openclaw CLI so we don't need to reverse-engineer the
+ * gateway REST API.  Fire-and-forget — errors are logged but never thrown.
+ */
+const OPENCLAW_BIN = `${process.env.HOME}/.npm-global/bin/openclaw`;
+
+function notifyUser(sender, message) {
+  if (!sender?.from || !sender?.channelId) return;
+  const safeMsg = message.replace(/"/g, '\\"');
+  const cmd = `${OPENCLAW_BIN} message send --channel ${sender.channelId} --target "${sender.from}" --message "${safeMsg}"`;
+  console.log(`[AMP Governance] Sending notification to ${sender.from}: ${message}`);
+  exec(cmd, (err) => {
+    if (err) console.warn('[AMP Governance] notifyUser failed:', err.message);
+  });
+}
+
 // ── HITL POLICY ENGINE ───────────────────────────────────────────────────────
 
 /**
@@ -209,12 +232,16 @@ async function checkToolPolicy(instanceId, tool, params) {
     console.log(`[AMP Governance] HITL required for "${tool}" — waiting for human decision...`);
     await ampLog(instanceId, `HITL requested for tool: ${tool} — awaiting human approval in AMP`, 'WARN');
 
+    // Notify the user immediately so they aren't staring at silence
+    notifyUser(_lastSender, `I need a reviewer to approve "${tool}" before I can proceed. I'll follow up once the decision is made — this may take a few minutes.`);
+
     let decision;
     try {
       decision = await pollHitlDecision(instanceId);
     } catch (err) {
       const msg = `Tool call "${tool}" timed out waiting for human approval — blocked.`;
       await ampLog(instanceId, msg, 'ERROR');
+      notifyUser(_lastSender, `No response from the reviewer — "${tool}" was blocked due to timeout.`);
       return { block: true, blockReason: msg };
     }
 
@@ -223,11 +250,13 @@ async function checkToolPolicy(instanceId, tool, params) {
 
     if (resolution === 'approve' || resolution === 'approved') {
       await ampLog(instanceId, `HITL approved: ${tool} | workitem: ${decision.workitem_id}`);
+      notifyUser(_lastSender, `The reviewer approved "${tool}" — continuing now.`);
       return {};
     }
 
     if (resolution === 'modify' || resolution === 'modified') {
       await ampLog(instanceId, `HITL approved with modification: ${tool} | workitem: ${decision.workitem_id}`);
+      notifyUser(_lastSender, `The reviewer approved "${tool}" (with modifications) — continuing now.`);
       return {};
     }
 
@@ -235,6 +264,7 @@ async function checkToolPolicy(instanceId, tool, params) {
     const info = decision.information ? ` Reviewer note: ${decision.information}` : '';
     const msg = `Tool call "${tool}" was rejected by a human reviewer.${info}`;
     await ampLog(instanceId, `HITL rejected: ${tool} | workitem: ${decision.workitem_id}`, 'WARN');
+    notifyUser(_lastSender, `The reviewer rejected "${tool}".${info}`);
     return { block: true, blockReason: msg };
   }
 
@@ -360,6 +390,14 @@ export default {
   register(api) {
     api.logger.info('AMP Governance registered. Phase 4 - eval policy enforcement active.');
 
+    // ── INBOUND MESSAGE: cache sender for HITL notifications ─────────────────
+    api.on('message_received', (event, ctx) => {
+      if (event.from && ctx.channelId) {
+        _lastSender = { from: event.from, channelId: ctx.channelId };
+        console.log(`[AMP Governance] Sender cached: ${event.from} on ${ctx.channelId}`);
+      }
+    });
+
     // ── BEFORE TOOL CALL: governance check + logging ─────────────────────────
     api.on('before_tool_call', async (event) => {
       const tool = event.toolName || event.name || 'unknown-tool';
@@ -403,7 +441,6 @@ export default {
       const instanceId = _instanceId || readSession()?.instanceId;
       const msgText    = event.content || event.text || event.message || JSON.stringify(event);
       const preview    = msgText.substring(0, 100);
-      console.log(`[AMP Governance] message_sending fired: ${preview}`);
       if (instanceId) {
         await ampLog(instanceId, `Agent reply: ${preview}`);
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inquiryon/openclaw-amp-governance",
-  "version": "1.0.4",
+  "version": "1.0.6",
   "description": "AMP governance plugin for OpenClaw",
   "type": "module",
   "files": [