pushary 1.9.69 → 1.9.70

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 1.9.70
+
+- Hermes: the plugin (0.8.1) no longer sends "Tool error in Hermes" to your phone each time a tool call fails. Hermes counts any command that exits non-zero as a failed tool call, so a `grep` that finds nothing or a test run that fails on purpose became a high-urgency push, up to three per Hermes session, and Hermes recovers from those on its own.
+- When Hermes marks a turn as failed, the push for that turn now reads "Hermes run failed" with the reason Hermes gives, instead of "Hermes finished", and the session keeps the reason. It is the same push that ends every turn and follows your updates setting, so a scheduled job that keeps failing sends no more pushes than before. A provider error that stops Hermes before it finishes the turn, such as a rejected API key, still sends nothing. Hermes redacts the reason before it leaves your computer.
+- The daily update and `pushary upgrade` replace a plugin folder that setup installed. Run `hermes gateway restart` to load the new copy. Pushary already drops the old copy's tool-error reports without a push, so they stop reaching your phone before you restart.
+
 ## 1.9.69
 
 - fx: upgrade and the daily update also switch a `pushary` server written as `"type": "streamable-http"`, which fx reads as HTTP. An entry with no `type` is no longer taken for an HTTP server, because fx runs it as a local process.

package/dist/bin/pushary-clean.js

@@ -53,7 +53,7 @@ import {
   defaultHermesHome,
   hermesReceiptPaths,
   removeHermesWiring
-} from "../chunk-Z3FMJ567.js";
+} from "../chunk-KLDUATG5.js";
 import "../chunk-QCGNGEGE.js";
 import {
   planClineRemoval,

package/dist/bin/pushary-codex-hook.js

@@ -808,7 +808,7 @@ var decidePreToolUse = async (input) => {
 };
 var repairCodexHoldsQuietly = async () => {
   try {
-    const { repairCodexHolds } = await import("../reapply-5SZSMZJJ.js");
+    const { repairCodexHolds } = await import("../reapply-TRR4XB6N.js");
     repairCodexHolds();
   } catch {
     return;

package/dist/bin/pushary-daemon.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import {
   inspectConfiguredManagedProviders
-} from "../chunk-SCKZVTFV.js";
+} from "../chunk-A7GTEZGH.js";
 import {
   HEARTBEAT_INTERVAL_MS,
   sendHeartbeat
@@ -10,7 +10,7 @@ import "../chunk-54O76CMR.js";
 import {
   readSignedOutDiagnosis
 } from "../chunk-QE22XDUK.js";
-import "../chunk-Z3FMJ567.js";
+import "../chunk-KLDUATG5.js";
 import {
   writeHuman
 } from "../chunk-QCGNGEGE.js";

package/dist/bin/pushary-doctor.js

@@ -74,7 +74,7 @@ import {
   routesHermesQuestions,
   runsAnOlderPlugin,
   shippedHermesPlugin
-} from "../chunk-Z3FMJ567.js";
+} from "../chunk-KLDUATG5.js";
 import "../chunk-QCGNGEGE.js";
 import {
   FX_PROFILE_REFUSALS,

package/dist/bin/pushary-session-start-hook.js

@@ -1,9 +1,9 @@
 #!/usr/bin/env node
 import {
   reapplyClaudeForVersion
-} from "../chunk-SCKZVTFV.js";
+} from "../chunk-A7GTEZGH.js";
 import "../chunk-54O76CMR.js";
-import "../chunk-Z3FMJ567.js";
+import "../chunk-KLDUATG5.js";
 import "../chunk-SSJDAX4E.js";
 import "../chunk-N2FDORN6.js";
 import "../chunk-D5G2HSQ6.js";

package/dist/bin/pushary-setup.js

@@ -117,7 +117,7 @@ import {
   routesHermesQuestions,
   runsAnOlderPlugin,
   shippedHermesPlugin
-} from "../chunk-Z3FMJ567.js";
+} from "../chunk-KLDUATG5.js";
 import {
   setHumanStream
 } from "../chunk-QCGNGEGE.js";

package/dist/bin/pushary-upgrade.js

@@ -28,7 +28,7 @@ import {
 import {
   ensureCodexTrustAfterReapply,
   reapplyAllAgents
-} from "../chunk-SCKZVTFV.js";
+} from "../chunk-A7GTEZGH.js";
 import {
   HERMES_RESTART_COMMAND,
   describeCodexTrustConfirmation
@@ -42,7 +42,7 @@ import {
   readMacAppAgentKey
 } from "../chunk-PK6WI4HH.js";
 import "../chunk-XLKO2XEG.js";
-import "../chunk-Z3FMJ567.js";
+import "../chunk-KLDUATG5.js";
 import {
   FX_RELOAD_AFTER_BRIDGE,
   fxHttpEntryKeptNote,

package/dist/{chunk-SCKZVTFV.js → chunk-A7GTEZGH.js}

@@ -7,7 +7,7 @@ import {
   installHermesPlugin,
   refreshHermesPluginFiles,
   shippedHermesPlugin
-} from "./chunk-Z3FMJ567.js";
+} from "./chunk-KLDUATG5.js";
 import {
   fxIgnoresInstructionsFile,
   fxInstructionsBody,

package/dist/{chunk-Z3FMJ567.js → chunk-KLDUATG5.js}

@@ -34,7 +34,9 @@ from . import api, approval, host, identity, installation, schemas, tools
 
 logger = logging.getLogger(__name__)
 
-MAX_ERROR_NOTIFICATIONS = 3
+FAILED_TURN_WITHOUT_REASON = "no reason given"
+REDACTION_UNAVAILABLE = "[redaction unavailable]"
+FAILED_TURN_REASON_MAX_LENGTH = 200
 SYSTEM_PROMPT_SECTION_ID = "pushary.reach-the-human"
 HERMES_QUESTION_TOOL = "clarify"
 QUESTION_ROUTING_PROMPT = (
@@ -82,7 +84,7 @@ TOOL_REGISTRATIONS = (
      "Ask one of your own end-users and block on a fail-closed answer (Partner plan)"),
 )
 
-_session_totals = {"tools": 0, "errors": 0, "notified": 0}
+_session_totals = {"tools": 0, "errors": 0}
 _last_turn = {"completed": False, "interrupted": False}
 _plugin_activity = {"attempted": False}
 _plugin_activity_lock = threading.Lock()
@@ -173,7 +175,7 @@ def _redact_outbound_text(text: str) -> str:
         return redact_for_egress(redacted) if redact_for_egress else redacted
     except Exception:
         logger.warning("[pushary] host redaction unavailable; omitting outbound detail")
-        return "[redaction unavailable]"
+        return REDACTION_UNAVAILABLE
 
 
 def _summarize_args(args):
@@ -312,52 +314,25 @@ def _on_pre_tool_call(tool_name=None, args=None, session_id=None, **kwargs):
         return _block_unapproved(tool_name)
 
 
-def _tool_failure(result, status, error_message):
+def _tool_failed(result, status):
     if status is not None:
-        return (error_message or "tool returned an error") if status == "error" else None
+        return status == "error"
     try:
         parsed = json.loads(result) if isinstance(result, str) else result
     except (json.JSONDecodeError, TypeError):
-        return None
-    if isinstance(parsed, dict) and parsed.get("error"):
-        return str(parsed["error"])
-    return None
+        return False
+    return isinstance(parsed, dict) and bool(parsed.get("error"))
 
 
-def _on_post_tool_call(tool_name, args, result, duration_ms=0, status=None,
-                       error_message=None, **kwargs):
+def _on_post_tool_call(tool_name, args, result, duration_ms=0, status=None, **kwargs):
     _session_totals["tools"] += 1
-
-    failure = _tool_failure(result, status, error_message)
-    if failure is None:
-        return
-
-    _session_totals["errors"] += 1
-    if not api.is_configured():
-        return
-    if _session_totals["notified"] >= MAX_ERROR_NOTIFICATIONS:
-        return
-    _session_totals["notified"] += 1
-
-    try:
-        failure = _redact_outbound_text(str(failure))
-        api.send_notification(
-            title="Tool error in Hermes",
-            body=f"{tool_name} failed: {failure[:150]}",
-            agent_name=identity.agent_name(),
-            context={
-                "type": "error",
-                "errorMessage": failure[:500],
-                "summary": f"Tool '{tool_name}' returned an error after {duration_ms}ms",
-            },
-        )
-    except Exception:
-        pass
+    if _tool_failed(result, status):
+        _session_totals["errors"] += 1
 
 
 def _on_session_start(session_id=None, **kwargs):
     identity.remember_session(session_id)
-    _session_totals.update(tools=0, errors=0, notified=0)
+    _session_totals.update(tools=0, errors=0)
     _last_turn.update(completed=False, interrupted=False)
     _report_session_event("session_start", session_id, can_drain=True)
     _report_plugin_active_once(session_id)
@@ -397,26 +372,40 @@ def _on_pre_llm_call(session_id=None, **kwargs):
     }
 
 
-def _report_session_event(event, session_id, can_drain=False):
+def _report_session_event(event, session_id, can_drain=False, error=None):
     if not api.is_configured() or not session_id:
         return
     try:
-        api.agent_event(event, session_id, can_drain=can_drain)
+        api.agent_event(event, session_id, can_drain=can_drain, error=error)
     except Exception:
         pass
 
 
-def _on_session_end(session_id=None, completed=False, interrupted=False, **kwargs):
+def _turn_failure_reason(failed, interrupted, turn_exit_reason):
+    if failed is not True or interrupted:
+        return None
+    reason = _redact_outbound_text(str(turn_exit_reason or "").strip())
+    if not reason or reason == REDACTION_UNAVAILABLE:
+        return FAILED_TURN_WITHOUT_REASON
+    return reason[:FAILED_TURN_REASON_MAX_LENGTH]
+
+
+def _on_session_end(session_id=None, completed=False, interrupted=False, failed=False,
+                    turn_exit_reason=None, **kwargs):
     identity.remember_session(session_id)
     _last_turn.update(completed=bool(completed), interrupted=bool(interrupted))
-    _report_session_event("session_end", session_id or identity.session_id())
+    _report_session_event(
+        "session_end",
+        session_id or identity.session_id(),
+        error=_turn_failure_reason(failed, interrupted, turn_exit_reason),
+    )
 
 
 def _on_session_finalize(session_id=None, **kwargs):
     _report_session_event("session_closed", session_id or identity.session_id())
     tool_count = _session_totals["tools"]
     error_count = _session_totals["errors"]
-    _session_totals.update(tools=0, errors=0, notified=0)
+    _session_totals.update(tools=0, errors=0)
 
     if not os.environ.get("PUSHARY_AUTO_NOTIFY_SESSION_END"):
         identity.forget_session()
@@ -635,7 +624,7 @@ def report_plugin_active(session_id):
         return False
 
 
-def agent_event(event, session_id, can_drain=False):
+def agent_event(event, session_id, can_drain=False, error=None):
     """Report a Hermes session and pick up a queued notch instruction."""
     try:
         pending = _read_pending(session_id) if can_drain else None
@@ -647,6 +636,7 @@ def agent_event(event, session_id, can_drain=False):
             "canDrainCommand": can_drain and pending is None,
             "ackMode": can_drain and pending is None,
             "skipCommandDrain": pending is not None,
+            **({"error": error} if error else {}),
         })
         with urllib.request.urlopen(req, timeout=10) as resp:
             raw = resp.read().decode("utf-8")
@@ -1073,7 +1063,7 @@ def record_plugin_load(approval_transport_registered, home=None, pid=None, now=N
         logger.debug("[pushary] could not record the plugin load: %s", exc)
         return None
 ` },
-  { path: "plugin.yaml", contents: "name: pushary\nversion: 0.8.0\ndescription: Push notifications and human-in-the-loop for Hermes Agent via Pushary\nauthor: RalphNex OU\nrequires_env:\n  - name: PUSHARY_API_KEY\n    description: Your Pushary API key\n    url: https://pushary.com/sign-up?from=hermes\n    secret: true\nprovides_tools:\n  - pushary_remind\n  - pushary_notify\n  - pushary_ask\n  - pushary_wait\n  - pushary_cancel\n  - pushary_propose_scope\n  - pushary_enroll\n  - pushary_ask_end_user\nprovides_hooks:\n  - pre_llm_call\n  - pre_tool_call\n  - post_tool_call\n  - on_session_start\n  - on_session_end\n  - on_session_finalize\nprovides_approval_transports:\n  - pushary\n" },
+  { path: "plugin.yaml", contents: "name: pushary\nversion: 0.8.1\ndescription: Push notifications and human-in-the-loop for Hermes Agent via Pushary\nauthor: RalphNex OU\nrequires_env:\n  - name: PUSHARY_API_KEY\n    description: Your Pushary API key\n    url: https://pushary.com/sign-up?from=hermes\n    secret: true\nprovides_tools:\n  - pushary_remind\n  - pushary_notify\n  - pushary_ask\n  - pushary_wait\n  - pushary_cancel\n  - pushary_propose_scope\n  - pushary_enroll\n  - pushary_ask_end_user\nprovides_hooks:\n  - pre_llm_call\n  - pre_tool_call\n  - post_tool_call\n  - on_session_start\n  - on_session_end\n  - on_session_finalize\nprovides_approval_transports:\n  - pushary\n" },
   { path: "schemas.py", contents: `PUSHARY_REMIND = {
     "name": "pushary_remind",
     "description": "Schedule a one-time personal reminder on Pushary, even after Hermes exits. Use body plus in_minutes or at (ISO timestamp with explicit UTC offset, within 30 days). Never guess the user's timezone. Confirm the returned time. No arguments lists pending reminders; cancel_reminder_id cancels a scheduled one. Use Hermes cron for agent work; use this for reminding the operator.",

package/dist/{reapply-5SZSMZJJ.js → reapply-TRR4XB6N.js}

@@ -12,9 +12,9 @@ import {
   reapplyOpenCodePlugin,
   reapplyVsCodePlugin,
   repairCodexHolds
-} from "./chunk-SCKZVTFV.js";
+} from "./chunk-A7GTEZGH.js";
 import "./chunk-54O76CMR.js";
-import "./chunk-Z3FMJ567.js";
+import "./chunk-KLDUATG5.js";
 import "./chunk-SSJDAX4E.js";
 import "./chunk-N2FDORN6.js";
 import "./chunk-D5G2HSQ6.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pushary",
-  "version": "1.9.69",
+  "version": "1.9.70",
   "description": "Permission hooks for AI coding agents: route tool approvals through Pushary push notifications",
   "keywords": [
     "pushary",