@plainconceptsplatform/agent-harness 3.3.0 → 3.4.1

package/cli/steps/source/index.js

@@ -1,123 +1,123 @@
-import { checkbox, select } from '@inquirer/prompts'
-import fse from 'fs-extra'
-import path from 'path'
-import { fileURLToPath } from 'url'
-import { header, info, success, warn } from '../../utils/exec.js'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const SOURCE_PRESET_PATH = path.resolve(__dirname, '../../presets/source.json')
-const sourcePreset = await fse.readJson(SOURCE_PRESET_PATH)
-
-async function listChildFolders(cwd) {
-  const entries = await fse.readdir(cwd)
-  const dirs = []
-
-  for (const name of entries) {
-    if (name.startsWith('.')) continue
-    const abs = path.join(cwd, name)
-    try {
-      const stat = await fse.stat(abs)
-      if (!stat.isDirectory()) continue
-      dirs.push({ name, abs })
-    } catch {
-      // ignore invalid entries
-    }
-  }
-
-  dirs.sort((a, b) => a.name.localeCompare(b.name))
-  return dirs
-}
-
-async function listParentFolders(cwd) {
-  const parent = path.resolve(cwd, '..')
-  const entries = await fse.readdir(parent)
-  const dirs = []
-
-  for (const name of entries) {
-    if (name.startsWith('.')) continue
-    const abs = path.join(parent, name)
-    try {
-      const stat = await fse.stat(abs)
-      if (!stat.isDirectory()) continue
-      if (path.resolve(abs) === path.resolve(cwd)) continue
-      dirs.push({ name, abs })
-    } catch {
-      // ignore invalid entries
-    }
-  }
-
-  dirs.sort((a, b) => a.name.localeCompare(b.name))
-  return dirs
-}
-
-export async function chooseSourceScope() {
-  header('Step 1, Source code scope')
-
-  const cwd = process.cwd()
-  info('Choose where agents should read source code from during init analysis.')
-
-  const mode = await select({
-    message: sourcePreset.message,
-    default: sourcePreset.default,
-    choices: sourcePreset.choices,
-  })
-
-  if (mode === 'current') {
-    success(`Source scope: ${cwd}`)
-    return { sourceMode: 'current', sourceRoots: [cwd] }
-  }
-
-  if (mode === 'children') {
-    const childFolders = await listChildFolders(cwd)
-    if (childFolders.length === 0) {
-      warn('No child folders found in current directory. Falling back to current folder.')
-      success(`Source scope: ${cwd}`)
-      return { sourceMode: 'current', sourceRoots: [cwd] }
-    }
-
-    const selected = await checkbox({
-      message: sourcePreset.childrenSelectionMessage,
-      choices: childFolders.map(d => ({
-        name: `./${d.name}`,
-        value: d.abs,
-        checked: true,
-      })),
-      required: true,
-    })
-
-    if (!selected || selected.length === 0) {
-      warn('No folders selected. Falling back to current folder.')
-      success(`Source scope: ${cwd}`)
-      return { sourceMode: 'current', sourceRoots: [cwd] }
-    }
-
-    success(`Source scope: ${selected.map(p => path.basename(p)).join(', ')}`)
-    return { sourceMode: 'children-selected', sourceRoots: selected }
-  }
-
-  const parentFolders = await listParentFolders(cwd)
-  if (parentFolders.length === 0) {
-    warn('No sibling folders found in parent directory. Falling back to current folder.')
-    success(`Source scope: ${cwd}`)
-    return { sourceMode: 'current', sourceRoots: [cwd] }
-  }
-
-  const selected = await checkbox({
-    message: sourcePreset.parentSelectionMessage,
-    choices: parentFolders.map(d => ({
-      name: `../${d.name}`,
-      value: d.abs,
-      checked: true,
-    })),
-    required: true,
-  })
-
-  if (!selected || selected.length === 0) {
-    warn('No folders selected. Falling back to current folder.')
-    success(`Source scope: ${cwd}`)
-    return { sourceMode: 'current', sourceRoots: [cwd] }
-  }
-
-  success(`Source scope: ${selected.map(p => path.basename(p)).join(', ')}`)
-  return { sourceMode: 'parent-selected', sourceRoots: selected }
-}
+import { checkbox, select } from '@inquirer/prompts'
+import fse from 'fs-extra'
+import path from 'path'
+import { fileURLToPath } from 'url'
+import { header, info, success, warn } from '../../utils/exec.js'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const SOURCE_PRESET_PATH = path.resolve(__dirname, '../../presets/source.json')
+const sourcePreset = await fse.readJson(SOURCE_PRESET_PATH)
+
+async function listChildFolders(cwd) {
+  const entries = await fse.readdir(cwd)
+  const dirs = []
+
+  for (const name of entries) {
+    if (name.startsWith('.')) continue
+    const abs = path.join(cwd, name)
+    try {
+      const stat = await fse.stat(abs)
+      if (!stat.isDirectory()) continue
+      dirs.push({ name, abs })
+    } catch {
+      // ignore invalid entries
+    }
+  }
+
+  dirs.sort((a, b) => a.name.localeCompare(b.name))
+  return dirs
+}
+
+async function listParentFolders(cwd) {
+  const parent = path.resolve(cwd, '..')
+  const entries = await fse.readdir(parent)
+  const dirs = []
+
+  for (const name of entries) {
+    if (name.startsWith('.')) continue
+    const abs = path.join(parent, name)
+    try {
+      const stat = await fse.stat(abs)
+      if (!stat.isDirectory()) continue
+      if (path.resolve(abs) === path.resolve(cwd)) continue
+      dirs.push({ name, abs })
+    } catch {
+      // ignore invalid entries
+    }
+  }
+
+  dirs.sort((a, b) => a.name.localeCompare(b.name))
+  return dirs
+}
+
+export async function chooseSourceScope() {
+  header('Step 1, Source code scope')
+
+  const cwd = process.cwd()
+  info('Choose where agents should read source code from during init analysis.')
+
+  const mode = await select({
+    message: sourcePreset.message,
+    default: sourcePreset.default,
+    choices: sourcePreset.choices,
+  })
+
+  if (mode === 'current') {
+    success(`Source scope: ${cwd}`)
+    return { sourceMode: 'current', sourceRoots: [cwd] }
+  }
+
+  if (mode === 'children') {
+    const childFolders = await listChildFolders(cwd)
+    if (childFolders.length === 0) {
+      warn('No child folders found in current directory. Falling back to current folder.')
+      success(`Source scope: ${cwd}`)
+      return { sourceMode: 'current', sourceRoots: [cwd] }
+    }
+
+    const selected = await checkbox({
+      message: sourcePreset.childrenSelectionMessage,
+      choices: childFolders.map(d => ({
+        name: `./${d.name}`,
+        value: d.abs,
+        checked: true,
+      })),
+      required: true,
+    })
+
+    if (!selected || selected.length === 0) {
+      warn('No folders selected. Falling back to current folder.')
+      success(`Source scope: ${cwd}`)
+      return { sourceMode: 'current', sourceRoots: [cwd] }
+    }
+
+    success(`Source scope: ${selected.map(p => path.basename(p)).join(', ')}`)
+    return { sourceMode: 'children-selected', sourceRoots: selected }
+  }
+
+  const parentFolders = await listParentFolders(cwd)
+  if (parentFolders.length === 0) {
+    warn('No sibling folders found in parent directory. Falling back to current folder.')
+    success(`Source scope: ${cwd}`)
+    return { sourceMode: 'current', sourceRoots: [cwd] }
+  }
+
+  const selected = await checkbox({
+    message: sourcePreset.parentSelectionMessage,
+    choices: parentFolders.map(d => ({
+      name: `../${d.name}`,
+      value: d.abs,
+      checked: true,
+    })),
+    required: true,
+  })
+
+  if (!selected || selected.length === 0) {
+    warn('No folders selected. Falling back to current folder.')
+    success(`Source scope: ${cwd}`)
+    return { sourceMode: 'current', sourceRoots: [cwd] }
+  }
+
+  success(`Source scope: ${selected.map(p => path.basename(p)).join(', ')}`)
+  return { sourceMode: 'parent-selected', sourceRoots: selected }
+}

package/cli/utils/agent-color.js

@@ -1,111 +1,111 @@
-/**
- * Deterministic colour for an agent, derived from its name.
- *
- * opencode accepts either a hex value or one of its theme keywords. Two of
- * those keywords are reserved for the primaries so they stay recognisable
- * across every project: build is `primary`, plan is `warning`. Every other
- * agent gets a hex derived from its name, so a `butterfly-engineer` is the
- * same colour in every repository that has one, with no central registry and
- * no chance of two agents in one project colliding by accident.
- *
- * Saturation and lightness are fixed rather than hashed. That is what keeps
- * the result usable: hashing them too would eventually produce near-grey,
- * near-black or near-white agents that are invisible against one theme or the
- * other. Only the hue varies.
- */
-
-/** opencode's theme keywords. Treated as unset so a name-derived hex wins. */
-export const THEME_COLORS = new Set([
-  'primary',
-  'secondary',
-  'accent',
-  'success',
-  'warning',
-  'error',
-  'info',
-])
-
-/** Reserved: the two primaries, which never take a derived colour. */
-export const RESERVED_COLORS = { build: 'primary', plan: 'warning' }
-
-const SATURATION = 68
-const LIGHTNESS = 52
-
-/**
- * FNV-1a, 32 bit. Chosen because it is short enough to duplicate verbatim in
- * the shipped plugin and produces the same hue everywhere: the same name must
- * resolve to the same colour on every machine and in every project.
- */
-function hash(value) {
-  let h = 0x811c9dc5
-  for (let i = 0; i < value.length; i++) {
-    h ^= value.charCodeAt(i)
-    h = Math.imul(h, 0x01000193) >>> 0
-  }
-  return h >>> 0
-}
-
-function hslToHex(h, s, l) {
-  const sat = s / 100
-  const light = l / 100
-  const c = (1 - Math.abs(2 * light - 1)) * sat
-  const x = c * (1 - Math.abs(((h / 60) % 2) - 1))
-  const m = light - c / 2
-
-  const [r, g, b] =
-    h < 60 ? [c, x, 0] :
-    h < 120 ? [x, c, 0] :
-    h < 180 ? [0, c, x] :
-    h < 240 ? [0, x, c] :
-    h < 300 ? [x, 0, c] :
-    [c, 0, x]
-
-  const toHex = v => Math.round((v + m) * 255).toString(16).padStart(2, '0')
-  return `#${toHex(r)}${toHex(g)}${toHex(b)}`.toUpperCase()
-}
-
-/** The colour for an agent, honouring the two reserved primaries. */
-export function agentColor(name) {
-  if (name in RESERVED_COLORS) return RESERVED_COLORS[name]
-  return hslToHex(hash(name) % 360, SATURATION, LIGHTNESS)
-}
-
-/**
- * Render a colour for YAML frontmatter.
- *
- * A hex has to be quoted. `#` opens a comment in YAML, so `color: #D83155`
- * parses as null and opencode rejects the agent outright with "Invalid input
- * color". Theme keywords need no quoting and are left bare.
- */
-export function yamlColor(value) {
-  return value.startsWith('#') ? `"${value}"` : value
-}
-
-/** The value of a frontmatter colour, with any quoting removed. */
-export function unquoteColor(raw) {
-  return (raw ?? '').trim().replace(/^["']/, '').replace(/["']$/, '')
-}
-
-/**
- * The frontmatter colour line an agent should have, or null when the current
- * one is already correct. Covers both jobs: choosing the colour, and quoting it
- * so the file actually parses.
- */
-export function colorLineFor(name, rawCurrent) {
-  const current = unquoteColor(rawCurrent)
-  const target = shouldDeriveColor(current) ? agentColor(name) : current
-  const rendered = yamlColor(target)
-  return (rawCurrent ?? '').trim() === rendered ? null : `color: ${rendered}`
-}
-
-/**
- * Whether a frontmatter colour should be replaced by the derived one.
- *
- * A theme keyword is replaced: those came from the old template, which asked
- * whoever ran /make-engineer to pick one and avoid collisions by hand. An
- * explicit hex is left alone, because someone chose it deliberately.
- */
-export function shouldDeriveColor(current) {
-  if (!current) return true
-  return THEME_COLORS.has(current.trim().toLowerCase())
-}
+/**
+ * Deterministic colour for an agent, derived from its name.
+ *
+ * opencode accepts either a hex value or one of its theme keywords. Two of
+ * those keywords are reserved for the primaries so they stay recognisable
+ * across every project: build is `primary`, plan is `warning`. Every other
+ * agent gets a hex derived from its name, so a `butterfly-engineer` is the
+ * same colour in every repository that has one, with no central registry and
+ * no chance of two agents in one project colliding by accident.
+ *
+ * Saturation and lightness are fixed rather than hashed. That is what keeps
+ * the result usable: hashing them too would eventually produce near-grey,
+ * near-black or near-white agents that are invisible against one theme or the
+ * other. Only the hue varies.
+ */
+
+/** opencode's theme keywords. Treated as unset so a name-derived hex wins. */
+export const THEME_COLORS = new Set([
+  'primary',
+  'secondary',
+  'accent',
+  'success',
+  'warning',
+  'error',
+  'info',
+])
+
+/** Reserved: the two primaries, which never take a derived colour. */
+export const RESERVED_COLORS = { build: 'primary', plan: 'warning' }
+
+const SATURATION = 68
+const LIGHTNESS = 52
+
+/**
+ * FNV-1a, 32 bit. Chosen because it is short enough to duplicate verbatim in
+ * the shipped plugin and produces the same hue everywhere: the same name must
+ * resolve to the same colour on every machine and in every project.
+ */
+function hash(value) {
+  let h = 0x811c9dc5
+  for (let i = 0; i < value.length; i++) {
+    h ^= value.charCodeAt(i)
+    h = Math.imul(h, 0x01000193) >>> 0
+  }
+  return h >>> 0
+}
+
+function hslToHex(h, s, l) {
+  const sat = s / 100
+  const light = l / 100
+  const c = (1 - Math.abs(2 * light - 1)) * sat
+  const x = c * (1 - Math.abs(((h / 60) % 2) - 1))
+  const m = light - c / 2
+
+  const [r, g, b] =
+    h < 60 ? [c, x, 0] :
+    h < 120 ? [x, c, 0] :
+    h < 180 ? [0, c, x] :
+    h < 240 ? [0, x, c] :
+    h < 300 ? [x, 0, c] :
+    [c, 0, x]
+
+  const toHex = v => Math.round((v + m) * 255).toString(16).padStart(2, '0')
+  return `#${toHex(r)}${toHex(g)}${toHex(b)}`.toUpperCase()
+}
+
+/** The colour for an agent, honouring the two reserved primaries. */
+export function agentColor(name) {
+  if (name in RESERVED_COLORS) return RESERVED_COLORS[name]
+  return hslToHex(hash(name) % 360, SATURATION, LIGHTNESS)
+}
+
+/**
+ * Render a colour for YAML frontmatter.
+ *
+ * A hex has to be quoted. `#` opens a comment in YAML, so `color: #D83155`
+ * parses as null and opencode rejects the agent outright with "Invalid input
+ * color". Theme keywords need no quoting and are left bare.
+ */
+export function yamlColor(value) {
+  return value.startsWith('#') ? `"${value}"` : value
+}
+
+/** The value of a frontmatter colour, with any quoting removed. */
+export function unquoteColor(raw) {
+  return (raw ?? '').trim().replace(/^["']/, '').replace(/["']$/, '')
+}
+
+/**
+ * The frontmatter colour line an agent should have, or null when the current
+ * one is already correct. Covers both jobs: choosing the colour, and quoting it
+ * so the file actually parses.
+ */
+export function colorLineFor(name, rawCurrent) {
+  const current = unquoteColor(rawCurrent)
+  const target = shouldDeriveColor(current) ? agentColor(name) : current
+  const rendered = yamlColor(target)
+  return (rawCurrent ?? '').trim() === rendered ? null : `color: ${rendered}`
+}
+
+/**
+ * Whether a frontmatter colour should be replaced by the derived one.
+ *
+ * A theme keyword is replaced: those came from the old template, which asked
+ * whoever ran /make-engineer to pick one and avoid collisions by hand. An
+ * explicit hex is left alone, because someone chose it deliberately.
+ */
+export function shouldDeriveColor(current) {
+  if (!current) return true
+  return THEME_COLORS.has(current.trim().toLowerCase())
+}