@vzn/vx 0.0.635 → 0.0.637

package/src/cli/show.ts

@@ -1,10 +1,10 @@
-// `vx show [target]` — introspect the workspace's LIVE resolved configs:
-// what a run would see here, now. Configs load through the same path a
-// run uses (`loadProjects`): the plugin `config` and `project` stages
-// apply, so a package a plugin gives tasks to shows them. No target: one
-// line per project. `<project>`: every task's resolved config. `<pkg>#
-// <task>`: one task. `<task>`: that task in every project declaring it.
-// Deliberately NOT the lock — vx-lock.json is already the frozen JSON.
+// `vx show [target]` — introspect the workspace's resolved configs: what a
+// run would see here, now. Configs load through the same path a run uses
+// (`loadProjects`): from the lock, with the plugin `config` and `project`
+// stages applied, so a package a plugin gives tasks to shows them. No lock
+// is `NO_LOCK`. No target: one line per project. `<project>`: every task's
+// resolved config. `<pkg>#<task>`: one task. `<task>`: that task in every
+// project declaring it.
 
 import path from 'node:path'
 import type { ProjectConfig, TaskConfig } from '../config.js'
@@ -23,11 +23,10 @@ import {
   type AffectedChanges,
   buildPackageGraph,
   findWorkspaceRoot,
-  type LoadReads,
-  loadWorkspace,
   type ProjectEntry,
   type ProjectMeta,
 } from '../workspace/index.js'
+import { runRules } from '../settings.js'
 
 interface ShowArgs {
   target?: string
@@ -83,9 +82,8 @@ export function parseShowArgs(args: readonly string[]): ShowArgs {
 export async function showCmd(args: readonly string[]): Promise<number> {
   const parsed = parseShowArgs(args)
   if (parsed.error) return refuse('show', args, parsed.error, 'VX_E_USAGE')
-  const reads: LoadReads = new Map()
-  const root = await findWorkspaceRoot(process.cwd(), reads)
-  const metas = await discoverCliProjects(await loadWorkspace(root, reads))
+  const root = await findWorkspaceRoot(process.cwd())
+  const metas = await discoverCliProjects(root)
   const byName = new Map(metas.map((m) => [m.name, m]))
 
   // `//#task` is Turbo's root package's task: the root project here (D-39),
@@ -149,7 +147,7 @@ export async function showCmd(args: readonly string[]): Promise<number> {
       affected = f
       raw.unshift(f)
     }
-    const r = await resolveFilters(process.cwd(), raw, { noCreate: true }, affected)
+    const r = await resolveFilters(process.cwd(), raw, {}, affected)
     if ('error' in r) throw new UserError(`vx show: ${r.error}`, r.code ?? 'VX_E_USAGE')
     if ('empty' in r) process.stderr.write(`vx show: ${r.empty}\n`)
     selected = new Set('empty' in r ? [] : r.names)
@@ -158,7 +156,7 @@ export async function showCmd(args: readonly string[]): Promise<number> {
   }
   const inSelection = (name: string): boolean => selected === undefined || selected.has(name)
 
-  const projects = await loadCliProjects(root, metas, scope, { noCreate: true, closure: true })
+  const projects = await loadCliProjects(root, metas, scope, { closure: true })
 
   if (parsed.target === undefined) {
     const shown = metas.filter((m) => inSelection(m.name))
@@ -200,7 +198,7 @@ export async function showCmd(args: readonly string[]): Promise<number> {
           plugins: ws.plugins,
           workspaceRoot: root,
           cacheDir: ws.cacheDir,
-          rules: ws.workspaceConfig?.rules,
+          rules: runRules(ws.workspaceConfig),
           warn: warnToStderr,
         }),
       )
@@ -342,7 +340,7 @@ function renderProject(
     // Round-trip so the printed object is exactly the JSON form of the
     // resolved config (drops `undefined` fields).
     const shown = shownConfig(config)
-    return `${JSON.stringify(JSON.parse(JSON.stringify({ name, dir, config: shown })), null, 2)}\n`
+    return `${JSON.stringify({ name, dir, config: shown })}\n`
   }
   const tags = config?.tags ?? []
   const tasks = Object.entries(config?.tasks ?? {})
@@ -365,7 +363,7 @@ function renderTask(
 ): string {
   if (format === 'json') {
     const obj = { name, dir, task: taskName, config: shownTask(task) }
-    return `${JSON.stringify(JSON.parse(JSON.stringify(obj)), null, 2)}\n`
+    return `${JSON.stringify(obj)}\n`
   }
   return `${name} — ${dir}\n\n${taskBlock(taskName, task)}`
 }
@@ -383,7 +381,7 @@ function renderTaskAcross(
       task: taskName,
       config: shownTask(p.config.tasks![taskName]!),
     }))
-    return `${JSON.stringify(JSON.parse(JSON.stringify(list)), null, 2)}\n`
+    return `${JSON.stringify(list)}\n`
   }
   return declaring
     .map(
@@ -406,7 +404,7 @@ function taskBlock(taskName: string, raw: TaskConfig): string {
   const exec = task.exec
   rows.push(['command', exec?.command ?? '(group)'])
   add('dependsOn', list(task.dependsOn))
-  add('timeout', exec?.timeout === undefined ? undefined : `${exec.timeout}ms`)
+  add('timeout', exec?.timeout)
   add('retries', exec?.retries)
   add('env.passThrough', list(exec?.env?.passThrough))
   add(

package/src/cli/task-verb.ts

@@ -5,7 +5,7 @@
 // name would change meaning the day a plugin declares that verb.
 
 import { isDefaultBuild, nxProjectTarget, projectNamed } from '../orchestrator/index.js'
-import { findWorkspaceRoot, loadWorkspace } from '../workspace/index.js'
+import { findWorkspaceRoot } from '../workspace/index.js'
 import { nearest } from '../util/index.js'
 import { findCwdProject } from './select.js'
 import { discoverCliProjects, loadCliProjects } from './workspace-config.js'
@@ -79,14 +79,7 @@ async function workspaceProjects(
 ): Promise<Awaited<ReturnType<typeof loadCliProjects>> | null> {
   try {
     const root = await findWorkspaceRoot(cwd)
-    return await loadCliProjects(
-      root,
-      await discoverCliProjects(await loadWorkspace(root)),
-      'all',
-      {
-        noCreate: true,
-      },
-    )
+    return await loadCliProjects(root, await discoverCliProjects(root))
   } catch {
     // No workspace, or a config that does not load: the caller's plain line stands.
     return null