nofax 0.2.0

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 Tomi Šeregi
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README.md

@@ -0,0 +1,351 @@
+# Nofax
+
+[![CI](https://github.com/AKzar1el/nofax/actions/workflows/ci.yml/badge.svg?branch=main)](https://github.com/AKzar1el/nofax/actions/workflows/ci.yml)
+[![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
+[![Node.js >=20](https://img.shields.io/badge/node-%3E%3D20-339933?logo=node.js&logoColor=white)](package.json)
+[![MCP](https://img.shields.io/badge/MCP-compatible-6f42c1)](https://modelcontextprotocol.io/)
+
+**Human-in-the-loop approvals and notifications for AI coding agents, without running a Nofax SaaS.**
+
+Nofax is a small open-source bridge between an agent and a human. Local mode can pause an AI workflow, notify your phone, and return an explicit decision. An optional self-deployed Cloudflare Worker exposes a deliberately narrower remote MCP surface for one-way notifications and safe request inspection.
+
+No Nofax account. No paid model API. No inbound port on your machine. MIT licensed.
+
+> **Current status:** local Nofax is `0.2.0`. The optional Cloudflare Worker is the upcoming `0.3.0` remote surface and is developed alongside the local package.
+
+## Why Nofax
+
+Agent workflows increasingly need a clean answer to one question: **when automation reaches a human decision boundary, how does it ask without pretending that silence means approval?**
+
+Nofax keeps that boundary explicit:
+
+- pending is never approval;
+- timeout and transport failure fail closed;
+- the first accepted terminal response wins;
+- agent-specific hook schemas stay isolated in adapters;
+- remote access is intentionally narrower than local access;
+- Nofax does not grant authority the calling agent did not already have.
+
+## Two operating modes
+
+| Capability | Local Nofax `0.2` | Remote Worker `0.3` |
+| --- | --- | --- |
+| Transport | stdio / CLI hooks | MCP Streamable HTTP |
+| One-way notification | Yes | Yes |
+| Allow / Deny | Yes | No |
+| Explicit choices | Yes | No |
+| Free-text refinement | Yes | No |
+| Wait for human response | Yes | No |
+| Read request metadata | Yes | Yes |
+| Durable state | Local files | Existing SQLite Durable Object rows |
+| Hosted by Nofax | No | No — self-deployed Worker |
+| Remote authentication | Local process boundary | Private bearer key |
+
+The remote Worker is **not** a hosted remote-approval service. It can send an informational notification and inspect existing request state, but it has no approval callback, choice, refinement, wait, webhook, or arbitrary remote-write endpoint.
+
+## Quick start
+
+### 1. Install
+
+Until a registry release is published:
+
+```bash
+npm install -g https://github.com/AKzar1el/nofax.git
+```
+
+Requires Node.js 20 or newer.
+
+### 2. Initialize
+
+```bash
+nofax init
+```
+
+Nofax creates `~/.nofax/config.json` and generates a high-entropy notification topic. With the default transport, subscribe to the displayed topic in the ntfy mobile app.
+
+### 3. Test
+
+```bash
+nofax test
+```
+
+### 4. Use it
+
+```bash
+nofax notify --title "Build finished" "All tests passed"
+nofax approve --title "Deploy?" "Release 1.4.0 is ready"
+nofax refine --title "Refine draft" "Tell me what to change"
+```
+
+An approval resolves to stable terminal JSON:
+
+```json
+{"decision":"allow"}
+```
+
+or:
+
+```json
+{"decision":"deny"}
+```
+
+If the request is still pending, times out, disconnects, or hits a transport error, Nofax never converts that condition into approval.
+
+## MCP
+
+Start the local stdio MCP server:
+
+```bash
+nofax mcp
+```
+
+Local MCP exposes:
+
+- `nofax_notify`
+- `nofax_request_approval`
+- `nofax_request_choice`
+- `nofax_request_refinement`
+- `nofax_wait_for_response`
+- `nofax_get_request`
+- `nofax_list_pending`
+
+Interactive requests return a durable request ID. `nofax_wait_for_response` performs a bounded wait; callers must repeat the wait while the request remains pending rather than infer approval.
+
+## Agent integrations
+
+### Claude Code
+
+Use Nofax as a local `PermissionRequest` hook in `~/.claude/settings.json`:
+
+```json
+{
+  "hooks": {
+    "PermissionRequest": [
+      {
+        "matcher": ".*",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "nofax hook claude"
+          }
+        ]
+      }
+    ]
+  }
+}
+```
+
+### Codex
+
+Codex hooks are enabled by default. Configure `~/.codex/hooks.json`:
+
+```json
+{
+  "hooks": {
+    "PermissionRequest": [
+      {
+        "matcher": ".*",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "nofax hook codex",
+            "statusMessage": "Waiting for Nofax approval"
+          }
+        ]
+      }
+    ]
+  }
+}
+```
+
+Restart Codex, run `/hooks`, and review/trust the exact Nofax hook definition before relying on it. Codex skips non-managed hooks until they are trusted, and a changed hook definition must be reviewed again. If an administrator or local policy has explicitly disabled hooks, re-enable them with `[features] hooks = true` in `~/.codex/config.toml`.
+
+### Gemini CLI
+
+Current Gemini CLI builds expose a synchronous `BeforeTool` hook that can allow or deny a tool call. Route selected tools through Nofax in `~/.gemini/settings.json`:
+
+```json
+{
+  "hooks": {
+    "BeforeTool": [
+      {
+        "matcher": "run_shell_command|write_file|replace",
+        "hooks": [
+          {
+            "name": "nofax-approval",
+            "type": "command",
+            "command": "nofax hook gemini",
+            "timeout": 305000
+          }
+        ]
+      }
+    ],
+    "Notification": [
+      {
+        "matcher": "ToolPermission",
+        "hooks": [
+          {
+            "name": "nofax-notification",
+            "type": "command",
+            "command": "nofax hook gemini"
+          }
+        ]
+      }
+    ]
+  }
+}
+```
+
+`BeforeTool` waits for an explicit Nofax Allow/Deny result. A Nofax timeout or transport failure emits valid no-decision JSON and leaves Gemini CLI's own policy/confirmation flow in control rather than converting failure into approval. The `Notification` hook remains advisory and is forwarded only as a phone notification.
+
+Adjust the matcher to the tools you want Nofax to gate. Keep the hook timeout longer than Nofax's configured approval timeout (`timeoutSeconds`, 300 seconds by default).
+
+## Optional remote Cloudflare Worker
+
+The `worker/` package provides a private, self-deployed MCP endpoint:
+
+```text
+remote MCP client
+       |
+       | authenticated Streamable HTTP
+       v
+Cloudflare Worker
+       |
+       +--> nofax_notify ------> ntfy ------> phone
+       |
+       +--> SQLite Durable Object
+              |
+              +--> get request metadata
+              +--> list pending requests
+```
+
+It exposes exactly three tools:
+
+- `nofax_notify` — one-way notification only;
+- `nofax_get_request` — read one safe request projection;
+- `nofax_list_pending` — read unresolved, unexpired request projections.
+
+Deploy from `worker/`:
+
+```bash
+npm ci
+npx wrangler login
+npx wrangler secret put NOFAX_REMOTE_KEY
+npx wrangler secret put NTFY_TOPIC
+npm run check
+npm run deploy
+```
+
+Preferred MCP connection:
+
+```text
+https://<worker>.workers.dev/mcp
+Authorization: Bearer <NOFAX_REMOTE_KEY>
+```
+
+Clients that cannot attach a static authorization header can use the compatibility capability path:
+
+```text
+https://<worker>.workers.dev/mcp/<NOFAX_REMOTE_KEY>
+```
+
+Treat the complete capability URL like a password.
+
+See [`docs/remote-mcp.md`](docs/remote-mcp.md) for deployment, threat boundaries, and qualification details.
+
+### Important: public ntfy + serverless egress
+
+The default public `ntfy.sh` service applies publisher quotas. Serverless platforms such as Cloudflare Workers may use shared outbound IP space, so a Worker can receive an ntfy `42908` daily-quota response even when that individual Worker has sent very little traffic. That limit is imposed by ntfy, not by the Cloudflare Workers request quota.
+
+For reliability-sensitive deployments, use a notification provider whose quota is tied to your own authenticated account/identity, or operate a trusted self-hosted transport. Do not build a critical workflow around anonymous public-topic quota assumptions.
+
+## Security model
+
+Nofax is a transport and human-interaction component, **not an authorization policy engine**.
+
+Local mode:
+
+- pending, timeout, disconnect, malformed state, and network failure never mean approval;
+- the first valid terminal response wins;
+- notification topics and one-time response topics are capabilities;
+- public ntfy is not end-to-end encrypted from the provider;
+- redaction is best-effort and cannot reliably identify secrets embedded in arbitrary free-form text.
+
+Remote mode:
+
+- only explicit `nofax_notify` performs an external messaging side effect;
+- request-inspection operations are read-only and do not perform hidden cleanup writes;
+- remote approval, callback, webhook, refinement, choice, and wait surfaces are absent;
+- `NOFAX_REMOTE_KEY` is a bearer credential;
+- remote projections omit callback capabilities, prompt/message text, and internal allowed-decision lists.
+
+Read [`SECURITY.md`](SECURITY.md) before using Nofax with sensitive information.
+
+## Configuration
+
+Default local config lives at `~/.nofax/config.json`:
+
+```json
+{
+  "version": 1,
+  "server": "https://ntfy.sh",
+  "topic": "nofax_<random>",
+  "timeoutSeconds": 300
+}
+```
+
+Override the home directory with `NOFAX_HOME`:
+
+```bash
+NOFAX_HOME=/path/to/nofax-home nofax config
+```
+
+Use another ntfy-compatible server with:
+
+```bash
+nofax init --server https://ntfy.example.com --force
+```
+
+## Development
+
+Local package:
+
+```bash
+npm ci
+npm run check
+npm test
+npm pack --dry-run
+```
+
+Remote Worker:
+
+```bash
+cd worker
+npm ci
+npm run check
+```
+
+CI qualifies Node.js 20, 22, and 24 for the local package. The Worker gate runs TypeScript, Vitest, a production-dependency audit, and a Wrangler deployment dry-run.
+
+## Project docs
+
+- [`docs/architecture.md`](docs/architecture.md) — trust boundaries and data flow
+- [`docs/remote-mcp.md`](docs/remote-mcp.md) — remote Worker deployment and qualification
+- [`SECURITY.md`](SECURITY.md) — security assumptions and vulnerability reporting
+- [`CONTRIBUTING.md`](CONTRIBUTING.md) — contribution and test expectations
+- [`CHANGELOG.md`](CHANGELOG.md) — release history
+
+## Non-goals
+
+Nofax deliberately does not provide:
+
+- a Nofax-operated approval SaaS;
+- a paid model API dependency;
+- persistent `always approve` policy;
+- an arbitrary remote shell endpoint;
+- a public multi-user Worker behind one shared deployment key;
+- a claim that MCP annotations themselves are a security boundary.
+
+## License
+
+MIT © Tomi Šeregi. See [`LICENSE`](LICENSE).

package/SECURITY.md

@@ -0,0 +1,144 @@
+# Security Policy
+
+## Supported versions
+
+Nofax is pre-1.0 software. Security fixes are applied to the latest release on the default branch.
+
+## Reporting a vulnerability
+
+Do not publish credentials, exploit details, private ntfy topics, one-time response URLs, remote MCP capability URLs, or sensitive hook/MCP payloads in a public issue.
+
+If GitHub private vulnerability reporting is enabled for this repository, use it. Otherwise, open a minimal public issue asking for a private maintainer contact channel without including sensitive details.
+
+## Local Nofax security
+
+### Public ntfy is not end-to-end encryption
+
+With the default `https://ntfy.sh` configuration, notification content transits and may be cached by the ntfy service. A random topic name reduces unauthorized discovery but does not encrypt content from the service operator.
+
+Use a trusted authenticated/self-hosted ntfy server for sensitive source code, production operations, credentials, regulated data, or confidential prompts.
+
+### Topics and one-time response URLs are capabilities
+
+On anonymous ntfy servers, knowledge of a topic can be sufficient to subscribe or publish. Treat the local ntfy topic as a bearer secret.
+
+Local Nofax generates fresh one-time response topics for interactive requests. Do not log, share, bookmark, or persist those callback URLs outside the state Nofax itself requires.
+
+### Durable local request state
+
+Local MCP human-response requests are stored under `~/.nofax/requests/` so an MCP/client restart does not erase an unresolved human decision gate.
+
+Nofax writes these files with user-only permissions where supported. MCP-facing request projections deliberately omit the secret response topic. Protect the Nofax home directory like other local application state.
+
+### Pending never means approved
+
+A local interactive request remains pending until a matching terminal response is accepted. Passage of time, timeout, tool failure, network failure, or client disconnect never means approval.
+
+`nofax_wait_for_response` uses bounded long-polls. If it returns pending, the caller must wait again rather than continue the guarded action.
+
+### First terminal response wins
+
+Local durable requests are single-use. Once a valid terminal response is accepted, later responses cannot intentionally replace it.
+
+### Native hook failures never become approval
+
+The Claude Code and Codex adapters fail closed. If the local transport times out, returns malformed data, or encounters a network/polling error, Nofax does not synthesize an Allow decision.
+
+### Redaction is best-effort
+
+Nofax redacts values under common secret-bearing object keys and bounds serialized payloads. It cannot reliably detect a credential embedded in arbitrary free-form command text. Treat all notification content accordingly.
+
+## Remote Worker security
+
+The optional Cloudflare Worker in v0.3 is a **one-way notification plus inspection remote MCP endpoint**.
+
+### Remote side effects are structurally bounded
+
+MCP tool annotations are descriptive metadata, not the security boundary.
+
+The remote Worker bounds side effects in code:
+
+- only `nofax_notify`, `nofax_get_request`, and `nofax_list_pending` are registered as MCP tools;
+- the remote handler object exposes one-way notification plus two read operations;
+- there is no remote approval, choice, refinement, wait, callback, or webhook handler;
+- public Worker routing is limited to health and authenticated MCP paths;
+- `/telegram/webhook` and `/r/*` are not routes and return 404;
+- pending-list reads filter expired rows without deleting them or performing hidden cleanup writes.
+
+The two inspection tools are annotated with `readOnlyHint: true`, `destructiveHint: false`, `idempotentHint: true`, and `openWorldHint: false`. `nofax_notify` is accurately marked side-effecting/non-idempotent/open-world and non-destructive.
+
+### Remote response minimization
+
+Remote MCP responses deliberately omit capability-bearing or unnecessary fields. Public request projections do not expose:
+
+- callback hashes;
+- callback tokens/URLs;
+- original request title;
+- original request message/prompt material;
+- internal allowed-decision lists.
+
+Terminal state may include the stored terminal decision and historical refinement text when present in existing durable state.
+
+### Remote MCP key is a bearer secret
+
+The Worker is a private single-user deployment protected by `NOFAX_REMOTE_KEY`.
+
+Preferred clients send:
+
+```text
+Authorization: Bearer <NOFAX_REMOTE_KEY>
+```
+
+Clients that cannot attach a static header may use:
+
+```text
+/mcp/<NOFAX_REMOTE_KEY>
+```
+
+The complete capability URL is equivalent to a password and may leak through browser history, screenshots, copied configuration, proxies, or third-party logging. Prefer the Authorization header whenever the MCP host supports it.
+
+Nofax uses equal-length constant-time credential comparison. After capability-path authentication, the Worker normalizes the request internally to `/mcp` before MCP protocol handling.
+
+Rotate `NOFAX_REMOTE_KEY` immediately if it is exposed.
+
+A single deployment-wide key is not sufficient for a shared/public multi-user service. Use a delegated authentication and authorization design before operating a multi-user deployment.
+
+### Durable Object compatibility
+
+The v0.3 Worker preserves the existing SQLite request schema so upgrading from experimental pre-read-only Worker builds does not require destructive storage migration.
+
+Only request-inspection methods are reachable for Durable Object state. Existing legacy rows may therefore be inspected after upgrade, but the Worker cannot create, resolve, or delete them through MCP or HTTP routes. The separate `nofax_notify` method can publish an explicitly requested one-way notification and does not mutate request state.
+
+### Cloudflare is the remote trust boundary
+
+Remote mode adds Cloudflare as an infrastructure boundary. For notification calls, ntfy is an additional provider boundary and receives the bounded notification title/message plus the configured topic identifier.
+
+The Worker does not automatically forward durable request records to ntfy. Only explicit `nofax_notify` content is published; Telegram, WhatsApp, SMS, and human-response callbacks remain absent remotely.
+
+Do not expose the deployment key in source, Wrangler vars, `.env`, `.dev.vars`, CI logs, PR text, screenshots, or issue reports. Use Wrangler secrets for production values and keep local secret files untracked.
+
+### Hosted ntfy quotas are an external availability boundary
+
+The public `ntfy.sh` service applies its own publisher limits independently of Cloudflare Workers quotas. Serverless egress may be shared between unrelated workloads, so a Cloudflare Worker can receive an ntfy `42908` daily-quota response even when that individual Worker has published little traffic.
+
+Treat public ntfy availability and quota policy as an external dependency. Reliability-sensitive deployments should use a transport with account-scoped quota/identity or a trusted self-hosted service rather than assuming anonymous public-topic capacity.
+
+## Nofax is not a policy engine
+
+Local Nofax answers approval requests that an upstream agent or workflow explicitly delegates to it. It does not decide which operations should require approval and must not be used to bypass an agent's deny rules, sandbox, or existing authorization boundaries.
+
+An `allow` result permits the caller to continue only within authority it already possessed.
+
+The remote Worker does not grant authority. Its only mutation is sending an informational notification; request-state operations remain inspection-only.
+
+## MCP transport notes
+
+### Local stdio
+
+`nofax mcp` is a local stdio server. Stdout is reserved for MCP protocol traffic. Do not wrap it with tooling that injects banners or diagnostics into stdout.
+
+### Remote Streamable HTTP
+
+The optional Worker uses authenticated Streamable HTTP for MCP. The only unauthenticated functional route is the secret-free `/healthz` health check.
+
+See [`docs/remote-mcp.md`](docs/remote-mcp.md) for the remote architecture and qualification checklist.

package/bin/nofax.mjs

@@ -0,0 +1,4 @@
+#!/usr/bin/env node
+import { runCli } from '../src/cli.mjs';
+
+process.exitCode = await runCli(process.argv.slice(2));

package/package.json

@@ -0,0 +1,56 @@
+{
+  "name": "nofax",
+  "version": "0.2.0",
+  "mcpName": "io.github.AKzar1el/nofax",
+  "description": "Human-in-the-loop approvals and notifications for AI coding agents via CLI, MCP, and agent hooks.",
+  "type": "module",
+  "bin": {
+    "nofax": "./bin/nofax.mjs"
+  },
+  "exports": {
+    ".": "./src/index.mjs",
+    "./mcp": "./src/mcp-server.mjs"
+  },
+  "files": [
+    "bin/",
+    "src/",
+    "README.md",
+    "LICENSE",
+    "SECURITY.md"
+  ],
+  "scripts": {
+    "test": "node --test test/*.test.mjs",
+    "check": "node --check bin/nofax.mjs && node --check src/cli.mjs && node --check src/config.mjs && node --check src/protocol.mjs && node --check src/ntfy.mjs && node --check src/requests.mjs && node --check src/mcp-tools.mjs && node --check src/mcp-server.mjs && node --check src/adapters/claude.mjs && node --check src/adapters/codex.mjs && node --check src/adapters/gemini.mjs"
+  },
+  "engines": {
+    "node": ">=20.0.0"
+  },
+  "dependencies": {
+    "@modelcontextprotocol/server": "2.0.0",
+    "zod": "4.4.3"
+  },
+  "keywords": [
+    "ai-agent",
+    "approval",
+    "claude-code",
+    "codex",
+    "gemini-cli",
+    "human-in-the-loop",
+    "mcp",
+    "model-context-protocol",
+    "ntfy",
+    "notifications",
+    "remote",
+    "developer-tools"
+  ],
+  "license": "MIT",
+  "author": "Tomi Šeregi",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/AKzar1el/nofax.git"
+  },
+  "homepage": "https://github.com/AKzar1el/nofax#readme",
+  "bugs": {
+    "url": "https://github.com/AKzar1el/nofax/issues"
+  }
+}

package/src/adapters/claude.mjs

@@ -0,0 +1,47 @@
+import { buildAgentSummary } from '../protocol.mjs';
+import { requestApproval } from '../ntfy.mjs';
+
+function assertClaudePermissionRequest(input) {
+  if (!input || input.hook_event_name !== 'PermissionRequest') throw new Error('NOFAX_CLAUDE_EVENT');
+  if (typeof input.tool_name !== 'string' || !input.tool_name) throw new Error('NOFAX_CLAUDE_TOOL');
+}
+
+export async function handleClaudePermissionRequest(input, {
+  config,
+  requestApprovalImpl = requestApproval,
+  onError = () => {}
+} = {}) {
+  assertClaudePermissionRequest(input);
+  try {
+    const result = await requestApprovalImpl({
+      config,
+      title: `Claude Code needs approval: ${input.tool_name}`,
+      message: buildAgentSummary({
+        source: 'Claude Code',
+        toolName: input.tool_name,
+        cwd: input.cwd,
+        toolInput: input.tool_input
+      })
+    });
+    if (result.decision === 'allow') {
+      return {
+        hookSpecificOutput: {
+          hookEventName: 'PermissionRequest',
+          decision: { behavior: 'allow' }
+        }
+      };
+    }
+    if (result.decision === 'deny') {
+      return {
+        hookSpecificOutput: {
+          hookEventName: 'PermissionRequest',
+          decision: { behavior: 'deny', message: 'Denied remotely via Nofax.' }
+        }
+      };
+    }
+    return null;
+  } catch (error) {
+    onError(error);
+    return null;
+  }
+}

package/src/adapters/codex.mjs

@@ -0,0 +1,47 @@
+import { buildAgentSummary } from '../protocol.mjs';
+import { requestApproval } from '../ntfy.mjs';
+
+function assertCodexPermissionRequest(input) {
+  if (!input || input.hook_event_name !== 'PermissionRequest') throw new Error('NOFAX_CODEX_EVENT');
+  if (typeof input.tool_name !== 'string' || !input.tool_name) throw new Error('NOFAX_CODEX_TOOL');
+}
+
+export async function handleCodexPermissionRequest(input, {
+  config,
+  requestApprovalImpl = requestApproval,
+  onError = () => {}
+} = {}) {
+  assertCodexPermissionRequest(input);
+  try {
+    const result = await requestApprovalImpl({
+      config,
+      title: `Codex needs approval: ${input.tool_name}`,
+      message: buildAgentSummary({
+        source: 'Codex',
+        toolName: input.tool_name,
+        cwd: input.cwd,
+        toolInput: input.tool_input
+      })
+    });
+    if (result.decision === 'allow') {
+      return {
+        hookSpecificOutput: {
+          hookEventName: 'PermissionRequest',
+          decision: { behavior: 'allow' }
+        }
+      };
+    }
+    if (result.decision === 'deny') {
+      return {
+        hookSpecificOutput: {
+          hookEventName: 'PermissionRequest',
+          decision: { behavior: 'deny', message: 'Denied remotely via Nofax.' }
+        }
+      };
+    }
+    return null;
+  } catch (error) {
+    onError(error);
+    return null;
+  }
+}