nofax 0.2.1 → 0.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nofax",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "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",

package/src/cli.mjs

@@ -1,10 +1,21 @@
+import { readFile } from 'node:fs/promises';
 import { initConfig, loadConfig } from './config.mjs';
 import { requestApproval, requestRefinement, sendNotification } from './ntfy.mjs';
 import { handleClaudePermissionRequest } from './adapters/claude.mjs';
 import { handleCodexPermissionRequest } from './adapters/codex.mjs';
 import { handleGeminiHook } from './adapters/gemini.mjs';
 
-const HELP = `nofax - remote approvals and notifications for coding agents\n\nUsage:\n  nofax init [--server URL] [--topic TOPIC] [--timeout SECONDS] [--force]\n  nofax test\n  nofax notify [--title TITLE] MESSAGE...\n  nofax approve [--title TITLE] MESSAGE...\n  nofax refine [--title TITLE] MESSAGE...\n  nofax mcp\n  nofax config\n  nofax hook claude\n  nofax hook codex\n  nofax hook gemini\n\nEnvironment:\n  NOFAX_HOME   Override ~/.nofax\n`;
+const HELP = `nofax - remote approvals and notifications for coding agents\n\nUsage:\n  nofax --version\n  nofax init [--server URL] [--topic TOPIC] [--timeout SECONDS] [--force]\n  nofax test\n  nofax notify [--title TITLE] MESSAGE...\n  nofax approve [--title TITLE] MESSAGE...\n  nofax refine [--title TITLE] MESSAGE...\n  nofax mcp\n  nofax config\n  nofax hook claude\n  nofax hook codex\n  nofax hook gemini\n\nEnvironment:\n  NOFAX_HOME   Override ~/.nofax\n`;
+
+const PACKAGE_JSON_URL = new URL('../package.json', import.meta.url);
+
+async function readPackageVersion() {
+  const metadata = JSON.parse(await readFile(PACKAGE_JSON_URL, 'utf8'));
+  if (typeof metadata.version !== 'string' || metadata.version.length === 0) {
+    throw new Error('NOFAX_PACKAGE_VERSION_INVALID');
+  }
+  return metadata.version;
+}
 
 function parseArgs(args) {
   const flags = {};
@@ -96,6 +107,11 @@ export async function runCli(args, overrides = {}) {
       return 0;
     }
 
+    if (args[0] === '--version' || args[0] === '-v') {
+      deps.stdout.write(`${await readPackageVersion()}\n`);
+      return 0;
+    }
+
     const command = args[0];
     if (command === 'init') {
       const { flags } = parseArgs(args.slice(1));

package/src/mcp-server.mjs

@@ -32,7 +32,7 @@ function result(value) {
 
 export function buildMcpServer({ handlers = createMcpToolHandlers() } = {}) {
   const server = new McpServer(
-    { name: 'nofax', version: '0.2.1' },
+    { name: 'nofax', version: '0.2.3' },
     { instructions: SERVER_INSTRUCTIONS }
   );
 

package/src/requests.mjs

@@ -1,4 +1,4 @@
-import { chmod, mkdir, readFile, readdir, rename, writeFile } from 'node:fs/promises';
+import { chmod, link, mkdir, readFile, readdir, rename, unlink, writeFile } from 'node:fs/promises';
 import { join } from 'node:path';
 import { resolveNofaxHome } from './config.mjs';
 
@@ -47,7 +47,12 @@ function validateRequest(input) {
 function paths({ home, env, requestId }) {
   const root = resolveNofaxHome({ home, env });
   const requestsRoot = join(root, 'requests');
-  return { requestsRoot, file: join(requestsRoot, `${validateRequestId(requestId)}.json`) };
+  const normalizedRequestId = validateRequestId(requestId);
+  return {
+    requestsRoot,
+    file: join(requestsRoot, `${normalizedRequestId}.json`),
+    terminal: join(requestsRoot, `${normalizedRequestId}.terminal.json`)
+  };
 }
 
 async function atomicWrite(path, value) {
@@ -58,6 +63,35 @@ async function atomicWrite(path, value) {
   try { await chmod(path, 0o600); } catch {}
 }
 
+async function loadTerminalClaim(path) {
+  try {
+    const terminal = validateRequest(JSON.parse(await readFile(path, 'utf8')));
+    if (terminal.status !== 'resolved') throw new Error('NOFAX_REQUEST_TERMINAL_INVALID');
+    return terminal;
+  } catch (error) {
+    if (error?.code === 'ENOENT') return null;
+    if (error instanceof SyntaxError) throw new Error('NOFAX_REQUEST_TERMINAL_INVALID_JSON');
+    throw error;
+  }
+}
+
+async function claimTerminal(path, value) {
+  const temp = `${path}.tmp-${process.pid}-${Math.random().toString(16).slice(2)}`;
+  await writeFile(temp, `${JSON.stringify(value, null, 2)}\n`, { encoding: 'utf8', mode: 0o600 });
+  try { await chmod(temp, 0o600); } catch {}
+
+  try {
+    await link(temp, path);
+    try { await chmod(path, 0o600); } catch {}
+    return true;
+  } catch (error) {
+    if (error?.code === 'EEXIST') return false;
+    throw error;
+  } finally {
+    try { await unlink(temp); } catch {}
+  }
+}
+
 export async function savePendingRequest({ home, env, request }) {
   const normalized = validateRequest(request);
   if (normalized.status !== 'pending') throw new Error('NOFAX_REQUEST_NOT_PENDING');
@@ -68,7 +102,9 @@ export async function savePendingRequest({ home, env, request }) {
 }
 
 export async function loadRequest({ home, env, requestId }) {
-  const { file } = paths({ home, env, requestId });
+  const { file, terminal } = paths({ home, env, requestId });
+  const claimed = await loadTerminalClaim(terminal);
+  if (claimed !== null) return claimed;
   try {
     return validateRequest(JSON.parse(await readFile(file, 'utf8')));
   } catch (error) {
@@ -89,8 +125,10 @@ export async function resolveRequest({ home, env, requestId, response, resolvedA
     decision: response.decision,
     ...(response.text === undefined ? {} : { text: response.text })
   });
-  const { file } = paths({ home, env, requestId });
-  await atomicWrite(file, resolved);
+  const { terminal } = paths({ home, env, requestId });
+  if (!await claimTerminal(terminal, resolved)) {
+    return loadRequest({ home, env, requestId });
+  }
   return resolved;
 }