@agfpd/iapeer 0.4.51 → 0.4.52

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agfpd/iapeer",
-  "version": "0.4.51",
+  "version": "0.4.52",
   "description": "Run Claude Code and Codex CLI as local background agents that message each other, wake on demand, and share one memory.",
   "license": "Apache-2.0",
   "author": "agfpd",

package/src/enable/index.ts

@@ -24,7 +24,7 @@ import { spawnSync } from 'child_process'
 import { existsSync, readFileSync, realpathSync } from 'fs'
 import { basename, join } from 'path'
 import { homedir } from 'os'
-import { MARKETPLACE_NAME, isMarketplaceRegisteredAs, refreshMarketplace, registerMarketplace } from '../onboard/index.ts'
+import { MARKETPLACE_NAME, isExecutable, isMarketplaceRegisteredAs, refreshMarketplace, registerMarketplace } from '../onboard/index.ts'
 import { findPeer, readPeersIndex } from '../registry/index.ts'
 import { IapError } from '../core/errors.ts'
 import { normalizeNameCandidate } from '../core/normalize.ts'
@@ -205,10 +205,14 @@ function resolvePeer(opts: EnableOptions): { personality: string; cwd: string; r
   return { personality, cwd: rec.cwd, runtimes: agentic }
 }
 
-function isExecutable(bin: string, env: NodeJS.ProcessEnv): boolean {
-  const r = spawnSync(bin, ['--version'], { stdio: 'ignore', env: env as Record<string, string> })
-  return r.error === undefined && r.status !== null
-}
+// В57 — the local `--version` ANSWER probe was removed: it spawned the runtime CLI with NO
+// timeout, so a wedged binary (the known macOS launch-approval park after a cask update —
+// onboard's live incident: probes sat 25+ min) hung `enable` forever. enable now uses
+// onboard's PRESENCE probe (accessSync X_OK / PATH scan — no spawn at all), and every real
+// runtime-CLI invocation below carries a hard timeout, mirroring onboard.
+const LIST_TIMEOUT_MS = 60_000 // read-only queries (plugin list)
+const MUTATE_TIMEOUT_MS = 120_000 // install / enable / add (may clone from GitHub)
+const SETUP_TIMEOUT_MS = 300_000 // a СЛОЖНЫЙ plugin's own installer — the longest legitimate leg
 
 /** Read the peer-scoped entry for this plugin from a fresh `claude plugin list --json`.
  *  CRITICAL: `enabled` is evaluated relative to the CURRENT cwd's project (verified
@@ -227,6 +231,7 @@ function claudeEntry(
     encoding: 'utf8',
     env: env as Record<string, string>,
     maxBuffer: 64 * 1024 * 1024,
+    timeout: LIST_TIMEOUT_MS,
   })
   return findPeerScopedEntry(parseInstalledPlugins(list.stdout ?? ''), plugin, marketplace, peerCwd)
 }
@@ -251,6 +256,7 @@ function enableClaude(plugin: string, marketplace: string, peerCwd: string, env:
       cwd: peerCwd,
       encoding: 'utf8',
       env: env as Record<string, string>,
+      timeout: MUTATE_TIMEOUT_MS,
     })
     if (inst.status !== 0) {
       return { runtime: 'claude', state: 'failed', detail: (inst.stderr || inst.stdout || `exit ${inst.status}`).trim() }
@@ -264,6 +270,7 @@ function enableClaude(plugin: string, marketplace: string, peerCwd: string, env:
       cwd: peerCwd,
       encoding: 'utf8',
       env: env as Record<string, string>,
+      timeout: MUTATE_TIMEOUT_MS,
     })
     const confirmed = claudeEntry(bin, plugin, marketplace, peerCwd, env)
     if (!confirmed?.enabled) {
@@ -300,7 +307,12 @@ export function parseCodexPluginStatus(jsonOutput: string, id: string): 'enabled
 }
 
 function codexState(bin: string, id: string, env: NodeJS.ProcessEnv): 'enabled' | 'disabled' | 'absent' {
-  const r = spawnSync(bin, ['plugin', 'list', '--json'], { encoding: 'utf8', env: env as Record<string, string>, maxBuffer: 64 * 1024 * 1024 })
+  const r = spawnSync(bin, ['plugin', 'list', '--json'], {
+    encoding: 'utf8',
+    env: env as Record<string, string>,
+    maxBuffer: 64 * 1024 * 1024,
+    timeout: LIST_TIMEOUT_MS,
+  })
   return parseCodexPluginStatus(r.stdout ?? '', id)
 }
 
@@ -314,7 +326,11 @@ function enableCodex(plugin: string, marketplace: string, env: NodeJS.ProcessEnv
   const id = `${plugin}@${marketplace}`
   const before = codexState(bin, id, env)
   if (before === 'enabled') return { runtime: 'codex', state: 'already-enabled' }
-  const add = spawnSync(bin, ['plugin', 'add', id], { encoding: 'utf8', env: env as Record<string, string> })
+  const add = spawnSync(bin, ['plugin', 'add', id], {
+    encoding: 'utf8',
+    env: env as Record<string, string>,
+    timeout: MUTATE_TIMEOUT_MS,
+  })
   if (add.status !== 0) {
     return { runtime: 'codex', state: 'failed', detail: (add.stderr || add.stdout || `exit ${add.status}`).trim() }
   }
@@ -346,6 +362,7 @@ function callSetup(
     cwd: installPath,
     encoding: 'utf8',
     env: setupEnv as Record<string, string>,
+    timeout: SETUP_TIMEOUT_MS,
   })
   if (r.error || (r.status ?? 1) !== 0) {
     return { ok: false, detail: (r.stderr || r.stdout || r.error?.message || `exit ${r.status}`).trim() }

package/src/identity/index.ts

@@ -120,6 +120,12 @@ export interface EnsurePeerProfileOptions {
    *  always-on plist so launchd's minimal PATH resolves it. Forwarded to
    *  installAlwaysOnPlist; ignored for warm-on-demand runtimes. */
   runtimeBin?: string
+  /** В36 — peer description, persisted into the LOCAL profile (the source of truth the
+   *  registry is reindexed FROM). Writing it only to the registry made it a stale copy:
+   *  the first reindexFromLocals (addRuntime on a dual-runtime create, connect telegram,
+   *  verify --fix) projected the profile's empty description over it, wiping the Layer-3
+   *  addressing signal fleet-wide. */
+  description?: string
 }
 
 // ─────────────────────────────────────────────────────────────────────────────
@@ -513,7 +519,9 @@ export function ensurePeerProfile(options: EnsurePeerProfileOptions): PeerProfil
       personality,
       runtime: options.runtime,
       runtimes: discoveredRuntimes,
-      description: '',
+      // В36 — the description lands in the LOCAL profile at birth (source of truth);
+      // the registry row is a projection of this file, not the other way around.
+      description: options.description?.trim() ?? '',
       intelligence: defaultIntelligenceForRuntime(options.runtime),
     }
     ensureLocalRuntimeScopes(cwd, profile.runtimes)
@@ -573,8 +581,17 @@ export function ensurePeerProfile(options: EnsurePeerProfileOptions): PeerProfil
     })
     intelligence = defaultIntelligenceForRuntime(options.runtime)
   }
-  if (mergedRuntimes.length !== existing.runtimes.length || intelligence !== existing.intelligence) {
-    const updated = { ...existing, runtimes: mergedRuntimes, intelligence }
+  // В36 — an explicit description on RE-provision updates the local profile too (H1 in
+  // writePeerProfileAtomic still guards: absent/empty never wipes an existing value).
+  const explicitDescription = options.description?.trim()
+  const descriptionChanged = Boolean(explicitDescription) && explicitDescription !== existing.description
+  if (mergedRuntimes.length !== existing.runtimes.length || intelligence !== existing.intelligence || descriptionChanged) {
+    const updated = {
+      ...existing,
+      runtimes: mergedRuntimes,
+      intelligence,
+      ...(descriptionChanged ? { description: explicitDescription! } : {}),
+    }
     writePeerProfileAtomic(cwd, updated)
     return updated
   }

package/src/launch/invocation.ts

@@ -50,7 +50,13 @@ export function buildLaunchInvocation(
     PEER_PERSONALITY: spec.personality,
     PEER_RUNTIME: spec.runtime,
     PEER_IDENTITY: spec.identity,
-    PATH: env.PATH ?? `${homedir()}/.bun/bin:${homedir()}/.local/bin:/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/usr/bin:/bin`,
+    // В45 — a per-peer launch.env PATH override WINS: this literal sits after the launchEnv
+    // spread, so `env.PATH ?? default` silently clobbered the operator's override (env.PATH
+    // is nearly always set). The legacy launcher sourced the file — there PATH took effect.
+    PATH:
+      launchEnv.env.PATH ??
+      env.PATH ??
+      `${homedir()}/.bun/bin:${homedir()}/.local/bin:/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/usr/bin:/bin`,
   }
   // Strip the claude-code-internal session namespace from the inherited env. When a launch is
   // INITIATED from inside a running claude session (an operator CLI, or a claude peer shelling out),

package/src/onboard/index.ts

@@ -71,7 +71,10 @@ function runtimeBin(runtime: OnboardRuntime, env: NodeJS.ProcessEnv): string {
   return env.IAPEER_CODEX_BIN?.trim() || 'codex'
 }
 
-function isExecutable(binOrName: string, env: NodeJS.ProcessEnv = process.env): boolean {
+/** PRESENCE probe (no spawn) — exported for enable/ (В57): the same wedged-runtime class
+ *  (macOS launch-approval parking every invocation pre-main) hung enable's `--version`
+ *  ANSWER probe forever; presence answers "is it installed" without executing anything. */
+export function isExecutable(binOrName: string, env: NodeJS.ProcessEnv = process.env): boolean {
   if (binOrName.includes('/')) {
     try {
       accessSync(binOrName, FS.X_OK)

package/src/provision/index.ts

@@ -89,6 +89,9 @@ export async function provisionPeer(opts: ProvisionPeerOptions): Promise<Provisi
     peers,
     personality: opts.personality,
     runtimeBin,
+    // В36 — description goes into the LOCAL profile (source of truth); the registry
+    // upsert below then persists a value reindexFromLocals can no longer wipe.
+    description: opts.description,
     warn: opts.warn,
   })
   // Birth-time native-memory lever (контракт «Слот памяти» §Native-память рантаймов):