@vzn/vx 0.0.636 → 0.0.638

package/src/cli/info.ts

@@ -1,15 +1,15 @@
 // `vx info` — workspace doctor printout. One screen of environment +
 // workspace + cache facts for bug reports and quick sanity checks.
 // The facts come
-// from the orchestrator's `collectInfo` (doctor.ts), which `vx mcp` reads
-// too; this file parses the flags and renders the rows.
+// from the orchestrator's `collectInfo` (doctor.ts); this file parses the flags and renders the rows.
 
 import os from 'node:os'
 import { collectInfo, type FlakyTask, type InfoFacts } from '../orchestrator/index.js'
 import { flagHint, formatValue, refuse, refusedWord, seeHelp } from './help.js'
 import { namedCacheDir, parseCacheDirFlag, warnToStderr } from './workspace-config.js'
 import { formatBytes } from './format.js'
-import { MIN_BUN } from '../util/index.js'
+import { detectColors, MIN_BUN, ui, type ColorSupport } from '../util/index.js'
+import { envName } from '../settings.js'
 
 interface InfoArgs {
   format: 'pretty' | 'json'
@@ -46,7 +46,7 @@ export function parseInfoArgs(args: readonly string[]): InfoArgs {
 export async function infoCmd(args: readonly string[]): Promise<number> {
   const parsed = parseInfoArgs(args)
   if (parsed.error) return refuse('info', args, parsed.error, 'VX_E_USAGE')
-  // `collectInfo` takes a resolved directory from `vx mcp` too, where one
+  // `collectInfo` takes a resolved directory, where one
   // not made yet is a workspace that never ran; the flag is the user's,
   // and a directory it names that is not there is a typo (item 900).
   if (parsed.cacheDir !== undefined) namedCacheDir(parsed.cacheDir)
@@ -55,14 +55,19 @@ export async function infoCmd(args: readonly string[]): Promise<number> {
     warn: warnToStderr,
   })
   process.stdout.write(
-    parsed.format === 'json' ? `${JSON.stringify(facts, null, 2)}\n` : `${renderInfo(facts)}\n`,
+    parsed.format === 'json'
+      ? `${JSON.stringify(facts)}\n`
+      : `${renderInfo(facts, detectColors(process.stdout))}\n`,
   )
   return 0
 }
 
-export function renderInfo(f: InfoFacts): string {
+export function renderInfo(f: InfoFacts, colors: ColorSupport = { enabled: false }): string {
   const rows: [string, string][] = [
-    ['vx', f.vx],
+    [
+      'vx',
+      `${f.vx} (${tildePath(f.vxPath)}${f.handedOffFrom === null ? '' : `, handed off from ${tildePath(f.handedOffFrom)}`})`,
+    ],
     // The prose stays in the rendered row; `f.bun` itself is the bare
     // version a script reads (doctor.ts says why).
     [
@@ -75,24 +80,25 @@ export function renderInfo(f: InfoFacts): string {
     ['git status cache', renderGitStatusCache(f.gitStatusCache)],
     ['workspace root', tildePath(f.workspaceRoot)],
     ['projects', describeProjects(f)],
-    // Only when a config did not load: the count above says "zero" for
-    // it, and this row says which and why.
-    ...(f.configErrors.length > 0
-      ? ([['config errors', f.configErrors.map((e) => `${e.path}: ${e.message}`).join('; ')]] as [
-          string,
-          string,
-        ][])
-      : []),
     ['plugins', describePlugins(f.plugins)],
     ['workers', describeWorkers(f.workers)],
     ['memory', describeMemory(f.memory)],
-    ['cache dir', tildePath(f.cacheDir)],
+    [
+      'cache dir',
+      `${tildePath(f.cacheDir)} (${formatBytes(f.cacheDirBytes)}, ${f.cacheStore === null ? 'the entries and ' : ''}this workspace's run history)`,
+    ],
     [
       'cache store',
-      f.cacheStore === null ? 'none: the cache dir holds the entries' : tildePath(f.cacheStore),
+      f.cacheStore === null
+        ? 'none: the cache dir holds the entries'
+        : `${tildePath(f.cacheStore)} (${formatBytes(f.cacheStoreBytes ?? 0)}, shared by this repo's checkouts)`,
     ],
     ['cache versions', `keys ${f.cacheVersion} · index schema ${f.schemaVersion}`],
     ['cache entries', describeEntries(f)],
+    // Only the settings a variable or the workspace changed.
+    ...(Object.values(f.settings).some((s) => s.source !== 'default')
+      ? ([['settings', describeSettings(f.settings)]] as [string, string][])
+      : []),
     ...(f.cacheRetention !== null
       ? ([['cache retention', describeRetention(f.cacheRetention)]] as [string, string][])
       : []),
@@ -101,7 +107,7 @@ export function renderInfo(f: InfoFacts): string {
       ? ([
           [
             'orphans',
-            `${f.orphans.artifacts} artifact${f.orphans.artifacts === 1 ? '' : 's'} (${formatBytes(f.orphans.bytes)}) the index does not know — \`vx cache prune\` reaps them`,
+            `${f.orphans.artifacts} artifact${f.orphans.artifacts === 1 ? '' : 's'} (${formatBytes(f.orphans.bytes)}) the index does not know — \`vx cache clean\` reaps them`,
           ],
         ] as [string, string][])
       : []),
@@ -116,10 +122,9 @@ export function renderInfo(f: InfoFacts): string {
     ],
     ['flaky tasks', describeFlakyTasks(f.flakyTasks)],
     ['sandbox', describeSandbox(f.sandbox)],
-    ['vx-lock.json', f.lockfile ? 'yes' : 'no'],
+    ['projects lock', f.lockfile ? 'yes' : 'no'],
   ]
-  const labelW = Math.max(...rows.map(([label]) => label.length))
-  return rows.map(([label, value]) => `${`${label}:`.padEnd(labelW + 1)} ${value}`).join('\n')
+  return ui.report('vx info', rows).lines(colors).join('\n')
 }
 
 /** `812 (1.2 GB)`, or against `cacheRetention.maxSize`: `812 (1.2 GB of 10 GB, 12%)`. */
@@ -132,6 +137,20 @@ function describeEntries(f: InfoFacts): string {
   return `${f.cacheEntries} (${size})`
 }
 
+/**
+ * The settings not at their default, with where each came from:
+ * `run.timeout 30m (VX_RUN_TIMEOUT), cache.policy local:rw (.vx/workspace)`.
+ */
+function describeSettings(settings: InfoFacts['settings']): string {
+  return Object.entries(settings)
+    .filter(([, s]) => s.source !== 'default')
+    .map(
+      ([path, s]) =>
+        `${path} ${String(s.value)} (${s.source === 'env' ? envName(path) : '.vx/workspace'})`,
+    )
+    .join(', ')
+}
+
 /** What a run's end evicts: `entries unused for 30d, and the oldest entries past 10G`. */
 function describeRetention(r: NonNullable<InfoFacts['cacheRetention']>): string {
   const parts = [
@@ -163,10 +182,10 @@ function describeSandbox(s: InfoFacts['sandbox']): string {
   return `unavailable — ${s.reason}; ${declared}${s.declared > 0 ? ' and will fail' : ''}`
 }
 
-/** `4 — the CPU count`, `2 — cgroup CPU quota 2 of 8 cores`, `8 — vx.workspace.ts (4 cores)`. */
+/** `4 — the CPU count`, `2 — cgroup CPU quota 2 of 8 cores`, `8 — .vx/workspace.ts (4 cores)`. */
 export function describeWorkers(w: InfoFacts['workers']): string {
   const quota = w.cpuQuota === null ? '' : `, cgroup CPU quota ${trimCores(w.cpuQuota)}`
-  if (w.source === 'workspace') return `${w.count} — vx.workspace.ts (${w.cores} cores${quota})`
+  if (w.source === 'workspace') return `${w.count} — .vx/workspace.ts (${w.cores} cores${quota})`
   if (w.source === 'cgroup')
     return `${w.count} — cgroup CPU quota ${trimCores(w.cpuQuota ?? w.count)} of ${w.cores} cores`
   return `${w.count} — the CPU count${quota}`
@@ -186,10 +205,7 @@ function trimCores(n: number): string {
 
 /** `2 — app#test (3 of 7 runs failed on unchanged inputs); api#e2e (1 of 4)`, or `none`. */
 function describeProjects(f: InfoFacts): string {
-  const tasks = `${f.tasks} task${f.tasks === 1 ? '' : 's'}`
-  const n = f.configErrors.length
-  if (n === 0) return `${f.projects} (${tasks})`
-  return `${f.projects} (${tasks} · ${n} config${n === 1 ? '' : 's'} did not load)`
+  return `${f.projects} (${f.tasks} task${f.tasks === 1 ? '' : 's'})`
 }
 
 function describeFlakyTasks(tasks: readonly FlakyTask[]): string {