open-claude-p 1.0.0

package/package.json

@@ -0,0 +1,68 @@
+{
+  "name": "open-claude-p",
+  "version": "1.0.0",
+  "description": "A PTY-backed compatibility shim for `claude -p` (Claude Code headless/print mode). Drives the interactive `claude` CLI via node-pty and exposes the same option surface and stream-json contract. Ships as both a library and an `ocp` CLI binary.",
+  "type": "module",
+  "main": "./src/index.js",
+  "exports": {
+    ".": "./src/index.js",
+    "./chat": "./src/chat/index.js",
+    "./options": "./src/options/index.js",
+    "./parsers": "./src/parsers/index.js",
+    "./output": "./src/output/index.js",
+    "./pty": "./src/pty/index.js"
+  },
+  "bin": {
+    "ocp": "bin/cli.js"
+  },
+  "files": [
+    "src",
+    "bin",
+    "scripts/postinstall.js",
+    "README.md",
+    "README.ko.md",
+    "README.ja.md",
+    "README.zh.md",
+    "LICENSE"
+  ],
+  "scripts": {
+    "test": "node --test test/**/*.test.js",
+    "postinstall": "node scripts/postinstall.js",
+    "prepublishOnly": "npm test",
+    "pack:check": "npm pack --dry-run"
+  },
+  "keywords": [
+    "claude",
+    "claude-code",
+    "claude-p",
+    "pty",
+    "node-pty",
+    "headless",
+    "cli",
+    "compatibility-shim"
+  ],
+  "author": "Lee Yongbeom <top6616@gmail.com>",
+  "license": "MIT",
+  "homepage": "https://github.com/empty-user77/open-claude-p#readme",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/empty-user77/open-claude-p.git"
+  },
+  "bugs": {
+    "url": "https://github.com/empty-user77/open-claude-p/issues"
+  },
+  "funding": {
+    "type": "github",
+    "url": "https://github.com/sponsors/empty-user77"
+  },
+  "publishConfig": {
+    "access": "public"
+  },
+  "dependencies": {
+    "node-pty": "^1.1.0",
+    "proper-lockfile": "^4.1.2"
+  },
+  "engines": {
+    "node": ">=18.0.0"
+  }
+}

package/scripts/postinstall.js

@@ -0,0 +1,60 @@
+#!/usr/bin/env node
+//
+// Postinstall fixup for node-pty prebuilds.
+//
+// npm sometimes drops the execute bit on prebuilt helper binaries during tar
+// extraction. node-pty ships a `spawn-helper` per (platform, arch) that MUST
+// be executable; without the +x bit every `posix_spawnp` call fails on macOS
+// with the unhelpful message "posix_spawnp failed.".
+//
+// This script restores the execute bit for whichever prebuild matches the
+// current platform/arch. It is a no-op everywhere else.
+//
+// Safe to run repeatedly; safe to fail (e.g. when node-pty is not installed
+// yet in some lifecycle ordering).
+
+import { chmod, lstat } from 'node:fs/promises';
+import { createRequire } from 'node:module';
+import path from 'node:path';
+import process from 'node:process';
+
+// Resolve via `require.resolve` against THIS script's location, not
+// process.cwd(). The npm-install case happens to use cwd = package
+// root and the relative path worked accidentally, but manual
+// invocation from another cwd would have chmod'd the wrong tree (or
+// nothing). With require.resolve the path is always the node-pty
+// actually loaded by our package.
+let nodePtyRoot;
+try {
+  const req = createRequire(import.meta.url);
+  nodePtyRoot = path.dirname(req.resolve('node-pty/package.json'));
+} catch {
+  // node-pty not installed yet (lifecycle ordering) — exit silently.
+  process.exit(0);
+}
+
+const targets = [
+  path.join(nodePtyRoot, 'prebuilds', `${process.platform}-${process.arch}`, 'spawn-helper'),
+];
+
+let fixedAny = false;
+for (const full of targets) {
+  try {
+    const st = await lstat(full);
+    // Refuse to follow symlinks: a malicious package preinstall hook could
+    // replace the prebuild path with a symlink to a sensitive binary and
+    // gain mode 0755 on it. lstat + bail keeps us on a regular file.
+    if (st.isSymbolicLink() || !st.isFile()) continue;
+    await chmod(full, 0o755);
+    process.stdout.write(`postinstall: chmod 0755 ${path.relative(process.cwd(), full) || full}\n`);
+    fixedAny = true;
+  } catch {
+    // file not present for this (platform, arch) — fine, skip silently
+  }
+}
+
+if (!fixedAny) {
+  // Quiet success: in some environments (Windows, or pruned installs) none of
+  // the targets exist; that is not an error.
+  process.exit(0);
+}

package/src/chat/event-filters.js

@@ -0,0 +1,116 @@
+// Shared event-level cleanup helpers used by the CLI text adapter, the
+// chat SDK, and the bundled sample server. Centralising them here keeps
+// the "what counts as TUI noise vs. real content" decision in one
+// place — when claude's TUI changes shape, a single edit propagates
+// everywhere.
+//
+// Imported by:
+//   - src/output/text.js  (CLI live spinner)
+//   - sample/server.js    (SSE chat stream)
+//   - chat.send()         (when `filterNoise: true`)
+
+// Forward-declared so cleanSpinnerLabel can use it before the function
+// definition below — keeps the export ordering readable.
+const SPINNER_ICON_PREFIX = /^[✻✶✺✹✸✷✵●✢✳✽·✾✿❀❁❂❃❄❅❆❇❈❉❊❋]\s*/;
+
+/**
+ * Convert an upstream `spinner` event label into a user-facing line of
+ * progress text. Returns `null` if the label is pure noise (token
+ * counters, lone digits, empty after trim, etc.).
+ *
+ * Examples:
+ *   "✻ Brewing… (2s · 47 tokens)"   → "Brewing…"
+ *   "✻ Deliberating…   10s 23 still"  → "Deliberating…"
+ *   "✻ thinking"                    → null
+ *   "✻"                             → null
+ *
+ * @param {string} label
+ * @returns {string|null}
+ */
+export function cleanSpinnerLabel(label) {
+  let s = String(label).replace(SPINNER_ICON_PREFIX, '').trim();
+  s = s
+    .replace(/\s{2,}\d.*$/, '')      // "  10s ..." time/token stats
+    .replace(/\s{3,}.*$/, '')        // TUI suffix after 3+ spaces
+    .replace(/\s+[·↓↑].*$/, '')      // " · N tokens …" separator
+    .replace(/\s*\(.*$/, '')         // "(stats)" suffix
+    .replace(/[\s\d·↓↑]+$/, '')      // trailing junk
+    .trim();
+  if (!s) return null;
+  if (/^(thinking|still|tokens?)$/i.test(s)) return null;
+  if (/tokens[\)·]/.test(s) || /^\d/.test(s)) return null;
+  if (!/\S{3,}/.test(s)) return null;
+  // Final defensive strip — claude's spinner text reaches the user's
+  // terminal as live progress; an exotic upstream that smuggled an
+  // ANSI sequence into the label would otherwise execute it.
+  return stripTerminalControl(s, 128);
+}
+
+/**
+ * `true` if an `assistant-text` event payload is TUI chrome rather than
+ * model output — TUI prompt characters, box borders, redraw artifacts,
+ * tree-view glyphs, "Found N" search status lines, status indicators,
+ * and tool-call announcements that other systems should not surface as
+ * assistant content.
+ *
+ * @param {string} text
+ * @returns {boolean}
+ */
+export function isAssistantTextNoise(text) {
+  const t = String(text ?? '');
+  if (!t) return true;
+  if (/^\s*[❯›❮‹✳✽·✾✿❀❁❂❃❄]\s*$/.test(t)) return true;       // lone TUI glyph
+  if (/^[\s\d\r\n]+$/.test(t)) return true;                  // whitespace / digits only
+  if (/⎿/.test(t)) return true;                              // tree-view marker
+  if (/MCP servers? failed/i.test(t)) return true;           // MCP status line
+  if (/^\s*(↑|↓|·|still running)\s*$/i.test(t)) return true; // arrow / indicator
+  if (/^(Found \d+|Did \d+)\b/.test(t.trim())) return true;  // search status
+  if (/^\s*(?:Web ?Search|WebFetch|Bash|Read|Write|Glob|Grep|LS|Edit|MultiEdit|TodoWrite|TodoRead|mcp__\w+)\(/.test(t)) return true; // tool-call announcement
+  // "Inferring…          8          word" — spinner + digit + word, pure animation frame
+  if (/\w(?:…|\.{2,3})\s{3,}\S.*$/.test(t)) return true;
+  // Heavy leading indent + ≤2 words — usually a single-line spinner element
+  { const tr = t.trim(); if ((t.length - tr.length) > 8 && tr.split(/\s+/).length <= 2) return true; }
+  // "Nucleating... 4 3 Nucleating... 5" — strip CapWord… tokens; if only digits/spaces remain it's pure spinner noise
+  if (/[A-Z][a-zA-Z]+[\.…]{2,3}/.test(t)) {
+    const rest = t.replace(/[A-Z][a-zA-Z\-]+[\.…]{2,3}/g, '').replace(/\s/g, '');
+    if (/^\d*$/.test(rest)) return true;
+  }
+  return false;
+}
+
+/**
+ * Strip control / escape characters that would execute as terminal
+ * sequences when echoed to a TTY. Use on any string sourced from
+ * untrusted-by-the-terminal channels (upstream JSONL `tool_use.name`,
+ * spinner labels, user prompt previews) before writing to stderr.
+ * Caps the length to keep meta-line / status-line wraps sane.
+ *
+ * @param {string} s
+ * @param {number} [max=64]
+ * @returns {string}
+ */
+export function stripTerminalControl(s, max = 64) {
+  // Strip C0 controls + DEL + the full C1 8-bit control range. The
+  // narrower `\x9b` (CSI) class missed `\x90` (DCS), `\x9d` (OSC),
+  // `\x9e` (PM), `\x9f` (APC) — all of which execute on VT100-compatible
+  // terminals when 8-bit forms are accepted. Also strips ESC (`\x1b`)
+  // which is the 7-bit prefix for the same sequences.
+  return String(s ?? '')
+    .replace(/[\x00-\x1f\x7f\x80-\x9f]/g, '')
+    .slice(0, max);
+}
+
+/** Tool-call announcement pattern → first capture is the tool name. */
+const TOOL_CALL_RE = /^\s*(Web Search|WebSearch|WebFetch|Bash|Read|Write|Glob|Grep|LS|Edit|MultiEdit|TodoWrite|TodoRead|mcp__[\w]+)\(/;
+
+/**
+ * Extract the tool name from an `assistant-text` line that announces a
+ * tool call. Returns `null` if the line is not a tool announcement.
+ *
+ * @param {string} text
+ * @returns {string|null}
+ */
+export function extractToolName(text) {
+  const m = String(text ?? '').match(TOOL_CALL_RE);
+  return m ? m[1] : null;
+}