martty 0.3.2-beta.1 → 0.3.2

package/lib/harness-registry.js

@@ -20,12 +20,17 @@ import { downloadFile } from './download.js'
 export const ACP_REGISTRY_URL = 'https://cdn.agentclientprotocol.com/registry/v1/latest/registry.json'
 
 export const OPENMA_PI_ACP = '@openma/pi-acp@latest'
+export const OPENMA_PI_ACP_REPOSITORY = 'https://github.com/openma-ai/pi-acp'
 
 function preferMarttyPiAcp(records) {
   return records.map((record) => record.id === 'pi-acp' ? {
     ...record,
-    label: 'Pi (OpenMA)',
+    label: 'pi-acp',
     version: 'latest',
+    // The official catalog also lists an unrelated unprefixed pi-acp package;
+    // Martty only ever launches @openma/pi-acp, and the repository URL is how
+    // users tell the two entries apart.
+    repository: OPENMA_PI_ACP_REPOSITORY,
     distributions: [{
       type: 'npx', command: 'npx', args: [OPENMA_PI_ACP], env: {},
     }],
@@ -144,6 +149,7 @@ export function normalizeAcpRegistry(value, options = {}) {
       ...(typeof agent.icon === 'string' ? { icon: agent.icon } : {}),
       version: agent.version,
       description: typeof agent.description === 'string' ? agent.description : '',
+      ...(typeof agent.repository === 'string' ? { repository: agent.repository } : {}),
       distributions,
     }]
   })

package/lib/harness-view.js

@@ -41,6 +41,15 @@ function findDescription(entry) {
   return `found locally · configure · ${commandText(entry)}`
 }
 
+// Registry entries carry their repository like the catalog's name /
+// description / repository triple; the URL distinguishes same-named
+// harnesses (e.g. @openma/pi-acp from the unrelated registry pi-acp).
+function describeCandidate(entry) {
+  const detail = findDescription(entry)
+  const repository = entry.registryRecord?.repository
+  return repository === undefined ? detail : `${detail} · ${repository}`
+}
+
 export function apply(ctx, options = {}) {
   const settingsPath = options.settingsPath
   let registryPromise
@@ -477,7 +486,7 @@ export function apply(ctx, options = {}) {
       openSelect({ id: 'harness-find', title: failure ? 'Add Harness · Registry offline' : pending ? 'Add Harness · checking Registry…' : 'Add Harness', searchable: true,
         value: entries[0]?.id ?? REFRESH,
         options: [
-          ...entries.map((entry) => ({ value: entry.id, label: entry.label, description: findDescription(entry),
+          ...entries.map((entry) => ({ value: entry.id, label: entry.label, description: describeCandidate(entry),
             ...currentOption(entry),
             group: locallyAvailable(entry) ? 'Installed / configured' : entry.status === 'Checking locally' ? 'Catalog' : 'Not downloaded' })),
           { value: REFRESH, label: failure ? 'Retry Registry' : 'Refresh Registry', description: failure ? errorText(failure) : 'Recheck the catalog and local programs' },

package/lib/harnesses.js

@@ -16,15 +16,20 @@ import {
   installRegistryBinary,
   managedBinaryPath,
   normalizeAcpRegistry,
-  OPENMA_PI_ACP,
   readAcpRegistrySnapshot,
 } from './harness-registry.js'
+import {
+  migrateLegacyAcpAgents,
+  readCustomAcpAgents,
+  removeCustomAcpAgent,
+  upsertCustomAcpAgent,
+} from '@openma/common/acp-harnesses/custom-agents'
 
 export { fetchAcpRegistry } from './harness-registry.js'
 
 const HARNESS_ID = /^[a-z0-9][a-z0-9-]*$/
-const PI_ACP_LATEST = OPENMA_PI_ACP
-const PI_ACP_SPEC = /^(?:@openma\/)?pi-acp(?:@.+)?$/
+const PI_ACP_ID = 'pi-acp'
+const PI_ACP_LEGACY_LABEL = 'Pi (OpenMA)'
 const ANSI = Object.freeze({ reset: '\x1b[0m', bold: '\x1b[1m', dim: '\x1b[2m', cyan: '\x1b[36m', green: '\x1b[32m' })
 
 // Kept as a source-compatible empty export. The source of truth is the
@@ -366,27 +371,50 @@ function validateHarness(value) {
   }
   const isNpx = /^npx(?:\.(?:cmd|bat|exe))?$/i.test(path.win32.basename(value.command))
   const normalizedArgs = isNpx
-    ? [...args]
-      .filter((arg) => arg !== '--yes' && arg !== '--prefer-offline')
-      // A pinned pi-acp spec (any old `pi-acp@x` or `@openma/pi-acp@x` entry)
-      // always resolves `latest` at runtime instead — harness upgrades ship
-      // through npm, never through a Martty release.
-      .map((arg) => PI_ACP_SPEC.test(arg) ? PI_ACP_LATEST : arg)
+    ? [...args].filter((arg) => arg !== '--yes' && arg !== '--prefer-offline')
     : [...args]
+  const label = typeof value.label === 'string' && value.label.trim().length > 0
+    ? value.label
+    : value.id
   return {
     id: value.id,
-    label: typeof value.label === 'string' && value.label.trim().length > 0
-      ? value.label
-      : value.id,
+    // Configs saved when the pi harness was branded "Pi (OpenMA)" keep
+    // working; only the stale default display label migrates to the id.
+    label: value.id === PI_ACP_ID && label === PI_ACP_LEGACY_LABEL ? PI_ACP_ID : label,
     command: value.command,
     args: normalizedArgs,
     ...(env !== undefined ? { env: { ...env } } : {}),
   }
 }
 
-function configuredHarnesses(settings) {
-  if (!Array.isArray(settings.harnesses)) return []
-  return settings.harnesses.map(validateHarness)
+// Harness definitions live in <oma>/agents.json, the shared store from
+// @openma/common — Backchat reads and writes the same file. Legacy rows
+// still inside settings.json (and Backchat's config.toml overrides) keep
+// reading through the shared module; the first shared write migrates
+// harnesses[] into config.toml and strips the key so removals stick.
+function settingsHome(settingsPath) {
+  return path.dirname(settingsPath)
+}
+
+function migrateStoredLabel(agent) {
+  // Configs saved when the pi harness was branded "Pi (OpenMA)" keep
+  // working; only the stale default display label migrates to the id.
+  return agent.id === PI_ACP_ID && agent.label === PI_ACP_LEGACY_LABEL
+    ? { ...agent, label: PI_ACP_ID }
+    : agent
+}
+
+function configuredHarnesses(settingsPath) {
+  // enabled=false is Backchat's disable flag: the definition stays stored
+  // but a disabled agent must not be selectable or launchable here.
+  return readCustomAcpAgents(settingsHome(settingsPath))
+    .filter((agent) => agent.enabled !== false)
+    .map(migrateStoredLabel)
+}
+
+function settingsWithoutHarnesses(settings) {
+  const { harnesses: _legacyHarnesses, ...rest } = settings
+  return rest
 }
 
 function withoutLegacyActive(settings) {
@@ -409,36 +437,37 @@ function persistedDefaultId(settings) {
 
 export function upsertHarness(settingsPath, harness) {
   const next = validateHarness(harness)
-  const settings = readSettings(settingsPath)
-  const harnesses = configuredHarnesses(settings)
-  const index = harnesses.findIndex(({ id }) => id === next.id)
-  if (index === -1) harnesses.push(next)
-  else harnesses[index] = next
-  writeSettings(settingsPath, { ...withoutLegacyActive(settings), harnesses })
+  // The shared write migrates legacy harnesses[] into config.toml and
+  // strips the key; read settings after it so a stale copy cannot race
+  // the migrated file back in.
+  upsertCustomAcpAgent(settingsHome(settingsPath), next)
+  const settings = withoutLegacyActive(readSettings(settingsPath))
+  writeSettings(settingsPath, settingsWithoutHarnesses(settings))
   return next
 }
 
 export function setDefaultHarness(settingsPath, id) {
-  const settings = readSettings(settingsPath)
-  const harnesses = configuredHarnesses(settings)
+  const harnesses = configuredHarnesses(settingsPath)
   if (!harnesses.some((harness) => harness.id === id)) {
     throw new Error(`unknown harness ${JSON.stringify(id)}`)
   }
-  writeSettings(settingsPath, { ...withoutLegacyActive(settings), harnesses, defaultHarness: id })
+  const settings = withoutLegacyActive(readSettings(settingsPath))
+  migrateLegacyAcpAgents(settingsHome(settingsPath))
+  writeSettings(settingsPath, { ...settingsWithoutHarnesses(settings), defaultHarness: id })
 }
 
 export function savedHarnesses(settingsPath) {
-  return configuredHarnesses(readSettings(settingsPath))
+  return configuredHarnesses(settingsPath)
 }
 
 export function removeHarnessConfiguration(settingsPath, expected) {
-  const settings = withoutLegacyActive(readSettings(settingsPath))
-  const harnesses = configuredHarnesses(settings)
-  const current = harnesses.find(({ id }) => id === expected.id)
+  const current = configuredHarnesses(settingsPath).find(({ id }) => id === expected.id)
   if (JSON.stringify(current) !== JSON.stringify(expected)) throw new Error('Harness configuration changed; review removal again')
-  settings.harnesses = harnesses.filter(({ id }) => id !== expected.id)
-  if (settings.defaultHarness === expected.id) delete settings.defaultHarness
-  writeSettings(settingsPath, settings)
+  removeCustomAcpAgent(settingsHome(settingsPath), expected.id)
+  const settings = withoutLegacyActive(readSettings(settingsPath))
+  const rest = settingsWithoutHarnesses(settings)
+  if (rest.defaultHarness === expected.id) delete rest.defaultHarness
+  writeSettings(settingsPath, rest)
 }
 
 // Kept as a source-compatible alias for integrations compiled against 0.2.30.
@@ -449,7 +478,7 @@ export function selectedHarness(settingsPath) {
   const settings = readSettings(settingsPath)
   const id = persistedDefaultId(settings)
   if (typeof id !== 'string') return undefined
-  return configuredHarnesses(settings).find((harness) => harness.id === id)
+  return configuredHarnesses(settingsPath).find((harness) => harness.id === id)
 }
 
 function executableFile(command) {
@@ -529,6 +558,7 @@ function registryRecords(options = {}) {
           : record.id,
         version: typeof record.version === 'string' ? record.version : 'unversioned',
         description: typeof record.description === 'string' ? record.description : '',
+        ...(typeof record.repository === 'string' ? { repository: record.repository } : {}),
         distributions,
       }]
     }
@@ -1004,7 +1034,7 @@ export function discoverHarnesses(settingsPath, options = {}) {
 }
 
 function discoverHarnessEntries(settingsPath, options, registryEntries) {
-  const configured = configuredHarnesses(readSettings(settingsPath))
+  const configured = configuredHarnesses(settingsPath)
     .map((harness) => ({ ...harness, source: 'configured' }))
   const defaults = (options.defaults ?? []).map((entry) => ({
     ...validateHarness(entry),
@@ -1071,7 +1101,7 @@ function searchScore(entry, query = '') {
 }
 
 function findHarnessCandidates(settingsPath, options = {}, query = '') {
-  const configured = configuredHarnesses(readSettings(settingsPath)).map((entry) => ({
+  const configured = configuredHarnesses(settingsPath).map((entry) => ({
     ...entry,
     source: 'configured',
     status: 'Configured',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "martty",
-  "version": "0.3.2-beta.1",
+  "version": "0.3.2",
   "description": "Terminal-native ACP client UI; Cordis client tree, any ACP agent",
   "license": "MIT",
   "repository": {
@@ -77,7 +77,8 @@
   "dependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@lobehub/icons-static-svg": "1.95.0",
-    "@openma/deepseek-harness-acp": "0.4.37",
+    "@openma/common": "^0.9.2",
+    "@openma/deepseek-harness-acp": "0.4.38",
     "@resvg/resvg-js": "2.6.2",
     "cross-spawn": "^7.0.6",
     "node-downloader-helper": "2.1.11"
@@ -97,10 +98,10 @@
     "agent-client-protocol"
   ],
   "optionalDependencies": {
-    "@openma/martty-darwin-arm64": "0.3.2-beta.1",
-    "@openma/martty-darwin-x64": "0.3.2-beta.1",
-    "@openma/martty-linux-x64": "0.3.2-beta.1",
-    "@openma/martty-linux-arm64": "0.3.2-beta.1",
-    "@openma/martty-win32-x64": "0.3.2-beta.1"
+    "@openma/martty-darwin-arm64": "0.3.2",
+    "@openma/martty-darwin-x64": "0.3.2",
+    "@openma/martty-linux-x64": "0.3.2",
+    "@openma/martty-linux-arm64": "0.3.2",
+    "@openma/martty-win32-x64": "0.3.2"
   }
 }