@miller-tech/uap 1.45.0 → 1.45.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@miller-tech/uap",
-  "version": "1.45.0",
+  "version": "1.45.1",
   "description": "Autonomous AI agent memory system with CLAUDE.md protocol enforcement",
   "type": "module",
   "main": "dist/index.js",

package/tools/agents/opencode_uap_agent.py

@@ -1409,14 +1409,7 @@ class OpenCodeUAP(BaseInstalledAgent):
     def create_run_agent_commands(self, instruction: str) -> list[ExecInput]:
         model = self.model_name or "llama.cpp/qwen35-a3b-iq4xs"
 
-        env = {
-            "OPENCODE_FAKE_VCS": "git",
-            # Transparent delivery: the uap-enforce plugin reads these to run
-            # the `uap deliver` convergence loop on the first source edit.
-            "UAP_DELIVER_ENDPOINT": self._api_endpoint,
-            "UAP_DELIVER_MODEL": os.environ.get("UAP_DELIVER_MODEL", "qwen35-a3b"),
-            "UAP_ENFORCE_DELIVERY": os.environ.get("UAP_ENFORCE_DELIVERY", "block"),
-        }
+        env = {"OPENCODE_FAKE_VCS": "git"}
 
         # --- Step 0: Build classified CLAUDE.md and enhanced instruction ---
         classified_claude_md = build_classified_claude_md(instruction)
@@ -1542,17 +1535,6 @@ class OpenCodeUAP(BaseInstalledAgent):
         # --- Step 5: Environment bootstrapping ---
         commands.append(ExecInput(command=ENV_BOOTSTRAP_CMD))
 
-        # --- Step 5b: Stage raw task instruction for transparent delivery ---
-        # The uap-enforce plugin reads /app/.uap-deliver/task.txt to seed the
-        # `uap deliver` convergence loop on the first source edit.
-        task_b64 = base64.b64encode(instruction.encode()).decode()
-        deliver_task_cmd = (
-            "mkdir -p /app/.uap-deliver && "
-            f"echo '{task_b64}' | base64 -d > /app/.uap-deliver/task.txt && "
-            "echo '[Deliver] task instruction staged for transparent delivery'"
-        )
-        commands.append(ExecInput(command=deliver_task_cmd))
-
         # --- Step 6: Run opencode with enhanced instruction ---
         # opencode.json baseURL points to proxy at http://127.0.0.1:11435/v1
         # which injects tool_choice="required" and forwards to the real LLM

package/tools/agents/plugins/uap-enforce.ts

@@ -41,11 +41,6 @@ export const UapEnforce: Plugin = async ({ $ }) => {
   let totalToolCalls = 0;
   const SOFT_BUDGET = 30;
 
-  // Transparent delivery: the first substantive source edit triggers the
-  // `uap deliver` convergence loop ONCE per session. Guarded so it never
-  // re-enters or fires per-edit.
-  let deliverRan = false;
-
   const TELEMETRY_PATH = '/tmp/uap-telemetry.jsonl';
 
   const simpleHash = (s: string): string => {
@@ -65,46 +60,14 @@ export const UapEnforce: Plugin = async ({ $ }) => {
     'tool.execute.before': async (input, output) => {
       totalToolCalls++;
 
-      // --- Transparent delivery (Layer 5): converge via `uap deliver` ---
-      // When delivery enforcement is active (the UAP default), the FIRST
-      // substantive write/edit hands the task to the `uap deliver` convergence
-      // loop, which iterates the model against the project's real gates
-      // (build/typecheck/test) before the agent's own edit lands. This is how
-      // deliver "uplifts" a small local model: plausible -> verified. It runs
-      // exactly once per session; the original edit then proceeds normally.
-      // Escape hatches: UAP_ENFORCE_DELIVERY=advisory or UAP_DELIVER_BYPASS=1.
-      if (
-        !deliverRan &&
-        (input.tool === 'write' || input.tool === 'edit') &&
-        process.env.UAP_ENFORCE_DELIVERY !== 'advisory' &&
-        !process.env.UAP_DELIVER_BYPASS
-      ) {
-        deliverRan = true; // set first — prevents re-entry / per-edit loops
-        try {
-          const endpoint = process.env.UAP_DELIVER_ENDPOINT || '';
-          const model = process.env.UAP_DELIVER_MODEL || 'qwen35-a3b';
-          const endpointArg = endpoint ? `--endpoint ${endpoint}` : '';
-          const deliverScript =
-            'source $HOME/.nvm/nvm.sh 2>/dev/null || true; ' +
-            'TASK="$(cat /app/.uap-deliver/task.txt 2>/dev/null)"; ' +
-            'if [ -n "$TASK" ] && command -v uap >/dev/null 2>&1; then ' +
-            `echo "[Deliver] converging via uap deliver (model ${model})..."; ` +
-            `UAP_DELIVER_ACTIVE=1 uap deliver "$TASK" ${endpointArg} --model ${model} ` +
-            '--project-root /app --max-turns 5 --no-until-delivered ' +
-            '> /logs/agent/uap-deliver.log 2>&1 || true; ' +
-            'echo "[Deliver] convergence loop finished"; ' +
-            'else echo "[Deliver] skipped (no task file or uap CLI absent)"; fi';
-          await $`bash -lc ${deliverScript}`.nothrow();
-          await $`echo ${JSON.stringify(
-            JSON.stringify({ event: 'transparent_deliver', tool: input.tool, ts: new Date().toISOString() })
-          )} >> ${TELEMETRY_PATH}`
-            .quiet()
-            .nothrow();
-        } catch {
-          /* fail open — let the original edit proceed */
-        }
-        // fall through: the agent's own edit still executes
-      }
+      // NOTE: a transparent "run `uap deliver` on the first edit" trigger was
+      // trialed here and REMOVED. Benchmarking showed it net-harmful: it ran a
+      // heavyweight second agent (the deliver convergence loop) against a
+      // self-authored PROXY gate that does not match the task's hidden verifier,
+      // mutating the workspace and corrupting tasks plain opencode solved
+      // (e.g. openssl-selfsigned-cert regressed; 4/6 -> 2/6). deliver's
+      // self-gate + agentic executor remain valuable as a STANDALONE `uap
+      // deliver` invocation, just not as a per-edit parasite on the agent loop.
 
       // --- Worktree enforcement (Layer 4) ---
       // Warn when file writes happen outside a worktree