Skip to content

[2/5] API command and workflow output foundation#98

Open
odyslam wants to merge 12 commits into
codex/pcl-human-output-polishfrom
codex/pcl-output-contract-workflows
Open

[2/5] API command and workflow output foundation#98
odyslam wants to merge 12 commits into
codex/pcl-human-output-polishfrom
codex/pcl-output-contract-workflows

Conversation

@odyslam
Copy link
Copy Markdown
Contributor

@odyslam odyslam commented May 23, 2026

Supersedes the closed oversized PR #90 and consolidates the previous raw-API extraction / output-contract slices (#97 + #98).

Review order

  1. [1/5] Human CLI output and agent contract guards #96: Human CLI output and agent contract guards
  2. [2/5] API command and workflow output foundation #98: API command and workflow output foundation ← this PR
  3. [3/5] Workflow contract and module architecture #100: Workflow contract and module architecture
  4. [4/5] OpenAPI operation routing for workflows #101: OpenAPI operation routing for workflows
  5. [5/5] Generated-client pruning and stale API cleanup #102: Generated-client pruning and stale API surface cleanup

Base / head

Base: codex/pcl-human-output-polish
Head: codex/pcl-output-contract-workflows

What changed

  • Splits raw pcl api internals by concern instead of keeping one growing command path.
  • Refactors CLI output contracts and workflow subcommands on top of the human-output work.
  • Preserves completion behavior, clap-native root help, auth recovery guidance, and dry-run guidance.

Review focus

  • Raw API command parity for list, inspect, call, coverage, and manifest paths.
  • Output contract compatibility for human, TOON, JSON, and JSONL login behavior.
  • Whether the API/output foundation gives later workflow PRs an obvious place to plug in.

Validation

Validated on the final stack tip codex/pcl-slim-generated-client:

  • make ci
  • Local ./target/debug/pcl discovery smoke for LLM guide, doctor, auth ensure, whoami, workflows, schema list, and API manifest.

@odyslam odyslam changed the title Refactor CLI output contracts and workflow commands [2/5] API command and workflow output foundation May 23, 2026
@odyslam odyslam changed the base branch from codex/pcl-api-command-extraction to codex/pcl-human-output-polish May 23, 2026 00:26
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.

1 participant