@7n/rules 1.6.0 → 1.7.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [1.7.0] - 2026-07-16
+
+### Changed
+
+- `skill cursor/codex` тепер виконуються через `@7n/llm-lib/acp` (napi-міст до Rust `llm_cascade::acp`) замість власного JS ACP-клієнта; deprecated `skill claude` лишається окремим JS-шимом (`acp-runner.mjs`), бо Rust-крейт `claude` не моделює
+
 ## [1.6.0] - 2026-07-16
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@7n/rules",
-  "version": "1.6.0",
+  "version": "1.7.0",
   "description": "CLI еталонних правил і skills (префікс n-): синк у репозиторій, дельта-lint, конформність",
   "keywords": [
     "cli",
@@ -85,7 +85,6 @@
   },
   "optionalDependencies": {
     "@agentclientprotocol/claude-agent-acp": "^0.59.0",
-    "@agentclientprotocol/codex-acp": "^1.1.2",
     "@earendil-works/pi-ai": "0.80.2",
     "@earendil-works/pi-coding-agent": "0.80.2"
   },

package/scripts/docs/skills-cli.md

@@ -3,7 +3,7 @@ type: JS Module
 title: skills-cli.mjs
 resource: npm/scripts/skills-cli.mjs
 docgen:
-  crc: ce3d7d60
+  crc: 1a8f83e4
   model: omlx/gemma-4-e4b-it-OptiQ-4bit
 ---
 
@@ -30,7 +30,7 @@ runSkillsCli виконує логіку командного інтерфейс
 
 ## Гарантії поведінки
 
-- Сам модуль лише читає файли й збирає промпт; **виконання** делегується агенту: `pi` (вбудований, мутує дерево, запускає bash) або зовнішньому ACP-агенту `cursor`/`codex`/`claude` через `runAcpRunner` (`./lib/acp-runner.mjs`) — JSON-RPC поверх stdio (`cursor-agent acp`, вбудовані адаптери `@agentclientprotocol/codex-acp` і `@agentclientprotocol/claude-agent-acp`), а не сирий `stdin`/`stdout`-pipe-передавання.
+- Сам модуль лише читає файли й збирає промпт; **виконання** делегується агенту: `pi` (вбудований, мутує дерево, запускає bash) або зовнішньому ACP-агенту `cursor`/`codex`/`claude` через `runAcpRunner` (`./lib/acp-runner.mjs`) — JSON-RPC поверх stdio (`cursor-agent acp`, бандловані адаптери `@agentclientprotocol/codex-acp` і `@agentclientprotocol/claude-agent-acp`), а не сирий `stdin`/`stdout`-піпінг.
 - Тира моделі для `pi`-runner береться з `main.json.tier` скіла (дефолт `max`).
-- ACP-раннер автоматично схвалює `session/request_permission` (без інтерактивних питань) — паритет із колишнім non-interactive режимом; `Client`-реалізація читає/пише файли напряму через `node:fs`.
+- ACP-раннер автоапрувляє `session/request_permission` (без інтерактивних питань) — паритет із колишнім non-interactive режимом; `Client`-реалізація читає/пише файли напряму через `node:fs`.
 - Лише `claude` — deprecated: друкує попередження й лишається як fallback, доки не налаштовано pi-модель. `cursor`/`codex` — повноцінні раннери без попередження.

package/scripts/lib/acp-runner.mjs

@@ -1,18 +1,18 @@
 /**
- * Виконання скіла через зовнішнього ACP-агента (Agent Client Protocol,
- * agentclientprotocol.com) — JSON-RPC поверх stdio замість сирого
- * `stdin`/`stdout`-передавання через pipe CLI.
+ * Deprecated JS ACP-шим для `skill claude` (Agent Client Protocol,
+ * agentclientprotocol.com) — JSON-RPC поверх stdio.
  *
- * Заміняє колишній `runLlmCli` (spawnSync + `-p`/`exec -`) для `cursor`/`codex`/`claude`:
- * підʼєднання йде через офіційний TS SDK `@zed-industries/agent-client-protocol`,
- * а дозволи на tool-calls (`session/request_permission`) автоматично схвалюються без участі
- * людини — паритет із сьогоднішнім non-interactive `-p`-режимом (full user-trust,
- * без write-guard, як задокументовано в `@7n/llm-lib/agent-skill`).
+ * `cursor`/`codex` мігрували на `@7n/llm-lib/acp` (napi-міст до
+ * `llm_cascade::acp` — та сама протокольна логіка, але в Rust, без
+ * дублювання в JS). `claude` лишається тут як окремий JS-шим, бо Rust-крейт
+ * `AcpAgentKind` його не моделює (лише `Cursor`/`Codex`); підʼєднання йде
+ * через офіційний TS SDK `@zed-industries/agent-client-protocol`, а дозволи
+ * на tool-calls (`session/request_permission`) автоматично схвалюються без
+ * участі людини — паритет із колишнім non-interactive `-p`-режимом
+ * (full user-trust, без write-guard, як задокументовано в `@7n/llm-lib/agent-skill`).
  *
- * `cursor` — нативний ACP-режим самого `cursor-agent` (`cursor-agent acp`).
- * `codex`/`claude` — офіційні адаптери (`@agentclientprotocol/codex-acp`,
- * `@agentclientprotocol/claude-agent-acp`), що вбудовують свій рушій — зовнішній
- * бінарник `codex`/`claude` у PATH більше не потрібен.
+ * `claude` — офіційний адаптер (`@agentclientprotocol/claude-agent-acp`), що
+ * вбудовує свій рушій — зовнішній бінарник `claude` у PATH не потрібен.
  */
 
 import { spawn } from 'node:child_process'
@@ -25,13 +25,11 @@ import { Readable, Writable } from 'node:stream'
 const require = createRequire(import.meta.url)
 
 /**
- * Команда запуску ACP-агента на провайдер. `cursor` — зовнішній бінарник у PATH;
- * `codex`/`claude` — резолвляться з `bin`-запису вбудованого npm-пакета-адаптера.
- * @type {Record<'cursor' | 'codex' | 'claude', { bin: string, args: string[] } | { adapterPackage: string }>}
+ * Команда запуску ACP-агента на провайдер. Резолвиться з `bin`-запису
+ * вбудованого npm-пакета-адаптера.
+ * @type {Record<'claude', { adapterPackage: string }>}
  */
 export const ACP_AGENT_COMMANDS = {
-  cursor: { bin: 'cursor-agent', args: ['acp'] },
-  codex: { adapterPackage: '@agentclientprotocol/codex-acp' },
   claude: { adapterPackage: '@agentclientprotocol/claude-agent-acp' }
 }
 
@@ -46,7 +44,7 @@ export function stopReasonToExitCode(stopReason) {
 
 /**
  * Резолвить абсолютний шлях до bin-файлу вбудованого ACP-адаптера з його `package.json`.
- * @param {string} adapterPackage назва npm-пакета адаптера (`@agentclientprotocol/codex-acp`, …)
+ * @param {string} adapterPackage назва npm-пакета адаптера (`@agentclientprotocol/claude-agent-acp`)
  * @returns {string} абсолютний шлях до виконуваного файлу адаптера
  */
 export function resolveAdapterBin(adapterPackage) {
@@ -121,12 +119,13 @@ class AcpSkillClient {
 }
 
 /**
- * Виконує скіл через зовнішнього ACP-агента (`cursor`/`codex`/`claude`).
- * @param {'cursor' | 'codex' | 'claude'} kind провайдер
+ * Виконує скіл через зовнішнього ACP-агента. Єдиний лишений провайдер —
+ * deprecated `claude` (`cursor`/`codex` — див. `@7n/llm-lib/acp`).
+ * @param {'claude'} kind провайдер
  * @param {string} prompt готовий промпт скіла
  * @param {string} projectDir робочий каталог сесії агента
  * @param {(line: string) => void} logError вивід помилок
- * @param {{ acp?: object, spawnFn?: typeof spawn, out?: (chunk: string) => void, resolveAdapterBin?: (pkg: string) => string, isBinaryInPath?: (bin: string) => boolean }} [deps] інжекти для тестів
+ * @param {{ acp?: object, spawnFn?: typeof spawn, out?: (chunk: string) => void, resolveAdapterBin?: (pkg: string) => string }} [deps] інжекти для тестів
  * @returns {Promise<number>} exit code (0 — `end_turn`, 1 — інакше)
  */
 export async function runAcpRunner(kind, prompt, projectDir, logError, deps = {}) {
@@ -134,13 +133,9 @@ export async function runAcpRunner(kind, prompt, projectDir, logError, deps = {}
   const out = deps.out ?? (chunk => stdout.write(chunk))
   const resolveBin = deps.resolveAdapterBin ?? resolveAdapterBin
 
-  const command = ACP_AGENT_COMMANDS[kind]
-  const { bin, args } =
-    'adapterPackage' in command ? { bin: process.execPath, args: [resolveBin(command.adapterPackage)] } : command
-
-  if ('bin' in command && deps.isBinaryInPath && !deps.isBinaryInPath(command.bin)) {
-    throw new Error(`\`${command.bin}\` not found in PATH. Install ${kind} CLI (ACP mode) or use \`skill pi\`.`)
-  }
+  const { adapterPackage } = ACP_AGENT_COMMANDS[kind]
+  const bin = process.execPath
+  const args = [resolveBin(adapterPackage)]
 
   const acp = deps.acp ?? (await import('@zed-industries/agent-client-protocol'))
   const child = spawnFn(bin, args, { cwd: projectDir, stdio: ['pipe', 'pipe', 'inherit'], env: process.env })

package/scripts/skills-cli.mjs

@@ -4,7 +4,9 @@
  * Скіли читаються з `npm/skills/<id>/SKILL.md` установленого пакета (або кешу `npx`).
  * Промпт збирає інструкцію скілу + контекст поточного CWD (`package.json`, `tsconfig.json`,
  * `.n-rules.json`) — далі stdout або виконання через один з раннерів: вбудований
- * pi-агент, чи зовнішній ACP-агент (`cursor-agent acp`, `codex-acp`, або deprecated `claude`).
+ * pi-агент, чи зовнішній ACP-агент. `cursor`/`codex` — через `@7n/llm-lib/acp`
+ * (napi-міст до `llm_cascade::acp`, без власного JSON-RPC у JS); deprecated
+ * `claude` — окремий JS-шим (`./lib/acp-runner.mjs`), бо Rust-крейт його не моделює.
  *
  * Підтримувані формати:
  *   `npx \@7n/rules skill list`
@@ -12,14 +14,14 @@
  *   `npx \@7n/rules skill pi taze` — виконати через вбудований pi-агент (рекомендовано)
  *   `npx \@7n/rules skill pi taze "онови залежності"`
  *   `npx \@7n/rules skill cursor taze` — Cursor CLI через ACP (`cursor-agent acp`)
- *   `npx \@7n/rules skill codex taze` — Codex через ACP-адаптер (`@agentclientprotocol/codex-acp`)
+ *   `npx \@7n/rules skill codex taze` — Codex через ACP (napi-міст `@7n/llm-lib/acp`, Rust спавнить `npx \@agentclientprotocol/codex-acp`)
  *   `npx \@7n/rules skill claude taze` — deprecated: Claude Code через ACP-адаптер
  */
 
-import { spawnSync } from 'node:child_process'
+import { runAcpAgent } from '@7n/llm-lib/acp'
 import { existsSync, readFileSync, readdirSync } from 'node:fs'
 import { dirname, join } from 'node:path'
-import { cwd } from 'node:process'
+import { cwd, stdout } from 'node:process'
 import { fileURLToPath } from 'node:url'
 
 import { runAcpRunner } from './lib/acp-runner.mjs'
@@ -28,7 +30,11 @@ import { readSkillMetaRaw, skillTier } from './lib/skill-meta.mjs'
 /** Виконавці скіла. `pi` — вбудований (рекомендований); `cursor`/`codex`/`claude` — зовнішні ACP-агенти (`claude` — deprecated). */
 const RUNNERS = new Set(['pi', 'cursor', 'codex', 'claude'])
 
-/** Раннери, що йдуть через зовнішнього ACP-агента (`runAcpRunner`), і чи deprecated (друкує попередження перед запуском). */
+/**
+ * Раннери, що йдуть через зовнішнього ACP-агента, і чи deprecated (друкує попередження
+ * перед запуском). `cursor`/`codex` — napi-міст `@7n/llm-lib/acp`; `claude` — окремий
+ * JS-шим `runAcpRunner` (Rust-крейт `llm_cascade::acp` `claude` не моделює).
+ */
 const ACP_RUNNERS = {
   claude: { deprecated: true },
   cursor: { deprecated: false },
@@ -47,17 +53,6 @@ const USAGE_LINES = [
   'Skill id: каталог у пакеті (lint, taze, …) або з префіксом n- (n-lint → lint).'
 ]
 
-/**
- * @param {string} name ім'я бінарника
- * @returns {boolean} чи знайдено бінарник у PATH
- */
-function isBinaryInPath(name) {
-  // Явний `env: process.env` — Bun без нього дає дітям snapshot оточення зі старту процесу
-  // і не бачить runtime-змін `process.env.PATH` (та сама причина, що в resolve-cmd.mjs).
-  const probe = spawnSync('command', ['-v', name], { shell: true, encoding: 'utf8', env: process.env })
-  return probe.status === 0
-}
-
 /**
  * @param {string} name ім'я скілу з CLI або каталогу `.cursor/skills/n-*`
  * @returns {string} id каталогу в `npm/skills/<id>/`
@@ -170,27 +165,41 @@ async function runPiRunner(prompt, rawSkillName, skillsRoot, projectDir, logErro
 }
 
 /**
- * Делегує виконання скіла зовнішньому ACP-агенту (`cursor-agent acp`, `codex-acp`,
- * або deprecated `claude`-адаптер) через `runAcpRunner` — JSON-RPC поверх stdio,
- * дозволи на tool-calls автоматично схвалюються (без інтерактивних питань, паритет із
- * колишнім non-interactive `-p`). `claude` лишається як deprecated fallback для
- * тих, у кого pi-модель ще не налаштована; буде прибрано (мігруй на `skill pi`).
+ * Делегує виконання скіла зовнішньому ACP-агенту. `cursor`/`codex` — через
+ * `@7n/llm-lib/acp` (napi-міст до `llm_cascade::acp`: спавн, `session/prompt`,
+ * автоапрув дозволів — усе в Rust, без JSON-RPC у JS). `claude` — deprecated
+ * JS-шим `runAcpRunner` (Rust його не моделює); буде прибрано (мігруй на `skill pi`).
+ * На відміну від колишнього стрімінгу по чанках, napi-шлях повертає повний текст
+ * лише по завершенню ходу — Rust-функція не стрімить проміжні токени в JS.
  * @param {'claude' | 'cursor' | 'codex'} kind якого ACP-агента запускати
  * @param {string} prompt промпт скіла
  * @param {string} projectDir робочий каталог агента
  * @param {(line: string) => void} logError вивід попередження/помилок
- * @param {{ runAcpRunner?: typeof runAcpRunner }} [deps] інжект для тестів
- * @returns {Promise<number>} exit code (0 — `end_turn`, 1 — інакше)
+ * @param {{ runAcpRunner?: typeof runAcpRunner, runAcpAgent?: typeof runAcpAgent, out?: (chunk: string) => void }} [deps] інжекти для тестів
+ * @returns {Promise<number>} exit code (0 — успіх, 1 — інакше)
  */
-function runLlmCli(kind, prompt, projectDir, logError, deps = {}) {
+async function runLlmCli(kind, prompt, projectDir, logError, deps = {}) {
   const runner = ACP_RUNNERS[kind]
-  const runAcp = deps.runAcpRunner ?? runAcpRunner
 
   if (runner.deprecated) {
     logError(`[deprecated] skill ${kind} → use 'skill pi'; зовнішній ACP-агент буде прибрано`)
   }
 
-  return runAcp(kind, prompt, projectDir, logError, { isBinaryInPath })
+  if (kind === 'claude') {
+    const runAcp = deps.runAcpRunner ?? runAcpRunner
+    return runAcp(kind, prompt, projectDir, logError)
+  }
+
+  const runNativeAcp = deps.runAcpAgent ?? runAcpAgent
+  const out = deps.out ?? (chunk => stdout.write(chunk))
+
+  try {
+    out(await runNativeAcp(kind, prompt, projectDir))
+    return 0
+  } catch (error) {
+    logError(error instanceof Error ? error.message : String(error))
+    return 1
+  }
 }
 
 /**