Skip to content

Add JSON output for remaining CLI list/status commands#143

Open
bolasse1234 wants to merge 1 commit into
profullstack:masterfrom
bolasse1234:cli-json-status-list
Open

Add JSON output for remaining CLI list/status commands#143
bolasse1234 wants to merge 1 commit into
profullstack:masterfrom
bolasse1234:cli-json-status-list

Conversation

@bolasse1234
Copy link
Copy Markdown

Closes #142. Related to #133.

This fills in a few CLI convention gaps where nested list/status commands only had human-readable stub output even though the docs say those commands should support --json.

What changed:

  • added --json output to the remaining list/status subcommands under the primary CLI verbs
  • added a command-tree test that walks �uild, config, promote, and scale so new list/status commands under those verbs keep the JSON option

Verified:

  • corepack pnpm --filter @profullstack/sh1pt typecheck
  • corepack pnpm vitest run packages/cli/src/input.test.ts packages/cli/src/commands/json-output.test.ts
  • corepack pnpm --filter @profullstack/sh1pt-core --filter @profullstack/sh1pt-policy --filter @profullstack/sh1pt build
  • smoke-tested a few commands via corepack pnpm exec tsx packages/cli/src/index.ts ... --json

I kept this intentionally small so it is easy to review and does not change the placeholder implementation behind the stubs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add JSON output to remaining CLI list/status commands

1 participant